/srv/irclogs.ubuntu.com/2010/11/17/#bzr.txt

pooliehi spiv, how are you? and what fabulous adventures will you have today?00:13
spivpoolie: working towards bug 309682; I have a patch that works, but polishing all the little issues the patch glosses over is surprisingly involved.00:16
ubot5Launchpad bug 309682 in Bazaar "tags are copied but their revisions may not be (affected: 0, heat: 10)" [Low,In progress] https://launchpad.net/bugs/30968200:16
pooliehow about a catch up call in a few minutes?00:16
spivSure00:16
vilahi all !07:26
GaryvdMHi vila, Hi all07:43
vilaGaryvdM: hey !07:43
GaryvdMvila: I hope you don't mind me asking about test writing a test. For bug 588698. The current tests that deal with merging merging new roots, crisscross merges, etc only check the shape of the graph after the merge. There are no asserts on the state of the files in the wt. This is done in other test that don't look at the shape of the graph. Would it be ok if my test for this bug did both?07:50
ubot5Launchpad bug 588698 in Bazaar ""bzr merge" fails "Branches have no common ancestor" (affected: 1, heat: 8)" [Undecided,Confirmed] https://launchpad.net/bugs/58869807:50
vilaGaryvdM: if you feel the need then go for it07:51
vilathere is no absolute rules about what a test *must* check07:51
GaryvdMOk07:52
vilatoo little is as bad as too much but this can be rather subjective at times07:52
GaryvdMI would like to do both, because I'm not sure exactly it should work, and I would like to define it in a test.07:53
vilaGaryvdM: may be two tests then ?07:57
vilaGaryvdM: or more :)07:58
vilaGaryvdM: ideally a test should check one thing to enhance the overall defect localization07:59
GaryvdMvila: But the actions that the 2 test would perform would overlap 95%07:59
vilaGaryvdM: that is, a single defect is caught by at least one test which makes it obvious what the defect is or at least narrow down the code that needs to be investigated08:00
vilaGaryvdM: it's hard for me to judge without concrete examples, but sometimes this indicates tests that are too high-level08:01
vilaGaryvdM: in this case drilling down to get closer to the code you want to test avoid the duplication08:01
vilaGaryvdM: in some cases though, due to some bad layering, you just can't avoid it so just go for the duplication but add comments or FIXME explaining the issue08:03
GaryvdMOk08:03
vilaGaryvdM: feel free to submit early versions for feedback, MPs can be used for that too (if properly explained)08:04
spivGaryvdM: the first thing to optimise for in a test is clarity08:04
spivGaryvdM: refactoring to reduce duplication is often helpful for that goal of course :)08:05
pooliehello vila08:05
vilapoolie: hey !08:05
spivGaryvdM: but it's important that the *intent* of a test is clear: if it's not, then when it fails, or when it needs to be updated for API or behaviour changes, then it's unclear what should be done.08:05
vilapoolie: dear PP, I havean MP approved but its pre-requisite, could you help me ? https://code.edge.launchpad.net/~vila/bzr/671050-config-policy/+merge/4034308:05
vilas/but its/but not its/08:06
spivGaryvdM: so if there are multiple aspects you want to verify it can be clearest to make multiple tests, so that each can have one explicit purpose.08:06
GaryvdMok08:07
spivGaryvdM: where I find it gets tricky is when I'm not sure yet what the behaviour should be, and I'm writing the test to help me discover that... :)08:12
GaryvdMspiv: And that is the case here...08:12
spivBut I guess it's like any other sort of authoring: you just keep revising till it makes good sense.08:12
poolievila, done, thanks for asking08:12
spivGaryvdM: so it's fine to start with an exploratory test (or tests) that probably do too much, so long as you are aware that once you've explored and hopefully understood the new territory it's probably worth reviewing the tests.08:14
spivGaryvdM: (yes that sentence has lots of qualifiers, I'm not sure these are hard-and-fast rules...)08:14
vilapoolie: great ! I'll tweak the comment, that's a point with quite deep implications that should be better addressed with the new config scheme but some parts are still unclear in my mind, so yes, a FIXME is appropriate08:15
vilaspiv: should we talk about https://code.edge.launchpad.net/~spiv/bzr/checkout-tags-propagation-603395-2.2/+merge/40406 ?08:16
spivvila: I chatted with poolie about it a bit08:17
spivvila: on one hand, I'm pretty confident about the backwards compat stuff... but on the other the bug report has been pretty quiet, so the urgency for getting it in 2.2 is low.  So I'll just land it in 2.3 for now.08:18
spivvila: (if later we decide it might be good to have in 2.2 after all... well, we know where to find the patch!)08:18
spivvila: thanks for the review08:18
vilaspiv: ha great, I feel relieved :)08:20
poolieyay :)08:32
pooliei had such a good day of mgmt today08:33
poolie(imho :-)08:33
vilapoolie: way to go :)08:44
vilapoolie: mgmt taks are the hardest to quantify which is a source of frustration when you try to summarize: "what the hell did I do today/this week/this month" :D08:46
poolieyep08:47
vilaAAAARGH, I hate vbox when is starts to randomly generates mouse clicks all over the place !08:48
poolie!09:07
pooliethat's what cats are for :)09:07
poolieok i think that's enough fun for today09:08
vilahehe09:09
=== Guest8056 is now known as jelmer
awilkinsHi, I'm running bzr 2.3b3 on CentOS 5 (python2.4), bzr-svn 0.7.5, subvertpy 1.0.4 and getting 'function' object has no attribute 'decode' when trying to pull svn branches ; stack trace http://pastebin.ubuntu.com/533419/10:55
vilaawilkins: bzr-svn bug, try trunk ?11:06
vilaawilkins: file a bug otherwise11:06
spivawilkins: I agree with vila11:07
awilkinsLooks like it's passing a generator to a function that expects a string11:08
awilkinsGoing to try it with 2.2.111:09
awilkinsAh, well that produces a different error, so that's progress. I think the previous thing is a change to the API in 2.3 vs 2.2 ; the contructor in 2.2 expects a generator function, in 2.3 it expects a string11:12
awilkinsNow it's a 2.5-ism I think ; 2.4 has no "any() " function, right?11:14
vilahmm, 2.2 was expecting a callable not a generator but yes, that's the offending change11:23
vilabut where is any() involved ?11:23
awilkinsvila, The calling code calls it a generator ... I'm not very pythonic I'm afraid11:23
awilkinsany() is in bzr-svn11:23
awilkinslogwalker.py11:23
awilkinsJust patching it11:23
awilkinsGrr. qbzr needs more context menus. Just want to be able to blame something and do a switch -b from the log menu11:24
awilkinsjelmer, Looks like you already noticed that logwalker.py python2.4 bug :-)12:00
GaryvdMawilkins: Noted12:06
=== oubiwann is now known as oubiwann-away
=== oubiwann-away is now known as oubiwann
=== Ursinha-dinner is now known as Ursinha
=== zyga is now known as zyga-food
vilamgz: testtools-0.9.6 has been installed on pqm14:28
vilamgz: don't rejoice too fast though, we lost subunit in the process :-}14:28
vilamgz: this was an announce from the be-ready-but-not-too-fast dept14:29
=== zyga-food is now known as zyga
hangfireI'm mainly an svn user, and I want to get a checkout from bzr://dxx-rebirth.bzr.sourceforge.net/bzrroot/dxx-rebirth . I've tried bzr co, bzr pull, bzr branch, and I get errors saying "not a branch". bzr init says that it's a "Shared repository". How can I get a copy?15:02
jelmerhey hangfire15:03
hangfirejelmer: hey15:04
jelmerhangfire: you'd want bzr branch or bzr co, but to check out an individual branch rather than the full repository15:04
hangfirejelmer: How can I get a list of branch nameS?15:04
Odd_Blokehangfire: There should branches underneath that repository, and those are what you deal with/check out in bzr.15:04
Odd_Blokejelmer's clearly on it. :)15:04
jelmerhangfire: Actually, looking at this again, is there any reason you're using bzr:// rather than http:// ?15:05
jelmerOdd_Bloke: hey! long time no see :-)15:05
hangfirejelmer: That's what's listed on sf.net :P15:05
Odd_Blokejelmer: Indeed.15:05
Odd_Blokejelmer: I've been hanging around a bit more because I heard there was a job going. ;)15:05
hangfirejelmer: ah! got it. Thanks15:06
hangfireOdd_Bloke: thanks too!15:06
jelmerOdd_Bloke: Ah, right, that one :-)15:07
Odd_Blokejelmer: Congratulations on that, by the way. :)15:07
Odd_BlokeSince I was really spending time contributing to bzr (and other FS projects) I've gained both a job and a girlfriend, so my time is considerably more scarce than it was.15:07
jelmerOdd_Bloke: Thanks!15:08
jelmerOdd_Bloke: Ahh! I wondered what had happened to you.15:08
Odd_BlokeThe job is with credativ, who work with free software, so it's not all bad.15:09
Odd_BlokeI'm just spending my time working on business logic stuff like OpenERP.15:09
Odd_BlokeAnd sysadmin stuff, and some website work in Django.15:09
jelmerOdd_Bloke: Ah, cool. Wasn't Chris or Johnny working there as well?15:09
Odd_Blokejelmer: jonnylamb works as Collabora.15:10
Odd_BlokeOr at least did.15:10
Odd_BlokeWhich may be where you're thinking of.15:10
jelmerI'm pretty sure I met another friend of yours who worked there, perhaps somebody other than Johnny or Chris.15:10
fullermdWell, that's hardly fair.  Messing with bzr hasn't gotten me a job or a girlfriend   :(15:10
Odd_Blokejelmer: Tim Retout?15:11
Odd_BlokeBrad Smith, and Chris Halls are the other DDs who do/have worked here.15:11
Odd_BlokeAnd I assume you only know DDs. :p15:11
jelmerOdd_Bloke: I think so.15:11
Odd_Blokefullermd: Waaah. :p15:12
jelmerOdd_Bloke: (-:15:13
Odd_BlokeOpenERP, which I spend most of my time working on, has a pretty lame upstream, I should probably get back into spending some spare time doing FS stuff.15:14
jelmerI vaguely recall seeing something about openerp and bzr recently15:15
jelmersomebody wanting to extend bzr-stats15:15
Odd_BlokeThey do use Launchpad.15:16
Odd_BlokeAt least for interfacing with contributors/users who are external to their partner programme.15:16
jelmerah, cool15:17
smoserwonder if anyone can help. i think i'm not sure if i'm using import-upstream correctly or not.16:29
smoser$ bzr branch lp:ubuntu/natty/euca2ools natty.dist16:29
smoser$ cd natty.dist16:29
smoser$ bzr import-upstream 1.3.1 ../dl/euca2ools-1.3.1.tar.gz16:29
smoserresults in http://paste.ubuntu.com/533507/16:30
smoseri'm on natty, but i think i'd seen this before16:30
maxbsmoser: I believe you need to add --version before the 1.3.116:46
smoserno. you do for merge-upstream, import-upstream wants it that way16:46
maxbAlso, isn't it merge-upstream16:46
smoser(at least per its usage)16:46
maxboh. ETOOMANYCOMMANDS16:46
smoseri actually want the import-upstream rather than merge-upstream. i just want the upstream pulled in, and then will merge it manually (basically dropping all ubuntu changes and patching in)16:47
smosermerge-upstream just leaves me with dozens of conflicts16:47
smoserjames_w, ^ you have thoughts on above ?16:47
maxbsmoser: is the debian/watch screwed up or is euca2ools-1.3.1-src-deps.tar.gz the right thing?16:48
smosermaxb, i'm not sure. i didnt' even look at the watch. silly me. probably should have looked there first.16:48
james_wsmoser, please file a bug16:48
maxbI think the watchfile is erroneously too permissive16:48
smoserjames_w, ok16:49
maxbsmoser: fwiw, commenting out the line it breaks on appears to allow it to function correctly16:53
maxbit would appear to simply be a missing "if upstream is not None:"16:53
james_wsmoser, lp:bzr-builddeb should have a quick fix for that problem16:54
james_wmaxb, indeed16:54
salgadodoes anybody know why lazr.uri is not included in the OSX bundle (http://wiki.bazaar.canonical.com/MacOSXBundle/SnowLeopard)?  it's needed by lp-propose16:54
james_wsalgado, does lp-propose depend on launchpadlib, or just lazr.uri?16:55
salgadojames_w, probably launchpadlib16:55
salgadohaven't checked, but it didn't work for martohls because lazr.uri was missing16:56
james_wsalgado, right, I'm wondering if launchpadlib is bundled, or it's just reporting the first of a few missing dependencies16:57
james_wI don't know who builds the bundle though16:57
salgadolaunchpadlib is not included in the bundle listing on the wiki16:58
maxbjames_w: Oh, btw, remember I said I was going to fix the missing tags issues blocking UDD imports?  Well, just so you know, I fixed one locally and then found the importer did all kinds of bizarre stuff with parallel importing into squeeze and sid, so I got blocked on having time to dig into that weirdness.17:02
james_wmaxb, bizarre how?17:02
maxbImporting the same upstream version multiple times was the most obviously wrong thing that I recall17:03
smoserbugger. i had opened a bug. but it appears launchpad isn't wanting me to do that now17:04
maxbAlso doing history-diverged distinct imports of the same debian version number on the squeeze and sid branches17:04
vilajam1: ping17:05
=== jam1 is now known as jam
james_wmaxb, the latter is intentional if the branches have previously diverged17:05
jamhi vila17:05
vilajam: how are things going ?17:05
jampretty good today. how about you?17:08
vilajam: testtools-0.9.6 has been installed on pqm BUT subunit has been removed17:08
vilajam: mthaddon is adding subunit-0.0.6 from our stable ppa again, but the build is still ongoing17:09
jamsigh17:09
vilajam: since I will EOD soon, it would be nice if you could check how it goes17:09
vilajam: I'm not asking for landing my stuff, just make sure we *can* land something :-/17:10
jamsure17:11
mgzha, upgrading testtools broke landing?17:11
vilajam: what are you working on these days ? hpss on lp ?17:11
vilamgz: AIUI testtools requires some version of subunit which wasn't the one installed17:12
vilamgz: my previous ping was to warn you that we *should* be able to finally land your branches so you may want to refresh them17:12
mgzah. it... shouldn't work like that, subunit depends on testtools not visa versa, but I can imagine something broke.17:12
mgzvila: well, they'll need to head merged, they predate the NEWS split.17:13
vilamgz: yeah, so that may means shuffling the news entries anyway17:13
jamvila: so what's up with the 'temp-commit-file' being sent every hour or so? Is this just PQM fallout?17:17
jamand is there an RT for the breakage that I should be following?17:17
jamvila: I'm planning on sending a mail to the list about some possible "next things" for me. Commit to stacked branch, Branch.open() round trips over hpss, etc.17:21
vilajam: yup, pqm fallout17:21
vilajam: yes, 41340, it has been closed while testtools was installed but I think it's the one to re-open17:22
vilajam: commit to stacked branches would be nice, AIUI, some udd devs prefer checkouts17:23
vilajam: and on lp that means stacked right ?17:23
jamvila: well, it can mean stacked, yes17:24
jamI think really what they want is shallow17:24
jamwhich is something I poked at a while ago, which could really use a good hpss verb for it17:24
awilkinsI'm now getting "Connection closed: Unexpected end of message" when trying to use bzr+ssh ... sftp works for the same branch17:25
awilkinsThe server logs the startup of the remote bzr instance...17:26
jamawilkins: can you try running with -Dhpss ?17:27
awilkinsHPSS calls: 2 (1 vfs) SmartSSHClientMedium(bzr+ssh://None@cu013/)17:27
smoserjames_w, bug 67660617:29
ubot5Launchpad bug 676606 in bzr-builddeb (Ubuntu) "import-upstream stack traces without upstream repository (affected: 1, heat: 6)" [Undecided,New] https://launchpad.net/bugs/67660617:29
exarkunWhy did this start happening?  http://buildbot.twistedmatrix.com/builders/hardy64-py2.5-select/builds/448/steps/bzr/logs/stdio17:29
jamawilkins: there will be some information in ~/.bzr.log indicating what requests were done, etc.17:29
awilkinsAnd the full log : http://pastebin.ubuntu.com/533527/ ; nothing unordinary on the server I can see17:30
james_wsmoser, thanks17:30
james_wsmoser, did you try the fix?17:30
smoserjust getting there.17:30
jamawilkins: so on the client side, it looks like we sent a request and got back no content in return17:31
jam"buf[:10]=''"17:31
jammeans that there was no content in the reply17:31
awilkinsjam, this is through an ssh tunnel ; I have a shell open on the same connection which works fine, as does SFTP.17:31
awilkinsSo there is a server though which tunnels are established to get access to the servers inside17:32
jamawilkins: sure, is it ssh inside the tunnel as well?17:32
jamor is it raw bzr:// ?17:32
jametc17:32
awilkinsjam, It's bzr+ssh:// through the same tunnel that the ssh shell connection is working through.17:32
jamThese lines are a bit surprising, too:17:32
jam0.152  ssh implementation is OpenSSH 0.974     result:   ('ReadError', 'home/awilkins/ace/.repo/workbench/trunk')17:32
smoserjames_w, fix seems to work.17:33
awilkinsjam, The version, or the lack of / on the front of the path?17:33
jam(it actually looks like we got an earlier failure, and are trying to continue working, when we shouldn't)17:33
jamawilkins: try this: "echo hello | ssh <that host over there> bzr serve --inet"17:33
jamthe full string we are sending is "bzr serve --inet --directory=/ --allow-writes" in case that matters (it shouldn't)17:33
jamlooks like "cu013" is <that host over there>17:34
awilkinsjam, this is an instance of bzr in my home folder on the server and is being accessed via setting BZR_REMOTE_PATH, not via the normal PATH17:34
jamThat *should" return "ok\x012\n"17:34
jamawilkins: well, then try that first17:34
awilkinsjam, returns ok17:34
jamawilkins: you are setting BZR_REMOTE_PATH and not via config, right?17:35
jamanother thing to check is that the remote bzr is executable, but you seem to have validated that with the "echo hello" check17:36
jamyou could try:17:36
awilkinsjam, setting it on the same command line - without, you get a command not found form the server17:36
jamecho hello | ssh cu013 $BZR_REMOTE_PATH serve --inet17:36
jamto make sure you have the remote path set correctly17:36
jamawilkins: I think you might have a username set wrong:17:37
jam[ 9024] 2010-11-17 17:27:21.522 INFO: HPSS calls: 2 (1 vfs) SmartSSHClientMedium(bzr+ssh://None@cu013/)17:37
jamNone@ ?17:37
jam(we may just be displaying it when we shouldn't, because it is using the default)17:38
vilajam: I see that almost everytime17:38
vilajam: exactly17:38
awilkinsjam, That's just the parameter in the stack trace ; I've been using this config for ages though17:38
awilkinsjam, It works fine17:38
awilkins(for ssh)17:38
awilkinsIt previously worked fine for a similar server17:38
james_wsmoser, excellent, thanks for testing17:38
smoserso, now for the general bzr education17:39
awilkinsBut had to start using a different one in the same cluster (it's CentOS 5 in a collabnet cluster)17:39
smoseri did the above, and imported upstream tarball17:39
smoserand i can bzr diff -r tag:upstream-1.3.117:39
smoserbut in bzr tags output i see:17:39
smoserupstream-1.3.1       ?17:39
maxbI believe that is expected, because whilst you have imported the revision, it is not (yet) in the ancestry of your current branch, so cannot be given a revno17:40
smoserwhy is that? and what is the repercussion of it.17:40
smoserhm.. so maybe import-upstream isnt what i want then?17:41
maxbWell, at this point you are free to 'bzr merge -r upstream-1.3.1 .'17:42
awilkinsjam, I think I might just be being an idiot about how to set an env variable17:42
james_wsmoser, import-upstream is only a building block for whatever you want to do17:42
james_wsmoser, so maxb is right that a merge is one of the possible next steps17:42
smoseryeah. ok. i think i'm set now.17:43
jamawilkins: what shell are you using?17:43
smosermaxb, regarding the watch file, it does seem its incorrect. it gets a -src-deps download. i'll fix that.17:43
jamthough if it is changing from "not found" to actually reporting something, that seems like progress17:43
jam(export BZR_REMOTE_PATH=...) seems like it should work for "bash"s, "set BZR_REMOTE_PATH=..." I think is tcsh.17:44
awilkinsjam,  I'm using bash locally and on the sever17:44
jamawilkins: fwiw it seems to be failing at the first RPC, so I'm not sure yet that bzr is able to start the remote bzr correctly.17:45
awilkinswas using  #  BZR_REMOTE_PATH="/home/awilkins/bin/bzr" bzr pull bzr+ssh://<url>17:45
awilkinsWithout the first part it complains about not finding bzr, but produces the same error17:45
awilkinsWith it, it doesn't17:45
awilkins(complain about not finding the command)17:45
awilkinsI know it's running on the server because the logs are showing up the serve --inet --directory=/ commands17:46
awilkinsjam, Hmm, the serve attempts that respond to the "hello" command have a return code 0, the others have no return code.17:49
jamawilkins: you pretty much have to have a return code...17:49
jamah, you mean on the server it doesn't show as returning?17:50
jamthat sounds like it might be segfaulting17:50
jamhow did you install the code there?17:50
jam(did you compile extensions on a different platform, and then copy it?)17:50
awilkinsjam, Yes, but they are not logging it - gets as far as "bzr-svn: using Subversion 1.4.2 ()" then stops17:50
awilkinsjam, compiled in the same folder it's running in17:50
jamawilkins: using "make" ?17:51
awilkinsjam, yes17:51
awilkinsjam, bzr 2.2.1 using the tarball and building from the included c files17:51
jamawilkins: so one quick check, try going to the server, and using ~/bin/bzr log /home/awilkins/ace/.repo/workbench/trunk17:53
jamin case that segfaults for some unknown reason17:53
jamand how is ~/bin/bzr running in the same dir that it was compiled? (just a symlink?)17:53
awilkinsjam, that runs just fine ; ~bin/bzr is a symlink, but I've tried the direct path as well with the same results17:53
awilkinsjam, I'll get the server to run the selftests17:55
jamand just to be sure "ssh cu013 bzr log /home/awilkins/ace/.repo" also works, right?17:55
jam(well, add 'trunk' there)17:56
awilkinsjam, .repo is actually a standard folder, the subfolders of it are repos17:56
awilkinsjam, that works... it also works on the server (accidentally put the command there)17:57
vilaawilkins: I would try 'bzr selftest -s bp.svn' first since you mentioned it17:58
awilkinsvila, Gah, no testtools17:59
awilkinsAs an aside, I hate CentOS17:59
* vila suggests >=0.9.6 :)17:59
awilkinsyum is especially pants18:00
vilaawilkins: no bzr packaged there ?18:00
awilkinsvila, Nope18:00
awilkinsvila, I dare not add too many repositories, because the last time I did it upgraded some component to be inbcompatible with the setup these servers have where they mount your home folder from a SAN somewhere18:01
vilaweird... I was sure I saw someone mentioning updating bzr for Red Hat with a simple command... where... where did I read that (a bug report ?)18:01
awilkinsBroken LDAP library or something18:01
vilaoh18:01
* awilkins tries yum install bzr18:01
awilkinsI broke it once like that and they rewarded me by taking away my ability to trash the server image... which seemed odd, because I was going to fix it by trashing the server image...18:02
awilkinsThis is a CollabNet build farm, what do you expect, they are still in love with svn.18:03
awilkinsI hope it really annoys them that I use my home folder to mirror all the SVN repos in the project as Bazaar branches, but frankly they probably haven;t noticed.18:04
fullermdMaybe if you checked that homedir into svn...18:04
txdvsvn is the best subversioning system, all more advancved once support it, so you can easily use git/bzr/hg insteand of svn18:05
awilkinsThat's ironically "meta". Even more ironic given that the lead "programmer" on this project has actually written a versioned object database that actually stored it's repository (note- repository, not working copy) in an SVN working copy   >-o-<18:05
awilkinsThat's a pinched agony face, by the way, not a goatse-ascii (accidental)18:06
fullermdYou get extra points if your svn->bzr conversion scripts automatically notice the new stuff in svn, and check out a bzr conversion of your homedir checkin in your homedir...18:06
awilkinsjam, It is a segfault18:18
jamawilkins: well, at least I was right... now to figure out why18:18
awilkinsjam, Started another port tunnel for plain bzr:// and  ran a server on the remote shell18:18
awilkinsjam, It's something in bzr-svn18:19
jamawilkins: do you need to have bzr-svn available?18:19
awilkinsjam, start it with --no-plugins and it works just fine18:19
jamI guess you were saying everything is running in svn upstream18:19
awilkinsjam, I need it on the server shell but not when I'm pulling remotely18:20
awilkinsIt's just a staging area to reduce my download bandwidth really - some of these branches are fricking huge18:20
brianjaritahey guys... i did an apt-get update  with an apt-get upgrade on hardy and now I get "bzr: ERROR: exceptions.ImportError: ('Unable to load subvertpy extensions: %s', 'No module named client')"18:21
awilkinsSo I can work around it by moving the plugin and putting a BZR_PLUGIN_PATH in the remote script for bzr18:21
brianjaritait was fine before I did the update ... I"m running ubuntu 8.04 server18:21
brianjaritaany ideas on how to fix that?18:23
awilkinsbrianjarita, were any of the components installed manually rather than via apt?18:24
brianjaritanope18:24
brianjaritai have bzr 2.1.1-1, bzr-svn 1.0.2-2 and bzrtools 2.1.0-1 installed18:25
vilaawilkins: or use BZR_DISABLE_PLUGINS, bzr help env-variables will tell you if it's available18:27
awilkinsvila, would that work for the remote server? Does SSH pass those env variables (or just a limited subset like LC)18:28
awilkinsbrianjarita, (I keep wanting to type "brainjar"), those are not the most recent versions of those packages for Hardy in the bzr PPA (I know they're not from the default Hardy repo, because that's still at 1.13)18:30
brianjaritai see  2.2.1-0~bazaar1~hardy118:31
awilkinsbrianjarita, Could you upgrade them to the newest versions in the PPA? (bzr-2.2.1-0~bazaar1~hardy1, bzr-svn-1.0.4-1~bazaar1~hardy1, bzrtools-2.2.0-2~bazaar1~hard1)18:31
brianjaritaroot@dev:~/Source# dpkg --list|grep bzr18:32
brianjaritaii  bzr                                   2.2.1-0~bazaar1~hardy1      easy to use distributed version control syst18:32
brianjaritaii  bzr-svn                               1.0.4-1~bazaar1~hardy1      Bazaar plugin providing Subversion integrati18:32
brianjaritaii  bzrtools                              2.2.0-2~bazaar1~hardy1      Collection of tools for bzr18:32
awilkinsbrianjarita, So why do you have 2.1.1-1 above?18:33
brianjaritasorry i was on the wrong terminal18:33
brianjaritai was on lucid at 10:24AM18:33
brianjaritathat last post is hardy18:33
brianjaritaLinux dev 2.6.24-26-server #1 SMP Tue Dec 1 18:26:43 UTC 2009 x86_64 GNU/Linux18:33
awilkinsI presume you need bzr-svn :-)18:34
brianjaritabzr    2.2.1-0~bazaar1~hardy1, bzr-svn   1.0.4-1~bazaar1~hardy1 and bzrtools 2.2.0-2~bazaar1~hardy1 are all installed18:35
knittl2a format has O(1) access time for text objects?18:35
jelmerbrianjarita: it looks like you have a broken python-subvertpy18:36
jamawilkins: I imagine we only pass a limited subset of env vars, probably just the ones that ssh itself defaults to passing18:36
jelmerawilkins: Did you figure out why bzr-svn was breaking your setup?18:37
jelmerhey jam!18:37
jamknittl: they are stored delta compressed, so not exactly O(1), and the index files are btrees, so looking up the location of the text is more O(log(n))...18:37
jamhi jelmer18:37
brianjaritapython-subvertpy is 0.7.5-1~bazaar1~hardy118:37
knittljam: is there a reference somewhere?18:37
jamknittl: is there a specific question I can help answer?18:38
awilkinsjelmer, Alas, no, it's segfaulting with no stack trace, I've not got testtools installed yet 'coz I can't find a CentOS package name for it (grr)18:38
knittljam: still writing on my thesis :P18:38
knittland i have a line in it, that says »O(1) lookup time for any version of a file \todo{cite,verify}«18:38
knittlso i came to verify18:39
jelmerawilkins: what version of subvertpy?18:39
jamIMO O(1) is pretty poorly defined. And usually  means you have to actually find the terms and determine that this part is a minimal part of the overall time, etc.18:39
awilkinsjelmer, 0.7.5, compiled from tarball. It pulls SVN branches just fine18:39
knittlO(1) is generally constant and not depending on number of commits, number of branches, number of files, etc.18:39
awilkinsjelmer, I may have patched it slightly for that not-importing-pycompat thing18:40
jamknittl: I would guess that we aren't strictly O(1). If I would guess, I would at least give it O(log(N)) of "stuff"18:40
jelmerawilkins: what are you trying to do ?18:40
knittli need evidence to back that up18:40
knittlthere are no books on bzr, so bazaar internal documentation/format documentation is best i can get18:41
jamknittl: inventories (mapping Revision id => tree => actual file content) is stored as a tree structure, so that is generally a log(N) lookup.18:41
knittlwhat is N in O(log(N))?18:41
awilkinsjelmer, I'm pulling SVN branches to my home folder on this server (because the SVN repo is in the same server rack and thus it's fast there), then pulling those bzr branches via bzr+ssh:// to my local machine18:41
jamknittl: Indices that map from a given text key (file_id, revision_id) to actual file content are btrees, which tends to be log100(N) lookup18:41
jelmerawilkins: and pulling the bzr branches over bzr+ssh:// fails?18:42
knittlwhat is N?18:42
jamknittl: the raw text content is stored in a group of zlib.compress(lots_of_delta_content)18:42
jamwhich is O(N_in_the_group) which is not strictly bounded, but significatly smaller than the N of all text content18:42
awilkinsjelmer, Or if you manually configure another tunnel and use plain bzr:// from a server started on the shell ; it segfaults, but not if you use --no-plugins. The only plugin outside the default is bzr-svn18:43
jaminventory would be O(log(Num_files_in_the_tree))18:43
awilkinsjelmer, Ill just be extra thorough and try just disabling bzr-svn18:43
knittli'm still very disappointed by bzr's technical documentation :(18:43
jamfile_key => pack file is O(log100(Num_all_texts_stored_in_the_repository))18:43
jelmerawilkins: I wonder why bzr-svn kicks in at all though18:43
jamknittl: it isn't a VCS for Comp Sci majors ...18:44
jelmerawilkins: the remote directory doesn't have .svn directories or anything like that?18:44
knittllet's say i can ignore compression18:44
jamjelmer: we're looking up a branch, that generally involves bzr-svn18:44
jamknittl: zlib.decompress is going to be O(size_of_file_content)18:44
knittli said i _can_ ignore it ^^18:44
jamand given everything I've stated, is likely to be the dominating factor18:44
jelmerjam: but we check for .bzr before checking for .svn, and the .svn check happens using os.path.isdir().18:44
knittlwhy btree and not a hashmap?18:44
awilkinsjelmer, The remote folder is a bare branch in a bare repository18:45
awilkinsSorry, slipping into git-ism - --no-trees18:45
jamawilkins: is it a shared repo?18:45
awilkinsjam, It is18:45
awilkinsjam, I will try one of the other repos18:45
awilkinsI have many here18:45
jamawilkins: then is is "repo/.bzr repo/branch/.bzr/" which would mean we have to 'search"18:45
=== oubiwann is now known as oubiwann-away
jamknittl: our keys aren't hashes to start with (vs git)18:46
jamand many operations look for "similar" data18:46
=== oubiwann-away is now known as oubiwann
jamlike log going over revisions tends to look at user-2001a, user-2001b, etc18:46
jamso you get locality by using a btree vs a hashmap18:47
jamsame for text content18:47
knittlok18:47
jamdiff looks for 2 texts of file-id1, rev1, rev218:47
awilkinsrepo layout is as at http://pastebin.ubuntu.com/533547/18:47
knittljam: so where is documentation that can back up everything you said?18:47
awilkinsI like this layout because you can do switch -b with ease18:47
jamknittl: ultimate documentation is in the code :)18:48
knittli'm afraid IRC will not be allowed as citing source18:48
jamlook in doc/developers/*18:48
jamand bzrlib/btree_index.py and bzrlib/groupcompress.py bzrlib/repofmt/groupcompress_repo.py18:48
jamthere should be some slightly technical docstrings18:48
jelmerjam: but in awilkins situation we'd still not have to check for .svn as we'd find a .bzr controldir in every directory we browse, right?18:49
jelmerawilkins: Can you perhaps run your bzr serve process in gdb ?18:49
knittljam: hm. ok18:49
awilkinsjelmer, I'll see if it's installed18:49
awilkinsHooray, something that CentOS actually has18:49
jamjelmer: I don't know that we would *have* to, but I'm pretty sure I've always seen bzr-svn get invoked18:49
jamwhen it is present18:49
maxbbzr-svn gets involved far too much. Just look at .bzr.log and see it crazily attempt to open the internals of bzrdirs as svn repositories18:50
knittlbtw, do i still need to sign contributors agreement for my branch? i still don't want to …18:50
jelmermaxb: how do you mean?18:51
maxb0.480  Unable to open <bzrlib.transport.local.LocalTransport url=file:///home/maxb/wc/bzr/bzr/.bzr/repository/indices/2b60105ff8068389b09d6c735928fddd.six/> with Subversion: Unable to open an ra_local session to URL18:51
maxbfor example18:51
jelmermaxb: that's when running 'bzr branches' ?18:52
maxbrepeat for every index and pack in the repo18:52
knittljam: i cannot find O(… in the three files you named18:52
maxbjelmer: hmm... multi-pull... which probably does exercise that code path18:52
jelmermaxb: yep18:52
awilkinshow do you pass args to a program in gdb?18:52
jelmerawilkins: gdb --args /usr/bin/python /usr/bin/bzr serve18:52
* awilkins is woefully spoiled by IDE attached debuggers18:52
jelmermaxb: The probe code (and the branch find code) are in bzr, not bzr-svn.18:53
jelmermaxb: we'd be trying to open those files with bzr, git and hg as well (provided you have those plugins installed), they're just a bit less verbose.18:53
knittl»When referring to revision numbers it is necessary to mention a branch URL as well« ← is that correct?18:54
maxbHmm. I think some rethinking is in order there, but it probably can't be made quicker without also being made less extensible18:54
awilkinsgdb stack trace at http://pastebin.ubuntu.com/533549/18:54
jelmermaxb: It should be possible to make it faster, the new Prober API should make that a bit easier.18:55
awilkinsWhy is it even throwing a SubversionException when serving a pure bzr branch ...18:56
jelmerawilkins: can you print the full backtrace ("bt")18:56
fullermdknittl: Well, maybe not a URL per se, but a branch one way or another, yes.18:57
awilkinsjelmer, http://pastebin.ubuntu.com/533550/18:57
jelmerawilkins: is the branch tied to a http repository or something like that?18:58
awilkinsjelmer, the parent branch is an SVN repo over https://18:59
knittlfullermd: URL can also be a path19:00
jelmerawilkins: can you to frame 3 and print ret->url ?19:00
awilkinsjelmer, I have a thought - one thing is that these repos don't work with just "https://" you have to use "svn+https://" because they don't like being probed (return 401 errors and the like)19:00
=== deryck is now known as deryck[lunch]
awilkinsjelmer, it says "no symbol ret in current context"19:03
brianjaritayou guys were right ..... python-subvertpy is broken in hardy .... i compiled it from source and everything works now19:04
brianjaritathanks19:04
jelmerawilkins: argh, darn optimizations19:04
jelmerbrianjarita: are you using the bzr ppa perhaps, or is this "plain" hardy?19:04
brianjaritaplain hardy19:04
brianjaritahardy server 8.04 lts19:05
brianjaritaright from the live cd from ubuntu19:05
awilkinsjelmer, he must be using the ppa because he's not on bzr-1.13, he's on 2.219:05
brianjaritayea i'm on 2.219:05
brianjaritamy repos are the regular ones19:06
brianjaritaI didn't add anything to /etc/apt/sources.list19:07
brianjaritaoh haha ... another IT guy did19:07
brianjaritai see     deb http://ppa.launchpad.net/bzr/ppa/ubuntu hardy main19:07
brianjaritaso yea I'm on ppa19:07
awilkinsjelmer, How about printing the url from the kwargs, is that hard?19:08
jelmerawilkins: should be possible19:09
jelmerawilkins: "print PyString_AsString(PyObject_Repr(kwargs))" I think19:09
jelmerawilkins: I have to run in a few minutes though19:09
awilkinsI was trying kwargs[0]-> nut not sure which member to try19:10
jelmerawilkins: any luck using "print PyString_AsString(PyObject_Repr(kwargs))" ?19:20
awilkinsjelmer, nope19:21
awilkins$2 = 14790565219:22
awilkinsjelmer, More coherent results from print kwargs[n], but you just get the refcount and object type19:23
awilkinsAs an integer19:23
awilkinsjelmer, print (char*) PyString_AsString(PyObject_Repr(kwargs)) seems to work a bit better.. but it;s truncated with an ellipsis19:44
=== deryck[lunch] is now known as deryck
awilkinsjelmer, the url is 'file://'19:47
jamlosa ping: Last I heard, bzr's PQM was still broken because of needing the new subunit to be installed. I don't know if that has been addressed or not. vila and mthaddon were working on it, but both are past their EOD20:40
joeyhowdy,20:59
joeyanyone know how to work around this?20:59
joeybzr: ERROR: No such file: u'/home/joey/canonical-docs/.bzr/repository/pack-names': [Errno 2] No such file or directory: u'/home/joey/canonical-docs/.bzr/repository/pack-names'20:59
joeyI went do to a commit and got that error20:59
beunoouch20:59
pooliehi joey, beuno, jam20:59
pooliejoey: what happened?20:59
beunoheya poolie!20:59
joeyhi poolie!21:00
joeybzr: ERROR: No such file: u'/home/joey/canonical-docs/.bzr/repository/pack-names': [Errno 2] No such file or directory: u'/home/joey/canonical-docs/.bzr/repository/pack-names'21:00
pooliethat's unusual21:00
joeyI hadn't used that branch in a while and went to do a commit ....21:00
pooliedid your machine crash recently?21:00
poolieah, or maybe not recently21:00
joeystatus, check, pull, update ...all get the same error21:00
jammorning poolie21:00
poolieyeah, i'm not surprised21:00
joeyyeah it has crashed but not in the middle of an update21:00
pooliethat's a pretty fundamental file to lose21:00
* joey looks in lost.files21:01
pooliejam, thanks for your mail21:01
=== Ursinha is now known as Ursinha-bbk
=== Ursinha-bbk is now known as Ursinha-bbl
joeylost+found is empty21:02
joeyI suppose I can just suck down the branch from LP and do a diff21:02
poolieso can you file a bug, and then21:02
poolieit should not be too hard to recreate the file, but i don't know if there's any command line way21:03
pooliebug titled 'pack-names file is missing'21:03
joeywell I have no idea how it would have happened21:04
poolieit's ok21:04
joeyno idea on how to reproduce it21:04
pooliewe should still have a way to recover from it21:04
poolie'rm pack-names' will reproduce it adequately :)21:04
joeyoh hell21:04
joeyyou are NOT going to believe this21:04
pooliehowever, it's probably only about low priority, since if your machine is going to lose files, there's a limit what we can do21:04
joeypack-names.u1conflict21:04
joeyI must have forgotten it was under RCS21:05
beunou1conflict21:05
beunoouch21:05
joeythe reason I don't use U1 :-)21:06
beunoso that was ubuntu one's fault21:06
beunohehe21:06
joeyI've invited Matt G and few others to my house to help them fix it :-)21:06
beunoI have Matt G trapped in my place all week21:06
jambeuno, joey: AIUI u1 is not transaction safe wrt what bzr does21:06
beunoplus rockstar and others21:06
jamwe write things in a certain order, but U1 doesn't know that21:06
joeyok so renaming that file worked!21:06
joeythanks O masters of all things bzr bizarre21:07
beunojam, I thought they had fixed that, bzr was one of the testcases21:07
jamso you can get repo level corruption if u1 decides to merge things21:07
jambeuno: if I commit here, and you commit there, we end up with 2 different pack files added to pack-names21:07
jamwhich would then conflict (I assume)21:07
jamand you need to merge those lines together.21:07
beunoah21:07
beunoright21:07
joeyjam: wc but any idea on how to tell u1 to stop synchronizing?  When I issue it from the gui it just fails silently21:07
jamnot to mention that you could sync pack-names before you upload the associated .pack files, etc.21:08
jamjoey: beuno is the one to ask about that21:08
beunojoey, so, where is this located on your filesystem?21:08
jambeuno: In an eventually-consistent state with a single committer, u1 will probably work21:08
jam(for example, you can rsync a live repository, as long as you eventually get a complete snapshot)21:08
beunojoey, go to:  https://edge.one.ubuntu.com/files/21:09
beuno"stop syncing folder" under the more option21:09
joeybeuno: that worked. Many thanks.21:11
beunojoey, np. I fixed that yesterday  :)21:11
beunosprinting FTW21:11
pooliejam, thanks for your mail about lp-serve21:14
pooliealso i spoke to spiv and he's very keen to work with you on hpss, and he's going to try to wake earlier to overlap more with you21:15
jampoolie: francis had a decent response to it21:16
pooliejam btw if you're still online, you might want to join #ubuntu-meeting21:37
spivjam: good morning, btw!21:39
jammorning sp21:39
jamspiv21:39
spivjoey: also, perhaps check if there's a pack-names.old file or something in .bzr/repository somewhere.21:40
spivjoey: oh, I see you found a file to rename :)21:41
pooliespiv, can you join #ubuntu-meeting too?21:45
jamspiv: there was a question about https://bugs.launchpad.net/udd/+bug/65330721:46
ubot5Launchpad bug 653307 in Ubuntu Distributed Development "Import fails with missing referenced chk root keys (affected: 1, heat: 6)" [Critical,In progress]21:46
peitschiemornin all :)22:10
spivjames_w: see my latest comment on 65330722:13
james_wspiv, replied, thanks22:16

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