[debhelper-devel] Bug#780133: Bug#780133: DH_VERBOSE under dh command

[Niels Thykier]

On 2015-03-10 00:23, Tianon Gravi wrote:
> [...]
> 
> Ok, here's my initial stab at improving this section to make the point
> more clear:
> 
> | diff --git a/debhelper.pod b/debhelper.pod
> | index 9415d78..f5d27f6 100644
> | --- a/debhelper.pod
> | +++ b/debhelper.pod
> | @@ -660,6 +660,12 @@ F<preinst>, F<postrm>, F<prerm>, and F<config> scripts, etc.
> |  
> |  =head1 ENVIRONMENT
> |  
> | +The following environment variables can influence the behavior of debhelper.
> | +It is important to note that these must be actual environment variables in
> | +order to function properly (not simply F<Makefile> variables). To specify
> | +them properly in F<debian/rules>, be sure to prefix their assignment with
> | +"B<export>". For example, "B<export DH_VERBOSE := 1>".
> | +
> |  =over 4
> |  
> |  =item B<DH_VERBOSE>
> 
> Would appreciate some prose review before I commit, though. :)
> 
> ♥,
> - Tianon
>   4096R / B42F 6819 007F 00F8 8E36  4FD4 036A 9C25 BF35 7DD4
> 
> [...]

Looks good to me.  The only thing I can think of is that it probably
also valid to do:

DH_VERBOSE := 1
export DH_VERBOSE

So maybe it should simply say that people should remember to "export"
them and not necessary require it to be exported during the assignment.

~Niels