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
app-text/nfoview-1.27 version bump #13038
Conversation
tillschaefer
commented
Sep 25, 2019
- migrate to EAPI 7
- add gettext dependency
- remove versions not tested against python 3.7
Pull Request assignmentSubmitter: @tillschaefer app-text/nfoview: @tillschaefer, @gentoo/proxy-maint Linked bugsNo bugs to link found. If your pull request references any of the Gentoo bug reports, please add appropriate GLEP 66 tags to the commit message and request reassignment. 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 |
app-text/nfoview/nfoview-1.27.ebuild
Outdated
|
||
PYTHON_COMPAT=( python3_{5,6,7} ) | ||
|
||
inherit distutils-r1 gnome2-utils xdg-utils |
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.
gnome2-utils isnt' needed here anymore, and using it for icon_cache_update on an EAPI-7 ebuild shouldnt even work. Use xdg_icon_cache_update
instead. R
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.
ok, good to know. Is changed now.
app-text/nfoview/nfoview-1.27.ebuild
Outdated
SLOT="0" | ||
IUSE="" | ||
|
||
BDEPEND="${PYTHON_DEP} |
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.
There's a typo here, but you are right in putting it into BDEPEND.
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.
done
migrate to EAPI 7 add gettext dependency Package-Manager: Portage-2.3.69, Repoman-2.3.16 Signed-off-by: Till Schäfer <till2.schaefer@uni-dortmund.de>
Package-Manager: Portage-2.3.69, Repoman-2.3.16 Signed-off-by: Till Schäfer <till2.schaefer@uni-dortmund.de>
7c08a9d
to
337cc1f
Compare
Pull request CI reportReport generated at: 2019-09-26 18:37 UTC No issues found |