Raphael Hertzog <hert...@debian.org> writes:

> tag 575304 - patch
> thanks
>
> On Sun, 28 Mar 2010, Jari Aalto wrote:
>> Here is a patch for dpkg to check DH_VERBOSE and if on, allow displaying
>> more information during patching. The patch is against:
>> 
>>     git://git.debian.org/git/dpkg/dpkg.git e170f70 2010-03-28 10:36 UTC
>
> I don't like this patch:
> - DH in DH_VERBOSE means "DebHelper" and we're not debhelper
> - it would be far more useful to always run patch in non-quiet mode, grab its 
> output
>   in a variable but only print it when the patching process failed
>   (The Dpkg::IPC API makes it relatively easy to implement)

Those who use debheler in their packaging could take advantae of it.
dpkg-source is not run in isolation when doing packaging work. Turning
on DH_VERBOSE would be a unified way for debhelper users.

The change does not affect interfere other packaging systems.

To "grab" output is not efficient. A full information is always welcomed
in debugging situations; without as little manual work as possible.

Jari



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to