On Mon, Aug 29, 2016 at 7:41 PM, IOhannes m zmölnig (Debian/GNU) <umlae...@debian.org> wrote: > On 08/29/2016 09:31 AM, IOhannes m zmölnig (Debian/GNU) wrote: >> On 08/28/2016 08:22 PM, Joël Krähemann wrote: >>> Last 2 months I have investigated much in stabilize GSequencer's 0.7.x >>> branch. What about packaging for debian? It would be great having the >>> new features in debian. >> >> just ask the main "uploader" (that is: maintainer within the >> pkg-multimedia-team) of gsequencer... oh that is you. >> so what are you waiting for ;-) ? > > hmm, the current package gives me 43 lintian issues (2 errors, 11 > warnings, 24 informational and 4 experimental), see attached file. > > i have already fixed a few of those in git, but there is still a largish > number to be done before the package is ready for upload. > > obviously the errors must be fixed and there is a strong suggestion to > fix the warnings. > > > apart from that: > - the libags-1.0.pc seems weird: there are duplicate entries, and it > adds /usr/local/ paths to both include and library search paths. i'm > pretty sure that this is wrong. > it also adds include-paths and libraries for quite a number of graphic > libraries (from cairo to png) - i wonder whether they are really needed > to use libags. > The /usr/local paths doesn't happen to me it is probably a dh_auto_configure issue with your package builder:
prefix=/usr exec_prefix=${prefix} libdir=${prefix}/lib/x86_64-linux-gnu includedir=${prefix}/include And yes, libags-1.0.pc should rather be libgsequencer-0.7.pc > - is there a specific reason to have the version encoded in library > names rather than the sonames (e.g. libags-1.0.so.0.0.0 rather than > libags.so.0.1.0)? > I just read this article: https://autotools.io/libtool/version.html > - since the package is now split into multiple binary packages, why are > the libraries shipped in the gsequencer package (rather than in libags > and/or libgsequencer). > > - the gsequencer-dev-doc ships >> usr/share/doc/gsequencer-dev-doc >> usr/share/doc/libgsequencer-0.7.x-doc >> usr/share/doc/libgsequencer-0.7.x > the last two seem to be out of place > I fix this upstream so I'll provide a new package gsequencer-0.7.58.tar.gz > there's probably more, but let's fix those first. > > > gfadsr > IOhannes > > _______________________________________________ > pkg-multimedia-maintainers mailing list > pkg-multimedia-maintainers@lists.alioth.debian.org > http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-multimedia-maintainers _______________________________________________ pkg-multimedia-maintainers mailing list pkg-multimedia-maintainers@lists.alioth.debian.org http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-multimedia-maintainers