/srv/irclogs.ubuntu.com/2010/06/04/#bzr.txt

spivThere's a bug open about that, https://bugs.launchpad.net/bzr/+bug/569361 I think.00:51
ubot5Launchpad bug 569361 in Bazaar "Remembering an URL different from the one the user entered is surprising (affected: 3, heat: 18)" [Wishlist,Confirmed]00:51
TresEquisAnybody seen bzr-svn create bogus tags when pushing to SVN after merging from another bzr-svn branch?03:54
spivTresEquis: I haven't, sounds like a good bug to file.04:10
TresEquisspiv: I would file it if I could figure out how to reproduce it04:19
TresEquishappened twice to me yesterday, but I'm not sure why /how04:19
TresEquisThis checkin was a push from a local bzr branch to the bzr-svn checkout :parent04:24
TresEquishttps://mail.zope.org/pipermail/checkins/2010-June/047200.html04:24
TresEquisfollowed immediately by creation of bogus tags:04:24
TresEquishttps://mail.zope.org/pipermail/checkins/2010-June/047201.html04:24
TresEquishttps://mail.zope.org/pipermail/checkins/2010-June/047202.html04:25
TresEquisetc04:25
spivHmm. :(04:25
spivDoes your local bzr branch have tags with those names in the 'bzr tags' output?04:25
TresEquisnone of those tags ever existed in that project, nor any such releases04:25
TresEquisspiv: hmm, I think I wiped out the local checkout ;(04:25
TresEquisyup ;(04:26
TresEquisI have a half-baked theory that bzr might have been confused by an attempt to merge an unrelated branch04:27
TresEquiswhich didn't actually make any changes in my working tree04:27
spivOh, yes, that does sound plausible off the top of my head.04:27
TresEquisbut maybe it messed up the tags bookkeeping?04:27
* spiv takes a peek at the code04:27
* TresEquis crosses his fingers04:28
spivHmm.04:28
spivYes, that does seem plausible, if you managed to get bzr to go as far as fetching revisions from the other branch into this one.04:29
TresEquisit seemed to try, then give up claiming "no common revisions" or something04:30
spivHmm, that doesn't sound right then.04:31
spivI'm pretty sure tags are only updated after fetching the actual revision objects, and that can only happen after determining the revisions to fetch, which is when that error would happen.04:32
spivIt's possible there's a code path I'm not noticing, of course.04:32
TresEquisif it matters, the two trees both point back to the same SVN repository04:32
TresEquisjust to different project areas in it04:32
spivI wouldn't expect that to matter, directly.  Well, that's presumably what allows bzr to know that it can even create those tags (because the revisions from the other repo are there), but doesn't address the mystery of why bzr/bzr-svn is mixing tags from an unrelated project.04:35
TresEquisall the branches (bzr-svn checkout, local branc) were inside a shared repo, if that matters04:35
TresEquisanyway, I will post a bug with as much information as I have04:40
TresEquiseven though reproducing it seems wicked04:40
spivThat would be good, I suspect jelmer may be able to guess the bug :)04:41
spivI don't suppose anyone knows much about configuring apache to use active directory?  If you do, https://answers.launchpad.net/bzr/+question/113185 could use your help.04:45
TresEquisPosted here: https://bugs.launchpad.net/bzr-svn/+bug/58951404:46
ubot5Launchpad bug 589514 in Bazaar Subversion Plugin "Bogus tags created after failed merge attempt (affected: 1, heat: 6)" [Undecided,New]04:46
TresEquisThanks for the sounding board, spiv04:54
pooliespiv, thanks for the import test and patch05:19
lifelessspiv: poolie: you guys might benefit by lurking in #ubuntu-devel, amongst other channels. Just had a query about udd reliability there - I don't think most folk having bzr related things go wrong will think to come here in the first instance.05:21
spivHmm, I'm usually in there.  I need to poke irssi harder to make that stick I guess.05:22
poolieyeah likewise05:36
pooliethanks for the reminder and for catching it05:36
spivIt's very wet today.05:45
pooliemm it is05:45
lifelessuhuh05:45
=== oubiwann is now known as oubiwann_
parthmpoolie, lifeless: regarding ignore handling. while i prefer 'fail early' i am ok with warning for 'ignore', 'ignored', 'ls' and 'st' as these command don't change anything.05:52
parthmdoing a warning for 'add' seems scary to me as the user might end up adding unwanted stuff to the repo.05:53
parthmhe could revert but then he may not notice and just add more revisions.05:53
parthmwe have seen tickets like thing regarding whoami guessing and .moved files being added.05:54
parthmi was hoping to catch jam but i guess its late in his timezone.05:54
pooliei think if you explicitly try to add an invalid file that should certainly fail05:55
lifelesspoolie: spiv: check my facebook feed - not wet today :)05:56
mwhudsonit's been very nice here today too05:56
lifelesspoolie: not invalid file; invalid .bzrignore making the regex compilation fail05:56
mwhudsonyou have better mountains though05:56
lifelessyou're in a mountain!05:57
pooliei think one interesting case is having a file with a bizarre name generated by a build process that you can't change05:57
poolieah sorry, different bug05:57
parthmpoolie: i am thinking if user does 'bzr add foo', it should work irrespective of bad ignore pattern. however a 'bzr add' should fail if there is a bad pattern in ignore files.05:57
pooliein that case i think it's within the person's power to fix it before doing anything else?05:57
lifelesspoolie: certainly bad files in the tree shouldn't stop bzr being usable as long as they aren't versioned05:57
lifelesspoolie: well thats what I've been arguing05:58
poolieparthm, right with 'add foo' (foo not a directory) we shouldn't need to even think about ignores05:58
pooliewell that's nice we agree then :)05:58
lifelessjam and vila have expressed an interest in just warning and not stopping05:58
lifelesspoolie: so you and I agree, jam and vila appear to agree with each other ;P05:58
poolieand they're both asleep...05:59
parthmpoolie: if a user is hoping to ignore 'x.zz' and it gets added. he may not notice and end up adding a few more revisions.05:59
poolie... so ... _-)05:59
lifelessthere is discussion in the bug05:59
lifelesspersonally, I've said my bit - and any improvement over a backtrace is an improvement05:59
lifelessso, I'm -> loom stuff for a bit more, then EOD06:00
poolieincremental improvements and working code generally get my approval06:00
lifelessparthm: what time is it for you, now ?06:00
poolieok06:00
pooliei'm going to concentrate on reviews for a bit06:00
parthmits 10:30 am ... i am planning to start working on a new patch for the ignore pattern issue.06:01
lifelessyou should look at my scenery porn first :)06:01
lifelesspoolie: ^06:01
* poolie coughs06:01
pooliethat is very pretty though06:01
lifelesshmm, facebook seem to discard lots of resolution.06:02
poolieinteresting choice of foreground06:02
lifelessit *was* a 4K by 3K06:02
poolieoh from what camera?06:02
lifelessI have some others without anything but water in foreground06:02
lifelesscoolpix s800006:03
lifelesshas a 10x optical zoom, which is pretty nice06:03
lifelessnothing compared to your beasty06:03
lifelessbut I wanted something I can hold for extended periods06:03
lifelessthe other ones though aren't as dramatic on the mountains06:04
lifelessand I thought the prosaic foreground item was fun06:04
mgzmorning all.06:09
pooliehi mgz06:12
pooliespiv, btw, you're pilot next week06:13
pooliei'm going to empty it a bit today if possible06:13
spivThanks :)06:13
pooliemgz, thanks for your review of the walkdirs patch06:35
mgzhappened to be something I'd just been looking at from the other end in testtools, and before that with vila in bazaar.tests._rmtree_temp_dir06:36
mgzif we've got a function that wants to guarentee unicode (or utf-8) outputs, and has a posix filesystem as its input, something has to give06:37
mgzthrowing a clear error like in the merge proposal is an improvement06:39
poolieyes06:39
mgzah, that's what I was doing, commenting on one of you bugs about this06:45
* mgz finishes typing that06:45
pooliei wonder if i have some still unpushed developer docs about handling unicode06:51
pooliewe could try to explain the general approach more06:52
mgzdid I write anything for my nonc bits...06:53
mgzhm, just:06:53
mgz+No API should take '``str`` or ``unicode``' expecting to use ``isinstance`` to06:53
mgz+tell them apart. Text that may need to include non-ascii characters on some06:53
mgz+platforms, such as filenames or error messages, should be decoded to unicode06:53
mgz+at the earliest opportunity. Much of this will also need looking at carefully06:53
mgz+when considering a py3k port.06:53
mgzwhich isn't all that useful.06:54
pooliemgz, "use instance to tell them apart ... without a very good excuse" :-)07:00
pooliesuch as coping with a platform api that can return either07:00
mgzright.07:00
mgzwe have to cope with such problems, don't want to propogate them07:00
pooliei've been meaning to write a tool that auto-fixes filenames07:02
poolieby guessing what encoding they're in and changing them to utf-807:02
pooliethere are libraries to do the guessing07:02
pooliemaybe such a tool exists07:02
mgzalso foreign branches, I filed a bug against bzr-git the other day while trying to track down a different issue07:03
mgza sensible guess is the right thing sometimes.07:03
poolielifeless, so what did we finally agree about sphinx in pqm? we'll try to backport it?07:04
mgzbug 587074 it twath.07:04
ubot5Launchpad bug 587074 in Bazaar Git Plugin "Branching tree with non-ascii filenames fails (affected: 1, heat: 6)" [Medium,Triaged] https://launchpad.net/bugs/58707407:04
vilahi all07:11
parthmhi vila07:13
pooliehi there vila07:13
vila\o/07:13
poolieare your machines feeling better?07:13
vilayes, and an additional disk for backups is on its way07:14
vilaOf course I had a kernel panic on OSX yesterday just to keep me on my toes, but I ignored it because enough is enough :)07:15
pooliegood idea :)07:15
pooliei'm still waiting for my rebuilt machine07:15
vilapoolie: errr, when did it broke ?07:16
pooliewow, two weeks ago now07:16
pooliebut it took a while to conclude what was wrong07:16
vilawow, I was lucky enough to get mine so quickly then07:17
vilayeah, the diagnosis for mine went a bit fast because it was locking quite fast07:18
vilabut in retrospect, I'm now convinced that the failure started manifesting itself long ago, but only once in a while (like once a month)07:18
vilaI'm toying with the idea of using a remote log facility for cases like that (fullermd mentioned raid but I can't do that on the imac)07:19
vilaBut that's low priority, backups are better :)07:20
poolieso vila if we started testing python2.4 only in babune, do you think that would be fairly safe?07:21
vilaSure, let me check hardy's default version07:21
vilaUnfortunately lucid doesn't support 2.4 anymore...07:22
mgzone concern I've got about that is Vincent's too good at just fixing things. when pqm fails, the patch submitter sees the fallout so there's some community learning about what isn't 2.4 safe.07:23
pooliethat's true07:23
vilahmm, 2.5 is the default07:23
poolieyes07:23
vilahehe don't make me blush,07:23
pooliehowever we could make him put coding-style additions up when he fixes something07:24
vilaThe thing is it's still faster to fix things than finding the time to configure email sending on errors07:24
pooliei think fixing them is fine07:25
pooliebut it should be visible somehow07:25
vilaso hardy's default is 2.5, I can either add a specific run for 2.4 or just force 2.4 on hardy (but that feels wrong since it's not the default config used by hardy's users and that's wht I want to test primarily)07:25
vilaI try to at least file bugs with a babune tags when I fix such things07:26
vilas/tags/tag/07:26
poolieif you put up a mp adding to doc/developers/coding-style.txt and say why you need the addition07:26
pooliethat will help07:26
pooliesomeone did that07:26
poolierecently07:26
pooliei'd care more about getting 2.4 on hardy than i would about retesting 2.5 on both hardy and jaunty07:27
vilaha no, the best solution for python2.4 is to add a centos/redhat slave07:27
pooliewhat's 'selftest-matrix' again?07:27
poolieoh that would be better07:27
pooliei'm not paying for a RHEL licence though :)07:27
vilathe old way to run on all slaves which was confusing as all tests were consolidated there07:28
vilaI will delete it once I get a blue babune including windows07:28
vilaIt's unfortunate that the default view is the 'all' one which by definition includes all jobs whether they are relevant or not (debug-* for example have a specific focus by definition)07:29
vilaAnd by the way, there are tooltips on the colored icons07:29
vilahmm, no example there, but grey can mean either disabled or aborted07:30
vilaselftest-matrix is disabled because it's obsolete07:30
mgzand here I was thinking you were testing bzr on human batteries07:31
vilatemplate-selftest is disabled because it's not intended to be run but is used by selftest-<slave> to keep some definitions unique07:31
vilamgz: what did I tell you about rule #1 ?07:31
mgzthere's no such thing as babune?07:32
mgzor was that rule #2...07:32
vilathre is no such thing as human batteries powered selftest :)07:36
=== radoe_ is now known as radoe
lifelesspoolie: I don't know what the consensus was08:11
poolieif you're talking about sphinx then i think it was that we'd try the backport08:18
spivThat was my impression too.  I don't recall anyone objecting to that suggestion, at least.08:26
poolielifeless, i don't want to be snarky but pqm failures now seem unreadable08:45
pooliesince they lose the crlf distinction08:45
pooliei guess i should write an alternate encoding08:45
mgzsubunit thing? what kind of unreadable?08:49
mgzI guess I need to try out my new powers and see what sort of mail sends soonish.08:50
mgzanyway, if it's because of \r in test output I might squash that at the testtools level08:50
mgzbecause it's a good way to break terminal output too.08:51
mgz>>> raise ValueError("\rgotcha!")08:52
mgzTraceback (most recent call last):08:52
mgz  File "<stdin>", line 1, in ?08:52
mgzgotcha!ror:08:52
lifelesspoolie: the fix is coded, see the lp rt queue for the request to deploy it09:04
lifelesspoolie: (fix here meaning to send as an attachment)09:05
lifelessmgz: there is a bug open on subunit to use a more robust framing09:05
mgzright, but my point was random control characters are bad anyway09:05
poolielifeless, that's nice09:07
pooliethis is rt39555?09:08
pooliewell, obviously09:08
lifelesspoolie: something like that, I can look it up if needed, but it should be pretty obvious09:09
lifelesspoolie: aaron wrote a sketch for me @ UDS, I polished and prepped, needs deployment09:14
poolieok, i bumped it up for attention09:15
lifelessa better protocol would be nice09:17
lifelessso would a summary in the body of the mail; but I figured getting the terrible bahviour sorted was the high priority09:17
bignoseThe ‘bzr-buildpackage’ tool is capable of doing the equivalent of an export, but taking the current uncommitted files from the working tree.09:18
bignoseis that a capability of the export command?09:18
bignoseI want to be able to export the current what-would-be-committed working tree state.09:18
pooliethere was a bug for that09:19
pooliei think someone fixed it?09:19
pooliebut maybe not09:19
bignose$ bzr version | head -n 109:20
bignoseBazaar (bzr) 2.1.109:20
bignose$ bzr export --help | grep uncommitted09:20
bignose$09:20
pooliei might sign off soon09:28
lifelessmgz: so09:49
lifelessmgz: matchers; did you look at the exception matcher in testrepository ?09:49
vilaurgh argh eeerk, ssl fingerprint do not match wth !09:54
vilafetchmail: despite my thanks for your wonderful support all those years: I hate you!09:54
lifelessayan: hi10:07
lifelessayan: what timezone are you in ?10:08
lifelesswell gnight y'all10:46
mgzdarn, came back too late10:58
mgzwill have to catch lifeless in another cycle.10:59
jelmermgz: 'morning Martin11:01
mgzmorning jelmer.11:01
parthmwhat does the '?' indicate in 'bzr status'? i am getting a status of '?  .bzrignore' for a test output.11:31
jelmerparthm: unknown file (not versioned, not ignored)11:32
parthmjelmer: thanks :)11:32
bignoseparthm: ‘bzr help status-flags’11:34
parthmbignore: neat. i didn't know about that.11:35
parthmsorry. i meant bignose :P11:35
bignoseyou may also want to know about nick completion in your IRC client :-)11:35
parthmbignose: yes. i think i should start using it :)11:35
parthmyup. it works :)11:36
bignoseheh11:36
bignoseparthm: for more interesting topics, ‘bzr help topics’11:36
parthmbignose: yes. i have used that one. somehow i missed status-flags.11:37
parthmthanks.11:37
peitschiehi everyone11:44
peitschiedoes anyone know if there is an official policy for how bzr deals with ghost revisions?11:45
jelmerhi peitschie11:46
jelmerpeitschie: Can you expand? Ghost revisions are just there (or rather, they aren't ;-)11:46
peitschiehi jelmer :)11:46
peitschiei'm chasing up 3 crashes that happen in qbzr tools... all related to ghost revs11:47
peitschiemost of the time bzrlib throws errors if the revision doesnt exist11:47
peitschiein some cases i think this behaviour is incorrect (e.g., in repository.py def _get_revisions(self, revision_ids) for example)11:48
peitschiebut just wondering if there was any disctussion about better ways of dealing with these?11:48
peitschiein some cases... qbzr can't possibly work around the iszsue, because it is calling a low-level bzrlib call with a bunch of revisions, which may or may not be ghosts11:49
jelmerpeitschie: in general the policy is to raise exceptions when you ask for a single revision that doesn't exist11:49
jelmerpeitschie: for calls that return multiple revisions such as get_revisions() it should probably either yield None or ignore the revision altogether if it's not present11:49
jelmerpeitschie: you shouldn't be calling _get_revisions directly though11:50
peitschiejelmer: sorry... ur right... it is calling get_revisions :)11:50
peitschiejelmer: would it be bad in your opinion to return a partially filled Revision object with a special "missing=True' property?11:55
jelmerpeitschie: In that case I'd opt for a GhostRevision object or something like that11:58
peitschiejelmer: ah..  That's a good idea!  I'll introduce one of those11:59
=== Meths_ is now known as Meths
=== nailuj24_ is now known as nailuj24
amanicahas any body seen the following? (google doesn't know anything about it):  ErrorFromSmartServer: Error received from smart server: ('error', 'Internal check failed: Cannot add revision(s) to repository: missing chk node(s) for id_to_entry maps')14:01
amanicaI did a reconcile on the server but that didn't help :(14:03
peitschieamanica: are you using bzr-svn?14:14
amanicanot in this case14:14
amanicado you think the plugin interferes?14:14
peitschieamanica: not unless you're mirroring from an svn repo :).  how many revs in your repo?14:15
amanicain my branch 200, in my repo much more14:16
amanicaI'm gonna try to upgrade the bzr on the server from 2.0.1 to 2.1.2 maybe that helps..14:16
mgzhey marius!14:17
mgzhow are you?14:17
amanicahey buddy mgz, I'm good thanks and you?14:17
mgznot too bad, sun is shining and lunch awaits.14:18
amanicaI'm trying to get some work done, but things just keeps getting in my way :(14:18
amanicamgz: btw. congrats with being granted pqm access14:27
bignoseBazaar offers me a command to break a lock, but using the command doesn't work.14:31
bignose“If you're sure that it's not being modified, use bzr break-lock chroot-13269072:///bzr/collab-maint/comixcursors/comixcursors.debian/.bzr/branch/lock14:31
bignosebut:14:31
bignose$ bzr break-lock chroot-13269072:///bzr/collab-maint/comixcursors/comixcursors.debian/.bzr/branch/lock14:31
bignosebzr: ERROR: Unsupported protocol for url "chroot-13269072:///bzr/collab-maint/comixcursors/comixcursors.debian/.bzr/branch/lock"14:31
bignoseso presumably Bazaar should either gain the ability to do what it's offering to do, or not offer it.14:32
jelmerbignose: please file a bug14:33
mgzamanica: I have yet to try it out though, need to get one of your merge requests ready to land so I can break it :)14:33
mgzbignose: this is on launchpad, right?14:34
bignoseI don't (yet) have a Launchpad account, I'd appreciate someone filing it on my behalf.14:34
bignosemgz: no, on Alioth14:34
amanicamgz: cool, will do :)14:34
mgzoh. you can still un-root the path to something you can access right?14:35
mgzI think I did that when I had a similar issue on launchpad14:35
mgz...or maybe I deleted the branch and pushed again, don't recall.14:35
bignoseI can work around the problem. but presumably the lock will remain until broken.14:35
bignosethe bug is rather that the error message suggests a command that will never work.14:35
mgzwhat I mean is, can you give break-lock that url in a form it understands?14:35
mgzI'm not disputing that the message is a bug.14:36
jelmerbignose: try using ' bzr break-lock' on the URL you originally used to access the branch14:36
bignosemgz: hmm. you're suggesting that the problem is the error message is displaying the wrong URL, and a correct URL could be used?14:36
mgzright.14:36
bignosejelmer: does it need to go all the way to the ‘…/.bzr/branch/lock’ file?14:36
jelmerbignose: no, that's not necessary (both should work)14:37
bignoseor could I just refer to ‘:bound’?14:37
jelmerperhaps, I'm not sure14:38
jelmerworth a try :-)14:38
bignose(the reason I'm asking, rather than trying it, is that I've already broken the lock by SSH onto the remote machine. so I can't test these questions.)14:38
jelmerI wasn't even aware the smart server over plain tcp worked on alioth these days..14:38
bignose“plain TCP”? it's over SSH, I'm not sure if that's different from what you're saying.14:39
jelmerit's different, I meant bzr://14:39
jelmeras opposed to bzr+ssh://14:39
bignoseokay. I've never tried the plain TCP protocol on Alioth.14:40
bignosenope, Alioth doesn't provide Bazaar plain TCP protocol.14:44
=== IslandUsurper is now known as IslandUsurperAFK
shrini1team16:52
shrini1 is there any way to rescue the disconnected "bzr branch" command?16:52
shrini1i have disconnecting net onnection16:52
maxbI don't think it's possible to resume bzr branch17:11
beunoshrini1, use a shared repositoru17:11
beunorepository even17:11
beunoit will save the revisions to it, and will continue branching just from the ones it doesn't have17:12
=== IslandUsurperAFK is now known as IslandUsurper
=== ericmoritz\0 is now known as Guest33830
csgeekhow would I checkout out a particular revision of a bzr repo? (I've already pulled the branch and should have all the history )20:46
lifelesscsgeek: bzr revert -r xxx20:53
lifelessor if you want to make a new branch at an old revision, bzr branch -r xxx . ../b220:53
csgeekthanks20:55
csgeeknah.. just wanted to look at a particular revision.  Just need to confirm something.  Thank you20:55

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