[00:00] bug reported, thanks. https://bugs.launchpad.net/bzr/+bug/637644 [00:00] Launchpad bug 637644 in Bazaar "Repository corruption, index file truncated by OS file system (affected: 1, heat: 6)" [Undecided,New] [00:01] hi folks, i branch a repo from lp in my VPS now i want to get it to my laptop i get this error: bzr: ERROR: Unable to connect to 173.230.141.58; [Errno 111] Connection refused [00:01] trying: bzr clone ftp://XXX.XXX.XXX.XXX/home/myuser/openerp/stable/addons [00:37] ah people... [00:38] like free online support has to be instant [00:43] hello ddaa [00:43] hello poolie, wassup? [00:43] mm i was going to answer but he just left [00:44] ah, things are pretty good here [00:44] so, my bet was "missing username from URL" :-) [00:44] just going to close all my other windows in a bit and look at this Inter2And2Helper bug [00:44] i was going to guess firewalls around the vps [00:44] oh right, missing username would be a different error [00:49] Good morning. [00:53] gotta say, ropemacs is da bomb! [00:54] when inlining a method that calls a function from another module, it adds the needed imports in the modules of the callsites [01:04] ah, i never managed to get it to work [01:05] Mh... I did need some non-trivial .emacs hacking here. [01:06] Here's what I did: http://pastebin.com/eiEVBJDJ [01:07] the python-ropemacs-hook is personal preference [01:09] I'm not sure why did this thing for on-startup loading, maybe on-demand loading was broken for me... [01:17] doesn't seem too bad [01:17] will have to try again at some point [01:19] poolie: huh, that Inter1And2Helper bug looks shallow, but also a bit depressing: [01:19] # XXX: not covered by tests, should have a flag to always run [01:19] # this. -- mbp 20100129 [01:20] s/source_repo/source/ looks like the obvious fix. [01:21] Also, in a probably unrelated issue, _get_rich_root_heads_graph looks suspiciously useless. [01:52] hm, thanks spiv === Ursinha-brb is now known as Ursinha-afk [03:41] so how should a --no-confirmations flag get down to the ui [03:41] poolie: via the context? ;) [03:42] hm, maybe :) [03:42] I've put up a patch for https://bugs.edge.launchpad.net/bzr/+bug/636930 btw, including test coverage. [03:42] Launchpad bug 636930 in Bazaar "Upgrading a repository fails with 'Inter1and2Helper' object has no attribute 'source_repo' (affected: 4, heat: 34)" [Critical,In progress] [03:42] should this be remembered in the uifactory, or in the context, or should it perhaps be somewhere else [03:43] extreme factoring would suggest there's a separate "user interaction policy" thing that is separate from the implementation of how to interact with the user [03:43] (legend) [03:45] and then that policy object would know whether to ask or not [03:46] Yes, I think that sounds about right. [03:46] I'd want to keep the policy object as simple as possible until more things use it. [03:47] But otherwise I think that sounds like a nice approach. [03:53] Hmm, maybe I should to fix http://bugs.python.org/issue9729 so that the test suite can pass on maverick. [04:06] could be good [04:06] there's another bug very slightly similar to that [04:06] just in bzr [04:06] which is something to do with socket.error being a bit odd [04:06] so not caught properly [04:06] perhaps because it's not a subclass of IOError or OSError [05:58] spiv, teddy bear? [05:58] i'm trying to decide if the user interaction policy layer should be a decorator layer on the uifactory [05:59] in some ways that's an easy way to intervene around all actions [05:59] * spiv listens [06:00] * spiv sits with arms sticking outwards a little bit, in his best imitation of a teddy bear ;) [06:00] :) [06:00] but it feels a bit wrong [06:00] Yeah, my initial reaction to that is it feels a bit weird. [06:00] if we do it, we could add a GenericDecorator that uses getattr to feed through to the underlying one [06:00] for instance you might like to look at the current state, or reset the state [06:01] So the decorator could intercept e.g. confirm_action and decide to ignore the decorated object and just return True if that was the policy? [06:03] So this is to save the individual uifactories from having to all implement the same "first, check the policy" logic? [06:04] right [06:04] how do you jump back a revision on a checkout? [06:04] and to make it easier to vary the policy [06:04] i can't believe i dont know how to do this [06:04] rryan 'bzr update -r -2' [06:04] for instance to have --confirmation=yes or --force options [06:04] https://bugs.edge.launchpad.net/bzr/+bug/397315 [06:04] Launchpad bug 397315 in Bazaar "break-lock should have a --yes, --force or --unconditional option (affected: 1, heat: 6)" [Medium,In progress] [06:05] facepalm, thanks! [06:05] So here's another use case that may or may not help to think about: [06:06] add the ability to selectively set a policy for just a specific question, e.g. "break_lock=yes/no/confirm" [06:07] I suppose the decorator would have to inspect the args passed to confirm_action (or whatever) to see if it matches the specific question. [06:09] (Hmm, I suppose a different variation is per-location or per-branch UI policy... I guess that's equally messy with the decorator or with every factory explicitly delegating to policy) [06:23] right, so we're going to want it somehow hooked up to the context, from which it can get the config object [06:23] i guess assigning a new object to ui.ui_factory feels like a very heavyweight action [06:23] to set the policy here [06:25] Yeah. [06:26] The decorator approach does sound like less code, which is a big plus. [06:30] What's the best way to maintain a changelog file with bzr? [06:51] lucidfox: well, there are two ways [06:51] you can do bzr log --format=gnuchangelog [06:51] or, you can manually commit to it and record those changes [06:58] Well, problem is [06:59] because of my "commit early, commit often" work style, automated log generation commits spams the log with one-line entries like "fix typo" or "add comments" [06:59] and the ChangeLog file is supposed to list *functional" changes, right? [07:00] right, so in that case i'd suggest probably maintaining it by hand, rather than automatically [07:30] Ok, patch submitted to http://bugs.python.org/issue9729. If upstream likes it I'll propose it for backporting to Python 2.6 in maverick, which will make our test suite a tiny bit happier. [07:32] It's a bit disconcerting fixing bugs in the Python standard library... the test suite doesn't seem anywhere near as comprehensive as I'm used to with bzrlib. [07:47] heh [07:47] feedreader was fun [07:47] "When the light of life has gone, no change for the meter. Then the king of spivs will come, selling blood by the litre." [07:47] they have a ton of tests but they're all sample data matched against the parsed form [07:48] indeed [07:49] so what do you think for a name? --yes? --jfdi? --no-confirmation? [07:49] --make-it-so [07:49] can't go wrong with Picard references [07:49] for bug 397315 [07:49] Launchpad bug 397315 in Bazaar "break-lock should have a --yes, --force or --unconditional option (affected: 1, heat: 6)" [Medium,In progress] https://launchpad.net/bugs/397315 [07:50] --thats-an-order [07:51] --crowbar [07:51] (lock breaking and all) [07:58] --yes sounds ok to me, it reminds me of fsck -y [08:05] hi all [08:05] :( i like --crowbar [08:07] poolie: why not a config var to define the ui itself ? [08:10] vila, how do you mean? [08:10] vila, istr that null output from script commands matches anything? [08:10] is that true? [08:10] it's just been confusing me [08:11] poolie: a yes_ui [08:12] null output matches anything... hmm, doesn't ring a bell, that would be quite a bug [08:12] mm it seems to be true though [08:13] vila, i agree, i think -O'ui=YesUI' should work [08:13] oh, wait, if *no* output is specified, then, yes, I think it's defined as meaning: "I don't care about the output", but that's not the same as specifying an output and getting none [08:13] that needs just a little more buildup and it's a somewhat longwinded way to get it [08:13] yes, that's consistent with what i'm seeing, but i think it's a bug [08:13] i recall discussing it [08:14] i think if you don't care, you should say '...' [08:14] poolie: but then you have to do it always, yeah may be we discussed that already [08:15] i should say i'm generally really super happy with scripts [08:15] I don't know if we have already enough data for that (nor if can define >/dev/null) [08:15] that's why i'm hitting bugs [08:15] :) [08:15] hehe [08:15] i think robert has a point they could create a pressure towards excessive blackbox testing [08:16] I fully agreed and still agree with that. I consider them a first rough shot to reproduce a bug [08:16] from there you drill down and write more precise tests [08:16] i think if you're going to write a blackbox test, you might as well write it this way [08:16] it's clearer and probably no slower [08:17] and when the bugs are shaken out, it should be no less precise [08:17] there's still the question of how we could test the command ui other than by full-stack tests [08:17] (how or whether) [08:17] https://bugs.edge.launchpad.net/bzr/+bug/637830 [08:17] Launchpad bug 637830 in Bazaar "null output from test script matches anything (affected: 1, heat: 6)" [Medium,Confirmed] [08:17] oh, for blackbox tests yeah, I was thinking more generally [08:17] no point making it _hard_ to write bb tests as a way to make people write unit tests [08:18] lol, no, that's not my point :) [08:22] vila, i think https://code.edge.launchpad.net/~mbp/bzr/ui-factory/+merge/34956 is now all ready to land [08:25] poolie: didn't I approved it ??? ... looks like it... weird [08:26] poolie: done [08:35] hmm, that's cleanup day :) I like it, go submitters, go ! [08:38] poolie: Why bzrlib.ui.testsupport instead of bzrlib.tests.ui (or something, but under tests) ? [08:45] heya GaryvdM [08:45] Hi bialix. [08:46] Gary, I've pushed small fix to lp:bzr-windows-installers branch, please, next time you'll build the installer use it [08:46] GaryvdM: btw, how's your own build host going? [08:47] bialix: I installed a 32 host, so I was not able to build the tbzr 64bit shell ext. So I'm gonig to have to start from scratch [08:47] vila, i actually did originally put it somewhere like that but i think python was getting confused by multiple things called ui [08:47] or at leoast i was :) [08:47] cleanup day? [08:48] GaryvdM: oops :-( [08:48] hi bialix, garyvdm [08:48] heya poolie [08:48] Hi poolie [08:48] ueah, I thought about that :) So how about bzrlib.tests.ui_utils ? [08:48] Hi vila [08:48] sure [08:48] hey GaryvdM, bialix ! [08:48] GaryvdM: which reminds me: why we still provides 32-bit bython version for 64-bit Windows? [08:49] vila: bonjour \o_ [08:49] i wouldn't mind having a rule that code outside .tests shouldn't call things inside it [08:49] aside of course from cmd_selftest etc [08:49] and this would advance that [08:49] poolie: yup, hence my remark about branchbuilder on the review comment [08:49] I'll probably build the 2.2.1 installer on the ec2 instance, and then have another go after that is done. I'll have to ask jam to revert the pyqt version. [08:50] GaryvdM: how hard it would be to have both 32 and 64 bits hosts? [08:50] bialix: Re 64bit python, - [08:50] bialix: at least twice as hard ? :-P [08:50] vila: exactly! [08:50] bialix: Re 64bit python, - I don't really know, I have not really looked at it [08:51] I have plans to get new computer soon, it should be 64-bit [08:51] bialix: My knowledge of the installer build is still small. I'm just following the docs.... [08:52] the only one question which bother me is: where to get the compiler for python extensions [08:52] bialix: good timing, 10.10 is nearly out :-P [08:52] vila: yep, I'd like to have dual boot machne [08:52] GaryvdM: which compiler are you using? [08:52] bialix: I was using VS C++ express, which is free beer, but not free speech. [08:53] bialix: even better: install 10.10 and use vms... way to go... ;) You will need a bit more RAM, but then you'll have both 32bits and 64bits windows available... [08:53] vila: I already have 32-bit Windows XP and pretty happy with it [08:53] bialix: you mean you'll get an *additional* computer ? [08:54] yes, new one [08:54] nice, have a look at synergy then, 2 computers is a nice setup, 2 mice, 2 keyboards... less nice [08:55] * vila looks around and see 5 mice 3 keyboards [08:55] hehehe [08:55] who's talking [08:56] I use synergy. [08:56] well, I use 1 kbd 1 mouse 99% of the time though [08:56] but there are cases were one PC needs a keyboard (boot, nasty problems) and 3 mices should just go in the attic :) [08:57] mice [09:01] vila i just got a magic touchpad today [09:02] it's pretty sexy [09:05] what's a magic touchpad ? [09:05] Oh, the apple one ? [09:05] yes [09:05] Can it call spells of 80 level? [09:06] :) [09:06] it can scroll both downwards _and upwards_ which my worn out old mouse couldn't do :) [09:07] poolie: yeah, I have a magic mouse (2 in fact :-p) it's a pity I had trouble configuring it properly for osx/vbox/lucid/synergys though :-/ [09:07] poolie: it works under maverick right / [09:07] spiv do you want to send your approved branches? [09:07] actually it's only on os x for now [09:08] poolie: ha. [09:08] i might try it on maverick later but my desktop pc has no bt adapter [09:08] oh nm, you did [09:17] vila i moved the uifactory stuff to tests.testui and it seems happy there [09:18] hehe, I thought about that too but didn't want you to say: oh, testui and test_ui are too close ;-p Please land ! [09:19] urk [09:19] they kind of are, but i think it's ok [09:21] sorry if it is a faq, probably I'm searching wrong keywords and cannot find the answer, how do I checkout a specific revision with bzr? [09:22] fargiolas: There are many ways to skin that cat [09:22] fargiolas: If you don't allready have a branch: bzr branch -r x [09:23] If you allready have a branch: bzr pull . -r x --overwrite [09:23] or [09:23] bzr revert -r x [09:23] or bzr update -r x [09:23] don't know if I have a branch, I ran brz branch lp:project-name [09:24] I still have to better understand bzr lexicon [09:24] I'm not sure what you want to do exactly. If you can tell me more, I can advise you better [09:24] you do [09:24] fargiolas: you want a copy of the tree as at a particular revision? [09:24] GaryvdM: I want to look at a project history looking at each commit changes [09:25] fargiolas: You do have a branch. do cd project-name; ls [09:25] if you're familiar with git, I'd like something like git log -p [09:25] 'bzr log -p' :-) [09:25] really, cool :D [09:26] as I said I'm pretty new to bzr, sorry :-) [09:26] fargiolas: no problem. [09:27] fargiolas: if you prefer fancy GUI, try 'bzr qlog' you need the qbzr plugin for that but it's worth it [09:27] vila: I'll try it, thanks [09:35] vila did you see spiv's neat "testdoc"? in lp:testdoc [09:36] prints a summary of the contents of a test module [09:36] i like '-f shiny' [09:36] poolie: yup, I need to use a dark backgrounf though or most of it is unreadble :) [09:37] ga <- fix typos [09:45] poolie: yeah, I'll submit them. I wanted to make sure the Inter1and2Helper fix got in because 2.2.1 is due quite soon. [09:46] How to launch bzr-gtk or bzr-explorer on Windows systems? [09:48] RomanMoz: If you installed with the windows installer, you should see a short cut in the start menu, and optionally on the desktop [09:48] RomanMoz: How did you install? [09:49] RomanMoz: From the console, you can run "bzr explorer" [10:05] hi [10:06] does someone here used bzr-git plugin? [10:11] hrw: sometimes [10:12] so far all my attempts ends with backtrace or other erros [10:13] bzr: ERROR: Invalid http response for https://github.com/hrw/linaro-armel-cross-toolchain.git/info/refs: Bad status line received [10:14] ok, found bug for it [10:14] bug 581933 [10:14] Launchpad bug 581933 in Launchpad Bazaar Integration "Imports from http://github.com fail, while ones from git://github.com work (affected: 1, heat: 6)" [High,Triaged] https://launchpad.net/bugs/581933 [10:15] hrw: Hi [10:15] hrw: "smart" http repositories aren't supported yet, but git:// should work for github repositories [10:16] 11:15 hrw@home:bzr-import$ bzr git-import git://github.com/hrw/linaro-armel-cross-toolchain.git [10:16] bzr: ERROR: exceptions.AttributeError: 'InterRemoteGitNonGitRepository' object has no attribute 'fetch_refs' [10:18] I prefer to create bzr branch with full git history but doing it by "git format-patch" + playing with bzr version of "git am" is what I do not want to play with [10:19] hrw: What version of bzr-git is that? Trunk should work. [10:20] *** 0.5.2-1 0 [10:20] 650 http://pl.archive.ubuntu.com/ubuntu/ maverick/universe amd64 Packages [10:20] hrw: Alternatively, you can have Launchpad do the import for you. [10:21] jelmer: where it is in LP? [10:22] hrw: The "Import a branch" link on the Code page of a project. [10:22] hrw: Or if you mean bzr-git itself, lp:bzr-git [10:23] ok, so time to create projects first [10:24] vila wow we're really going to need to see about giving better errors on failure [10:24] the assertionerror is not always super helpful [10:25] 'morning poolie, vila [10:25] hi there jelmer [10:26] poolie: ECONTEXT [10:27] jelmer: hey ! [10:29] morning all. [10:30] mgz: _o/ [10:30] vila there's not much explanation of which bit of the script it's unhappy about [10:30] but i just improved it slightly [10:30] I see pqm doesn't have news merge. [10:31] poolie: ha, will lineno help ? [10:31] it looks like only a few tweaks are needed to make '' not be a wildcard [10:31] mm, or just giving the command line [10:31] mgz: hmm yeah, I should probably file an RT to get it turned on. It only helps a bit, but that's still worth something. [10:31] that's not quite unambiguous [10:31] spiv you should [10:32] ok :) [10:32] spiv fixed bug 625574 during the night? [10:32] Launchpad bug 625574 in Bazaar "testtools details should not be shared between parameterised tests (affected: 1, heat: 9)" [Medium,Confirmed] https://launchpad.net/bugs/625574 [10:33] mgz: ha, thanks, I mentioned you to spiv but couldn't find back the mp... :) [10:35] spiv, mgz : I let you discuss it :-) [10:35] actually spiv i thought there was one already? [10:38] is this a known bug i sometimes get EBADF inside paramiko running parallel=fork? [10:38] adding a copy method was what Robert mentioned as well, just hadn't got as far as digging into what would need fixing where myself [10:38] probably thread leaks of some kind [10:40] thx [10:48] poolie: I dunno it a bug ws filed for it, but yes, it's more apparent now that the leaks have been fixed but it's an old one [10:49] poolie: the root cause is that when a client and server communicate via socket the events related to the socket being EOLed do not always occur in the same order [10:50] poolie: we catch EBADF on both the client side and the server side, but paramiko use its own private thread, where EBADF, sometimes, occurs [10:50] poolie: IIRC it doesn't make the test fail thoug, it's just annoying to have it in the output at all :-/ [10:53] ah ok [10:53] vila ah i see the naughty test_conflicts blackbox tests, but not in that directory :) [10:53] [10:54] hm [10:54] so they are nice and readable [10:54] they are kind of using the command line to do setup, which will tend to be slow [10:54] poolie: yeah, I should rewrite them into more precise tests as hinted earlier :) [10:54] and i think was lifeless's objection to adding this facility [10:55] just thinking out loud, not trying to give you a hard time [10:55] no no, np [10:55] I was in exploratory mode when I wrote them so they fitted the bill: how to create such and such conflicts [10:56] some other things it's teaching me [10:56] AFAIK there are the most complex setups we have and spiv and I weren't fully happy about the parametrization either [10:56] we're a bit inconsistent about stdout vs stderr (not totally news) [10:57] and also, a universally respected -q would be good for these tests [10:57] also apparently -1 doesn't work with --parallel [10:58] no, at best it could stop one thread (well process) [10:59] these tests are a good example of why I didn't care about command output... until the very end, so adding '...' for every line will make them *far* less readable [10:59] actually -q is pretty good [10:59] on the other hand, they are not here to stay... [11:00] adding it to every bzr invocation in the script ? wfm [11:00] to the ones you don't care about [11:00] an alternative would be to add an option to the run_script method to keep the actual behavior [11:01] hi guys, i'm trying to write a plugin to add credentials into gnome keyring in the format that bzr-gtk expects [11:02] but i'm getting lost in the code somewhat - can anyone point me to how to get the credentials stuff from a URL? [11:03] Glenjamin: look into bzrlib/transport/__init__.py split_url and unsplit_url [11:03] Glenjamin: be aware that the credential stores doesn't provide any facility for storing new credentials [11:04] Glenjamin: the rationale is that keyring, ssh-agents are best fitted for that [11:04] the gnome-keyring bit in bzr-gtk uses a hash of attributes to store [11:04] the gnome seahorse program doesn't let you set them [11:06] so i can't see a way to non-programatically set the password [11:07] I see, but still :) [11:07] * vila searches [11:08] so the good news is that -q does consistently suppress most stuff [11:09] i suspect there are still some where it's not correct [11:11] Glenjamin: meh, seahorse let you create passwords or keyrings or ssh keys or pgp keys, what are you trying to create ? [11:12] http://bazaar.launchpad.net/%7Ebzr-gtk/bzr-gtk/trunk/annotate/head%3A/keyring.py#L60 [11:13] you can create a named password, but you cant set the attrs dictionary from seahorse - although it will display it for existing passwords [11:14] Glenjamin: did you look at oython-gnomekeyring ? [11:15] yes, it has bindings to let me store a password [11:15] i managed it manually [11:15] but now i want to be able to do "bzr gpass some://url" and have it prompt me for username/password, and store it [11:16] i'm struggling to convert the URL argument into the relevant bits however [11:16] Glenjamin: haaa, ok, so you don't try to store them in authentication.conf ? [11:17] as ideally i'd like to be able to plug in the same url that any other command would take - in this case it's a custom directory service which maps to bzr+https [11:17] vila: nope, if the password is in the keyring it seems to pick it up [11:17] Glenjamin: then look at bzrlib/transport/__init_.py [11:17] that would be a nice feature [11:18] i'm intrigued as to how jelmer manages to get the passwords into keying - as he's the author of this bit of bzr-gtk [11:18] Glenjamin: that's the trick, it doesn't, he just uses the ones that are there [11:18] s/it/he/ sorry jelmer :-/ [11:18] but they'd have to get there somehow, no? [11:19] Glenjamin: other apps can add them [11:19] yay, done, and not too tedious to change [11:19] vila https://code.edge.launchpad.net/~mbp/bzr/scripts/+merge/35386 - no rush [11:19] Glenjamin: circa line 1354 there is one example [11:19] thanks for the reviews! [11:19] poolie: ok [11:19] lunch time here, bbl [11:28] annoyingly, parsing the url just gives me bzr+https. which isn't much use. Since this is only for one repo i'm just going to make a more specific command. :( [11:51] i've run into a weird error. I've got an https smart server running, and http redirecting to https. if i try and do a command to bzr+https, i get an error about 401 being an invalid response. If i do bzr+http it redirects to https and prompts me for my password. :s [11:56] and it only happens with pycurl [13:09] hmm. how do i check if a revision is valid from a revision_dwim object? [13:13] Glenjamin: what do you mean by 'only bzr+https' ? Care to paste some code demonstrating that ? [13:13] !paste [13:13] For posting multi-line texts into the channel, please use http://paste.ubuntu.com | To post !screenshots use http://tinyurl.com/imagebin | !pastebinit to paste directly from command line | Make sure you give us the URL for your paste - see also the channel topic. [13:19] knittl: hmm, I don't think you can, the dwin processing is about trying various kind of revisions until one succeeds, see bzrlib.revisionspec circa line 325 [13:19] vila: well, i want to test if it's 0 [13:19] so -r0 [13:20] and print revision outputs [13:20] knittl: what's wrong with -rnull: ? [13:20] vila: it crashes [13:20] * [13:21] knittl: what crashes ? [13:21] bzr [13:21] ok, doing what ? [13:21] inventory -r0 [13:21] inventory -r0 . works (giving filelist) [13:22] but it should output nothing right ? [13:22] yes [13:22] file a bug, probably pretty shallow [13:22] or a warning 'r0 does not have an inventory' or something like that [13:22] vila: i try to fix it myself [13:23] knittl: good, thanks ! [13:23] that's why i asked how to check the revision ;) [13:23] knittl: but once the bug is filed you can assign to yourself and everybody will know it's a know one if they encounter it [13:24] nobody is as stupid as i am and will try to show the inventory of r0 ^^ [13:24] yada yada [13:25] I'm surprised it didn't show up in other contexts... and that we don't have a test for that [13:25] if type(dir_ie) is type(None): return; [13:26] in inventory.py entries(self).descend(dir_ie, dir_path) [13:26] seems to fix it [13:27] in descend ? yeah, I wsa there too (note that we use the 'if dir_ie is None' idiom instead [13:27] that did not work [13:27] but it was 4am when i tried last [13:27] :D [13:27] oh ok, works too :) [13:29] should this go inside descend or should i check self.root before calling descend? [13:29] checking root.self will be called only once instead of each subdirectory [13:30] knittl: I think it's an old code path, http://paste.ubuntu.com/493618/ works [13:30] lol, ok [13:31] knittl: well, you did the hard work :) Please send a merge proposal ! [13:32] although entries(self) says "this may be faster than iter_entries" [13:32] no, it's bogus, it breaks the test suite [13:33] ok, iter_entries works too [13:33] knittl: no it's not, it breaks the test suite :) [13:34] what? entries()? [13:34] hu [13:34] what now? :D [13:35] hmm, may be the test suite is wrong and you uncovered more than you thought :) [13:38] knittl: told you :) File a bug, there are weird things there [13:38] meh, i wanted to patch!! [13:38] !bugs [13:38] If you find a bug in Ubuntu or any of its derivatives, please file a bug using the command « ubuntu-bug » - See https://help.ubuntu.com/community/ReportingBugs for other ways to report bugs - Bugs in/wishes for the IRC bots (not Ubuntu) can be filed at http://bugs.launchpad.net/ubuntu-bots [13:38] !bugs bzr [13:39] stupid bot xD [13:39] knittl: Please keep working on the patch ! :) You can then do a merge proposal and with that and the bug, we are sure to track the issue until it's fixed [13:40] well for me both iter_entries() and entries().descend() with the if dir is none works [13:40] i haven't run the testsuite though [13:41] knittl: I did run it in a dirty bzr, cleaned now, re-running [13:41] knittl: you could run of subset with 'bzr selftest -s bt.test_inventory [13:43] bzr: ERROR: No module named testtools? [13:43] knittl: argh, yes, OS/version ? [13:43] maverick [13:43] knittl: cool, so just apt-get or synaptic will do [13:43] * GaryvdM really really really would like to have shallow branches... [13:44] GaryvdM: hm? [13:44] GaryvdM: even if your mirror updated your repo at night ? [13:44] thought bzr would already allow such a thing [13:44] vila: what's the package name? [13:45] GaryvdM: or say, every hour (which should be fast enough) [13:45] python-testtools? [13:45] knittl: let me check, but if this one exists, sure [13:45] yes [13:46] knittl: it's a bit old, (0.9.6 has been released) but it shouldn't matter (/me crosses fingers) [13:46] vila: I'm branching a piece of code I don't normally work on [13:46] lp:ubuntu/ubiquity [13:46] vila: http://paste2.org/p/987536 [13:47] and my internet line is being very slow, I'm only doing 2kB/s [13:47] knittl: ok, now try: 'bzr selftest -s bt.test_inv -s bb.test_inv -s bb.test_non_ascii' [13:48] ok, running [13:50] testtools 0.9.4' is in ~bzr/proposed, ftr, though Martin gz said it may be more buggy than 0.9.2 in some areas [13:51] How does one copy an entire colo workspace and its branches? colo-sync-* requires that there already exists a mirror workspace. [13:51] Is one supposed to do a bzr colo-init followed immediately by a colo-sync-from? [13:52] vila: "ran 729 tests in 164.478s OK" [13:54] knittl: sounds good, what's your patch ? [13:54] if self.root is not None: descend(self.root, u'') [14:01] vila: http://paste.ubuntu.com/493632/ [14:01] knittl: hmm, there are valid cases where self.root is None while the inventory is not, this sounds like bad test coverage :-/ [14:02] Glenjamin: well, in that case user and password are None indeed [14:02] vila: meh… [14:02] vila: but no [14:02] self.root cannot be none in entries [14:02] this is the bit where i'm trying to get the attributes of the URL to store against the keyring [14:03] because if it is None the .children call will fail [14:03] which would be protocol = https, server = bzr.genesys-solutions.co.uk; I know i can just chop bzr+ off the front - but that kinda feels like cheating [14:04] knittl: typed too fast, checking something (in this code path you're right, but the Inventory __init__ allows it to be None) [14:05] i'm only checking in descend [14:05] ideally i want to try and access the url, and then where it would normally go hit the credential stores, just grab those attributes and prompt for a user/password that then gets saved. [14:05] but i dont think the stack is really set up for that. [14:05] how can i set name and email locally for a branch? [14:05] and how can i amend the last commit? (override author information) [14:06] and what's the equivalent of git's format-patch? [14:06] Glenjamin: indeed the stack is not setup for that yet (well, only in a rudimentary form which doesn't interact with credential stores IIRC) [14:07] knittl: bzr whoami [14:07] knittl: bzr uncommit; bzr whomai; bzr commit [14:07] vila: no, whoami is globally [14:07] uncommit will clear my message? [14:08] in my particular case the repository is always the same, so i added a command which can store credentials for that repo. But my plan of making a generic "gpass" command is pretty much kaput [14:08] knittl: bzr push lp:~knittl/bzr/bug#-fix-inv-r0 ; bzr lp-open [14:08] whoami accepts a --branch option [14:08] knittl: If you use qcommit, it will uncommit will remember it for you [14:09] thanks for the support guys :-D [14:10] knittl: I think to set whoami for a branch, you have to edit .bzr/branch/branch.conf . I'm not sure if there is a ui for that. [14:10] knittl: Not you can also do bzr commit --author [14:10] GaryvdM: whoami --branch works [14:10] s/Not/Note [14:11] knittl: Oh - learn something everyday :-) [14:11] bzr help whoami revealed its secrets :) [14:13] ok, so i need to push to launchpad? [14:13] knittl: that's where you will be able to create a merge proposal and where we will review it [14:14] bzr-pqm is correct 'parent'? [14:15] ugh, now for the reporting part [14:15] knittl: Yes [14:16] https://bugs.edge.launchpad.net/bzr/+bug/379740 [14:16] knittl: yes, bzr-pqm is our automated gate keeper that ensures that all commits on the mainline pass the whole test suite [14:16] Launchpad bug 379740 in Bazaar ""bzr inventory -r 123" requires a working tree (affected: 1, heat: 0)" [Medium,Confirmed] [14:16] looks like the bug? [14:17] knittl: nope, some command but different bug [14:17] same ... [14:17] vila: but -r0 will also have no working tree? [14:17] knittl: no, it can have an empty working tree [14:18] knittl: this bug is about using 'inventory' on treeless branches [14:18] whatever. report a bug *click* [14:18] vila: ah ok. understood! [14:23] done: https://bugs.edge.launchpad.net/bzr/+bug/638034 [14:23] Launchpad bug 638034 in Bazaar "`bzr inventory -r0` crashes because self.root is None (affected: 1, heat: 6)" [Undecided,New] === Meths_ is now known as Meths [16:32] what can i do with a file_id? is there a way to `bzr cat` it? [16:33] i'm trying to fix up the test suite in tarmac to use bzr's TestCaseInTempDir, and super() instead of direct calls up to the parent, but now i'm getting a weird error: http://pastebin.ubuntu.com/493658/ [16:33] can anyone help me figure it out? [16:39] dobey: it's still going through TestCaseWithMemoryTransport, I didn't think TestCaseInTempDir was a subclass of that [16:43] james_w: nothing in my tarmac tre eis using TestCaseWithMemoryTransport though [16:44] knittl: it's used internally in bazaar, not really exposed at the UI level. why would you want to ? [16:45] jelmer: showing an inventory -> displaying file contents [16:45] james_w: i guess super() just really hates multiple inheritence :( [16:47] class TestCaseInTempDir(TestCaseWithMemoryTransport): [16:48] knittl: is there a particular reason for not doing that by file path though? [16:48] inheritance doesn't look wrong to me. [16:48] jelmer: file paths may change? [16:48] knittl: Not in a particular inventory. [16:49] but between revisions [16:49] knittl: But you just mentioned you wanted to lookup paths in an inventory [16:49] s/paths/files? [16:49] s/paths/files/ [16:50] what's wrong with using file_ids? [16:50] dobey: a minimal test case that gets you that stack would be helpful [16:50] (god i hate bazaar) [16:51] when i want to know how to display contents of file ids i want to do that, i don't want to show contents of a normal file or use its filename … [16:51] (sorry, pet peeve) [16:51] knittl: What are you trying to do exactly? [16:52] knittl: You seem to be mixing storage-specific things with UI concepts. [16:52] i'm not interested in UI [16:52] but i'd prefer having a command in bzr which does what i want [16:53] then that would be UI, no? [16:53] knittl: can you give us some more context? [16:53] i'm analysing storage/object models of different dvcss [16:53] and bzr documentation is really awful [16:54] i'm reading pydoc of inventory classes [16:54] so bzr inventory -v --show-ids will give me names and their id [16:54] i guess bazaar tracks history per file (not easy to find either) [16:54] and a file_id identifies a specific file [16:55] so i should be able to show its content by using the file_id [16:55] you would need both the file id and the revision id [16:56] ok, i don't care if i need revid as well [16:56] how can i display it? you can also link me to a page which describes the command (or tell me the command name) [16:56] knittl: bzr cat [16:57] knittl: though you specify the path and the revision [16:57] i don't want to use the path [16:57] i'm specfically asking for file_id [16:57] knittl: Why not? the path is unambiguous if you also specify the revision [16:58] is there a *single* identifier of a file in a revision? [16:58] knittl: yes, the path. [16:58] that's path+rev [16:58] that's two things, not a single one [16:58] knittl: right [16:58] you said "in a revision" [16:59] "of a certain revision of a file" <- better? [16:59] knittl: a tuple of a revision and a path, or a tuple of a revision and a file id [16:59] i'm trying to find an easy bug to fix; 521452 looks doable, but I can't reproduce it with 2.1.0rc1 (where it was reported) or the tip. Any hints? I've tried on Ubuntu 9.10 and using Wine on the same system, but no luck [17:00] bzr cat -r5410 bzrignore-20050311232317-81f7b71efa2db11a [17:00] roryy: it may have been incidentally fixed by poolie [17:00] not working … [17:00] knittl: Why would you prefer that to "bzr cat -r5410 .bzrignore" ? [17:00] rorry: do you want suggestions for starter bugs? [17:00] mgz: please [17:00] jelmer: because i say so -.- [17:02] knittl: Anyway, the short answer is that there's no way to cat by file id from the UI. [17:02] rorry: first, if you'd like to test my theory on that you could try checking out a rev before poolie's change, and if the bug exists there but not now, closing that bug as fixed [17:03] jelmer: that sucks … [17:03] rorry: appears to have been merged in r5371, so try r5370 [17:03] mgz: cool, thanks. I'll try that [17:06] jelmer: hg has `hg debugdata` and `hg debugindex` [17:06] rorry: for bugs to work on, see https://bugs.launchpad.net/bzr/+bugs?field.tag=easy - but again some of these may have already been fixed, and some may not actually be easy at all, so feel free to ask questions [17:07] mgz: will do. and some tagged "easy" definitely don't look that easy :-) [17:08] any luck repoing your first bug on an earlier bzr? [17:08] mgz: no. still checking that i haven't made a mistake, tho [17:09] it might help to go back even earlier, to a rev from the same time when Gary filed the bug [17:10] i tried that by checking out tag 2.1.0rc1 [17:11] sorry, that's for 515569, for which the other is marked as a dupe [17:11] was just about to ask that. [17:11] https://bugs.launchpad.net/bzr/+bug/202374 looks like a nice first bug [17:11] Launchpad bug 202374 in Bazaar "pull and update should accept --show-base (affected: 0, heat: 0)" [Medium,Confirmed] [17:17] or how about bug 417922 [17:17] Launchpad bug 417922 in Bazaar "treat MemoryError as a special/environmental error (affected: 1, heat: 0)" [Medium,Confirmed] https://launchpad.net/bugs/417922 === spike_ is now known as spikeC [17:17] I know that code so can help you write a test and everything. === spikeC is now known as spike_ === spike_ is now known as spikeWRK [17:21] I probably need to be able to reproduce this simple bug before i try anything more ambitious [17:21] ...I'll try and repo here, it may well be you're doing nothing wrong [17:22] ...quick thought though, you are using ./bzr in the directory of the checkout right? [17:22] yeah [17:26] okay, I can't repo with either of those old revs either. [17:28] mgz: http://pastebin.ubuntu.com/493724/ [17:28] can i use a hash instead of a revno? [17:28] and how do i find the hash of a revision [17:28] mgz: testing that with trial gives the same error === Ursinha is now known as Ursinha-lunch [17:28] knittl: bzr help revisionspec [17:29] ok. and how do i get the revid? [17:29] dobey: okay, yeah, that's you getting screwed by multiple inheritance [17:30] bzr log -v -r i guess [17:30] no, cannot see it [17:30] --show-ids [17:30] mgz: so what can i do? [17:30] ^^ [17:30] i think bzr qlog shows it [17:30] -v often does nothing [17:30] not create the tempdir in BaseTestCase I'd suggest [17:30] inventory -v: same as inventory [17:31] vila: thax [17:31] * thanks [17:31] inventory is an hidden command, it get far less love than the exposed ones :) [17:31] it got a patch from me today … [17:31] -v is not used coherently [17:31] that means a lot [17:32] but it's displayed in the help of inventory [17:32] "bzr rocks -v" is the same too [17:32] knittl: revids and file-ids are *internal* ids, if you want to manipulate them, you'd better use bzrlib (the library, developer oriented) not bzr (the command-line user-oriented tool) [17:33] i don't want to manipulate them [17:33] just use them [17:33] s/manipulate/address|get your hands on/ same thing [17:34] no, quite different [17:34] mgz: and why does the first test pass, but all subsequent tests fail? [17:34] bzr do not accept them anywhere from the command-line, except for the revid but it's not the recommanded way [17:34] first passing others failing sounds like not resetting state to me [17:34] because you remove the attribute tempdir, which the bzr class uses for a non-test-specific tempdir [17:34] and you removed it. [17:35] Glenjamin: yeah, in this specific case, it's even: blowing up the state ;D [17:37] dobey: just don't do this: tempfile.tempdir = os.getcwd() [17:37] but ideally, if you want to be using TestCaseInTempDir, don't do any of that stuff in the other base test case [17:38] we have to do stuff in the other base test case, because we have to create other files [17:39] if i just make the base test case use TestCaseInTempDir instead, then all the tarmac tests fail [17:39] dobey: and inheriting from TestCaseInTempDir ? [17:39] vila: hmm? [17:40] dobey: make your base class inherit from TestCaseInTempDir [17:40] i did that [17:40] and everything else failed. [17:40] single inheritance [17:41] class BaseTestCase(TestCaseInTempDir) [17:41] bzr revision-history is nice [17:41] vila, I think ideally he'd do that, but he's trying to update an existing suite and that means lots of changes [17:41] right [17:42] well ideally i wouldn't be here having this discussion, because the code i wrote to change this over, would just work [17:42] it works fine if i don't do super(), and instead just call up directly [17:42] life is never that simple :D [17:42] dobey: you can also calls both setUp and tearDown explicitely, but that won't address stepping on other toes^W attributes [17:43] anyway, just don't assign to global state in the tempfile module [17:43] it's dodgy and unneeded [17:44] does that change then get your real tests passing? [17:45] (there's no reason to change to using super for the cases it actually breaks) [17:45] they appear to pass with that change, but i suspect there will be other problems [17:46] and if changing it breaks the bzr test case, then the bzr test case is probably doing it wrong [17:46] why does bzr revision-history not take a revision as its argument? [17:48] knittl: because hidden commands are generally written for a specific need and then forgotten [17:49] dobey, no, do you want me to walk you through why that's bust? [17:49] vila: it would make a lot of sense [17:49] like git's rev-list [17:49] the point of TestCaseInTempDir is it creates a tempdir and chdirs into it [17:49] so bzr revision-history -r1000 [17:49] super call like that means it gets called first [17:49] show history from r1000 to 0 [17:50] knittl: seen the see also for revision-history ? That's what 'bzr log' does [17:50] you then set the global tempfile default creation to the cwd, which, remember, is going to get deleted in teardown [17:51] vila: hm? [17:51] 'seen the see'? [17:51] also bzr log shows a lot more [17:51] so the next tempdir that is going to get created is put in a dir that was deleted at the end of the last test, hence error [17:52] knittl: bzr help revision-history | grep 'See also' [17:52] i don't understand [17:54] knittl: why add -r to revision-history when it's already implemented for log whose job is to display the history of the branch [17:54] vila: log shows a lot more info than revision-history [17:55] knittl: indeed, listing the revids only doesn't make a lot of sense that's why the command never get more love [17:55] *sigh* === Ursinha-lunch is now known as Ursinha [17:59] vila: listing only revids makes a lot of sense when analysing low level architecture of different systems [18:00] knittl: the point is, that it wouldn't be a command commonly used. most people wouldn't care about it. [18:00] bzr is not doing well so far: little documentation, everything is hidden and locked away from the user, nobody can properly explain object/history model and different types of objects, etc. [18:00] knittl: I understand that and I think we've already had this discussion. [18:00] trying to analyse low level architecture from the command line doesn't make any sense at all. why don't you use a Python repl like any sane person would? [18:01] vila: yes, i say that a lot after i've been asking for something in #bzr :D [18:01] mgz: because the command line tools are (generally) there already [18:01] i don't understand [18:02] so, they aren't, and you'd just spent a couple of hours complaining about that. [18:02] mgz: yes. [18:02] why not drop that particular hammer and pick up a more suitable tool? [18:05] also, I'm interested in why you think documentation isn't there. We specifically picked bzr here for a few reasons: Better windows support, and good end user documentation. The things that 90% of people use are documented and documented well. Are there holes, yes, but no more so than other projects. [18:05] mgz: i've more-or-less given up on bug 515569 after trying it on several revisions: 4933, 4934, 4967, 5019, 5365, 5310, which seem to be when bzrlib/ui/text.py had relevant changes. I think generating a test case might be harder than it appears. [18:05] Launchpad bug 515569 in Bazaar "progress bar is not removed when bzr exits (affected: 3, heat: 12)" [Medium,Confirmed] https://launchpad.net/bugs/515569 [18:05] rubbs: end user documentation might be good, but i don't read end user documentation [18:06] it also lacks a lot when it comes to history/object model [18:06] rorry, I fear you're probably right there. leave it for the moment, and maybe bug garyvdm or bialix about how they hit it if you see them in the chat [18:06] i have yet to find good architectural documentation [18:06] knittl: ah, thank you for that clarification. [18:06] mgz: on bug 417922, i see martin pool did do something in 4634.26.1 -- did you have something else in mind? [18:06] Launchpad bug 417922 in Bazaar "treat MemoryError as a special/environmental error (affected: 1, heat: 0)" [Medium,Confirmed] https://launchpad.net/bugs/417922 [18:07] rubbs: with 'i don't read end user docs' i mean, that end user documentation is irrelevant here for me. it does not contain the information i need [18:08] knittl: i understand, I just didn't understand when you said bzr had little documentation. I've found that teaching it to my devs here was much easier because they had a lot of good documentation, but I have no experience with dev documentation, only user documentation. [18:08] so I can not comment one way or the other on it. [18:08] maybe i've just been missing it the last month [18:08] but nobody could point me to good docs [18:09] also the wiki has links to interesting articles [18:09] just to find those articles are empty [18:09] rorry, ha, another bug poolie fixed? [18:09] or have a single line of content [18:13] mgz: i think so. i must go now; i'll have a look at the bug Glenjamin suggested. [18:13] okay, sorry for the bumpy start they rorry. [18:13] *there [19:29] how do i go about adding a testcase for bzr inventory -r0? [19:32] knittl: I just replied to your email [19:33] I don't think we really care up at the 'bzr inventory' level, but it is good to have it at the Inventory level [19:36] jam: ok [19:36] that way it was easiest to explain here [19:36] knittl: sure [19:37] I'm just saying that you *could* explicitly test 'bzr inventory -r0' in a blackbox test, but I think testing closer to the change is better [19:38] i'm not interested in `bzr inventory` either, it's just the command where i found it to fail [19:38] how can i run only that test? [19:38] because i updated to the previous revision and the tests pass [19:39] `bzr selftest -s bt.test_inv -s bb.test_inv` [19:48] knittl: roughly that, you have to make sure to not just create "Inventory()" since the default creates a root [19:49] so overwrite root after that? [19:49] no, root_id is None [19:50] self.assertEqual(true, false) should always fail, right? [19:50] the test is not called :( [19:50] afk for 30 min … [19:51] Hi, how could I capture an error status of a bzr update in a bash script? [20:05] Hi, how can I capture an error status of a bzr update in a bash script? [20:07] danto: you could check the return code ($?), and/or pipe the stdout to a file. [20:08] danto: Or put the stdout in a var [20:08] var = $(bzr update) [20:09] Oh - that does no work, not sure why. Sorry [20:11] thanks GaryvdM, I'm already trying to pipe the stdout and doing grep -i error but doesn't work; bzr's error seems to broke any pipe [20:12] danto: Oh - let me try it myself. [20:13] GaryvdM, sure [20:14] danto: That's odd. bzr update | more does not work, but bzr update | less does. [20:14] danto [20:15] GaryvdM, I'm here [20:15] danto: I'm not sure. Maybe someone else may be able to help [20:15] danto: Sorry, I misspress enter after typing your name. :-) [20:15] thank you GaryvdM :) [20:40] danto: have you checked stderr vs stdout? [20:41] bzr update 2>&1 | less [20:51] jam not yet, I will try and let you know, thank you :) [21:43] I [21:43] I'm running python 2.5, but can't compile the extensions runnin 'make' b/c of missing python pyrex. Aptitude install python-pyrex gives me python 2.4. [21:43] Should I downgrade python? :S [21:44] try cython instead. [21:46] I'm guessing you're on lenny or an older ubuntu. [21:47] mgz: thanks, that's right, lenny [21:48] mgz: will cython give me pyrex? [21:48] it's an alternative, yes. [21:48] mgz: i installed it, do i need to take extra steps? [21:49] mgz: the make command still fails :/ [21:49] you may need to purge pyrex as well. [21:49] seems we try for pyrex first in the setup script for some reason. [21:50] mgz: No Pyrex, trying Cython... [21:50] mgz: bzrlib/_annotator_pyx.c:4:20: error: Python.h: No such file or directory [21:50] mgz: Cannot build extension "bzrlib._annotator_pyx". [21:50] ah, you need the build deps as well. [21:50] mgz: any clues? [21:51] mgz: build-essential is installed [21:51] I just installed bzr 2.2 from PPA to be able to run bzr resolve --take-other and i got bzr: ERROR: Tree transform is malformed [('duplicate', None, 'new-1', None)] [21:51] sender: sudo apt-get build-dep bzr ? [21:53] Hi mgz [21:53] thanks gary, couldn't remember the incantation. [21:54] chx: bug 430129 look like it? [21:54] Launchpad bug 430129 in Bazaar "merge aborts with Tree transform is malformed [('unversioned executability', 'new-1')] (affected: 1, heat: 2)" [Medium,Confirmed] https://launchpad.net/bugs/430129 [21:55] or bug 611739 instead? [21:55] how can i run a specific test? [21:55] Launchpad bug 611739 in Bazaar "shelve problem on shelving directory with ignored file inside (affected: 1, heat: 4)" [Medium,Confirmed] https://launchpad.net/bugs/611739 [21:55] mgz: i dunno. the merge finished. [21:55] GaryvdM: thanks [21:55] mgz: it's not shelve either. [21:55] file a new bug then. === chx_ is now known as chx [21:58] so this is a bug. [21:58] any way to get somethin gmore detailed to help you guys with? [21:59] yeah, look in bzr log for the traceback and extact command you ran [21:59] and paste that into the bug [21:59] need help with bzr tests please … [21:59] `bzr help selftest` [22:00] … [22:00] how can i run tests for inventory? [22:00] bzr selftest inventory does not work [22:00] oh good lord, bzr logs everything? [22:01] it's alright, we don't tell your mom. [22:01] `bzr selftest -s bt.test_inventory` [22:01] mgz: it does not run a test [22:02] self.assertEqual(true, false) [22:02] oh, wait [22:02] no, still. it should fail, but does not [22:02] paste the full diff of your change to the file I might be able to help you write a working test. [22:02] true and false should raise NameError if nothing else, this isn't cpp [22:03] asserting true and false should fail … [22:03] seriously, pastebin, bzr diff. [22:03] mgz: https://bugs.launchpad.net/bzr/+bug/638451 enough? need more? [22:03] Launchpad bug 638451 in Bazaar " bzr resolve --take-other bzr: ERROR: Tree transform is malformed [('duplicate', None, 'new-1', None)] (affected: 1, heat: 6)" [Undecided,New] [22:04] looks pretty good chx, did you notice anything else odd? [22:05] yes. bzr 2.2 is even more awesome than usual :p [22:05] http://paste2.org/p/988130 @mgz [22:05] what did the merge consist of, how did you work around etc. [22:05] uuuuuuhuuu [22:06] just ideas, generally the more you can braindump now the less you might have to remember later [22:06] knittl: that's in test_inv [22:06] so? [22:06] so, I guessed the name of the file wrong when I said -s bt.test_inv....ENTORY [22:07] mgz: I have a branch (alas, private project on lauchpad) stacked on top of a two months old lp:drupal. There are changes. I took a new lp:drupal , applied those changes, resolving the conflicts etc and then wanted to merge this back [22:07] mgz: so it's a bit complicated. [22:07] it's in the inventory.py file [22:07] no, test_inv [22:07] the test is on the path bzrlib.tests.test_inv.... [22:07] jelmer: Are there any neat tricks for causing bzr-svn to redo a find_tags_between, after the initial attempt after a 'bzr branch' threw an exception? [22:07] which is what -s wants [22:08] chx: that sounds relevent I'm afraid, so if you could try and explain it in the bug I'm sure it'd help [22:08] selftest -s bzrlib.tests.test_inv? [22:08] mgz: can you guys reach private LP repos? [22:08] yeah, or the bt. shortening [22:08] chx: I can't personally, but canonical support people probably can. [22:09] ok, it fails now [22:09] good :) [22:09] knittl: You can check if your test is included in your regex/-s by running bzr selftest xxx --list [22:09] or just -v, yeah. [22:09] ah [22:09] included in my regex? [22:10] ok, test failed. test runs without errors. WIN! [22:10] or your -s - selftest has a few different ways of selecting tests [22:10] a winner is very nearly you. [22:10] mgz: okay i included the examiner trunk uri then [22:10] no, it failed and after applying my patch it succeeds [22:10] chx: fantastic, thanks. [22:11] Is there some where (like a faq) that explains why bzr-svn often ends up with ghost revs? [22:11] knittl: right, you've nearly got your fix landed. [22:11] fix done, test done. [22:11] you want to deal with all the annoyance in one day? then... [22:11] I would like to link to such a thing in bug 638422 [22:11] Launchpad bug 638422 in QBzr "qlog: Delta loading crashes on GhostRevisionError (affected: 1, heat: 6)" [Medium,Confirmed] https://launchpad.net/bugs/638422 [22:11] knittl: have you signed the canonical contributor agreement? [22:11] no [22:12] have fun: http://www.canonical.com/contributors [22:12] do i have to? [22:12] yup, but it's reasonably non-onerous [22:14] then you'll have nearly seen your fix through from >_< to :D and it'll just want approval and landing [22:15] garyvdm: might be a question for the mailing list. [22:15] some kind of ghost writeup is needed if it doesn't already exist. [22:15] mgz: Ok === spike_ is now known as spikeWRK [22:50] Getting: bzr: out of memory on a simple branch from 1 server to an other [22:51] danto: have you checked stderr vs stdout? + bzr update 2>&1 | less <-- yes, it works thanks again :) [22:51] source branch is 40K (as outputted by du -h) [22:56] sender, see du -h .bzr [22:56] I have 137M of videos in a branch but .bzr is about 1.4G :S [22:57] danto: ok, well it seems that the repo is fine, i just get an out of mem on a branch [23:04] sender: what bzr versions are you using on the client and server? [23:05] lifeless: thanks for the reply [23:05] lifeless: Bazaar (bzr) 2.2.0 on the client (target) [23:06] lifeless: Bazaar (bzr) 1.13.1 on the server (source) [23:07] sender: try nosmart+ for the server url. [23:08] mgz: thanks, running now [23:08] mgz: lifeless: i do get "Doing on-the-fly conversion from RepositoryFormatKnitPack1() to RepositoryFormat2a()." Problematic? [23:09] can be, yeah. [23:09] thats a very memory intensive operation [23:10] mgz: lifeless: must admit I kinda got lost with repo formats during updates of bzr over the last 2 years or so [23:23] mgz: lifeless: mem 100% and process seems halted [23:27] sender: bzr has tried to reduce the format churn. 2a has been arround for more than a year, and there is a strong commitment to stick with it as the default for a lot longer. [23:27] yup [23:27] okay, sender, in which case [23:27] hi gary, lifeless [23:27] you'll want to branch to a fresh repo, so no format conversion [23:27] then upgrade [23:27] GaryvdM: great [23:27] Hi poolie [23:27] mgz: ok, can I do that with that old version of bzr, or first install 2.2? [23:28] you can do that as is. [23:28] and later push the upgraded version back to the server if you put a newer version of bzr on there [23:29] so, branch to a new dir not inside a shared repo is step one. [23:29] hi poolie [23:34] mgz: ok, new repo is Standalone tree (format: pack-0.92). I run bzr upgrade? [23:34] yup, which may again run out of memory, but hopefully won't. [23:35] if it does, just stick with the old format locally for the moment. [23:55] mgz: out of mem on the upgrade :( [23:56] mgz: would it be smart to do a (lightweight) checkout instead of a branch and later unbind? to split up operations.. [23:58] well, there's no reason you shouldn't use the old format for the moment locally [23:58] spiv: does bzr have any double-parameterised tests? [23:58] spiv: I mean foo(Bar)(Quux) [23:59] spiv: if so, the __copy__ hack will silently break them. [23:59] spiv: (including in plugins or launchpad or other subclassers.