Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/16354 )
Change subject: [build] Adjust JAVA_HOME candidates ...................................................................... Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/16354/1/build-support/java-home-candidates.txt File build-support/java-home-candidates.txt: http://gerrit.cloudera.org:8080/#/c/16354/1/build-support/java-home-candidates.txt@51 PS1, Line 51: /Library/Java/Home > This came from the updated version of bigtop-detect-javahome which is the o Thank you for the clarification. Just for posterity, could you include this information in the commit message as well? I think it will help in to maintain this file in the future. -- To view, visit http://gerrit.cloudera.org:8080/16354 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ie51cde4153be23e8f33fcc061531c2c022791b16 Gerrit-Change-Number: 16354 Gerrit-PatchSet: 1 Gerrit-Owner: Grant Henke <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Bankim Bhavsar <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Wed, 26 Aug 2020 19:00:23 +0000 Gerrit-HasComments: Yes
