/srv/irclogs.ubuntu.com/2011/04/14/#bzr.txt

pooliehi all00:08
mgzhey poolie.00:09
pooliehi there00:09
mgzpoolie: what do you want done with the Ubuntu bits of theses new exciting bugs you've made public? jam suggested closing them after opening an upstream task, but I'm not sure I have perms to do that.00:25
pooliemgz i don't know00:26
mgz...where 'bits -> 'bug task'00:26
pooliehm00:26
* mgz needs to clearer00:26
pooliei guessed00:26
pooliein some ways, if they are not going to be *specifically* fixed in upstream00:27
pooliethey should be 'wontfix'00:27
pooliebut, that sounds a bit negative to the reporter perhaps00:27
poolieyou could just make them mirror the upstream tasks?00:27
maxbWouldn't it make sense to leave the Ubuntu tasks open, until we eventually close them via LP: #nnnnn in debian/changelog ?00:28
pooliemm that could be good00:28
pooliein that case then triaged or confirmed, and matching importance00:29
mgzwell, that's like the 'fix released' issue poolie posted about the other day.00:29
mgzunless there's some automation there already?00:29
maxbWell, for Ubuntu package tasks, there's an automated robot which closes bugtasks from debian/changelog uploads00:30
mgzhow does it get in the changelog though?00:31
pooliemanually i think00:31
pooliewell, perhaps for a new major release there could, or perhaps is, a script to gather it?00:32
dchiltonHi I just launched a series of lp:whatever updates with bazaar @ a bunch of different machines ... everyone's returning: "ssh: connect to host bazaar.launchpad.net port 22: No route to host".  Is anyone else seeing anything?  Not sure if there's a status page to check ...00:32
mgzI got that just now, branched locally instead, launchpad likes midnight updates.00:33
dchiltonmgz: ah, didn't realize launchpad's in Europe ...00:35
mgzhttp://identi.ca/launchpadstatus/ <- doesn't look planned this time.00:35
dchiltonmgz: a status page!  evern better, thx!00:35
dchiltongood excuse to go get a beer early ... thx!00:37
mgzand I need to sleep, will have to do mps tomorrow.00:40
pooliesleep well00:50
mgz...fell into the just one more bug trap. now I'm really going. :)00:58
spivGood morning.01:01
pooliehaha01:04
pooliehi spiv!01:04
=== r0bby is now known as robbyoconnor
=== psynaptic|away is now known as psynaptic
* spiv blinks01:26
spivmutt crashed.  That's new!01:26
bob2heh, it reliably crashes every night for me while doing imap idle01:27
spivI use offlineimap to handle the actual imap01:28
spivI'm not sure I really want to send a 1.7GB apport report though!01:30
spivI'm rather impressed than an 8MB crash file can apparently expand that much…01:32
spivAh, no, apport's GUI is just lying to me.01:33
AfCjelmer: Doubt you noticed in the avalanche of bugmail, but regarding the problem pulling from GNOME git trying to clone GTK, I discovered04:22
AfCjelmer: that you can duplicate with the public (anonymous [sic]) git:// URL as well as the git+ssh:// one. So with any luck you might be able to duplicate the crash I'm experiencing04:22
AfCjelmer: [all on the bugs are bad but encountering someone else's reproducible bug might find something deeper that does matter]04:23
pooliespiv do you use tribunal or subunit on pqm failures?04:45
poolietribunal seems to not be coping with them recently04:45
pooliejust started digging into it before lunch04:45
poolienm, stupid bug04:47
pooliespiv i'm going to backport the importwarning fix to 2.1, 2.2 and 2.305:05
poolieit seems necessary to land anything on those branches in lucid05:05
pooliethat code didn't exist in 2.005:05
pooliedo you see any risks?05:05
pooliesent anyhow05:17
spivpoolie: sorry, was out for a bit over lunch.  The importwarning fix seems ok... oh, but is it in 2.4?05:37
spivpoolie: Hmm, no, docs say ImportWarning is new in 2.505:37
spivpoolie: I don't use tribunal for that, partly because I happily don't get too many failures back from PQM, and partly because it just doesn't work quite well enough :/05:38
poolie:/05:38
spivSubunit streams seem a bit fragile, so it seems like half the time I have to try random filtering options to use them.05:39
pooliethat is true05:39
pooliethere was an embarassing total-failure bug in trunk, the presence of which indicates not that many people are using it05:39
pooliewhich is fine05:39
poolieanyhow, if you hit stuff that's not stream fragility, let me know05:39
spivBut also each time I use tribunal I get itchy to make it better, rather than work on what I was originally doing :)05:39
spivIt's been a while since I've tried though05:39
poolieyeah05:39
pooliethat actually puts me off using it :)05:40
poolietoo many yaks05:40
pooliein this case i did though05:40
pooliei might fix bug 760387 now though05:40
ubot5Launchpad bug 760387 in Bazaar "failures are not easily visible in pqm make check error output" [Medium,Confirmed] https://launchpad.net/bugs/76038705:40
pooliehow is the pack code?05:40
spivFairly good.  I've mostly extracted out the state and logic I wanted to extract into its own class, although it's still a bit more bound up with the original class than I'd like05:47
spivI've got some basic tests using that, and I'm about to start writing tests for the edge cases that brought me here in the first place.05:48
poolielifeless: hi, just quickly, can you tell me what if any special expectations pqm has for subunit support?06:36
pooliedoes it look for a selftest.log file?06:36
lifelessno, it parses stdout06:36
poolieor does it just expect that stdout might be a stream?06:36
pooliethat's all?06:36
lifelessyup thats it06:36
pooliedo you recall what it is that generates the summary in the pqm rejection mail?06:40
pooliewell, not a summary, but a list of skipped tests etc06:40
lifelesspoolie: sorry, I'm looking at this cisco issue07:05
lifelessthe mail body is generated by pqm using the subunit library07:06
lifelesspretty straight forward to change if you want it different07:06
poolienp07:07
pooliethat'd be something to change in pqm's code?07:07
lifelessyup07:11
lifelesspqm/output.py07:14
lifelessclient = subunit.test_results.TestResultFilter(client)07:14
lifelessconstructs the used filter07:14
lifelesswhat do you need changed?07:15
pooliei'd like it to just tell me about the tests that actually failed (!)07:16
poolieat the moment the output is enormous and mostly irrelevant07:17
pooliehttps://bugs.launchpad.net/bzr/+bug/76038707:17
ubot5Ubuntu bug 760387 in Bazaar "failures are not easily visible in pqm make check error output" [High,Confirmed]07:17
lifelessI think thats a combination of things; its showing skips which changing the line to include filter_skip=Tre as a parameter will address07:19
lifelessand the skips have an attachment which is useful when debugging why something skipped07:20
lifelessbut not great en masse07:20
lifelessspm: ping07:20
spmyo07:20
lifelesson balleny, in the pqm checkout that pqm runs from07:20
lifelesscan you change07:20
lifelessclient = subunit.test_results.TestResultFilter(client)07:20
lifelessto07:20
lifelessclient = subunit.test_results.TestResultFilter(client, filter_skip=True)07:21
lifelessin the file07:21
lifelesspqm/output.py07:21
lifelessif this works and makes poolie happy, I'll commit said change to the public branch07:21
spmlifeless: hrm. that'd be a no. I can't get to balleny atm07:22
lifelessah07:22
lifelessE-pic routing07:22
spmpurple routing?07:22
spmI lie! I'm in!07:22
lifelessyou are such a tease07:22
spmit was staring at me for ages diong nothing....07:23
lifelesssadface07:23
lifeless    raise errors.NoWhoami()07:23
lifelessbzrlib.errors.NoWhoami: Unable to determine your name.07:23
pooliehooray cowboys07:23
poolielifeless: on where?07:23
lifelesspoolie: pqm test suite07:23
spmlifeless: ~ line 97?07:24
lifelessspm: zigactly07:24
spmlifeless: changed07:24
lifelesspoolie: toss something broken at it07:25
pooliei think it's full at the moment07:26
pooliei'm sure i will give it something broken in good time07:26
pooliethanks for that07:26
pooliere: reliability, the problem may be that other output from bzr or pqm or make sometimes gets into the stream07:30
pooliein some ways it is in an unhappy compromise between being strictly defined and requiring a reliable transport, and being fuzzy07:30
lifelesspoolie: I would like it if you can tell me in the next week whether its better or not07:31
lifelessso we don't have an extended cowboy07:31
lifelesspoolie: pqm keeps stdout and stderr separate; it is still possible to have things messed up of course.07:32
lifelessbut stderr/stdout interleaving is the most common cause07:32
pooliei will tell you07:32
pooliethere are some things in bzr that leak to stdout07:33
pooliealso ,tribunal is kind of buggy07:33
poolieok i sent in an intentionally buggy merge07:35
poolieit should get to the top of the queue in a couple of hours07:35
lifelesscool, thank you07:41
poolielifeless: btw next in the queue is my fix for bug 751824 which might be related to your nowhoami thing07:53
ubot5Launchpad bug 751824 in Bazaar "whoami-related test failures with bzr.dev" [High,In progress] https://launchpad.net/bugs/75182407:53
pooliewell, it's _related_ though i don't know if it will fix it07:54
lifelesspossibly07:54
lifelessbasically pqm gets few edits07:54
lifelessso it only encounters bzr api changes infrequently07:54
jammorning poolie07:58
pooliehi there jam07:58
=== Ursinha is now known as Ursinha-afk
spivpoolie: I think the ImportWarning fix needs to be updated to support Python 2.4, which doesn't have ImportWarning.08:48
poolieah, yuck08:54
pooliejust when it finished merging all the way up too08:54
pooliespiv i actually found a better way to do it, which is to just set a -W option08:55
poolieImportWarning is ignored by default even in 2.7 and i don't think it will actually help with anything to turn it on08:55
pooliejust for curiousity, did you just think of that, or did you notice in babune?08:57
pooliespiv: a great example that nothing is truly trivial i guess09:04
pooliespiv: hm the last ubuntu release with 2.4 is karmic which is eol now, or really soon09:12
pooliebut better not to accidentally break it09:13
poolielifeless: i got the result from your pqm tweak09:13
poolieit's still including knownfailure/xfail results09:14
poolieit would be nice to exclude them too because they're not relevant to any particluar landing09:14
poolieaside from that it looks really good09:14
poolie(not urgent)09:15
lifelessthat may need a tweak to subunit to know about that class in the filter module (or I may have an old version in my lp vm)09:16
lifelesspoolie: remind me monday09:16
pooliesure09:16
pooliethanks for the patch09:16
pooliespiv yes that does fail on 2.4, not surprisingly09:20
poolieok i'm out for a bit, good luck09:30
=== Ursinha-afk is now known as Ursinha
=== zyga_ is now known as zyga
lifelesspoolie: http://iatp.typepad.com/thinkforward/2011/04/us-subsidizes-brazilian-cotton-to-protect-monsantos-profits.html may entertain you10:34
spivpoolie: I didn't check babune, the 2.4 issue is just something I thought of glancing at that patch for probably the 4th time in the last week or so12:29
spivpoolie: the definition of “trivial” certainly isn't trivial!12:29
poolie:)12:31
jelmerspiv: +112:34
jelmervila and I explicitly talked about the fact that ImportWarning wasn't in python2.4 when we were discussing why the ImportWarning didn't happen on hardy.12:35
jamjelmer, poolie: So... time to deprecate python 2.4 support?12:43
jamWe've been searching for an excuse12:43
jelmerjam: I don't remember what came out of the discussion in January; does bumping the minimum version to 2.5 gain us much?12:44
jamjelmer: clearly it gets us ImportWarning12:44
jelmerwell, apart from that extremely exciting feature :)12:44
jamjelmer: with statements12:44
pooliejam, i think we should drop it in trunk12:45
jamtry/except/finally12:45
jamjelmer: a much easier way to migrate to a python3 code base12:45
jamnote, though, that 2.5 still doesn't let us use "bytes()"12:45
jampoolie: AIUI, the primary reason to support it was that RHEL5 only has python-2.4 and will be supported by RH for another 5 years or so12:46
=== psynaptic is now known as psynaptic|away
jamand if we introduce a new repo format that users need to upgrade to, they won't be able to get that on their RHEL systems.12:46
jampoolie: but personally, I'm probably in favor of dropping 2.4 support in trunk. Especially since RHEL6 is officially out, so RHEL people have an upgrade path they can use.12:48
rom1hi all14:04
rom1I am checking a problem that i have with bzr-xmloutput14:05
rom1about the ssh connections not closed until the rpc server is closed14:05
rom1I am a bit lost to understand the flow to open a ssh connection through bzrlib14:05
rom1Is there a developer doc describing this ?14:06
rom1Or can you give me an api that i could  check ?14:06
jelmerrom1: hi14:08
jelmerrom1: The ssh connection is opened from a SFTP or SSH Transport, both of which get created when you open a Branch, Repository, BzrDir, etc at a remote URL14:09
jelmerrom1: there isn't any functionality to close ssh connections as far as I know, I think there's a bug about it in bzr14:10
pooliemm, i think there is a disconnect method14:10
pooliebut can you say more about just what's going wrong?14:11
rom1thx jelmer14:11
rom1poolie : in fact, i have many users using bzr-eclipse14:11
rom1and for each commands sent to a remote repository, a connection is openned and never closed14:12
rom1connections in bzr+ssh14:12
rom1I had up to 400 connections for one user14:12
rom1I have detected in fact taht's the xmlrpc server that keeps openning connections without nor closing them nor reusing them14:12
rom1and i've seen that closing the rpc server, the connections are closed.14:13
rom1i haven't this issue using directly the bzr command line14:13
rom1so i try to see if i can force the connection closure in the rpc server (i am talking about the server from bzr-xmloutput)14:14
poolierom1, there is a Transport.disconnect method14:15
pooliethe key thing is to decide when it ought to disconnect14:15
poolieand indeed why it's not reusing the existing connection if it is keeping them open14:15
rom1indeed14:15
rom1from what i see in the code, it calls the commands.main method14:16
rom1code of the rpc server14:16
rom1does bzr normally reuse existing connections ?14:16
poolierom1 only when the transport object is reused14:18
pooliethere's no implicit cache14:18
poolienow i need to sleep though...14:18
rom1good night poolie, and thx !14:18
jamnight poolie14:18
=== psynaptic|away is now known as psynaptic
AfCOh man, I'm doing a bzr clone of a git repo, and it's "fetching revisions" at about 1 per second with 100% CPU. That's ok. Only 30,000 to go.15:34
LeoNerdLast time I tried doing that, it ended up bombing out because it'd filled the /tmp partition with a huuuuuge binary blob file15:35
LeoNerdI haven't touched it since15:36
AfCI'm going to have to leave this running overnight. Wow.15:41
sorenAfC: A public get repo?15:43
AfCsoren: yes,15:44
AfC$ bzr clone git://git.gnome.org/gtk+15:44
sorenhttps://code.launchpad.net/~vcs-imports/gtk/trunk15:45
sorenIf you'd rather go that route.15:45
AfCsoren: that's a _conversion_ isn't it?15:45
AfCsoren: not a bzr-git repo?15:45
maxbAfC: It's a bzr-git repo15:46
AfCoh15:47
maxbwell, branch15:47
AfChm15:48
AfCwell, shit, ok15:48
AfCDo you know if they're globally unique and repeatable? If I then use bzr-git myself against upstream am I going to end up with the same revids?15:50
james_wAfC, yes, it is deterministic15:51
james_wthat branch is what you would get if you waited for yours to finish15:52
AfCjames_w: That's the word I was looking for, thanks. Clearly it's zzz time15:52
jelmerAfC: Are you using bzr and dulwich with compiled extensions?15:53
AfCjames_w: very cool. Thanks. Pity it takes so long. Most people trying this wouldn't come here to ask / wait for it to finish [in the damn, more bad press sense]15:53
AfCjelmer: hey15:53
jelmerAfC: The gtk+ conversion took less than an hour here15:53
jelmerAfC: Hi! Sorry, missed the conversation here earlier.15:53
AfCjelmer: I couldn't say - I'm using whatever is packaged in the Bazaar PPA15:53
jelmerthat should be the packaged bits.15:54
jelmerAfC: Are you using the nightly PPA or another one?15:54
AfCjelmer: no releases. You guys have years of track record of warning people that dailies are broken. I don't need to try them myself to experience it.15:54
AfCdeb http://ppa.launchpad.net/bzr/ppa/ubuntu maverick main15:55
AfCjelmer: ^ that one15:55
jelmerAfC: snapshots may be broken, but the PPA packages run the testsuite before they're built15:55
jelmerAfC: The followup on the bzr-git bug report, was that using a release?15:55
AfCjelmer: yes15:55
AfCjelmer: (I said so there :))15:56
AfCjelmer: I've been working on your request to blow it away and start again15:56
jelmerAfC: is that 0.6.0 or something earlier?15:57
AfCwha? It's whatever you released into the Bazaar PPA15:57
* AfC looks15:57
AfCyes15:57
AfCgit 0.6.015:57
jelmerok, that should be recent enough15:58
AfCWow. It did a repack at 10,000 revisions and now it's back to 1/second.15:58
AfCOnly 22,000 to go15:59
* AfC is astonished this only took you 1 hour15:59
jelmerAfC: do you have bzr 2.3 or 2.4 ?15:59
AfCjelmer: whatever is released by the Bazaar project in the PPA.15:59
AfC$ bzr --version16:00
AfCBazaar (bzr) 2.3.116:00
=== psynaptic is now known as psynaptic|food
jelmerI'm running 2.4. I'd be surprised if it made that much of a difference though.16:00
* jelmer tries another clone of gtk+16:01
AfC{shrug} don't want to waste your time16:01
jelmerit's not that much effort :)16:01
AfCbut it seems like some profiling of this run might bear some fruit16:02
AfCI guess I should abort this and just use ~vcs-imports16:02
jelmerit'd be great to know if it was indeed related to the shared repository you were doing the clone in16:03
AfCjelmer: but I will let it continue & then attempt to pull in it periodically over the next few weeks so as to confirm that bug is gone16:03
AfCjelmer: I'm disappointed that the branch's revisions got corrupted like that, though. I've never experienced that with Bazaar before.16:04
AfCjelmer: obviously we don't want that to happen, so I wish I could give you more information.16:05
jelmerAfC: it's not corruption, it's just that bzr-git can't regenerate the original git commit from the bzr revision16:05
jelmerthough I admit the error message looks a bit scary16:05
AfCjust a little [and to an end user, it's corruption :)]16:05
AfCWell, I will leave this running. I'll check back tomorrow16:06
=== beuno is now known as beuno-lunch
=== psynaptic|food is now known as psynaptic
=== Ursinha is now known as Ursinha-afk
=== beuno-lunch is now known as beuno
=== psynaptic is now known as psynaptic|away
BasicOSXdo! heh Adium on Colloquy!22:12
pooliehi BasicOSX!22:20
BasicOSXhello22:23
BasicOSXwrong channel :-)22:23
poolie:)22:28
kirklandmaxb: hey, i finally got around to testing your lp:~maxb/bzr-builddeb/no-error-unknown-distro branch ...22:58
kirklandmaxb: it gets me past the first error, but still another blocker:22:58
maxbhi22:58
kirklandmaxb: bzr: ERROR: Inconsistency between source format and version: version is native, format is not native.22:58
maxbuh, well, that's slightly good :-)22:58
kirklandmaxb: :-)22:58
maxbcat debian/source/format and head -n1 debian/changelog say what?22:59
kirklandmaxb: okay, i worked around that one by removing debian/source/format23:00
LeoNerdIt strikes me that 'bzr missing' is slightly misnamed, if it reports that the local branch has more than upstream does..23:06
maxbnah, it tells you what upstream is missing :-)23:09
LeoNerdHmmm... I suppose that's true23:11
knighthawkI just did a bzr svn-import23:17
knighthawknow when I try to create a branch I get the following errr23:17
knighthawkzr: ERROR: Not a branch: "/home/pluvious/wc-grc/.bzr/branch/": location is a repository23:18
knighthawkis there something I need to do first to access it?23:18
knighthawkI also tried to do a checkout but I think I got the same error23:19
mr-russ1knighthawk: you don't branch the .bzr23:19
mr-russ1wc-grc is the repository that hosts branches.23:19
mr-russ1you should be able to branch from  wc-grc/trunk23:20
knighthawkmr-russ1, thanks23:20
knighthawkI may have a layout issue.23:20
mr-russ1knighthawk: probably, I've found it painful to get svn converted to bzr the way that I'd like.23:20
mr-russ1repository is a collection of branches.  it's there to save space as the common revisions are only stored once.23:21
mr-russ1repository is really good for projects with multiple branches.23:22
mr-russ1off to work for me now.23:23
maxbknighthawk: So, wc-grc was the root directory created by the svn-import?23:23
maxbIt should contain within it a number of other directories which mirror the structure of branches within the svn repository23:23
knighthawkmaxb yeah I just had to drill down to the trunk23:30
knighthawkseems to be working now. only I did a branch instead of a checkout so I'm not all that sure how many extra steps I'm going to have to do to have it work like svn (well sort of like svn)23:30
pooliehi all23:36
spivpoolie: Good morning23:38
spivpoolie: http://albertomilone.com/wordpress/?p=50223:38
pooliethat's good23:39
poolieo/ cinerama :)23:39
cineramahi!23:39
pooliespiv i'm finding i get by fairly ok with hamster just in the launcher23:41
jelmer'morning poolie, spiv23:42
pooliehi jelmer23:42
pooliei guess first of all i should revert the importwarning landings, unless someone else already did23:42
jelmernobody has yet, I wasn't sure what the right thing to do was23:43
jelmerI guess we can fix it to use ImportWarning if available and otherwise just catch ImportError /23:43
pooliei think the right thing is -Wignore::ImportWarning as long as that works on 2.423:44
pooliei think it will, but i'll test it now23:44
poolieit seems to23:45
poolieso i'll revert the change to the except statement and add that to the makefile?23:45
jelmerwfm23:45
spivjelmer, poolie: see my comment on the MP23:47
spivYou can just do "try: errors = (ImportError, ImportWarning) except NameError: errors = ImportError" then use "except errors"23:47
spivOh, jelmer just said basically that.23:48
spivI don't have a strong opinion on -Wignore::ImportWarning vs that23:48
spivI guess we don't need that normally because normally we don't use -Werror?  Does the -Wignore::foo stack nicely with -Werror?23:49
poolieyes, as long as you specify them in the opposite order to what the manual implies :)23:50
poolie-Wignore is shorter :)23:50
pooliearguably we should actually just blacklist with -Werror warnings we do actually care about23:50
poolieif this happens again i'd probably do that23:51
pooliebiab, natty upgrades...23:53

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