/srv/irclogs.ubuntu.com/2011/08/31/#bzr.txt

=== yofel_ is now known as yofel
pooliehi all00:14
=== AuroraBorealis is now known as aurora|afk
=== pjdc_ is now known as pjdc
vilahi all !06:19
mgzmorning vila!06:24
vilamgz: hey !06:24
vilamgz: I did comment on your mp but lp dropped my mail06:24
vilamgz: td;lr : please land whatever you can06:25
vilamgz: I share your curiosity about the consequences on pqm slowness06:25
mgzI will merge up to dev and see where I'm at.06:25
vilamgz: see my last comment on https://code.launchpad.net/~jelmer/bzr/config-file-permdenied/+merge/7336107:02
mgzvila: that branch should just land, but there does need to be a bug at least, and maybe a comment there07:04
mgztwas a comment, not a needsfixing07:05
vilamgz: can you file it and comment on the proposal ?07:05
vilayeah07:05
vilahmm07:05
vilamgz: is there an idiom we can use today that would avoid the mangling ?07:06
vilamgz: because this patch calls trace.warning twice, but the second call hides the path shuffling into a method that already contains a semi related FIXME (config.Store.external_url)07:08
mgzdelay as long as possible, and keep the actual objects around to stringify when there's an output stream for them to go to07:08
mgzthe logging module doesn't help much here07:08
vilahmm07:09
vilawe are calling trace.warning to output *now*, so basically you're saying that trace itself should provide some way to encode the path/urls properly ?07:10
mgzbut it's going to two places07:10
vilaoh my, right07:11
mgzone, a file where we should (but don't really) enforce a utf-8 encoding07:11
mgzand (possibly) one to the console07:11
vilaso what ? define and use .format() instead of '%' and inject some path/url specifier in the format string ?07:12
vilaand will that be enough to cover cases where the fs encoding and the terminal encoding can't both provide a meaningful representation of the path ?07:13
mgzthis is where knowing it's a path/url and not just some string is useful07:14
mgzbecause then you can percent-escape and get a correct reference07:14
vilapercent-escape == url-encoded ?07:15
mgzif you've already stringified and just have the message to output, there's no safe transform you can do to get a correct path back again07:15
vilaright, I agree that this is the root cause07:15
vilaand I'm not amused by the fact that jelmer used 'utf8' instead of fs encoding either ;)07:16
mgzit's correct for the log, and his terminal :)07:17
jammorning all07:23
jammgz: its correct for how the api of trace.note() and trace.warning() are currently defined.07:25
mgzwhich is a bug.07:25
jammgz: I don't disagree, but it is not his place to fix it here07:25
jamI've *personally* decided not to focus on the issue, because I think if you really wanted to help windows users, you'd be better of polishing something like bzr-explorer07:26
mgzI don't think it is, but the line added will need changing when that bug is addressed.07:26
jamrather than trying to handle the 5-different possible-non-unicode encodings Windows has07:26
jammgz: actually, if we just changed how trace.note/warning worked to have them write to the Windows Console Unicode api07:26
jamthen utf-8 is just fine07:26
jamit can be trans-coded to UTF-16 losslessly07:27
jamwhich is where I'd *much* rather see the fix07:27
vilajam: hey !07:52
vilajam: ping07:54
jamvila: /wave07:54
vilajam: what exactly do you require to approve this safety net is brittle patch to be approved without delaying it for weeks ?07:55
jamvila: I thought I was clear about what I would like to see, a way to stop the test suite if a critical piece of infrastructure fails, that doesn't require us to kill the running process07:57
jamI'm willing to hedge on it that sys.exit is the best we can get07:57
jamit isn't playing nicely as a test suite, but it works fine for "bzr selftest"07:58
vilabut you objected on using it07:58
jamvila " doesn't require killing the running process"07:58
jamthat would be sys.exit()07:58
jamif someone uses another test runner, it will die on them which could be rather surprising.07:58
vilayou objected on using sys.exit(), I removed it, you object again07:58
jamvila: #1) The original point is that we don't want 20k tests failing in the same way, right? Otherwise we would have left it at the status quo07:59
jamI'm not trying to go in circles. I do have a position I would like to see, all of the alternatives seem sub-optimal for different reasons07:59
viladoesn't apply to jelmer's case and for mthaddon's case we already have -107:59
jamso I'm a bit ambivalent about picking one07:59
vilathen let's land self.fail() and continue discussing 'FatalError' so we progress on both fronts08:00
jamvila: if it doesn't apply to either case then why do we have a patch?08:00
jamvila: good enough08:00
vilaboth cases are addressed: we get a meaningful error, mthaddon's case is less well addressed but also highly unusual08:01
jamvila: ISTR that jelmer was surprised to have so many tests failing, but sure08:03
vilaI think jelmer (and I and mthaddon) were all confused because the error was masked and as such made the diagnosis hard08:04
vilathe root of mthaddon case was: running as a user not declared in /etc/passwd. OMG ! How the hell do you end up in such a mess on a posix system ?08:04
vilathe root of jelmer case was: you pretend to be root, yet, you don't have rights to even read files in your home dir. wtf ?08:05
vilajam: I totally see your point about FatalError, but we are playing tricks by handling a shared resource without telling the runner already, detangling that when a 10lines fix is available ? Worth the time ?08:07
jamvila: well, I was hoping it wouldn't be a lot of time by asking people who knew more about it. Unfortunately, one just got engaged and the other is going to have a baby RSN08:08
vilaexactly08:08
vilaso better separate the issues08:09
vilajam: do you mind voting on that proposal ?08:12
jamvoted08:16
jamapprove08:17
Riddellmorning all08:17
vilajam: thanks !08:17
jammorning Riddell08:18
vilaRiddell: _o/08:21
=== quicksil1er is now known as quicksilver
pooliehello europa08:40
vilapoolie: hellllo AU :)08:41
jamih eiloop08:41
pooliehi maj08:41
jam(i couldn't easily write it upside down, so I wrote it backwards)08:41
vila:)08:42
fullermdHello Ganymede?08:43
poolievila, re bug 83792609:06
ubot5Launchpad bug 837926 in Bazaar "needs a way to get precise timings on pqm landings" [Medium,In progress] https://launchpad.net/bugs/83792609:06
pooliedon't the subunit streams have enough data?09:06
vilapoolie: only if you're the submitter09:08
poolieso this bug could be caused by logging the output?09:09
poolie*closed09:09
vilapoolie: https://code.launchpad.net/~vila/bzr/837926-log-make-check/+merge/73492 is a simple and pragmatic way to address it09:09
vilapoolie: I don't want to dive into solutions that requires modifying pqm or anything else, I just want a couple of `date` in the log files (which are already mirrored on chinstrap09:10
poolieif that's enough that's fine with me09:10
pooliehm09:10
pooliewell, approved09:17
vilapoolie: the '.bzr' invocation already redirects stderr, so I'm more comfortable with adding statements *around* its invocation that risking breaking it09:19
vilapoolie: thanks for the review !09:19
pooliethen time $(SHELL) -c './bzr ... '09:19
pooliebut, ok, i see your point abotu risk09:19
vilapoolie: re quoting, simple of double quotes ? (Can't remember which one is more likely to fail ;)09:20
pooliefailing and non-failing quotes? :)09:22
vilabah, I'll just remove  the brackets and go with double ones ;)09:23
poolieit seems to all be rediceted already09:23
pooliealso, i thought we got rid of the separate run in ascii mode?09:23
pooliedid that come back? perhaps that accounts for the slow down09:23
vilapoolie: only for 2.009:24
poolieoh, i hadn't noticed this was into 2.009:24
pooliethough you do say so09:24
vilagone in 2.109:24
vila2.2 added subunit09:24
pooliei'm a bit skeptical about changing this in old series09:25
vila2.3 added robustness09:25
vilapoolie: I can start anywhere you want, I'm mostly interested by 2.3/2.4/trunk myself09:25
poolieso, changing this stuff has historically tended to throw up environment-dependent failures09:26
poolietherefore i wouldn't change it in a stable series without a really good reason09:26
vilaright, me too,09:27
pooliewhat are you actually going to do with this?09:27
vilathe effect should be seen when landing the patch itself, so either it works or it breaks09:27
pooliethat hasn't always been true in the past09:27
vilasure, I don't mind starting at 2.4/trunk or even only trunk if you prefer09:28
vilathe intent is to get timings and a poor-man's monitoring09:29
pooliei think just trunk09:29
vilaok09:29
poolieso are you hoping to find out that eg it's consistently slower, or it's very variable from one revision to the next, that sort of thing?09:30
vilayeah, I don't have any pre-conception09:31
poolieok09:43
poolieso i wouldn't like to break anything just for kind of snooping-around investigation09:43
pooliesidnei sent me some tarmac/etc instructions which i hope to set up some time09:44
vilapoolie: I'll stop and revert on first sign of an issue, I swear ;)09:44
=== Quintasan_ is now known as Quintasan
jamvila: for the datestamp, would you want to use: `date "+%F %T"` ? It seems a bit easier to parse than just date09:58
poolieor date -I10:01
pooliesorry, -Isec10:01
jamthat works, too10:01
vilajam: th... err, ok, we still have some time before pqm catches up ;)10:02
jampoolie: odd, 'man date' doesn't seem to report it, but 'date' supports it10:02
vilaright, I was about to ask about portability :)10:02
poolieyou have no mandate! i demand a new election.10:02
fullermddate: illegal option -- I10:02
poolieit is a gnu extension10:02
vilafullermd: does BSD use gnu date ?10:03
vilaosx doesn't10:03
Riddellis there an equivalent in python of  isinstance(foo, function) ?10:03
poolieis it a function?10:03
ccxCZyou have callable()10:04
poolieprobably you want 'callable(foo)'10:04
jamRiddell: do you want something like callable() ? which has been deprecated10:04
fullermdOf course not!  We ain't lettin' no commie control something as important as time!10:04
poolie:)10:04
jamnote that a Class that implements __call__ is 'callable()'10:04
jamwell, an object ...10:04
poolieso it depends on exactyl what you meaon10:05
poolieanyhow, i should stop10:05
pooliehave a good day all10:05
Riddelljam: if it's deprecated what replaces it?10:05
jamhave a good evening poolie10:05
jamRiddell: something about abstract base class Callable, let me see if I can find it10:05
Riddelldocs don't say it's deprecated http://docs.python.org/library/functions.html#callable10:05
vilajam, poolie, fullermd : I think I'll stick with bare `date` , parsing can be addressed10:05
ccxCZtype(foo) == types.FunctionType10:06
jamRiddell: maybe it is deprecated only in py310:06
ccxCZif you want to check for functions only ^10:06
jambut it certainly exists in 2.7 so feel free to use it10:06
jamRiddell: http://docs.python.org/dev/whatsnew/3.2.html10:06
jamsays that they restored it10:06
jambecause they felt it was more readable than isinstance(x, collections.Callable)10:07
vilapoolie: cu10:07
jamso I missed that part. And yes, it is not deprecated after all10:07
Riddelllovely, thanks10:07
fullermdvila: Well, if it's meant to be "parsed" by a human, plain output is at least as good as any other.10:07
fullermdFor auto-parsing, epoch would be simplest probably.  Ignoring the mess of nonmonotonicity/continuity of timekeeping in general anyway...10:09
jamfullermd: I still find -I easier to parse as a human when I'm trying to do something like *compare* the dates10:09
pooliethisi sa kludge; let's not overoptimize it10:09
poolieremember you have the option to just ask sysadmins to run experiments for you10:09
jamEspecially if you have to wrap the months (though that shouldn't be a factor here)10:10
jampoolie: well, I've been pqm-submitting branches that are known-broken which works fine. But for the historical record, it would have been nice to have the datestamps10:10
jamand then "oh, in rev X things are a lot slower"10:10
vilajam: python -c 'import datetime; print datetime.datetime.strptime("Wed Aug 31 12:16:18 CEST 2011", "%a %b %d %H:%M:%S %Z %Y")'10:23
jamvila: still harder to think about, etc. Anyway, it isn't like it really matters, it was just a suggestion.10:24
vilajam: sure, but poolie preferred trunk only to lower the risks, I'd rather not add risks for portability10:25
jamI'm pretty sure +%F is portable10:25
jamit is -Isec that isn't10:25
ccxCZ%F doesn't seem to be in posix http://pubs.opengroup.org/onlinepubs/000095399/utilities/date.html10:29
fullermdIt's part of strftime in C99 (though not C89)10:30
jelmerRiddell: hi10:40
jelmeris it common practice for uploaders to request syncs from Debian, or is it also ok if I just re-upload packages directly to Ubuntu, by myself?10:40
Riddelljelmer: it's better to request a sync10:41
Riddellassuming there's no changes to be made10:41
jelmerRiddell: Cool, thanks.10:42
RiddellI can do the sync if there's no freezes to care about, just need the bug number10:42
jelmerRiddell: that'd be great, I forgot you were an archive admin10:43
vilajelmer: will be afk (lunch), let me know if/when bzr-2.4.0 is deployed on jubany, and thanks for doing it  ;)10:44
jelmerRiddell: bug 83169010:44
ubot5Launchpad bug 831690 in bzr-gtk (Ubuntu) "Sync bzr-gtk 0.100.0+bzr734-2 (universe) from Debian unstable (main)" [Undecided,New] https://launchpad.net/bugs/83169010:44
jelmerRiddell: I'll have a couple of others later today for the other bzr plugin FTBFS bugs, if that's okay.10:48
jelmervila: will do :)10:48
Riddellsure10:48
=== nyuszika7h` is now known as nyuszika7h
doudeHi, all. I try to log with my launchpad login on bzr: bzr lp-login nickname12:14
doudebut I work behind a firewall, and I cannot contact launchpad on SSH port12:15
doudeand now each bzr branch commands fail12:15
doudeHow I can unlogin to just clone some launchpad repository ?12:16
viladoude: you need to remove launchpad_username from bazaar.conf12:16
viladoude: do you have a bzr recent enough to run 'bzr config' ?12:16
doudevila: yes12:20
jamugh. just accidentally passed a .gz stream to subunit, and it decided the whole thing should be "passthrough", which meant it sent 600kB of garbage to the terminal, which got the system bell *really* excited12:20
jamif the bell is 3s long, and system-bell is 1/255 of the chars, I estimate it will take 2 hours for it to stop beeping12:21
vilajam: ouch ;)12:21
doudevila: thanks a lot for your help12:21
jamfortunately 'mute' works, but volume itself doesn't12:21
viladoude: then bzr config --remove launchpad_username in your home dir should do12:21
doudevila: ok, it works12:22
vilawow, the garbage killed jam :-/12:24
doudevila: Is it possible to log to launchpad if the SSH port is blocked ? by another way ?12:24
viladoude: short answer: no12:24
viladoude: there are various tricks to route ssh but that will need cooperation from the firewall admin in most cases12:25
vilajam is back \o/ death to the garbage ! ;)12:27
jam:)12:27
doudevila: ok12:28
viladoude: you can access lp in read-only mode without ssh though12:28
doudevila: ok12:29
doudebye bzr men12:29
viladoude: and if I remember correctly you can even make merge proposals via mail but I don't remember the details12:29
vilano fast enough :-}12:29
vilanot12:29
jelmerjam: just to confirm, you were happy with my move-hashcache MP if I added a common base class for WT2 and WT3 ?12:32
jamjelmer: I think I approved it already, but if not, yes12:33
jamah,I think I forgot to actually vote12:34
jelmerjam: yep, your last vote was needsinfo. anyway, thanks12:34
jamI just updated it12:37
jelmerI noticed, already sent it off to pqm :)12:39
Peakerhey, what's the difference between -c and -r in bzr merge?13:16
Peakerhttp://doc.bazaar.canonical.com/latest/en/user-reference/merge-help.html  doesn't make it clear13:16
Peakeroh, I just read that cherry-picks via "bzr merge" are not tracked by bzr :-(13:17
PeakerI was wondering whether my current gripe with git (cherry-picks don't track the history, merges force non-modular resolution of all conflicts at the same time) was solved by bzr13:18
jelmerPeaker: hi13:25
jelmerPeaker: unfortunately we don't really have either of those features either (yet)13:26
jelmerwe would like to support both, in particular tracking cherry pick merges, but it's a hard problem to solve if you want to get it right.13:27
Peakerjelmer: well, we're using git, and I noticed some serious problems -- for example:  if you have two branches, A, and B.  You cherry-pick a bad commit A->B.  Now you revert the bad commit only in A. Then you merge A and B -- you will get the bad commit again, because a 3-way merge considers (bad+revert = 0  <  bad)13:33
Peakeralso, a "merge" in a large project that involves many people is pretty horrible -- conflict resolution requires the expertise of many different people!13:34
Peaker(and they're all forced to share a working tree to work together on the problem)13:34
Peakerand the conflict resolutions are later very hard to view in isolation, because merge diffs are hard to view (relative to which parent?)13:34
PeakerSo my conclusion is that cherry-picks are really the right way to move work around -- but without their metadata being tracked, it works horribly13:34
PeakerI think darcs has the right model of "cherry picks" are the only way to move work around and are tracked.. but it has performance issues13:35
jelmerPeaker: three way merges are relatively easy because there's really only three trees you have to care about13:43
jelmerPeaker: with cherrypicks you could have to consider all changesets in a repository13:44
jelmerit would be relatively easy to track cherrypick merges in bzr (in revision properties, for example) but using that information intelligently and without making merge scale badly is the hard bit13:45
Peakerjelmer: I think darcs just makes merges scale badly :)13:49
Peakerbut I'm thinking it might be worth it... We're getting horrible horrible merges, and cherry-picking everything means we lose all DVCS tracking of everything13:50
Peakerthe only solution we have is to just merge extremely often so no divergence picks up13:50
jelmerPeaker: btw, I'm pretty sure bzr will pick up the revert in the case you mention13:54
jelmerPeaker: bzr does track per file history, unlike git, which just tracks content13:54
Peakerjelmer: but you do a 3-way merge here, no?13:56
Peakerthe per-file history here is that in branch A, the file had changed, and then changed back, so sum of all changes = 0.   In branch B, it just changed. So the sum = change.   3-way merge of 0 and change = change, no?13:57
jelmerPeaker: bzr doesn't look at the checksum to see if a file has changed. It keeps track of what revision it was last changed.13:57
Peakerso what would bzr do here? it would see the file changed in branch A, and then changed back, and consider it a conflict?13:58
jelmerPeaker: it should have enough info to create a conflict; I'm not sure if it actually creates one14:00
Peakerjelmer: git has the same info, of course, it is also a question of whether it looks at that info14:00
jelmerPeaker: git doesn't know what revision a file last changed in, unless it's going to scan history (which it doesn't)14:01
jelmerit just finds the common base to use for the 3 way merge14:01
jelmerabentley: hi14:01
Peakerjelmer: ah, yes, perhaps14:01
abentleyjelmer: hi.14:01
Peakerjelmer: though I think merge does scan history in various cases14:01
jelmerPeaker: it scans it to find the base for the three way merge14:02
Peakerjelmer: Well, that scan is per-file, so it had already encountered all of the version that changed it14:03
jelmerPeaker: finding the base for the merge? that can only be done based on the revision graph14:03
Peakerjelmer: yeah -- and when walking that graph, it sees whether that file changed in each of the revisions14:04
jelmerPeaker: it doesn't look at the files during that scan, just the revisions14:04
Peakerjelmer: I'm pretty sure it does the 3-way merge basis lookup on a per-file basis, according to the debug prints visible when encountering conflicts. But I'm not entirely sure14:05
Peakerand if it does that -- it should not be expensive to see whether the file changed or not, in those revs, if it didn't already14:05
jelmerPeaker: it is fairly expensive because to know what file has changed you have to apply the rename/copy detection heuristics14:06
jelmerPeaker: to follow the changed files I mean14:07
jelmeras far as I understand it it uses the revision graph to find the base revision, and then applies the heuristics between just the base revision and the other/this trees14:07
jelmerPeaker: bzr's merge doesn't generate a conflict either14:10
vilajelmer: care to have a look at https://code.launchpad.net/~vila/udd/806425-append-revs-only/+merge/73531 ?15:05
jelmervila: sure15:09
jblueQ: is there a way to commit/operate on a specific set of changes after a 'bzr status'?  For example, if bzr status shows removed, modified, and unknown files, and I want to work only with the removed files, is there some way to specify that?15:09
vilajblue: 'bzr commit file1 file2' should just work15:11
jblueit does, but was looking for a method working with that set, in an instance where I have a lot of files being removed.15:12
jbluebut also have a lot of files modified, and their locations are intermixed15:12
jblueso specifying them one by one is a bit tedious15:12
vilamay be 'bzr qcommit' provides some help ?15:13
vilaDid you get there after a merge or some heavy refactoring from an IDE ?15:14
jblueheavy refactoring.. basically several files have been removed, modified, and added, and I'd like to do the removal first, then work with the others15:14
vilabut no file has been both removed and added right ?15:15
jbluewell it was, but I quickly saw that was going to be a headache, so I've reverted, and am doing the removal first15:16
vilaI meant, you didn't *rename* files by doing remove/add right ?15:17
jblueno15:17
vilaok, good15:17
=== jpds_ is now known as jpds
jelmervila: r=me15:26
vilajelmer: thanks, I'm deploying15:26
vilaha, damn, been interrupted and forgot to answer15:26
vilajelmer: my understanding is that a_r_o should only be applied in *some* cases so having a global switch doesn't make sense at this point15:27
jelmervila: I mean, having a global variable that's set to True or False15:28
vilawhat for ? switch it back to True ?15:28
jelmeryeah15:29
vilaAIUI, using it for local branches is just a Bad Idea, so after some checking I will probably just get rid of the calls themselves15:30
vilafor the lp branches, AIUI again, the rules are not well understood yet so we probably end up replacing the calls by some more logic15:31
vilaso in the end, none of the actual calls should survive15:31
vilajelmer: or do you expect such a breakage that we want to come back to a_r_o=True ?15:31
jelmervila: I'm not really bothered either way :)15:32
vilahmm, looks like lp has or had some transient issues...15:32
gypsymauroI've some mess in my repo and if I try to do a ci it says: bzr: ERROR: Bound branch BzrBranch7('file:///....') is out of date with master branch BzrBranch7, To commit to master branch, run update and then commit. but I want to be sure that nothing in my working directory will be changed.. I'm sure my current version is the correct version15:32
gypsymauroany hint?15:32
vilagypsymauro: hi15:35
jelmergypsymauro: I think "bzr diff -rbranch:/path/to/master/branch" should tell you what it would change15:35
vilawhat does 'bzr missing' says ?15:35
gypsymaurois says that I've two extra revision (and is true, i done some ci --local)15:37
vilaany uncommitted changes ?15:38
gypsymaurothe bzr diffs displays me 29645 lines..15:38
jelmergypsymauro: in that case it shouldn't change your working tree, although it will change your local commits to be pending merges15:39
gypsymaurovila: what do u mean? I committed locally, only this , how can I now commit to the remote repository?15:40
jelmergypsymauro: if you run "bzr up" and then "bzr commit" that will commit your changes to the local repository15:41
jelmerargh15:41
jelmers/local/remote/15:41
gypsymaurojelmer: and this for sure doesn't change anything in my local repository? (the update term scares me :)15:48
jelmergypsymauro: it will update bzr metadata in the local tree - it will change your two local commits to be pending merges, and your branch tip to match that of the remote branch15:49
jelmergypsymauro: if the remote branch doesn't have any extra revisions (as "bzr missing" seems to indicate) then it shouldn't change any of the files in your tree15:50
gypsymaurojelmer: doh..15:52
gypsymauro bzr conflicts15:52
gypsymauroConflict: can't delete  foo/bar  because it is not empty.  Not deleting. Conflict because  foo/bar is not versioned, but has versioned children.  Versioned directory.15:52
gypsymaurodamn.. it creates a lot of .BASE .OTHER and .THIS file...15:53
gypsymauroit's a nightmare..15:58
jelmergypsymauro: ouch, that sucks :(15:59
fullermdI keep wishing we'll just squirrel ci --local away somewhere deep hidden...15:59
jelmergypsymauro: you had local commits that added foo/bar ?15:59
jelmerfullermd: yeah, me too. I think we should just get rid of them.16:00
gypsymaurodamn...16:00
gypsymauroif I recover from a backup my foo/bar? this resolves my conflicts?16:01
gypsymauroor I've to merge manually ?16:01
jelmergypsymauro: I suspect "bzr resolve --take-other" might fix this for you, but I'm not entirely sure16:02
vilagypsymauro: sry was called elsewhere16:02
=== beuno is now known as beuno-lunch
vilagypsymauro: if you have conflicts, a merge already occured (triggered by update)16:03
gypsymaurojelmer: uhm i suppose the .THIS version is the correct16:03
vilagypsymauro: how conflicts ?16:03
vilagypsymauro: how many conflicts ?16:03
gypsymaurovila: 4716:03
gypsymaurowc -l16:04
vilahmm16:04
vila!paste16:04
ubot5For posting multi-line texts into the channel, please use http://paste.ubuntu.com | To post !screenshots use http://imagebin.org/?page=add | !pastebinit to paste directly from command line | Make sure you give us the URL for your paste - see also the channel topic.16:04
vilagypsymauro: can you pastebin the output of 'bzr conflicts' ?16:04
vilagypsymauro: and 'bzr st'16:04
gypsymaurohttp://pastebin.com/TCYWVLy616:05
gypsymauroand http://pastebin.com/nWPRg99U16:06
=== deryck is now known as deryck[lunch]
jelmerRiddell: still there?16:08
jelmerRiddell: another sync is bug 83795616:08
ubot5Launchpad bug 837956 in bzr-dbus (Ubuntu) "Sync bzr-dbus 0.1~bzr51-1 (universe) from Debian unstable (main)" [Undecided,New] https://launchpad.net/bugs/83795616:08
vilagypsymauro: :-/ Looks like you added files that were also added remotely...16:09
vilagypsymauro: but you said 'bzr missing' only reported local revisions right ?16:09
jelmervila: the problem is that update works in two phases; first it updates to the remote branch, then it merges in the local commits again16:10
jelmervila: I *think*16:10
vilajelmer: but we stop as soon as we get a conflict (or was it discussed but not implemented ?)16:10
vilagypsymauro: what bzr version are you using ?16:10
jelmervila: sure, but he was up to date16:10
fullermdThe usual cause of kerfufflage there is when you have uncommitted changes as _well_ as local commits.16:10
vilafullermd: I asked for uncommitted changes first ;)16:11
fullermdI thought we added some guard on that, but maybe we didn't.  I dunno, I stay too far away from those dragons to remember   :p16:11
gypsymauroBazaar (bzr) 2.1.216:11
vilagypsymauro: ouch, OS/version ?16:11
gypsymaurodebian squeeze16:12
jelmerRiddell: and I have a bzr upload waiting for approval16:13
Riddelljelmer: waiting where?16:13
vilagypsymauro: rrright, let's do it the old fashion way then16:13
vilagypsymauro: do you have the qbzr plugin installed ?16:14
jelmerRiddell: waiting for approval by an archive admin; looks like it was just rejected16:15
ScottKjelmer: I rejected 2ubuntu1.  You just uploaded 3ubuntu1.16:15
ScottKAt this point it needs to wait until after Oneiric Beta 1 is done.16:16
jelmerScottK: ah, my mistake. Thanks!16:16
ScottK(I also pinged you on #ubuntu-devel about it ...16:16
ScottK)16:16
gypsymaurovila: now yes :D16:17
vilagypsymauro: great, try 'bzr qlog'16:17
jelmerScottK: yep, not sure why I missed that16:18
gypsymaurovila:  ok16:18
vilagypsymauro: we need to find the last revision committed locally (we'll need the revision on your remote branch to, but this one will be easy to get)16:18
gypsymauroi've two light blue circles16:18
vilagypsymauro: 'bzr revno' ?16:19
gypsymauro ok the last is 1416:19
vilado you see a 'pending merges' label ?16:19
gypsymauroyes16:19
gypsymauro14.1.216:19
gypsymauroand another blue 14.116:20
gypsymauro14.1.116:20
vilaexellent, that's your local commits16:20
gypsymaurowithout the pending msg16:20
gypsymauroexactly16:20
vilajustasec16:20
vilagypsymauro: roughly, we will create a branch from remote and 'merge ../with-conflicts -r14.1.2'16:22
gypsymaurovila: I'm in ur hands... take care about 2 weeks of work :) I've a backup btw16:25
vilagypsymauro: i.e.: in a new directory: bzr branch <remote> recover ; cd recover ; bzr merge <conflicts> -r14.1.216:26
gypsymaurovila: doing the first step16:29
vilagypsymauro: no worries, try (above the dir with conflicts) : bzr branch <conflicts> -r14.1.2 my-precious16:29
gypsymaurowhat do u mean with <conflicts>16:29
vilayour checkout dir16:30
gypsymauroyou mean where I've my precious sources?16:30
vilaI call it <conflicts> to denote that's the one where the conflicts are *now*16:30
vilahehe, yeah, except you've committed them, so they are safe in a revision in the repo16:30
gypsymauromy local repo16:31
vilayes16:34
fullermdWait, he's committed the conflicted merge?16:35
vilano16:35
=== deryck[lunch] is now known as deryck
fullermdSo how would the local commits (now pending merges) have a dotted revno available?16:35
vilacan't commit with conflicts16:35
fullermdWouldn't you need to go by revid?16:36
vilahlol, good point, we need the revids16:36
vilagypsymauro: sry, in that qlog window, note the revid for 14.1.2 checking that's indeed your last local commit16:37
* fullermd is a cheenius sometimes.16:37
vilafullermd: thanks for the heads-up16:38
fullermdvila: You'll get my bill in the mail   ;p16:38
vilahehe, as usual, don't forget to bill for some more goats too, say half a dozen ?16:38
gypsymaurovila: ok now I try , and all that .THIS and .OTHER will disappears?16:45
vilagypsymauro: so you've got a copy of your remote branch ?16:45
gypsymauroyes16:45
gypsymauroI don e  bzr branch <remote> recover ; cd recover ;16:46
gypsymaurosteps16:46
gypsymauronow I'm doing that bzr merge <conflicts> -r14.1.216:46
vilago there and do 'bzr merge ../<conflicts -r<revid>'16:46
vilagypsymauro: as fullermd pointed out the revno is probably not available (qlog lied ;)16:46
vilaso you need,16:46
vilagypsymauro: sry, in that qlog window, note the revid for 14.1.2 checking that's indeed your last local commit16:46
gypsymaurobzr: ERROR: Requested revision: u'14.1.2' does not exist in branch:16:48
gypsymaurodoh...16:48
vilaright, fullermd was :)16:48
* fullermd buffs his nails and looks important.16:49
vilagypsymauro: qlog *pretends* the revnos exist, but they don't until you commit16:49
gypsymauroI'm getting lost16:49
vilagypsymauro: just go back to qlog, select 14.1.2 and it should display a pane with some details on the revision16:50
gypsymaurowhich detail do u need?16:50
vilagypsymauro: one of them is the revid which is used internally and in some recovery contexts, normally you only deal with revnos (14.1.2 is a so-called dotted revno)16:51
vilagypsymauro: a revid is ugly, except it probably has your id/email in it so it's not that ugly ;)16:52
gypsymaurovila: it's something like email@....bla bla bla?16:52
gypsymauroah ok16:52
gypsymauroso -rrevid?16:52
vilayup, pedantically -rrevid:email@bla bla bla16:52
gypsymauroAll changes applied successfully.16:52
vilabzr st ?16:53
vilayou should get something that looks like a summary of your two local commits16:53
viladitto for 'bzr diff'16:53
gypsymaurovila: well where is applyed the merge?16:55
vilagypsymauro: what does 'bzr st' says ? It should finish with 'pending revisions' blah16:55
vilas/finish/ends/16:56
gypsymauroI mean in the recover folder I've the old version of files16:56
vilayou did the merge there right ?16:56
gypsymauropending merge tips: (use -v to see all merge revisions)16:56
gypsymauroyes of course16:57
vilaso you just merged the last 'commit --local' you did in <conflicts>16:57
vilagypsymauro: there was no uncommitted changes when did update in <conflicts> right ?16:57
gypsymaurono wait I didnt' made the last commit --local I tried to commit directly on the remote16:58
vilaargh16:59
vila><16:59
gypsymauroI forgot the --local command..16:59
gypsymauroif I move all the .THIS file to the original file?16:59
vilathe fatal trap :-(16:59
sidneiwhat's that really shiny github-like site for bzr hosting which i can never remember the name?17:00
vilawell, now it depends which merge failed, it's unclear from there, so, chose a file you know well and check whether the right one is .OTHER or .THIS17:00
vilagypsymauro: well, now it depends which merge failed, it's unclear from there, so, chose a file you know well and check whether the right one is .OTHER or .THIS17:00
gypsymaurovila: is the .THIS17:01
jelmersidnei: http://wiki.bazaar.canonical.com/Hosting17:01
jelmersidnei: I'm not sure which one of those you mean17:01
vilagypsymauro: two choices from here17:02
vilagypsymauro: either you go back to <conflicts> and resolve the conflicts there OR17:02
sidneijelmer, might be mergebox, but it does not look like what i remember17:02
sidneithe name i remember was codemaster or something17:02
vilagypsymauro: you go to recover and do: 'bzr revert --no-backup; bzr pull --overwrite -rrevid:<revid of 14.1.2>'17:03
gypsymaurovila: what does that?17:03
gypsymaurovila: uhm i think is better to resolve conflicts17:04
vilagypsymauro: the first one is going forward resolving the conflicts17:04
gypsymauroyeah yeah I mean the second17:04
gypsymaurobut if is it what I suppose I prefer the second :)17:04
gypsymaurothe first sorry17:04
vilagypsymauro: the second one is stepping back, restore your working tree as it was when you miss --local17:05
vilagypsymauro: it all depends on what the conflicts are which is hard for me to judge17:05
gypsymaurovila: just can you help me telling what to do when there is a . THIS .OTHER conflict?17:05
jelmersidnei: mergebox isn't really shiny17:05
gypsymaurovila: I've to move the .THIS over the original file and then run bzr resolve myfile?17:06
vilagypsymauro: is there a conflict-type topic in 'bzr help topics' ?17:06
vilagypsymauro: that's the general idea, yes17:06
jelmersidnei: it took me a while to hear about bzr.bz too, it's been there for a long time without anybody knowing17:06
vilagypsymauro: one by one (especially with bzr-2.1 where I don't remember which bugs were still there)17:07
vilagypsymauro: for the record, we have a new series every 6 months and 2.4.0 has just been released so 2.1 is 18 months old17:07
gypsymaurovila:  ok, mv or cp? and then the generated files wil disappear? (.this .other..)17:07
vilagypsymauro: both should work17:08
gypsymaurodoh... bzr: ERROR: Could not acquire lock17:09
gypsymauro/.bzr/checkout/dirstate": [Errno 11] Resource temporarily unavailable17:09
vilagypsymauro: 'bzr break-lock'17:09
vilano wait !17:09
vilaquit qlog first17:09
gypsymaurowho is locking that? I've nautilus plugin17:09
vilatry again17:09
vilain 2.1, could be qlog, can't remember the details17:09
sidneijelmer, it totally escapes me now, but i rememember poolie mentioning it in an email, which i can't find. hopefully im not dreaming. it definitely doesn't look like mergebox, unless they rebranded.17:10
sidneifor worse that is ;)17:10
jelmersidnei: :)17:10
gypsymaurovila: tanx for all ur help :) last question17:12
gypsymaurohow to solve the Conflict: can't delete foo/bar because it is not empty.  Not deleting =17:12
gypsymauro?17:12
vilaright, so, what is inside foo.bar ?17:12
gypsymauroother folders17:13
vilagypsymauro: which are also involved in conflicts ?17:13
=== beuno-lunch is now known as beuno
jamsidnei: sloecode?17:21
jamhttps://launchpad.net/sloecode17:21
jamnot really a website, though17:21
sidneijam, nope. the site i remember had a black background and some shiny polished metal like images, very apple-y. oh well :/17:23
jamsidnei: I remember a pastebin sort of website17:26
jamsidnei: http://chopapp.com/#17:26
jambut it isn't bzr17:27
jamelliot just mentioned http://bzr.bz17:30
vilagypsymauro: so you're back on track ?17:31
gypsymauroI hope17:31
gypsymaurotank you vila17:31
vilagypsymauro: no you know why fullermd....17:31
gypsymauroI'm sorry but my bzr skills ar so low :( I need to study it again!17:32
vilafullermd: I just can't believe you went out/in just around the time we needed to fix gypsymauro issues17:32
gypsymaurofullermd: :D17:32
vilafullermd: you're just faking those connections msgs right ?17:32
gypsymauroyou are great guys17:32
gypsymaurotanx tanx tanx again17:32
gypsymauroreally17:32
gypsymauroyuhu!17:32
vilagypsymauro: well, as fullermd was hinting, --local is.... unusual17:32
fullermdMy spidey sense made me do it.17:32
vilagypsymauro: enjoy !17:33
vilaand on that positive note, I will join family for dinner ;)17:33
fullermdNeed to me break something else so you can get out of that?   ;)17:34
vilafullermd: so, just read the logs, you went out when he realized he... forgot --local...17:34
TA1CZhello17:35
TA1CZmay I ask something about Bazaar explorer for windows.17:35
TA1CZRun command: bzr branch lp:~openerp/openobject-addons/trunk C:/launchpad/openobject-addons/trunk --use-existing-dir17:36
TA1CZConnected (version 2.0, client Twisted)17:36
TA1CZbzr: ERROR: [Error 5] Access is denied: 'c:\\users\\user\\appdata\\local\\temp\\tmpiawygw.pag'17:36
TA1CZI get this error...17:36
jelmerTA1CZ: pag rather than pack?17:37
jamTA1CZ, jelmer: I believe that is a failure of pageant17:37
TA1CZis there any suggestion?17:37
jamtry re-setting up putty17:37
jamor making sure your key is set up, etc.17:37
jamor just stop pageant entirely17:38
TA1CZI used puttygen for generating the keys..17:38
TA1CZboth private and public..17:38
jamTA1CZ: the issue with $TEMP is usually while communicating to the local pageant server17:38
jamit doesn't really matter what the keys are17:38
TA1CZthe putty agent is also working on the right side of windows17:39
TA1CZI added private.ppk key17:40
jamTA1CZ: try stopping the local putty agent for a bit17:40
TA1CZo.k.17:40
TA1CZRun command: bzr branch lp:~openerp/openobject-addons/trunk C:/launchpad/openobject-addons/trunk --use-existing-dir17:41
TA1CZConnected (version 2.0, client Twisted)17:41
TA1CZbzr: ERROR: Connection error: Unable to authenticate to SSH host as17:41
TA1CZ  karen-denki@bazaar.launchpad.net17:41
TA1CZsupported auth types: ['publickey']17:41
TA1CZRun command: bzr branch lp:~openerp/openobject-addons/trunk C:/launchpad/openobject-addons/trunk --use-existing-dir17:41
TA1CZConnected (version 2.0, client Twisted)17:41
TA1CZbzr: ERROR: Connection error: Unable to authenticate to SSH host as17:41
TA1CZ  myID@bazaar.launchpad.net17:41
TA1CZsupported auth types: ['publickey']17:41
jamTA1CZ: is the user-id correct?17:41
TA1CZthe result is like this when I stop the agent!17:41
jam(just to start from the beginning)17:41
TA1CZo.k.17:42
jamWe can't auth via public-key because you don't have the key agent around, and putty uses a different key17:42
jamformat by default17:42
jam.ppk vs id_rsa.pub17:42
jamhowever, start putty again17:42
jamand I think you might see a $TEMP failure17:42
jamunfortunately, I've seen this on one machine (our ec2 build host) only, and haven't been able to reproduce it17:42
TA1CZwhich putty?17:42
jamon my home machine, etc.17:43
jam"pageant"17:43
jamI should have been clearer17:43
TA1CZ:(17:44
jamTA1CZ: it is failing with TEMP again?17:45
jamTA1CZ: what os version/17:45
jam?17:45
TA1CZwindows 717:45
TA1CZso could you say step by step what should I must do is you dont mind?17:46
TA1CZbecause I totally confused..:(17:46
TA1CZis this relevant with authentication.conf?17:48
jamTA1CZ: not really. I think I can work out a workaround for you17:49
jamyou need to run puttygen again17:49
TA1CZo.k. now I am running puttygen17:49
jam(note, I'm discussing it from memory, and I'm not logged into Windows right now, but I'll do my best)17:49
jamyou should be able to load the private.ppk that you created17:49
vilajam: nice mail about fdatasync, make check timing patch landed, first log rsync'ed, the stamps are lost in the noise but nothing a crude parser can't decipher, dinner time here but you'll got a parser first thing I touch this keyboard again ;) Unless you beat me to it that is :-p17:50
* vila really goes to dinner now ;)17:50
jamvila: for now, I'm focusing more on test cases individually. If I have 'pqm-stdout.gz' it has the subunit timestamp stream17:51
jamso 'date' doesn't really matter.17:51
TA1CZo.k. when the putty gen is on the stage.. I load the private.ppk key by pressing Load button.. it is loaded.17:51
vilajam: great17:51
jamTA1CZ: Now you want to export the key into the "openssh" format17:51
jamwhich you've already done a bit of, since you uploaded the openssh public key form to launchpad17:51
vilajam: spurious spikes for tests have been encountered when dealing with socket leaks, cycle checks from mgz and so on, you may want to give a spin merging it into one of your born-to-not-land branches17:53
jamvila: could be interesting. But I believe Martin also indicated the machine is shared17:54
jamso if they're doing stuff at the same time, its gonna get whacky17:54
jamthe test suite was 2.5hrs for a couple of runs, then 3.5 for the other times of the day17:54
vilajam: sure, primary cause, but 3x...17:54
jamvila: just the source of variability17:54
jamprimary cause is still probably fdatasync, which is another failed log I'm looking at17:55
jambut there seems to be slightly more than that as a real cause, and then hard-to-measure-because-of-system variability17:55
jam2.3 can finish as fast as 45min, but took 1:45 last night17:55
jametc.17:55
=== med_out is now known as medberry
=== medberry is now known as med_out
AuroraBorealisdoes bzr hash files when it does a commit?23:17
PengAuroraBorealis: Hash in what way?23:26
AuroraBorealisjust wondering if this statement is true for bazaar as well23:26
AuroraBorealiscontext: kernel.org was compromised recently23:26
PengIt was?23:27
AuroraBorealishttp://pastebin.com/8CKqBPQC23:27
AuroraBorealisyeah it was23:27
AuroraBorealis(see https://www.kernel.org/, first news item)23:28
PengYeah.23:28
AuroraBorealisso basically, can someone change something without bazaar knowing about it?23:28
PengBazaar doesn't uses hashes as revision IDs, but it does hash everything.23:28
AuroraBorealisso if someone did do that you would be able to detect it if you rechecked the repo or something23:28
PengI'm not quite awake enough to think about this problem.23:29
AuroraBorealisxD23:30
AuroraBorealisi dont know enough about the actual structure of the repo itself to figure it out23:30
PengIt would most definitely be possile to verify it somehow, but I don't know how easily.23:30
AuroraBorealisisn't there a --recheck option23:32
=== micahg_ is now known as micahg
PengYes.23:33
mathrickhiya23:34
AuroraBorealiswould be an interesting test to see if someone could modify the repo and see if said recheck catches it.23:34
mathrickhttp://doc.bazaar.canonical.com/latest/en/mini-tutorial/index.html#creating-your-own-copy-of-another-branch <-- why does it even talk about init-repo?23:34
AuroraBorealisas modifiying the repo malitiously would be bad.23:34
mathrickwithout explaining what it is and why it is there?23:34
mathrickthat's really outside of the scope of a mini tutorial, IMHO, and should be removed23:34
AuroraBorealisfile a bug about it23:35
AuroraBorealisit is probably overkill for said mini tutorial.23:36
PengAuroraBorealis: "bzr check" would certainly find ordinary corruption, but a clever hacker would modify all the hashes, no? That would probably blow up pretty quickly as people with uncompromised copies of the repo interacted with it, but it's not something I have any knowledge about.23:37
AuroraBorealiswell what that git comment said was that all the future commits require that all the previous commits are correct23:38
AuroraBorealisalthough i don't really know how that works, but another example is with bitcoin blocks, each block has the hash to the previous block, creating a 'chain', so you can't modify a block or else the hashes won't match up23:38
PengBut it can't actually verify that the hashes of all previous revisions are *correct* in every operation.23:41
PengCan it?23:41
AuroraBorealiswell true.23:41
AuroraBorealisif the person manages to maliciously modify commit 100 for example23:42
AuroraBorealisand manages to change the hashes for 1-99 then i guess it would be undetectable23:42
AuroraBorealisbut i'm sure something else would break...23:42
AuroraBorealislike a merge would show it as a conflict23:43
AuroraBorealisand thats how bitcoin does it as well, is that its a p2p network, so you would have to get everyone to have the malicious block or else you would realize that its wrong23:44
PengMight be an uncommonly-modified file.23:44
PengLike I said, I'm too tired to think about this.23:44
AuroraBorealisxD23:44
AuroraBorealiswell wake up23:44
PengWorst case, even without any hashing, you can obviously do a diff between a questionable copy and a known-good copy.23:44
AuroraBorealisyeah, even if its uncommonly modified, if anyone had branched the good copy and tried to merge it back in, it would show up as a conflict23:45

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