[00:14] <samthewildone> whats up
[03:03] <samthewildone> mwhudson, hey
[03:03] <samthewildone> well then
[13:50] <fredw> Hi, can someone please review https://code.launchpad.net/~fred-wang/ubuntu/wily/firefox/fix-for-1473552/+merge/268213 ?
[15:45] <sidi> i'm building a library with a handmade makefile, and i'm struggling to find the proper rules to build debian library and header packages for it. Essentially i'm installing the files in $(DESTDIR)/usr/lib/ in my Makefile (which I suspect is wrong) and dh_install is not finding them in debian/tmp/usr/lib (since they apparently go to debian/pkg-name1/usr/lib) and fails
[15:45] <sidi> Anyone got an example of a library package without autotools, by any chance?
[15:46] <cjwatson> sidi: can you pastebin your debian/rules?
[15:46] <cjwatson> and the Makefile as well, ideally
[15:53] <sidi> cjwatson, sure, one sec
[15:54] <sidi> Makefile: http://pastebin.com/6MhFyQKG // debian/rules: http://pastebin.com/NRNu3Qqn (pretty much as was generated)
[15:57] <samthewildone> Straight outta bed
[16:00] <cjwatson> sidi: that looks OK (except for the MAKEVARS setting which doesn't seem to do anything and if it did ought to be in a different place).  how about the debian/*install files?
[16:02] <sidi> my preloadlogger1.install is usr/lib/lib*.so.* (which I've taken from other library-like packages)
[16:02] <sidi> the -dev is usr/lib/lib*.so, plus etc/security/*
[16:03] <cjwatson> sidi: should be fine, how about debian/compat?
[16:04] <sidi> cjwatson, 9?
[16:04] <cjwatson> sidi: that should be OK.  at this point I think it'd be helpful if you could put a full example source package somewhere so I can test it
[16:04] <sidi> cjwatson, bzr+ssh://bazaar.launchpad.net/+branch/activityfinder/preload-logger/
[16:05] <sidi> building the whole thing is really fast, it's a tiny library
[16:05] <sidi> ... most of the commits being me attempting silly things to build the package
[16:07] <cjwatson> this isn't the problem, but your Makefile would work better (i.e. install wouldn't have to repeat the build) if you had better dependencies - perhaps as a starting point, use "libPreloadLogger.so.0.9" as a target name rather than "lib", so that make knows it's already built
[16:08] <cjwatson> this is the sort of thing not using a handmade Makefile does better at :-)
[16:09] <cjwatson> oh I see
[16:09] <cjwatson> sidi: you don't have preload-logger-dev defined in debian/control
[16:09] <cjwatson> so debhelper is using its there's-only-one-package logic which is inappropriate here
[16:10] <sidi> ah.
[16:10] <cjwatson> (also, should be libpreload-logger1 and libpreload-logger-dev, really)
[16:12] <sidi> well, the file i build is indeed a library but it's not really intended to be linked against for building other apps
[16:12] <sidi> it's a LD_PRELOAD module loaded with PAM for process monitoring
[16:13] <sidi> though I can change it
[16:13] <cjwatson> I understand that but it doesn't change the library binary package naming rules
[16:14] <sidi> i'll change it
[16:20] <sidi> i think it's working better...
[16:20] <sidi> still had a mistake in the libpreload-logger-dev.install
[16:22] <sidi> cjwatson, once again, thank you!
[16:22] <sidi> I suspect you're gonna end up in my thesis's acknowledgements section :-)
[16:26] <cjwatson> heh
[16:26] <cjwatson> no problem
[17:14] <fredw1> Does anyone know who is in charge of reviewing patches for the Firefox package?
[17:14] <fredw1> chrisccoulson is listed in the latest git revisions but these date back from 2012...
[18:04] <cjwatson> fredw1: It would be chrisccoulson, yes, and if you're only seeing patches from 2012 then that must be because you're looking at an inactive branch.
[18:04] <cjwatson> https://code.launchpad.net/~mozillateam/firefox/firefox.wily
[18:05] <fredw1> cjwatson: thank you. I was looking at https://code.launchpad.net/~ubuntu-branches/ubuntu/wily/firefox/wily