Skip to content

Implement support for pex-tool/pex#3314. - #215

Merged
jsirois merged 6 commits into
pex-tool:mainfrom
jsirois:pex/issues/3314
Oct 28, 2026
Merged

jsirois merged 6 commits into
pex-tool:mainfrom
jsirois:pex/issues/3314

Conversation

@jsirois

@jsirois jsirois commented Oct 28, 2026 •

Copy link
Copy Markdown
Member
  • Support binding resource directories using --bind-resource-path with trailing .. components.
  • Support control of zip file --modes with original, <octal> or dir=<octal>,exe=<octal>,file=<octal>.
  • Support interpolation of {pex.env.*} for --inject-env values.

+ Support binding resource directories using --bind-resource-path with
  trailing `..` components.
+ Support control of zip file `--modes` with `original`, `<octal>` or
  `dir=<octal>,exe=<octal>,file=<octal>`.
+ Support interpolation of `{pex.env.*}` for `--inject-env` values.
Comment thread python/tests/test_pex_issue_3314.py Outdated
pathsep=os.pathsep
)
),
"--exe",

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This should set --modes 755 and check the pex zip modes as well.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Done and added a test of pexrc inject --modes 755 which Pex will use via pex --rc.

@jsirois
jsirois merged commit 57313d2 into pex-tool:main Oct 28, 2026
57 of 64 checks passed
@jsirois
jsirois deleted the pex/issues/3314 branch October 28, 2026 19:24
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.

1 participant