-
Notifications
You must be signed in to change notification settings - Fork 2k
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
media-gfx/gpicview: update GTK+ icon cache and xdg desktop database #6920
Conversation
ghost
commented
Jan 21, 2018
- fix QA warnings
Pull Request assignment Areas affected: ebuilds media-gfx/gpicview: @gentoo/graphics, @gentoo/lxde No 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 ping us to reset the assignment. If you do not receive any reply to this pull request, please open or link a bug to attract the attention of maintainers. In order to force reassignment and/or bug reference scan, please append |
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.
More than you intended, probably, but there's some easy fixes we should have done.
src_install() { | ||
emake DESTDIR="${D}" install | ||
dodoc AUTHORS | ||
} |
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.
src_install is unnecessary. It doesn't deviate from the default. Go ahead and delete it.
LICENSE="GPL-2" | ||
SLOT="0" | ||
KEYWORDS="~alpha ~amd64 ~arm ~arm64 ~ppc ~x86 ~arm-linux ~x86-linux" | ||
IUSE="" |
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.
Delete empty variables.
DEPEND="${RDEPEND} | ||
>=dev-util/intltool-0.40 | ||
virtual/pkgconfig | ||
sys-devel/gettext" |
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.
Alphabetical, please, grouping use flag constraint, version constraint, and bare depends together in the same order for each *DEPEND var.
Package-Manager: Portage-2.3.19, Repoman-2.3.6
Thanks for review, hope I didn't miss something. |
👍 All QA issues have been fixed! |