Skip to content

Conversation

PeytonAllworth
Copy link
Contributor

Adds a short note for zsh users explaining that they must quote extras
(e.g. pip install '.[fast]') when installing to avoid "no matches found"
errors caused by shell globbing.

@PeytonAllworth
Copy link
Contributor Author

This PR only modifies README.md, so the failing test_payment_multipart_trampoline_e2e in test_lnpeer.py seems unrelated. Could we re-run the CI job?

> when installing. To fix this, quote the extras like so:
>
> ```bash
> python3 -m pip install --user '.[fast]'
Copy link
Member

Choose a reason for hiding this comment

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

AFAICT all mentions of "pip install ..." with extras already has it quoted.
For example:

$ ELECTRUM_ECC_DONT_COMPILE=1 python3 -m pip install --user ".[gui,crypto]"

> python3 -m pip install --user -e ".[gui,crypto]"

$ python3 -m pip install --user -e ".[gui,crypto]"

If you find one that does not, you can add quotes at there.

Copy link
Member

Choose a reason for hiding this comment

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

I guess if you think that helps, we could quote even the lone dots. e.g.
$ python3 -m pip install --user -e "."


### Development version (git clone)

$ python3 -m pip install --user -e .
Copy link
Member

Choose a reason for hiding this comment

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

why add this command here? In terms of ordering, this does not make sense for this section.
The command already appears on line 107:

$ python3 -m pip install --user -e .

@SomberNight
Copy link
Member

Thanks but I think this is overkill :)

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.

2 participants