HADOOP-19819. Remove obsolete OS platform Dockerfiles#8257
HADOOP-19819. Remove obsolete OS platform Dockerfiles#8257pan3793 wants to merge 4 commits intoapache:trunkfrom
Conversation
|
💔 -1 overall
This message was automatically generated. |
| # Install packages | ||
| #### | ||
| RUN pkg-resolver/install-maven.sh debian:13 | ||
| RUN pkg-resolver/install-cmake.sh debian:13 |
There was a problem hiding this comment.
Debian 13 APT repo contains CMake 3.31.6, which satisfies the version requirement - 3.19 or newer
chengpan@18fd9dedaa29:~/hadoop$ cmake --version
cmake version 3.31.6
| # Install packages | ||
| ###### | ||
| RUN pkg-resolver/install-maven.sh rockylinux:8 | ||
| RUN pkg-resolver/install-cmake.sh rockylinux:8 |
There was a problem hiding this comment.
RockyLinux 8 YUM repo contains CMake 3.26.5, also satisfies the version requirement - 3.19 or newer
[chengpan@045e7222babb hadoop]$ cmake --version
cmake version 3.26.5
CMake suite maintained and supported by Kitware (kitware.com/cmake).
| @@ -1,55 +0,0 @@ | |||
| #!/usr/bin/env bash | |||
There was a problem hiding this comment.
previously used by CentOS 7, useless now.
|
(!) A patch to the testing environment has been detected. |
|
💔 -1 overall
This message was automatically generated. |
|
💔 -1 overall
This message was automatically generated. |
steveloughran
left a comment
There was a problem hiding this comment.
not a docker code but cutting old stuff does make sense. I do think I used that ubuntu_20 for the 3.4.3 release so this isn't for backporting
@steveloughran, absolutely, it is for trunk only. I might be more conservative about backporting than you, generally. |
|
(!) A patch to the testing environment has been detected. |
|
💔 -1 overall
This message was automatically generated. |
|
(!) A patch to the testing environment has been detected. |
|
💔 -1 overall
This message was automatically generated. |
c2c5227 to
8488805
Compare
|
(!) A patch to the testing environment has been detected. |
|
💔 -1 overall
This message was automatically generated. |
|
💔 -1 overall
This message was automatically generated. |
|
(!) A patch to the testing environment has been detected. |
|
💔 -1 overall
This message was automatically generated. |
8488805 to
d5d5eb7
Compare
|
(!) A patch to the testing environment has been detected. |
|
💔 -1 overall
This message was automatically generated. |
|
💔 -1 overall
This message was automatically generated. |
|
(!) A patch to the testing environment has been detected. |
|
💔 -1 overall
This message was automatically generated. |
|
💔 -1 overall
This message was automatically generated. |
| ], | ||
| "debian:13": [ | ||
| "fuse3", | ||
| "libfuse3-dev" |
There was a problem hiding this comment.
it's a bug, fuse and libfuse-dev should be used, otherwise, -Drequire.fuse causes failure.
Description of PR
fuseandlibfuse-devshould be used, instead offuse3andlibfuse3-devHow was this patch tested?
CI
For code changes:
LICENSE,LICENSE-binary,NOTICE-binaryfiles?AI Tooling
No usage.