Bug#771926: unblock: debmake/4.1.7-2

["Adam D. Barratt" <adam@adam-barratt.org.uk>]

On 2014-12-04 15:14, Osamu Aoki wrote:
> Hi
>
> On Wed, Dec 03, 2014 at 08:42:11PM +0000, Adam D. Barratt wrote:
> > Control: tags -1 + moreinfo
> >
> > On Thu, 2014-12-04 at 00:14 +0900, Osamu Aoki wrote:
> > > Please unblock package debmake
> > >
> > > Fix important bug and documentation fixes.
> > >
> > >  * Add sanity check for copyright lines. Closes: #767666
> > >  * Update template for autogen.sh. Closes: #771920
> > >  * Work around os.getlogin. Closes: #769392
> > >  * Update documentstion. Closes: #769392
>                                       ^^^^^^
>                This should have been #765579
>
> > ++<td class="content">Due to <a
> > href="https://bugs.debian.org/606542";>Bug #606542</a>, you need to
> > manually install <strong>EXTRAPACKAGES</strong> into the chroot.  See 
> > <a
> > href="#chroot">[chroo
> > t]</a>.</td>
> >
> > Is that not meant to be a list of packages?
>
> Hmmm... I do not quite understand the question ... but No.
>
> 4 "Note" items and only the second item has a package mentioned.  There
> is no "list of packages".

Okay, let me try and explain differently then.

This sentence:

> | Note
> |  Due to Bug #606542, you need to manually install EXTRAPACKAGES into 
> the
> |  chroot. See [chroot].

sounds odd. One cannot "install EXTRAPACKAGES". I assumed that it was 
meant to say "install foo, bar and baz" instead. From looking at the 
bug, I now assume that it's intended to mean "you need to manually 
install the packages listed in EXTRAPACKAGES in your pbuilder 
configuration". Which makes more sense, but at least wasn't obvious to 
me out of context.

I don't have time to look at the rest of the diff right now, so will 
have to come back to it later.

Regards,

Adam