faad2 2.7 Transition

Fabian Greffrath greffrath at leat.rub.de
Wed Jun 3 08:02:23 UTC 2009


All done in git this morning. IMHO the package is ready for upload now.

Fabian Greffrath schrieb:
> - Why are you hardcoding -DPIC and -fPIC into the CFLAGS in 
> debian/rules? This should really be unnecessary. It is also unnecessary 
> to set the optimization flags on purpose anymore.
> 
> - Why do you run the entire autoreconf sequence in the 
> override_dh_auto_configure rule? BTW, if you insist on doing so, please 
> replace the ./configure call in line 20 with dh_auto_configure.
> 
> - Why do you enable DRM support? In the past it has proven to make the 
> library unusable.
> 
> - libmp4ff-dev and faad2-dbg must depend on libmp4ff0.
> 
> - Please avoid the .{a,la,so} bashism in the -dev install files.
> 
> - Please use a *sane* watch file. The one you added is nearly unreadable.
> 
> - "* Move libfaad-dev to the front to be 'default' package debhelper 
> acts on." WTF is the purpose of changes like this?!
> 
> So far for the first check...
> 
> Fabian
> 
> 


-- 
Dipl.-Phys. Fabian Greffrath

Ruhr-Universität Bochum
Lehrstuhl für Energieanlagen und Energieprozesstechnik (LEAT)
Universitätsstr. 150, IB 3/134
D-44780 Bochum

Telefon: +49 (0)234 / 32-26334
Fax:     +49 (0)234 / 32-14227
E-Mail:  greffrath at leat.ruhr-uni-bochum.de



More information about the pkg-multimedia-maintainers mailing list