=== tchan1 is now known as tchan [06:52] ok back to memory usage [06:52] so much state to pick up [07:21] hi all, hey poolie [07:22] hi there vila [07:24] hi jam :) [08:01] * jelmer waves [08:03] morning all [08:04] hey mgz [08:10] poolie, vila: standup? [08:11] oh hi [08:11] was having so much fun [08:11] good idea [08:58] mgz, jelmer, poolie: I found the pad for last week standup, will post it (thought I did...) as well as the one for this week [08:58] ok thanks [08:59] so... I guess for the installer work I need access to the ec2 instance and urk, probably something other than just ssh [08:59] probably you do [09:00] do you have an AWS account? [09:00] nope. [09:00] standard signup okay? [09:07] vila: I've got a pending MP for bzr-upload from October, I was wondering if you'd seen it: https://code.launchpad.net/~jelmer/bzr-upload/only-strip-slash/+merge/80111 [09:08] err, yeah I saw it, I thought I merged it 8-/ Let me check [09:12] hmm, obviously I didn't... [09:12] mgz, hm, i might need to send you credentials [09:12] i don't think you actually have to sign up [09:12] jam1, hi? [09:13] poolie: that might be better, otherwise they want card etc from me [09:15] I may have trouble till PSU gets delivered, as I lack windowsy things currently === jam1 is now known as jam [09:17] poolie: hi [09:19] jam, could you help mgz get the windows ami going so he can build an installer etc? [09:24] poolie: sure, mgz, you around? [09:25] urk [09:25] that's enough [09:25] jam if you want you could read https://code.launchpad.net/~mbp/bzr/890085-add-bytes/+merge/83735 [09:25] it doesn't directly improve anything though [09:26] was kinda hoping it would [09:26] perhaps will leave it on the shelf until it's more clearly useful [09:27] jam: I'm around but what will I need this end? A spider did for my windows machine. [09:28] rdesktop will do [09:28] mgz: yeah, its on an ec2 instance [09:31] poolie: did you want to make add-bytes have a prereq of BigString so that the review is simpler? [09:31] or you want them together [09:32] it's actually independent [09:32] i rebased it out [09:32] .. [09:32] perhaps lp needs to be told [09:34] jam, should be better now [09:34] the web page looks good [09:34] the email didn't [09:35] i think it doesn't always email updates [09:35] not sure of the logic [09:35] poolie: so we actually need to change the Pack code to take a [chunks] parameter instead of a 'bytes' one. [09:35] i know [09:35] i'd really better go [09:35] i know this isn't sufficient by itself [09:36] i hoped it would show an incremental improvement but apparently not [09:36] I'll comment on why it isn't sufficient, have a good night. [09:36] i'm a bit surprised there is nothing though [09:36] i do understand other stuff is joining up the strings [09:37] poolie: well, we have to ''.join() the zlib chunks to pass it to the pack parameter [09:37] and then we delete the chunks [09:37] so there is one peak [09:37] then the pack code does it *again* [09:37] and we have the second peak [09:37] we have to get rid of both [09:37] yep [09:37] got that [09:38] so mostly for this review i'm hoping for "not wrong" and "in the right direction" [09:41] jelmer: sorry, in parallel with your review, I realized I missed the points you re-raised, was about to say so in the proposal ;) [09:41] hello [09:42] who is responsible for http://bazaar.launchpad.net? [09:42] today's Xubuntu CDs failed to build because http://bazaar.launchpad.net/~xubuntu-dev/ubuntu-seeds/platform.precise/ ceased to exist [09:42] but https://code.launchpad.net/~xubuntu-dev/ubuntu-seeds/xubuntu.precise exists, and bzr pull works fine, etc. [09:43] pitti: #launchpad in general [09:43] Peng: ok, will try there, thanks [09:44] pitti: I have a feeling a $platform forgot to be expanded. [09:44] $platform vs platform, etc. [09:44] Ooh, good catch [09:44] http://bazaar.launchpad.net/~xubuntu-dev/ubuntu-seeds/xubuntu.precise/ works [09:45] And https://code.launchpad.net/~xubuntu-dev/ubuntu-seeds/platform.precise does not exist. [09:45] yeah, you can go to: https://code.launchpad.net/~xubuntu-dev/ubuntu-seeds to see what branches exist underneath it. [09:45] oh, I see [09:45] pitti: Sorry for the hopping back and forth. [09:45] sorry, missed that [09:46] I'll go back to poking the cdimage folks then :) [09:48] actually, it seems it's meant to be like that, it just had trouble reading http://bazaar.launchpad.net/~ubuntu-core-dev/ubuntu-seeds/platform.precise [09:49] but that works, so maybe it was just a glitch; I'll retry the build [09:49] pitti: ~ubuntu-core-dev/.../platform.precise is not the same as ~xubuntu-dev/.../platform.precise [09:50] "(10:42:12 AM) pitti: today's Xubuntu CDs failed to build because http://bazaar.launchpad.net/~xubuntu-dev/ubuntu-seeds/platform.precise/ ceased to exist" [09:50] jam: right, as I said I just misunderstood the logs [09:50] ah, ok [09:50] the names are pretty similar [09:50] it apparently tries to pull the "platform" seed for both the derivative and ubuntu [09:50] vila: np, I figured you just missed it [09:50] and both failed, I just looked at the wrong one [10:17] jelmer: ping, po_merge options registrations, [10:18] good catch (my god, what was I thinking !), but lazy won't gain us anything here [10:18] it's useful only it it helps *not* loading additional stuff from the plugin but nothing is required here [10:19] importing config is required but it is also required to register the options anyway [10:23] vila: it prevents running the constructor of the option objects, not sure how heavy that is [10:24] and potentially having to import whatever method is used for from_unicode [10:26] Option() constructor should be << import config time, if a specific from_unicode method is needed, yes, lazy should be used to get access to it [10:26] overall, I should use lazy to set a good example :) Will fix [11:16] ahh, a windows machine, it's been so long... [11:16] (well... since thursday) [11:18] ...a java update is available, okay, novelty has worn off [11:28] mgz: probably a TZ update :-} [11:42] jelmer: are bug #894461, bug #368717 and bug #608640 fix released or not ? tool/check-newsbug.py is confused :) [11:42] Launchpad bug 894461 in Bazaar "HPSS call for Repository.pack" [Medium,In progress] https://launchpad.net/bugs/894461 [11:42] Launchpad bug 762330 in Bazaar "duplicate for #368717 lightweight checkout over hpss is slow, many roundtrips" [Medium,Fix released] https://launchpad.net/bugs/762330 [11:42] Launchpad bug 608640 in Bazaar "bzr export from remote branches downloads way more data than it should" [Medium,In progress] https://launchpad.net/bugs/608640 [11:43] the duplicate is weird in itself but what about the others ? [11:43] vila: I've marked #894461 fixreleased [11:43] vila: it's debatable whether the other two are fixreleased yet [11:44] the HPSS branch for Repositor.iter_files_bytes is the first part of it, I consider the HPSS call for Repository.iter_inventories also necessary for it [11:44] * vila nods [11:50] hi [11:50] hi zyga [11:50] is there any way to speed up accessing bzrlib.branch.Branch().nick on a checkout? [11:50] it takes veery long, much longer than revno() which is the other thing I'm intereted in [11:50] zyga: there has been some talk on making branch.Brnach().nick only access the local branch, and synchronising it on "bzr up" [11:50] zyga: in the mean time, you can use: [11:51] bzrlib.branch.Branch._get_nick(local=True) [11:51] oh! [11:51] excellent [11:51] how will that differ from the other method? [11:51] will it raise exceptions or return out-of-date data? [11:51] it only accesses the local branch nick, not the remote one [11:52] so you'll also have to explicitly set the local branch nick [11:53] * zyga checks what it does on his branches [12:08] hi === mrevell_ is now known as mrevell [12:19] hi hrw [12:22] I have question (again). probably one of stupid ones. I have lp:~hrw/ubuntu/precise/gcc-4.6/cross-fixes branch and want to generate diff against lp:ubuntu/precise/gcc-4.6 branch. is there other way then fetch both and then diff directories? [12:23] hrw: in theory you could do the diff without downloading the branches. with the current smart server deployed on Launchpad, that will probably be about as slow as actually pulling down the branches. [12:24] jelmer: first branch I already have on disk [12:24] s/have/had [12:24] I fetched second one and diffed dirs [12:25] have to experiment with bzr-colo later [12:25] as with git I would have both in one .git [12:25] bb in ~20 minutes [12:25] hrw: yes [12:26] hrw: bzr diff --old one-branch --new other-branch [12:26] hrw: you can skip either argument if you are in a branch already [12:26] hrw: you can fetch into a shared repository without trees, that should have the same behaviour as bzr-colo or a single .git directory [12:27] bzr init-repo --no-trees gcc; bzr branch lp:~hrw/ubuntu/precise/gcc-4.6/cross-fixes gcc/cross-fixes; bzr branch lp:ubuntu/precise/gcc-4.6 gcc/4.6 [12:27] jelmer: I think hrw already has a shared repo for gcc [12:27] so he has 99% of the data locally already [12:28] also, with 'bzr diff' you don't need trees [12:29] zyga: in that case, I don't understand why bzr-colo / .git would are better [12:29] jelmer they are not [12:29] they are identical [12:57] jelmer: I have a feeling that when I ask about bzr and branches I will always see 'bzr init-repo --no-trees' ;) [12:57] zyga: lp:linaro-gcc != lp:ubuntu/gcc-4.6 [12:58] hrw: sure but they may share ancestry [12:59] zyga: they have one thing in common: 3 letters in branch name ;D [12:59] jelmer: will try your stuff [13:02] hrw: '3 letters in branch name' seriously ? Not a single revision is shared ? parallel imports ? [13:04] vila: lp:linaro-gcc is copy of gcc source code. lp:ubuntu/gcc-4.6 is debian/ubuntu packaging for gcc-4.6 [13:04] like you would compare linux kernel and qt5 repos [13:05] jelmer: is there actually a dulwich 0.8.1 release? I didn't see a tag for it. [13:05] oh, upstream and packaging-only branches ? sry, missed that detail [13:07] ..there is... I just don't have the tag [13:42] mgz: ah, you're trying to build the windows installers? [13:44] yes and it's all your fault >_< [13:44] ...I have one that now works with `bzr info git://...` [13:58] mgz: what about "bzr clone git:// [13:58] ? [13:58] I'm not sure much network activity bzr info generates [14:02] jelmer: bug #897690 filed for ObjectNotLocked on jubany [14:02] Launchpad bug 897690 in bzr-builddeb "bzrlib.errors.ObjectNotLocked raised when calling _mark_native_config" [High,Confirmed] https://launchpad.net/bugs/897690 [14:02] jelmer: I think I have a fix (testing) [14:02] yup, seems to work, proposing [14:05] also works, but tells me 'clone' is deprecated :) [14:18] vila: ah [14:19] jelmer: https://code.launchpad.net/~vila/bzr-builddeb/897690-object-not-locked/+merge/83790 [14:19] with hopefully enough explanations [14:29] vila: looking [14:32] vila: I think you're missing one of the return paths in _default_config_for_tree [14:33] vila: (still returns 2 elements in the tuple rather than 3) [14:33] crap, you're right [14:34] vila: other than that, it looks good to me [14:34] jelmer: including the debian/changelog ? Not sure if I got this right [14:35] missed return path fixed [14:35] though it is slightly odd that path2id doesn't require a read lock.. [14:35] it does [14:36] it has a @needs_read_lock decoratr [14:36] tor [14:36] vila: right, but's slightly surprising imo [14:36] yeah, was about to guess a rationale but ended up with a poor one :) [14:37] vila: not sure if this needs mention in the changelog btw, the regression isn't in any releases [14:37] but anyway, digging this route won't address the current issue so I punted [14:37] ha, so I just remove that and merge to lp:bzr-builddeb ? [14:37] vila: right - +1ed [14:38] ECANTPARSE :) [14:38] vila: yep [14:38] ha [14:38] vila: approved :) [14:44] done, deploying on jubany (including a newer bzr.dev as you've fixed set_root_id(not None) which needs to be deployed too) [14:44] bah, of course, chromium-browser is running... shudder [14:44] s/running/imported/ [14:49] hmm, tiplog fall in the infamous 'access self.branch.control_transport on a smart branch' trap :-/ [14:50] vila: it seems tiplog has no choice but to fallback to VFS anyway though [14:50] when it's pushing stuff, etc [14:51] yeah, was thinking along the same lines, I never realized it was pushing a file to lp though which is... surprising, [14:52] I thought lp was strict about which files were allowed...is *anything* allowed below .bzr ? [14:52] scary [14:52] vila: yes [14:53] vila: but don't dare creating a file outside of .bzr [14:53] I haven't looked closely, but the trap is really that even if it doesn't *need* the control_transport later, ensure_real has fired anyway [14:53] vila: the scanner will without warning delete your branch [14:53] ha good === vednis is now known as mar === mar is now known as mars [15:02] uploading windows installers now. [15:06] vila: did you run the testsuite before landing? [15:06] vila: https://launchpadlibrarian.net/86194438/buildlog_ubuntu-oneiric-i386.bzr-builddeb_2.7.9%2Bbzr649~oneiric1_FAILEDTOBUILD.txt.gz [15:07] nope, I only tested on my case :-/ [15:08] fixing [15:11] fixed [15:12] jelmer: pushed, really sorry about that :-/ [15:15] vila: no worries, thanks for fixing the other bug [15:15] which was a regression in *my* code :) [15:17] what I don't quite get is why it didn't trigger earlier but I suppose there are more code paths [15:17] than the one used on the importer [15:18] I say that, was wondering why it prompted for password, and after uploading everything now it says there was an auth problem [15:20] mgz: enable js ;-p [15:20] (just kidding) [15:22] I don't really see how curl is meant to be doing auth here, presumably via the single sign on somehow? [15:22] curl ? Is that the recommended way ? Isn't there a script that will use lp API ? [15:23] the script uses curl, I'm guessing there's no lp api for uploading files [15:23] note that I don't know what I'm talking about here... [15:24] lp:bzr-windows-installers ./upload_to_launchpad.py [15:25] ok, I reverse my advice: Are you sure this script is used ? :0p [15:25] jam manages it! [15:25] so it must work somehow. [15:26] mgz: --user USERNAME:password [15:27] okay, that worked, need to manually enter lp email address and pa... each time [15:27] but yes, Launchpad lets you post your entire document before it tells you your password is wrong [15:27] or not, like that [15:27] it is because uploading the file is a POST [15:27] which doesn't validate the info until the post has finished streaming the body. [15:27] or three whole documents, in the last case, because the curl output wasn't flushed :) [15:42] hm, I didn't add '-1' in the filenames [15:43] do I really want 20 minutes more upload time to fix that... [15:44] is it really that much? [15:44] nope, you have an allowance of 3 bad uploads across 2 releases as the new maintainer :) [15:56] ghaaaa Alarm clock [15:56] time to wake up vila! [15:56] hehe, I wasn't sleeping, I was thinking ! [15:57] while debugging a test and the new test timeout triggered losing my context ;) [15:57] I'm probably not the last to fall into this trap ;) [16:01] ehehe [16:01] yeah, need to cancel out on pdb break in I guess === zyga is now known as zyga-ill [16:12] mgz: yup, my thought exactly (but invading pdb.break is hard, TestCase.debug on the other hand...) === shadeslayer_ is now known as shadeslayer === quicksil1er is now known as quicksilver === beuno is now known as beuno-lunch [16:33] not sure what to do about my last git related issue from earlier [16:34] I guess I could put up the patch I used ripping setuptools out of dulwich, on moral grounds [16:35] but looks like losing the `./setup.py test` thing might upset some people [16:36] and I didn't record the exact failure but just hacked around it as it was clearly setuptools sucking as usual [16:37] jelmer: if you have any thoughts^ [16:38] basically dulwich didn't work with buildout till I made it believe setuptools didn't exist as it was installing to a path it objected to for whatever reason [16:38] there's probably a buildout level fix but I didn't try and find it. [16:53] bingo, 13 failures less on jubany (if only new ones could stop appearing ... ;) [16:55] mgz: hmm [16:56] mgz: dulwich' setup should work without setuptools, can't you remove it on your local system? [17:01] the build bot needs it for other things, I can possibly kill it at the correct level next time [17:10] \o/ 'bzr checkout' and 'bzr checkout --lightweight' are now VFS-free === beuno-lunch is now known as beuno [17:39] what's the best practice for merging a feature branch to a mainline while preserving history of each changeset in the feature branch, without rebasing? I rebased a mainline on a project last night, and now I feel a bit icky. [17:40] ...just merge it? [17:42] mgz: right, tried merging, resolving conflicts, then committing, but I want to preserve the diffs per-changeset on the feature branch on my mainline. If I push my merge to launchpad, and I view history, IIRC, I don't see this. [17:42] supton: that history is still there, you can view it with e.g. "bzr qlog" or "bzr log -n0" [17:42] so I uncommitted a merge and had the mixed luck of thinking, I'm the only one using the mainline, why not just rebase it from my dev/feature branch. [17:43] jelmer: cool. [17:43] * supton needs to figure out what plugin qlog is in [17:43] it's part of qbzr. [17:43] now, I need to figure out how to undo my rebased branch pushed to lp, so I can do this right [17:43] mgz: thanks [17:45] but just for the history of the last merged change, try `bzr log -c-1 -n0 BRANCH` [17:45] I wonder if I can rebase back to a common revision, then merge properly? [17:45] I shouldn't have pushed this, but for lack of sleep, I did [17:46] `bzr uncommit lp:PROJECT` works, you just want to be quick enough that no one pulled yet :D [17:46] (otherwise, it's not the end of the world, and you can just leave the merge proper in next time) [17:49] sometimes with very messy branches you're better off flattening them, like `bzr merge ../FEATURE && bzr revert --forget-merges && bzr commit --author contrib@example.com -m"...description of change..."` [17:50] mgz: sounds like a plugin... [18:00] mgz: good idea, thanks. [18:05] I despair of ever getting this guy to set up a http smart server correctly. [18:12] hi [18:12] hi tkamppeter [18:13] I have cleaned a file from UTF-8 characters to make it pure ASCII and then committed it to bzr+ssh://till-guest@bzr.debian.org/bzr/pkg-cups/cups/debian-trunk. [18:13] Then the bzr client crashes with a traceback: [18:14] bzr: ERROR: exceptions.UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in position 349: ordinal not in range(128) [18:14] Traceback (most recent call last): [18:14] File "/usr/lib/python2.7/dist-packages/bzrlib/commands.py", line 946, in exception_to_return_code [18:14] return the_callable(*args, **kwargs) [18:14] File "/usr/lib/python2.7/dist-packages/bzrlib/commands.py", line 1150, in run_bzr [18:14] ret = run(*run_argv) [18:14] File "/usr/lib/python2.7/dist-packages/bzrlib/commands.py", line 699, in run_argv_aliases [18:14] return self.run(**all_cmd_args) [18:14] File "/usr/lib/python2.7/dist-packages/bzrlib/commands.py", line 721, in run [18:14] return self._operation.run_simple(*args, **kwargs) [18:14] File "/usr/lib/python2.7/dist-packages/bzrlib/cleanup.py", line 135, in run_simple [18:14] self.cleanups, self.func, *args, **kwargs) [18:14] File "/usr/lib/python2.7/dist-packages/bzrlib/cleanup.py", line 165, in _do_with_cleanups [18:14] result = func(*args, **kwargs) [18:14] File "/usr/lib/python2.7/dist-packages/bzrlib/builtins.py", line 3316, in run [18:14] lossy=lossy) [18:14] File "/usr/lib/python2.7/dist-packages/bzrlib/decorators.py", line 217, in write_locked [18:14] result = unbound(self, *args, **kwargs) [18:14] File "/usr/lib/python2.7/dist-packages/bzrlib/workingtree_4.py", line 209, in commit [18:14] result = WorkingTree.commit(self, message, revprops, *args, **kwargs) [18:14] File "/usr/lib/python2.7/dist-packages/bzrlib/decorators.py", line 217, in write_locked [18:14] result = unbound(self, *args, **kwargs) [18:14] File "/usr/lib/python2.7/dist-packages/bzrlib/mutabletree.py", line 210, in commit [18:14] *args, **kwargs) [18:14] File "/usr/lib/python2.7/dist-packages/bzrlib/commit.py", line 289, in commit [18:14] lossy=lossy) [18:15] File "/usr/lib/python2.7/dist-packages/bzrlib/cleanup.py", line 131, in run [18:15] self.cleanups, self.func, self, *args, **kwargs) [18:15] File "/usr/lib/python2.7/dist-packages/bzrlib/cleanup.py", line 165, in _do_with_cleanups [18:15] result = func(*args, **kwargs) [18:15] File "/usr/lib/python2.7/dist-packages/bzrlib/commit.py", line 440, in _commit [18:15] self.rev_id = self.builder.commit(self.message [18:15] How can I commit this file? Problem is probably that the diff between the old file and the new file contains non-ASCII characters. [18:16] amusingly despite all that, I don't see the bit of the traceback I need [18:16] !pastebin [18:16] For posting multi-line texts into the channel, please use http://paste.ubuntu.com | To post !screenshots use http://imagebin.org/?page=add | !pastebinit to paste directly from command line | Make sure you give us the URL for your paste - see also the channel topic. [18:16] for future reference tkamppeter [18:18] mgz, http://paste.ubuntu.com/753917/, with calling command. [18:19] Bazaar (bzr) 2.4.2 on Oneiric with Python 2.7.2. [18:21] how did you supply the commit message? are you using bzr-builddeb or did you manually enter it with an editor? [18:22] retrying with `bzr commit -m "A message that's really just ascii"` will probably work, [18:22] mgz, seems that bzr-builddeb is in use and already pre-configured, as "bzr commit" does not open an editor if I commit in that repository. [18:23] okay, what bzr-builddeb version? [18:23] "bzr diff" output is on http://paste.ubuntu.com/753921/. [18:23] I will try the -m metyhod now ... [18:23] nope, don't. [18:23] you probably just need to update bzr-builddeb [18:24] the file is a red herring, it's 'Martin-Éric' causing the problem (which is fixed in the latest bzr-builddeb version) [18:26] bug 853664 [18:26] Launchpad bug 853664 in bzr-builddeb "tries to decode debian/changelog as ascii, and fails when it's not" [Medium,Fix released] https://launchpad.net/bugs/853664 [18:26] mgz, thank you. Used the "-m" to not need to wait for the update. [18:28] hmm, I don't think that's actually released yet [18:29] ...you marked it as such though! [18:31] ah, it's in 2.7.9 [18:36] jelmer: that's where the version markers in launchpad are useful :) [18:37] mgz: unfortunately it's not possible to assign a closed milestone to a bug [18:37] mgz: which is often useful after the release has happened [18:46] there's a (slow, manual) trick to that [18:46] re-activate the milestone, set the bug to it, re-de-activate the milestone [19:08] hi all [19:22] hey poolie [19:23] hi there [19:28] hey guys. [19:30] hi grid [19:31] jelmer, bug 806904 says you committed a fix. is this likely be working on launchpad soon then? if there's something left you do and you can point me in the right direction, i can take a look. [19:31] Launchpad bug 806904 in Launchpad itself "Mercurial import fails from Google code" [High,Triaged] https://launchpad.net/bugs/806904 [19:32] i realize now that i'm getting a different error than the one in this bug. but it is a failing google code import. [19:33] http://launchpadlibrarian.net/85941722/weyrick-crack-lang-trunk.log [19:39] "something left to do", rather. i'm not sure how much of this is bzr vs. launchpad code. i suppose a first step is trying a local import with the latest bzr-hg. [19:40] that's generally useful [19:41] actually that would be highly useful to see if you can reproduce it locally [19:41] then we can update it in lp [19:41] yep ok i'll give it a shot in bit here. [19:57] GRiD: the hg import is still very much beta at this point, so I wouldn't be surprised if there are multiple issues affecting one import [19:58] question: on mac...does bzr know about resource forks? [19:59] jelmer, ok. well i'll see what happens when i try locally. does it work well enough to rightly be offered as on option on launchpad at this point, then? [19:59] nup [20:00] GRiD: bug #889530 [20:00] Launchpad bug 889530 in Launchpad itself "mercurial imports should be marked 'beta'" [High,In progress] https://launchpad.net/bugs/889530 [20:00] AuroraBorealis: no [20:00] jelmer, ah ok :) === zyga-ill is now known as zyga === zyga-ill is now known as zyga