Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 593768 - =dev-util/debhelper-9.20160814 stable request
Summary: =dev-util/debhelper-9.20160814 stable request
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Stabilization (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Debian-related package maintainers [DISBANDED]
URL:
Whiteboard:
Keywords: STABLEREQ
Depends on: 595782
Blocks:
  Show dependency tree
 
Reported: 2016-09-14 14:46 UTC by Jeroen Roovers (RETIRED)
Modified: 2017-07-07 09:07 UTC (History)
0 users

See Also:
Package list:
=dev-util/debhelper-9.20160814 =sys-apps/fakeroot-1.20.2
Runtime testing required: ---
stable-bot: sanity-check+


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Jeroen Roovers (RETIRED) gentoo-dev 2016-09-14 14:46:15 UTC
Arch teams, please test and mark stable:
=dev-util/debhelper-9.20160814
Targeted stable KEYWORDS : alpha amd64 arm hppa ia64 ppc ppc64 sparc x86
Comment 1 Tobias Klausmann (RETIRED) gentoo-dev 2016-09-22 08:45:32 UTC
It seems debhelper with USE=test requires fakeroot to be installed, otherwise the tests fail:

>>> Test phase: dev-util/debhelper-9.20160814
make -j5   test
printf "package Debian::Debhelper::Dh_Version;\n\$version='9.20160814';\n1" > \
        Debian/Debhelper/Dh_Version.pm
./run perl -MTest::Harness -e 'runtests grep { ! /CVS/ && ! /\.svn/ && -f && -x } 
@ARGV' t/* t/*/*
t/dh_compress.t ..................... ok   
t/dh_install.t ...................... ok     
t/dh-lib.t .......................... ok     
t/dh_link.t ......................... ok     
t/maintscript.t ..................... sh: fakeroot: command not found
cat: t/tmp/debian/debhelper.postinst.debhelper: No such file or directory
t/maintscript.t ..................... 1/8 # Failed test 1 in t/maintscript.t at li
ne 19
#  t/maintscript.t line 19 is:  ok(grep { m{^dpkg-maintscript-helper rm_conffile /
etc/1 -- "\$\@"$} } @output);
# Failed test 2 in t/maintscript.t at line 20
#  t/maintscript.t line 20 is:  ok(grep { m{^dpkg-maintscript-helper mv_conffile /
etc/2 /etc/3 1\.0-1 -- "\$\@"$} } @output);
cat: t/tmp/debian/debhelper.preinst.debhelper: No such file or directory
# Failed test 3 in t/maintscript.t at line 19 fail #2
# Failed test 4 in t/maintscript.t at line 20 fail #2
cat: t/tmp/debian/debhelper.prerm.debhelper: No such file or directory
# Failed test 5 in t/maintscript.t at line 19 fail #3
# Failed test 6 in t/maintscript.t at line 20 fail #3
cat: t/tmp/debian/debhelper.postrm.debhelper: No such file or directory
# Failed test 7 in t/maintscript.t at line 19 fail #4
# Failed test 8 in t/maintscript.t at line 20 fail #4
t/maintscript.t ..................... Failed 8/8 subtests 


