Skip to content

Implement pkg_install(symlink) also for pkg_zip#982

Merged
tonyaiuto merged 1 commit intobazelbuild:mainfrom
adrianimboden:sync_20251022131430
Nov 13, 2025
Merged

Implement pkg_install(symlink) also for pkg_zip#982
tonyaiuto merged 1 commit intobazelbuild:mainfrom
adrianimboden:sync_20251022131430

Conversation

@adrianimboden
Copy link
Copy Markdown
Contributor

I start to use hermetic python environment and saw #929.

It mostly works, but in my case I use pkg_zip instead of pkg_tar.

4eed546 implements raw_symlink/ support for pkg_tar.
This MR adds the same functionality for pkg_zip.

The exception when the feature is not implemented yet:

Exception: ('Unknown type for manifest entry:', ManifestEntry<{'type': 'raw_symlink', 'dest': 'system_impl.runfiles/_main/example/_impl_py_client_install.venv/bin/python3', 'src': 'bazel-out/k8-fastbuild/bin/example/_impl_py_client_install.venv/bin/python3', 'mode': '0555', 'user': None, 'group': None, 'uid': None, 'gid': None, 'origin': '@@//example:system_impl'}>)

Copy link
Copy Markdown
Collaborator

@cgrindel cgrindel left a comment

Choose a reason for hiding this comment

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

LGTM. I will wait to let @aiuto review before merging.

@cgrindel
Copy link
Copy Markdown
Collaborator

@adrianimboden It looks like there may be some linting issues. Can you investigate?

@tonyaiuto tonyaiuto merged commit 0026581 into bazelbuild:main Nov 13, 2025
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.

3 participants