[Letsencrypt-devel] Bug#810720: Bug#810720: libdpkg-perl: check_patches_applied does not apply patches if the first patch emits a warning

[Guillem Jover]

Hi!

On Sat, 2016-01-16 at 17:51:06 -0500, Harlan Lieberman-Berg wrote:
> Apollon Oikonomopoulos <apoikos at debian.org> writes:
> > On 21:22 Tue 12 Jan     , Guillem Jover wrote:
> >> In any case you should fix this in the source package, either by
> >> merging the hunks into a single patch (what's the point of a patch
> >> series correcting the same previously introduced changes anyway? :),
> >> or by splitting the different patches into independent files. Or
> >> rearranging the patch as you mentioned.
> >
> > I agree. I'm Cc'ing the package maintainers, since I'm not directly 
> > involved with the package myself.
> 
> Indeed!  The rather odd patch series came from me collapsing a branch of
> patches into a single file from the upstream source control, rather than
> melding the patch series into a single patch before dumping it into the
> file.  Since quilt was happy with it, I didn't think to check other
> problems that might have caused.
> 
> The patch has been since applied upstream, so the troublesome file was
> deleted in the last upload of letsencrypt.
> 
> I'm fine considering this one as user-error and having it closed,
> Guillem, unless you want it open for some other reason.

I've turned the warnings in that case into an error, so that this
situation can be catcher earlier. (Also cleaned them up to only be
emitted once and changed the output message to mention that it might
be repeated more than once, not just twice.)

Thanks,
Guillem