[00:01] lifeless: pong; just figuring out my after-work commitments now so I can plan that. [00:06] mwhudson: Now with more evil and depravity :) http://bzr.mattnordhoff.com/loggerhead/loggerhead/yui-cdn/revision/324/loggerhead/templates/macros.pt [00:14] spiv: kk [00:16] lp's ssh refuses to connect :( [00:16] weigon: LP's down for maintenance. [00:16] maintanence? Eh, whatever. [00:17] *doh* thx [00:21] crap, revision properties contain arbitrary binary data? [00:22] jelmer: it's just {str:str} isn't it, so i guess so? [00:23] mwhudson: what's the easiest way to escape arbitrary data in something representable in unicode? [00:24] jelmer: repr()? s.encode('unicode-escape')? base64? [00:25] jelmer: i guess it might be meant to be utf-8 or something [00:25] i guess i'm the wrong person to ask [00:26] jelmer: why do you say they contain arbitrary data? [00:26] Peng_: base64 is overkill, unicode-escape requires a unicode string (and I have no way of creating one) [00:27] Peng_: your increased depravity looks fine to me :) [00:30] mwhudson: Is that a bb:approve? :D [00:30] lifeless: at least one of the tests tries to add "\xb5" as property value [00:32] (in other words, a object and invalid utf8) [00:34] jelmer: ah interesting [00:34] and we what, base64 enciode them ? [00:35] it seems like it silently gets casted to unicode [00:35] in the XML serializer [00:35] so "\xb5" goes in, u"\xb5" comes out [00:36] but you said \xb5 isn't valid unicode? [00:37] lifeless: it's invalid utf8 [00:46] jelmer: is the test testing that or is it a bug in the test? [00:49] lifeless: the bug is in the test [00:49] lifeless: "No shells on this server." ... Connecting with putty with same certificate seems to work. You said if that was so, then bzr didn't pick up Pageant. How would I fix that? [00:56] MizardX: now we've reached the end of my knowledge :( I haven't used bzr on windows [00:56] Ok. Thanks. [00:56] MizardX: but we now know that your launchpad and putty stuff is setup [00:56] uhm [00:57] did you install using the windows binary, or the 'python installer' ? [00:57] binary [00:57] bzr --version will give you the path to a log file [00:57] I think it's compiled with py2exe [00:57] you might look in that log and see if there is anything about pagent/putty [00:59] Just "ssh implementation is OpenSSH" and "ConnectionReset: Connection closed: please check connectivity and permissions" [00:59] ok [01:00] that suggests it found a ssh.exe on your machine and is using that [01:00] there is an environment variable to control this I think, one sec [01:01] do you have plink installed? [01:02] yes [01:02] ok [01:02] try [01:02] setting the BZR_SSH environment variable to 'plink' [01:04] "FATAL ERROR: Disconnected: No supported authentication methods available", not it doesn't find the certificate [01:04] now* [01:06] lifeless: I'll head to you. ETA is 11:50 by the time I buy my coffee :) [01:06] Ah, pageant wasn't running... [01:07] "Branched 8 revision(s)." :D [01:07] Thanks, lifeless. [01:10] mwhudson: ping [01:10] mwhudson: Mind if I make an old-style class in Loggerhead new-style without getting it reviewed? (Well, asking is almost as inconvenient as a review...) [01:11] MizardX: cool [01:16] what does this mean: error: unable to create symlink 'themes' on this platform [01:16] Would I be correct at guessing you're running Windows? [01:16] Also, how did this come about? What command caused it? [01:16] unfortuneatly yes [01:17] i was just getting a branch [01:17] Then I can only assume the branch contains symlinks, which have no equivalent on Windows... I dunno what else it could mean. [01:18] ugh, ok... the thing is is that it gave me any of the other files in the branch [01:20] Launchpad is running bzr 1.13 on production now. [01:21] ~18s & 21 hpss calls to push no revisions to a stacked branch. ~7s of that is SSH connection time. [01:23] ~27s & 32 hpss calls to push one revision to a stacked branch. [01:24] jml: is this better :) [01:24] yes. [01:27] g'day all [01:27] how long should bzr selftest take on a macbook ? [01:27] it has been running an hour and a hlaf [01:28] sm: it takes about 20 minutes here [01:28] lifeless: bzr pull is noticeably faster. [01:28] jelmer: on a macbook ? [01:28] sm: ah, no, sorry - linux [01:29] lifeless, spiv: congratulations! [01:29] thanks [01:32] hmm. making a new stacked branch is still kind of slow. [01:32] I wonder if bumping our format to 1.9 would change that. [01:32] do you have the patches we landed yesteday? [01:33] lifeless: only if they are in the nightly -- I'll update my bzr.dev and try pushing with that. [01:33] well, whats your nightly version? [01:34] lifeless: 1.14~bzr8.10-4223-1 [01:35] lifeless: down from 81 -> 66 calls using bzr.dev. still the same elapsed time. [01:37] jml: ok, thats interesting [01:37] pop the log up somewhere [01:39] lifeless: http://paste.ubuntu.com/142445/ -- first is using nightly, second is bzr.dev [01:40] both are pushes of the same branch to Launchpad but to different locations. branch was made locally from a mirror of the default stacked-on branch. so no new revision data for either push. [01:44] What would be the best way to have a particular file updated every commit to include a revision number? I think I could use a hook for it, but it looks like I would have to require it be installed on all committer's computers. [01:45] Snova: Well, that's what the new keyword plugin is for, although it just applies to the working tree, not the history itself, like svn:keywords. [01:45] Snova: That'd mean you only need the people who actually need the revision numbers to install it. [01:45] Snova: Anyway, the usual workaround is to have your Makefile run "bzr version-info >whatever". [01:46] Snova: what do you want to achieve with this? [01:46] lifeless: I'd like the server I'm writing to send the revision number along with the version. [01:47] Snova: of the codebase its running? [01:47] Yes. [01:47] as Peng says, invoking bzr version-info during server startup is probably better [01:48] I could, but then it assumes it's always in a branch... a source tarball or any form of distribution wouldn't... although I primarily want this during development, if I actually released it there wouldn't be much point. === ja1 is now known as jam [01:49] I guess that works, and it's the simplest way. [01:49] you can hook that into your tarball export logic too [01:55] lifeless: http://bugs.python.org/issue5660 [02:00] jml: well, we think we have stuff in the works that will cut another ~10s out of that fairly soon (needs new verbs, though). [02:01] jml: thanks for the log [02:01] Hooray for Bazaar! [02:01] spiv: cool. [02:02] jml: so if you're lucky, 1.14 on the server and client will fix it [02:02] if you're not, it won't :) [02:02] if I'm extra double lucky, we might get an edge codehosting service so we can run 1.14 on the server before the next LP release. [02:04] when is the next release? [02:04] lifeless: four weeks from today [02:09] What would the edge codehosting do? Have a second copy of everything? Would the main and edge instances both pull mirrored branches? [02:12] jml: btw, my locations.conf for my bzr branches now defaults to lp [02:14] spiv: nice! [02:14] 67 HPSS calls, 48s (including SSH handshake) and 207k to push up a new branch with a new revision. Not too bad. [02:14] Peng_: it would just be the SSH server & surrounds. There would be no edge puller. [02:14] Peng_: it would use the same data and be in every way analogous to the edge web service [02:15] jml: uh [02:15] jml: bzr: ERROR: Server sent an unexpected error: ('error', '') [02:15] jml: that was on a Branch.unlock RPC on an empty push [02:16] spiv: hmm. I bet it's an operational glitch. [02:16] jml: maybe the XML-RPC server was unreachable temporarily for the "hey please rescan this branch" ping? [02:16] jml: how hard would it be to get that message to a) have an OOPS code, and b) make it clear that LP is to blame, not the client? :) [02:18] spiv: if you asked me, I would have thought that the OOPS was already in it. [02:18] jml: I see! [02:19] jml: So that means there is probably already an OOPS generated, at least? That's nice! [02:20] jml: btw, it does seem to be a transient issue, a second attempt at the empty push worked just fine, rather than erroring on the final (13th! spooky!) RPC. [02:21] jml: In my magical land of perfect software and ponies, and temporary inability to reach xmlrpc.lp.internal wouldn't break anything... [02:21] https://bugs.edge.launchpad.net/launchpad-bazaar/+bug/281042 [02:21] Error: This bug is private [02:21] s/and/an [02:21] describes such a land. except that it's private, for reasons that elude me at present. [02:22] jml: ah, yes, I guess it was a timeout, judging from the 20s + a bit it took between request and error response. [02:23] * spiv clicks the hard-to-find "me too" link. [02:23] (Something seems wrong when it's easier to add a "me too" comment than use the "me too" feature...) [02:26] spiv, yes. Quite a few things, actually. Hopefully, we'll address that in 2.2.5 with the change in the bug lauout page [02:27] spiv: fwiw, https://bugs.edge.launchpad.net/launchpad-bazaar/+bugs?field.tag=codehosting-ssh is actually a fairly complete list of bugs in the codehosting server [02:27] spiv: btw, what's the status of your ~ patch? [02:28] jelmer: limbo; I need to find some spare time to rewrite along the lines poolie wanted to see if I'm happy with his approach [02:29] jelmer: I think I'll be fine with poolie's approach (although his approach does need a new server, which is a minor drawback), but it needs to be implemented... [02:30] spiv: ah [02:31] (on the other hand my approach needs a new client and his doesn't...) [02:44] Could someone take a look at lp:xubuntu-docs and see if there is anyway we can improve performance? Branching and pushing seems to take a very long time on it [02:45] cody-somerville: use bzr 1.14 [02:45] cody-somerville: and be sure you have done bzr launchad-login [02:45] I have [02:45] I'm using 1.13 [02:46] is 1.14 in the ppa? [02:46] in the nightly ppa yes [02:46] jelmer: jam has suggested that the filter registration function just take a callable, not a dict and a callable [02:46] jelmer: I think it's a good idea. OK with you if I put a patch up for that? [02:46] we want the API stable for 1.14 [02:47] cody-somerville: also, have you tried branching and pushing since the lp rollout that happened an hour or so ago? [02:47] I just branched [02:48] cody-somerville: I'm doing a branch now, and it's streaming to me here in Sydney at ~150K/s [02:49] (which I believe is the capacity of this ADSL link) [02:49] (I have a 1.5MB connection, so thats 1.2MB used by bzr [02:50] cody-somerville: and how long did it take to branch? [02:50] cody-somerville: [you are aware you have 30Mb of history] [02:51] Yea, I figure thats what the problem is [02:51] I did some nasty stuff to get the revision history to show correctly [02:51] cody-somerville, and, you could use a shared repo, so you only download whatever revision once in a life time [02:51] I am using a shared repo [02:52] cody-somerville: "12:46 < lifeless> cody-somerville: and how long did it take to branch? [02:52] " [02:52] so it should be very fast after you did it oncec...? [02:52] once even [02:52] lifeless, about a minute [02:52] "time bzr branch lp:xubuntu-docs" here just took 4m23.953s, which seems to be about as fast as this ADSL link allows. [02:52] Given that it had to fetch ~36M [02:53] cody-somerville: do another branch with -Dhpss please, and we can look at the network activity if you grab the log for that bzr command from your ~/.bzr.log [02:53] okay === timchen119 is now known as nasloc__ [02:55] 60 seconds is long to figure out you have everything locally, unless you didn't branch into your shared repo [02:55] I did and am again [02:55] oh; also try 'time bzr checkout --lightweight $SOME_LOCAL_BRANCH /tmp/foo' [02:57] lifeless: FWIW, that takes less than 1s here. [03:00] cody-somerville: ping; 1 minute should have finished ages back [03:00] lifeless, It takes a long time to copy and paste the log :P [03:00] cody-somerville: I'm kindof paying attention here to give you a fast turnaround; if you're going away to do stuff let me know and I'll get back to coding [03:00] http://pastebin.ubuntu.com/142472/ [03:01] No, just takes a long time to copy and paste 6078 lines [03:01] you missed the top of the log even so :) [03:01] Before that is log stuff about another command I did [03:02] oh wait [03:02] nvm [03:02] cody-somerville: don't worry [03:02] you're not using a shared repo [03:02] so its copying everything [03:02] http://pastebin.ubuntu.com/142475/ [03:03] oh... I guess you're right [03:03] I thought I had created that [03:04] btw, I did bzr check and it said this: http://pastebin.ubuntu.com/142477/ [03:05] Is an inconsistent parent bad? [03:05] its something to fix [03:05] bzr reconcile will fix it [03:06] lifeless, So I moved a bunch of files - when I merge from the parent, which hasn't moved the files, will bzr do the right thing or will they go to the old location? [03:07] cody-somerville: why don't you try ?:) [03:07] lifeless, cause the branch is reconciling atm or I would ;) [03:07] oh, that was quick [03:09] lifeless, would a repack help make the branch smaller? [03:10] cody-somerville: no [03:10] bzr repacks itself as it needs to [03:12] what about that obsolete_packs directory? [03:12] is that maybe causing the download to be more than it needs to be? [03:13] No, it doesn't affect downloads at all. [03:13] hmm... the branch seems to be bigger now than before [03:14] cody-somerville: 1 minute is not long [03:14] You have 36M of actual revision history, which doesn't sound so unlikely for nearly 200 revisions of a 5.9M tree of files. [03:14] Its 77M on disk [03:14] cody-somerville: all seems to be working reasonably; its under a second to make a working tree, and with a shared repo new branches hsould be about 3 seconds or less [03:14] cody-somerville: thats because of the reconcile; ignore it [03:15] cody-somerville: housekeeping will take care of it [03:15] lifeless, so will this reconcile get pushed to launchpad? [03:15] cody-somerville: no [03:15] launchpad needs to be reconciled separately [03:15] should I do that? [03:16] if you like [03:16] what exactly am I fixing? [03:16] Will it make merging easier or something? [03:16] what revisions annotate will attribute content to [03:18] cody-somerville: generally speaking you shouldn't need to worry or do anything special with bzr to make it perform well; the one key exception is shared repositories [03:18] and the second key exception is using a smart server if one is available (which lp: does for you once you login, and bzr+ssh/bzr+http do elsewhere) [03:26] Hey I discovered the super awesome "bzr join" command without any help from mwhudson but the only problem is that I can't figure out how to use it would anyone like to assist me [03:26] I have two unrelated branches and I want to make one of them just be part of the other one. [03:28] hmm... [03:28] bzr doesn't seem to have done the right thing :( [03:28] and ugh.. "1432 conflicts encountered". [03:29] exarkun: use the bzr merge-into plugin; join is part of an as-yet unreleased feature [03:29] cody-somerville: did it print 'criss cross merge' at the top ? [03:29] no [03:30] thats very odd then [03:30] what are the conflicts? [03:30] and when you say you did nasty things, could you be more specific [03:31] I'm going to guess you've been doing history-glueing in some nasty manner, and it is [predictably] biting you [03:31] Well, I took branch A and branch B [03:32] lifeless: Oops! I figured out how to use bzr join while I wasn't looking at IRC. [03:32] A wasn't based on branch B [03:32] lifeless: Should I start over with merge-into? [03:32] But really was [03:32] exarkun: if it has worked, go for it. [03:32] okay cool, thanks :) [03:32] So I merged B into A using -c 1 [03:32] That worked fine and I got what I wanted except I lost all the revision history from branch B and some people didn't like that [03:33] cody-somerville: when you say 'a wasn't based on branch b' do you mean 'they were completely separate projects' ? [03:33] No, it was Xubuntu docs and Ubuntu docs [03:33] They're similar but the branches didn't have shared revision history because Matthew restarted all the branches from scratch for some reason [03:33] I imagine for performance reasons [03:34] ok, which means 'they were completely separate projects' [03:34] no history -> separate projects [03:34] Right [03:34] B was based on C [03:34] you should use 'bzr merge-into' to join them. [03:34] and now I'm trying to merge C into B [03:35] Oh I missed a part [03:35] I removed everything in B and put in the contents from A to get the history back [03:35] and then I merged A into B to get the metadata [03:36] B *is* based off of C and now I'm trying to merge C into B [03:37] well [03:37] you might try merge --weave [03:37] I moved a ton of stuff into a subdirectory [03:37] it may do better [03:37] did you move using 'bzr mv' ? [03:37] so now I have a bunch of files in the old directory ending with .BASE and .OTHER [03:37] lifeless, I did in A, yea [03:37] or did you move by doing 'mv; bzr add' ? [03:38] I did bzr move [03:38] but in A [03:38] is A merged into C or B or neither? [03:38] into B [03:41] so should I do: bzr remerge --merge-type weave --reprocess [03:42] * cody-somerville tries [03:43] lifeless, If this doesn't work, would branching C and doing the moves over again and then merging that into B work and make future merges of C work? [03:43] hmm... only 1324 conflicts this time [03:44] but didn't do the right thing. bzr diff desktop-guide (the directory I moved almost all the directories into) returns nothing. [03:45] cody-somerville: what would you expect it to return? [03:45] lifeless, The changes bzr merged in? [03:47] lifeless, bzr isn't touching any of the files already in the branch [03:47] lifeless, all the conflicts are on files its added [03:50] I'm also getting this error now trying to branch inside the shared repository I created: http://pastebin.ubuntu.com/142494/ [03:52] cody-somerville: you ran 'bzr init-repo' to make the shared repository, yes? [03:52] yup [03:52] cd to the root of the shared repo [03:52] okay [03:52] and run bzr upgrade --default-rich-root [03:53] bzr: ERROR: no such option: --default-rich-root [03:53] 'use bzr 1.14' :) [03:53] hell you're canonical, you should be using the nightly ppa :) [03:55] ;p [03:55] also, I just got this when trying to do bzr remove */po/*.po: bzr: ERROR: [Errno 24] open: Too many open files: '.' [04:40] hmm, I wish that the GNOME vcs-imports branches weren't the default branches :-( [05:20] SamB: if there's a better branch to use, file a request at https://answers.launchpad.net/launchpad-bazaar asking for the default to be changed. [05:21] spiv: well, in #launchpad they've got me seeing if I was using a bleeding-edge enough bzr that they should have fixed the problem where it takes forever to push to a branch with little to no shared history with the branch it is to be stacked on ... [05:22] SamB: ah, yes. We keep fixing bugs :) [05:23] SamB: but even so, if there's a better branch, do let the relevant people know [05:23] and they said something about how GNOME is supposed to be switching to git [05:24] so that the bzr-playground branches might not be that great for too long [05:31] SSH handshake time is now well over a third of the overall time in random incremental pushes to my branches on LP. [05:32] oh, incremental pushes seem fine [05:49] SamB: I'm opening to changing the message if it can be less confusing [05:49] SamB: but I figured I should explain why it is saying that (and why it's not "lying"). [05:49] Well, "transferring" is a much more neutral word [05:49] if it can't be bothered to figure out which of the two activities it is engaged in is taking all the time ... [05:50] I'm not sure what you mean by that comment? [05:50] The activity is (accurately, by its choice of perspective) called "fetching". [05:50] well, it's not waiting on the fetches from the local repository ... it's waitng on the stores to the other one [05:51] Not really, those things aren't really decoupled. [05:51] well, okay, in that case the word is "transfer" [05:52] so [05:52] we're going to overhaul this [05:52] and make it prettier [05:52] at least, in en_US that's the case [05:52] transfer is indeed a synonym [05:52] or nearly so [05:52] at the right point in the code [05:52] maybe in en_AU or en_UK, fetch is more synonymous with transfer? [05:53] but one of the basic problems is that the code doesn't *know* which end of the pipeline the human is sitting [05:53] so really, a more directional adverb to describe what is going on isn't easy to correlate with the UI [05:53] well ... saying "transfer" is a nice and neutral thing to do in that case [05:53] or when, say, both repos are local [05:53] sure, thats one way we can alter the message [05:54] all I'm trying to say is: [05:54] - it is broken at the moment [05:54] okay, yeah, I gathered ;-) [05:54] - its not top of the pops for time until the rc [05:54] - until then, I don't plan on thinking too hard about it [05:55] (so please forgive me for simply acking that its an issue, until we're looking at the code, finding a combination that reads well and fits the code isn't necessarily easy [05:55] the code itself *is* highly directional [05:57] you know, I guess I don't need to finish this push ;-) [05:57] SamB: is it still slow? [05:58] it's just that I was just pushing the same thing to a new branch [05:58] so now that I've established that it was just a confusing progress message, I don't need to finish [05:58] oh :) [05:58] makes sense to me [06:29] lifeless: did you see my patch about --parallel ? [06:31] no [06:33] lifeless: ~one liner to make it work but I think you're the one that can review it [06:49] where is it [06:57] vila: ^ [06:57] http://bundlebuggy.aaronbentley.com/project/bzr/request/%3Cm2iqlozk7x.fsf%40free.fr%3E [06:58] # A registry where get() returns a suite decorator. [06:58] parallel_registry = registry.Registry() [06:58] + [06:58] + [06:58] def fork_decorator(suite): [06:59] those changes are bogus , PEP8 allows adjacent for things that are closely related [06:59] specifically I want the FOO_decorator and register..FOO adjacent [07:05] vila: reviewed [07:15] lifeless: still ok to separate each couple ? [07:18] sure === serg_ is now known as serg === serg is now known as anyone === anyone is now known as serg [11:43] If I do bzr revert on a file, does that count as a change? IOTW, if I commit it, will it commit the reverted file as a head revision? [11:45] bzr revert is no different from manually changing the working tree [11:45] plain 'bzr revert' will just revert to the head revision, so it wouldn't commit anything [11:46] Ah OK - thanks. I used to use perforce, and that behaved quite differently IIRC. It would rememeber what revision number of each file you currently have in your working tree. [11:46] hm? [11:46] So pulling out an old version to look at was not a change. [11:46] ah [11:47] no, bzr revert operates purely on the working tree, you can do something like 'bzr cat -r file >file' and it will be more or less the equivalent [11:47] Right - makes sense. Thanks! [12:18] jelmer/vila: would authentication.conf have problems with passwords that contain @ characters? [12:19] Specfically, if I put my password in the file, it fails, remove it and I get prompted more than once for each bzr-svn push [12:21] awilkins: quote it [12:21] Aha [12:22] vila: Failed [12:23] awilkins: :-/ quoted as in user@host => user%40host ? [12:23] Ah, as in "c@r" [12:23] awilkins: sorry, I meant url encode it [12:24] Hmmph, still not working [12:25] Although by now I may have locked out my account I suppose [12:25] Nope, still works fine at the command line [12:25] awilkins: what protocol are you using ? [12:25] svn+http [12:26] you mean bug #256612 ? [12:26] Launchpad bug 256612 in bzr "should prompt for usernames during HTTP Basic/Digest auth" [High,In progress] https://launchpad.net/bugs/256612 [12:26] or some related problem ? [12:27] THe username is correctly being picked up from auth.conf with is set up for the host [12:27] If I add the password it failes [12:27] I thought connecting succesfully with *svn* was the current work-around [12:27] Yes, I thought I'd done that [12:27] Although it was with TSVN [12:27] I'll try getting it cached in the CLI client also [12:29] That works [12:30] good, then you encountered the bug above or something very close [12:33] It's obviously using the password when it's provided in the conf file, but it appears to be wrong === ja1 is now known as jam [12:56] lifeless, jml: they've fixed the deepcopy(TestCase(...)) bug [12:58] lifeless, jml: but you might want to take a look at the new TestCase.addTypeEqualityFunc method [13:33] jelmer: I imported an svn repository with bzr-svn's svn-import. I see "svn revno: x" in log output, but I can't use -r svn:x. Why not? === sabdfl1 is now known as sabdfl === NEBAP is now known as NEBAP|work === NEBAP|work is now known as NEBAP|away === NEBAP|away is now known as NEBAP|work [14:17] fbond: you need a newer version of bzr-svn to do that on imported branches [14:18] igc: yeah, just taking a callback sounds good to me [14:18] jelmer: Okay, thanks. [14:19] jelmer: I'm using trac-bzr; it'd be really swell if I could get the old references to svn revisions work in the diff viewer. ;) [14:19] I mean [283] links and whatnot. === ja1 is now known as jam === abentley1 is now known as abentley [15:04] vila: bleh, I'm not sure how I missed that appearance of 'password' [15:04] jelmer: :-) [15:05] vila: I did actually refactor that function but changed it back after I noticed it broke for LC_ALL=C [15:07] jelmer: huh ? the one you copy from has been passing for ages or did I miss something ? [15:07] vila: Yes, but I changed it after I copy pasted it [15:07] haaa :) [15:08] vila: and that broke the tests, so I fixed it and ended up with what the original did :-) [15:08] vila: Your NotATerminal patch looks ok, with one minor concern [15:08] vila: won't this caused backtraces when there's no terminal? [15:08] getpass() will raise an exception that's not caught atm [15:10] jelmer: which exception ? EOFError ? [15:10] vila: termios.error [15:11] in my case: [15:11] termios.error: (25, 'Inappropriate ioctl for device') [15:11] including backtrace [15:12] and the backtrace leads to what ? 'sys.stdin.readline()' or some termios call ? [15:13] vila: getpass.py line 32 [15:14] that's indeed a termios call in getpass() [15:14] jelmer: grr [15:14] stream=None => stream = sys.stdout, damn, I explicitely avoid using a prompt and see what happens ! :) [15:15] vila: oh no, it's indeed related to stdin [15:17] hmm, so that's why NotATerminal was there :-/ [15:20] vila: ah, right [15:21] good I did some testing then :-) [15:21] vila: So NotATerminal can't be removed, but your other refactoring is still valid right? [15:21] jelmer: indeed ! [15:22] jelmer: No [15:22] hmm, or may be we just need to put raise NotATerminal in get_password... [15:22] Urgh, how ugly, the user can enter its login but not its password [15:25] no, I think the right fix is to either call readline() or getpass depending on stdin.isatty() or something like that [15:25] turning echo off makes sense only on a tty and that's the reason we want to use getpass [15:26] vila: sounds reasonable === thunderstruck is now known as gnomefreak === thunderstruck is now known as gnomefreak [16:02] If I committed with a bad username, is there a way to fix those revisions to use a new value for that field? [16:05] Only by uncommit/recommit [16:05] Or variants thereof.. The internal revision ID uses a hash of the data, collectively. So changing the data would change the hash, and hence the revid [16:06] vila: will you submit a new patch? [16:07] I think so, but not in the coming minutes nor hours I'm affraid [16:08] vila: thanks, no hurry :-) [16:09] vila: and thanks for the work on this auth stuff, hopefully bzr/bzr-svn 1.14 will be the first bzr where this stuff works properly [16:09] jelmer: np, I'd still like feedback from the other devs about utf8 encoding [16:09] LeoNerd: Thanks [16:09] jelmer: I noted that you didn't *call* get_username, are you planning to work on that ? [16:10] jelmer: yeah hopefully, but as you see, evils is in the details :) === serg_ is now known as serg [16:10] vila: don't call it in what sense? [16:11] vila: get_user() would use it [16:11] oooh, yes, indeed, the other patch, ok [16:11] vila: speaking of which.. [16:12] then will you update that one to take into account the other calls to getpass.getuser() / [16:12] s!/!?! [16:12] vila: you mentioned that http uses get_user() atm and the others use getpass.getuser() [16:12] yup, that point :) [16:13] vila: Just to make sure; they should all call get_user() but get_user() should default to getpass.getuser() ? [16:13] vila: You also mentioned something about only prompting if a prompt was specified [16:13] jelmer: yes, except that defaulting should be the default but http can overrride it [16:14] ha, I didn't think about using the prompt for that... [16:14] vila: so defaulting is directly returning getpass.getuser() or rather printing "[DEFAULT]" and letting the user just hit to use that? [16:15] ...I was thinking about a new, optional, parameter... [16:15] no [16:15] we don't want to require pressing enter where we didn't before [16:15] ok [16:16] when we default to getpass.getuser... we default period. You can ask for feedback but I doubt we want to change that [16:16] vila: ok, so I won't touch anything but http at the moment [16:16] vila: what would you like to change in case of http? [16:17] is it ok to require pressing or entering a username there if there's no user specified in authentication.conf ? [16:17] jelmer: I think you should change ftp/ssh/http, call ui.get_username() from config.get_user() but only if required via a parameter (i.e. http only) and get rid of the getpass.getuser() calls in ftp/ssh [16:18] just do a grep getpass you'll understand easily I'm sure [16:19] vila: ah, ok [16:19] vila: so, just to make sure: [16:19] vila: I'd add a new parameter to get_user() that specifies whether or not to ask the user [16:20] vila: and only http for now sets that parameter to true [16:20] exactly [16:21] So once I do the uncommit/commit dance to get my username right in the revisions, if I want to fix what's on launchpad, should I do a push --overwrite? [16:21] sounds like the cleanest way while still allowing get_username to get overwritten by GUIs with the usual mechanism [16:21] exarkun: yes [16:22] cool === abentley1 is now known as abentley === thunderstruck is now known as gnomefrak === gnomefrak is now known as gnomefreak === LenzGr is now known as LenzGr|dinner [18:42] hi, I've made bzr rm file, on the wrong file, is there anyway to get it back simply? [18:42] I haven't committed yet [18:46] is there a bzr-cvs plugin? [18:50] hi, I've made bzr rm file, on the wrong file, is there anyway to get it back simply? [18:50] bzr revert file? [18:52] luke-jr, it works thanks a lot (I got scared for a second) [19:43] hi jelmer, I am having some issues with bzr-svn, that I saw were supposedly fixed in bzr 1.6. I am using bzr 1.13 with bzr-svn from launchpad, and am still getting python: subversion/libsvn_subr/path.c:414: svn_path_is_empty: Assertion `is_canonical(path, strlen(path))' failed. [19:43] when trying to push to upstream [19:53] Any way to get a series of patches for a series of commits? [19:53] (In one command, or so.) [19:54] Hm, I guess I'll just shell script it. [19:56] Any thoughts on https://bugs.launchpad.net/bzr/+bug/354036? [19:56] Launchpad bug 354036 in bzr "ErrorFromSmartServer - AbsentContentFactory object has no attribute 'get_bytes_as' exception while pulling from Launchpad" [Undecided,New] [19:57] I suspect it may have to do with the recent Launchpad bzr 1.13 upgrade [19:57] But that's speculation. [19:57] Hello! If I have pushed a commit but forgot to sign it, how can I push just the signature afterwards? [19:57] klbate: You can run "bzr sign-my-commits", but I dunno if pushing afterwards will push the new signatures. [19:59] Sorry, was disconnected... [20:00] In Git there is something like 'push --tags', is there a bzr equivalent? [20:02] klbate_: "bzr push" will silently push new tags, so it's a reasonable assumption that it will do the same with signatures. [20:03] It's worth a try, at least. [20:04] jelmer, really sorry to insist... but whenever you can I would really appreciate it, as it's a blocker for me right now [20:17] can somebody in the ~bzr lp group please copy the qbzr 0.9.9 packages from https://launchpad.net/~qbzr-dev/+archive/ppa to https://launchpad.net/~bzr/+archive/ppa ? [20:29] Huh, bzr just started whining about an invalid config file somewhere. [20:33] lifeless: ping [20:37] lifeless: ping [20:38] emmajane: Oh, hi. [20:38] ola abentley :) [20:39] Ça va? [20:39] what options should I use in order to find out why bzr-svn is crashing? [20:39] presque. :) [20:39] I'm inside and it's nice outside. :) [20:41] abentley, how's the weather down south today? [20:41] abentley, someone told me we're getting another dump of snow. which makes me cry inside. [20:42] emmajane: It's not bad. You can get away without a jacket for a bit. But the clouds have rolled in today. [20:43] abentley: What are you pinging him about:? :D [20:43] abentley, boo to the clouds. [20:43] Peng_: WorkingTree4.iter_references [20:43] abentley: Oh. [20:45] emmajane: Did you know Mike Audet's up in Peterborough nowadays? [20:46] abentley, the nerve! what's he doing up there? [20:47] abentley, my cousin is at Trent. :) [20:47] emmajane: Teaching, doing some programming on the side. [20:47] abentley, sweet! [20:57] it's fairly standard in bzr to use "r800" as an abbreviation for "revno 800", right? [20:57] (Users Guide seems to indicate so, but only by implication...) [20:58] kfogel: i certainly do that quite a lot [20:58] mwhudson: thx [20:58] kfogel: I picked up the habit from svn and still use it with bzr. [20:59] Peng_: more confirmation, good [20:59] does -r handle "r800"? [21:00] james_w: in svn, yes :-) [21:00] I thought I remembered a bug about it not doing so [21:04] Oh crap. I just reverted bzr to r3234 instead of r4234. Now it doesn't even recognize the branch format. :D [21:04] Thank goodness I had another copy of bzr around. [21:07] Peng_: pong [21:07] abentley: pong [21:07] lifeless: DirStateWorkingTree.iter_references raises and AttributeError if you don't lock the tree before use. I found this surprising. [21:08] s/and/an [21:08] abentley: thats certainly a bug; I would expect it to raise, but not AttributeError [21:09] lifeless: Oh hi. I filed bug 354075 about what I was pinging you about, but forgot to unping you. [21:09] Launchpad bug 354075 in bzr "Branch.get_config_file smart verb breaks get_bzr_remote_path" [Undecided,New] https://launchpad.net/bugs/354075 [21:09] abentley: btw, how long does it take you to run the bzr test suite ? [21:09] hi umm, how do i merge two different lp branches? [21:10] lifeless: This is because you initialize DirStateWorkingTree._repo_supports_tree_reference when the tree is locked. [21:10] nsh22: cd branch1 && bzr merge branch2 && bzr commit && bzr push [21:10] lifeless: Rather than at __init__ time, as you do with the RevisionTree. [21:11] lifeless: thx [21:11] lifeless: What's the reason for delaying that? [21:12] abentley: I guess I was thinking of cases where 'bzr switch' was used and the backing repo facilities changed [21:12] lifeless: I dunno how long the suite takes. Maybe 25 minutes? [21:12] abentley: with the bzr-ec2test plugin & 2 warmed up instances, its takes ~90 seconds. You might like to play :) [21:22] lifeless: In the 'switch' case, the tree would remain write_locked during the switch, so the _repo_supports_tree_reference would get stale, wouldn't it? [21:22] abentley: possibly yes; I guess you're trying to see whether to work around [e.g. getattr, init to None etc] or to move it to __init__ - I'm happy either way, or even a third [21:23] I'd suggest avoiding a getattr(), I don't think one is justified [21:24] lifeless: I'm also wondering if the win of caching that data on WT is worth it. Thoughts? [21:25] if its accessed during the iter_changes outer loop it may be measurable on 'bzr st' timings [21:28] abentley: oh, possibly it wasn't caching at all; maybe WT2 can't be right root ? [21:28] and so we have tree code that correctly needs to ask the tree not the repo. [21:30] lifeless: I suppose that's useful when you're detecting subtrees and then forcing them to directories because of the repo. [21:31] abentley: well also for 'add-reference' to fail correctly when the tree can't represent the add [21:31] lifeless: In that case, I don't think the caching is essential. [21:32] WT2 can implement add_reference to always raise... [21:46] Peng_: so, what is 'bytes' [21:50] lifeless: Eh? [21:53] ricardokirkner: hi [21:53] ricardokirkner: are you using the latest bzr, bzr-svn and subvertpy? [21:53] hi jelmer, thanks [21:53] bzr 1.13 [21:53] latest bzr-svn [21:53] and subvertpy [21:53] yes [21:53] I have been tracing bzr, until I got to line commit.py:600 [21:53] ricardokirkner: please run inside of gdb and pastebin the bracktrace [21:54] I'll do that [21:54] but maybe I have found the reason. let me explain [21:54] ricardokirkner: subvertpy 0.6.5? [21:54] yes [21:54] and python 2.6 (64 bits) [21:55] the line that causes an abort, is when doing [21:55] ret.append(ret[-1].add_directory(name, copyfrom_url, base_rev)) [21:55] now, I just saw that copyfrom_url ends with a '/' [21:56] I from what I read, that is causing the svn abort [21:56] does this make sense? [21:56] ricardokirkner: copyfrom_url doesn't matter in this case [21:56] ricardokirkner: does path end in / ? [21:57] what path? [21:57] name? [21:57] ricardokirkner: anyway, that backtrace would really be useful here.. [21:57] ok [21:57] sec [22:00] jelmer, http://ricardokirkner.pastebin.com/d3ec659ab [22:04] mwhudson: is loggerhead bust? [22:04] mwhudson: it shows me no diff for http://bazaar.launchpad.net/~geoff.bache/bzr/trunk/revision/4158 [22:04] lifeless: 'expand all' [22:05] mwhudson: does nothing [22:05] lifeless: It's all Ajaxy now. Do you have JS disabled, or a weird browser? [22:05] lifeless: hmm, maybe shift-reload [22:05] lifeless: It works for me. [22:05] Oh, good point. [22:05] i keep meaning to vary the static urls by version... [22:06] Peng_: epiphany, [22:06] mwhudson: thanks [22:06] mwhudson: and please do that, kthanksworking [22:08] ricardokirkner: please try again with latest rev on subvertpy [22:09] jelmer, I installed subvertpy using easy_install, should I try downloading from trunk? [22:09] ricardokirkner: yeah [22:09] ok [22:09] lifeless: any chance you can have a look at my BzrDir.push() patch ? [22:12] jelmer: sure, url [22:13] lifeless: http://bundlebuggy.aaronbentley.com/project/bzr/request/%3C20090401141753.GA16842%40vernstok.nl%3E [22:14] jelmer, same thing :-( [22:14] ricardokirkner: are you sure you're running the latest subvertpy [22:14] ricardokirkner: oh, whoops [22:14] ricardokirkner: push failed, one sec [22:14] oh [22:15] ricardokirkner: pushed properly now, r2051 [22:16] lifeless: Oh, sorry I didn't understand you. I updated the get_bzr_remote_path bug. [22:17] Wait a minute, it's *supposed* to be the contents of branch.conf, so why is ConfigObj balking? [22:19] jelmer, :-( [22:20] ricardokirkner: ? [22:20] same thing still happening [22:20] I checked and doublechecked [22:20] ricardokirkner: backtrace? [22:20] ricardokirkner: at least the line numbers should be different.. [22:21] ok, one sec === abentley1 is now known as abentley [22:23] jelmer, http://ricardokirkner.pastebin.com/d767040a9 === abentley1 is now known as abentley === abentley1 is now known as abentley [22:32] jelmer, I have to go now,.. will try again later [22:32] thanks a lot for your help [22:33] ricardokirkner: I'm working on a fix [22:33] ricardokirkner: to subvertpy [22:33] should be committed in a few minutes [22:39] lifeless: ping [22:45] Peng_: pong [22:46] lifeless: Hi. [22:46] hawwo, was just responding, again, to the 1.14 delay thread [22:46] hopefully Ian wins troll of the year award [22:46] for his april first post [22:46] ? [22:47] new 1.14rc date, was posted april 1st [22:47] Oh. [22:47] he really needs to jump out about now in the thread and say 'april fools' [22:47] :) [22:47] anyway [22:48] lifeless: I was just pinging you because I want hand-holding through the get_config_file bug. :D [22:48] right [22:48] just change it to bytes.splitlines('\n') is probably enoug [22:48] rather than [bytes] [22:49] also config objs constructor blows chunks [22:49] I just used "splitlines()". That's what I wanted to ask. [22:49] oh [22:49] yeah that shold be fine [22:49] working >> anything else [22:49] Is that alright? Should I only split on \n? [22:49] splitlines should be ok [22:50] jelmer: does bzrdir.push work for you? [22:52] lifeless: thanks for the response regarding 1.14 RC date [22:53] I think mbp should never go on another holiday again, ever! :-P [22:54] BasicOSX: :) [23:01] lifeless: Anyway, the other thing I wanted to ask you about was adding a test. [23:01] Um. [23:01] Either expand one of the test_remote.py tests for _get_config [23:01] or add another in the same class [23:01] just set two keys and then get one back [23:01] should trigger [23:03] Um. [23:06] Pardon my dumbness, but I'm completely lost in the test suite. [23:07] una momento [23:07] Wait, hold on. [23:07] I think I can just copy test_get_branch_conf. [23:08] Or I could just modify it to use a 2-line file. [23:10] Oh, interesting. ConfigObj(['# line 1\n# line 2\n'], encoding='utf-8') works. [23:10] You need non-comment lines to trigger the failure. [23:15] lifeless: Where should I add the test? test_remote.TestBranchControlGetBranchConf? That seems to be the only place it's tested. [23:15] lifeless: (I mean, in that file..) [23:20] Peng_: yes, thats the unit test for it [23:21] lifeless: Should I modify the current test or add a new one? I'd prefer to do the latter, just...cuz. [23:22] either is fine, new test is clearer [23:24] Okay. [23:24] Then...send it to the mailing list? :D [23:26] lifeless: If that email *was* an April Fool's joke, he definitely wins this year. That sparked a huge discussion! [23:29] Peng_: sadly it wasn't [23:29] Peng_: yah, [MERGE] it up [23:29] lifeless: Oh, too bad. [23:29] it will need to be cherry picked to 1.14 too now :) [23:29] lifeless: ok [23:33] jelmer: ping [23:33] jelmer: does the BzrDir.push patch meet your needs, and is it mainly moved code? [23:34] lifeless: yes and yes [23:34] Hello there :) [23:34] jelmer: land it [23:34] lifeless: is that a bb:approve ? :-) [23:35] a (perhaps stupid) question: provided I want to keep a set of config files in a bazaar repository, is there a way of saying 'get me this revision of this file from repo'? [23:35] jelmer: not quite [23:35] yacoob: bzr cat -r X this_file [23:36] lifeless: so what do you mean exactly? [23:36] I've reviewed [23:37] BasicOSX: I too am a little confused [23:37] BasicOSX: did you mean 'land bbc now' too ? [23:38] lifeless: yes [23:38] ok cool. [23:38] we'll land on trunk and merge to the rc branch then - you might like to clarify that on the list :) [23:38] lifeless: ? land it where ? [23:39] lifeless: oh never mind [23:39] lifeless: I hadn't seen your email reply [23:41] What's being landed where? [23:42] lifeless: Anyway, I think I sent it. My IMAP server is slow today, but SMTP probably works. [23:42] Peng_: if I understand BasicOSX correctly; brisbane-core will go to bzr.dev asap (meaning a once-through review) and be cherry picked to 1.14 [23:42] BasicOSX: ^ is that what you mean to have happen? [23:43] Cherry-picking hundreds of revisions consisting of thousands of lines of code sounds like "fun". [23:44] hmm [23:45] * jelmer thinks one of the hot topics at the bzr sprint this year should be "parallel imports" [23:47] 'ello. bzr newbie here. i just did a couple commits, then did "uncommit" twice, then commited that. all seems to have behaved as epected, and bzr status shows no changes. BUT, the actual source tree has not been changed! [23:48] johnjosephbachir: it would't have [23:48] ORLY [23:48] johnjosephbachir: "uncommit" and "commit" don't change the working tree. [23:48] uncommit pops a commit off, it doesn't revert the changes in your tree [23:48] I was under the impression BC was in 1 branch that could be merges in bzr.dev? [23:48] BasicOSX: it is [23:48] :%s/merges/merged/ [23:48] lifeless Peng_ okay cool cool... sooooo what do i do? :) [23:49] johnjosephbachir: well, what do you want to achieve? [23:49] Sorry on the busy and very bumby, typing going to be hard [23:49] busy=bus :-) [23:49] type [23:49] :) [23:49] lifeless: i waaaant.... to make my source tree reflect... my commits? (maybe i have the wrong terminology) [23:50] BasicOSX: so am I understanding correctly? [you can say yes/no] - we merge it to .dev, and to 1.14 [23:50] johnjosephbachir: After uncommitting, if you want to throw out all of your noncommitted changes, use "bzr revert". [23:50] * igc breakfast [23:50] Peng_: hmm... just tried it... no change to files [23:51] yes .dev, yes 1.14 [23:51] BasicOSX: thanks! will get right on it [23:51] johnjosephbachir: Because you went and committed all of the changes again. [23:51] Peng_ lifeless oh maybe i didn't commit... the uncommits properly, because i didn't revert after doing the uncommits? [23:51] haha, okay [23:52] johnjosephbachir: You don't commit an uncommit. When you run uncommit, it removes the revision from the branch's history. [23:52] can yyuou clarify on the maikkling llist (eerrr!) so people aren't freaking out? <- lifeless [23:52] johnjosephbachir: After running "uncommit", it's done. There's nothing left to do. [23:52] Peng_: OIC [23:52] thanks [23:52] i will follow up properly once I get home [23:52] johnjosephbachir: When you ran "commit" afterwards, all you did was commit all of the changes in your working tree, including the changes from the 2 revisions you uncommitted. [23:52] oh noes! [23:53] Peng_: okay, thanks, i'll try to apply all that knowledge and see where i get [23:53] Peng_: yes, working perfectly now. hooray for bzr! [23:53] :D [23:54] BasicOSX: sure thing === abentley1 is now known as abentley [23:55] almost out of battery back tonight [23:55] ciao === thewrath_ is now known as thewrath [23:58] hey all [23:58] i got bzr working