Hi again It's been a few days, so forgive me if I bug you again.
The known issues have been fixed. Probably Pd people would like to have a final look and confirm that the package in question is ready for upload. I'd be happy if eventually a DD could upload it. Thanks in advance. Roman On Mon, 2011-10-03 at 14:11 +0200, Roman Haefeli wrote: > All the issue below have been fixed or do not belong to the package in > question. > > I'd be grateful if someone could have a look again and eventually upload > it. > > Cheers > Roman > > > > On Fri, 2011-09-30 at 16:19 +0200, Roman Haefeli wrote: > > Hi IOhannes > > > > First of all, thanks a lot for having such a thorough look. > > > > On Fri, 2011-09-30 at 13:24 +0200, IOhannes m zmölnig wrote: > > [...] > > > debian/control: > > > current standards-version is 3.9.2 > > > > fixed > > > > > debian/control: > > > Uploaders field has a stray trailing comma > > > > oops... fixed. > > > > > debian/control: > > > any reason why you are so picky about the debhelper version? > > > as jonas has pointed out before (e.g in the recent pd-zexy thread), > > > debhelper-7 is even in oldstable, so you might happily use "debhelper". > > > > I'm using short-form dh with dh overrides. Lintian tells me that those > > features are only available since 7.0.50. I read the thread about > > pd-zexy, but I _think_ the situation might be different there, because > > it is using cdbs (and thus probably not using dh overrides). > > > > http://lintian.debian.org/tags/debhelper-overrides-need-versioned-build-depends.html > > > > > (i know you care about ubuntu a lot, and i don't know the exact > > > situation there) > > > > I'm currently packaging on Debian/unstable and testing on both. I got > > the exact same lintian warnings/errors on both. > > > > > debian/control: > > > Depends on "pd",but there "pd" is only a virtual package, and you > > > should provide a real one first. > > > this is also caught by lintian: > > > W: pd-pdstring: virtual-package-depends-without-real-package-depends > > > depends: pd > > > something like this should fix the problem: > > > Depends: puredata-core | pd > > > > fixed. > > > > > debian/copyright: > > > is it really true that moocow holds the copyright for files in debian/? > > > no file in debian/ has an explicit copyright notice (why), but i still > > > doubt that moocow did the debian packaging. > > > > fixed > > > > > also config.* and some other files seem to have different copyright > > > holders > > > > Hopefully fixed. If someone could have look, that be nice, as this is > > the most tedious (I find) and probably rather error-prone part. > > > > I was told on #debian-devel, that auto-generated files such as > > Makefile.in (generated by autotools) are not required to be listed in > > debian/copyright. So I left them out. Is that compliant with the > > practice of pkg-multimedia? > > > > > debian/patches/add-meta-file.patch: > > > this patch is actually unneeded. > > > you could simply put the pdstring-meta.pd into debian/ and install it > > > directly from there > > > btw, you could also use debian/install to install the file, rather than > > > adding a dh_override > > > > Converted from patch to a simple debian/install command. > > > > > debian/README.Debian > > > quite a long line :-) > > > more important, i cannot load pdstring following your advice in > > > README.Debian: [declare -stdlib extra/pdstring] will do nothing (on > > > reload), only [declare -lib pdstring] helps > > > > How did you test? The '-lib' flag searches relative to your patch, > > whereas '-stdlib' searches relative to pd. You only can correctly test > > it by effectively installing the package and run pd (/usr/bin/pd). > > > > However, it turned out, that the advice was not complete, since the > > library also contains abstractions, which are not found with only > > '-stdlib extra/pdstring'. The full and correct declaration is: > > > > [declare -stdlib extra/pdstring -stdpath extra/pdstring] > > > > Yeah, that's a lot for loading only a library, but unfortunately that is > > how it currently works in Pd. > > > > > debian/watch > > > it seems that the name is not mangled correctly, i get > > > <snip> > > > Newest version on remote site is 0.10-2, local version is 0.10.2 > > > pd-pdstring: remote site does not even have current version > > > </snip> > > > try something like: > > > opts="uversionmangle=s/-/./" > > > > Thanks for the hint. > > With this I get: > > uscan warning: malformed opts=... in watchfile, skipping line: > > > > I'll look later into that. > > > > Roman > > > > > > > > _______________________________________________ pkg-multimedia-maintainers mailing list [email protected] http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-multimedia-maintainers
