Skip to content

Strip whitespace from paths before prefixing ${pwd}#4054

Merged
slackito merged 1 commit into
bazelbuild:mainfrom
gergondet-woven:fix/strip-path-before-prefixing-pwd
May 28, 2026
Merged

Strip whitespace from paths before prefixing ${pwd}#4054
slackito merged 1 commit into
bazelbuild:mainfrom
gergondet-woven:fix/strip-path-before-prefixing-pwd

Conversation

@gergondet-woven
Copy link
Copy Markdown
Contributor

Fixes #3992

This patch strips the relative paths before prefixing ${pwd}. This also tweaks an existing test to test this situation.

Copy link
Copy Markdown
Collaborator

@slackito slackito left a comment

Choose a reason for hiding this comment

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

Thanks for the fix!

Fixes bazelbuild#3992

This patch strips the relative paths before prefixing `${pwd}`. This
also tweaks an existing test to test this situation.
@scentini scentini force-pushed the fix/strip-path-before-prefixing-pwd branch from 7b7dcdb to bc8abc3 Compare May 28, 2026 12:44
@slackito slackito enabled auto-merge May 28, 2026 12:49
@slackito slackito added this pull request to the merge queue May 28, 2026
Merged via the queue into bazelbuild:main with commit 0eca149 May 28, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Wrong handling of leading spaces in cargo build script arguments

2 participants