-
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
net-print/gutenprint: bump to 5.3.4 #20715
Conversation
Pull request CI reportReport generated at: 2021-05-07 09:09 UTC There are existing issues already. Please look into the report to make sure none of them affect the packages in question: |
@juippis thanks for your suggestions :) fixed |
|
Pull request CI reportReport generated at: 2021-05-07 12:14 UTC There are existing issues already. Please look into the report to make sure none of them affect the packages in question: |
EDIT I see it as well during checking phase, but package build correctly |
Pull request CI reportReport generated at: 2021-05-07 14:09 UTC There are existing issues already. Please look into the report to make sure none of them affect the packages in question: |
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 can't merge since I'm not part of the team. I assume you have some sort of deal with someone from that project? If not, please open a bug and link this PR to that bug, then ping me 2-4 weeks after opening the bug and I can merge.
https://devmanual.gentoo.org/general-concepts/package-maintainers/index.html#maintainer-authority
I had left a message to @mattst88 on gentoo-dev but not sure if he see it |
Pull request CI reportReport generated at: 2021-05-08 09:19 UTC There are existing issues already. Please look into the report to make sure none of them affect the packages in question: |
This release drops 2 patches, these named files/5.3.3-* Closes: https://bugs.gentoo.org/674078 ^^ does not appear on 5.3.4 with -j64 ^^ Closes: https://bugs.gentoo.org/788880 Package-Manager: Portage-3.0.18, Repoman-3.0.3 Signed-off-by: Marco Scardovi <marco@scardovi.com>
Pull request CI reportReport generated at: 2021-05-08 09:29 UTC There are existing issues already. Please look into the report to make sure none of them affect the packages in question: |
Pull Request assignmentSubmitter: @scardracs net-print/gutenprint: @gentoo/printing Linked bugsIn order to force reassignment and/or bug reference scan, please append Docs: Code of Conduct ● Copyright policy (expl.) ● Devmanual ● GitHub PRs ● Proxy-maint guide |
Pull request CI reportReport generated at: 2021-05-08 09:49 UTC There are existing issues already. Please look into the report to make sure none of them affect the packages in question: |
Pull request CI reportReport generated at: 2021-06-08 17:34 UTC There are existing issues already. Please look into the report to make sure none of them affect the packages in question: |
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.
If I emerge this with
USE flags: 'cups -gimp gtk nls ppds readline static-libs'
it ends up in an error:
rmdir: failed to remove '/var/tmp/portage/net-print/gutenprint-5.3.4/image/usr/share': Directory not empty
make[5]: [Makefile:2211: install-data-hook] Error 1 (ignored)
rmdir `dirname /var/tmp/portage/net-print/gutenprint-5.3.4/image/usr/libexec/cups`
rmdir `dirname /var/tmp/portage/net-print/gutenprint-5.3.4/image/etc/cups`
libtool: install: /usr/lib/portage/python3.9/ebuild-helpers/xattr/install -c gutenprint.5.3 /var/tmp/portage/net-print/gutenprint-5.3.4/image/usr/libexec/cups/driver
rmdir: failed to remove '/var/tmp/portage/net-print/gutenprint-5.3.4/image/etc': Directory not empty
make[5]: [Makefile:2213: install-data-hook] Error 1 (ignored)
make[5]: Leaving directory '/var/tmp/portage/net-print/gutenprint-5.3.4/work/gutenprint-5.3.4/src/cups'
/usr/bin/install: cannot create regular file '/var/tmp/portage/net-print/gutenprint-5.3.4/image/usr/libexec/cups/driver': No such file or directory
install-xattr: failed to stat /var/tmp/portage/net-print/gutenprint-5.3.4/image/usr/libexec/cups/driver: No such file or directory
make[4]: *** [Makefile:971: install-cupsexec_driverPROGRAMS] Error 1
make[4]: *** Waiting for unfinished jobs....
libtool: error: '/var/tmp/portage/net-print/gutenprint-5.3.4/image/usr/libexec/cups/filter' is not a directory
make[4]: *** [Makefile:1020: install-cupsexec_filterPROGRAMS] Error 1
make[4]: Leaving directory '/var/tmp/portage/net-print/gutenprint-5.3.4/work/gutenprint-5.3.4/src/cups'
make[3]: *** [Makefile:1930: install-am] Error 2
make[3]: Leaving directory '/var/tmp/portage/net-print/gutenprint-5.3.4/work/gutenprint-5.3.4/src/cups'
make[2]: *** [Makefile:481: install-recursive] Error 1
make[2]: Leaving directory '/var/tmp/portage/net-print/gutenprint-5.3.4/work/gutenprint-5.3.4/src'
make[1]: *** [Makefile:597: install-recursive] Error 1
make[1]: Leaving directory '/var/tmp/portage/net-print/gutenprint-5.3.4/work/gutenprint-5.3.4'
make: *** [Makefile:901: install] Error 2
* ERROR: net-print/gutenprint-5.3.4::gentoo failed (install phase):
* emake failed
*
Can you reproduce?
I can't reproduce it |
Package-Manager: Portage-3.0.19, Repoman-3.0.3 Signed-off-by: Marco Scardovi <marco@scardovi.com>
Pull request CI reportReport generated at: 2021-06-09 08:36 UTC There are existing issues already. Please look into the report to make sure none of them affect the packages in question: |
KEYWORDS="~alpha ~amd64 ~arm ~hppa ~ia64 ~ppc ~ppc64 ~sparc ~x86" | ||
|
||
IUSE="cups gimp gtk nls readline ppds static-libs" | ||
RESTRICT="test" |
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 has this been done?
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.
That's an old thing: I haven't touched it because I don't know why it is 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.
That's why when updating ebuilds such as this, always try to remove this restriction and test the ebuild. If it doesn't work, add a comment above the RESTRICT what exactly fails, and preferably either a link to a gentoo bug, or upstream issue about it. Test phase is pretty much the only reliable way for us to be sure the program works like upstream intends to.
|
||
src_configure() { | ||
local myeconfargs=( | ||
--enable-test |
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.
... yet this is enabled? What's the interaction here?
BDEPEND=" | ||
${RDEPEND} | ||
nls? ( virtual/libintl ) |
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.
Should this be DEPEND
instead? Is virtual/libintl needed in DEPEND too?
BDEPEND="${RDEPEND}"
is almost always wrong though.
Package-Manager: Portage-3.0.19, Repoman-3.0.3 Signed-off-by: Marco Scardovi <marco@scardovi.com>
Pull request CI reportReport generated at: 2021-06-10 08:26 UTC There are existing issues already. Please look into the report to make sure none of them affect the packages in question: |
RDEPEND=" | ||
dev-lang/perl | ||
cups? ( >=net-print/cups-1.1.14 ) | ||
gimp? ( | ||
media-gfx/gimp:0/2 | ||
x11-libs/gtk+:2 | ||
) | ||
gtk? ( x11-libs/gtk+:2 ) | ||
readline? ( sys-libs/readline:0= ) |
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.
These are most likely needed in DEPEND
too, I'll add it after merging.
KEYWORDS="~alpha ~amd64 ~arm ~hppa ~ia64 ~ppc ~ppc64 ~sparc ~x86" | ||
|
||
IUSE="cups gimp gtk nls readline ppds static-libs" | ||
RESTRICT="test" |
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.
That's why when updating ebuilds such as this, always try to remove this restriction and test the ebuild. If it doesn't work, add a comment above the RESTRICT what exactly fails, and preferably either a link to a gentoo bug, or upstream issue about it. Test phase is pretty much the only reliable way for us to be sure the program works like upstream intends to.
This release drops 2 patches, these named files/5.3.3-*
Closes: https://bugs.gentoo.org/674078
^^ does not appear on 5.3.4 with -j64 ^^
Closes: https://bugs.gentoo.org/788880
Package-Manager: Portage-3.0.18, Repoman-3.0.3
Signed-off-by: Marco Scardovi marco@scardovi.com