build(deps): remove duplicate pyyaml from requirements

piksel / pytouch-cube

Qt5 Label Editor for Brother P-Touch Cube label maker

MIT License

48 stars 9 forks source link

build(deps): remove duplicate pyyaml from requirements #37

Closed Mimoja closed 2 years ago

Mimoja commented 2 years ago

While modern pip3 versions are absolutely okay with an double entry if only one refers to a version, the python3.8 shipped pip3 from ubuntu 20.04 does fail in this case very much.

ERROR: Double requirement given: pyyaml (from -r requirements.txt (line 13)) (already in pyyaml~=5.4.0 (from -r requirements.txt (line 8)), name='pyyaml')

Signed-off-by: Mimoja git@mimoja.de

piksel commented 2 years ago

Yeah, that was added in #32, since pyaml wasn't installed for some reason. Had the same problem, but didn't realize it was actually in the requirements already

Mimoja commented 2 years ago

Yes, I found pip install . to not work as it is not using the requirements.txt. pip install -r requirements.txt worked just fine on a bunch of ubuntu versions for me