> > Updated with some @owner/@group changes that were missed previously and
> > highlighted by this regeneration. I regen'd patches as well.
> 
> Thanks! Sorry, I'm still getting the hang of this.

It was wrong before, your update just made it obvious :-)

> > I'm not sure what is going on with the existing jade/man2html patch as
> > it doesn't depend on any of them and the port doesn't build the docbook
> > documentation ..
> 
> I'm not sure what's going on with the jade/man2html/docbook patch, 
> either. Out of curiosity I wiped my build env and rebuilt the package 
> without that patch, and, at least on amd64, I wasn't able to spot a 
> difference. According to cvs, that snippet has been in 
> patches/patch-configure_in since import in 2002, and the commit 
> doesn't say why it's there. I motion to drop it (in the interest of 
> code clarity), and we can re-add it if we later find some 
> still-current reason to keep it.
> 
> Updated patch, dropping the jade/sgml/man2html/docbook bits and 
> switching the url to https per Rafael:

Thanks. I've committed this with a further change to use upstream's
install target (with USE_GMAKE), which installs the version of docs that
was bundled by upstream (the tools that configure searches for are only
needed if they aren't already present).

Reply via email to