[debhelper-devel] "Rules-Requires-Root: no" vs ExtUtils::Install + dh_strip_nondeterminism

[Axel Beckert]

Hi Mattia,

Mattia Rizzolo wrote:
> > Did your example contain a PNG which needs a fixup?
> 
> Yes.  I tried the same package you brought as an example,
> systray-mdstat, which has png files that gets modified.

Huh? Maybe there has been a misunderstanding, but it didn't do that
for me:

Neither dh_strip_nondeterminism emitted anything during a fakerooted
build (which it AFAIK should if it changes anything), nor did
"strip-nondeterminism share/*" cause "git status" to show any changes
in the files afterwards.

In contrary to that, running "strip-nondeterminism -t png
t/fixtures/png/tEXt.png.in" in strip-nd's git repo clearly shows such
a change:

→ git status --short
→ strip-nondeterminism -t png t/fixtures/png/tEXt.png.in
→ git status --short
 M t/fixtures/png/tEXt.png.in
→ 

But for systray-mdstat:

→ git status --short
→ strip-nondeterminism -t png share/*
→ git status --short
→ 

> https://anonscm.debian.org/git/reproducible/strip-nondeterminism.git/commit/?id=e32fed6

Thanks!

I've build a strip-nondeterminism snapshot of it. And with that
snapshot, systray-mdstat indeed builds fine with just "R³: no". So
instead of the override_dh_strip_nondeterminism, I'll just add
"Build-Conflicts: dh-strip-nondeterminism (<< 0.040~)" to
systray-mdstat and everything should be fine. Thanks again!

> Pending release, shall I cut one right away?

I'd be happy if it would only take days (and not weeks) to get this
fix into unstable. But that's just my wish. :-)

		Regards, Axel
-- 
 ,''`.  |  Axel Beckert <abe at debian.org>, https://people.debian.org/~abe/
: :' :  |  Debian Developer, ftp.ch.debian.org Admin
`. `'   |  4096R: 2517 B724 C5F6 CA99 5329  6E61 2FF9 CD59 6126 16B5
  `-    |  1024D: F067 EA27 26B9 C3FC 1486  202E C09E 1D89 9593 0EDE