On Tue, May 03, 2011 at 02:18:16 (CEST), Jonathan Nieder wrote: > Hi, > > Reinhard Tartler wrote: > >> * don't try to install non-existing documentation, fixes FTBFS on powerpc >> * add NEWS.Debian file >> * install doc/APIChanges and refer to them in NEWS.Debian (Closes: >> #623682) >> * readd deprecated avcodec_thread_init in libavformat > > Thanks for this. I tried the package and it seems that NEWS.Debian > isn't actually installed. While working on that, I picked a few other > nits: > > * the APIChanges should only be relevant to people using the > development libraries, so I think it makes more sense to only > install them to lib*-dev and libav-doc > > * using source format 3.0 (quilt), so "git reset --hard" and > other habits don't produce confusing results > > * allowing "debian/rules clean" as a non-root user. In > modern times, most people don't run "debian/rules binary" as true > root anyway, and in the worst case "debian/rules clean" will error > out and the user will know what to do (just a pet peave). > > If you have any complaints, I'd be glad to roll a new version of the > patches addressing them.
Thanks, I've applied (most) of them mostly unmodified. -- Gruesse/greetings, Reinhard Tartler, KeyID 945348A4 -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
