/srv/irclogs.ubuntu.com/2008/06/02/#bzr.txt

lifelessmorning y'all00:16
Verterokmoin00:17
beunohey lifeless!  feeling better?00:24
lifelessbeuno: yes thanks00:36
spivpoolie: that fix of mine just landed00:44
* igc out for a while - bbl00:52
pooliespiv, great01:03
lifelessabadger1999: ping01:04
lifelessabadger1999: sorry01:04
lifelessabentley: ping; I have a bundle confusion thing; I'm debugging but perhaps you've seen it in the past and can advise01:05
jmlspiv: which bug was that a fix for?01:05
jmlspiv: because my attention is being drawn to bug 236380 in Launchpad, and I wonder if they are perhaps related.01:05
jmlhow can I get a list of changes to trunk filtered by whether they changed files in a particular directory?01:12
jmlbzr log --short foo/bar does *something*01:12
spivjml: I don't think my fix is related to that bug01:14
jmlspiv: ok. thanks.01:14
spivjml: I can reproduce 236380 with bzr.dev, btw01:17
pooliehello jml01:26
jmlpoolie: hi01:26
* markh waves01:49
markhI'm looking through the API docs, but I can't see a way to do iter_changes or changes_from non-recursively - ie, if I pass a directory name, I do *not* want children checked.  Any clues?01:50
lifelessI'm not sure we have that01:51
markheg, I'd like to know the dir has been "added", but I don't yet care about children01:51
markhfor tortoise, I'd obviously like to avoid doing a full tree status if a user happens to browse to the root of a large working tree.01:52
markhI can see how to work around it for files - but it seems that to get the status of a single directory anywhere in a tree requiring me to do a full tree status update is going to kill in some cases.01:59
markhat least '--no-recurse' is listed as a todo ;)02:06
brandon_rhodesAfter numerous experiments (and learning to delete ~/.bazaar/subversion.conf after each one!), I have managed to make "bzr svn-import" import a project from my subversion repository (yay!)02:54
brandon_rhodesBut it's treating my tags as branches02:54
brandon_rhodesRather than giving the mainline trunk a list of actual tags that can be listed with "bzr tags"02:54
brandon_rhodesIs there anything that can be done about that?02:54
brandon_rhodesOr should I march boldly into the future without my old tags around as tags? :-)02:55
bob2tag support is planned but not implemented yet, aiui02:55
brandon_rhodesRoger.  I'm about to move a project to Launchpad and wanted to make sure I wasn't missing something first02:57
brandon_rhodesThanks.02:58
=== lifeless changed the topic of #bzr to: launchpad bazaar hosting is shutdown for urgent maintenance | Bazaar version control system | http://bazaar-vcs.org | bzr-1.5 is out! | http://irclogs.ubuntu.com/ | http://planet.bazaar-vcs.org/
pickscrapeWhat would I need to call in bzrlib to get a Tree of a branch at some given revision?03:12
jameshpickscrape: branch.repository.revision_tree(rev_id)03:14
pickscrapeThanks. Could you point me at somewhere I could have looked that up myself?03:15
pickscrapeI've looked around for a 'bzrlib for plugin developers' doc but couldn't find one.03:15
lifelessdoc/developers/ has some stuff03:15
pickscrapeThanks03:15
lifelesssorry, doc/en/developers I think.03:16
pooliehm there is a document aimed at just that03:17
pooliepickscrape: http://bazaar-vcs.org/Integrating_with_Bazaar03:18
pooliei'm sure there are things plugin developers might want to know that aren't answered there though03:18
poolieso if you hit something please ask here or on the list03:18
pickscrapepoolie: thanks for that!03:19
pickscrapeIt's a very rich API, which is great but a tad bewildering when you first look into it. :)03:19
poolieyeah03:19
poolieand it does change over time too03:20
pooliebut we are pretty friendly, do ask03:20
pickscrapeYes, this is actually what's I'm doing (accounting for a removed method)03:20
poolieit's the best way to guide documentation improvements too03:20
lifelessyay 5 errors to go in bundle tests03:50
markhIt seems the bzr tests fail to remove the test directory when the test fails (Permission denied: unable to remove testing dir tmpfq3hof) - is that known?03:52
lifelessmarkh: it is a bug indeed03:54
markhany idea if its already reported?03:54
markhsave me searching :)03:54
lifelessI know it has been discussed/partially fixed etc in the past03:55
markhgreat, thanks03:56
markhlifeless or poolie: any idea if adding a 'recursive' param to iter_changes and changes_from would be controversial?  ISTM it would have to be implemented in inventory.iter_entries_by_dir() and might not be that hard or deep.  Should I give it a shot, or discuss the requirement itself on the mailing list first?03:59
lifelessmarkh: inventory.iter_changes* are largely deprecated03:59
lifelessmarkh: look at workingtree_4.py's iter_changes04:00
lifelessmarkh: and definitely discuss on the list; I mean - it seems to me you actually are looking at a single path rather than a dir?04:00
markhlifeless: I'm not sure what the last part of that means.  The way TSVN works is that when a filename or directory name is requested, the status of all items in the parent are queried and cached.04:02
markhbut actually that doesn't seem relevant - even if we just fetch one item at a time, I don't want to recurse when the status of a directory is requested.04:03
lifelessmarkh: to a large extent this depends on the definition of 'status' for a directory04:04
lifelessmarkh: say you are putting an emblem on altered directories04:04
lifelessshould a directory with an altered child get an emblem ?04:04
markhwell, its more like "get the full status of a dir in the backgroumd, but the "immediate" status in the foreground"04:04
markhwe will manage the background part, but do need a way to say "give me the 'immediate' status of the folder itself, not of its children"04:06
markhif we want to show *any* emblem before the full recursive status is calculated04:06
markhwhich we do04:06
markh(and the fact the status command has a comment indicating 'no recurse' is a todo item gave me hope the requirement itself is kinda 'obvious')04:08
lifelessI am behind on a deadline due to being sick for a week04:09
markhno worries - I hope you are better - thanks for the comments and I'll take it to the list04:09
lifelessso can't really offer good interactive discussion on this; it certainly doesn't need to block on me - the pointer already given, to the fast-path iter_changes function should give you an idea of the code that you'd need to modify04:10
lifelessnice:04:17
lifelesstime bzr log megarepo/pool/main/p/pyvorbis/04:17
lifelessreal    0m1.274s04:17
lifelessuser    0m0.150s04:17
lifelesssys     0m0.050s04:17
lifelesscold cache04:17
lifelessreal    0m0.178s04:18
lifelessuser    0m0.150s04:18
lifelesssys     0m0.020s04:18
lifelesshot cache04:18
pickscrapelifeless: what's the context of those timings?04:25
lifelesspickscrape: http://www.advogato.org/person/robertc/diary/83.html04:27
lifelessrobertc@manganese:~$ du -sh megarepo/.bzr/04:27
lifeless22G     megarepo/.bzr/04:27
lifelessits a single repository with ~all of ubuntu04:27
pooliepickscrape: for curiousity what are you working on?04:27
pickscrapeAh yes, I did read that blog post. Very nice :)04:27
pickscrapepoolie: getting diffstat working again04:28
poolieoh yay04:28
pickscrapeProbably a 5 minute job for someone well versed in bzrlib, but good learning material for me :)04:28
jmlI was just reminded how neat 'merge --uncommitted' is, so I blogged about it: http://mumak.net/2008/06/02/neat-bazaar-feature/04:29
pooliejml, we should add you to planet bazaar hey?04:30
jmlpoolie: oh yeah maybe04:30
pooliepickscrape: that would be great to have04:30
poolieactually it might be nice to have it builtin, at least if it just adds new options to diff, status, send, etc04:31
pickscrapepoolie: that's what I was hoping to do to it next04:31
pickscrapeI actually don't think it needs to add any new command at all04:31
pooliegreat04:32
pickscrapeDo you know if Michael Ellerman (its origina author) is still around?04:33
pooliei haven't seen him for a while04:33
pickscrapeIt's not been touched since 2006, so I have the impression that it's basically unmaintained at this point.04:34
pooliei think you're right04:34
pickscrapeThat being the case I was thinking of hosting it on launchpad (once I've got it working), but I'm unsure of the etiquette with regard to picking up things like this.04:35
pooliei think just go ahead and publish it04:36
poolieafter all he can always merge back if you want04:36
pickscrapeYes04:36
pickscrapeok, I'll do that.04:36
pooliemarkh: i think that 'permission denied' might be windows-specific?04:38
pooliedoes it happen with all failing tests?04:38
markhyes, I've no doubt its windows specific, and best I can tell, all tests04:38
pooliecould you maybe put a breakpoint at the place it is raised so we can work out what specifically is going wrong?04:39
markhactually, not all tests now it has finished :)  No transport tests04:39
markhok, I'll see how I go04:39
markhlikely to be a handle to that dir is open, so the trick might be finding it.  I'll see what I can find.04:40
pooliei think that would be it04:42
beunomwhudson, around?  I have good news  :)04:45
pooliebeuno: he has a public holiday toda, but you can tell me :)04:46
beunopoolie, well, let me get you the URL.  I got Loggerhear to *not* request all diffs for each revision, and you can request each diff via ajax, cutting by 4 the time/memory needed for each diff  :)04:47
poolienice04:48
beunoso, with the new templating system, and diffs being cheaper, it should be feasable to run it without it explodind a dozen times a day   :)04:49
beunos/explodind/exploding04:50
beunoI still think we should throw away the current code  :)04:51
* mwhudson is here but worrying about loggerhead definitely seems like work04:55
beunomwhudson, :)04:55
pooliebeuno: ok with me, as long as you agree with mwhudson and other folks that it's the best thing to do04:55
beunolet me try and get this port open so you can peak04:55
poolieand what specifically to do instead04:55
poolieand hopefully it'll be something that's both easy for users to deploy and works in well as part of lp04:55
pooliehello keir04:56
keirhey martin04:56
mwhudsontbh, the main concern i have with beuno's ideas is the requiring javascript stuff04:56
keirso i guess launchpad bzr is down :(04:56
keiri was here to ask about that04:57
mwhudsonkeir: will be back very soon04:57
pooliekeir, yes, jml is working on it, should be back soon04:57
keircool04:57
keiri was thinking i was doing something wrong :)04:57
mwhudsoni think a gmail-for-bzr could be really really cool though :)04:57
keirwoot! did i hear gmail+bzr?04:57
pooliewell, i think we need to have some kind of reasonable fallback if they don't have js (enabled)04:57
pooliei think he meant gmail-style interface04:58
mwhudsonyes04:59
beunomwhudson, http://intranet.pentacorp.net:8080/bazaar/bzr_dev/revision/pqm%40pqm.ubuntu.com-20080530010236-e3x7ckdc25s57pgc?start_revid=pqm%40pqm.ubuntu.com-20080530010236-e3x7ckdc25s57pgc04:59
beunomwhudson, well, the way I'm doing things now, we can fall back to an html-only version pretty easily04:59
mwhudsonbeuno: seems to work05:00
mwhudsonsome kind of progress report would be a good idea05:01
beunoI haven't done extensive benchmarking, but a 8k diff uses very little memory, and, even more important, now, it's optional  :)05:01
mwhudson(maybe just a spinning gif)05:01
beunomwhudson, right, there are *tons* of improvements to do on top of that05:01
beunobecause of the way loggerhead works, it's getting the diff for those files anyway, I'm just not parsing them05:01
pooliebeuno: sweet05:02
beuno:)05:02
poolieis that tested at all?05:02
pooliewe should think about that too05:02
poolieand when i say 'we' i mean 'you' :-) :-)05:02
beunopoolie, test what, sorry?05:02
pooliewell05:02
poolieactually i don't know if it has any automatic tests at present05:03
pooliebut it would be nice to get some05:03
poolieincluding that the js works as expected05:03
mwhudsonloggerhead has some _very_ basic tests currently05:03
beunomwhudson, I haven't figured out how to run them  :)05:03
mwhudsonbeuno: nosetests05:03
mwhudsonor py.test05:03
beunomwhudson, I'll take a look at that (we should document that :p)05:04
jmlpoolie: the corporate 'we' :)05:04
=== lifeless changed the topic of #bzr to: Bazaar version control system | http://bazaar-vcs.org | bzr-1.5 is out! | http://irclogs.ubuntu.com/ | http://planet.bazaar-vcs.org/
beunocool, tests kinda of work!05:05
beunomwhudson, so, getting diffs individually seem to be *way* cheaper05:06
mwhudsonbeuno: interesting05:06
mwhudsoni think it's all in the rendering probably05:06
beunoI'm using the zpt branch05:06
mwhudsonoh cool05:07
mwhudsonyou got that working in the end?05:07
beunoso, it we combine zpt + not rendering everything + individual diffs, it should be a big improvement peformance-wise05:07
beunoyeap05:07
beunoI pulled your branch, and tweaked a little05:07
beunoI'm going to add a progress while it's fetching the diff05:08
beunoand then update the branch to use the new methods (it still has the deprecated ones)05:08
beunoclean up, and push05:08
beunoand the font seems to be smaller for some add reason. Css is ignoring me today05:09
beunos/add/odd05:09
beunopoolie, I'm not sure how we can test javascript. Especially since each browser renders it differently05:10
beunoI am using a very popular js library, so it's almost guaranteed to work on all popular browsers05:11
mwhudson'selenium' is one answer05:11
markhpoolie: I noticed my debugger changes the behaviour, which made me think "gc" - and sure enough, a gc.collect() before the osutils.rmtree(dirname) in tests._rmtree_temp_dir() seems to solve the problem.  Apart from the problem that a number of tests are still failing ;)05:12
beunomwhudson, looks good, I'll look into it05:14
beunoand stop bothering you on a holiday  :)05:15
jmlsome of the JS could be unittested with spidermonkey & integrated into the main suite using subunit.05:15
jmlbut I don't think very much could.05:16
abentleylifeless: pong05:45
lifelessabentley: I mailed the list05:45
lifelessabentley: I think there is a conceptual bug in the mpdiff interface/output05:45
abentleylifeless: I believe the bundle specifies what the parents are, but I can double-check that.05:50
lifelessabentley: even if it does I think that the lower level api will break because each individual text can have hit different orderings or different ghost texts05:51
abentleyThe orderings used are the orderings specified in the bundle.05:54
beunomwhudson, loader added: http://intranet.pentacorp.net:8080/bazaar/bzr_dev/revision/pqm%40pqm.ubuntu.com-20080530010236-e3x7ckdc25s57pgc?start_revid=pqm%40pqm.ubuntu.com-20080530010236-e3x7ckdc25s57pgc05:54
mwhudsonbeuno: sweet05:55
mwhudsona 'view all' might be appropriate i guess05:55
beunomwhudson, yeah, I'm not putting too much work into the interface considering I'm going to be implementing the new theme next week05:56
lifelessabentley: looking at mpdiffs, I just don't see were the text parent ids are grabbed - the diffs list contains MultiParent.from_lines output only05:56
beunomwhudson, I have to do major code cleanup now, and change the way loggerhead builds that page, so it doesn't still internally get the diffs and discard them05:57
abentleympdiffs don't list the parents.  The bundles list the parents.05:57
mwhudsonbeuno: fair enough05:57
beunoand, well, tests and such would be nice05:58
abentleylifeless: The metadata entries for a record include a "parents" entry, and this is what should be used when applying the mpdiff.06:02
lifelessyay only reconcile to go06:05
lifelessabentley: I'm concerned about a mismatch between the actual used parents in the knit and that record06:05
lifelessconcretely, looking at add_mp_records, ghosts will be present in the record06:06
lifelessbut ghosts are filtered out of the diff generation06:06
abentleyWhere are they filtered out?06:08
lifelessbzrlib/versionedfile.py line 26606:08
lifelessparents = [lines[p] for p in parent_map[version_id] if p in knit_versions]06:09
abentleylifeless: I think I agree.  Whatever parents were used to generate the mpdiff should match the parents in the metadata.06:13
jmlbeuno: btw, your branch should be working now.06:13
beunojml, ah, let's see....06:13
beunojml, the problem was with all knit branches?06:14
jmlbeuno: not all. just many.06:15
jmlbeuno: anything that needed to write an escaped path to disk.06:15
beunojml, works perfect, thanks06:16
jmlbeuno: my pleasure.06:16
=== BasicPRO is now known as BasicOSX
mwhudsondoes upgrade work over bzr+ssh yet?06:27
lifelessno06:29
beunomwhudson, bzr branch lp:~beuno/loggerhead/zpt.ajax07:44
beunoand I'm off to bed07:44
beunoit still needs cleaning up, and a lot of work to do on top of it, so I'll keep working on that branch from now on07:45
beunoI replaced the deprecated methods, and updated the templates with the latest from your branch07:45
poolieok i'm preparing a beta...08:39
Frank_HeinenHi, can someone help me with a problem I got with a branch. I accidentally removed a file manually. Now I'd like to get the latest version of the branch again without checking out again, is this possible (getting back to the original branch again)? bzr update doesn't work, it mentions it is allready up to date.08:42
PengFrank_Heinen: What? Do you want bzr revert?08:49
tcaIs http://pastebin.org/39994 a safe workaround for https://bugs.launchpad.net/bzr/+bug/235055 ?09:04
ubottuLaunchpad bug 235055 in bzr "bzr log fails with KeyError in graph.get_parent_map" [Undecided,Confirmed]09:04
pooliehi tca09:05
tcahello09:05
TFKyletca: might want to do the graph.get_parent_map call outside of the try block so you couldn't get any exceptions from it, but probably isn't needed09:07
poolietca, i _think_ so09:07
poolietca, could you send it to bazaar@lists.canonical.com quoting that bug number?09:07
poolieit does look like this is the case that comment was looking for09:08
poolieit may be something unusal in the history coming from arch09:08
* tca subscribing to the list09:09
tcaYes, I think lots of weird stuff happened in arch ;-)09:11
awilkinsFrank_Heinen: bzr revert <my_file>09:18
Frank_HeinenPeng: Awilkins: tanks, lets try.09:23
Frank_HeinenPeng: Alwilkuns: It works! :-) Stupid that I didn't see this....09:26
jrydberglifeless: ping?09:33
=== pmezard_ is now known as pmezard
pooliejrydberg: he's feeling tired, probably won't be back for 12h10:10
poolieie tomorrow morning10:10
jrydbergi see.10:13
jrydbergthanks!10:13
jdubhey duderinos10:23
jdubwhat are the chances of having bzr-svn in the bzr ppa?10:24
jdubi can't follow bzr easily without it :|10:25
mwhudsonbzr get lp:bzr-svn ~/.bazaar/plugins/svn10:26
mwhudsonreleases are for weenies :)10:26
awilkinsAnyone here working on the xmlrpc server?11:43
=== kiko is now known as kiko-afk
=== _[PUPPETS]Gonzo is now known as [PUPPETS]Gonzo
semsliehi. I'm in a situation where I've updated, which has converted some commits to a pending merge. I'd prefer to go back and rebase now. Is it possible for me to convert those commits involved with the pending merge back into commits and basically rollback to before the update?13:07
james_wsemslie: I'm not sure of a way to get back to what you had before, I can talk you through getting back to an equivalent.13:12
james_wsemslie: I'm not sure if "bzr revert" will do what you want though, I think it may do the opposite of what you want, so I'm wary of telling you to try it.13:12
semsliejames_w: fortunately I've got a backup13:13
luksI'd unbind the checkout, commit the pending merges, then lookup the revid of the merged revision and branch of that13:13
semslieluks: thanks, I'll give that a try13:13
james_wluks: exactly13:13
luksafter that you should have the branch you had before and you can use rebase on those revisions13:13
james_wthough "pull --overwrite" instead of "branch" will save you a temporary branch.13:14
james_wand I was wrong before, that will get you back to what you had.13:14
luksI just don't like destructive operations13:14
lukshaving two branches feels safer to me13:14
semsliejames_w: when you say you were wrong before, do you mean that "bzr revert" wont delete the pending merge, but rather get me back to where I was before the commit?13:15
luksno, revert will get rid of your local commits13:15
luksthey are still in the repository, but not so easy to look up13:16
semslieluks: right, I'll avoid that then13:16
james_wsemslie: no, I meant that I was wrong when I said that I wasn't sure whether I could get you back to what you had before. I was going to suggest luks' method, but that will get you back to what you had before.13:16
semslieokay, I'll try that13:16
semslieftr I'm merging a fairly complicated branch back into a subversion rep that I've diverged from. It now makes sense that update did what it did, but I think what I was looking for was rebase13:18
semslierebasing seems to have worked nicely. nearly there now. Thanks for the advise!13:34
abentleylifeless: I don't think the mpdiff ghotst bug is a conceptual bug.  It's just a bug bug.14:07
visik7can I commit a filechange on a revision ?14:20
bob2you only want to commit the changes made to a paricular file?14:22
visik7the problem is that I've a working copy at a revision xy and the deploied app to a revision older14:25
visik7now I want to submit some modification to the deploied app14:25
visik7maybe I need to branch and revert14:27
vilavisik7: may you need two branches14:32
vilabe even14:32
visik7another question14:33
visik7I've a shared repo14:33
visik7at the beginning I've commited on it a lot of files14:33
visik7now I deleted those files from the working tree14:33
visik7and I don't want them anymore14:34
visik7but the shared repo still has it14:34
visik7has them14:34
visik7I can't find any command on how to manage a shared repo14:34
bob2that is unrelated to using a shared repository or not - bzr doesn't currently have an (easy) way to purge things from history14:36
visik7but it's 1gb of useless data14:36
visik7:(14:36
visik7and a way to purge a repo from a revision to the beginning ? I mean I've the problem at revision 714:40
visik7a way to remove from 1 to 714:40
andrea-bsI don't know if this is the best way, but you can make an another branch, "bzr revert -r1" the original branch, copy all needed files to the new branch and commit, "bzr revert -r 2", copy and commit, ...14:44
=== mw|out is now known as mw
awilkinsvisik7: Export r 8 ; initialize it, then merge revisions 9.. with it15:11
visik7my current revision is at 3715:18
visik7it's ok to do it ?15:19
mtaylorvisik7: try bzr pack15:24
visik7they are png15:25
=== asabil is now known as asabil_
PieterHmm, creating a bidirectional git-bzr thing should be really easy15:40
PieterI might even try it someday15:40
jelmerPieter, there already is something like that15:41
jelmerPieter, http://launchpad.net/bzr-git15:41
Pieterbut that won't write to git, right?15:41
jelmerPieter, not yet, no15:41
jelmerPieter, or is what you're intending to do different from a bzr plugin?15:41
PieterI was thinking about using fast-import/export15:42
PieterYou'd just need to extend git-fast-export and bzr fast-import to import/export marks files15:42
Pieterand then you can go both ways incrementally15:42
datoI'm planning to add incremental support to bzr-fast-export15:43
datoit already exports a marks file15:43
Pieterbzr-fast-export already has a --import-marks thing15:43
jelmerPieter, that won't work because you'd break referential integrity15:44
datoPieter: I know, I wrote it :)15:44
Pieterjelmer: you'll have to maintain the marks files, then it should be ok15:45
dato(though I'm not planning in bidirectional, no)15:45
dato(just incremental bzr -> git)15:45
Pieterthat already works15:45
semslieI'm constantly getting an AssertonError when pushing from a rebased bzr branch to a subversion repository: assert self._new_revision_id is None or self._new_revision_id == revid15:49
semsliehas anyone else had experience with this?15:49
Jc2ksemslie: do you mean you pushed once, rebased and pushed again>15:50
semslieJc2k: no, but after I branched the subversion branch diverged from the bzr one. Rather than merge and loose my changelog in a single subversion commit, I rebased off the new subversion head and tried to svn-push15:51
semslieinterestingly, the commit that should have been last (at the bzr head) was the only one that got committed to svn before it crashed15:53
semslieI wonder if it has somehow gotten turned around?15:53
jelmersemslie, hi15:54
jelmersemslie, there is an open bug report about this15:55
semsliejelmer: thanks. I'll take a look in launchpad15:55
semsliejelmer: is that a bug against bzr or bzr-svn?15:56
jelmersemslie, bzr-svn15:56
semsliethanks15:56
=== kiko-afk is now known as kiko
PeakerWorkhey, I used "bzr commit --local" when I had some connectivity problems, then I used "bzr commit" and it failed cause it is out of date. So I used "bzr update" (when I have some local changes) and it gave me a bazillion merge conflicts! Nothing needs merging, only I have changes, the remote tree was untouched the whole time16:21
PeakerWorkwhy is that?16:21
PeakerWorkI removed all the .THIS/.OTHER files (I took the .OTHER.moved files) and now I have lots of "Conflict adding file raytrace/Plane.hs.BASE.  Moved existing file to raytrace/Plane.hs.BASE.moved." conflict messages and I can't get rid of them16:22
PeakerWorkis this a bug or did I do something wrong?16:22
PeakerWorkIts pretty terrible :(   only revert worked to get rid of the conflicts so I manually re-applied the ~1~ backup files, and all directory tree changes had to be manually re-done..16:25
james_wPeakerWork: hi, it sounds like the file ids ended up different between the remote and local branches.16:28
PeakerWorkI think it happened to me before -- whenever I commit --local, later updates are trouble16:29
james_wif nothing happened on the remote side what did you do locally? was it just edit and commit, or did you add and remove any files?16:29
PeakerWorkdoes bzr have tests for "commit --local" followed by updates?16:29
PeakerWorkI renamed/moved some files, I don't think I added/removed anything16:30
KeybukI'm clearly missing something, because one of my most common "other branch" operations is damned hard to do with bzr16:31
james_wit does have tests, yes16:31
james_wPeakerWork: it may be the renames then, did you use "bzr mv"?16:31
Keybukthe operation is "what changed in this branch when compared to the mainline?"16:31
PeakerWorkjames_w, I think I consistently get merge conflicts for all modifications made with --local when I try to update later16:31
PeakerWorkjames_w, Yeah I did16:31
james_wKeybuk: hi. what changed as in diff, or revisions?16:31
Keybukjames_w: diff16:32
james_wKeybuk: "bzr diff -r ancestor:submit:" might be what you are looking for16:33
james_wor it may give you something completely different.16:33
Keybukbzr: ERROR: Not a branch: "/home/scott/tmp/sadmac/submit:/".16:33
james_woh wow16:34
james_wdoes "bzr diff -r submit: work?16:34
LarstiQor bzr diff --new path/to/branch16:34
Keybukjames_w: nope, it uses a totally random path16:34
Keybuk(also this appears to be require me to checkout the branch I want to look at ... which sucks)16:34
Keybukif I'm in my upstart branch16:35
Keybukand somebody gives me the URL to their branch of changes16:35
Keybukwhich is branched off my upstart branch, though probably an earlier revision16:35
Keybukwhat's the recipe for that?16:35
LarstiQbzr diff --new http://bazaar-vcs.org/bzr/bzr.dev16:35
LarstiQworks for me?16:35
james_whi LarstiQ16:36
LarstiQI can imagine cases where that is not the most readable diff.16:36
KeybukLarstiQ: shows the reversal of changes in local branch ?16:36
LarstiQKeybuk: diffs the other branch with the current one, you can also do --old16:36
LarstiQthat's just for left and right16:37
james_wKeybuk: there's another possibility, "bzr merge --preview URI"16:37
KeybukLarstiQ: which isn't what I want ... I want to see what _they_ changed16:37
Keybuknot what I changed since16:37
Keybukjames_w: has random side-effects like setting the submit branch16:37
LarstiQKeybuk: aha16:37
Keybukaha!16:37
Keybukjames_w: you were onto the right idea originally16:37
Keybukbzr diff -r ancestor:. OTHER_BRANCH16:37
Keybuk:p16:37
LarstiQif you are in their branch that is?16:38
james_wKeybuk: yeah, it should definitely be something that is possible. I think that "merge --preview" would be the suggested way to ask what they changed, but I'm not sure what to do about the submit branch thing.16:38
Keybukalso merge --preview doesn't work with difftools, so I can't use meld to see it ;)16:39
james_wif you haven't got a submit branch set then that may be why "-r submit:" didn't work.16:39
Keybukit's not me who needs to set that, no? :p16:39
LarstiQjames_w: hey :)16:39
Keybukit's if _they_ haven't got a submit branch? :P16:39
james_wyou did ask for your differences to the mainline first though, which I think is a "diff -r ancestor:" operation.16:39
james_wKeybuk: maybe you would like to post to the list so that we can get some discussion on this, it should definitely be something that is easy to do.16:40
PeakerWorkjames_w, does it sound like a bug though?16:41
Keybukwhat's the address?16:41
james_wKeybuk: bazaar@lists.canonical.com16:41
james_wPeakerWork: it does sound like one, though there may have been something that I missed.16:42
james_wPeakerWork: is it a public project?16:42
james_wPeakerWork: or better yet, can you isolate a test case on a couple of dummy branches?16:42
PeakerWorkjames_w, I'll try that at home yeah16:43
james_wPeakerWork: thanks. You can file a bug either way, but a simple test case will probably get it looked at far quicker.16:43
schierbeckabentley: hey dude16:53
abentleyschierbeck: Hey there.16:54
schierbeckabentley: it's nice to see the branch review functionality coming to lp -- do you have any idea when it will match the functionality of bundle buggy?16:54
schierbecki'm thinking of starting to use it in bzr-gtk16:55
abentleyschierbeck: They're never going to have the same feature set.16:55
KeybukYour mail to 'bazaar' with the subject16:57
Keybuk    Two things that bzr makes hard :-(16:57
KeybukIs being held until the list moderator can review it for approval.16:57
Keybuk:-(16:57
KeybukFAIL16:57
pickscrapeIt's because you're not subscribed I think.16:58
schierbeckabentley: in what way do you mean? i'm thinking: when will it allow users of bundle buggy to switch to lp without changing workflow16:58
schierbecki.e. reviewing over email must be supported16:58
abentleyschierbeck: I don't know.  What's important to you?16:59
schierbeckabentley: i know jelmer must have email support if he's to switch over -- personally, i only really need better integration with the code browser16:59
schierbeckso i can see a full diff17:00
abentleyEmail integration is what I'm working on right now.17:00
abentleyI'm not sure when we'll add the ability to view a diff.17:00
jelmerschierbeck, abentley: Voting support is another thing I think launchpads merge request integration doesn't have atm17:00
schierbeckjelmer, abentley: as far as i know it's just been introduced17:01
jelmerschierbeck: Also, is there any reason to switch to launchpad from bundlebuggy once these things have been resolved in launchpad?17:01
abentleyjelmer: No, it has voting.17:01
jelmerschierbeck, I prefer using free over non-free software when possible17:02
schierbeckjelmer: that's true, but if we disregard that very valid point, having the bugs, branches and review proposals in one place offers a very real benefit, at least for me17:02
schierbecki feel it would be a good idea to start linking bug reports to hosted branches, and using lp to request code reviews would mean that i would have one single location from which i could view the status of my work17:04
jelmerThe linking between bug reports and hosted branches should already be happening if you use --fixes17:05
abentleyschierbeck: So, that requires actually having branches.  BB doesn't require that, which gives you a lower barrier to entry.17:05
schierbeckabentley: that's true, but support for patches/bundles in lp merge proposals would be possible17:06
abentleyThat's true.  What's actually been proposed is that you could use a bundle to create a branch.17:10
=== yacc_ is now known as yacc
semsliejelmer: I've dug around a bit and it seems like the first revision to be pushed is off by one. So todo in branch.py line 360 is missing the first revision that needs to be pushed. Not sure if this helps, but I am rebasing before I push17:18
schierbeckabentley: that would be cool too, but i think the best solution would be to allow uploading bundles when making a merge proposal17:41
abentleyschierbeck: Well, supporting bundles as first-class objects rather than just a way of getting branches isn't on the radar, so please file a bug to help put it there.17:45
schierbeckabentley: sure17:46
schierbeckabentley: but would it technically be the same, if the internal representation is in fact a branch17:46
schierbeckit would just be anonymous to the lp user17:46
abentleyI don't know what you mean by "anonymous to the lp user".17:47
schierbeckabentley: i mean that all they see is a file chooser dialog, they don't need to first create the branch from a bundle, then point to it from the merge proposal17:48
schierbeckmany changes are small, and figuring out a unique name for it is perhaps overkill17:48
abentleyschierbeck: So you're saying accepting a bundle should create a branch without a name?17:49
schierbeckabentley: yeah, if done from the merge proposal form17:50
abentleyI highly doubt that will fly.17:50
schierbeckthe internal name could correspond to the merge proposal id17:50
schierbeckabentley: why wouldn't it?17:50
abentleyIf a branch doesn't have a name, it's hard to access it.17:51
=== kiko is now known as kiko-fud
schierbeckabentley: since they'd be read-only, they could be exposed through the review interface17:52
schierbecki'm not familiar with the url scheme, but something like /merge-proposals/<id>/branch17:52
abentleyConcretely, you're not going to be able to get a diff of a branch that doesn't have a name through the code browser.17:53
abentleyBecause you won't be able to construct a URL for that.17:53
=== yacc_ is now known as yacc
jelmerabentley, I'm trying to track down an error in bundlebuggy18:18
jelmer"freeze" for emails is used to store them somewhere else for reprocessing?18:18
jelmerone of the emails is raising UnicodeDecodeError and that is causing bundlebuggy to reprocess it over and over again18:19
jelmersince UnicodeDecodeError isn't handled in MailQueue.handle()18:20
abentleyfreeze is used for emails that have permanent failures.18:21
abentleyWhile UnicodeDecodeError is permanent, it shouldn't be happening at all.18:22
jelmerI've  made it print a traceback next time this happens18:27
nevanshas anyone ever had a problem committing to a bound branch, even though unbinding, committing, and pushing works just fine?18:31
jelmernevans, what sort of problems?18:31
nevansFWIW, I have this problems with a couple of subversion branches (using bzr-svn 0.4.9)18:31
nevanscommit tells me to update and try again, and update tells me that I'm up to date18:32
semsliejelmer: hi again. I've got an idea why I might be seeing #230863 (AssertionError on push -r x)18:33
jelmersemslie, ok18:33
jelmersemslie, what do you think is causing it?18:34
semsliejelmer: it looks like it is following the branch when choosing the next revision id, but there are revisions in the repository that are newer than in the branch, so we are choosing something that collides with something elsewhere in the repository.18:35
jelmersemslie: any chance you can write up a script that reproduces this?18:36
jelmerthat would be a huge help18:36
jelmerand would be useful to make sure this doesn't regress18:36
jelmeroncewe fix it18:36
semsliejelmer: sure18:37
jelmernevans, what does "bzr missing" say?18:39
nevansI'll try to reproduce it... I've got everything committed now.  ;-)18:39
=== kiko-fud is now known as kiko
jelmernevans, thanks18:56
abentleyjelmer: You should already have a traceback in the message it sends you.19:06
jelmerah, I didn't actually bother to look at those19:07
jelmerabentley, http://pastebin.ubuntu.com/16447/19:07
abentleyWell, that's cute.19:08
abentleyThe message claims to be utf-8, but ain't?19:08
abentleyWell, for this purpose, 'replace' should be fine.19:09
=== tchan1 is now known as tchan
antoranzHi, Guys!19:29
semsliejelmer: a friend and I have started a test script, and confirmed that when the head revision is on the target branch then a the push is successful19:31
jelmersemslie: can you attach that script to the bug report?19:31
semsliejelmer: sure, as soon as its finished I'll do that19:32
=== bigdo3 is now known as bigdog
semsliejelmer: adding  something to the target branch fixed the problem in practice, but we are finding that creating a new repository, then making some revisions such that the last revision is not to the target branch is not producing the error on our script. I'll post it if I can get it to reproduce the problem19:49
jelmersemslie, ah, ok - thanks19:49
=== andrea-bs_ is now known as andrea-bs
jelmerbeuno, bundlebuggy is up again20:28
beunojelmer, you rock  :)20:28
jamjelmer: so why have you decided to switch from pyrex back into plain 'C' wrappers, and why not just use the latest svn-1.5 bindings?20:55
jamJust curious on your rationale20:55
* fullermd has given up believing that svn 1.5 exists.20:55
datocan I expect the listing in TreeDelta.renamed to be comprehensive on directory renames?20:55
datohm20:56
jamdato: define "comprehensive"20:56
jamI believe it just lists the directories20:56
jamnot their children20:56
jamunless they have *also* been renamed underneath20:56
datowell, I'm confused, because in one bzr-export-case, I get both 'dir1 -> dir2', and then a full list of their children20:56
datoI get20:56
dato dir1 -> dir220:56
dato dir1/fileA -> dir2/fileA20:56
datoetc20:56
jamdato: I don't really know what your 'bzr-export-case' is, but generally you don't get that20:57
jamI'm not sure how you could20:57
datolet me see if I can reproduce from a python prompt20:58
jamI need to clean out my spam folder more often.... 15.3k spam messages20:58
datoah, you are right jam20:59
datoi was just blind; fileA gets renamed too21:00
jamwhat do you mean 'gets renamed too', shows up in the TreeDelta.renamed? Or just that it was physically moved?21:00
datoyes, that it was dir1/fileA -> dir2/fileB21:01
datoand dir1 -> dir221:01
jamright, *that* will show up21:01
jambecause we check if the 'parent' or the 'name' has changed21:01
datonow I'm not sure how to make git-fast-import happy about this21:01
jamdato: yeah, because IIRC it *doesn't* handle directory renames21:02
jamso, I think you have to iterate the children and give them new homes21:02
jamif kind == 'directory': iterate_children21:02
jamsomething like that21:02
pygipoolie, around? :)21:03
datojam: no, I think it handles them just fine, except this degenerate case in which fileA is the only file in dir121:04
jampygi: poolie is sleeping for a little while longer21:04
pygijam, oki ^_^21:04
jamI think he'll be around in maybe 2hrs or so21:04
jamcan I help?21:04
datojam: so if I rename the directory first, the file rename fails; and if I rename the file first, the dir rename fails because the dir is empty so it does not exist for git21:04
pygijam, lemme pm you :)21:05
jamdato: so do you need to track that the directory was renamed, and then rename the new location?21:05
datoI guess21:05
jamthat's... crummy21:05
=== emgent_ is now known as emgent
jam"I caught you naked bazaar-commits-owner"21:06
jamuh-oh, what has poolie been doing21:06
jamAt least it is obviously spam when it comes through the bazaar mailing list owner like that21:07
=== acuster is now known as noone
=== noone is now known as acuster
fdvjelmer: you mentioned something a few days ago about some memory leaks in the python-subversion bindings being fixed. If I recall correctly, this was not ported fully to hardy, but fixed someplace else. I'm not sure which packages are actually fixed (or having problems), is this something that's available anywhere?21:52
Jc2kfdv: subversion 1.5 has the fixes22:04
fdvJc2k:22:05
fdvwhoops22:05
fdvJc2k: they're nowhere else?22:05
Jc2kafaik that means you have some compiling to do :)22:05
fdvwell, compiling is no problem22:05
Jc2k1.5 hit rc822:05
Jc2kafaik that is their gold release22:06
Jc2kif all goes well22:06
fdvone can always hope22:06
fdvthen maybe I won't care so much for the python bindings anymore22:07
fdvsvnmerge is very cumbersome, and being able to use a bazaar mirror would likely be progress22:08
fdvbut when I try to import the repo, memory is soon exhausted22:09
fdvso basically, I need a bzr mirror while waiting for decent functionality in svn 1.5, but now I also need svn 1.5 to get a bzr mirror working :p22:10
Jc2kfdv: pretty much ;D22:12
Jc2kfdv: your branch much be special in some way...22:13
fdvmaybe22:13
Jc2ki converted 20gb worth of SVN without breaking the hardy python-subversion, 200 modules. A few weighed around 2gb22:14
fdvit's rather big, with lots of history (also from aeons of CVS) and I think they've omitted some conventions22:14
fdvhm22:14
Jc2khow many revisions?22:14
Jc2kevolution had about 23,00022:14
fdv~150k22:15
Jc2kyeah, that would explain it22:15
fdvbut I think that's only the svn revs22:15
Jc2k:P22:15
Jc2kevo only had 23,000 svn revs but still took 2gb of space on disk - maybe binary data?22:15
Jc2kfdv: are there lots of branches?22:16
fdvprobably22:16
fdvyep22:16
fdvplenty :p22:16
Jc2kbugger :)22:16
fdvyep :)22:16
Jc2kdo you need them all :D22:17
fdvthe svn-import doesn't rise above 0/n22:17
fdvwell, not at all22:17
fdvI only need trunk22:17
Jc2kwhen i did the initial GNOME mirror i only mirrored trunk of each module22:17
fdvbut afaics, svn-import won't work with branches22:17
Jc2kand then used svn-import to top them up22:17
fdvhow'd that work?22:17
Jc2khttp://live.gnome.org/JohnCarr/Bazaar22:18
Jc2kthat script creates a repository called $MODULE and then branches trunk of the SVN repo into $MODULE/trunk22:18
Jc2kit used bzr pull to fetch 1000 revs at a time, dodging the leaking22:19
fdvah :)22:19
fdvneat trick22:19
Jc2kthen when i'd done, i discovered svn-import22:20
Jc2k:p22:20
Jc2kso ran that on my mirror of trunk and it figured out what i was missing and started topping stuff up22:20
Jc2kit was pretty good about recovering from errors, too22:20
fdvbut that requires the whole repo, right?22:20
fdvI'm not sure I have understood this completely22:21
sabdflpoolie: 1.6rc1 today?22:21
Jc2kfdv: so svn-import works out what the branches are and then makes a bzr repository dir with multiple branch dirs22:22
Jc2kfdv: my bash script does that too, but only for trunk rather than all the branches22:22
fdvright22:22
Jc2kfdv: you could pull the import branches by hand...22:22
fdvI'll certainly give it a shot22:22
fdvwell, they're not interesting, really22:22
Jc2k*important branches, even22:23
fdvI just want a non-central repo I can commit to frequently22:23
fdv:)22:23
Jc2kfdv: the page i linked to also has my python script for updating the mirror on svn commit22:24
Jc2kthough its a tad specific to the GNOME svn-commits mailing list22:24
fdvI saw it, looks neat :)22:24
fdvwell, that could probably be tweaked, I guess22:25
=== abentle1 is now known as abentley
beunojelmer, btw, BB still won't let me vote via email, just through the web interface. I don't mind at all, but if you happen to trip over the DB and see something funny...  :)22:25
jamsabdfl: 1.6beta1 was today, 1.6rc1 is this Fri22:35
jamJc2k: well, starting with your script might be a good start, depending on if 'svn-import' is smart enough to work in 1k revision batches like yours is22:36
Jc2kjam: it doesn't, i know from locking up my server ;D22:37
pickscrapeIs there any chance someone well versed in bzrlib could give me some feedback on the diffstat patch I posted to the mailing list?22:37
pickscrapeSince it's my first work using bzrlib I'm uncertain as to whether I've made any monumental screwups or not. :)22:38
jampickscrape: this is the 'make it work with 1.5' ?22:38
pickscrapejam: yes22:38
jampickscrape: giving you feedback on the list22:44
pickscrapejam: thanks22:45
jampickscrape: basic summary, lock your objects, and reuse them as much as possible22:51
jamI sent a modified form to the list22:51
jamI think its right, but you should certainly test it to make sure22:51
pickscrapejam: thanks for that, much appreciated :)22:51
pooliesabdfl: no 1.6rc1 is for friday, beta1 was monday22:52
jamhi poolie, we on for a call in 10min or so?22:52
fdvJc2k: the last pull in the script, from FINAL_REP, what's the exact purpose of that?22:54
jelmerbeuno: I'll have a look at it next time I deal with bundlebuggy22:54
fdvis it so that the bzr repo there is set to "point" to the central svn repo?22:55
Jc2kfdv: in my case i was converting a local rsync'd copy of svn.gnome.org22:55
pickscrapejam: some very useful pointers there: thanks. I didn't actually know about locking at all.22:55
Jc2kso the FINAL_REP just points it at the http://svn.gnome.org/ instead of my local rsync'd copy22:55
jampickscrape: a lot of api's do it 'automagically' for you, but if you hold the lock, you get more caching22:55
pickscrapeThe method of parsing diff output is how the inherited code did it (__diff was also inherited), but these are things I could look at next.22:55
fdvright, then I got i correctly (I think) :)22:55
jamespecially helpful for multiple 'as_revision_id()' lookups22:55
jametc22:55
fdvJc2k: thanks!22:55
pickscrapeThe first priority was to just get it working with the current version of bzr :)22:56
beunojelmer, I keep getting emails telling me I'm not allowed to vote. It must be processing my votes through the web interface. Anyway, don't worry about it, just thought I'd mention it22:58
lifelessabentley: agreed; I hadn't fully got the layerings straight in my head23:32
timelyxhrm23:37
timelyxis "versionedfiles" a technical term?23:37
lifelesstimelyx: bzrlib.versionedfile is a module in the bzr code base23:42
lifelesstimelyx: versionedfiles is an upgraded api I am working on23:42
timelyxok, and it's spelled in lowercase?23:43
timelyxwould it be wrong for me to stick it in ``ticks`` ?23:43
lifelesstimelyx: context?23:45
timelyx    * New annotate merge (--merge-type=weave) implementation is fast on23:45
timelyx-     versionedfiles withough cached annotations, e.g. pack-0.92.23:45
timelyx+     versionedfiles without cached annotations, e.g. pack-0.92.23:45
timelyx      (Aaron Bentley)23:45
lifelessah, that is versionedfile pluralised23:45
timelyxgrr23:45
timelyxthat's just wrong then23:46
timelyxcan i do ``versionedfile``s?23:46
timelyxor /something/23:46
timelyxbecause your thing really makes what was written there amazingly ambiguous23:46
timelyxor just plain wrong, your choice :)23:46
lifelessI'm looking for a better name23:46
timelyxyeah, i saw23:46
lifelessI wouldn't go change all the existing things just yet :P23:46
timelyxanyway... suggestions? versionedfile's ?23:46
timelyxi'm fixing spelling, as you can see, that line's going to change either way23:47
lifelessversionedfiles is appropriate plural of versionedfile. versionedfile's would indicate ownership23:47
timelyxyeah, but people write url's and stuff23:47
lifelessthey are wrong23:47
timelyxit's not right, but it is common :/23:47
timelyxrather, it isn't right :(23:47
* timelyx cries23:48
timelyxanyway, would ''versionedfile''s be bad?23:48
timelyxor however that magic markup workls23:49
lifelessI think it is fine as it is, given the context and detail present23:50
timelyxok23:50
timelyx     * Add scenarios as a public attribute on the TestAdapter classes to allow23:51
timelyx       modification of the generated scenarios before adaption and easier23:51
timelyx       testing. (Robert Collins)23:51
timelyx+XXX this doesn't make sense23:51
timelyx"adaption" is generally not considered much of a word23:51
lifelessif you wish to refer to the class name more precisely it is ``bzrlib.versionedfile.VersionedFile``, and I would probably phrase it as 'on $above implementations' rather than using the plural ABC. But its not a problem as it is :)23:51
timelyxthe tail of that "sentence" doesn't work (before X and easier Y)23:52
lifelessto allow (modification of the generated scenarions before adaption) and (easier testing)23:52
lifelessAFAIK its ok english23:52
lifelessalso23:53
lifelesshttp://dictionary.reference.com/search?q=adaption23:53
timelyxyeah, i know... it's just that things like: http://www.merriam-webster.com/dictionary/adaption23:55
timelyxbasically say "not common, use X"23:55
lifelessit would be a boring world if we only used the most common ways to say something23:55
MarcoHello23:56
MarcoI'm doing bzr up on a branch I checked out from a launchpad project23:57
Marcoand it insists that it's up to date at revision 332 even though the most recent revision is 33523:57
lifelessMarco: bzr info23:57
lifelessMarco: if it does not say 'checkout of' then you made a new branch (e.g. bzr branch) rather than a checkout (e.g. bzr checkout)23:57
Marcois there a way to "convert" a new branch into a checkout?23:58
MarcoI haven't made any changes23:58
thumperif someone was looking to switch from CVS to Bazaar, what is the currently accepted "best converstion tool"?23:58
lifelessMarco: you can convert it to a checkout with 'bzr reconfigure  --checkout'23:58
timelyxlifeless: if i reordered it as to allow easier testing and modification...23:58
timelyxwould you object?23:58
lifelesstimelyx: generally I think we shouldn't alter NEWS that has been issued23:59
timelyxok23:59

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