Michael Smith has posted comments on this change. ( http://gerrit.cloudera.org:8080/21111 )
Change subject: IMPALA-12689: Change TPC-H and TPC-DS builds to respect CFLAGS ...................................................................... Patch Set 2: Code-Review+2 (2 comments) http://gerrit.cloudera.org:8080/#/c/21111/1/source/tpc-ds/build.sh File source/tpc-ds/build.sh: http://gerrit.cloudera.org:8080/#/c/21111/1/source/tpc-ds/build.sh@38 PS1, Line 38: sed -i -r "s/^(LINUX_CFLAGS\s*=.*)/\1 ${CFLAGS}/" makefile > Yes, I think this is going to accumulate if we run it multiple times. It's Ack http://gerrit.cloudera.org:8080/#/c/21111/1/source/tpc-h/build.sh File source/tpc-h/build.sh: http://gerrit.cloudera.org:8080/#/c/21111/1/source/tpc-h/build.sh@50 PS1, Line 50: # The build makefile doesn't respect the CFLAGS environment variable directly, > Synced these up Done -- To view, visit http://gerrit.cloudera.org:8080/21111 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: native-toolchain Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I3f999b71c56a72c14f1beeea99a3689b82a4d45a Gerrit-Change-Number: 21111 Gerrit-PatchSet: 2 Gerrit-Owner: Joe McDonnell <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Comment-Date: Tue, 12 Mar 2024 18:41:32 +0000 Gerrit-HasComments: Yes
