Skip to content

Conversation

@alesancor1
Copy link
Member

@alesancor1 alesancor1 commented Apr 2, 2025

  • Have you followed the guidelines for contributing?
  • Have you signed the CLA?
  • Have you successfully run make lint && make test?

This PR adds a note explaining why python3-venv is required in stage-packages inside pypi tutorial.


See built docs

This example uses ``python`` plugin along ``python-packages``, so ``pip`` is
invoked implicitly to install such packages. `PEP 668`_ changes the
behaviour of Python package managers like ``pip`` by introducing a special
marker (``EXTERNALLY-MANAGED``) that prevents ``pip`` from installing
Copy link
Contributor

Choose a reason for hiding this comment

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

Would this be more precise?

Suggested change
marker (``EXTERNALLY-MANAGED``) that prevents ``pip`` from installing
environment variable (``EXTERNALLY-MANAGED``) that prevents ``pip`` from installing

Copy link
Collaborator

Choose a reason for hiding this comment

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

It's not an environment variable - it's a file. Personally I think it's fine to keep it as the vague "marker"

This example uses ``python`` plugin along ``python-packages``, so ``pip`` is
invoked implicitly to install such packages. `PEP 668`_ changes the
behaviour of Python package managers like ``pip`` by introducing a special
marker (``EXTERNALLY-MANAGED``) that prevents ``pip`` from installing
Copy link
Collaborator

Choose a reason for hiding this comment

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

It's not an environment variable - it's a file. Personally I think it's fine to keep it as the vague "marker"

@tigarmo tigarmo merged commit 1fe946a into canonical:main Apr 3, 2025
16 of 17 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