Hi Andy,


the packaging looks good.

I did two builds with DH_VERBOSE=1 in a sid chroot
(the unstable and mentors versions)

I did a meld between them, and I see some "issues"

-"Wall" flag has disappeared

(I guess not a real problem here :) )


- many lines are like this one
gcc -O3 -DEAPI -fPIC -pthread $(pkg-config --cflags apr-1) -pthread 
$(dpkg-buildflags --get CFLAGS)  -c -I../../vm -I/usr/include/mysql mysql.c

evaluating $(dpkg-buildflags --get CFLAGS) at each file build (and link, the 
same is for LDFLAGS) is somewhat an overkill.

I see they come from debian/patches/harden.patch,
just as TODO you might ask upstream to use *FLAGS from the environment, and 
drop that patch.


but we are nitpicking here :)

So if you can readd the -Wall flag I guess we are fine there :)

cheers,

G.

Reply via email to