[00:04] poolie, spiv: call on today? [00:05] igc, yes, but i can't get in [00:06] still trying [00:06] poolie: I'll hang up in case that helps [00:07] * spiv hangs up too [00:07] jelmer: ping [00:08] poolie: I got straight back in ok [00:08] is it your local phone maybe? [00:12] it might be my phone [00:12] when i tried i got through to varying extents [00:12] anyhow [00:13] let's just catch up here on irc? [00:13] Sounds good. [00:13] me too [00:13] so i think the PPA stuff is all settled now [00:14] today, I'm planning to review patches, particularly the xml8 serializer one [00:14] i'm going to look at what bugs i currently have assigned to me and either finish some of them, do reviews, or do startup time hacking [00:14] I need to learn more about that so reviewing that patch will help me [00:15] i'd like to suggest the whole project do a bug day tomorrow, will send mail about that [00:15] also i seem to have some Ubuntu bugs atm [00:15] so will file them [00:16] poolie: were you planning to tweak james_w's patch on lp: urls or shall I do that? we both voted tweak [00:16] i'll do that [00:16] thanks [00:17] also when lifeless is back i'll call him re his conference submission [00:17] and i owe thumper a call or sketch of the branch listing page [00:17] if you get a chance, I'd like some feedback on my hook patch/rfc too [00:18] I've sent off the core bits of the fix for bug 207558. I want to do a little more on it (as mentioned in the most recent email to the list about it), but it's essentially done. [00:18] Launchpad bug 207558 in bzr ""bzr branch http://bazaar.launchpad.net/...." fails with bzr.dev >= r3309" [Critical,In progress] https://launchpad.net/bugs/207558 [00:18] ok [00:18] i'll try to read that for you spiv [00:18] what is next for you? [00:18] So today will be mainly server-side hooks. [00:20] poolie: another thing for me today - kill & resume the OOo import [00:21] it's at 157k out of 305k, taking 18 minutes per 100 revisions so dog slow [00:22] I took the call to enable_cache out of the code yesterday at lifeless' request so I'll resume it with that code gone and see if it helps [00:31] ok, good [00:39] igc: We should find what the cache was masking and fix it. [00:44] igc: I strongly suspect that using commit builder will give you better performance and let you get to an incremental-change approach safely for the importer [00:45] igc: with appropriate tweaks to the builder of course; but thats what its meant to be used for - install_revisions shouldn't, IMO, need to exist. [00:46] lifeless: maybe. commit-builder currently does things much slower than the importer does [00:47] it simply can't make some of the assumptions than an importer can [00:47] which make big differences to speed [00:48] igc: without details I can't comment; but if we're to continue to accelerate commit, commit builder has to learn how to work with very smart trees; which an import source can be considered to be. [00:48] igc: in dealing with merge revisions, if commit builder is slower, I have to assume that the importer is wrong :> [00:48] I'm going to do a couple of spikes today [00:48] and get back to VF this afternoon [00:50] I chatted with benno last night about python performance [00:50] and he concurred with my import analysis, gave some commentary on os call performance readdir etc [00:50] so I'm going to do this long threatened import hook [00:53] lifeless: based on my stracing and experiments in London, I'm only expecting a very modest win (300ms improvement in cold cache time). I'll be interested to see if the reality is different. [00:55] spiv: strace doesn't show how much time python spends marshalling [00:55] spiv: I suspect that there is some compiler overhead that won't be addressed by a simple import hook [00:59] holy cow 'import bzrlib' hits a lot of shit [00:59] 34 modules [00:59] yes, i know [01:00] many are demand load stubs I think [01:00] or something strange [01:01] -weird [01:01] >>> type(sys.modules['bzrlib.struct']) [01:01] [01:02] That's expected. [01:03] When a module inside the bzrlib package tries "import struct", python first treats it as a relative import. [01:03] And when an import fails, it caches misses as well as hits in sys.modules. [01:03] ah good [01:04] (Bring on python 2.5...) [01:04] 12 real modules imported then [01:20] lifeless: I see 58 new modules imported by "import bzrlib" with bzr.dev. (Only 12 with http://people.ubuntu.com/~andrew/bzr/faster-startup, though...) [01:21] (as measured with PYTHONPATH=. python -c "import sys; get_mods = lambda: set(name for name, mod in sys.modules.items() if mod is not None); before = get_mods(); import bzrlib; after = get_mods(); print len(after - before)") === mwhudson_ is now known as mwhudson === kiko is now known as kiko-zzz [01:27] spiv: what does your faster-startup branch do ? [01:27] spiv: (I am only planning to alter the behaviour of bzrlib specific imports) [01:28] lifeless: basically, various hackery to defer unnecessary imports [01:28] (or avoid) [01:29] lifeless: it's cumulatively a few hours of iterating on "bzr --profile-imports" on simple commands like "rocks" and "status" and seeing what cheap wins I could get. [01:30] has anyone tried running freeze on bzr ? [01:30] Oh, and specifically also reducing the amount of the universe that "import bzrlib" imports :) [01:30] e.g. making *everything* import and running freeze [01:31] spiv: so my thesis is that we should make importing everything fast === bigdo2 is now known as bigdog [01:31] because in the general case most of the code is needed [01:46] spiv: what do you mean about 2.5? [01:46] i don't think he meant 2.5 [01:47] at some point, imports written the normal way will be absolute [01:47] which means less looking for things like bzrlib.struct [01:49] I meant file:///usr/share/doc/python2.5/html/whatsnew/pep-328.html [01:49] 2.5 introduces "from __future__ import absolute_import" [01:50] oh, heh [01:50] shows when i stopped paying microscopic attention to python development [01:54] damn, pkgutil doesn't exist in 2.4 [02:08] so, I'm saving 130/4900 open calls [02:08] all of which look like failures [02:18] question, how do I change the "parent branch" of a branch? [02:19] igc, is there a BB enttry for the patch from james_w you mentioned before? [02:20] igc, i take it you were referring to the one to give a warning when getting a readonly transport? [02:20] poolie: that's right [02:21] * igc looks [02:21] http://bundlebuggy.aaronbentley.com/request/%3C1205831646.7247.295.camel@flash%3E [02:21] poolie: ^^^ [02:22] thanks [02:23] LaserJock: pull --remember [02:26] lifeless: awesome [02:35] * igc lunch [02:46] * mwhudson tries to work out if it's strange or not for merge --preview to complain about uncommitted changes [02:51] mwhudson: there was a thread about that on the list recently [02:51] mwhudson: it shouldn't complain, IIRC [02:51] there was? must have missed that in my catchup [03:04] mwhudson: it should tell you, and take your changes as part of the merge [03:04] mwhudson: IMO [03:06] i guess that makes sense [03:14] ok, I now have my code being called consistently [03:21] lifeless: well, I'm not convinced. [03:21] It's not a very accurate preview that way. [03:21] abentley: why is it less accurate? [03:23] because merge won't make any of those changes. It will barf. [04:07] abentley: thanks for the review ofthe help change [04:07] np [04:08] Something else I meant to mention is that people will still get criss-cross warnings if they are using --weave or --lca [04:09] Even though those algorithms handle criss-cross better. [04:10] So perhaps it should say not to remerge if you're already using --lca or --weave. [04:10] so would you overall suggest using remerge, or revert and merge again? [04:13] back in a few minutes; getting lunch [04:24] poolie: I'd suggest revert and merge. [04:25] Fewer commands to learn that way. [04:29] abentley: when is remerge a good thing to learn and use then ? [04:30] I generally use it when I've got a single file with really wacky conflicts, and I want to try out a bunch of different mergers to get the best result I can. [04:31] If there are basic, intermediate, and advanced commands, I'd consider it intermediate. [04:31] ok [04:31] one thing I've found with loom is that there was a bunch of 'when I use it' implicit in the thing that I hadn't written down. [04:31] I think we may have something similar here. [04:32] that is basically when i use remerge too [04:32] rarely on the whole tree [04:33] could we at least make it fail safe on looms - like give an error rather than mess things up? [04:34] lifeless: are you familiar with the problem? [04:35] abentley: no, I've never used remerge like that [04:35] oh, you mean with looms - I've seen the bug report [04:36] So the problem is that remerge assumes the .THIS file coresponds to the left workingtree parent. [04:37] but with up-thread, it is actually the rightmost parent. [04:37] I think the things are about-face for performance [04:37] I didn't want to transform; then merge [04:38] I'm not sure your approach is wrong. A different approach might lead to more conflicts. [04:39] its on my long todo to teach bzr's merge the ability to be told different file suffixes [04:40] so we can drop the 'right files' in the right places [04:40] and give better herringbone labels [04:40] You'd also want to swap the conflict markers. [04:40] :-) [04:40] I'd actually like to call them by their thread names [04:40] Speaking of merge, did --reprocess stop doing its thing sometime? Or did it just never really cut things down a lot? [04:40] Sure. [04:41] fullermd: I think it became default [04:41] abentley: could you suggest some better help text? [04:41] should we say "use remerge FILE, or revert and merge again?" [04:42] Let's say "revert and merge again". [04:42] lifeless: So we understand the problem-- is there a short-term fix we can do like poolie suggested? [04:43] fullermd: I don't think it became the default. But it only helps a narrow class of merges, and only applies to --diff3 [04:44] lifeless: The problem so far is that we can't even tell that the merge was different. [04:45] I've got kernel configs in bzr to easily merge upstream changes, and every time I do (last was some months ago), I get hyuge spurious conflicts that it doesn't help. === mw is now known as mw|out [04:53] fullermd: line endings? [04:53] No, but changes in single lines ended up resurrecting giant blocks that I'd deleted. [04:53] Imagine we pushed some changes for April Fools day. Like, 4 revisions. What would the easiest way, hypothetically, to check an older revision to head? [04:54] I guess it's some artifact of the diff hunk for the revs where I whacked the blocks. [04:55] awmcclain: bzr push -r NNN --overwrite [04:55] awmcclain: (NNN == -4, I guess...) [04:55] spiv: And if it's a bound branch, can we do bzr ci -r NNN --overwrite? [04:56] fullermd: I'd normally offer to help, but I'm behind already. [04:56] awmcclain: ah, with a bound branch, use uncommit [04:56] spiv: 4 times? [04:56] awmcclain: which also takes a -r arg. [04:56] Or just run it 4 times, sure. [04:56] spiv: Gotcha! [04:56] Or just pull -r-4 --overwrite . [04:56] fullermd: heh. [04:56] fullermd: pull overwrite affects master branch :) [04:57] abentley: *shrug* It's not that big a deal. Bugs me for 30 seconds or so a couple times a year when I do it. Hasn't been a high enough priority for me to build up a test case for it. [04:57] fullermd: You might try --lca. It's kinda got --remerge built in. [04:57] s/--remerge/--reprocess [04:58] Last time I tried was probably pre-lca; I tried both diff3 and weave, with/out reprocess; came up the same every time. [04:58] But if I want to commit _over_ the current changes, not remove them.. could i bzr revert -r NNN then ci? [04:58] I'll try getting creative next time it comes up. [04:58] awmcclain: right [04:59] abentley, BB seems to be stuck, can you whack it? [05:00] poolie: did it give you a lock error? [05:01] abentley: I'm currently getting 502 Proxy Error [05:02] "Reason: Error reading from remote server" [05:04] abentley: me too [05:04] " The proxy server could not handle the request GET /." [05:05] I have increased the database timeout to unreasonable levels. [05:05] So there's an active lock, but I'm not sure whether it will resolve itself. [05:06] The last time I got a 502, it did. [05:07] same [05:07] you're using sqlite right? [05:07] Okay, whacked. [05:07] lifeless: right [05:08] there is something funny in the python bindings AFAICT [05:08] sqlite3? [05:08] abentley: how about this: [05:08] In complex merge cases, `bzr merge --lca` or `bzr merge --weave` may give [05:08] better results. You may wish to `bzr revert` the working tree and merge [05:08] again, or use `bzr remerge` on particular conflicted files. [05:09] i feel kind of reluctant not to mention remerge because it's what i usually run myself [05:09] i see your point though about involving less commands [05:09] Okay, fair enough. [05:09] and if they did help criss-cross just after the failure they will not lose anything [05:09] Perhaps we should omit the revert; remerge thing. [05:09] Pardon? [05:10] lifeless: Any suggestions on a band-aid for up-thread? [05:11] abentley: w.r.t remerge ? [05:11] Yes. [05:11] I don't think there is a good band-aid; I think it just needs the bzrlib work done. [05:11] """In complex merge cases, `bzr merge --lca` or `bzr merge --weave` may give [05:11] better results. You may wish to `bzr revert` the working tree and merge [05:11] again. Alternatively, use `bzr remerge` on particular conflicted files. [05:12] "'" [05:21] * poolie reads fix for 207558 [05:23] poolie: call ? [05:23] in 6m? [05:23] sure [05:38] poolie: re that TT bug, it's due to adding a file to a deleted directory, when the deleted directory was the root directory. [05:39] ! [05:39] ok [05:40] spiv: ping [05:40] Anyhow, now that I can reproduce it, it shouldn't be a hard fix. [05:40] I have a 'wtf' moment with my import hook [05:41] lifeless: your phone's engaged? [05:41] its trying to import 'debian_bundle' from '' in the Numeric path path_hook object [05:41] poolie: no?? [05:44] lifeless: pong [05:44] does Numeric do crack? [05:59] lifeless: I don't know if Numeric does crack or not. [06:00] beuno: around? [06:13] spiv: you have mail [06:13] spiv: should be self evident if you want to apply it to bzr.dev and play [06:14] spiv: I get quite reduced syscalls with this but its buggy :/ [06:38] lifeless: bzr-gtk has now been updated so that it functions correctly with my bzr-dbus patches, btw [06:39] they're waiting on you for review though. [06:40] thanks [06:51] abentley: bb seems to be wedged again === mwhudson_ is now known as mwhudson [07:34] lifeless: regarding loom-specific conflict markers, please be sure you won't break external tools that are looking for them (i.e. may be *add* thread name after the usual markers instead of replacing MERGE-SOURCE and/or TREE) [07:52] Want to make $$$ just by clicking? Sign up on http://bux.to/?r=eg0trip, and begin recieving checks! [07:54] no thanks [07:54] yes you do [08:04] lifeless: still around? [09:03] dunno who egotrip was, but we don't want spam here. [09:08] He didn't explain himself very well, certainly. I kept clicking, but I got @@@ instead of $$$ :| [09:15] and I'd prefer to receive cheques rather than checks [09:30] Hmm, how can I force a baseless merge with bzr? [09:32] jelmer: ping [09:32] schierbeck: I second that: ping jelmer [09:32] :) [09:32] schierbeck: what's it for you? In my case bzr-svn seems to have forgotten about branch relationships ;( [09:33] tough luck [09:33] nah, i just want him to give his 2 cents before i send in a merge request [09:34] yacc: merge -r1..-1 [09:35] james_w: is that a good idea, considering that the branch contains the same revisions as the remote one? (bzr just seems to think that they are unrelated) [09:35] yacc: probably not [09:35] how did you create your local branch? [09:35] bzr branch svn+https://... [09:36] from the same branch you are trying to merge now? [09:37] Look at: https://bugs.launchpad.net/bzr-svn/+bug/210705 [09:37] Launchpad bug 210705 in bzr-svn "bzr-svn forgets relationship" [Undecided,New] [09:37] james_w: but basically yes. [09:38] james_w: => it remembers where it pulled from, and pushed to in the past. [09:38] james_w: just when doing a merge it complains about being unrelated. [09:38] Oh, bzr missing lists all revisions as missing on both sides, plus the one that I did commit locally, which is only listed on one side. [09:39] but the merge did not help anything, it still misses 3/4 revisions ;( [09:40] james_w: despite the above merge -r1..-1, it still claims that the branches are diverging. [09:40] yacc: can you pastebin a bzr missing --show-ids [09:40] james_w: just a moment, first I'm curious ;) [09:42] james_w: they don't match :( [09:42] http://rafb.net/p/a1aZXh19.html [09:43] james_w: guess that is what jelmer meant with the ids can change? [09:43] jelmer!!!!!!!!!!!!!!! [09:43] they don't match is the reason that you are seeing this issue. [09:43] calm down, he's been highlighted, if he's around he'll see it [09:44] james_w: yeah, BUT: I had to change the scheme to include new trunks [09:44] revision-id: svn-v3-list-QlpoOTFBWSZTWTtPdCgAACpRgAAQAAK3r94gIABIbVT1Mj0QeptEEpAAANMcdP3o3SqXf2KNSpL4ShxMTeA8adCslybHoqgxGgqoZA-LuSKcKEgdp7oUAA..:3e2713fb-81e7-48d5-8ee4-88d6b0cf85af:trunk%2Flogs%2Flookery-memberfind:56 [09:44] revision-id: svn-v3-list-QlpoOTFBWSZTWYDqAEcAADdRgAAQAAK3r94gIABISkMo0zUaemoEkkaZMAm1IM83f4FQYGxfzVWsvATONoa96yIoS8IEx1BxVRIGB_MCYIMwX5NxaET_F3JFOFCQgOoARw..:3e2713fb-81e7-48d5-8ee4-88d6b0cf85af:trunk%2Flogs%2Flookery-memberfind:56 [09:45] see the base64 encoded list of paths <= I had to change that to accomodate a new branch. [09:45] you changed the svn branching scheme? [09:45] james_w: list-path1-path2 => list-path1-path2-path3 <= I had to add path3 because it's a new branch I need access to? [09:47] james_w: basicaly the revids for branch X contain the a list of all other branches that I'm accessing in the given SVN repo. [09:48] james_w: Hmmm, I think the fix would be to make bzr-svn use svn-v3-list-$PATH_USED_IN_THIS_CASE instead of svn-v3-list-$PATH_OF_THE_COMPLETE_SCHEME? [09:50] james_w: Any explanation what the different branching schemes are good for? [09:53] yeah, so it's caused by the change to the branching scheme [09:54] the scheme is to tell bzr-svn which paths in svn to consider branches [09:56] yeah, but the problem is, appending one new "path" in the repository is a quite expected thing to do. [09:56] It should not break all branches/checkouts in the wild, just because I need to track one additional location? [09:57] yacc: add this information to the bug report. [09:57] Already done so. [10:00] james_w: single would mean that I can only access exactly one path inside the svn repo? [10:00] I think so [10:01] Hmmm. [10:01] So basically I would need to parametrize the "schema name for property name" lookup thing. [10:02] It basically needs to take the URL being accessed into account ;( [10:06] * yacc wonders what the schemes are good for at all? [10:10] james_w: I'm hoping to review that gnome bug tracker patch tomorrow [10:10] if you put through abentley's tweak tonight, that would be nice ... [10:10] though I realise you may not have time [10:10] I'm heading off now [10:10] night all [10:11] igc: thanks, I'll try and get to it. [10:11] night [10:13] lifeless: ping [10:22] jelmer: ping [10:27] jelmer: would it be ok by you to implement a "EveryPathIsABranchScheme"? [10:40] Hello, when using bzr, can't I checkout just a single folder instead of the whole branch ? [10:41] right, you can't [10:41] bzr can only work with whole branches [10:42] unlike svn [10:43] svn doesn't have branch :) [10:43] *branches [10:44] luks: and branches is (almost) all that bzr has ;) [10:44] How do I make bzr send my local commits as a patch to the maintainer? [10:45] bzr send [10:46] I just discovered that just switching schemes won't work => the svn repos contains pointers to the old scheme name, ... [10:47] My only hope for a sane scheme is rapidly crashing. [10:51] jelmer: ping [11:05] I am trying to convince a colleague to use bzr for our collaboration. However, he has a mac OSX 10.4 intel. Is there a ready to use disk image for that machine ? [11:13] there's a macport I think [11:18] New bug: #210665 in bzr-svn "bzr svn-import fails" [Undecided,New] https://launchpad.net/bugs/210665 [11:20] New bug: #210705 in bzr-svn "bzr-svn forgets relationship" [Undecided,New] https://launchpad.net/bugs/210705 [11:21] New bug: #210723 in bzr "Some tests failed when selftest is invoked with -v" [Low,Confirmed] https://launchpad.net/bugs/210723 [11:22] yacc: pong [11:23] yacc: EveryPathIsBranchingScheme will be incredibly slow [11:24] jelmer: why? [11:25] jelmer: and it misses to_list, which makes --set break, but I guess it breaks for other schemes too. [11:25] yacc: When searching for revision ids, bzr-svn checks every possible branch path for properties [11:26] Well, it seems the that list paths get embedded in the properties somehow, so just doing a fresh branch with the ammended paths is not an option either. [11:26] in the case of EveryPathIsBranchingScheme that means one call per file/directory in the remote repository [11:27] And let me guess it does not cache the results in any way ;) [11:27] it does cache the results [11:27] but that's still too slow [11:28] brb [11:32] so how to fix the list scheme then? [11:34] how do you mean? [11:52] Well, the list scheme embeds the paths in the revision stuff. [11:53] Meaning that when somebody adds a new path (because a new path got added), bzr-svn implods. [11:53] jelmer: : #210705 [11:54] jelmer: easy to demonstrate: with a branch that uses a list scheme, add another path to the list => et voila, bzr thinks that the svn remote branch and your local bzr branch do not share any history. [11:54] Which is wrong. [11:55] Considering the fact that some svn users use very very creative directory setups, ... [11:55] yacc: that's correct because they have a different branching scheme [11:55] Ok, so how do I handle the case that my upstream project just got a new subproject? [11:55] yacc: What you really want is a fix for bug 130372 [11:55] Launchpad bug 130372 in bzr-svn "Abandon branching schemes" [Medium,Triaged] https://launchpad.net/bugs/130372 [11:55] yacc: you can use wildcards in listbranchingscheme [11:56] jelmer: Only if I would have started with wildcards. Take a look at the later comments in 210705 and you'll notice that it seems impossible to switch the scheme for a given repo and recreate the bzr branch [11:57] yacc: yes, that's correct [11:57] yacc: that's why there's bug 130372 [11:57] Launchpad bug 130372 in bzr-svn "Abandon branching schemes" [Medium,Triaged] https://launchpad.net/bugs/130372 [11:58] so what do I do? write myself scripts that replace ~/.bazaar/subversion.conf based on the directory I'm in when calling bzr? [12:00] yacc: The error fetching after changing branching schemes is a bug [12:01] So I open up a time loop window, and go studying the source code to fix that bug? <= I mean the scheme is part of the revision IDs? [12:05] Serious, new bug or known bug? [12:06] yacc: the branching scheme is part of the revision id [12:06] yacc: that's bug 130372 I mentioned earlier [12:06] Launchpad bug 130372 in bzr-svn "Abandon branching schemes" [Medium,Triaged] https://launchpad.net/bugs/130372 [12:07] jelmer: 130372 is not about a bug fix, it's about joined branches, right? Meaning that you could drop to supporting only trunk/none for bzr-svn and make the people use split out parts of these repositories, right? [12:08] yacc: no, it's about abandoning branching schemes completely [12:08] as soon as by-value branches are supported [12:09] well, replacing branching schemes with a "repository layouts", which are not included in the revid [12:11] So nothing simple for a newbie that has only a limited understanding of bzr internals might want to tackle ;( [12:13] yacc: not really, though it should be possible to get that "not present in "... error fixed [12:13] I'll see if I can fix that [12:14] hey everyone, the docs on this function are a little bit unclear in what order the branch and tree objects are returned [12:14] http://starship.python.net/crew/mwh/bzrlibapi/bzrlib.bzrdir.BzrDir.html#open_tree_or_branch [12:14] I think it is mentioned at the end, but I am still unsure [12:15] rexbron: tuple with tree and branch [12:15] jelmer: K, it mainly that the other funtions make that more clear [12:16] the first sentence should be fixed then [12:17] How do I undo local commits? [12:17] yacc: bzr uncommit? [12:17] or revert [12:18] bob2, jelmer: I think it should be changed to something like the method below: Returns (tree, branch) [12:18] famen [12:18] tr -d f [12:19] yacc: in bzr.dev, "bzr uncommit --local" [12:19] I don't think that's in a release yet, though :( [12:19] spiv: Well, uncommit seems to have worked :) [12:19] rexbron: tree will be None if there isn't one if you hadn't figured that out yet. [12:20] yacc: oh good :) [12:20] james_w: I think I need to refresh exactly what the difference between a branch on the file system and a workingtree (beyond different methods for those objects) [12:20] rexbron: i see ":return: (tree, branch)" for that method [12:20] rexbron: you can have a branch with no tree [12:20] i.e. you have .bzr/branch/ [12:21] but there are no files in the working directory, and there will be no .bzr/checkout/ [12:21] james_w: My comment was that it could be more clearly stated, like in the other functions [12:21] james_w: ok [12:21] you can get this with bzr remove-tree, or with a treeless repository [12:21] james_w: the furst sentence says the opposite, and I don't think it's unreasonable to assume that it should be correct also :-) [12:21] and it will also happen for remote trees. [12:21] bob2: that's true [12:21] rexbron: that is markup that used to denote the return value. [12:22] james_w: is the doc system supposed to format it differently that it currently is? [12:22] ah, it appears to be spelt wrong, as other functions have it formatter === mrevell is now known as mrevell-lunch [12:23] see open_containing_from_transport [12:24] http://starship.python.net/crew/mwh/bzrlibapi/bzrlib.bzrdir.BzrDir.html#open_repository seemes to also have a formatting issue [12:24] for :param _unsupported: [12:26] now that I look for it, I see a lot of method docs that are (perhaps) not formatted correctly [12:26] yeah, they do seem to be a bit bad. [12:27] the API documentation is very hit and miss [12:29] james_w: what system is being used, as I would be happy to submit patches [12:30] ok, looks like pydoctor [12:31] is mail from the bazaar list getting doubled up? [12:32] :) [12:33] sabdfl: I don't see it, you mean you receive multiple copies? [12:34] Is there a page which documents the syntax for pydoctor? or is it the same as something else? [12:34] rexbron: I think it uses the pydoc ones or whatever they are. [12:35] rest markup in python docstring might get you some useful pages from google. [12:35] james_w: yes, i do get multiple copies [12:35] peregrine% ./bzr pull [12:35] Using saved location: http://bazaar.launchpad.net/~bzr/bzr/trunk/ [12:35] bzr: ERROR: Not a branch: "http://bazaar.launchpad.net/~bzr/bzr/trunk/". [12:35] anybody know what would cause this? [12:36] when I go to that URL I see the loggerhead page [12:36] sabdfl: you're using bzr.dev [12:36] sabdfl: I think the default is for the list software not to forward it's copy to you if you are already in the to list, but it sounds like it's all mail. [12:36] sabdfl: the fix is on the list [12:36] ok thanks spiv [12:36] sabdfl: the bug is 207558 [12:36] ubotu: bug 207558 [12:36] Launchpad bug 207558 in bzr ""bzr branch http://bazaar.launchpad.net/...." fails with bzr.dev >= r3309" [Critical,In progress] https://launchpad.net/bugs/207558 [12:42] hmm, I can not seem to find what markup syntax bzr is using to generate the api docs [12:42] pydoctor's site does not really mention anything [12:43] rexbron: epydoc [12:43] epydoc [12:43] (the ReST variant of it, IIRC) [12:48] rexbron: http://epydoc.sourceforge.net/manual-epytext.html [12:48] james_w: [12:48] but it uses : to start, rather than @ [12:48] ya, I found that [12:48] it is the rest variant it uses, but that seems to suggest other markup. [12:49] From a quick look, the reason some is being formatted correctly and others not is a lack of a blank line [13:10] jelmer: i've made the bugs tab look good: lp:~dasch/bzr-gtk/revisionview-bugs-page [13:10] what do you think? [13:16] schierbeck: yeah, that's definitely an improvement [13:17] hmm, the bug isn't clickable anymore [13:17] try double-clicking [13:17] i'll fit a button in there [13:17] for now, i removed the status string, since "fixed" is the only one supported... [13:18] double-clicking doesn't work [13:19] schierbeck: I'd rather keep it in there and make the message say something like "This revision changes the affects one or more bugs." === mrevell-lunch is now known as mrevell [13:19] jelmer: the status message? [13:20] i think we should only do that when bzr supports more than one type [13:20] yeah, or otherwise we should be making sure the status message == "fixed" [13:20] i could do that [13:20] otherwise if bzr supports more than one type in the future, users will get very confusing errors [13:22] jelmer: okay [13:23] jelmer: i think the bugs tab should always be visible [13:24] it's pretty confusing when ui elements magically appear on the screen [13:26] I'd be fine with having it insensitive [13:26] if there are no bugs [13:26] but one of the nice things now is that you don't actually have to switch to the bugs tab to see if a revision touches any bugs [13:26] okay [13:26] jelmer: the same thing applies for signatures [13:27] but if the user is not able to open the tab, he may think that he needs to install something [13:30] schierbeck: Alternatively, we could show an icon in the branchview to indicate a revision touches bugs [13:30] jelmer: just my thought [13:31] i just need a smaller icon then [13:31] currently i just copy-paste from the shared icons i've got lying around... [13:31] hope i don't get in any trouble [13:32] schierbeck: yeah, we need to check what the license on those icons is.. [13:33] i'll get on it before we release, but most likely it won't be an issue [13:34] are there any documents on how to solve permission problems with shared repositories? i have a group stbaz that can commit to the shared repository, but it seems that after bzr push'es or bzr commits from a lightweight repos using sftp [13:34] some files get owned by my user [13:34] both as a user and as a group [13:35] Hi, I was just wondering, does BZR work with Kerberos? [13:36] it doesn't have specific kerberos support, afaik, but it would work with kerberised ssh [13:37] yeah, I used it with kerberized ssh without problems [13:38] kerberized ftp is not supported yet though (kerberized sftp should be fine) [13:39] ignas: have you set g+s on the directories? [13:40] yes, it seems so [13:40] let me check [13:40] jelmer: okay, i've pushed the changes to lp [13:41] ignas: there's also a limitation of sftp in this area, bzr+ssh will serve you better. [13:41] james_w: oh, i see, do i need to do something special? or just replace sftp with bzr+ssh and that's it? [13:42] ignas: you need bzr installed on the server [13:42] but otherwise probably not [13:42] i see [13:42] (the exception would be if you had bzr installed somewhere outside the $PATH on the server) [13:42] it's in $PATH [13:43] by the way - is there a way to get something like a lightweight checkout [13:43] that would not need network connection [13:43] for bzr status or bzr diff [13:43] bzr checkout --lightweight [13:43] i am using --lightweight [13:44] but every operation like bzr st or bzr diff seems to be connecting to the server [13:44] oh, not need network connection, sorry. [13:44] that's the 'lightweight' - most stuff is on the server [13:44] s/most/all/. [13:44] you can use a normal, non lightweight, checkout. [13:44] well - i want a compromise [13:45] something that is not 70mb [13:45] and does not need network to tell if i have changed a file [13:45] like with svn ;) [13:45] that puts all of the data on your local disk, but pushes it to the server when you commit, so you only need network access for write operations. [13:45] ignas: There is nothing like that yet. [13:46] i see [13:46] :/ [13:46] ignas: All or nothing. [13:46] ouch [13:46] how far along are lifeless's shallow branches? [13:46] * Peng points out that 70 MB isn't huge. [13:46] bob2: quite close I think, it's just a case of making everything "stackable-on" [13:47] the smart server for one thing I think [13:47] ah [13:47] hopefully 1.4 then [13:48] schierbeck: I think the message in the bugs tab should say "This revision fixes one or more bugs." since we're not showing the status anymore [13:48] bob2: They aren't shallow branches. [13:49] jelmer: i'm editing it as we speak :) [13:49] abentley: oh [13:49] They're not intended to allow you to work when the source branch is unavailable. [13:50] ah [13:50] jelmer: should we use "fixes" or "claims to fix"? [13:50] Stacked branches are mostly a storage optimization. [13:51] sounds about equivalent to co's bound to a shared repo? [13:52] jelmer: for now i'll go with "claims to fix", as we don't really know [13:54] Well, stacked could let you create a checkout of emacs without downloading all its revisions. And diff/status would be zippy. But log would not be. [13:56] ah, of course [14:04] schierbeck: yeah, claims to fix sounds fine [14:06] In a true shallow checkout, non-local revisions would be handled as ghosts, so log would be speedy but short. [14:07] Is "bzr branch" identical to "bzr init" + "bzr pull"? [14:11] re [14:12] bronger: I wouldn't think so, but I could be wrong [14:21] hello [14:21] The background of my question is: If I want to update a remote SVN repo, I can create a branch of it locally, merge the new changes and pull it back. Would it be different in any way if I pulled it to an empty revisioned directory first? [14:22] Sorry "... push it back" [14:28] what is a "submit branch"? [14:28] is it another name for a "push branch"? [14:31] gioele: It may also mean a commit in the branch is bound. [14:32] "... if the branch is bound" === kiko-zzz is now known as kiko [14:33] bronger: bzr info in a bound branch (a checkout) says [14:33] Location: [14:33] light checkout root: . [14:33] checkout of branch: http://... [14:33] is that the submit branch? [14:35] bronger: init + pull is the same as branch, yes. [14:36] ah, the submit branch is the location where "bzr send" sends changes [14:43] james_w: Thank you! [14:45] is there a revspec with the meaning "last time it was changed"? [14:45] For example, current revision is 20 and file foo has been modified at revision 12 and 8. Is there a revspec that I can use instead of 12 and 8 in "bzr diff -r 8..12 foo" [14:46] That would be useful, but I don't know of one. [14:46] The docs list all the types of revspecs. [14:46] * Peng goes back to bed. [14:47] jelmer: ping [14:47] schierbeck: pong [14:47] jelmer: do you think we should rename the "Signature" tab to "Authenticity"? [14:47] gioele: I don't think there is one. [14:47] after all, that's really all it's about [14:49] schierbeck: We need to verify the testament first [14:49] schierbeck: but otherwise, sounds fine to me [14:51] jelmer: yeah, i was gonna ask that, too [15:10] Hi, guys! [15:11] morning [15:11] I was merging the changes from another branch yesterday (both local FS branches) and I got an error and don't know if It's a bug in bazaar [15:12] I had made a post in linuxquestions but nobody replyed to it, that's why I looked if there was an IRC channel... and so I'm here. [15:13] antoranz: link? [15:13] the post is here: http://www.linuxquestions.org/questions/linux-software-2/problem-with-bazaar-632262/ [15:13] in future, I'd say that posting to the list is a lot more likely to find someone who knows bzr thatn linuxquestions :) [15:13] list = mailing list on bazaar-vcs.org [15:14] k [15:14] someone reported almost that exact error yesterday [15:14] bzr in gutsy? [15:14] mmm........... yes, it's gutsy [15:16] I think this deserves a bug at https://bugs.launchpad.net/bzr/+filebug === mw|out is now known as mw [15:17] there's already a bug for it I think [15:18] try upgrading (add 'deb http://ppa.launchpad.net/bzr/ubuntu gutsy main' to /etc/apt/sources.list, 'sudo aptitude update', 'sudo aptitude dist-upgrade') [15:18] I don't think it's fixed [15:18] ah, ok [15:19] I forget the conversation yesterday, but did -q workaround? [15:19] what's the problem? My use of "á" in the directory name or something? [15:19] cause I haven't had a problem so far with it in the branch where I created it committing to it. [15:20] is that repository stable? [15:21] bob2: I think this is a different location to yesterday's [15:22] https://bugs.edge.launchpad.net/bzr/+bug/135320 [15:22] Launchpad bug 135320 in bzr "bzr merge - exceptions.UnicodeDecodeError" [Undecided,New] [15:23] I'll append my info to that bug. [15:28] The bug a little old though [15:29] anyway... I've already appended my trace to it. [15:30] How do you debug bzr plugins? [15:30] Is there a way to make the plugin hook the eric debugger at a certain point? Any other IDE is OK [15:31] I'll try changing the name of the directory to see what happens [15:32] F***! It failed when I tried to commit the mv [15:33] all I did was: $ bzr mv doc/Cities/Bogotá/ doc/Cities/Bogota [15:38] Peng: by the way - if 70 mb is not much, why is it taking ~30 minutes or more tu make a branch :/ how long does it take with "much" ;) [15:39] antoranz: can you pastebin the traceback you get for that please? [15:40] gioele: what I normally do is edit the source and add "import pdb; pdb.set_trace()" where I want it [15:40] if the problem is a plugin not loading then ~/.bzr.log will have the error. [15:40] sure [15:41] james_w: is there an API documentatio for bzrlib? [15:41] I'm filling for a new bug including both bugs in https://bugs.launchpad.net/bzr/+filebug. Want to see it? [15:41] gioele: sort of yes [15:42] Here it is: https://bugs.launchpad.net/bzr/+bug/210838 [15:42] Launchpad bug 210838 in bzr "Problems with non-ascii characteres in names" [Undecided,New] [15:42] http://starship.python.net/crew/mwh/bzrlibapi/ [15:42] that was fast! :-D [15:42] there's also http://bazaar-vcs.org/Integrating_with_Bazaar [15:43] antoranz: he responds to what you say [15:43] they'll be the new bug notification in a minute or two [15:44] ignas: btw, cp -r is a valid way to branch a standalone branch to anothoer standalone branch [15:45] bob2: i know, but cp -r does not work if your branch is on launchpad [15:45] ah [15:45] ignas: that was good! [15:51] New bug: #210838 in bzr "Problems with non-ascii characteres in names" [Undecided,New] https://launchpad.net/bugs/210838 [15:55] I'll try the repository you provided mye with... hold on to your butts. :-) [15:58] hell! I tried to commit just like that and it "added" the directory (didn't move it) [15:59] I just checked the repository and I have 5 inconsistent parents [16:00] how do I get rid of those "parents"? [16:05] jelmer: what do you think about a "Tags" page in the revision view? i think it would boost the discoverability. [16:05] i know they belong in the branch, but to the user who wants to add a tag to a revision, it's natural to select that revision in the viz and go from there. [16:06] schierbeck: tags are already shown in the revisionview, in the main tab [16:06] jelmer: yeah, but i'm thinking more like a treeview, where you can "rename" and add tags [16:07] can I delete .bzr/branch/lock so I get rid of the lock on the repository? [16:07] jelmer: the current ui doesn't allow for deleting tags in the revisionview [16:07] I've already done it, so I think it's safe, right? [16:08] schierbeck: yeah, makes sense [16:08] jelmer: let's get the bugs and signature page branches merged first, then i'll refactor the revision view and go from there [16:09] antoranz: You're supposed to use the bread-lock command. [16:09] jelmer: do you think there's anything missing from those two branches? i've already sent a mail to the tag icon artist [16:10] I've should know better, wasn0't I? after I removed the lock (by deleting, by the way :(), I was able to commit into the stable branch even with the names with non-ascii characters [16:10] my godness... what a gramatical mess did I write! :-D [16:10] schierbeck: the signatures branch disables the signature tab [16:10] schierbeck: but that still says "this revision has one or more signatuers" [16:11] schierbeck: huh? [16:11] schierbeck: that's all I can think of right now [16:11] jelmer: the seahorse-integration branch? [16:11] i think that's the one we should ship [16:17] I think my repository got completely busted (both the stable and the development one... as a matter of fact, I deleted the developemt one and can't branch from the stable) [16:18] I just noticed I lost quite some hours of work goys [16:20] say-.... 2 days of work [16:21] hi, can i get bzr serve to prompt clients for a password? [16:23] there's something strange going on here. [16:24] I was able to commit the changes brought from the development branch in the stable branch.... I can see the diffs applied [16:25] however, there are not "efective" in the physical files of the stable repository [16:25] they, I mean [16:46] jelmer: okay, i've sent in the merge requests [16:47] once they're approved, i'll start refactoring the revision view [16:48] schierbeck: olive? gtk? [16:49] gioele: bzr-gtk [16:49] the bottom part of the viz [16:50] I love viz [16:51] How do I get the arguments passed to bzr from my RevisionSpec? [16:52] why is bzr push to a remote repository is a mega pain? and what should I do instead? [16:53] unter linux siehts auf jeden fall anders aus [16:53] fc [16:53] i checkout the "trunk", checkout the branch, bzr merge the branch into the trunk, and am trying to bzr push the mainline back so everyone (especially our mega release tools) would see the changes... [16:53] bzr push seems to be taking ages [16:54] even though there was only 1 small change made to the trunk [16:58] are they the same branch formats (should be unless you use a shared repo) [16:58] hmm, ok it seems that it was taking so long because it was synchronizing the fact that i have enabled tags on the remote repository [16:58] i am using a shared repository [16:59] for my remote branch [16:59] because all the release branches will be stored in the same location [16:59] but yes, now that the first push got through it is bearable (takes only 10-20 seconds) [17:02] what can I do about "Value "bzr+ssh://schooltool.org/var/local/bzr/schooltool/lyceum/trunk/" is masked by "bzr+ssh://bazaar.launchpad.net/%7Eignas/schooltool/lyceum/" from locations.conf"? [17:03] and where is that locations.conf thing? [17:03] i have tried doing "bzr push --remember bzr+ssh://schooltool.org/var/local/bzr/schooltool/lyceum/trunk/" === kiko is now known as kiko-afk [17:06] ~/.bazaar/locations.conf [17:07] * gioele wonders when bzr will conform to XDG Base Dir spec [17:08] emm, why is it there? and why is it overriding parameters i am passing through command line? [17:08] i mean - maybe there is a reason, and i am doing something wrong... [17:08] that file is used for the parameters that you didn't specify on the cmd line [17:09] so when i do "bzr info" [17:09] paths are taken from that file [17:09] not from some place in .bzr ? [17:09] ignas: old branch formats used to save it to locations.conf, new formats use .bzr/branch/branch.conf [17:09] oh, i see [17:09] so i should upgrade my branch and remove that file? [17:10] if it was complaining about masked location, your branch is probably already upgraded [17:10] you can just remove the branch from locations.conf [17:11] ok [17:24] jelmer: ping [17:24] schierbeck: hi [17:25] jelmer: i've got permission to use the tag icon [17:25] cool [17:25] can i get you to review the merge request? [17:25] yeah, just give me a few minutes [17:25] okay [17:27] how do i convert a branch into a checkout? [17:28] ignas: bzr help bind [17:29] schierbeck: thanks === schierbeck is now known as schierbeck_afk === Peng__ is now known as Peng_ [18:21] how do I "un-add" files? [18:26] revert [18:26] antoranz: ah! thanx! [18:26] if you haven't commited [18:27] antoranz: no, I haven't. :) [18:28] there you go then. === weigon_ is now known as weigon [19:00] hello [19:01] does the mailing list accept mails from Gmane? [19:16] New bug: #210938 in bzr "bzr status -v should show also non-modified files" [Undecided,New] https://launchpad.net/bugs/210938 [19:29] gioele: I don't know. The list is moderated, so it might be that. [19:36] james_w: no, it passed fine. Maybe Gmane is a bit slow today === mw is now known as mw|food [19:39] ah, ok [20:02] hi, I have question about merge.py [20:05] abentley, may I ask you about merge.py? [20:05] Sure. [20:06] there is Merger, Merge3Merger and others [20:06] it seems like Merger is main of them [20:06] it's correct? [20:06] Yes. It's been suggested that Merger should be renamed to MergeController. [20:07] so, it's always created Merger instance for merge operations? [20:08] Yes. It's not required, but I believe all callers do go through Merger. [20:08] my actual question is: where I should put my code for support merge of file properties === mw|food is now known as mw [20:09] is it Merger correct place, or Merge3Merger? [20:09] Probably in Merge3Merger. [20:09] i.e. actual merge done in Merge3Merger, right? [20:10] Yes, the actual operation is handled by Merge3Merger or a subclass. [20:11] all other merge types is subclased Merge3 as I could see. [20:11] Right. [20:11] okay, thank you. [20:12] np === kiko-afk is now known as kiko [20:17] moin [20:19] eddyMul: More specifically, you use "bzr remove --keep --new". That will avoid reverting any other changes. [20:23] abentley: I used `bzr revert specific_files`, and it worked for me. [20:24] abentley: judging from `bzr help remove`, it appears like `bzr remove --keep --new` is the like the "anti" of `bzr add`. Very handy. Thanx! [20:24] np [20:31] hi bialix, Verterok [20:32] hi james_w [20:32] bialix: sorry I haven't had a chance to look over your file properties mail yet [20:33] it seems like nobody have this chance yet [20:33] there was a lot of people who cry without line-endings support [20:33] james_w: hi [20:33] bialix: yeah, it would be great to have. [20:34] they already worked [20:36] I need only to beat merge and to teach it write files to disk with proper eol [20:39] abentley: Merge3Merger.__init__: has arguments working_tree, this_tree, base_tree, other_tree -- what's difference between working_tree and this_tree? it's not the same? [20:41] this_tree is the input tree that you read from in order to calculate the merge. working_tree is the output tree that you write the changes to. [20:41] They are commonly the same, but this_tree is sometimes a RevisionTree. [20:42] in which case? [20:46] I see in constructor's code: self.this_tree = working_tree [20:46] and it seems like this_tree argument is not used actually [20:46] so I'll ignore it. [20:49] Don't be a jerk. [20:55] abentley: as far as I can see the "this_tree" argument to Merge3Merger is not used in Merge3Merger.__init__, and none of the subclasses in the same file override __init__. [20:55] Why can't I bzr mv from one branch in a repository to another? [20:56] so is it just there in case a subclass wants to? [20:56] db-keen: because bzr is not like subversion? [20:56] james_w: WeaveMerger used to do that, and future mergers may also do that. It's quite likely to be a performance win. [20:56] ah, ok, thanks === Toksyuryel` is now known as Toksyuryel [20:56] db-keen: well it's more complicated for us than for svn [20:56] we should make it possible [20:56] is there a way to have "external" branches inside a branch? [20:57] abentley: I think your reaction might have been a bit strong then, but maybe I misread it [20:57] I'm trying to take a project and split it into smaller branches, what's the best way to do this? [20:58] db-keen: are you splitting on directory boundaries? [20:58] james_w: If writes to this_tree or reads from working_tree, I'll veto his patch so fast his teeth will shake. [20:58] my initial reaction was to create new branches and move stuff from the old large one into these new small ones, but obviously that doesn't work [20:58] james_w: mostly [20:59] db-keen: there's a split command that may help [20:59] abentley: but the default mergers read from working_tree? [20:59] No [20:59] They read from this_tree [21:00] abentley: they read from self.this_tree, which working_tree is assigned to in the constructor [21:01] which is entirely different. [21:02] thumper: hi, are we meeting now? [21:02] poolie: yeah [21:02] abentley: well, I don't still fully understand, even with the extra discussion, so I still think your response was too strong. [21:06] james_w: think what you like. When people ask me for advice, I expect them not to ignore it. [21:08] james_w: I guess that the API/contract is that you read from this_tree, without caring what gets assigned to it before. or so. from my limited understanding. :) [21:08] They may decide that they don't agree, or that it doesn't apply. But it's not okay if they just ignore it. [21:09] I read his statement as ignoring this_tree, just like the existing code does, not ignoring your answer, but I don't know what he meant [21:10] The existing code does not ignore this_tree. [21:14] maybe I am wrong to say ignore, but the this_tree parameter passed to Merge3Merge.__init__ is unused as far as I can see. This may well be because working_tree fulfils the contract of this code just as well as this_tree. [21:20] We're not talking about constructors, so when you say this_tree, I assume you mean self.this_tree [21:23] sorry for the misunderstanding then, I tried to be clear that I was talking about this_tree passed to the constructor as opposed to self.this_tree [21:25] So it looks like the Merge3Merge constructor is a bit broken. [21:26] Regarding the "fix botched log-message" discussion: Does any of the storage mechanisms actually have an API for "turn revision X into a ghost"? [21:26] It's probably due to the shuffling around we did to enable merge previews. [21:26] hmeland_: No, all of our formats are append-only. [21:27] You would need to create a new repository, and not add those revisions. [21:27] Right. [21:32] In principle you could repair revision X by adding a X' (which has X and the parent(s) of X as its parents) with the fix, convert X into a ghost, and repeat for all revisions that are descendants of X. [21:33] But without an easy way to do the "convert X into a ghost" step, that sounds somewhat cumbersome. [21:35] The approach might also have bad (performance) implications for e.g. merge, especially if such fixing were to be as common as I gather git's "commit --amend" is. [21:37] "commit --amend" only works for the tip of the branch, and is another form of rebase. [21:37] Ah, good. [21:38] I think there are two cases being discussed, one is the "nuclear launch codes" and the "abusive commit message" case, and the other is the "spelling mistake case" [21:39] "abusive commit message"? [21:39] the latter could be solved by just adding a note somewhere saying "when displaying it, set the message of X to Y" [21:39] dato: it's an example from the list of where there was evidence of sexual harrasment in a commit message. [21:39] oh, wow, I missed that [21:40] james_w: I think *abusive* commit message is a form of nuclear launch codes, actually. [21:40] hmeland_: yes, it is, but I was just using the two examples from the list discussion. [21:40] OTOH, an *incorrect* commit message might be solvable in the way you describe. [21:41] for the former you need to remove X from history so that it is unreachable. [21:41] so this could either be a rebase type operation, or introducing a ghost, both of which have their issues. [21:42] Yes. Both NLC and ACM goes in the former category, to my mind. [21:43] The nice thing about the ghost-introduction solution is that other existing branches would still be related to the original branch. [21:43] exactly [21:44] You might have to take care that merges from any such branches didn't re-introduce the ghosted revisions, though. [21:44] however, currently ghosts have no information whatsoever, not even their parents, which means that you can't get at the history previous to the ghost. [21:45] and it would also be information disclosure, which some people in some circumstances wouldn't accept. === schierbeck_afk is now known as schierbeck [21:46] If you branched from revision Y, which has now been converted into a ghost, there will be a new revision Y' in the source branch, whose mainline history will take you all the way back to before the fixed revisions. [21:47] hmeland_: no, when you branch you are based of Y still [21:47] it's not like svn where you have to make a commit to start a branch. [21:47] but you are right, I think care would have to be taken not to get unwanted ghosts again when merging. [21:47] Yes, that's what I'm saying. In the *source* branch there will be a Y'. [21:48] hmeland_: ah, in your proposal for creating Y' from Y, sorry [21:49] there would be the peverse case where you made Y a plain ghost, as now you can't do 3 way merging as you can't get the text from the base revision. [21:52] hmeland_: the conversion of X's descendants to X''s descendants will not propogate [21:52] james_w: Right, you'd have to pull before merging, I think. [21:53] lifeless: In the case of multiple fixes, you mean? [21:54] X' will have (X + parents(X)) as parents. X'' will have (X' + parents(X')) as parents. [21:54] hmeland_: you can't pull before merging, if you have diverged then pulling is not allowed. [21:54] hi lifeless [21:54] hi james_w [21:55] hmeland_: no, I mean in the case of having more than one repository which needs the correction done [21:55] hmeland_: all other repositories will have the original X as their semi-ultimate descendant [22:00] Not X, but the tip of the branch X was in when X was fixed, I think? [22:00] But yes, ghosts do not propagate. [22:04] jelmer: ping [22:04] schierbeck: pong [22:05] I've fixed the remaining issues with the Seahorse integration branch, and I've sent in an updated merge request. Could I get you to take a look at it? [22:11] hmeland_: and neither does the conversion to 'fixed' revisions, only the ones actually fixed will propogate [22:15] wow, apparently you can be allergic to electricity... [22:15] that's pretty f****d up! [22:17] schierbeck: Iyou don't appear to be catching the errors if there is no seahorse interface [22:19] jelmer: have you tested it? i check for the seahorse service name [22:19] if seahorse is installed, the object paths and interfaces should be there [22:19] it worked when i uninstalled seahorse... [22:20] lifeless: Assuming a mainline-only history: If you actually fix revision 2 by introducing 2' (with 2 (now ghosted) and 1 as parents), and "fix" revision 3 by introducing 3' (with 3 (now ghosted) and 2' as parents), wouldn't a branch at e.g. 3 doing a pull cause both 2' and 3' to propagate? [22:20] schierbeck: ah, I missed that - sorry [22:20] jelmer: :) [22:20] schierbeck: any chance you can make two more changes? [22:21] schierbeck: I'll reply to the email [22:21] jelmer: which? [22:21] ok [22:27] hmeland_: a new branch from 3' will take 3' of course. But *all* the other existing branches won't be fixed. [22:28] I think the suggestion is to mark some ghosts as propagating, so the next merge or pull will fix them [22:30] james_w: That's not what I'm suggesting; I'm just trying to understand if the algorithm I suggested above might be a viable starting point for the requested functionality. [22:31] ah, ok, sorry [22:31] the general thing here is that revisions are immutable [22:32] having efficient, safe trustable mutable revisions is possible, buts its a lot of complexity [22:32] you basically need a meta-timeline for versioning revisions too [22:32] If you the requires-fixing revision is present in branches you don't control, I'm leery of trying to propagate the fix to them automatically. [22:37] lifeless: Yeah. So I'm trying to see whether the "convert to ghost and add alternative timeline" approach could get us at least some of the benefits without all the complexity. [22:40] Anyway, thanks for the input, I guess I'll sleep on it and see if I can come up with something. [22:52] 'lo #bzr [22:55] hi [22:55] jelmer: okay, i've sent in yet another iteration [22:56] i've also contacted the author of the bug icon [23:02] schierbeck: ah, thanks [23:02] schierbeck: (wasn't strictly necessary btw, I voted bb:tweak) [23:04] is it me or the proportion for BB to be unavailable is increasing day after day ? [23:05] it's been months since I've seen bundlebuggy downtime for bzr.dev [23:14] morning [23:15] Hmmm, does anyone maintain a bzr branch of the current Python code? [23:15] I'm not completly sure if branching the svn URL would be a very enjoyable experience. [23:17] yacc: it's not too bad with 0.4.9 [23:20] yacc: you mean CPython? I'm pretty sure I've heard others talking about such a thing, but I'm not sure where. [23:20] yacc: you mean like http://www.python.org/dev/bazaar/ ? [23:25] jelmer: you are trying to sell me on going back to using a released version of bzr-svn ;) [23:26] yacc: :-) [23:26] yacc: If performance or reliability matters to you, you should be using a release [23:26] performance certainly matters for the python repo, given its size [23:26] jelmer: does it support bzr 1.3? [23:26] yacc: yep [23:27] jelmer: And I ended up using a trunk scheme, so I guess this should work too? [23:27] yacc: http://www.python.org/dev/bazaar/ [23:28] yacc: yep [23:28] oh, oops, too slow [23:37] bob2: well, maybe he'll tell _you_ why that isn't what he wants :) [23:45] bob2: what's too slow?