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.
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
Flexget: 2.17.14 -> 2.17.20 & Python: 2.7 -> 3.6 #52529
Conversation
''; | ||
|
||
# ~400 failures | ||
doCheck = false; | ||
|
||
propagatedBuildInputs = [ | ||
python27 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This does not look right.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it needs python2.7 for teh deluge plugin. Taking it out will cause build fail.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Then, that plugin has to be removed. We can't have two python versions in here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why should it be removed? Deluge is a different tool.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't know what deluge is nor do I care. It's simple: Either python36
or python27
has to removed from the expression.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay, can you please once again explain why and where python27
is needed? Maybe we can find a solution.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
because it's needed for deluge and the deluge plugin.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A little detail would be appreciated. E.g., why did you put it in propagatedBuildInputs
. Can you point me to some source code that explains this requirement?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
try to build it without. Then try to build it with it. I don't speak python. I'm just a user of flexget.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In that case, you're probably not the right one to do this. An alternative might be to keep using Python 2.7 and override the cherrypy version.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
propagatedBuildInputs
should be updated to reflect https://github.com/Flexget/Flexget/blob/2.17.20/requirements.txt
And what exactely there? |
For example, I don't find pathlib anywhere. |
Building with
which tells us that somehow, |
Then feel free to fix it with python 2.7 |
According to Flexget/Flexget#2102, Flexget only uses |
I could try to update this myself in the next few days so you can test it, if you have the time to wait. |
I don't use deluge but a lot of people do. |
Superseded by #52681. |
Motivation for this change
Flexget fails on current master because of Cherrypy for python 2.7, so updated flexget to using python 3.6 and updated it to v2.17.20
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)