cxzl25 commented on code in PR #1819:
URL: https://github.com/apache/orc/pull/1819#discussion_r1503564090
##########
site/_docs/building.md:
##########
@@ -18,19 +18,18 @@ You'll want to install the usual set of developer tools,
but at least:
* cmake
* g++ or clang++
-* java ( >= 1.8)
+* java ( >= 17)
* make
* maven ( >= 3)
For each version of Linux, please check the corresponding Dockerfile, which
is in the docker subdirectory, for the list of packages required to build ORC:
-* [CentOS 7]({{ page.dockerUrl }}/centos7/Dockerfile)
-* [Debian 10]({{ page.dockerUrl }}/debian10/Dockerfile)
* [Debian 11]({{ page.dockerUrl }}/debian11/Dockerfile)
* [Debian 12]({{ page.dockerUrl }}/debian12/Dockerfile)
* [Ubuntu 20]({{ page.dockerUrl }}/ubuntu20/Dockerfile)
* [Ubuntu 22]({{ page.dockerUrl }}/ubuntu22/Dockerfile)
+* [Ubuntu 24]({{ page.dockerUrl }}/ubuntu24/Dockerfile)
Review Comment:
Thanks. I originally wanted to link this URL
https://github.com/apache/orc/blob/main/docker/os-list.txt directly in the
document, but now I have completed all OS in the document.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