Hand-installing fakeroot makes the tests pass.
Comment 2 Agostino Sarubbo gentoo-dev 2016-09-26 12:40:16 UTC
amd64 stable
Comment 3 Jeroen Roovers (RETIRED) gentoo-dev 2016-10-01 13:59:16 UTC
Stable for HPPA PPC64.
Comment 4 Jeroen Roovers (RETIRED) gentoo-dev 2016-10-01 14:45:24 UTC
(In reply to Tobias Klausmann from comment #1)
> It seems debhelper with USE=test requires fakeroot to be installed,
> otherwise the tests fail:

What happened to "file a new bug report"?

Fixed in 10.
Comment 5 Tobias Klausmann (RETIRED) gentoo-dev 2016-10-01 17:43:43 UTC
(In reply to Jeroen Roovers from comment #4)
> (In reply to Tobias Klausmann from comment #1)
> > It seems debhelper with USE=test requires fakeroot to be installed,
> > otherwise the tests fail:
> 
> What happened to "file a new bug report"?

What happened to "check the dependencies of the packages you file stablereqs for"?
Comment 6 Jeroen Roovers (RETIRED) gentoo-dev 2016-10-01 19:26:21 UTC
(In reply to Tobias Klausmann from comment #5)

> What happened to "check the dependencies of the packages you file stablereqs
> for"?

You're still commenting on the wrong bug report.
Comment 7 Tobias Klausmann (RETIRED) gentoo-dev 2016-10-02 09:50:48 UTC
So should we stabilize 10/10-r1? or do you insist on me saying pretty please and jumping through hoops? I can also add the dep to 9.20160814 and bump it to -r1.
Comment 8 Jeroen Roovers (RETIRED) gentoo-dev 2016-10-02 10:26:49 UTC
(In reply to Tobias Klausmann from comment #7)
> So should we stabilize 10/10-r1?

Not necessarily.

> I can also add the dep to 9.20160814 and bump it to -r1.

Not necessary. These are development tools, so I expect users to know what to do when the test dependency turns out to be a run-time dependency as well.

Also, you're still talking on the wrong bug.
Comment 9 Tobias Klausmann (RETIRED) gentoo-dev 2016-10-02 10:58:08 UTC
As an AT, I have strong reservations against putting a package in stable whose test suite is known to fail due to a missing ebuild-level dependency.

Since you seem unwilling to fix&bump any v9 ebuild (and v10 hasn't been around for long enough, I guess?), and I am unwilling to stable v9 in its broken state. I guess alpha will skip this bug. Feel free to re-CC us if that changes (or if I missed something here).
Comment 10 Jeroen Roovers (RETIRED) gentoo-dev 2016-10-02 11:07:47 UTC
(In reply to Tobias Klausmann from comment #9)
> As an AT, I have strong reservations against putting a package in stable
> whose test suite is known to fail due to a missing ebuild-level dependency.

As an AT, the package maintainer and the God of Hellfire, I ask you to disregard the missing test dependency (or provide it yourself for testing purposes) and stabilise the version set out in the Summary.
Comment 11 Tobias Klausmann (RETIRED) gentoo-dev 2016-10-02 12:42:14 UTC
Stable on alpha (with USE=test masked).
Comment 12 Markus Meier gentoo-dev 2016-10-26 16:39:24 UTC
arm stable
Comment 13 Agostino Sarubbo gentoo-dev 2016-12-28 08:55:22 UTC
Dear Maintainer (or who is mainly involved in this stable request),

This is an auto-generated message that will move the current component to the new component Stabilization.
To ensure that the stabilization will proceed correctly, please fill the fields "Atoms to stabilize" and "Runtime testing required" as described here:
https://archives.gentoo.org/gentoo-dev/message/4b2ef0e9aa7588224b8ae799c5fe31fa
Comment 14 Stabilization helper bot gentoo-dev 2017-01-04 09:25:24 UTC
An automated check of this bug failed - repoman reported dependency errors (12 lines truncated): 

> dependency.bad dev-util/debhelper/debhelper-9.20160814.ebuild: DEPEND: arm(default/linux/arm/13.0) ['sys-apps/fakeroot']
> dependency.bad dev-util/debhelper/debhelper-9.20160814.ebuild: DEPEND: ia64(default/linux/ia64/13.0) ['sys-apps/fakeroot']
> dependency.bad dev-util/debhelper/debhelper-9.20160814.ebuild: DEPEND: ia64(default/linux/ia64/13.0) ['sys-apps/fakeroot']
Comment 15 Markus Meier gentoo-dev 2017-02-21 20:35:27 UTC
arm stable
Comment 16 Michael Weber (RETIRED) gentoo-dev 2017-02-26 17:07:49 UTC
ppc stable.
Comment 17 Sergei Trofimovich (RETIRED) gentoo-dev 2017-06-04 16:38:41 UTC
ia64 stable
Comment 18 Thomas Deutschmann (RETIRED) gentoo-dev 2017-06-05 18:01:53 UTC
x86 stable
Comment 19 Agostino Sarubbo gentoo-dev 2017-07-07 09:07:35 UTC
sparc stable. Closing.