Hi Niels,

Niels Thykier wrote:
> > Will upload anyway as it clearly improves the situation and we need a
> > fix quickly.
> > 
> > Niels: Feel free to improve my fix in the next upload in case I
> > understood some of the new code wrongly.
> 
> Thanks for finding and working around the bug.

You're welcome.

> It was indeed an oversight.

Yep, looked like that: Was the only occurence far away from the
remaining calls.

Also wondered if that call better should have been a simple
install_file() instead.

> I will probably not change anything for now. Having slept on it and given
> additional evidence, I think the underlying issue was a bug in fakeroot
> (#1023286 is most my most likely candidate) and not a bug in debhelper

I see. Given that comment and the fact that #1023286 only affects 32
bit architectures, I think that #1024261 (in debhelper) is related and
might have the same cause as only 32-bit files seem to be affected.
(Or was #1024261 the whole trigger for that 4-argument conversion? Saw
no comment from you there so far.)

> (meaning the change to 4 parameters should probably be undone).

Ack, sounds as if this is probably the way to go once it's clear that
#1023286 was really the cause. I also wonder if some versioned Breaks
or Conflicts would be helpful, too.

                Regards, Axel
-- 
 ,''`.  |  Axel Beckert <a...@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

Reply via email to