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

@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

3 participants