python-appimage icon indicating copy to clipboard operation
python-appimage copied to clipboard

Allow < and @ sign in requirements.txt

Open overbost opened this issue 2 years ago • 1 comments

Every line in requirements is injected in shell, if there are "<" or "<"= sign it break because "<" is special symbol in shell.

Before this line the line should be incorporated in apices like:

"'{r}'".format(r=requirement)

# not works
$ pip install package<=0.5
# works
$ pip install 'package<=0.5'

i temporary solved by adding apices in my requirements

EDIT git repository with specific branch doesn't work, and putting apices doesn't solve. I must use main branch

# not works
git+https://github.com/overbost/myrepo@devbranch

overbost avatar Aug 01 '23 16:08 overbost

Hello @overbost ,

thank you for pointing this out. Implementing your proposal seems straightforward. But, unfortunately, it does not seem to solve the second issue, which I do not quite understand.

I currently have little availability for this project. Since there is an easy solution (manually wrapping with ''), I'll leave this as so for now. But, I am open to a PR, if someone manages to solve both issues.

niess avatar Oct 10 '23 07:10 niess

@overbost fyi git requirements are working now

related PR: https://github.com/niess/python-appimage/pull/73

dotzborro avatar May 23 '24 22:05 dotzborro