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
virtual/python-funcsigs: require correct USE flags of python/funcsigs #10603
Conversation
e3ab12e
to
8c34ac5
Compare
8c34ac5
to
861092e
Compare
Copyright policy changePlease note that on 2018-09-15 Trustees have approved new Gentoo copyright policy. All contributions made to Gentoo need to follow this policy. If you include the Signed-off-by line in your commit message, you indicate that you have read the policy and agree to its terms. For more detailed explanation, please see the new Gentoo copyright policy explained article. Pull Request assignmentSubmitter: @DarthGandalf virtual/python-funcsigs: @gentoo/python Linked bugsBugs linked: 672890 In order to force reassignment and/or bug reference scan, please append Docs: Code of Conduct ● Copyright policy (expl.) ● Devmanual ● GitHub PRs ● Proxy-maint guide |
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.
Please make this version 2 instead of rev-bumping version 1.
|
||
RDEPEND=" | ||
${PYTHON_DEPS} | ||
$(python_gen_cond_dep '>=dev-python/funcsigs-1[${PYTHON_USEDEP}]' python2_7 python3_{3,4,5,6,7} pypy{,3} )" |
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.
Please do not add python3_6 and 3_7 on this line.
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.
Sure. Can you explain why?
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.
The functionality provided by dev-python/funcsigs is included in the standard library shipped with python 3.6 and 3.7.
Closes: https://bugs.gentoo.org/672890 Signed-off-by: Alexey Sokolov <sokolov@google.com>
861092e
to
4259d88
Compare
Pull request CI reportReport generated at: 2018-12-29 21:57 UTC No issues found |
Closes: https://bugs.gentoo.org/672890 Closes: gentoo#10603 Signed-off-by: Alexey Sokolov <sokolov@google.com> Signed-off-by: Mike Gilbert <floppym@gentoo.org>
Disclaimer: I don't know whether it's the right fix.
Here's what happened: I tried to get rid of python 2, but eventually gave up, and tried to bring python 2 back. I have FEATURES=test, and various builds were failing with python2_7 because they couldn't open module funcsigs in the test phase.
It turns out they depend on virtual/python-funcsigs[python2_7], but virtual/python-funcsigs depends on python/funcsigs without requiring any specific python.
emerge -DNu @world
does attempt to update it:[ebuild R ] dev-python/funcsigs-1.0.2-r1::gentoo USE="test" PYTHON_TARGETS="python2_7* python3_6 (-pypy) -pypy3 -python3_4 -python3_5 (-python3_7)" 0 KiB
, but does that after trying to build other packages which depend on it.Adding
[${PYTHON_USEDEP}]
fixes it for me.