On Fri, 21 Dec 2012 02:50:34 -0800, Vincent Cheng wrote: > Hi David, > > I hope you don't mind, but since I took a bit of time to update the > packaging and try to build the latest release of codeblocks myself > this morning, I went ahead and updated codeblocks' collab-maint git > repo. :)
Uops. I had 12 commits locally which I didn't push :) I merged your work, thanks! > I just wanted to ask a few quick questions about the packaging: > - Why do you set /usr/share/codeblocks/lexers/lexer_bash.sample as > executable in postinst? Since that file is shipped in > codeblocks-common, couldn't you just do it after it's installed into > the build tree? I went ahead and removed the postinst script in favour > of adding an override_dh_fixperms target in debian/rules that does the > same thing; this also fixes lintian warning script-not-executable. ACK. > - Is it possible to simplify override_dh_strip a bit? Instead of > manually moving the files around, I was thinking of something like > this: > > override_dh_strip: > dh_strip -p codeblocks --dbg-package=codeblocks-dbg > dh_strip -p libcodeblocks0 --dbg-package=codeblocks-dbg > dh_strip -p codeblocks-contrib --dbg-package=codeblocks-contrib-dbg > dh_strip -p libwxsmithlib0 --dbg-package=codeblocks-contrib-dbg > > ...although I admit I haven't actually tested that yet and am not sure > if it actually works. I believe that wouldn't work, but you could try :) Also, with debhelper compatibility 9, I usually move all debugging symbols to a common -dbg package (let's say, codeblocks-dbg), because there's no easy way to distinguish which symbols belongs to which package. I admit I don't have much time for this package -- if you want, you can take full maintainership, or just add yourself to Uploaders. I don't mind if you want to take it all for you :) (I have too many packages to take care of, and I'm not a heavy user of C::B). Thanks! David -- . ''`. Debian developer | http://wiki.debian.org/DavidPaleino : :' : Linuxer #334216 --|-- http://www.hanskalabs.net/ `. `'` GPG: 1392B174 ----|---- http://deb.li/dapal `- 2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174
signature.asc
Description: PGP signature