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

Clarify how to set $0 properly in run_in_shell_command #19019

Merged
merged 1 commit into from May 16, 2023

Conversation

thejcannon
Copy link
Member

Fixes #19017 (well turns out it was a non-issue) by suggesting to users how to get $0 properly in their binaries.

Copy link
Contributor

@chrisjrn chrisjrn left a comment

Choose a reason for hiding this comment

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

Yup, this seems fine.

Definitely need that new target for executing binaries :)

@thejcannon thejcannon enabled auto-merge (squash) May 16, 2023 20:40
@thejcannon
Copy link
Member Author

If/when we do that, feel free to change this text to instead nudge people to that target.

@thejcannon thejcannon merged commit 1026adc into pantsbuild:main May 16, 2023
17 checks passed
@thejcannon thejcannon deleted the fix19017 branch May 17, 2023 01:12
github-actions bot pushed a commit that referenced this pull request May 17, 2023
Fixes #19017 (well turns out it was a non-issue) by suggesting to users
how to get `$0` properly in their binaries.
github-actions bot pushed a commit that referenced this pull request May 17, 2023
Fixes #19017 (well turns out it was a non-issue) by suggesting to users
how to get `$0` properly in their binaries.
github-actions bot pushed a commit that referenced this pull request May 18, 2023
Fixes #19017 (well turns out it was a non-issue) by suggesting to users
how to get `$0` properly in their binaries.
github-actions bot pushed a commit that referenced this pull request May 18, 2023
Fixes #19017 (well turns out it was a non-issue) by suggesting to users
how to get `$0` properly in their binaries.
github-actions bot pushed a commit that referenced this pull request May 18, 2023
Fixes #19017 (well turns out it was a non-issue) by suggesting to users
how to get `$0` properly in their binaries.
thejcannon added a commit that referenced this pull request May 18, 2023
… of #19019) (#19031)

Fixes #19017 (well turns out it was a non-issue) by suggesting to users
how to get `$0` properly in their binaries.

Co-authored-by: Joshua Cannon <joshdcannon@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

AdhocProcessRequest isn't replacing $0 correctly
3 participants