[01:16] maxb: thats probably because the maintainer script to clean them up is not present in the new package === jelmer_ is now known as Guest40138 === Guest40138 is now known as jelmer____ === mr-russ1 is now known as mr-russ [15:56] maxb: hmm, I'm not sure I follow [15:57] maxb: some sort of shim? [17:27] hmm, odd [17:27] Bad Gateway error during recipe build :_/ [17:52] hi vila! [17:52] any bzr devs around? [17:52] beuno: hey. No, I'm not really there, just relaunched xchat [17:53] :) [18:07] hey beuno [18:08] hey jelmer! happen to be bored? :) [18:08] I'm working on a branch to add downloading tarballs to loggerhead [18:09] lifeless pointed out that saving them to a dir to serve them isn't super optimal, but it seems I don't really have an (easy) choice with the way things are set up in bzr [18:09] wanted some smarter eyes to confirm [18:10] *download tarballs of revisions _from_ loggerhead [18:10] bzrlib.export.export only takes a path [18:11] yeah, it seems like we should be able to improve on that [18:11] though we should watch out for keeping lp:linux tarballs in memory, too [18:12] right [18:13] I started digging in bzrlib, and it's not trivial to change this [18:13] it looks like there already is some support for writing to stdout [18:13] oh? [18:14] '-' is treated specially [18:14] in bzrlib.export.tar_exporter [18:14] ah [18:15] interesting, I missed that [18:16] this might also be a good moment to fix some other related issues [18:16] bug 513752 and bug 711226 [18:16] Launchpad bug 513752 in Bazaar "cannot export a zip archive to stdout" [Low,Confirmed] https://launchpad.net/bugs/513752 [18:16] Launchpad bug 711226 in Bazaar "deterministic output for tar.gz exports with --per-file-timestamps" [Medium,Confirmed] https://launchpad.net/bugs/711226 [18:17] (also cases where it would help if the exporters could write to an arbitrary stream rather than a specific file) [18:17] hm, if I specify -, it creates a dir called - :/ [18:17] * beuno nods [18:17] you need to specify the format I think [18:19] right, that seems to work [18:19] well, "work", now I need to capture stdout and stream that [18:20] I suspect this may be harder to actually deploy on launchpad, though [18:20] beuno: urgh [18:23] beuno: we should just factor the relevant code out of bzrlib.export.tar_exporter [18:24] * beuno nods [18:24] doesn't seem too hard, it's just quickly getting out of hand :) [18:24] beuno: a temp *file* would be ok [18:25] beuno: its a persistent temp dir I was whining about [18:25] lifeless, ah, hi lifeless :) [18:25] lifeless, well, that makes things much simpler [18:26] a temp file will clean up when its closed [18:28] lifeless, ok, so I'll fiddle with this stdout approach for a bit so at least I have it handy, and then change it to use a tmp path [18:29] beuno: tmp *file* specifically - tempfile.TemporaryFile [18:30] lifeless, ack [18:33] jelmer: So, what I was thinking was either a straight backport of dh_python2, or a fake dh_python2 executable or debhelper sequence file that would run pycentral instead [18:34] Either way, a solution which does not require modifications to each package's packaging [18:49] maxb: sounds reasonable [18:51] lifeless, so, since bzrlib.export only really takes a path, it seems I'd need to poke at bzrlib in order to use a TemporaryFile [18:52] or capture stdout and write to this new tempfile [18:52] or... maybe this temp file has a path, which I guess it would [18:53] of course it does [18:56] beuno: you may need to change bzrlib.export [18:56] beuno: TemporaryFile doesn't have a name [18:56] NamedTemporaryFile is not crash proof [18:56] lifeless, no, but NamedTemporartFile... ah [18:56] NTF depends on __del__ [18:57] if we kill the process - not unheard of :P - it will leak and slowly mess up /tmp [18:57] beuno: fwiw I've got a branch which should let you do what you need [18:57] hmeland_, w00t! that's great [18:57] er [18:57] jelmer, I meant :) [18:58] although, it kinda sucks for this feature to depend on a future bzr [18:58] it factors out the bit of bzrlib.export.tar_exporter that writes to the tarball, so it requires you to pass in a tarfile.tarfile object and a tree [18:59] I guess I could hack in using stdout for older bzr versions, and the cleaner one for newer ones [19:06] jelmer: Hi. Shall I file bugs to have the ftpmasters update the overrides for qbzr and wikkid, both of which got override disparity emails on their most recent uploads, or is that on your todo list? [19:23] maxb: Oh, I don't think I have seen those. [19:53] jelmer: they went to you@debian.org and pkg-bazaar-maint@lists.alioth [19:55] maxb: wikkid probably should be web [19:55] I think in both cases the package has it right [19:56] maxb: yeah [19:57] i.e. wikkid s/vcs/web/ because its defining characteristic is being a wiki [19:57] maxb: if you can file bugs that'd be great [19:57] will do === jasono_ is now known as jasono [21:19] beuno: https://code.launchpad.net/~jelmer/bzr/export-tgz-711226/+merge/53183 [21:36] hmm, like Launchpad Bazaar has an interesting definition of "easy" when it comes to bugs [21:54] jelmer, \o/ thanks, I'll take a peak in a little while [21:58] beuno: 'peek' [23:15] Morning folks [23:28] mm-mysql: hey mark! [23:28] spiv: mørning! [23:30] jelmer: hï :P [23:54] jelmer: btw, I've just committed "Take 02_external_configobj out of debian/patches/series because it causes tests to fail." to the daily-ppa branch. Any thoughts on what we do about that for Debian? [23:54] maxb: submitting the fix to the python-configobj package [23:56] it's on my todo list but I haven't had time to do it yet