[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#799498: RFS: neko/2.0.0-4



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 to: