Skip to content

Conversation

@jeanas
Copy link
Collaborator

@jeanas jeanas commented May 22, 2022

The previous method could lead to problems if a dependency installed
within venv/ shipped with its own PO files.

The previous method could lead to problems if a dependency installed
within venv/ shipped with its own PO files.
@jeanas jeanas mentioned this pull request May 22, 2022
@jeanas
Copy link
Collaborator Author

jeanas commented May 22, 2022

Cf. #1880 (comment).

@jeanas jeanas requested review from JulienPalard and deronnax May 22, 2022 18:00
@JulienPalard
Copy link
Member

Wow TIL git does its own glob expansion by default.

Wow that's funny then in bash, as git and bash don't interpret *.po the same way, and as bash leaves the raw *.po when no matches are to be found:

  • on python-docs-fr git ls-files *.po only matches 7 files (bash does the glob expansion)
  • on python-docs-fr git ls-files '*.po' matches all files (git does the glob expansion)
  • on git repo git ls-files *.po matches all files (bash fails to glob expand so it left the literal *.po for git to expand it using its semantics...).

But there's no bash involved in your PR so it's 100% right, git will expand properly every time, nicely done!

@JulienPalard JulienPalard merged commit 3fc81dd into python:3.10 May 22, 2022
@python python deleted a comment from PyDocTeur May 22, 2022
@jeanas jeanas deleted the merge-files branch May 22, 2022 23:57
@deronnax
Copy link
Collaborator

hat down 👏

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