Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fetch_pip_metadata fails if the pwd contains a space #795

Open
bddvlpr opened this issue Nov 21, 2023 · 0 comments · May be fixed by #1089
Open

fetch_pip_metadata fails if the pwd contains a space #795

bddvlpr opened this issue Nov 21, 2023 · 0 comments · May be fixed by #1089

Comments

@bddvlpr
Copy link

bddvlpr commented Nov 21, 2023

If the project is nested in a parent directory that contains a space, it will fail as the space will separate the path and thus will be interpreted as two (or more) parameters to fetch_pip_metadata.

Can be reproduced by renaming the python-project (or any parent directory) to python project and running nix run .#default.lock

󰄛 /packages/single-language/python project on  main [✘+] is 📦 v1.0.0 via 🐍 
at 11:49   ❯ nix run .#default.lock
usage: fetch_pip_metadata [-h] --json-args-file JSON_ARGS_FILE --project-root PROJECT_ROOT
fetch_pip_metadata: error: unrecognized arguments: project
Traceback (most recent call last):
  File "/nix/store/wvp0c85z9gnzq5cgi4dqq6cb389n057b-refresh/bin/refresh", line 37, in <module>
    lock_data = run_refresh_scripts(refresh_scripts)
  File "/nix/store/wvp0c85z9gnzq5cgi4dqq6cb389n057b-refresh/bin/refresh", line 33, in run_refresh_scripts
    refresh_scripts[name] = run_refresh_script(value["script"])
  File "/nix/store/wvp0c85z9gnzq5cgi4dqq6cb389n057b-refresh/bin/refresh", line 19, in run_refresh_script
    subprocess.run(
  File "/nix/store/pzf6dnxg8gf04xazzjdwarm7s03cbrgz-python3-3.10.12/lib/python3.10/subprocess.py", line 526, in run
    raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command '['/nix/store/qi3s83wi9z1l0gc3bf7kpji0jrkhzgr7-script.sh']' returned non-zero exit status 2.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(fetchPipMetadata): quote path arguments to fetch_pip_metadata script alex-fu27/dream2nix
1 participant
@bddvlpr