/srv/irclogs.ubuntu.com/2010/12/01/#bzr.txt

abentleypoolie: by "relevant plugins" do you mean ones that do iter_changes on PreviewTrees, or ones that might do RevisionTree.is_executable, or...?00:02
poolieim' not sure if it will be easy to identify which ones could be affected00:04
pooliewe could guess based on the apis00:04
poolieor just test the ones that are packaged00:04
abentleypoolie: I don't think it will be easy.00:04
abentleypoolie: Okay, I'll go find out what's packaged and make sure I've got it all installed, and then run the plugin tests.00:05
pooliedo you think this is too laborious and we should just land it?00:06
pooliei think it would be worth at least just running them00:06
poolieif it turns out they're too noisy, then we could decide what to do00:06
abentleypoolie: I think I can run them and delta the test failures against vanila 2.2.00:11
pooliethat sounds good to me if that works for you00:11
poolieso to keep this under control as a contract00:21
pooliei think i should mark out what we want in each particular future release00:21
pooliewould that work for youL00:21
pooliemkanat: actually just generally, feel free to request review from me, jam, spiv, or all of ~bzr01:18
mkanatpoolie: Okay.01:19
mkanatmwhudson: If I make the change about the None, do you want me to re-request review or just go ahead and merge it after doing it?01:34
mwhudsonmkanat: if you're confident in your change, just merge it01:35
mkanatmwhudson: Okay.01:35
mkanatmwhudson: I'm also wondering if there might be an off-by-one error in the contents array, so that it has an extra line that it shouldn't have.01:36
mwhudsonseems a bit unlikely somehow01:38
mkanatmwhudson: Well, I think that annotate_iter may not consider a final "\n" to be adding an extra line.01:39
mwhudsonoh right maybe01:40
mkanatSo I suppose I could just have the generator create an infinite number of empty containers at the end.01:41
mkanatSince we iterate over the contents array now instead.01:41
mwhudsonthat works i guess01:41
mkanatI think it'd be the simplest solution, code-wise.01:41
mkanatWell, launchpad outage stops me from doing any more development or merging. :-(01:46
jdobrienis there an issue with launchpad right now?01:54
mwhudsonjdobrien: one of the data centres is offline01:55
jdobrienmwhudson, aha! that would answer it01:55
jdobrienmwhudson, I hope it was planned :o01:56
AfCmkanat: why would the lack of availability of a centralized web resource have anything to do with you being able to get work done with a distributed revision control system like Bazaar?02:01
abentleypoolie: I've run all plugin tests, and none fail with is_executable that pass with 2.2.  Actually, an extra fails with 2.2, so I assume it's a flaky test.02:41
mkanatAfC: Because my code exists only on LP right now, I reverted my patch locally.02:50
poolieabentley: nice03:10
abentleypoolie: shall I go ahead and land it on 2.2, then?03:11
poolieplease do03:11
abentleypoolie: btw, I looked up who committed this code that's causing me grief.  It was me.03:14
pooliehaha :)03:14
pooliethanks for persisting with fixing it03:14
abentleypoolie: no worries.03:16
pooliemkanat: reading your patch now05:02
mkanatpoolie: Okay.05:20
pooliedone!05:20
pooliegood to go05:20
mkanatpoolie: Thanks! Good timing by me. :-)05:20
mkanatpoolie: BTW, I'm on the launchpad-devel mailing list still, so I do get the messages even if they're not CC'ed to me. (I usually just watch for subjects about loggerhead or codebrowse.)05:27
mkanatpoolie: There is a test framework in place, BTW.05:27
pooliek05:28
poolie... but it's hard to use for this?05:28
mkanatpoolie: No, not that hard, I could do it. But I think it mostly already covers this, since annotate is already tested...I believe.05:29
mkanatYeah, annotate is already tested.05:29
poolieit seems like if you're changing the default view05:29
poolieand the tests don't fail05:29
pooliethere must be a gap in coverage?05:29
mkanatpoolie: I haven't run the tests, because they're failing anyway. :-)05:29
pooliereally, just generally? :/05:31
mkanatpoolie: Well, they were last time I ran them.05:31
poolieto the extent that it's not worth adding a new one?05:35
mkanatpoolie: Ah, no, it could be worth adding a new one. Let me get the tests running on my new system (I just upgraded my OS and am missing some packages) and I'll see how much work it would be.05:35
pooliewell, it could be worth a try05:39
mkanatI'm not quite figuring out why the tests won't run on my machine. Maybe it's some python 2.7 thing.05:42
pooliewhat os do you use?05:44
spivmkanat: hmm, IIRC bzr's tests are fine on 2.7 now.05:44
mkanatpoolie: Fedora05:44
mkanatspiv: Yeah, but these are loggerhead's tests.05:44
mkanatI seem to recall just being able to "nosetests" on python 2.6 and things would be fine.05:45
mkanatjam, mwhudson: Do you know what it takes to get the loggerhead tests to run right?05:48
jammkanat: at one point I tried to integrate it with "bzr selftest -s bp.loggerhead"05:48
mkanatjam: Ohh, right. Since then, I think the tests have been broken with nosetests.05:49
mkanatjam: So I have to have it symlinked in as a plugin to do that, right?05:49
jamsomething like that, yes05:49
mkanatAttributeError: 'module' object has no attribute '_WritelnDecorator'05:50
mkanatAre bzr's tests only working with python 2.7 on trunk?05:50
mkanatI have 2.2.1 installed.05:51
pooliemkanat: probably yes05:52
mkanatOkay.05:52
pooliethere's no py26 in fedora 14(?)05:52
mkanatpoolie: Right.05:53
spivI run bzr trunk's tests with 2.6 all the time.05:56
spivPerhaps you need a newer testtools or something like that?05:56
mkanatspiv: No, I just easy_install'ed it.05:57
mkanatThis is the traceback: http://pastie.org/133754605:57
mkanatIt's pretty apparent that it's just related to the unittest changes in 2.7.05:58
pooliemm06:00
poolieyou might be able to cherrypick-back the changes from 2.3 to fix it?06:00
poolieor perhaps testing it against bzr 2.3 is still useful06:00
mkanatAh, well, I don't imagine LP runs 2.3.06:01
mkanatI'll have to figure out how to make these tests work with nosetests again.06:02
mkanatWow, how is this SO FAST? Probably because of crazy custom Hg implementation. http://code.google.com/p/python-nose/source/list06:09
pooliethat's what we should aim for06:15
mkanatYeah.06:16
pooliemight need more than little slices of time though :/06:16
mkanatI think we'd have to actually cache the rendered HTML to get that fast.06:16
mkanatBut maybe not.06:16
poolieperhaps an intermediate representation is cached06:16
pooliethey're probably not touching a magnetic disk to have it consistently that fast06:16
lifelessthat sort of speed is what cassandra is aimed at06:17
lifeless- its a bigtable backend for hg06:17
mkanatpoolie: Well, if it's anything like their Svn implementation, it's a mapreduce thing with, yeah, bigtable.06:17
mkanatFirst off we'd have to figure out what and how to cache things.06:19
mkanatAlthough if you had a cassandra backend for bzr, we may not even have to cache them, I don't know for sure.06:19
lifelessyou'd denormalise things like revnos06:20
lifelessin a sense such a thing is a cache06:21
mkanatRight, we already have a revno cache in loggerhead.06:21
lifelessif we can finish the 'does not need vfs for regular use' work, we could seriously consider a cassandra impl06:22
lifelessfailing that we could push (thus limiting the operations that take place) into a ro cassandra mirror06:22
poolieusing hpss to talk to a process that talks to cassandra?06:22
poolieoh nm, yes06:23
lifelessyes, exactly.06:23
mkanatjam: Do you know why your changes would have caused this (when trying to run the loggerhead tests via nosetests)? http://pastie.org/133758406:25
jammkanat: not specifically, but I know that loggerhead doesn't know whether it is a plugin or a main script, and does some ugly hacks with the python import path to try and work both ways06:26
mkanatjam: Ah, yeah.06:26
mkanatpoolie: It's probably going to take me quite some time to get the test suite working again, just as far as figuring out what's wrong and making things work right again. Do you still want me to add a test for the new controller?06:30
spivmkanat: I'm not poolie, but I think investing in the test suite will pay off06:31
mkanatspiv: I fully and completely agree with you; this is more of a "how does Canonical want me to spend my hours" sort of a deal. :-)06:31
pooliemkanat: let's not block this merge on that, but i do think it would be worth spending some canonical time on it06:32
mkanatpoolie: Okay.06:32
mkanatSounds good.06:32
poolieit seems like it will help do other work in future06:32
mkanatpoolie: Okay, agreed.06:33
lifelessit would be good to switch it to using testtools.run rather than nosetests06:34
lifelesswhich will give it easy hudson integration via subunit.run06:34
mkanatlifeless: It can already run under bzr selftest, I just can't run bzr selftest using my version of bzr and my version of Python.06:34
lifelessmkanat: why not? actually scratch that question. Different one: perhaps fixing that is more important.06:35
mkanatlifeless: Supposedly that's been fixed on trunk, just not in 2.2.x.06:35
mkanatlifeless: (This is with python 2.7.)06:35
mkanatSo, TAL is not a very good template language. :-(07:36
bob2the chameleon implementation is nicer07:38
bob2gives you $foo for when you're feeling lazy07:38
mkanatYeah, it's not so much the convenience of not of the language, as much as it is that it's just very difficult to work out how to do certain things.07:40
mkanatParticularly if you can't wrap an element in another element, or don't want to.07:40
mkanatI really liked Mako last time I used it, that's probably what I'd prefer, although I have no idea if there are any serious problems with it in production (or if it's still actively maintained).07:41
vilahello all !07:46
vilafull red babune07:46
vilaI upgraded to testtools trunk yesterday, direct link, url: http://babune.ladeuil.net:24842/job/selftest-maverick/lastFailedBuild/07:47
vilaNice tests as only reading their names strongly suggests a shallow problem07:48
mkanatI could do template inheritance with my patch that I'm working on right now, too, if I was using Mako instead. :-/07:56
vilalifeless, jml: bug #683505 rings a bell regarding a recent change in testtools ?08:05
ubot5Launchpad bug 683505 in Bazaar "bzrlib.tests.test_selftest.TestSubunitLogDetails failures with testtools trunk (affected: 1, heat: 6)" [High,Confirmed] https://launchpad.net/bugs/68350508:05
vilamgz: may also be related to a too aggressive cleanup of the test details ? ^08:06
vilajml, lifeless: ping09:39
vilatesttools introduced a change in revno 142 (regarding the return value of running a test suite with failing or erroring test),09:41
vilaI'd like to guard against that by testing the testtools version09:41
vilaI can use 0.9.8 for that except if you expect to change the behavior again before releasing 0.9.809:42
vilajml, lifeless: when do you expect to release 0.9.8 ?09:43
lifelessvila: its going to be backed out, see the bug09:45
vilalifeless: rats, too bad I missed that one, this sounded like a nice change though09:47
lifelessonly part of the patch is buggy09:48
lifelessonly that bit will be backed out09:48
lifelessit breaks subunit too09:48
vilaOk, I'll subscribe to the bug to see how it goes then09:49
vilameh, lp already subscribed me :)09:50
vilaI'll revert my babune testtools in the mean time09:50
viladOxxx: Hey !13:06
dOxxxvila: hey :)13:07
dOxxxvila: everything go okay with the release?13:07
viladOxxx: Thanks for the reviews and merges ! I haven't looked at the s/update/pull/ bit you mentioned, do you plan to: 1) fix it, 2) revert to use checkouts 3) something else ?13:07
dOxxxvila: fixed it when I merged your changes.13:07
viladOxxx: AFAIK, yes, there are a couple of people having troubles with svn on OSX for reasons unclear to me, I was wondering if you knew more about that (they used 2.2.1 if that's relevant)13:08
viladOxxx: Cool!13:08
dOxxxvila: Which version of OSX?13:08
dOxxxwere there bugs filed?13:09
vilawow, that was ambigous, re-try: yes, the 2.2.2 release seems okay. Then some people with 2.2.1/10.5 have reported svn related problems13:09
dOxxxok13:09
viladOxxx: I think there were bugs, but linked to/from answers13:09
viladOxxx: at least one of them has installed macports and had related problems but didn't confirm that uninstalling macports addressed them13:10
viladOxxx: the other(s?) didn't mention macports13:11
dOxxxif you could point me to the bug reports, I'll take a look13:11
vilaok, let me find them13:11
vilabut... just wondering, was it for 2.3bn that svn/subertpy tip was needed ?13:12
dOxxxchecking13:13
dOxxxNo, I think it was for all branches13:14
dOxxxBut it's a compilation problem not a runtime thing13:14
dOxxxI needed r2219+ to actually get subvertpy to compile on 10.513:15
jelmerdOxxx: oh?13:15
vilabug #68128513:16
ubot5Launchpad bug 681285 in Bazaar "Can't push my changes in Bazaar" [Undecided,Incomplete] https://launchpad.net/bugs/68128513:16
vilajelmer: ^ you may know better from the error message in the last comment13:16
dOxxxjelmer: yep, as I recall you made the changes for me13:16
vilawell, the *2* error messages even13:16
jelmerdOxxx: can you fetch without problems from that location?13:17
dOxxxsorry still reading bug13:18
jelmerdOxxx: ahh, ok. That was a while ago though.13:19
dOxxxjelmer: yeah, so I don't think that's the problem here.13:19
viladOxxx: right, the other is harder to find back: bug #682904 and https://answers.launchpad.net/bzr/+question/135989, read the question before the bug may be13:20
ubot5Launchpad bug 682904 in Bazaar "Can't push my changes in Bazaar--need to update libsvn?" [Undecided,Invalid] https://launchpad.net/bugs/68290413:20
dOxxxvila: could you check what version of svn you have on your 10.5 build machine?13:20
vilahow ?13:20
vilaI *have* svn ??13:20
dOxxxof course, how do you think the installers build? :)13:21
dOxxxOS X comes with svn stock13:21
vilano idea, I've just pushing buttons ;)13:21
vilas/I've/I'm/13:21
dOxxxhah13:21
jelmerdOxxx: Ahh, you're Gordon!13:21
dOxxxjelmer: yers ;)13:21
vilajelmer: meet dOxxx aka Gordon13:22
dOxxxsorry, shoulda introduced myself :)13:22
viladOxxx: meet Jelmer :)13:22
vilasvn --version reports: 1.4.4 (r25188)13:22
jelmerdOxxx: np, I know who you are.. but I hadn't connected you with your nick name :-)13:22
dOxxxhmm....13:23
viladOxxx: is this as bad as I think ? :-/13:23
dOxxxjelmer: would compiling subvertpy against libsvn 1.4 cause a problem like described in these questions?13:23
jelmerdOxxx: yeah13:23
vilai.e. is bzr-svn compatible with svn 1.4.4 ?13:24
jelmerit is compatible but certain functionality won't be available13:24
jelmerand performance will be impacted13:24
dOxxxI don't want to include new svn libraries with the OSX bzr installer13:25
jelmerdOxxx: why not?13:25
dOxxxjelmer: 1) I'm not sure how to do it correctly, 2) that's touching system files, which is kinda icky13:25
vilajelmer: I don't think the OSX installer can use private binaries/libraries easily so they are installed sys... what he said :)13:26
jelmervila: urgh :-(13:26
jelmerI guess it makes to use whatever the system is providing13:26
vilait is !13:27
dOxxxthe OSX installer puts all the bzr stuff into the system python's site-packages and stuff like the Qt framework in the system location too13:27
vilabut the fix here may be to just tell people to upgrade their svn if needed13:27
dOxxxso it's already potentially overwriting Qt libraries13:27
vilabut that's different than pushing a newer one13:27
dOxxxbut those aren't installed stock with OSX so I didn't feel so bad about that13:27
dOxxxbut even if they upgrade their system svn, it seems that since subvertpy is compiled against 1.4, it will still cause problems?13:28
jelmeryes13:28
vilaoh my13:28
dOxxxright, so we'd need to compile subvertpy against svn 1.5 at least and tell users that they must install svn 1.5 first13:29
vilano, wait, I don't get it13:29
dOxxxvila: subvertpy build against svn 1.4 on our build machine will make it behave like a svn 1.4 client even if there are newer svn  libraries installed on the user's machine13:29
vilaare you both implying that 2.2.0 was ok, but 2.2.1 is not anymore because of subvertpy ?13:29
dOxxxvila: I don't think this is related to the bzr version13:30
vilaI was referring to the installer version13:30
vilaso the set of whatever we package there, including bzr, plugins and their dependencies13:31
dOxxxurgh13:32
vilaif we can't provide an all-in-one installer including bzr-svn, it means (IMHO) that we should have specific packages for svn-1.4, 1.5 and/or whatever other versions are available and install only the relevant ones ?13:32
vilas/ones/one/13:32
vilabut back to my question: bzr-svn was working with the 2.2.0 installer right ?13:33
dOxxxlike I said, I don't think this is a problem that was introduced in a particular version of the installer, it may have always existed13:34
viladOxxx: or did you use a 10.5 host with an upgrade svn to 1.5 ?13:34
vilaha, ok :-/ So these are the first reports that bzr-svn doesn't work on OSX ?13:34
dOxxxvila: honestly, I can't remember what version of svn it had installed. I know that it was a fresh install of OS X 10.5, so I'm assuming that it was svn 1.4 like yours13:35
vilajelmer: did you get feedback from osx 10.5 users ?13:35
dOxxxvila: as far as I know, I don't seem to get emails for bug reports on bzr-svn13:35
viladOxxx: honest ignorance is better than silence :D13:35
jelmervila: please note that these bug reports are not an indication that bzr-svn doesn13:35
jelmer't work at all, it's just certain kinds of pushes13:35
vilaoh13:36
vilaright :)13:37
vilaI was shaking the tree to see which fruits will fall :)13:37
vilaWasn't meant to be negative13:37
vilameh, especially when the report starts with: "For a while, Bazaar was working fine, but now when I push, I get"13:38
viladOxxx, jelmer : how does one check which svn and which subvertpy is used by bzr (or is subvertpy the only relevant part) ?13:40
jelmervila: newer versions of bzr-svn will mutter that information to .bzr.log13:43
vilaSaid otherwise: how can one install an upgraded svn/subvertpy to make bzr happy ?13:43
vilahmm, or may be s/how can one/is it possible to/13:43
maxbIf you just install svn 1.5.x and reinstall subvertpy, things should be happy13:43
dOxxxjelmer: the bzr 2.2.2 OSX installer uses r2219 of subvertpy, has there been an official release since then?13:44
dOxxxlooks like 0.7.4 and 0.7.5?13:45
jelmerdOxxx: yep13:46
dOxxxok, I should update the installer configs to use 0.7.5 then I think13:46
dOxxxvila: do you think we should rebuild the OSX installers for 2.2.2 to use a newer subvertpy?13:46
dOxxxthe version its currently using is from July and about 2 releases out of date13:47
vilahmm13:48
dOxxxI don't know if it will solve George's problem but it seems like a generally good idea13:48
dOxxxsince it looks like there have been some svn 1.4 compatiblity fixes13:48
vila2.2 is stable, if we broke it we should fix it, but upgrading too many things may break it even more13:49
jelmerdOxxx: yeah, it wouldn't solve George's problem13:49
dOxxxjelmer: would subvertpy 0.7.5 still be compatible with bzr-svn 1.0.3 or should we use a newer version of that?13:49
jelmerdOxxx: there's no need for a newer bzr-svn13:49
dOxxxok thanks13:49
vilaso, testing 2.2.1, 2.2.2, 2.3b3 locally with 'bzr info http://svn.apache.org/repos/asf/subversion/trunk' succeeds13:59
viladoes it mean I need another svn server to test against ?13:59
vilato reproduce George's problem ?13:59
dOxxxvila: have you tried the URL George is using?13:59
vilanot yet, I though it required some auth, let me try13:59
maxbjelmer: Did you deliberately turn the bzr-pqm-daily recipe on again for karmic?14:00
* maxb turns it off again14:00
viladOxxx: indeed, it asks for a suer/password and when I enter none, fails with a 500 server error14:01
dOxxxdarn14:01
dOxxxwhat I don't understand is why, if George upgraded his svn to 1.6.9 and subvertpy to 0.7.4, why is still not working? It gives a different error though.14:02
jelmerdOxxx: did he recompile his subvertpy against svn 1.6 ?14:03
dOxxxhe doesn't say.14:03
jelmermaxb: I didn't realize it was intentionally disabled. I enabled the natty builds yesterday evening.14:03
maxbThe recipe description field could do with being more prominant, and supporting line-breaks14:04
dOxxxvila, jelmer: I've updated https://answers.launchpad.net/bzr/+question/135989, asking George for specifics on how he upgraded svn and subvertpy14:06
dOxxxI need to go do my actual day job now unfortunately. :P14:07
vilajelmer: he says (in bug #682904) : "Note that I am running svn 1.6.9 and subvertpy 0.7.4."14:07
maxbjelmer:  Oh, whilst I think about the recipes, I had a question: Why are the bzr daily recipes using version numbers like 1.4.0~bzr{revno}~ppa{revno:packaging}+{revno:debian} ? The ordering of {revno:packaging} and {revno:debian} is inverted to what feels natural to me?14:07
ubot5Launchpad bug 682904 in Bazaar "Can't push my changes in Bazaar--need to update libsvn?" [Undecided,Invalid] https://launchpad.net/bugs/68290414:07
viladOxxx: no worries, thanks for the help !14:07
vilajelmer: he didn't say *where* he installed them though14:08
jelmervila: but that doesn't say against which version of the libsvn API his subvertpy was compiled against though14:08
dOxxxyeah that's what I'm wondering14:08
dOxxxconsidering he was using MacPorts before14:08
viladOxxx: not sure that's the same guy14:08
dOxxxthey seem to be the same George14:08
vilaha, good, well done to file different bugs then :-/14:09
dOxxxyeah and two Questions with the same content :)14:09
vilaha, now we need duplicates for questions too...14:09
jelmervila, dOxx: If he has svn 1.6.9 installed but compiled his subvertpy against svn 1.4 then it still won't work.14:10
vilaha no, poolie turned the bug into a question, oh my14:10
jelmermaxb: Something can be said for either14:10
dOxxxdammit, I'm not getting notified of bugs filed on bzr-mac-installers14:10
vilajelmer: /me nods14:10
maxbjelmer: ok :-) I'd like to understand what can be said for this way?14:11
dOxxxok... set myself as the bug supervisor maybe that will do it14:11
mgzvila: I don't understand the testtools from the log or Robert's bug entry14:12
mgz+issue14:12
vilamgz: just a sec, let me push my wip branch, that should be clearer there14:13
mgzthe three failing tests look wrong to me, but why were two of them passing before?14:13
vilathree/two ?14:14
vilaThree were passing three are now failing14:14
mgzwe don't seem to use a custom wasSuccessful method, and the default one should return False if there's a failure14:14
mgzunexpected success flipped, but the other two should be the same14:14
vilamgz: in a nutshell, the bug I filed is a duplicate, more fixes should be done in testtools14:15
mgzI'm guessing from the bug entry Robert knows what's going on, I just can't divine what the fix should be from what he's written14:15
vilahmm, he said the change doesn't take into account decorated was_successful or something, including subunit14:16
mgzah, maybe it's a subunit only thing.14:16
smoseri'm not sure this is bzr or launchpad question.14:17
vilamgz: now, I don't understand the specifics, I've only been looking at making the tests more precise about whether running a test suite should succeed or not depending on which tests were provided14:17
dOxxxargh14:17
smoserbut this used to work:  bzr branch lp:~ubuntu-on-ec2/vmbuilder/automated-ec2-builds14:17
smoserbut now gives bzr: ERROR: Not a branch: "bzr+ssh://bazaar.launchpad.net/~ubuntu-virt/vmbuilder/0.12/".14:17
dOxxxvila: looks like we need to do a new 2.2.2 installer build anyway to update bzr-svn to 1.0.414:17
dOxxxbug 62144614:17
ubot5Launchpad bug 621446 in Bazaar Mac Installers "AttributeError: paths when trying to svn-import" [Undecided,New] https://launchpad.net/bugs/62144614:17
jelmersmoser: that's a #launchpad question really. You have a branch stacked on another branch that no longer exists.14:17
smoserhttps://code.launchpad.net/~ubuntu-virt shows no 0.12 branch there anymore.  I'm guessing someone deleted that, and my branch shared data14:18
smoserjelmer, yeah.14:18
smoserjelmer, thanks.14:18
jelmersmoser: Soren had the same issue the other day.14:18
viladOxxx: unless this is already fixed in 2.3b3 (or b4 which will be released tomorrow)14:18
dOxxxvila: no it's an installer issue. the installer is configured to use bzr-svn 1.0.3.14:19
smoserjelmer, funny, soren is probably who deleted my branch :)14:19
jelmermaxb: if somebody fixes something in the packaging branch then that change should sort before any fixes in the debian branch.14:19
dOxxxnothing to do with bzr version per se14:19
sorensmoser: I didn't delete your branch. I moved mine to ~ubuntu-virt.14:19
dOxxxjust that 2.2 is current stable branch so I should make sure it works ;)14:19
viladOxxx: if 1.0.4 is not specifically targeted at bzr-2.2, we shouldn't include it in 2.2.[23]14:19
smoserwell, one its gone from where it was, so move or delete, ... potAto potato14:19
jelmermaxb: ~bzr has write access to the packaging branch but not the debian branch so the packaging branch can always merge in the debian branch if necessary, never the other way around.14:20
viladOxxx: otherwise we end up packaging the same thing in 2.3 and 2.2 and this will break the stability guarantee14:20
smosersoren, do you happen to know, then what i need to do to make that function again ?14:20
viladOxxx: that's the rule,  does it apply in this case ?14:21
dOxxxvila: not sure I understand your previous comment14:21
vilamgz: lp:~vila/bzr/683505-test-no-log, wip, doesn't work yet14:22
maxbjelmer: but, every build will always take the latest revno of the debian branch anyway, so I'm not seeing where this ordering issue would arise14:22
viladOxxx: let me look at the bug14:22
jelmermaxb: Imagine somebody made a commit in the debian branch14:22
jelmermaxb: or, in fact, did a merge in the debian branch (changing the revno's)14:23
maxbbut... that would be a silly thing to do! :-)14:24
mgzthanks vila.14:24
viladOxxx: dang ! Same here, I never got gplyph answer !14:24
mgzso, I think the conditions are backwards in those three tests, but it's subunit being funny.14:25
mgzso again, probably wants something changing in all three projects.14:25
sorensmoser: I'd talk to mvo. I think he sorted it out.14:26
dOxxxokay really going to work now14:27
dOxxxseeya14:27
vilamgz: since babune was very update, I've reverted to testtools-0.9.7 waiting for a consensus to emerge :)14:27
vilamgz: what is backwards ?14:28
vilamgz: that subunit says all is well even when running a single failing test ?14:28
viladOxxx: argh, no one more second !14:29
dOxxx?14:29
viladOxxx: my comment was: if bzr-svn doesn't say: this release is compatible with bzr-2.2 and 2.2 users are expected to upgrade, we should include it in 2.2.2, otherwise we should not14:30
viladOxxx: i.e.: the plugin authors should be explicit, like qbzr maintaining a specific series target at 2.2 and another targeted at 2.314:30
dOxxxthat would be nice14:30
vilajelmer can confirm that bzr-svn is targeted at 2.2 and 2.3, so in this case, yes, we should rebuild the 2.2.2 installers14:31
dOxxxbzr-svn 1.0.414:31
vilameh, bzr-svn-1.0.4 yeah, thanks :D14:31
dOxxxargh14:32
jelmerdOxxx, vila: The supported versions for a bzr-svn branch should be available in info.py14:32
dOxxxGeorge says he "replaced the system-installed svn with the new version using MacPorts"14:32
vilajelmer: by the way, the 1.1 and 2.0 series for bzr-svn are not that clear to me ;D14:32
dOxxx /facepalm14:32
viladOxxx: be sure to check the dates ?14:32
dOxxxthis is a new comment on the question14:33
dOxxxposted 15 min ago14:33
dOxxxhttps://answers.launchpad.net/bzr/+question/13598914:33
dOxxxas far as I know MacPorts installs into /opt so it wouldn't replace the system svn14:33
jelmervila: Yeah, I haven't really gotten to those. I have Plans, though :-)14:34
vilajelmer: hehe great !14:34
vilaso, bzr-svn 1.0.5 is targeted at 2.x for x in [0,1,2,3] :)14:34
vilabzr-svn 1.0.3 is targeted at 2.x for x in [0,1,2]14:35
vilaand of course I don't have bzr-svn-1.0.4 available here ;D14:36
dOxxxanyway so I think his svn is not properly upgraded and his subvertpy not compiled against the right version.14:36
dOxxxand now I really must go to work, already 30 min late.14:36
viladOxxx: right, happy work day ! Thanks again !14:36
dOxxxbye!14:36
vilajelmer: one more thing: bzr-svn is aimed at being compatible with svn-1.[456] right ? (relying on subvertpy compiled against the right library ?)14:56
jelmervila: yep, and 1.714:57
vilahmm, so the issue with OSX installers is to be able to build various subvertpy packages on a host with the right(s) svn installed :-/14:59
vilaand provide some way to select the right one :-(14:59
=== zyga is now known as zyga-food
=== Meths_ is now known as Meths
=== zyga-food is now known as zyga
mgzvila: yes, that's my understanding of the subunit aspect.16:06
vila. o O (Oh my god, what does he mean with this 'yes'....)16:07
vila:D16:07
mgz<vila> mgz: what is backwards ?16:07
mgz<vila> mgz: that subunit says all is well even when running a single failing test ?16:07
mgz:P16:07
vilaha, ok16:07
=== deryck is now known as deryck[lunch]
vilaI wonder if he does this manually in order to carry some subliminal messages16:08
mgzthe lunch? or lifeless quitting? :D16:08
vilamgz: not you ;D16:09
vilaThis has occurred several times these last weeks with an astonishing synchronicity :)16:09
vilamgz: random thought: about 'lost connection during...' errors, could that be explained by 'time' events going backwards in time ?16:13
mgznope, they should be pure decoration.16:13
vilaI hate them16:14
mgzit's not a very helpful starting point for debugging I agree.16:15
mgzcould perhaps file a bug against subunit to make it provide more information in those circumstances16:17
mgzit's from falling off the end of a read-from-stream loop, printing state information about the subprocess where available might be helpful16:18
mgzbreaks through the abstraction a bit, but otherwise you don't have any clue what caused the eof.16:19
vilamgz: is there a way to save these streams ?16:20
mgztee?16:20
vilawhere ?16:20
mgzhm, point.16:20
vila:D16:20
mgzinside fork_for_tests I guess.16:21
vilayeah, I was afraid you'll say that ;)16:21
=== beuno is now known as beuno-lunch
ccxCZis there way to make bzr non-interactive when working on bzr+ssh branches, so it won't ask for password?16:40
=== deryck[lunch] is now known as deryck
mgzhttp://babune.ladeuil.net:24842/job/selftest-windows/237/ <- are you aware of this regression vila?16:53
mgzbt.test_msgeditor.MsgEditorTest.test_deleted_commit_message has been consistently failing since then, though it passes locally for me.16:53
vilaccxCZ: like... using an ssh key and an ssh agent ?16:54
mgzit's something of a hacky test, perhaps tripping over something with del.16:54
vilamgz: shudder, I noticed and forgot and bialix also mentioned something related and I was waiting for his feedback16:55
vilaccxCZ: so the answer is yes, it would drive me crazy otherwise to type my passphrase hundreds of time each day16:56
vilamgz: 'since then' is related to bug #676637 ?16:57
ubot5Launchpad bug 676637 in GStreamer "Wrong aspect ratio with DV AVI files in PiTiVi" [Low,New] https://launchpad.net/bugs/67663716:57
vilaprobably not16:58
vilamgz: 'since then' is related to bug #673637 ?16:58
ubot5Launchpad bug 673637 in Bazaar ""bzr commit" fails when BZR_EDITOR is emacsclient" [Medium,Fix released] https://launchpad.net/bugs/67363716:58
vilamore like it16:58
mgzsince that set of revisions, but I'm guessing bug, yeah, that one16:58
viladamn, I should fix this asap then16:58
vilathe bug report was for a windows user, too bad he didn't test it, lack of installer I presume16:59
ccxCZvila: of course I know about ssh keys, but in case they're not loaded (eg. cronjob) I want bzr fail instead of hang17:01
ccxCZbzr to fail*17:01
mgzthe problem is bzr delegates to your ssh client17:01
mgzhow would you prevent that from promting?17:02
ccxCZwhich is paramikoI presume?17:02
mgzwell, depends on what you have installed.17:02
vilathe use case is valid though a bit weird17:02
ccxCZyou can stop openssh by options and maybe by env17:03
mgzvila, I'm not sure the fix for the bug is broken, just that test (and not always, as it seems to pass here)17:03
vilaccxCZ: you can force the ssh implementation via...17:03
vilaBZR_SSH ?17:03
vilayeah, see 'bzr help env-variables'17:04
mgzyeah, making a sh script with ssh --some-no-prompt-option then setting BZR_SSH to that script would work17:04
mgzif you have an ssh client that supports something like that.17:04
vilaccxCZ: so you have a cronjob that pass where you have given access to some key via your agent and you want it to fail otherwise ?17:04
vilas/where/when/17:04
vilaccxCZ: I use password-less keys for unattended jobs, so I'm curious about your use case here17:05
vilaccxCZ: also there have been discussions about a true non-interactive mode but I'm not sure the related changes have landed17:07
vilaccxCZ: better file a bug with your precise use case, I'm sure I will forget otherwise :(17:08
ccxCZactually I want automate checking whether there are changes on remote server, probably by changing prompt on the shell that is cd'ed in a dir with .bzr in it17:09
ccxCZwhat should I put in BZR_SSH? would "/usr/bin/ssh -o BatchMode=yes" work?17:10
vilaI'm afraid we don't accept arguments in this variable, IIRC, it's either a pre-defined value or a path17:11
vilamgz: hmm, this may be babune specific...17:13
mgzyup.17:14
lifelessccxCZ: you can put BatchMode=yes into a ~/.ssh/config stanza17:19
ccxCZyes, but that would disable it for *every* ssh instance, which I don't want, since I might ssh into a server where I don't have a key17:21
lifelessno17:22
lifelessput it in a host stanza17:22
lifelessonly affects that host17:22
ccxCZthat's not really what I want, it has several shortcomings17:27
ccxCZneeding to remember to add host when I branch from new one for example17:28
ccxCZinteresting thin is this still manages to ask for password:17:29
ccxCZTTY="" ssh -p 2222 localhost </dev/null &>/dev/null17:29
ccxCZthing*17:29
jamecho -e '#!/bin/sh\nexec ssh -o BatchMode=yes "$*"\n' > ~/bin/my_ssh; chmod ~/bin/my_ssh; export BZR_SSH=~/bin/my_ssh17:29
jamccxCZ: openssh knows how to do direct-to-tty communication, not via stdin/stdout17:29
jamso that you can't get your password intercepted, etc17:30
vilajam: hello  :)17:30
jamhi vila17:30
jamvila: I just found # cypthon: profile=True which seems very powerful. Of course, it just changed my runtime from 3min to >1hr so far...17:31
jambut you can also selectively disable it for specific functions17:31
vila:) :-/17:31
jami thought the 2x overhead from profiling regular python code was bad, 100x for compiled functions?17:32
vilamay it's more precise :D17:32
vilait runs everything 100 times to avoid noise ;)17:33
vilajam: what BZR_SSH setting are you using on windows when running from a cygwin shell ?17:34
jamvila: BZR_SSH=paramiko17:34
viladoesn't work for me:17:34
vilaerr, not the same failure :(17:35
jamfrom cygwin shell, you'd still need "export BZR_SSH=paramiko", and you'd also want to make sure you actually have paramiko installed17:35
ccxCZjam: yeah, I've done just that17:36
ccxCZthanks everyone17:36
vilajam: hmm, will that be able to use a forwarded agent ?17:36
jamvila: paramiko doesn't talk to forwarded agents, AFAIK17:37
jamI'm not sure if "BZR_SSH=openssh" would be able to (if you have openssh installed on that machine)17:37
jamactually, AFAIK, you can't forward agents to windows, but I don't know the cygwin ssh server all that well17:37
jamI've never tried to use it in anger17:37
vila'ssh-add -l' gives me the expected result17:38
=== beuno-lunch is now known as beuno
vilaha,right, that's where I got the error: 4 [main] ssh 3980 C:\cygwin\bin\ssh.exe: *** fatal error - couldn't initialize fd 2 for /dev/tty017:38
vilaI can connect "manually" but not via bzr+ssh :-/17:39
ccxCZ/msg *status detach #bzr17:41
ccxCZoops17:41
vilaccxCZ: bye ;)17:41
jamvila: probably something about TTY hiding/subprocess spawning17:42
jamvila: I would guess the problem is that your "bzr" is a win32 native process17:42
jaminterposing itself from your cygwin shell that you sshed into, and the ssh you are going out of17:43
jamyou could try running cygwin bzr17:43
vilawhich one is that ? ;D17:43
* mgz uses plink17:44
mgzmixing cygwin ssh and native bazaar could certainly cause oddness17:45
vilamgz: and you can ssh *to* a windows host *and* get bzr+ssh to say lp, working ?17:45
mgzwhy would I want to ssh to a windows host?17:45
vilamgz: to help me ssh to a windows host and have a babune setup working ;D17:46
jamvila: why not use pageant with a decent key and paramiko?17:47
mgz(In other words, I've never tried to run an ssh server on windows and probably can't help)17:48
vilajam: the windows host is started on demand, there is nobody logged there17:48
mgzI'd stick with screen sharing, even though that's not as nice for you as pretending it's another nix box.17:49
jamvila: well, *somebody* gets logged in...17:49
vilayes, via ssh, but how will pageant be started there ?17:50
vilano display17:50
vilamgz: screen sharing is good for interactive use, running the test suite doesn't care about a screen :)17:51
mgzhudson needs ssh access to work?17:52
vilahudson itself, no, but I don't want to duplicate everything just for windows, anyway, EOD for me, dinner is ready ;) I'll try plink tomorrow17:54
jamvila: I don't think plink knows how to talk to openssh agents17:55
mgzyeah, I'm not sure I'm solving your problem with that.17:55
jamthe other option is paramiko without pageant, and just use password-less keys17:55
vilaif this means having one dedicated ssh key for windows, I can leave with that17:55
vilaor two (one for babune, one for lp)17:56
jamI believe paramiko will look in ~/.ssh/* which is C:/Documents and Settings/user on older windows and C:/Users/user on newer17:56
vilajam: ha ! excellent, that's certainly the missing bit, as <user> is *not* set correctly so far17:59
* vila shouts: yeah, I'm coming darling, I'm coming17:59
Ddorda‎hey guys. how do i push on the last commit?18:01
dashwhich one is the last one? :)18:03
dashor do you mean "on every commit"18:03
Ddorda‎dash: commit that overwrite the last commit18:04
dashDdorda: commits don't "overwrite" stuff18:04
Ddorda‎dash: i mean revisions (woops)18:05
dashright, those don't either18:05
Ddorda‎to push overwriting the last rev.18:05
Ddorda‎so?18:25
Ddorda‎i want to remove the last rev. and make a new one instead18:25
beunoDdorda, "bzr uncommit"18:26
Ddorda‎beuno: does it remove the changes?18:27
beunoDdorda, it removes the commit, and leaves the changes uncommited18:27
Ddorda‎beuno: thank you :)18:27
beunonp18:28
maxbvila: Your lucid bzr-landing-dependencies has a version number less than the hardy one. This is bad, if there's ever an attempt to upgrade from one to the other18:49
vilamaxb: thanks for the tip, I'll fix it, 1.0.2~lucid1ubuntu1 will do >18:56
vila?18:56
maxbvila: Well, it's a pet peeve of mine - why include "ubuntu" when it's not a Debian derived package modified in ubuntu?18:57
vilamaxb: no idea, monkey see monkey do, no thinking there :) I'm all ears18:57
maxbIf it's a lucidified version of 1.0.0, I'd call it 1.0.0lucid118:58
vilamaxb: oooh, that's the '~' that makes it smaller ?18:59
maxbYes, the ~ is the magic that is smaller than nothing at all18:59
maxbUnfortunately, the rise of PPAs has taught people to use it as a generic separator18:59
vilamaxb: because PPAs are supposed to provide preview versions right ?19:00
maxbBecause they are often used that way19:00
jamvila: right, you usually want ppas to be overwritten by real releases19:01
jamI've seen similar tricks with nightly builds. So you have bzr-2.3b4~r553619:02
vilamaxb: done and dput'ed19:02
maxband mark-uploaded and pushed? :-)19:02
vilajam: yeah, as maxb said, I thought it was just a separator19:02
jamvila: there is a whole lot of voodoo magic that goes into debian versions :)19:02
jam- means something, as does ~, and everything else19:03
vilamaxb: what ? Hmm, more tricks coming :)19:03
vilamaxb: I just do a debchange/debcommit/debuild -S/dput dance19:03
maxbbut don't you have this in a bzr branch?19:03
vilamaxb: I do19:04
maxb"bzr mark-uploaded" is "bzr tag <figure it out from the changelog>"19:04
vilahmm, done19:05
maxbyay :-)19:05
vilapushed19:05
vilaso the branch lacks a proper tag for the previous version, but we can forget about that right ?19:06
vilaI should do the same in the hardy branch though19:06
vilaor should I create 1.0.0hardy1 first to catch up ?19:07
vilaOMG that's why ubuntu version follow the alphabet ?19:08
jamsometimes I wonder what the Unicode people are smoking: http://www.fileformat.info/info/unicode/char/1f63c/index.htm19:09
jam"Unicode Character 'CAT FACE WITH WRY SMILE'"19:09
vilamaxb: ^ ?19:14
maxbYes that is one of the reasons for ubuntu versions following the alphabet :-)19:15
vilamaxb: cool :) And about creating a 1.0.0hardy1 version ?19:16
maxbThere is no particular need to do 1.0.0hardy1, I would call the hardy branch the 'trunk' and let it use plain version numbers19:16
vilaok19:16
radixjam: FINALLY I've been wanting that codepoint for *ages*19:34
maxb  File "/home/maxb/wc/bzr/qbzr/ppa/karmic/lib/tests/modeltest.py", line 69, in nonDestructiveBasicTest19:53
maxb    assert(self.model.columnCount(QtCore.QModelIndex()) >= 0)19:53
maxbTypeError: QAbstractItemModel.columnCount() is abstract and cannot be called as an unbound method19:53
maxbAnyone qbzr-involved around who can interpret that?19:53
maxbThe current qbzr packaging turns on the tests on the buildds, and it fails like that for karmic19:54
maxbI am now wondering if anyone actually uses qbzr from the ppa on karmic19:54
maxbor jaunty even19:54
vilagary gary where are you ??? garyvdm (hoping to trigger a curl/grep daemon)20:01
vilamaxb: this smells like a qt dependency, filing  a bug on qbzr may be your best bet20:02
vilamgz, jam: What will happend if bzr selftest has to access to the TEMP dir ?20:03
vilad for devil20:03
vilamgz, jam: in windows that is20:04
jamvila: if it has to, then it will access it...20:05
jamI don't quite understand your question20:05
jamare you wondering *where* temp is?20:05
vilait turns out babune has no write access (nor read even) to TEMP (c:\WINDOWS\TEMP)20:05
vilayet most of the tests pass... So I wonder how :D20:06
jamare you sure $TEMP isn't C:\Documents and Settings\<user>\Application Data\Local\Temp ?20:06
jamcheck what20:07
jamtempfile.mkstemp() returns20:07
vilanothing :-DF20:09
vilaha, no typo of course20:09
vilac:\\windows\\temp\\tpmxxxx20:09
jamor tempfile.NamedTemporaryFile().name20:10
vilablink20:10
jamvila: are you *sure* it doesn't have access :)20:10
jamI'm not sure the protection model for winxp, but it may be impossible to block the standard temp dir, or something20:10
vilathat's weird, if I set TEMP to  a dir I create myself the offending test pass20:10
vilajam: right, it has access, but... something goes wrong just for this test...20:17
jamwe use $TEMP for the whole test suite, so I would be surprised if that was the specific failure20:19
jammaybe the path depth is getting too long?20:19
vilano, simple file name20:19
jam(more than 260 chars?)20:19
vilaand my test dir is longer than windows/temp20:19
vilaright, and from a cygwin shell TEMP is set as you said and the test pass20:24
vilaso definitely some weird user setup, I fixed some, but more is needed, but enough for tonight, probably the ssh server fail to setuid properly, I may have to tweak the service definition and reboot or something-ish :)20:26
pooliehi jam, spiv21:01
jamhi poolie21:01
pooliejam, join #ubuntu-meeting?21:02
dOxxxI return!21:52
jelmer\o/21:53
dOxxxso the eventual decision was that the installer for bzr 2.2.2 should include subvertpy 0.7.5 and bzr-svn 1.0.5?21:54
jelmerdOxxx: I don't remember releasing 1.0.5 :-)21:54
jelmerdOxxx: but other than that, yes.21:55
dOxxx(09:34:48) vila: so, bzr-svn 1.0.5 is targeted at 2.x for x in [0,1,2,3] :)21:55
dOxxxdid he mean 1.0.4?21:55
jelmerdOxxx: no, 1.0.5 is in development at the moment21:56
dOxxxah21:56
dOxxxum21:56
dOxxxjelmer: So, just to be sure, bzr-svn 1.0.4 should be bzr 2.2 compatible, right?21:59
maxbjelmer: speaking of which, when are you going to release 1.0.5 :-)22:00
maxbWould be nice to get those kde code imports that are currently being imported by my desktop on to launchpad proper22:00
pooliejam, hi, still around?22:14
jelmermaxb: I don't have any immediate plans, would like to get those ghost issues resolved first.22:14
jelmermaxb: It should be possible to cherrypick the new release for launchpad though, new bzr-svn releases have to be cherrypicked anyway.22:15
maxbjelmer: cherrypick as in merge just the relevant fixes into the ~launchpad-pqm branch?22:19
jampoolie: yeah, I'm still here22:20
jelmermaxb: No, sorry - I used the wrong word. I meant to say new bzr-svn releases have to be merged manually for lp anyway.22:21
pooliesorry otp at the moment22:22
maxbyes... but that is dependent on there being a new upstream release first :-)22:22
jamnp, you were looking for me :)22:22
jelmermaxb: not really, we've merged snapshots in the past.22:22
maxbSo I should review whether there's a comfortable point on lp:bzr-svn/1.0 which we can be happy about using22:23
jelmermaxb: Yes; hopefully that will be trip.22:23
jelmer*tip22:24
viladOxxx: if 1.0.5 is not released, it's ok to put it in 2.3, hoping for jelmer to release it before 2.3 is final, for 2.2, we should stau with officially released versions, so 1.04, sory for the confusion, I was looking at the source in bzr-mac-installer/2.3/src/bzr-svn22:26
vilas/stau/stay/22:26
* vila off to bed ;)22:26
peitschienite vila22:30
peitschie(mornin jelmer, poolie, jam, maxb, dOxxx... woah.. busy today!)22:31
pooliebye vila22:35
maxbDoes anyone know what the graph.sh in hydrazine is intended to be used with?23:38
maxbBecause the data collection part of the puzzle appears to be missing23:39
maxbI am wondering if we should delete graph.sh from hydrazine23:39
maxb(poolie perhaps?)23:39
_habnabitIs there a thing anyone knows of that visualizes branching and merging?23:47
dash_habnabit: 'bzr qlog'23:47
dashor 'bzr vis'23:47
_habnabitAha.23:47
dashdepending on if you have the qt or gtk plugins installed23:48
_habnabitApparently I have neither.23:48
maxbI recommend qlog. Even though I am a GNOME user.23:50
maxbqlog is just too shiny not to use, even if I have to install all of Qt to do it23:50
_habnabitIs there something that can spit out a dot file, maybe?23:50
pooliemaxb, it's pretty old, i think we should delete it23:56
poolieif the collection code is checked in at all, it will be there23:56
poolieit never got to a super useful state though23:56
pooliei thought there was a count-bugs.py or something?23:56
maxbThere is, but it emits human-readable text, not rrd anything23:57
spiv_habnabit: there is somewhere23:57
spiv_habnabit: the graph-ancestry command from bzrtools23:58
_habnabitAha.23:58

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