KeDeng has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/23033 )

Change subject: [Rocky] Add support for Rocky Linux in build toolchain scripts
......................................................................


Patch Set 6:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/23033/6//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/23033/6//COMMIT_MSG@9
PS6, Line 9: proper C++17 compilation on Rocky Linux
> I'm not sure I understand why to bring in GC C12 via devtoolset when it's w
Hi Alexey,

Thank you very much for your sincere feedback. Kudu has done a great job 
adapting to RHEL and SUSE-related distributions. In China, most companies used 
to rely on CentOS. After CentOS reached its end of official support, many have 
chosen Rocky Linux as a replacement. Although our current customers are still 
primarily using CentOS and RHEL, it's undeniable that Rocky is becoming an 
important option for the future.

Rocky 9.5 comes with GCC 11 by default, but during my research I found that the 
minimum available version of gcc-toolset is gcc-toolset-12. To facilitate 
future standardization and compatibility work, I intended to standardize the 
GCC version in this submission to make it easier for other users to compile 
Kudu moving forward.

Since you feel this change may not be appropriate, let's ignore this submission 
for now. I sincerely apologize for taking up your time.



--
To view, visit http://gerrit.cloudera.org:8080/23033
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ibb232f6ec4c36369e79b03370b487ae64aac6f3d
Gerrit-Change-Number: 23033
Gerrit-PatchSet: 6
Gerrit-Owner: KeDeng <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Ashwani Raina <[email protected]>
Gerrit-Reviewer: KeDeng <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Thu, 26 Jun 2025 02:46:57 +0000
Gerrit-HasComments: Yes

Reply via email to