-
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
x11-themes/silver-xcursors: use readme.gentoo-r1.eclass and EAPI=6 (bug #637686) #6205
Conversation
Pull Request assignment Areas affected: ebuilds x11-themes/silver-xcursors: @gentoo/desktop-misc Bugs linked: 637686 |
@floppym suggested me to use the |
cp -R "${WORKDIR}"/${MY_P:5}/Silver/cursors "${D}"/usr/share/cursors/xorg-x11/Silver/ || die | ||
dodoc "${WORKDIR}"/${MY_P:5}/README | ||
dodir /usr/share/cursors/xorg-x11/Silver/cursors | ||
cp -R "${S}/Silver/cursors" "${D}/usr/share/cursors/xorg-x11/Silver" || die |
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.
remove these two lines and try:
insinto /usr/share/cursors/xorg-x11/Silver/
doins -r Silver/cursors
dodoc "${WORKDIR}"/${MY_P:5}/README | ||
dodir /usr/share/cursors/xorg-x11/Silver/cursors | ||
cp -R "${S}/Silver/cursors" "${D}/usr/share/cursors/xorg-x11/Silver" || die | ||
dodoc "${S}/README" |
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 einstalldocs
(and see whether it doesn't install additional junk)
Sorry for the big delay. I just fixed this ebuild as you suggested. Is it OK now? |
I removed the DOCS array due to @monsieurp's suggestion. |
elog should be used instead of einfo for important messages that should
be read by users; moreover elog is logged by default while einfo isn't
https://dev.gentoo.org/~zmedico/portage/doc/portage.html#package-ebuild-helper-functions-output
Closes: https://bugs.gentoo.org/637686