Opened 3 years ago
Closed 3 years ago
#63481 closed defect (fixed)
youtube-dl: can't install when system has py-cryptodome, due to conflict with py-crypto
Reported by: | catap (Kirill A. Korinsky) | Owned by: | mascguy (Christopher Nielsen) |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | |
Keywords: | Cc: | xeron (Ivan Larionov), reneeotten (Renee Otten) | |
Port: | youtube-dl, py-crypto, py-pycryptodome |
Description (last modified by ryandesign (Ryan Carsten Schmidt))
Python has pycryptodome which is for of crypto.
youtube-dl port depends on py-crypto and it is impossible to install it on the system which has pycryptodome
Change History (10)
comment:1 follow-up: 2 Changed 3 years ago by ryandesign (Ryan Carsten Schmidt)
Cc: | xeron added |
---|---|
Description: | modified (diff) |
Port: | youtube-dl py-crypto py-pycryptodome added |
comment:2 Changed 3 years ago by catap (Kirill A. Korinsky)
Replying to ryandesign:
First question: do we need both in MacPorts? It is inconvenient to have conflicting ports. Could we remove one and switch all ports to depend on the other?
Base on https://pycryptodome.readthedocs.io/en/latest/src/vs_pycrypto.html I'd like to say that pycrypto can be replaced by pycryptodome. Or at least we may try it.
comment:3 Changed 3 years ago by catap (Kirill A. Korinsky)
@ryandesign seems that py-crypto is dead: https://github.com/pycrypto/pycrypto/issues/301
comment:4 Changed 3 years ago by mascguy (Christopher Nielsen)
Cc: | mascguy added |
---|
comment:5 Changed 3 years ago by mascguy (Christopher Nielsen)
Cc: | reneeotten added |
---|
comment:6 Changed 3 years ago by mascguy (Christopher Nielsen)
Summary: | Can't install youtube-dl when system has py-cryptodome → youtube-dl: can't install when system has py-cryptodome, due to conflict with py-crypto |
---|
Just ran a quick test of both youtube-dl
as well as subport yt-dlp
, and both seem to work fine with py-pycryptodome
. No surprise, given that the latter is intended to be a drop-in replacement.
Overall, given that py-crypto
is now at end-of-life, should we migrate everything to py-pycryptodome
instead? (With the caveat that we'll need to run some basic regression tests of each, to ensure everything still works.)
Renee, what do you think?
comment:7 Changed 3 years ago by mascguy (Christopher Nielsen)
Owner: | set to mascguy |
---|---|
Status: | new → assigned |
comment:8 Changed 3 years ago by mascguy (Christopher Nielsen)
Cc: | mascguy removed |
---|
comment:9 Changed 3 years ago by mascguy (Christopher Nielsen)
PR submitted to migrate dependents to py-pycryptodome
, and obsolete py-crypto
:
comment:10 Changed 3 years ago by Christopher Nielsen <mascguy@…>
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Ok, so I see that pycryptodome is a fork of pycrypto.
There are currently several ports that depend on pycrypto:
And a few that depend on pycryptodome:
First question: do we need both in MacPorts? It is inconvenient to have conflicting ports. Could we remove one and switch all ports to depend on the other?
If we need to keep pycrypto, the port should be renamed from py-crypto to py-pycrypto. py-crypto was added in the early days of MacPorts before we had decided that all python ports should be named with the "py-" prefix followed by the upstream project name, even if that project name begins with "py".
If we need to keep pycryptodome, it looks like it can be installed two different ways: one that conflicts with pycrypto (this is apparently what we have now) and one that doesn't. Could we switch to the second installation method to avoid the conflict?
If we need to keep both ports and keep them conflicting, then we should modify all ports that depend on either of them so that either of them could be used (i.e. use a
path:
dependency rather thanport:
) and the ports should be marked as being conflicting.