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 -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org