[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: RFS: libvigraimpex (updated package)



* George Danchev <danchev@spnet.net>, 2009-09-12, 18:44:
Package uploaded.

Thanks.

Some suggestions, you might want to address in the next versions:

* 'copy-all-remove-some' processing is best to be avoided, i.e.:
you want:
       dh install -XLICENSE.txt
instead of:
       dh install
       rm -f debian/*/usr/share/doc/*/html/LICENSE.txt

Right, I'll fix that.

* if you manage to convince upstream that they need a text version of their
changelog, then the pipe horror of lynx.cur|sed|tail|cut would be avoided, and
lynx-cur dropped as a build-dependency as well.

Actually, the HTML changelog is generated from this Doxygen file:
http://www.informatik.uni-hamburg.de/~meine/hg/vigra/raw-file/tip/docsrc/credits_changelog.dxx
It could be easily converted to plain text using only essential tools.

Unfortunately, *.dxx files are not included in the .orig.tar.gz.
That looks like a DFSG#2 violation, so I need to talk to upstream anyway.

--
Jakub Wilk


Reply to: