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

Windows, python: add inc. flag to fix escaping #7973

Closed

Conversation

laszlocsomor
Copy link
Contributor

@laszlocsomor laszlocsomor commented Apr 8, 2019

Incompatible flag: #7974
See #7958

RELNOTES[NEW]: Windows, Python: the --incompatible_windows_escape_python_args flag (false by default) builds py_binary and py_test targets with correct command line argument escaping.

Change-Id: I789f9370e2cf59fa1a179716ca1c6ad80e1d583e

See bazelbuild#7958

RELNOTES[NEW]: Windows, Python: the --incompatible_windows_escape_python_args flag (false by default) builds py_binary and py_test targets with correct command line argument escaping.

Change-Id: I789f9370e2cf59fa1a179716ca1c6ad80e1d583e
@laszlocsomor laszlocsomor changed the title Windows, python: add inc. flag to fix esacping Windows, python: add inc. flag to fix escaping Apr 8, 2019
@laszlocsomor laszlocsomor requested review from meteorcloudy and removed request for lberki April 8, 2019 16:13
@lberki
Copy link
Contributor

lberki commented Apr 8, 2019

/cc @brandjon

(I'll be travelling this week so I won't be able to do this review)

@laszlocsomor
Copy link
Contributor Author

Thanks Lukács. No worries. I didn't expect your review, GitHub added you automatically.

@brandjon
Copy link
Member

brandjon commented Apr 8, 2019

Looks cool.

Copy link
Member

@meteorcloudy meteorcloudy left a comment

Choose a reason for hiding this comment

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

Nice, LGTM!

@bazel-io bazel-io closed this in 4c6fb96 Apr 9, 2019
@laszlocsomor laszlocsomor deleted the pybin-launcher-flags branch April 9, 2019 09:34
laszlocsomor added a commit to laszlocsomor/bazel that referenced this pull request May 10, 2019
Follow-up to PR bazelbuild#7973

Copy the --incompatible_windows_escape_python_args
value also to the host config options.

Incompatible flag: bazelbuild#7974
See bazelbuild#7958
bazel-io pushed a commit that referenced this pull request May 10, 2019
Follow-up to PR #7973

Copy the --incompatible_windows_escape_python_args
value also to the host config options.

Incompatible flag: #7974
See #7958

Closes #8289.

PiperOrigin-RevId: 247596101
aehlig pushed a commit that referenced this pull request May 13, 2019
Follow-up to PR #7973

Copy the --incompatible_windows_escape_python_args
value also to the host config options.

Incompatible flag: #7974
See #7958

Closes #8289.

PiperOrigin-RevId: 247596101
aehlig pushed a commit that referenced this pull request May 17, 2019
Follow-up to PR #7973

Copy the --incompatible_windows_escape_python_args
value also to the host config options.

Incompatible flag: #7974
See #7958

Closes #8289.

PiperOrigin-RevId: 247596101
aehlig pushed a commit that referenced this pull request May 17, 2019
Follow-up to PR #7973

Copy the --incompatible_windows_escape_python_args
value also to the host config options.

Incompatible flag: #7974
See #7958

Closes #8289.

PiperOrigin-RevId: 247596101
aehlig pushed a commit that referenced this pull request May 20, 2019
Follow-up to PR #7973

Copy the --incompatible_windows_escape_python_args
value also to the host config options.

Incompatible flag: #7974
See #7958

Closes #8289.

PiperOrigin-RevId: 247596101
aehlig pushed a commit that referenced this pull request May 21, 2019
Follow-up to PR #7973

Copy the --incompatible_windows_escape_python_args
value also to the host config options.

Incompatible flag: #7974
See #7958

Closes #8289.

PiperOrigin-RevId: 247596101
aehlig pushed a commit that referenced this pull request May 22, 2019
Follow-up to PR #7973

Copy the --incompatible_windows_escape_python_args
value also to the host config options.

Incompatible flag: #7974
See #7958

Closes #8289.

PiperOrigin-RevId: 247596101
aehlig pushed a commit that referenced this pull request May 23, 2019
Follow-up to PR #7973

Copy the --incompatible_windows_escape_python_args
value also to the host config options.

Incompatible flag: #7974
See #7958

Closes #8289.

PiperOrigin-RevId: 247596101
aehlig pushed a commit that referenced this pull request May 23, 2019
Follow-up to PR #7973

Copy the --incompatible_windows_escape_python_args
value also to the host config options.

Incompatible flag: #7974
See #7958

Closes #8289.

PiperOrigin-RevId: 247596101
aehlig pushed a commit that referenced this pull request May 24, 2019
Follow-up to PR #7973

Copy the --incompatible_windows_escape_python_args
value also to the host config options.

Incompatible flag: #7974
See #7958

Closes #8289.

PiperOrigin-RevId: 247596101
irengrig pushed a commit to irengrig/bazel that referenced this pull request Jun 18, 2019
Follow-up to PR bazelbuild#7973

Copy the --incompatible_windows_escape_python_args
value also to the host config options.

Incompatible flag: bazelbuild#7974
See bazelbuild#7958

Closes bazelbuild#8289.

PiperOrigin-RevId: 247596101
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants