Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix https://github.com/bazelbuild/bazel/issues/18493. #18509

Closed
wants to merge 1 commit into from
Closed

Conversation

Pavank1992
Copy link
Contributor

The use of the pipe in
b8e92cc made it swallow the exit code, so we need to set pipefail.

Fix #18493

Closes #18498.

PiperOrigin-RevId: 535520323
Change-Id: Idf1a5c39bf5b7deec29b76c10ece2825b568ebf2

The use of the pipe in
b8e92cc made it swallow the exit code, so we need to set pipefail.

Fix #18493

Closes #18498.

PiperOrigin-RevId: 535520323
Change-Id: Idf1a5c39bf5b7deec29b76c10ece2825b568ebf2
@Pavank1992 Pavank1992 requested a review from keertk as a code owner May 26, 2023 11:01
@Pavank1992 Pavank1992 requested a review from iancha1992 May 26, 2023 11:01
@Pavank1992 Pavank1992 added the team-OSS Issues for the Bazel OSS team: installation, release processBazel packaging, website label May 26, 2023
@Pavank1992 Pavank1992 requested a review from meteorcloudy May 26, 2023 11:05
@Pavank1992 Pavank1992 added the awaiting-review PR is awaiting review from an assigned reviewer label May 26, 2023
@@ -125,4 +125,90 @@ eof
echo "$output" | grep --fixed-strings 'ExecuteProgram(C:\first_part second_part)' || fail "Expected error message to contain unquoted path"
}

function test_run_with_runfiles_env() {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This test case is not in the original PR: https://github.com/bazelbuild/bazel/pull/18498/files

Did you have to deal with some merge conflict? Let's remove this one since it's unrelated.

@keertk
Copy link
Member

keertk commented May 26, 2023

Created this one instead: #18512

@keertk keertk closed this May 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting-review PR is awaiting review from an assigned reviewer team-OSS Issues for the Bazel OSS team: installation, release processBazel packaging, website
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants