Xiang Yang has posted comments on this change. ( http://gerrit.cloudera.org:8080/20263 )
Change subject: IMPALA-12362: (part-4/4) Refactor linux packaging related cmake files. ...................................................................... Patch Set 9: (6 comments) Thanks quanlong's carefully review! http://gerrit.cloudera.org:8080/#/c/20263/8/package/CMakeLists.txt File package/CMakeLists.txt: http://gerrit.cloudera.org:8080/#/c/20263/8/package/CMakeLists.txt@25 PS8, Line 25: install(FILES ${CMAKE_SOURCE_DIR}/LICENSE.txt DESTINATION ${IMPALA_INSTALLDIR} RENAME LICENSE) > Thanks for adding the LICENSE file. I just filed IMPALA-12989 for this. Let Done http://gerrit.cloudera.org:8080/#/c/20263/8/package/CMakeLists.txt@36 PS8, Line 36: if("${gcc_lib}" STREQUAL "") > Let's show the file name and the search dir in the message, e.g. Done http://gerrit.cloudera.org:8080/#/c/20263/8/package/CMakeLists.txt@42 PS8, Line 42: if("${cpp_lib}" STREQUAL "") > Same as above, e.g. Done http://gerrit.cloudera.org:8080/#/c/20263/8/package/CMakeLists.txt@50 PS8, Line 50: if("${kudu_lib}" STREQUAL "") > Please update this as well. Done http://gerrit.cloudera.org:8080/#/c/20263/8/package/CMakeLists.txt@56 PS8, Line 56: if("${hadoop_lib}" STREQUAL "") > Please update this as well. Done http://gerrit.cloudera.org:8080/#/c/20263/8/package/CMakeLists.txt@97 PS8, Line 97: ild) > nit: missing "hive" Done -- To view, visit http://gerrit.cloudera.org:8080/20263 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: If3914dcda69f81a735cdf70d76c59fa09454777b Gerrit-Change-Number: 20263 Gerrit-PatchSet: 9 Gerrit-Owner: Xiang Yang <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Laszlo Gaal <[email protected]> Gerrit-Reviewer: Quanlong Huang <[email protected]> Gerrit-Reviewer: Xiang Yang <[email protected]> Gerrit-Comment-Date: Wed, 10 Apr 2024 14:47:03 +0000 Gerrit-HasComments: Yes
