[00:38] morning [00:38] morning Ian. [00:38] request: could you look at https://code.launchpad.net/~simon-kersey/bzr-explorer/568728-fix/+merge/24306 at some point? [00:39] I did a drive-by comment on in a while back, and don't want the guy thinking I'm ignoring him (I just don't know if the code is right) === davidstrauss_ is now known as davidstrauss [00:41] mgz: it would be churlish of me to chime in on the subject of Launchpad as an OpenID consumer, wouldn't it? :-) [00:42] anyway, I'm given to understand that wheels are in motion to get a proper OpenID consumer in Launchpad, so that makes me optimistic. [00:43] mgz: I hope to get back to code reviews today [00:44] igc: glad to see you back in the saddle [00:44] thanks bignose! [00:44] great, thanks. [01:10] hi guys [01:10] anybody care to help a bit? [01:11] I just nuked my bzr repo [01:11] but I have a crapload of checkouts (not branches, checkouts) here and there - what would be the best way to get the newest revisions in the new repo? [01:12] 1. restore a backup (40 revisions back), and in checouts do bzr up; resolve conflicts, bzr ci? [01:12] 2. just ignore the backups and move a checkout as the new repo? [01:13] 3. do bzr pull IN the backup-restored repo FROM the checkout? [01:13] 3 sounds plausible to me [01:13] 4. bzr push the checkoput INTO the backup-restored repo [01:13] as long as you hav heavyweight checkouts, not lightweight (the default is heavy) [01:13] 3 and 4 are equivalent [01:14] heavy, heavy [01:14] okay, great [01:14] so, restore a backup, bzr push/pull [01:14] trying. === spiv_ is now known as spiv [01:22] spiv: hi [01:22] mwhudson: also ping [01:22] lifeless: hello [01:22] bzr smart server in lp [01:22] what diagnostics can I get on that ? [01:22] not terribly much easily :/ [01:23] also, is it still running as a subprocess; and what happens to its stdin/stdout - are they connected to twisted ? [01:23] i think unhandled exceptions should go into oops reports [01:23] yes and yes [01:23] trying to figure out why we only get 21KB/s from lp; gathering data as first step [01:23] it's running bzr lp-serve user-id --inet [01:23] shoving this under udd as 'slow checkouts hurt distro users' [01:23] ish ayway [01:26] ok [01:26] do we have statistics / info gathering on the twisted master [01:26] I'm particularly interested in socket windows (raw and ssh wrapped), packet delays and so on [01:26] lifeless: morning. [01:27] spiv: python-dev, antoine's message 'Summing up' may be of interest to us and the smart server in standalone mode [01:27] lifeless: not in any very structured way [01:27] mwhudson: is that yes if I ask nicely? or 'no, but I didn't want to say that' ? :) [01:28] lifeless: ta, will look [01:28] lifeless: it's 'conch dumps some crap into a log file about this sort of thing, but i doubt it's of any use' [01:29] * mwhudson sucks a 50 meg file off launchpad over bzr+ssh using hitchiker and watches the network in system monitor [01:29] it's quite surprising [01:30] do tell [01:30] lifeless: http://people.canonical.com/~mwh/sm.png [01:31] yes [01:31] there is something wrong ;) [01:31] it ramps up slowly to a decent rate, then drops right down and slowly ramps up again [01:32] it's a bit odd [01:32] I would expect this as a buffering interaction [01:33] if a pipe somewhere fills [01:33] process swaps/stops getting time slices [01:33] the tcp window may not open enough, and can take time to recover [01:34] http grab of the same file is also behaving a bit strangely, but very differently [01:36] http://people.canonical.com/~mwh/sm2.png [01:44] lifeless: worked like a charm, thanks [01:46] guys, maybe it would be a good idea to estimate, what's the average bandwidth in both situations? [01:47] I mean, the graphs show completely different behaviour, true, but maybe the overall average bandwidth at the end of the downloading is more or less the same? [01:47] Hey, does anyone know if there's already a bzr equivalent of the git-version-gen autoconf script? [01:48] I know I can adapt that to be for bzr, but I figured I'd see if someone had already done it [01:48] if so, maybe it would be a good idea to try wgtetting or whatever with a fixed max rate, just below that average [01:48] to see if anything strange happens, or not [01:55] lamalex: not that I know of [01:55] lamalex: I've done something similar for setup.py [01:55] lamalex: and drizzle does something similar too [01:56] mtaylor: ^ show lamalex the bzr version stuff for drizzle [01:56] lifeless: aroo? [01:56] mtaylor: so http peaks at 1mb [01:56] bah [01:56] lifeless: oh - gotcha [01:56] mwhudson: so http peaks at 1MB [01:56] and bzr+ssh at 512KB [01:57] lamalex: it's all in pandora-build ... (which you should use for your autoconf needs anyway) ... but specifically [01:58] lifeless: yeah, after the initial fast burst though http followed a vaguely similar pattern to bzr+ssh [01:58] lamalex: http://bazaar.launchpad.net/~mordred/pandora-build/devel/annotate/head:/m4/pandora_vc_build.m4 [01:58] lamalex: I've got to run at the moment - are you going to be around in 5 hours? [01:58] spm: whats that darn tool you swear by [01:58] less of a sawtooth i guess [01:58] lifeless: 'find' [01:58] spm: not swear at [01:58] * mwhudson was going for 'awk' [01:59] mtaylor, no probably not [01:59] but maybe [01:59] lifeless: a hammer? [01:59] context may help - what are you trying to do :-) [01:59] spm: for network window / efficiency / analysis [01:59] lamalex: ok. well, I can certainly help you out on this if you still need it when I get back or tomorrow [01:59] AHhhhhh! tcptrace + xplot [01:59] mtaylor, thanks [01:59] lamalex: sure! [02:00] yikes, m4. [02:00] isn't there a better templating solution that's dependably installed? [02:00] http://www.tcptrace.org/ and http://www.tcptrace.org/manual/node12_mn.html specifically. sheer gold. [02:01] mwhudson: what file were you testing with [02:01] bignose: well, not that's integrated with autoconf, no [02:01] s/better/more human-maintainable/ [02:01] lifeless: http://bazaar.launchpad.net/~vcs-imports/linux/trunk/.bzr/repository/packs/7590053cc142a5cb7ca8d0e33022ce2a.pack [02:03] lamalex: that particular macro encodes YYYY.MM.BZR-REVNO into the version ... as we're the only user of that particular function, I haven't generalized it yet. [02:03] ok. I'm off. have fun all [02:17] hi [02:17] how can I move everything in the root dir of my branch to a subdir? [02:17] like: [02:17] bzr mkdir src [02:18] bzr mv . src/myproject [02:18] which of couse does not work ;) [02:19] bzr mv $(ls | grep -v src) src/myproject [02:19] hmm....that will fail because of some files that aren't under version control [02:19] well, adjust appropriately [02:20] perhaps bzr ls --versioned [02:21] Need a bzr pivot_root... [02:21] fullermd: wouldn't address this case [02:21] fullermd: (and we have it) [02:21] lifeless, well, thx....that works ;) [02:21] only .bzignore shouldn't be moved I guess ;) [02:21] right [02:21] but I will simply bzr revert it [02:21] you'll want to mv that back :) [02:21] or that, sure [02:23] We do? Where? [02:24] tree transform [02:24] it can handle a merge that moves the root [02:24] we don't have a userspace command for it, yet. [02:24] Yeah, but that's not much of a UI ;) [02:27] mwhudson: what ISP do you have? [02:30] lifeless: vodafone [02:31] but my local connection is through a telecom 'cabinet' [02:31] for ADSL ? [02:31] yar [02:31] I'm peaking at ~ 128K [02:31] which is, frankly, sucky. [02:31] oof [02:31] what does your modem think you're synced at? [02:32] dunno, don't have the admin password for it [02:33] are you still in the short term rental place? [02:33] yes [02:33] 3 months 1 week to go [02:34] Not that he's counting down or anything. [02:34] i'm not too surprised that your internet sucks then [02:34] hah [02:34] its a private residence the owner is on sabbatical, as opposed to a serial rent environment [02:34] ah ok [02:34] we've taken over the phone account etc [02:34] I suspect a call to telecom is in order [02:35] maybe the owner was on an old, capped plan? [02:35] 128K is 1Mbit [02:35] noone seems to bother with that any more though [02:35] I changed the plan to the upsized one [02:35] one possibility is an adsl1 modem [02:35] another is bad wiring [02:36] distance to the exchange? [02:36] who knows [02:36] telecome knows [02:36] thats who knows [02:39] spm: is ParseOptions: packet 41290 TCPOPT_SACK option truncated, skipping other options [02:39] spm: a meaningful worry - just use a longer snaplen and repeat? [02:50] lifeless: not sure. I usually grab the entire snaplen - debugging http so you want to see the data. [02:50] * igc lunch [02:50] heh, in this case the data is just a pack so 'meh' [02:51] or ssh, so equally [02:51] though [02:51] we may want to write a ssh analyzer [02:51] if we can get the session key out of twisted ? [02:51] mwhudson: ^ [02:51] mwhudson: would that be a lot of work, do you think? [02:52] lifeless: no idea at all [02:52] it would be at least a moderate amount of work [02:52] mwhudson: try suggesting it's impossible perhaps? [02:52] spm: -s doesn't seem to give me a graph [02:53] lifeless: -S create time sequence graph[s] [02:54] wrong case. [02:55] the stock xplot via apt, *may* not be able to deal with the output generated tho. on just about every other system I've used, I've had to manually compile 'his' version to have it work. [02:55] also, for nuisance value, the microwave here is on the same freq as the ap [02:55] \o/ [02:55] so dinner time is no-internets-time [02:56] awesomely awesome! [02:56] I know my wireless mouse used to get horribly laggy when the wireless was busy. [02:56] hmm, it decided a2b was empty [02:56] but b2a has dome something [03:00] that sounds ... vaugely right. a2b would be likely just acks. [03:01] tcptrace -l, to get a view? [03:01] summary text view as in. [03:01] otp [03:01] err - not summary. whatever. :-) [03:36] turns out to be wifi fail [03:36] however, am now getting 800KB/s to melbourne [03:38] sweet [03:38] still a bit bursty [03:39] drops down to 200 every now and then [03:39] going to be interesting to debug on top of :P [03:42] verra noisy [03:43] I wonder if I'm going to need a local ping background measurement ;P [03:53] 700K to b.l.n [03:53] \o/ [03:54] 800 [03:54] 850... you can do it [03:54] 100 :( [03:54] sawtooth [03:56] lifeless: i guess doing a regular bzr+ssh transfer off say devpad would be an interesting comparison [03:57] does hitchhiker have a non interactive mode ? [03:58] echo 'get /~vcs-imports/linux/trunk/.bzr/repository/packs/7590053cc142a5cb7ca8d0e33022ce2a.pack' | hitchhiker bzr+ssh://bazaar.launchpad.net :P [03:59] boom [03:59] I saw the same massive dropoff you did [04:05] spm: take pity on me; give me a sample session with tcptrace + xplot [04:05] I hav a trace [04:05] tcptrace -l shows sensible things [04:05] :-) [04:05] sure. [04:06] have you the trace somewhere I can munge at the same data? [04:06] sure [04:06] let me shove it on devpad [04:06] ta [04:07] the files ssh and http on devpad:~robertc - copying now [04:09] done [04:09] or thereabouts [04:10] ta, grabbing [04:10] You don't have permission to access /~robertc on this server. haha [04:11] I don't? [04:11] I beg to differ [04:11] oh wait. nm. I was using http access. lemme just scp it. [04:11] yah, not in ~public_html [04:14] righto. so looking at http. tcptrace -l http ;; on a bigger more complex snarf, i'd start with -b (brief) to get an idea of which streams to look at, but here is easy. [04:14] so no supriose that all the data is coming from crowberry; a is just acking. [04:15] right, I tried to capture just one file [04:15] well, there is the initial request ;) [04:15] :-) [04:15] tcptrace -G http <== builds *all* the graphs [04:16] xplot -x b2a_tsg.xpl a2b_tsg.xpl & [04:16] the -x and specifying both will allow you to zoom in on 1, and see the same zoom on the other at the same time. way cool. [04:17] I find it useful to not touch the vertical adjust, but stretch the two graphs fully horizontally. ymmv. [04:17] bad flag argument -x [04:17] right. you need his version of xplot then. [04:17] grah [04:17] ok [04:17] has he considered getting his patches upstream ? [04:17] no idea :-) [04:18] gawd dammit. one day I will remember that right click exits. [04:19] I wonder if tim sheppard even knows about this patchset [04:19] possibly not [04:19] shep at xplot.org [04:19] ^ go, mail him. [04:19] will do [04:19] but first... :-) [04:20] hmm, C code needs a little love [04:20] * spm starts to play spot the yak ;-) [04:20] ok, much better [04:21] it could use a window bitmap cache [04:21] if you stretch horiz, you'll be able to better zoom. vert if really useful imho. [04:21] so flat bits are stalls ? [04:21] so here we're basically looking for 1 main thing; other maybes. is the tcp window full. which - apparently is sure as hell isn't. [04:21] slowdowsn of some sort, yeah [04:22] aroud 12:54:20 "something" happened [04:22] hah! it does tmiezones [04:23] oh wow. zoom on ~ 12:54:19 window full. [04:23] And around 13:22, lunch happened. [04:23] * spiv makes it so [04:23] :-) [04:23] 14:54:19 you mean :P [04:23] lifeless: whatever :-D [04:23] but note also the large numbers of '1's and '2's. [04:23] chatty [04:23] looks like a fail for acks to be received [04:24] how do you you zoom out [04:24] left click [04:24] zoom by selecting; then left click [04:24] so whats a 1 and a 2 - or should I e reading at this point [04:25] rtfm, resends, once, twice of the same packet istr. [04:25] so you're thinking massive packet loss ? [04:26] the yellow band is the window size, right ? [04:26] actually - 1/2 I think are duplictae acking. [04:26] the delta between the horizonatal lines is the window [04:26] yellow and green [04:26] 53:24.5 has a similar region [04:27] hah, is one green?! [04:27] yeah. a couple of window fulls, imho, isn't bad. not great, but not bad. keeping in mind this is *your* window (aiui), not crowberrys. [04:28] sorry, I'm unclar [04:28] if you zoom in on the start, you can see the scaling take off. [04:28] is the gap 'data in the window' or 'amount that can be in the window' [04:28] amount that can be in the window. [04:28] ok [04:29] and can we tell amount in flight/in the receiver buffer [04:29] each white cross, is in fact a line with 2 arrows, zoom right in on one. that's a data packet. [04:29] I see black lines with arrows [04:30] http://www.tcptrace.org/manual/node12_mn.html <== check the 2nd diagram by way of example [04:32] based on that - it *looks* to me like crowberry isn't hammering you as hard as it could; or your pipe is full. ?? [04:32] tho the backoff from ~ 3/5ths to end is... odd [04:33] * spm checks throughput... [04:34] hrm. ~ 500kps. not too shabby. [04:34] avg [04:34] peaked at 880 or so [04:34] so lots of room for improvement [04:35] which, assuming default tcpwindow sizings on crowberry is about max for london to AU given latency. [04:35] I'm in nz ;) [04:35] so slightly closer eh? ;-) [04:35] no, sadly, no [04:35] can/should we tweak the window settings up ? [04:36] 350ms 64 bytes from crowberry.canonical.com (91.189.90.11): icmp_seq=5 ttl=42 time=328 ms [04:36] we can, trivially. I'd advise yes. but would want to cross check 1. what they actually are. [04:36] bah [04:36] 64 bytes from crowberry.canonical.com (91.189.90.11): icmp_seq=5 ttl=42 time=328 ms [04:36] and ask why the defaults aren't large enough to handle the earth. [04:36] 2. watch carefully JIC of funkies (I doubt we'd see any) [04:36] nfi. that's the default that linux kernels come with aiui. [04:36] yeah [04:37] the aarnet mirror admin [04:37] ho, Bazaar folk. what is the preferred pastebin? [04:37] swears that everyone should up their window settings [04:37] yeah. that supersize macca's theirs [04:37] they* [04:37] nod [04:37] bignose: one that works. And not pastie. [04:37] me, I use pastebinit [04:37] spm: thanks for this; I think there is more to nut out here [04:37] my favourite, , has been responding 500 errors for weeks. [04:38] spm: I need to do a couple of 'new country' chores before shops close. [04:38] lifeless: np. i suspect there is, but it's all clues. [04:38] I'll try that one, lifeless [04:38] shoo [04:38] spm: but I might grab a little more of your time to look at the ssh trace after, if thats ok with you [04:38] surely [04:38] lifeless: what URL? [04:38] its been .... some time, since I did this sort of analysis myself [04:38] bignose: apt-get install pastebinit [04:38] eek. crowberry is the, small, default. [04:39] spm: I can has RT ticket for 'we run servers please' ? [04:39] ooh! naughty Ubuntu worker, everyone knows ‘aptitude’ is the command to recommend :-) [04:39] spm: I mean, should I file an RT saying all our servers facing the net might need stuff ? [04:39] if you create said RT, I don't see why not ;-) [04:39] lifeless: sure. it'll help us track it at least. [04:44] fwiw. the *best* way to do these is get two sniffs at the same time. in this case, one on your machine, and one on crowberry. and display the sender from crowberry (b2a) and receiver at yours. you get a better view of the packet flows with times and whats missing etc. [04:50] I've had a crash on ‘bzr shelve’, with a “MalformedTransform” exception [04:50] there appear to be some bug reports with that same error, but different use cases [04:51] Launchpad is not displaying bug reports properly for me. can someone check whether the crash I'm getting is already reported, and/or submit that traceback to the report? [04:52] it's reproducible (I've pasted the version where I used ‘--no-plugins’), and I can make the repository available if it would help diagnosis. [04:55] spm: that would be awesome. I can has packet capture privs on crowberry ? :) [04:55] hahaha. no. *I* wouldn't even ask for that. :-) [05:00] mwhudson: how do I get that linux pack onto the staging bazaar.launchpad.net ? [05:01] mwhudson: /win 67 [05:03] spm: um, no quick idea, i guess you can sftp it up... [05:04] it seems to be because the shelve involves undoing a rename [05:04] but meanwhile the original filename is occupied by a new file, that isn't versioned. [05:04] so I'm not sure what I expect ‘shelve’ to do in that case. no crash, of course, but I don't know the right behaviour. [05:05] what bzr version? [05:06] also is there a bug about the rendering for you ? [05:06] [05:06] no, I don't have a Launchpad account and it's not working well for me in any case :-) [05:06] Bazaar version 2.1.1 [05:31] bignose: ok, we'll need to open a bug for you for that [05:31] bignose: you say there isn't a bug open about launchpad rendering badly? [05:32] bignose: could you put an email together, to me, with screen shots and so on that shows lp behaving badly, and I'll make a bug report out of it [06:01] any upload anything to sourceforge using bzr? [06:02] should I create a seperate trunk and branches directory (I am more familiar with doing this in svn) or just push it all up to SF? [06:14] PainBank: I haven't uses sf in a long time but bzr doesn't require specific directories like trunk and branches. You can probably push it all up as long as the project devs agree to treat dir x as trunk / mainline. [06:16] * fullermd eep!'s at broken dirstate code :( [06:20] PainBank: this article seems to be doing what you want http://sina.salek.ws/content/migrating-subversion-bazaar-including-special-guide-sourceforge-projects [06:57] fullermd: ? [06:59] lifeless: https://bugs.launchpad.net/bugs/582656 :( [06:59] Launchpad bug 582656 in Bazaar "Compiled _dirstate_helpers causes crash with specified file commands (affected: 1, heat: 6)" [Critical,Confirmed] [07:41] lifeless: shelve crash reported in Debian BTS now, [07:41] lifeless: the Launchpad rendering problems seem to be a strange interaction between Iceweasel and my proxy; change either of those and it's alright. [07:42] lifeless: shelve crash reported in Debian BTS now, (fixed the URL) [07:45] bignose: what proxy are you using ? [07:45] launchpad is on https generally, so I wouldn't expect anything in it except -perhaps- css files to hit your proxy [07:45] nevertheless, there is a problem :) [07:55] hi all [07:55] vila_: Sorry for using your nick, I'll leave === vila_ is now known as vila [07:59] tsk tsk [08:05] vila: hi [08:07] vila: i noticed you have claimed the reivew https://code.launchpad.net/~parthm/bzr/538868-message-for-heavy-checkout/+merge/25336 ... i could land it if it looks ok it you [08:07] parthm: hi ! [08:07] vila: or if i can get another reviewer :) [08:08] parthm: I didn't claim it, I suspect jam assigned me instead, I plan to review it soon [08:08] Maybe vila_ claimed it for you. [08:08] vila: ah. ok :) .. no hurry. was just wondering. [08:08] fullermd: ha :) [08:08] annoying squatter this vila_ ... [08:10] lifeless: I can't trivially run selftest, lacking testtools :| [08:10] lifeless: Won't have time the next few days to try and fudge it into place; got a week of stuff to do before I leave town Friday. [08:14] fullermd, lifeless : babune has a freebsd8 slave, can I help ? [08:14] Mmm. Building the bzr-2.1.1 tag in my 2.1 branch seems to die too, though the installed 2.1.1 doesn't [08:15] It was rebuilt since my last OS/python upgrade though, so it's not that... [08:15] Holy crow it takes a long time to branch bzr.dev on local disk. [08:18] die ? How ? [08:19] gnight [08:19] Hm. It seems to work on my server, which is fbsd8/i386/py2.6.4. Failing on my workstation, fbsd-current/amd64/py2.6.5. So, if everything's different, the result is different :p [08:19] Oh, pyrex 0.9.8.6 vs 0.9.9 too. Hm. [08:20] Maybe I can test that switch quick-like... [08:20] That could explain why the system-wide 2.1.1 works, but hand-building it doesn't; the system-wide has the generated files from the tarball. [08:20] In fact... [08:21] * vila holds his breath and hopes fullermd will finish his sentence soon enough [08:22] Yes, if I build from the 2.1.1 tarball, it works. If I build from the bzr-2.1.1 tag it blows chunks. diff shows only the generated .c/.h files as different between the trees. [08:22] So it sounds like pyrex 0.9.9. [08:24] hmm, pyrex 0.9.9 here, but maybe the .c files has not been modified since I upgraded.... [08:24] err, the .pyx files of course [08:24] one more hole in babune setup.... [08:24] a lucky one this time\ [08:25] Oh, that should be quick to test with that repro script (or manually) [08:25] Actually, don't even need a modified file in the tree. [08:25] So just 'bzr stat bzrlib' in the bzr.dev tree should tell you. [08:26] tell me what ? what repro script ? [08:26] https://bugs.launchpad.net/bzr/+bug/582656 [08:26] Launchpad bug 582656 in Bazaar "Compiled _dirstate_helpers causes crash with specified file commands (affected: 1, heat: 6)" [Critical,Confirmed] [08:26] Or just 'bzr stat bzrlib'; I get the crash (with 0.9.9-built stuff) there too. [08:27] * vila branching (don't ouch the babune working one !!!) [08:27] yeah, ouch, exactly, not a typo ! [08:27] Pfft. Wimp. [08:28] boom [08:28] KeyError: '' [08:28] Oh, good. I've successfully ruined your day; now I can go sleep in peace 8-} [08:31] fullermd: hehe, I'll check which pyrex version were used to generate my files and add a comment [08:31] (which I'm about 3 hours overdue for at the moment, so *bamf*) [08:31] It leaves a comment right on the first line. Handy. [08:31] I thought so [08:32] Generated by Pyrex 0.9.8.6 on Thu Apr 1 17:19:38 2010 [08:32] haha, so funny [08:32] not [08:32] Just confirms my long-held suspicion that dirstate is a conspiracy against me. [08:33] 656 diffs... [08:33] Between the generated files? === mrevell__ is now known as mrevell [08:34] Yeah, I think the diffs are big even between runs of the same version. Suckage :| [08:34] * vila searches the magic incantation to ignore the comment lines mentioning the source file [08:35] Well, at least we could build with cython instead... except [last time I tried] that yields stuff that doesn't work at ALL. [08:35] But that would fix THIS bug. Sorta ;) [08:36] urgh, you're right, bunch of moved around code or added "registers" .... go figure the problem :-/ [08:37] wow __Pyx_AddTraceback call added 8-{ [08:38] err, no, calls deleted in fact (/me switches the windows to a sane order) [08:39] lucid is still using 0.9.8.5, yeah for responsible distro maintainers using well tested versions :-P [08:40] Figures. CD I ordered last week showed up in my mailbox today, so obviously something UNpleasant had to happen to counterbalance and leave the day centered. [08:43] ok, got the failure in the test suite, that should narrow down the problem [08:44] bah of course it's a blackbox test :-/ [08:44] 7 failing tests already [08:44] Oh well. I'll still have the CD after the bug is fixed. Nya nya nya, Fate! [08:45] lol [08:45] * fullermd quickly hides. [08:47] * vila setup a dedicated babune job :) [08:48] 32^W41^W53 failures so far, one of them ought to be easier to debug than the others [08:49] Actually, I'd WAG that tracking it would be easier by just dumping some debug printf's around where it crashes to see what unexpected structure (or lack thereof) the pyx is giving back, and dig down from that direction. [08:49] * vila goes back to fixing its keyboard layout before turning insane [08:50] ... of course, just using the phrase "debugging printf" probably shows I'm not qualified to deal with python in the first place :p [08:50] fullermd: except a pdb.set_trace() with a simplified context may be even more handy, I'll look into it when the run finish [08:50] fullermd: same concept, different tool ) [08:50] :) [08:51] Well, I require_feature(zzz), so I'll leave it in your hands for the next fartoofew hours. [08:51] I would probably wait for jam to turn up anyway he knows pyrex far better than me [08:51] fullermd: sleep well [08:52] and thanks for the heads-up ! [08:52] * fullermd <--- canary in the bzr-mines. [08:52] hmm, I was about to joke about sending some gas and thought better :) [09:03] bonjour vila [09:04] hello Sacha :) [09:04] my patch for backslashes landed to 2.1 last night, thanks jam [09:05] I think I should provide a patch for bzr.dev to merge it cleanly now? [09:05] because that was for 2.1 only [09:05] and 2.2 has different command-line parser [09:05] or the person who will do merge simply ignore my changes? [09:06] vila: ^ [09:06] bialix: a patch to merge it cleanly would be good, I think [09:06] that's my understanding as well [09:06] bialix: it would be better if you merge 2.1 to trunk to ensure the conflicts are properly resolved [09:07] I'm just double checking [09:07] ok, so maybe I can adapt some my new tests [09:08] but if you propose a patch including your 2.1 merge in its ancestry everything should be fine [09:09] grr damn xmodmap bug ! [09:24] morning all. [09:25] vila: just posted thingy about leak experiements, basically what I said yesterday evening [09:25] mgz: Hi ! I haven't checked my mail yet :-/ [09:25] am now going to try and do some actually-useful things. [09:26] I mean just, like, that second. [10:13] heya mgz [11:58] igc: hello, is bug #541626 bzr or fastimport's stuff? [11:58] Launchpad bug 541626 in Bazaar Fast Import "'BTreeBuilder' object has no attribute '_find_ancestors' (affected: 2, heat: 6)" [Undecided,Confirmed] https://launchpad.net/bugs/541626 [12:01] jelmer: ping ? [12:01] asabil: hi [12:02] hey jelmer, I just wanted to know if you would have an idea about this assertion error failing: [12:02] File "/home/asabil/.local/lib/python2.6/site-packages/dulwich-0.5.1-py2.6-linux-i686.egg/dulwich/pack.py", line 926, in iterentries [12:02] assert isinstance(offset, int) [12:02] AssertionError [12:03] asabil: not without context... [12:03] I am trying to clone an internal git repo, and it fails with that error [12:03] I can paste the crash file somewhere [12:04] what versions of bzr-git/dulwich ? [12:06] from source [12:06] freshly installed [12:07] the crash trace is here: http://pastebin.com/SkbAkPeE [12:09] asabil: from the tarballs, from bzr ? [12:10] from bzr [12:16] Is there any good reason why bzrtools releases are strictly bound to bzr releases? [12:20] asabil: can you check what the type of offset is ? [12:26] yep, give me few minutes please [12:26] it takes a lot of time to clone a 6Gb repo [12:26] oh, if it's a 6gb repo I suspect the type might be a long [12:27] I don't think we support repositories that big yet [12:30] is there a way I can help fixing this ? [12:31] maxb: by request of the author [12:31] ...? [12:32] abentley prefers that the plugin aborts early rather than claims to work and fails late [12:32] asabil: we'd need support for larger pack files in dulwich first I think [12:32] asabil: do you have any pack files > 2Gb? [12:32] let me check [12:33] vila: I'm awake, but not officially working yet [12:40] jelmer, yes I think there is a 6Gb pack file [12:41] asabil: in that case, you'd really need to patch dulwich first [12:41] at the moment it doesn't support index files for pack files larger than 2Gb [12:42] ok, I will start there then [12:42] you might also want to check nobody else is working on that support yet, I think one of the Google folks was interested in it as well [12:44] jelmer, on the mailing list ? [12:44] asabil: yep [12:44] ok I will, thanks [13:13] I'd like to access the revno from inside a program running out of a bzr repo. Does anyone have a code snippet that can show me how to do that? [13:13] I've been browsing bzrlib for a function, but not been lucky [13:16] branch.last_revision() [13:18] bialix, what's the path to "branch" ? [13:18] branch is the instance of bzrlib.branch.Branch [13:18] e.g. br = bzrlib.branch.Branch.open('path') [13:19] print br.last_revision() [13:19] or maybe br.last_revision_info() [13:19] check the API [13:20] bialix: got it working, thanks a bunch! [13:20] np [13:24] Now I need to figure out how best to include that in my django site [13:30] http://wiki.bazaar.canonical.com/Integrating_with_Bazaar can be useful === JFo-afk is now known as JFo [14:01] bialix: indeed [14:01] jam: ok, back from shopping (a *real* chair for my desk :) === joerg_ is now known as joerg === CardinalFang_ is now known as CardinalFang [14:41] jml: hi [14:43] jml: .. sorry. i meant to ping jam. [14:54] jam: hi [15:09] hi folks, when i push in lp i give the message, pgrade the repositories to the same format for better performance. [15:11] i found in the web this command: bzr upgrade --2a [15:11] so i want confirm this before to do it [15:11] its ok? [15:11] ovnicraft: yep [15:12] jelmer, thx [15:12] ovnicraft: it might also be that the remote branch is out of date while the local one is newer [15:12] ovnicraft: in that case you might want to upgrade the branch on lp [15:13] in lp has format 7 [15:13] jelmer, i get that is 2a [15:21] format 7 sounds like a branch format [15:21] Bazaar formats can get confusing, because there's a repository, a branch, and a working tree, all with independent formats [15:21] it is, but I think it's probably the branch format that's part of 2a [15:21] maxb: yeah [15:21] bzr info -v tells all [15:21] launchpad displays the long names [15:35] sorry, branch format 7, repo format 2a [15:36] ovnicraft: did the upgrade help? [15:36] that info from lp, i am upgrading the repo === radoe_ is now known as radoe [16:50] jam: Latest bug comment is as far as I can take it. [16:50] fullermd: that's the "its a really big diff that is hard to understand" ? :) [16:51] can you email me the output? [16:51] No, just tracked to one line that's differing in whether it succeeds or not. [16:51] ah, didn't get that one yet [16:51] Hmph. I just rm'd the dir :p [16:51] you still have 0.9.9 pyrex installed, right? [16:52] Yah, regenning. [16:53] thanks === IslandUsurper is now known as IslandUsurperAFK [16:54] <[1]reggie> jam (or anyone else) -- how would I setup bazaar.conf to use sendmail to send mail? [16:55] <[1]reggie> I'm on windows but have a sendmail binary [16:55] <[1]reggie> just setting the config line in bazaar.conf doesn't appear to do it [16:55] [1]reggie: first off, "to send mail" [16:55] Under what circumstances? [16:55] <[1]reggie> bzr commit [16:55] Mailed. [16:55] fullermd: thx [16:55] [1]reggie: I presume you are then using 'bzr-email' ? [16:55] <[1]reggie> let me check === beuno is now known as beuno-lunch [17:06] fullermd: I think you mentioned that cython doesn't work for you. Can you give a bit more detail? (I use it here) [17:08] and I've at least reproduced the failure with your test script. [17:08] Interestingly you do have to pass a path, which may be a decent hint [17:08] <[1]reggie> jam, we have a emailer.py that is in our mysql_plugins folder. if post_commit_mailer = smtplib then it tries to use that. otherwise it uses whatever post_commit_mailed is set to [17:08] <[1]reggie> I have mine set to my sendmail instance [17:08] <[1]reggie> I just ran a test commit and got no error and no error in .bzr.log but also no email [17:08] <[1]reggie> very frustrating that smtplib doesn't support SSL 465 [17:08] [1]reggie: k. I'd start by saying I don't have a copy of, nor access to the mysql plugin [17:09] so there isn't a huge amount of support I can do on it [17:09] <[1]reggie> I know [17:11] I will point out that bzrlib.smtp_connection.SMTPConnection *can* take an optional "_smtp_factory" parameter (originally designed for the test framework). And you could pass SMTP_SSL in there instead. === davidstrauss_ is now known as davidstrauss [17:11] If mysql cribbed the email sending from bzr-email [17:12] that also has a "_smtplib_implementation" attribute, which instantiates an SMTPConnection [17:12] which would be another way to inject your own SMTP_SSL object [17:12] jelmer: re https://bugs.launchpad.net/bzr/+bug/480102, the only thing odd about the last-imported revision for lp:karl3 is that it includes a change to an svn:eol-type property [17:12] Launchpad bug 480102 in Launchpad Bazaar Integration "assert len(tview) == tview_len (affected: 5, heat: 48)" [High,Triaged] [17:14] [1]reggie: another option is too look at how bzrlib/mail_client.py works [17:14] which is where 'bzr send' hooks into sending notifications to your existing mail client [17:14] so that you have an opportunity to write messages to it. [17:15] probably not exactly what you want for the mysql post-commit effect, though [17:15] that change shows up here: svn diff -c 5140 http://osi.agendaless.com/bfgsvn [17:15] bug not here: bzr log -p --limit=1 lp:karl3 [17:15] TresEquis: bzr-svn doesn't import changes to svn:eol-type properties [17:16] OK, but would that explain the checksum issues on the next revision? [17:22] it also added 'svn:keywords' property, with 'Id' as the value [17:23] that might cause SVN to believe something different about the content of the file than bzr, maybe? [17:24] jam: Don't remember offhand, it was a couple months ago. But it just flat didn't work at all for anything. [17:24] jam: Yes, need the path. Don't need changes though; I found I could just 'bzr stat bzrlib' in a pristine tree to bring it up. [17:25] so my immediate guess is that pyrex is leaving an exception on the stack, and then accidentally checking the exception state as part of the "x == y" statement [17:26] Mmm. The object comparison does that? [17:27] My fiddling fingered that one specific line of the C. [17:28] jelmer: the next commit on the karl/trunk branch (the one which fails to import) is svn r5144, which touches (only) the file whose properties got changed in svn r5140 [17:28] fullermd: adding a "PyErr_Clear()" just before that line makes everything just peachy [17:28] *ugh* [17:28] Wacky. [17:29] Looking at: http://www.cosc.canterbury.ac.nz/greg.ewing/python/Pyrex/version/CHANGES.txt [17:29] - Exceptions caught by an except clause are no longer put into the thread state and cannot be retrieved using sys.exc_info(). To access the caught exception, it must be bound to a name in the except clause. A third name can be supplied to capture the traceback. [17:29] So I have another thought, we'll see [17:29] looks like I got it [17:30] change the bare "except KeyError:" to "except KeyError, e:" [17:30] and things are happy again [17:30] can you try it on your system? (line 1241) [17:33] Hm. Well, that makes 'bzr stat bzrlib' work. The test script still fails, though, as does stat bzrlib/_dirstate_helpers_pyx.pyx [17:33] (it's 1239 in my bzr.dev) [17:33] fullermd: my guess is that we just have to trap all of these bare exceptions, because pyrex is fubar without it [17:33] we use it in a fair number of other places [17:34] basically, look for "except[^,]*:" [17:34] Changing the one at 1222 as well fixes it. [17:34] a) This is definitely a bug in pyrex, and I'm trying to write a simple test case to show it [17:34] b) we can probably work around it for now [17:34] (well, fixes this incarnation of it anyway; who knows what evil lurks in the hearts and minds of any other except's in there) [17:35] TresEquis: hmm [17:35] TresEquis: *looking* [17:36] fullermd: reproduced [17:36] I'll file a bug on pyrex [17:37] I guess this just shows how version control systems can help you find bugs :p [17:38] svn collapses the $Id: ...$ string in its internal representation when that svn:keyword is set, I think [17:38] a stupid CVS compatibility hack [17:38] TresEquis: bzr doesn't touch that, it shouldn't really affect it [17:39] it wouldn't cause bzr and svn to have different ideas about the payload for that file? [17:55] fullermd: as near as I can tell, dirstate_helpers is the only extension that uses exceptions without objects, I'll put up a patch === IslandUsurperAFK is now known as IslandUsurper === deryck is now known as deryck[lunch] === beuno-lunch is now known as beuno [18:06] jelmer: I'm trying now to check out the karl/trunk stuff from a copy of 'bzr' with a pdb.set_trace() just before the call to main() [18:06] so that I can look at the stack when the exception happens [18:11] crap, bzrlib is bailing out without giving me a shot at the traceback / stack frame [18:11] fullermd, vila, lifeless: https://code.edge.launchpad.net/~jameinel/bzr/pyrex_099_bug_582656/+merge/25625 [18:13] Purdy. [18:14] * fullermd thanks jam. [18:20] amusing pyrex bug, well tracked down jam. [18:22] Oh, right, "amusing" was totally the first word that came to my mind last night :p [18:22] maybe it's one of those things that's only funny after the fact ;) [18:24] It'll give us something to chuckle over in the rest home, anyway. [18:26] jam: setting a run on babune [18:28] jam, fullermd : http://babune.ladeuil.net:24842/job/debug-freebsd/2/ [18:29] * vila dinner, will check later [18:29] rhymes :) [18:29] I'm on a selow connection, and I want to only make changes to a branch then push them up somewhere on launchpad, how can I minimize the data transfer? (I don't really care about full history) [18:29] jelmer: sigh, the traceback is no help, as the caller (who has computed the window) is implemented in C [18:34] vila: I'm not sure what in that sentence rhymed... but maybe in french? Anyway, thanks for the run. I'm pretty confident, having used pyrex 0.9.9 here. [18:36] jam: dinner with later ? [18:36] I've never pronounced "later" as "linner" but maybe you do :) [18:37] damn, for once there was no typo :) [18:37] Now if you said "date: will check late" [18:39] That would doubly rhyme, since late dates cause strife with the wife. [18:40] talking about wife.... see you later :-P [18:44] Hm, should I file a new bug for my three hashcache-related test failures, or should I piggyback on bug 70272 from 2006 [18:44] Launchpad bug 70272 in Bazaar "test failure (TestHashCache) (affected: 0, heat: 0)" [Medium,Confirmed] https://launchpad.net/bugs/70272 [18:47] changing the three tests not to sleep for four seconds each would be nice as well, can't they be utimed into the past? [18:51] ...think I'll file a new one, mine's certainly FAT related, just tried unsetting TMP and they pass === deryck[lunch] is now known as deryck [19:26] what is eclipse doing constantly running bzr in the background? [19:28] TresEquis: :-(I [19:29] because it keeps ending up running out of heapspace - and complains about Decorators [19:29] Calculating missile trajectories for when it becomes self aware and takes over. [19:29] jelmer: the offending svn revision is actually 5147, not 5140 [19:30] and the oddity is that the 'tview' string looks like a real revision of the file [19:30] but it corresponds to neither the prior version (5146) nor the updated one (5147) [19:31] the tview_len value is the actual length of the file in r5147 [19:31] so I have no idea where the bogus version comes from [19:40] jelmer: OK, the 'view' string is *not* a valid revision: it contains a Python syntax error, at least [19:58] TresEquis: it doesn't occur anywhere in history? [19:58] jelmer: nope [19:59] In bzr explorer, is there any way to run 'bzr pull' a lot of branches at once? [19:59] jelmer: it looks like a partial / intermediate change between the version of the file svn r4908 and the version in svn r5090 [20:02] but it also includes text which wasn't added until 5147 [20:03] the developer who committed 5147 maybe was using bzr-svn? [20:04] I don't know if there is any way to see "fingerprints" left by such a commit, assuming bzr-svn made it [20:04] hi jelmer TresEquis [20:05] ah, ree is the developer I spoke of [20:05] ze one who messed up things, right :) [20:06] ree: do you think you might have committed svn r5147 via bzr? [20:06] not sure. [20:07] or bzr, or merge from our imagedrawer branch... which was also bzr'd [20:07] so this may well be the root of the problem but I don't know how to be sure [20:08] merge from our imagedrawer branch = with pure svn, that is [20:10] I do see a bunch of bzr: properties on the svn r5090 dump [20:10] which is the one where you merged the imagedrawer branch [20:11] yes. I am not even sure if that one was merged with bzr or with svn [20:11] bzr:revision-info, bzr:ancestry, bzr:file-ids, bzr:revision-id, bzr:text-parents, bzr:text-revisions [20:12] plus svn:mergeinfo and svk:merge ;( [20:12] how nice [20:12] the svk bit was someone else ;) [20:13] TresEquis, in fact it is possible that I done that one with svn [20:13] sorry, usage history longer then memory [20:15] jelmer, would it not be possible, in theory, that bzr-svn just ignores the wrong info, and misses "something" but not the whole lot? [20:15] 'cause it may have been my fault, but if I could do it others will do it I can guarantee [20:15] so that's maybe something bzr-svn should be prepared of [20:15] ^ [20:21] I think it may be possible to scrub out the bzr: revision properties [20:21] I will want to try on a scratch repo first [20:24] TresEquis, that would solve it [20:38] ree: those bzr: properties are not "revision properties" (which are unversioned, and can be changed later), but "tree properties" [20:38] which are revisioned [20:40] d'oh [20:41] does this prove that it was me with svn? or it could have been bzr? [20:46] I don't see how svn would have been adding bzr: properties [20:47] but using svn to merge a branch to which those properties had been added earlier via bzr-svn would be plausible [20:47] I also have no evidence that those properties have anything to do with the error [20:49] none of those properties are on the affected file, or its parents [20:52] TresEquis: but they mention the affected file perhaps? [20:52] TresEquis: what version of bzr-svn are you testing with? [20:52] 2.1.1 [20:54] jelmer TresEquis it was definitely the case, that the properties were on the branch being merged [20:54] TresEquis: of bzr-svn I mean? [20:54] but, what buggers me is that there is no evidence that those properties have anything to do with the error [20:55] jelmer: http://paste.ubuntu.com/436359/ [20:55] jelmer, I have the issue with 1.0.2 [20:55] ree: Have you tried bzr-svn trunk ? [20:55] no, but I can [20:56] please do [20:56] jelmer bzr plugins says 1.0.2 [20:56] TresEquis: can you please also try with trunk ? [20:56] the file which blows up the checkout / import is karl/trunk/karl/content/views/blog.py [20:57] jelmer: like so? $ bzr co lp:bzr-svn ~/.bazaar/plugins/svn [20:58] TresEquis: yep [20:59] How do I temporarily go back to a specific revision? Neither update or switch seem to take revision specifiers. [20:59] TresEquis, tx [21:00] I can do a checkout, but that seems a bit overkill.. [21:01] hno: bzr up takes a -r / --revision arg [21:02] TresEquis: Maybe by bzr on this box is too old then.. 1.18.1. [21:02] long overdue for upgrade anyway. [21:03] hno: I really like the new "shared revisions" repo format [21:03] jelmer, checkout of our project will take some time, cc 10 min [21:03] * ree doing with trunk [21:04] * TresEquis as well [21:04] and halfway there [21:05] TresEquis: "shared revisions"? Is this part of the 2a format? [21:05] "Shared repository with trees" [21:05] right, 2a [21:05] I misremembered [21:06] hno: no, we've supported shared repos since 1.0 [21:06] (actually a bit older still, but meh) [21:06] I just wish joining worked solidly with svn. (to overcome externals) [21:07] we had a project (kss) that we could never switch to bzr for this reason, mainly [21:07] jelmer: I'm back at the pdb.set_trace I added in bzrlib [21:07] lifeless: I know. [21:07] jam: land that pyrex patch; its like coding on top of a sea [21:08] jelmer: same crash [21:08] 'morning lifeless [21:08] same busted not-really-a-revision ;( [21:09] TresEquis: :-( [21:09] * ree still just at 488/1089 [21:09] hi jelmer [21:10] jelmer: the 'window' tuple has the bits which cause the borked file version in the 'new_data' field [21:11] TresEquis: I suspect the base text is invalid and that's causing the problems [21:12] is that the 'sview'? [21:13] hno: yes, -r on update is new [21:13] hno: or you can use revert -r [21:13] which has been around for ages. [21:15] lifeless: I do not want to revert, just switch to that revision.. [21:15] jelmer: the 'sview' has the same text as svn r4908 [21:15] TresEquis, why was it so much faster for you? Working from a server or something? [21:16] ree: dunno -- I started earlier, maybe [21:16] TresEquis, did you do a bzr get, or a bzr co? [21:16] or that shouldn't matter. [21:17] so, it is trying to apply the patch from r5090 - r5147 against the base from r4908 [21:17] ree: I did 'bzr co' [21:19] hno: yes, so using your version of bzr: bzr shelve --all; bzr revert -r X; bzr unshelve [21:20] hno: will get you onto that revision the same as update -r [21:21] jelmer: the 'apply_txdelta_window' curried 'apply_window' has a value for 'sbuf' which is from the wrong source revision [21:22] but I can't see who called that code in the first place [21:23] TresEquis: it's called from fetch.py [21:27] jelmer: does that mean that the checksum matched before the currried bit was returned? [21:27] TresEquis: the base text is wrong [21:28] TresEquis: so yes [21:28] I mean no [21:28] sorry :) [21:28] lifeless: Think I prefer checkout -r then... [21:29] * hno starts a 60 seconds countdown of the remaining lifetime of this install. Time for a complete system update. [21:30] hno: why? [21:32] ugh [21:33] lifeless: Because I like to? And the distro release used becomes end-of-life in some weeks.. [21:33] hno: I meant 'why do you prefer checkout -r' [21:33] I am very happy that you're upgrading :) [21:34] Because I do not want to accidently commit that revert. [21:34] k [21:35] hno: just figured out I knew you from Squid list [21:35] :) [21:35] TresEquis: Thanks [21:35] free software is a small small world [21:35] indeed. And hi lifeless ;-) [21:36] :) [21:37] Ok, countdown ended. Time for system update. See you later and meanwhile let #bzr return to normal high squid developers presence level. [21:38] ciao [21:39] ack , curried functinos are a pain to debug [21:39] curried functions called by C code are even worse [21:40] TresEquis: that's not really the problem here though, the issue is before the curried code [21:41] jelmer: yeah, but the curried code is the only place I can detect the problem ;( [21:41] TresEquis: look at SvnEditor.apply_textdelta [21:42] the currying gives me no way to get back to the 'self' of the think which created it [21:42] TresEquis: if you go up in pdb? [21:46] up lands in 'report_inventory_contents' [21:50] TresEquis: but down from that there is apply_textdelta? [21:52] no [21:53] down from there is the curried 'apply_window' [21:53] ah, right [21:53] have you tried setting a breakpoint in apply_window when the problematic file is processed? [21:54] I have the breakpoint in the apply_txdelta_window function, right before the failing assert [21:54] that is the bottom of the traceback [21:56] that's too late though [21:57] you need the place where the base text is retrieved [21:59] yep [22:00] Greeting, everyone. Is there any kind of UTF-8 to native encoding when commiting/pulling to/from remote repository in bazaar, as it is done in subversion? [22:00] NightNord: for paths, commit messages, etc - yes [22:00] NightNord: not for file contents (similar to subversion) [22:02] jelmer: I have added yet-another-layer-of-indirection in fetch.py [22:02] Too bad, it seems that none of git,hg,bzr capable of this feature. Curious, that CRLF/LF conversion is done in bazaar... [22:02] jelmer: thanks [22:02] currying the curry with a method, to get me access to self [22:04] vila: around ? [22:06] NightNord: what do you mean ? [22:08] lifeless: any known for me DVCS incapabable for switching text-file encodings, despite of that eol conversion could be done without marking all converted files as 'changed'-ones [22:09] In theory, same trick could be used for encoding conversion [22:13] where is .bazaar/plugins/ in windows? [22:13] NightNord: sure, you could write a plugin to do that for bzr [22:13] MTecknology: bzr --version should show you [22:13] lifeless: thanks [22:17] lifeless: if plugin is something the same as for git are 'hooks', I suppose it's quite hacky operation... At least if not checking bzr code =) [22:17] jelmer: how do you run bzr-gtk's unit tests? [22:17] "bzr selftest bzrlib.plugins.gtk" [22:18] guijemont: bzr selftest plugins.gtk [22:18] NightNord: filters to do changes like that are a supported feature, not hacky at all. [22:19] Hm... That's could done a trick. I'll dig in, thanks, lifeless [22:19] aha, didn't know about that command [22:19] NightNord: if you're new to python you'll probably find the learning curve a little steep; folk here or on the mailing list will be happy to help, I'm sure. [22:20] jelmer: ok, different stab: I'll make 'self.chunks' on the FileRevisionBuildEditor an instance of a class derived from list, but which holds onto a reference to the editor [22:21] time to see how many tests I broke [22:21] Waiting 10 minutes between tries is not good for staying in the flow ;( [22:23] lifeless: Actually, I'll need to run single very simple shell command (find -name '*.cpp' -exec enca -x KOI8-R '{}' \;), so, I hope, my knowledge in python and some working plugin as example are sufficient. But thanks for invite, anyway =) [22:25] NightNord: you'll need to do the reverse as well, for the input filter [22:25] otherwise you'll commit in your local encoding [22:26] Yes, I know, I just simplified description to write less ;) [22:27] ok [22:27] I found checkeol plugin, I suppose it will serve as good base [22:27] jelmer: ok, that hack gets me to the editor instance [22:27] only I lost my pdb prompt, gotta run again ;( [22:28] TresEquis, got to go, good luck! [22:29] jelmer, thanks! [22:33] jelmer: what information is interesting from the editor? base_ie? [22:39] lifeless: hah, looks like plugins are actually like git hooks. And I see no plugins that actually 'change' data without marking it as changed [22:40] NightNord: The eol plugin [22:41] Ah, you mean embedded one... [22:42] Now I finally got your point. There is a special type of plug-ins, called 'filters', which are exactly what I need [22:44] yes, we hope [22:44] jelmer: the base_ie shows that it bzr-svn does think that the parent revision for svn r5147 is svn r4908, instead of r5090 [22:44] not widely used, but if it doesn't fit file a bug and we'll fix. [22:45] now trying to figure out why [22:45] TresEquis: please note that this is the per-file base, not the per-revision base [22:45] and it's the revision against which the diff is sent, that's also not necessarily the base [22:46] s/the revision/the file revision/ [22:46] jelmer: (Pdb) parent.base_ie.revision [22:46] 'svn-v4:4f889dee-8c54-0410-98c1-98789d956ae4:karl/trunk:4908' [22:47] doesn't that mean that we think the base version of the file is from svn r4908? [22:47] lifeless: simple hack over eol.py show that it fit well. It's even simplier then I thought initially. Thanks again. [22:47] because svn thinks that the base revision is 5090 [22:48] is there any installer package for tortoisebzr on windows? [22:48] TresEquis: it only means we expect svn to send the delta against the base text in that revision [22:49] TresEquis: how can you tell svn thinks the base revision is 5090? [22:49] is that the base revision of the overall revision or of this particular file revision? [22:49] svnlog shows 4098, followed by 5090, followed by 5147 (the last is the one which blows up) [22:50] is the file changed at all in r5090? [22:50] NightNord: cool === jelmer_ is now known as jelmer [22:52] lifeless: Actually, quick googling "git content filter" told me, that git supports just the same feature as well. But I will probably stick with bzr on this project, as folks at #git didn't know an answer for same question =) [22:53] jelmer: svn log of this particular file [22:53] the diff is a diff against 5090 [22:53] 5090 was, however, a revision which involved merging a branch [22:53] so maybe we are deceived by the metadata about the merge? [22:53] yes [22:54] jelmer: I set a conditional pdb.set_trace() inside FileRevisionBuildEditor.__init__ [22:55] the first time it hits, it is being called from open_file with base_revnum=5089 [22:57] is there any way of doing what bzr replay does that works on paths rather than fileids? [22:58] jelmer: the second time, the base_revnum of the caller is 5146 [23:04] jelmer: interesting, the 'len' attribute of the base_ie object matches the len of the r5090 version of the file, but its revision is still 4098 [23:07] so it looks like we're storing the wrong file under the r4098 revid [23:08] or mapping to it inappropriately? [23:08] the file's text matches the 4098 revid [23:08] but not its length [23:09] anything you's like me to look at about the editor? [23:09] its parent_trees are both tied to svn r5146 [23:09] as its base_tree [23:10] its revid is svn r5147 [23:13] in the call to _open_file, the file_id and base_file_id are the same [23:15] TresEquis: sorry, don't have time to help debug this at the moment :-/ [23:19] jelmer: ok, thanks [23:20] hm [23:21] is there any way i can cherry pick changes to just one file? [23:26] yes [23:27] mwhudson: merge filename [23:29] lifeless: ah right [23:29] thanks [23:35] jam: tsk, you skipped mentioning pyrex in your fix ;P [23:35] in NEWS, I mean. [23:38] mentioning slash blaming? [23:39] oh blaming, most definitely [23:39] doing 2.0->2.1 [23:39] to propogate the pyrex fix [23:39] hmm, I want a health checker for our branches [23:49] hi all [23:50] hello! [23:50] martin? [23:50] yup, vila made me get a nick that doesn't take half an hour to type ;) [23:53] bah. let them eat tab completion. [23:59] jelmer: I tried to summarize my findings on the LP issue