[Impala-ASF-CR] IMPALA-4096: Allow clean.sh to work from snapshots
Jim Apple has submitted this change and it was merged. Change subject: IMPALA-4096: Allow clean.sh to work from snapshots .. IMPALA-4096: Allow clean.sh to work from snapshots buildall.sh calls bin/clean.sh, which fails when not in a git directory. This skips the "git clean" steps when not in a git checkout. Todd Lipcon found this when testing a snapshot created using git archive. Change-Id: I12dd9035298151557491009680d66d25c8f58c1d Reviewed-on: http://gerrit.cloudera.org:8080/4336 Tested-by: Internal Jenkins Reviewed-by: Jim Apple --- M bin/clean.sh 1 file changed, 5 insertions(+), 3 deletions(-) Approvals: Jim Apple: Looks good to me, approved Internal Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/4336 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I12dd9035298151557491009680d66d25c8f58c1d Gerrit-PatchSet: 5 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Jim Apple Gerrit-Reviewer: Dan Hecht Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Jim Apple Gerrit-Reviewer: Lars Volker Gerrit-Reviewer: Michael Brown
[Impala-ASF-CR] IMPALA-4096: Allow clean.sh to work from snapshots
Jim Apple has posted comments on this change. Change subject: IMPALA-4096: Allow clean.sh to work from snapshots .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4336 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I12dd9035298151557491009680d66d25c8f58c1d Gerrit-PatchSet: 4 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Jim Apple Gerrit-Reviewer: Dan Hecht Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Jim Apple Gerrit-Reviewer: Lars Volker Gerrit-Reviewer: Michael Brown Gerrit-HasComments: No
[Impala-ASF-CR] IMPALA-4096: Allow clean.sh to work from snapshots
Internal Jenkins has posted comments on this change. Change subject: IMPALA-4096: Allow clean.sh to work from snapshots .. Patch Set 4: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/4336 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I12dd9035298151557491009680d66d25c8f58c1d Gerrit-PatchSet: 4 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Jim Apple Gerrit-Reviewer: Dan Hecht Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Jim Apple Gerrit-Reviewer: Lars Volker Gerrit-Reviewer: Michael Brown Gerrit-HasComments: No