/srv/irclogs.ubuntu.com/2009/02/19/#bzr.txt

Kobazwhoops00:18
Kobazi broke bze00:18
Kobazbzr00:18
=== Iso_ is now known as IsoSchiz
Kobazhttp://pastebin.com/m3850ffa200:19
spivKobaz: you upgraded bzr, but have an old bzrtools shelf00:20
spivKobaz: you can use "bzr shelve1"/"bzr unshelve1" to keep using the bzrtools shelf, IIRC.00:20
Kobazoh00:20
Kobazshould probably be a warning for that or something00:21
Kobazii  bzr                                  1.10-1~bazaar1~intrepid1            easy to use distributed version control syst00:22
Kobazii  bzrtools                             1.10-1~bazaar1~intrepid1            Collection of tools for bzr00:22
Kobazboth are the same version00:22
spivRight; what happened was that bzr shelve became part of the core, rather than provided only by a plugin.00:23
Kobazk00:23
spivBut the implementation in the core is new and improved compared to the bzrtools one :)00:23
Kobazk00:23
Kobazso how would i use the new one?00:23
spiv"bzr shelve", in theory.00:24
Kobazhmm00:24
Kobazbut that is what croaked00:24
spivI'm not sure why it's breaking for you, there seems to be a broken shelve file there.00:24
spivMy guess is it's a bzrtools shelf, but that's just a guess.00:25
spivI could easily be wrong about that...00:25
spivPlease file a bug.  At a minimum it shouldn't give a traceback just because of a damaged file.00:25
Kobazk00:26
Odd_BlokeThere already exists a bug for that.00:31
Odd_BlokeBug 303569.00:32
ubottuLaunchpad bug 303569 in bzr "shelving a patch with no new line at end of file fails" [Undecided,Confirmed] https://launchpad.net/bugs/30356900:32
Odd_BlokeKobaz: ^00:33
lifelessspiv: ping00:33
IsoSchizHi, I was wondering if someone could help. I am attempting to create a bzr repository from an svn one. I'm running Intrepid (bzr 1.6.1-1, bzr-svn 0.4.13-2) and used the following command: "bzr svn-import --all --scheme=trunk --prefix=endets/ svn+ssh://<hostname>/var/repository endets". It seems to work initially (build a cache, then 'Importing branches with prefix /endets/', which takes a while), but it appears to stop abruptly a00:34
lifelessIsoSchiz: your coment cuts off at 'abruptly a'00:34
IsoSchizDarn it. :-) Ok, after that it goes:00:34
lifelessrather ironic I thought00:35
IsoSchizbut it appears to stop abruptly and gives the following error when I try to co or branch from it: 'bzr: ERROR: Not a branch: "/home/martin/bzr/endets/.bzr/branch/".' What's gone wrong? :-)00:35
IsoSchizYes, it is rather..00:35
lifelessIsoSchiz: svn-import creates a set of branches00:35
lifelessso run 'bzr branches endets' and you should get the relative url of the branches it made00:35
spivlifeless: pong00:35
IsoSchizThat gives no output at all...00:36
lifelessspiv: I want to chat [irc] briefly about a refactoring00:36
lifelessIsoSchiz: so, it sounds like it may have errored/failed - what return code did it give? Is there anything in ~/.bzr.log00:36
spivlifeless: btw, I discovered https://edge.launchpad.net/compiz-colorfilter.  I wonder if it could be used for non-colour-blind people to simulate e.g. red-green blindness?00:38
IsoSchizThe svn-import command ends with: 202.125  return code 0 - I haven't checked that's actually the return code though.00:38
spivlifeless: fire away00:38
lifelessspiv: ok, so I'm going to tackle the stuff we discussed just-in-time, working on the round trips for that blackbox test00:38
lifelessspiv: format.intialize_on_transport() is the code called to make a .bzr dir00:39
lifelessspiv: MetaDirFormat is the concrete class used00:39
lifelessspiv: so I'm thinking in MDF' initialize_on_transport can do a check to see if there is a remote capable transport00:40
lifelessnow the question00:41
lifelessat this point I could create a RemoteBzrDirFormat and pass on the format details to it00:42
lifelessand then call that new instance's initialize_on_transport00:42
lifelessor I could do something different00:42
lifelessmaking a RemoteBDF seems appropriate to me00:43
Odd_BlokeShould the Windows installers work on 64-bit Windows?00:44
spivHmm.  So asking a MDF to initialize_on_transport might actually give you a Remote thing instead?00:44
spivThat feels a bit weird, but I don't know of anything that would actually break.00:44
lifelessspiv: ok00:45
lifelessspiv: do you know of a good place to put a test that this happens?00:45
IsoSchizlifeless: Re-ran the import: it returns 0. The log file claims it returns 0, and looks fine otherwise (though I am unsure what to expect). bzr branches gives no output, but returns 0 too.00:45
lifelessIsoSchiz: are there any subdirs in the output?00:45
spivHmm.  I suppose you could put that test in test_remote.  Although I'd probably expect to find it in a test_bzrdir_format module -- if we had one ;)00:46
IsoSchizlifeless: Only the .bzr, which contains branch-format, branch-lock/, README, repository/ and svn-import-revision00:46
lifelessspiv: we have that00:46
lifelessIsoSchiz: sounds like its not finding any branches to import00:47
spivWell there's TestBzrDirFormat in test_bzrdir, yeah00:47
lifelessIsoSchiz: have you tried the simpler 'bzr svn-impoprt svn+ssh://<hostname>/var/repository outputdir'00:47
spivI'd probably lean towards adding it there, I think.00:47
IsoSchizlifeless: I am just trying that now :-)00:48
IsoSchizlifeless: slightly more exciting: bzr branches now prints a blank line. :-)00:51
IsoSchizlifeless: I don't quite understand why it would find *no* branches. Surely at least the trunk would become a branch (even if it doesn't correctly identify the copies in the 'branches' directory as actual branches)?00:52
lifelessIsoSchiz: you might like to try bzr-svn 0.5 which has lots of improvements, you need a newer bzr at the same time too00:53
IsoSchizlifeless: Is there a PPA containing these for Intrepid? Or do I need to install manually?00:55
lifelessIsoSchiz: yes00:57
lifelessIsoSchiz: http://bazaar-vcs.org/Download00:57
baxissimoQ about this tutorial: http://doc.bazaar-vcs.org/bzr.dev/en/tutorials/centralized_workflow.html00:58
baxissimoIt says create a new branch by first doing "bzr init  REMOTE/newbranch" then "bzr checkout REMOTE/newbranch" locally.01:00
baxissimoSeems like it's also legit to do "bzr init LOCAL/newbranch" and later "bzr branch LOCAL/newbranch REMOTE/newbrach"01:00
bob2yes01:01
baxissimoSound ok?01:01
bob2+ bzr bind REMOTE/newbranch01:01
baxissimobob2: Ok, yeh, bind if you want automatic pushes.01:01
bob2if you don't do bind, they do NOT do the same thing01:02
baxissimobob2: ok.  I see.01:02
bob2but both are reasonable ways to use it01:02
baxissimobob2: and if you don't bind, then "bzr send" will do the equvalent of "hg push"01:03
baxissimobob2: ?   (if you know hg at all...)01:04
bob2no idea what hg push does01:04
bob2bzr send will generate a bundle (ie patch with metadata) and dump it in a file or open your email client01:04
baxissimoIt send revisions in the local branch that aren't in the remote branch to the remote branch.01:05
bob2bzr push will push changes back to the remote branch01:05
baxissimoOhh.. ok yeh that's the equiv of "hg push"01:05
baxissimobzr send is I guess analogous to "hg bundle", though sounds like it does a bit more.01:06
baxissimohg bundle just doesn't have the --mail-to option.01:08
baxissimobob2: thanks!01:08
lifelessspiv: I'm moving the smart call log to TestCaseWithMemoryTranspirt01:10
IsoSchizlifeless: newer versions don't work either. :-( Without the --prefix arg, 'bzr branches' produces blank line; with --prefix gives the error: bzr: ERROR: The specified path is inside a branch. Specify a different URL or a different repository layout.01:13
spivlifeless: +101:13
spivlifeless: I've been increasingly tempted to add something like "start a smart server backed by self.get_url(...)" to TestCaseWithMemoryTransport too01:15
lifelessIsoSchiz: I suggest filing a question on bzr-svn01:17
IsoSchizlifeless: never done that before. Is this some feature of Launchpad? Perhaps the 'Answers' section...?01:19
IsoSchizOk, worked that out, ignore the last question.01:20
IsoSchizThanks for your help lifeless.01:21
lifelessIsoSchiz: np01:23
lifelessspiv: so, its 15 round trips to init a bzr meta dir remotely01:30
lifelessspiv: hows the polish on that branch coming01:30
lifelessspiv: 15 -> 2 for initialize_on_format01:59
mwhudsonthat sounds like a nice improvement :)02:01
lifelessmwhudson: cheap too, server already has the RPC02:07
lifelessit's use got lost during stacking cloning behaviour rearrangments02:08
mwhudsonoh right02:08
lifelessbundle sent02:09
jelmerlifeless: thanks!02:15
jelmerlifeless: you forgot to mention that this also fixes initial push in a format unknown to the remote server02:15
jelmerwhich I think has a bug or two open02:15
lifelessjelmer: I don't think it alters that case at all02:16
jelmerlifeless: not completely, no02:18
jelmerlifeless: but at least you don't end up with a unusable branch on the remote side02:18
jelmeroh, you do02:19
jelmerit just gives the warning earlier02:19
jelmernever mind me02:19
lifelessjelmer: :P02:23
jelmerI guess I was just too eager to see that bug fixed...02:23
lifeless:)02:25
* igc lunch02:43
=== abentley1 is now known as abentley
lifelessso jelmer03:20
lifelessif you read my patch, you could bb:approve it :)03:20
pooliei'm going to push that EC2 build and its docs along a bit further03:22
lifeless40 calls to do init-repo url03:33
pooliedown from 100?03:45
lifelessno03:45
lifelesstested right now as a reasonable component to measure while working on push03:45
lifelessspiv: ping; a test for a new verb - its test_remote for the client, test_smart for the request method backend?03:54
pooliejam, if you're still up, which i suppose you're not, i'm going to mess with the ec2 machine now03:54
spivlifeless: that's right04:04
spivlifeless: nice results for the RPC count04:04
spivlifeless: I just fired off external_references / scan_unvalidated_index patch.04:04
=== abentley1 is now known as abentley
lifelessspiv: ping; it seems really hard to write a test for this using FakeClient, because of all 40 calls I need to emulate for backwards compatibility04:35
lifelessspiv: I'd be happier using a hook to capture the calls and assert that it tried a newer one, and worked.04:35
lifelessspiv: assuming there is some way to say 'give me a old server'04:36
lifelessspiv: -> is there?04:36
spivAn old server?  bzr-v2:// ?04:39
lifelesslet me pastebin my 'old servers work test' one sec04:40
spiv(The client code usually knows better than to try new methods if the protocol encoding is only v2)04:40
spivYou could also modify the request_handlers dict buried in the server instance somewhere...04:42
lifelessspiv: uhm04:44
lifelessso I don't really care if its a really-old-server04:44
lifelessor just the server-before-I-write-this-method04:44
lifelesshttp://paste.ubuntu.com/119949/04:45
lifelessI want to be able to have that be the test04:45
lifelessbecause it seems clear and complete, if we can be sure the server doesn't support the verb04:45
lifelessthoughts/suggestions?04:46
spivTestRemotePackRepositoryAutoPack.test_backwards_compatibility does a small monkey-patch to achieve something a bit similar04:47
spivI don't quite follow the intent of that pastebinned test, though.04:48
spivWhy is asserting that exactly 1 BzrDir.create_repository call a test that the client is backwards compatible with old servers?04:49
lifelesswell04:49
lifelesswe should try the call (most servers will be up to date04:49
lifelessand it should still work :)04:49
lifelesswe shouldn't try more than once04:50
spivOh, the missing bit of that test as written is the bit that makes that call not exist on the server?04:50
lifelessyes04:51
spivIf the assertion could be "there's exactly 1 BzrDir.create_repository call, and it got a no-such-method response", that'd be much clearer.04:51
spivBut the existing logging doesn't lend itself to that...04:51
lifelessdoes the smart server client hook show the result?04:51
lifelessI don't think the precision gained is that important, if we're sure the server can't handle the call04:52
lifelesswe can test that by calling it directly, if we care04:52
spivIt doesn't, but it'd be nice if it did.04:52
spivA brief comment can make it clear enough, we don't have to fall down a rabbit-hole...04:53
spivSo I think ideally you'd be able to modify the server in the test to have no implementation for that method.04:57
spivUnfortunately it's a bit hard to get at the relevant commands dict atm, there's a little more refactoring needed to make that possible.04:57
spivYou could modify the global request_handlers registry, though.04:58
lifelessyeah try:finally for the win04:59
spivYeah.  Or addCleanup, and hide all the mucking about in a helper.05:00
lifelesshttp://paste.ubuntu.com/119961/05:02
lifelesswell05:02
lifelessone line != mucking aorund05:02
spivI count six, but sure.  The test is readable enough.05:04
lifelessspiv: in the finally block, but yeah I see what you mean. Next time, refactor++05:06
* spiv nods05:07
stubI suspect I have a rather unusual problem I want to solve. I have a copy of a CVS tree, and I want to export a single file and its commit history to a bzr branch.05:26
stubIs there any tool available that will import a single RCS style ,v file? I think it contains everything that is needed.05:26
lifelessspiv: test_smart vs test_smart_request05:34
lifelessspiv: the former has everything in it I'd expect to be in the latter05:34
lifelessstub: take a full copy of the CVS tree. delete everything in the module the ,v file is in except the ,v file, and move it to the top of the module05:34
lifelessstub: then use any CVS history importer (such as cvs-fastexport or cscvs or cvsps-import)05:35
spivlifeless: yeah, I can see that. And what's in test_smart_request could stand to be split up a bit too I think.05:36
lifelessspiv: so I'm going to add to test_smart05:36
lifelessspiv: because that has the test for FindRepository05:36
lifelessspiv: also, exception handling05:37
* spiv nods05:37
lifelessyou've been improving that, how should I handle the main failures (no bzr dir present, repository already present)05:37
lifelessor is it still just return an error with a custom string?05:38
stublifeless: Just trying that :-) Got to the point where cvsps-import is spitting out horrible errors, possibly due to me running 1.12? AttributeError: 'MinimalTree' object has no attribute 'get_file_with_stat'05:38
lifelessstub: yes, seems like a version skew issue05:40
lifelessspiv: so errors?05:46
spivlifeless: errors?05:53
lifeless16:33 < lifeless> spiv: also, exception handling05:56
lifeless16:33  * spiv nods05:56
lifeless16:34 < lifeless> you've been improving that, how should I handle the main failures (no bzr dir present, repository already present)05:56
lifeless16:34 < lifeless> or is it still just return an error with a custom string?05:56
spivOh, I totally failed to read a couple of lines of IRC.06:00
spivlifeless: the currently defined errors can all be seen in the function at the bottom of bzrlib/remote.py06:01
spivlifeless: no bzr dir present should be 'nobranch' on the wire IIRC, which mirrors the confusing NotBranchError that bzrlib gives.06:02
spivlifeless: I don't think there's an appropriate error for repository already present.06:05
lifelessI'll skip that case until it turns up then :P06:07
spiv:)06:08
=== montywi|sleep is now known as montywi
vilahi all07:04
* fullermd waves vila around a bit.07:07
pooliehi vila07:12
vilaI'm sure that will make for a good quote but I can't find the right title :-)07:17
lifelessspiv: success07:21
lifelessI'm really slow today :( cotton wool in the head07:21
lifelessbut its done07:21
lifeless19 rpc's less07:24
jfroyso huh07:32
jfroyis there a fast-import importer for Subversion that does support branches and tags and such trivial things :|07:32
Peng_jfroy: Does it have to be fast-import, or would bzr-svn work?07:33
jfroyI want to move away from svn07:34
jfroycompletely and for ever07:34
Peng_Sounds good.07:34
jfroyI guess I could force people to use rich roots.07:34
jfroy(for the particular project in question)07:34
jfroyBut it felt that fast-import was a better fit than bzr-svn for the task of permanently migrating a set of branches.07:34
Peng_Shrug. bzr-svn can do it. It doesn't stop you from moving away from svn forever.07:36
lifelessspiv: bombs away07:37
lifelessjfroy: svn-import is the recommended migration tool :)07:37
lifelessjfroy: if you feel strongly about rich roots you could regenerate the output via fast-export | fast-import, but I wouldn't bother.07:38
jfroyI don't actually know what rich roots are :p07:38
jfroyBeyond the concern of carrying extra data for no reason.07:38
lifelessits essential data we left out of the early formats07:38
lifelessgoing to extra effort not to have it would be... counterproductive07:38
jfroy"--standalone      Create standalone branches." <-- huh07:40
jfroylifeless: if it's essential, why is there still a distinction in the newer formats, e.g. 1.9 versus 1.9-rich-roots07:40
Peng_jfroy: Because converting is slightly inconvenient, so it hasn't been forced on users yet.07:41
lifelessit takes time and effort to convert; data discrepancies can show up so care needs to be taken; abentley is going to work on the overall thing soon I believe07:43
Peng_"data discrepancies"?07:44
jfroyI see07:44
lifelessPeng_: parent mismatches mainly07:44
Peng_You mean it exposes standard "bzr check/reconcile" stuff, or *causes* problems?07:44
lifelessPeng_: exposes07:44
Peng_Okay.07:44
lifelessPeng_: unchecked branches don't always convert completely 100%, even though they should07:45
lifelesswe're regenerating all the inventories after all07:45
Peng_That sounds ominious.07:45
=== thekorn_ is now known as thekorn
lifelessspiv: if you're still doing stuff, network_name could use a review, as could BzrDir.create_repository RPC verb08:03
pooliebug 331416 sucks a bit08:06
ubottuLaunchpad bug 331416 in gdm "gdm fails with "the greeter application appears to be crashing" after upgrading to jaunty" [Undecided,New] https://launchpad.net/bugs/33141608:06
mae^ohnoes! you have a valid .bzr control directory, but not a branch or repository. This is an unsupported configuration. Please move the target directory out of the way and try again.08:10
mae^i did a push that timed out, now I'm getting that error08:11
lifelessspiv: also I can't see the patch you said you sent in08:11
lifelessmae^: right, do what it says :)08:11
mae^what target directory? I'm trying to push to launchpad.08:13
=== serg_ is now known as serg
lifelessgot to launchpad, there is a 'delete branch' button on your branch08:13
mae^hmm.. what if someone else has made changes to the branch that I dont have locally? tough luck?08:15
Peng_If it has no branch or repository, what changes could it be storing?08:16
mae^oh, I found it .. the branch location changed when I changed the branch ownership08:17
mae^thx08:20
beunopoolie, what do you think about adding a space next to the semicolon on the progress bars?08:31
beuno"Build phase: Adding file contents" instead of "Build phase:Adding file contents"08:31
lifelessbeuno: patches :)08:33
beunolifeless, of course. Just wanted to get extra agreement nods  :)08:34
=== montywi is now known as monty|meeting
asabilhi all08:36
asabildoes anyone see the problem with this code: http://bazaar.launchpad.net/~asabil/%2Bjunk/bzr-filter-branch/annotate/head%3A/__init__.py ?08:36
asabilI get the following traceback: http://pastebin.ca/134153508:36
asabilonly when using the plugin inside a rich-root repository08:37
Mezjelmer: using bzr-svn can I just import it to bzr and then "forget" that it was part of svn?08:56
Lo-lan-doMez: Yes.08:57
Mezhow ?08:57
Lo-lan-dorm -r $svn_repo :-)08:57
MezLo-lan-do: but surely bzr will still think "oh, I'm parented to $SVN_REPO"08:58
Lo-lan-doDepends on how you did the import.08:58
Peng_Mez: "bzr pull --remember", then?08:58
Mezbranched svn, so I can keep the history08:58
Lo-lan-doIf you ran "bzr checkout svn+ssh://...", then "bzr unbind" will unbind.08:59
Lo-lan-doIf you ran "bzr branch svn+ssh://...", then you're not bound.08:59
Lo-lan-dobzr remembers the SVN URL as the parent location for subsequent pulls and merges, but that's about it.08:59
Lo-lan-doYou can remove the parent_location from .bzr/branch/branch.conf if you don't like to see it in "bzr info".09:00
=== timchen1` is now known as nasloc__
beunostatik, has your cron for updating bzr nightly gone to sleep and not woken up again?09:11
=== thunderstruck is now known as gnomefreak
beunovila, why is bug 331288 incomplete?11:46
ubottuLaunchpad bug 331288 in bzr "Push should have an option to not stack" [Undecided,Incomplete] https://launchpad.net/bugs/33128811:46
vilaBecause I didn't have time to test that it already works and I thought the bug reporter may do that :-)11:46
vilabeuno: well to test *again* should I say, because I indeed tested it days ago, but was in a hurry and didn't check all the things I should have checked :)11:47
beunovila, so maybe add a comment?  I'm sure jml will cry if he sees that you think he is incomplete11:48
vilaby marking it incomplete, I know either jml will come back after testing or *I* will get more time to test it properly11:48
vilabeuno: huh ? I added a comment saying it should work by using '--no-stacked' instead of '--not-stacked'11:49
vilaI sure remember having used --no-stacked11:49
beunovila, ah, I did not see that11:49
beunoignore me11:49
vilabeuno: Argh ! I hate that bug in launchpad: start typing a new comment, realize you should change the status or something, do that, watch your comment vanish :(11:50
vilabeuno: comment added again11:51
beunovila, that will be fixed soon-ish11:52
beunoin an ajax-y kind of way11:52
vilabeuno: good :)11:52
vilabeuno: so you know about the problem already ?11:52
beunovila, yes, it's happened to me a few times as well11:56
asabilanyone ?12:03
=== UdontKnow is now known as root
=== beuno is now known as beuno-lunch
=== beuno-lunch is now known as beuno
=== mvo__ is now known as mvo
domashi! how to make 'bzr log' faster? it takes >24 hours on quite powerful machine13:29
* Lo-lan-do suddenly feels *his* performance problems are not that bad after all13:30
beunouhm13:31
lifelessdomas: wow, it shouldn't be *that* slow13:31
beuno24 hours seems a bit much13:31
beunodomas, what version of bzr is that using?13:31
lifelessdomas: are you running log on a particular file, with -v, or just 'bzr log' ?13:31
luksit's aliased to log -v, I bet13:31
domasbeuno: any, 1.1213:31
domaslifeless: 'bzr log -v' on whole tree13:31
domasit allocates ~800MB of memory13:31
domasand oprofile says it is walking dictionaries all the time13:32
lifelessdomas: its a known limit of bzr, it has to do a whole diff on every step in the log output; we're fixing it13:32
domaslifeless: I'd be happy to betatest :)13:32
domasany kind of caching, etc :)13:32
lifelessdomas: there is an alpha quality branch at the moment, lp:~bzr/bzr/brisbane-core, where we are assembling the fix13:32
domaslifeless: interesting, thanks!13:33
lifelessdomas: we need better disk data structures though; so there is a [somewhat costly] conversion to do to test13:33
domaswould that take 24 hours? :)13:33
lifelessdomas: there's also discussion on the list, and on the Roadmap page13:33
lifelessdomas: I don't know how long it will take on your repo; if its huge or very deep history it could take a while; but I would expect a matter of hours not days13:34
lifelesswe're only starting to optimise the conversion logic13:34
domasI was tempted to start hacking bzr too13:35
lifelessdomas: I guess i'm saying - please do track this. We're planning a beta quality release for this in March13:35
domasit was spending way more time in walking than calculating13:35
domaslifeless: thanks, subscribed ;-)13:36
lifelesscool13:36
lifelessnight everyone13:36
domaslifeless: I'm playing with opengrok13:36
domasit is awesome :)13:36
domasit has bzr integration, but thats too slow on big trees13:36
ronnyopengrok?13:37
domashttp://dammit.lt:8180/source/xref/mysql-5.1.31/sql/13:37
domasit is source repository browser with indexing13:37
domasthink, lxr on steroids13:38
ronnyhmm13:38
ronnythat reminds me how far i will have to go with anyvc if i want to make it neat13:38
=== ja1 is now known as jam
vilajam: ping14:30
jammorning vila14:30
vilamorning :)14:31
vilaI'm trying to fix test_autopack_rpc_is_used_when_using_hpss failing for dev5 formats14:31
vilaIt turns out we don't use hpss because InterPackToRemotePack rules out the specific autopack14:31
vilaFixing that reveals that we don't have the right Packer14:32
vilaWhich in turn can't be easily fixed because of our design14:33
ronnyjelmer: sup? whats missing in dulwich to give status info about the index/the workdir?14:33
vilajam: because Packer is the root of two class hierarchies: one for the format one for some features (like reconcile)14:34
jamvila: are you talking about for GC repos, or for chk repos?14:35
jamAIUI chk repos use the same Packer14:35
vilaI need CHKReconcilePacker._process_inventory_lines ... somwehere it can't be accessed14:36
ronnyjelmer: btw, any chance to move dulwich to a bsd style license?14:37
vilajam: hmm, I'm wrong, Packer doesn't define a hierarchy for formats, yey, I need to override Packer._process_inventory_lines without using a ReconcilePacker...14:39
jamSo... to start with, I wasn't sure where to push in the autopack RPC, which is why I left that test alone :)14:40
jamI don't quite understand why to get autopack working, you need _process_inventory_lines ?14:40
vilajam: let's rollback14:41
jamAs for stuff like "Packer", it would be reasonable to define class attributes14:41
jelmerronny: hi14:41
jamSo PackRepository._packer_class = Packer14:41
jamCHKPackRepository._packer_class = CHKPacker14:41
jametc14:41
jelmerronny: We'll be merging some stuff from pyrite soon, that should include an index parser14:41
jamor possible a registry14:41
ronnyjelmer: an api that compares to bzr's WorkingTree.iter_changes would be nice14:42
vilaInterPackToRemotePack rules out using InterPackToRemotePack if any of the repo supports_chks14:42
vilaInterPackToRemotePack rules out using InterPackToRemotePack if any of the repo *doesn't* supports_chks14:42
vilajam: see InterPackToRemotePack.is_compatible in repository.py14:43
vilaThat seems wrong if both repo supports_chk, so I fixed that but then it fails because Packer._process_inventory_lines calls repo._find_file_ids_from_xml_inventory_lines14:43
jelmerronny: wrt BSD licensing; there would have to be a very good reason, such as another Open Source project that is definitely going to use Dulwich but is blocked by the licensing14:44
ronnyjelmer: at least lgpl?14:44
vilajam: but ReconcilePacker._process_inventory_lines adresses precisely that14:44
vilajam: is that clearer ?14:44
vilajam: and do you agree about InterPackToRemotePack.is_compatible ?14:45
ronnyjelmer: since my encounter of not being able to combine different things cause one was glp2 only and the other was gpl3 only im not exactly accepting gpl any more14:45
jelmerronny: dulwich is GPLv2+ at the moment14:46
jamvila:  think you were right in your first statement. InterPackToRemotePack rules out if any repo *does* support chks14:46
jamSecond, I don't really think we want to switch to using Packer based fetch14:47
jamI'm pretty sure we are trying to get away from that14:47
jaminto the insert_record_stream(get_record_stream()) based APIs14:47
jamSo IMO, the real fix is to help insert_record_stream() && commit_write_group() trigger a remote autopack()14:48
jamrather than trying to get it to use InterPackRepo code.14:48
jamvila: does that make sense?14:48
vilajam: At the highest level yes. Concretely it means I will push making that test pass lower in my task list and fix simpler ones first :)14:49
vilatime to shelf debug statements :)14:50
vilajam: hmm, not yet, I should first understand where the insert_record_stream route will start14:50
vilas/will start/should start/14:50
jelmerronny: I wouldn't be opposed to LGPL, but there would have to be a good enough reason and of course everybody else involved would have to be ok with relicensing as well14:51
ronnyhmk14:53
jamvila: also note that Robert and Andrew are working on a new fetch command14:53
jamwhich also should be preferred to Packer based work14:54
jamand deals with Remote calls14:54
jamas there is some overlap here, I agree that we should wait14:54
jamif you *want* turn those into KnownFailure14:54
jamwith a  comment like14:54
vilajam: reason enough to wait ? Or can you point14:54
jam"Remote autopack calls is being postponed for improved fetching"14:54
vilalol, s/RETURN/DELETE/ :)14:54
vilajam: deal, I'll do that14:55
beunostatik, has your cron for updating bzr nightly gone to sleep and not woken up again?15:02
statikbeuno: oh my i hope not.15:02
* statik looks15:02
statikbeuno: they are confused with this error: bzr: ERROR: Unknown repository format: 'Bazaar RepositoryFormatKnitPack6 (bzr 1.9)\n'15:04
* statik upgrades bzr on the build machine15:05
beunostatik, ah! the build machine doesn't install it's own versions?  :)15:05
statikbeuno: all fixed, and new packages are churning in the PPA now. thanks for letting me know it was stalled!15:20
beunostatik, awesome, thanks for keeping this rolling, it's very useful to dogfood this stuff15:21
statiknp, i'm happy to be able to help the project in a small way15:21
vilajam: in bzrdir.py we replaced bzrlib.workingtree_4.WorkingTreeFormat4 by bzrlib.workingtree.WorkingTreeFormat4. Is there  a resaon for not doing the same for bzrlib.workingtree_4.WorkingTreeFormat5 ?15:41
jamno15:43
jamwe should use the workingtree.XXX ones15:43
vilaok, will fix it in bzr.dev then15:44
vilajam: already fixed there by.... you :)15:56
jamyep, I thought so15:56
asabilanyone to help out with the multiple root issue ?16:07
=== kiko is now known as kiko-fud
=== nekohayo_ is now known as nekohayo
=== Guest94594 is now known as jelmer
jamstatik: i'm gettting "build failures" fo rthe ~bzr-nightly-ppa16:41
jamdo you know why?16:42
=== jfroy|work is now known as jfroy
statikjam: i don't, i just fixed the system to start uploading daily builds again a couple hours ago (it was failing due to an old version of bzr on the build machine not understanding the branch format).17:26
statikjam: i'll try to take a look today and see what is failing17:26
mwhudsonbeuno: thanks for the LH reviews :)17:31
beunomwhudson, I tried to eat through as much LH as I could. Not sure if I managed or not.17:32
mwhudsondid you manage to try the diff js goo in IE?17:33
beunono, I'm in Berlin still17:34
beunoand not back home til next week17:34
beunowhere I have all the evil stuff like windows machines17:34
mwhudsonok17:35
mwhudsoni'll merge anyway i think :)17:35
beunoyeah, we can fix it later on17:35
vilajam: nearly off for the day, I pushed revno 3832 to bbc, the test suite is passing except for 1 failure and 4 errors17:38
=== jkakar_ is now known as jkakar
vilajam: 2 errors being AttributeError: 'CHKInventory' object has no attribute 'apply_delta' coming from bzrlib.tests.test_repository.TestDevelopment5FindRevisionOutsideSet, so dev5 specific17:39
mwhudsonhm17:41
mwhudsoni'17:41
mwhudsoni'm not sure i like the level of deviousness you can use in css17:42
mwhudsonall too tempting :)17:42
vilamwhudson: Just do as you I feel, you will regret it anyway :)17:42
vilamwhudson: Just do as you feel, you will regret it anyway :)17:43
vilaand one more joke ruined by a typo :)17:43
mwhudsonheh17:43
mwhudsonbeuno: got time to give a visual opinion on coloring line numbers?17:44
beunomwhudson, I hear we will eventually get a full blown operating system made in css17:45
beunosure, shoot17:45
mwhudsonbeuno: :)17:47
mwhudsonbeuno: wondering what colour the indicated "cells" should be http://people.ubuntu.com/~mwh/ss.png17:47
beunomwhudson, one line is for removes17:48
beunoand the other for adds, right?17:48
beunoah17:48
mwhudsonyes17:48
beunook17:48
beunoso17:48
beunohow about a lighter tone of red, and a lighter tone of green?17:49
mwhudsonhm, that probably works17:49
Lo-lan-doMaybe do both cells as light red and both cells as light green?17:51
Lo-lan-do(Yay bikeshedding)17:51
mwhudsonbeuno: try lp:~mwhudson/loggerhead/unified-by-default-sbs-by-ajax rev 28718:00
beunomwhudson, I like it!18:02
mwhudsonbeuno: cool18:03
mwhudsoni replied to your review, but i think i'm going to merge the branch now18:03
beunomwhudson, sure, I'll reply back, but as I said, it's in a good state to land18:04
mwhudsoncool18:04
mwhudsonbeuno: i spent most of yesterday when i wasn't being the CHR dude trying to move away from mootools, btw18:07
beunomwhudson, oh, that sounds like a fantastic thing to do18:09
beunothe slider animations are going to be a PITA18:09
mwhudsonyeah, the animations are just commented out at the moment18:09
mwhudsonthere seem to be totally weird ass interactions between window.addEvenHandler (mootools) and Y.on() (yui) wrt domready :/18:10
mwhudsonbeuno: oh right, something I meant to ask18:10
mwhudsonbeuno: i don't really understand how to spread code between different js files but have them share a Y object18:11
statikjam: the nightly build seems to be failing because it's trying to use python2.6. Any idea why? http://launchpadlibrarian.net/22882024/buildlog_ubuntu-jaunty-i386.bzr_1.13~bzr9.04-4020-1_FAILEDTOBUILD.txt.gz18:11
statikjam: I'm building the source packages on a hardy machine, definitely don't have any python2.6 there18:12
jamstatik: I don't know. Is python2.6 the default python on jaunty ?18:12
jamThat said, we *should* support python2.618:12
jamas vila works hard to ensure compatibility18:13
mwhudsoni think jaunty might *only* have 2.6...18:13
statikjam: no, it's not even available in jaunty. yeah, things should absolutely run with python2.6, but the problem is that it's not installed in the chroot in the buildd18:13
jamI see what you mean, though:18:13
jamcd . && python2.6 setup.py build --build-base="/build/buildd/bzr-1.13~bzr9.04-4020/./build" --executable "/usr/bin/python" /bin/sh: python2.6: not found make: *** [python-build-stamp-2.6] Error 127 dpkg-buildpackage: failure: debian/rules build gave error exit status 218:14
statikxactly18:14
beunomwhudson, I've been trying to get that sliding animation to work in lzr-js. Will let you know if I do.18:14
jamthat is part of "update-config"18:14
jamcan you check your packaging branch?18:14
statiksure, will have a look now18:14
jamIt at least seems to be generated from "debian/rules"18:14
beunomwhudson, as to sharing a Y object among different files...  it sounds tricky18:14
jamAnd it is possible that the latest version from jelmer (for debian) uses python2.618:14
=== mvo_ is now known as mvo
statikthats probably what it is18:16
jelmerjam: debian doesn't have python2.6 yet :-)18:20
statikjelmer: yeah, i'm stumped where this reference to 2.6 is coming from18:21
jamstatik: It sure looks like it is something with the build daemons themselves18:21
jamthe first thing it does is:18:21
jamdh_clean  cd . && python2.6 setup.py clean -a /bin/sh: python2.6: not found make: [python-clean-2.6] Error 127 (ignored) cd . && python setup.py clean -a18:21
jamstatik: I'm also seeing a problem with "Pyrex" not being found18:22
jamwhich would also be sort of a bad thing for the final built versions18:22
jamsince they don't have the .c extensions18:22
jamI suppose we are a bit stuck....18:23
statikjam: yeah, the packaging doesn't Build-Depends on python-pryex18:23
jamwe don't really want to depend on Pyrex for the final build, because we put the .c files in the tarballs18:23
jambut for intermediate ones18:23
jamwe don't bother18:23
jelmerstatik: if you have 2.6 it might be trying it if it is thre defualt python18:29
jelmerstatik: we just define that we support all pythons since 2.418:29
statikjelmer: i don't, thats the weird thing. I'm building a source package on hardy, and uploading it to a jaunty PPA on launchpad18:29
statikand python2.6 is nowhere in sight18:29
jelmerstatik: strange indeed :-/18:42
james_wone thing is that it lists python2.4-dev, python2.5-dev when python-all-dev is probably a good idea18:44
james_wit may be that it got caught in the middle when python2.6 was only partly default18:45
=== chx is now known as chx_sleeping
jelmerjames_w: ah, good point19:19
=== kiko is now known as kiko-afk
jelmerjames_w: strange, it's fine in the version in Debian20:16
=== root is now known as UdontKnow
ronnylifeless: any plans to have signed tags? (something like git's tag object)20:41
jelmerronny: fwiw, there are already signed revisions20:42
ronnyjelmer: yeah, but not tags20:47
ronnyhmm20:47
ronnytime to add push/pull support to anyvc20:48
mwhudsonhm21:05
mwhudsonbzr search -s just broke on me21:05
mwhudsonoh, my bzr-search was way out of date21:07
lifelessronny: I thought gits tag object was just a blob/commit - I didn't see a special object type for it21:17
lifelessasabil: I don't hink anyone trivially knows21:18
ronnylifeless: its an own object type21:18
lifelessasabil: I know I probably need to jump into a debugger to figure it out21:18
ronnythey have blob, tree, commit, tag21:18
lifelessasabil: perhaps mail the list?21:18
ronnytags can be put on any objects21:18
lifelessronny: ok21:18
ronny(ie also blobs/trees)21:18
ObmarHi there, hopefully I am not asking a question that has been asked 1000 times before, but here goes.  I have bzr on my local computer.  I have init the project, add the files, commit the project.  Now I am trying to merge it to a centralized copy.  I have bind and merge and commit, but the files are not on the server.  The .bzr folder is there, and it says everything is fine, but the actual source code files are not in the folder.  What could I possibly be doin21:24
garyvdmObmar: 1. did you push to the central server?21:24
ObmarYes.21:25
ObmarThis is to a ftp://... if that makes any difference.21:25
RAOFObmar: bzr doesn't update the working tree when pushing.  This is what you're seeing.21:25
garyvdmObmar: 2. When you push to a remote protocal - it does not update the working tree.21:25
ObmarThe project folder did not exist before, but it was created, along with the .bzr subfolder.21:25
garyvdmObmar: Is this for a website?21:26
ObmarSo... maybe I am confused, how do I get it to update the working tree on the server?21:26
ronnyObmar: all the history is within the bzr folder21:26
Obmargaryvdm: No.21:26
=== chx_sleeping is now known as chx
ronnyObmar: is this for some software on the server, or do you just need a repository there?21:26
ObmarJust the repository.21:27
ronnyObmar: if all you want to do is keep the history there, then you dont need to update on the server-side21:27
ObmarBut I develop on serveral different computers.21:27
ObmarI need all the files and the history there.21:27
ronnyObmar: you dont need a workingtree on the server, only the .bzr folder with the history21:27
ObmarSo when I go to a new computer, how will I work on the code?21:27
ronnyor is this an actual software that has to run on the server?21:27
ronnyObmar: you pull the history21:27
ObmarNo, it does not run on the server.21:28
ronnyObmar: the history implies the files21:28
ObmarAre they in that subfolder somewhere then?21:28
Obmarthe .bzr folder I mean?21:28
ronnyObmar: the history is in the .bzr dir21:28
ObmarAh, so it is not going to look like my local copy.21:28
ObmarSince I have .bzr, src/, build, etc...21:29
ObmarBut I only see .bzr on the server copy.21:29
ronnyObmar: on your local copy you have a branch with a working directory, on the server you only need the branch, not the working directory21:29
ObmarOk, I think I follow.21:29
ObmarSo the files are safely up there then.21:29
ObmarI was getting a bit worried.21:29
ObmarSo when I do a pull on a different computer, I will get a working directory there?21:30
ronnyyes21:30
ObmarAh ha.  That is what I was missing.21:30
ObmarSo I was doing it right.  Just missed a concept.21:30
ObmarOk, second question, only sort of related.  I am using Eclipse, and I have the bzr plugin installed, but the only options under the team menu is Apply Patch and Disconnect.  All the other options are disabled (greyed out)21:31
ronnyno idea about eclispe21:31
Obmarlol21:32
ObmarHopefully somebody does.21:32
ronnyi avoid it as its mostly wasting my ram and cpu time21:32
ronnysame goes for netbeans and other fscking big & laggy ide's21:32
ObmarIt is big.  Agreed.21:33
ObmarIt is what I hae for now.21:33
ObmarIt would be nice if I could get the bzr integration working though.21:33
lifelessObmar: did you follow the bzr eclipse faq?21:34
ObmarYes, as best as a could.21:34
ObmarIt is installed.21:34
ObmarAnd the init command works.21:34
ObmarThe rest are disabled though.21:34
ronnyhmm21:37
lifelessverterok: ^21:37
lifelessObmar: verterok is the author21:37
Obmarverterok:  Any ideas?21:38
ObmarSo... on question 1...  I have a local repository..  the best path to "upload" that to the server is... push/bind/commit?21:40
ObmarI mean, is that the right order?21:40
ronnyObmar: if you bind, it will imply that each commit is pushed, thats nice for working on more than one computer, but not so nice if you want to work as interuption-free as possible21:41
ronnylifeless: btw - i think i need bzr at least twice as fast as its currently21:42
ObmarI like the bind idea, for me at least.  I am on fast links between all the locations.21:42
ronnyObmar: ah, ok21:43
ObmarOne less thing to worry about.21:43
pooliehello all21:43
ObmarI just have to update...work...commit21:43
ObmarI think that is all I have to do, if I understand the docs correctly.21:43
ronnyObmar: im not fluent on the bind workflow, im on slow links/disconnected often21:44
lifelessronny: ok21:49
lifelessronny: we're working on that :P21:49
spivlifeless: gar, I screwed up my from address when sending a patch yesterday, re-sent.21:50
igcmorning21:53
pooliehello igc21:56
poolietoday is going to be a good day for you :)21:56
pooliei have a big block in my diary for content flittering21:56
lifelessspiv: ah :P21:58
verteroklifeless: thanks & hi21:58
verterokObmar: hi21:58
verterokObmar: could you check if there are errors in the error log view?21:59
verterokObmar: or in $WORKSPACE/.metadata/.log, related to bzr-eclipse?21:59
Obmarverterok: http://pastebin.com/m6b21c84d22:01
ObmarLooks like it22:01
igchi poolie. Time for a review today? :-)22:01
poolieexactly22:01
pooliesee above22:02
verterokObmar: what version of xmloutput have you installed?22:02
verterokObmar: let me guess: bzr 1.12 and bzr-xmloutput <= 0.8.2?22:03
ObmarNot sure how to tell.22:03
Obmarverterok: Genius.22:05
ObmarI didnt know which one.22:05
ObmarSo I went and got the latest.22:05
ObmarAll the menu options lit right up.22:05
ObmarThank you.22:06
verterokObmar: no problem, glad to help22:08
* verterok back to the sprint, bbl22:08
Obmarverterok: Still here?22:09
Obmarverterok: Trying to authenticate against FTP.22:09
=== serg is now known as serg|away
=== serg|away is now known as serg
Obmarverterok: Nevermind just saw the limitations part of the site.22:10
lifelessspiv: with patch please :)22:10
asabillifeless: ok thanks, I just sent a mail to the ML22:27
lifelessspiv: reviewed; needs a little work22:32
lifelessspiv: I forgot a vote line - bb:tweak22:32
jml:(22:46
lifelessjam: ping; like to high bandwidth discuss network_name22:46
jmlI'm pushing up a stacked branch to Launchpad, one that's already been pushed, and it is still pushing after ~40mins22:46
lifelessjml: anything in the log?22:47
jmlyeah.22:47
lifelessare you sure its stacked?22:47
jmlIt was, at least: https://code.edge.launchpad.net/~jml/launchpad/cloud-style-changes22:47
lifelessjml: 'bzr info lp:~jml/launchpad/cloud-style-changes' should tell you22:48
jmllifeless: the Launchpad page I referred you to does basically the same thing :)22:48
jmlbut still22:49
jml     stacked on: /~launchpad-pqm/launchpad/devel22:49
pooliejam, thanks for your comments on the EC2 thing.  i replied - how does that idea suit you of just leaving it up?22:49
jmllifeless: https://pastebin.canonical.com/14025/22:50
jmllifeless: the logs are filled with variations on that.22:50
lifelessjml: its probing for a tonne of revisions it doesn't have22:51
jmllifeless: what's causing that?22:53
jmland what do I do about it?22:53
lifelessjml: I don't know and let it run22:53
lifelessfile a bug afterwards with the log (trimmed to elide the repeated sections in the middle)22:54
poolieigc, could you see today or soon about updating orcadas to run more benchmarks, including if possible the network ones?22:54
pooliethere should now be scripts there that let us turn on the slow network22:54
jmlHPSS calls: 7898 <bzrlib.smart.medium.SmartSSHClientMedium object at 0x29a7b50> :)22:55
jmlit just finished then.22:55
spivwalk_to_common_revisions is the usual problem.22:58
lifelessI've 3 branches pending, all approved except the bottom one (network-names)23:01
lifelessI get the feeling jam is done for the day, he commented - if someone could review and read our thread that would be great23:02
jelmerlifeless: fwiw I'm happy to review some of the smart server stuff23:06
lifelessjelmer: anytime :)23:07
jmllifeless: https://bugs.edge.launchpad.net/bzr/+bug/331823 fwiw23:07
ubottuUbuntu bug 331823 in bzr "Pushing an update of a stacked branch to Launchpad sometimes takes nearly an hour" [Undecided,New]23:07
lifelessjelmer: the Repository.initialize branch probably addresses some of your 'getting branches we can't use' issues23:09
jelmerlifeless: ok, I'll have a look23:10
igcpoolie: I'd like to leave that to Monday if I can - knee deep in code today that I need to get running before the weekend23:10
lifelessjelmer: the network-name one is the only currently unapproved one23:10
lifelessok, running a full regression and getting food concurrently23:10
igcpoolie: as then I can run some big imports on the weekend23:10
jelmerlifeless: I don't feel comfortable enough with that code to do review23:11
lifelessjelmer: ok23:11
lifelessjelmer: given all the work you do with custom formats I'm surprised though :)23:11
poolieigc, ok makes sense23:12
jelmerlifeless: that bit isn't the problem :-) It's more the smart server side of things23:13
lifelessnetwork-name doesn't touch the smart server23:18
* igc breakfast23:20
jelmerlifeless: The code seems ok, the problem is the rationale behind the change23:22
=== TheMuso_ is now known as TheMuso
jelmerlifeless: In particular, you seem to make it possible to have streams that are not necessarily tied to a particular bzrdir format23:39
jelmerlifeless: and potentially having multiple repository formats returning the same string23:39
jelmerlifeless: wouldn't you just want a different format object for the stream ?23:42
lifelessjelmer: they are tied to a particular repository format23:46
lifelessjelmer: bzrdir formats are not particularly useful for the metadir set of flavours23:46
lifelessjelmer: its a registry, so we have the registry uniqueness guarantee (and repositories *could* claim to be a byte-compatible format for compatibility, on the wire)23:47
lifelessjelmer: so consider knits - the record_stream from knits, pack-0.92 are identical23:47
lifelessfrom pack-1.6 and pack-1.9 they are identical too23:47
lifelessthe different between knits, pack-0.92 and pack-1.6,pack-1.9 is that the latter may generate different parents for file texts during reconcile, but for network streams all four will be identical - same serialisers23:49
lifelessjelmer: so the goal is to be able to reconstitute a RepositoryFormat object in the smart server, to handle the stream even though we don't have the source repository available23:50
lifelessbrb23:50

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!