Roger Leigh wrote:
> Note that there's one issue with the code that confuses me.  This might
> be a pre-existing issue with dh that the patch exposes:
> 
> % fakeroot dh binary --no-act
>    debian/rules install
>    debian/rules override_dh_strip
>    dh_makeshlibs
>    dh_shlibdeps
>    dh_installdeb
>    dh_gencontrol
>    dh_md5sums
>    dh_builddeb
>    debian/rules binary-arch
>    debian/rules binary-indep
> % fakeroot dh binary
> [build completes]
> % fakeroot dh binary --no-act 
>    debian/rules binary-arch
>    debian/rules binary-indep
> 
> Notice that the initial 'debian/rules install' performed
> by dh binary isn't repeated on the second invocation.  But AFAICS
> there's no .log file that would make dh skip this, so I'm unsure
> why it's skipped.  The same applies to the override to an extent
> (at least here the override is for a single specific purpose which
> we can assume was completed if the .log exists); install is rather
> more generic and I would expect it to be called even if it
> subsequently does nothing due to being up-to-date).  Not yet found
> which bit of the code causes this to be skipped.

Does this still need to be understood?

> Subject: [PATCH 2/2] dh: Add hook support

I don't know why this was included or worked on again.
I showed that hooks make rules files worse in
http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=604563#47

-- 
see shy jo

Attachment: signature.asc
Description: Digital signature

Reply via email to