/srv/irclogs.ubuntu.com/2010/12/08/#bzr.txt

=== Ursinha is now known as Ursinha-afk
achianghello, i'm reading the man page for bzr merge and find it a little confusing. let's say i have r1, r2, and r3. i want to revert r2, obviously keep r1, and am ambivalent about r3 (it can stay, i don't care). do i want bzr revert -r r2, or do i actually want merge . -r r2..r3 ?00:16
spivachiang: "revert" is a command to revert the working copy back to the a checked in version.00:19
spivachiang: to undo a committed change you need merge00:20
achiangspiv: oh hm. sorry, i am using git terminology; i should have studied the bzr man page a little more00:20
spivachiang: specifically, to undo the changes in r200:20
spivachiang: you'd do "bzr merge . -r 2..1"00:20
spivachiang: note the backward revision range, because you want to reverse the change :)00:21
achiangspiv: hm, what happens to r3 in that case?00:21
spivNothing.00:22
spivIt's a cherrypick.00:22
achiangok. i'll try that00:23
spivIt doesn't add a pending merge to the working copy, it is like a more convenient and robust version of generating a diff of that revision and applying it backwards.00:23
achiangok, i just did that, and here is some sample output: http://pastebin.ubuntu.com/540859/00:28
achiangwhy does the merge message tell me that debian/changelog was modified, but bzr status does not?00:28
fullermdYou missed the '.' in the merge command line.  You want to 'merge' from the local branch, not the parent.00:29
achiangugh00:30
fullermd(it MAY be the same rev of course, if both branches are up-to-date identical)00:30
fullermdThe merge message probably means whatever that rev range is made a change to the chanelog, but the end result was the same as what you already have.00:31
achiangoh, ok. i can buy that explanation00:31
achiangthanks spiv and fullermd00:32
achiangbtw, i know you must hate comparisons to git, but this is one operation i find to be much more conceptually straightforward in git. i just did a quick google of 'svn revert' and realize bzr has similar behavior, but perhaps our man page could be improved?00:33
maxbachiang: Educate us, how does this work in git? :-)00:34
achiangspecifically, in the 'bzr merge' page, the line that says, "To merge the changes introduced by 82, without previous changes:"; might be nicer to say, "To drop the changes introduced by 82..."00:34
achiangmaxb: git revert <revspec>00:35
fullermdmaxb: 'git revert X' ~= 'bzr merge -rX..last:X . ; bzr ci'00:35
maxbhmm00:35
achiangmaxb: git reset is the equivalent of bzr revert00:35
fullermdgit reset is sometimes bzr reset, and sometimes uncommit, and sometimes pull --overwrite.00:35
achiangand again, i know the primary use case is probably svn users, so i'm not trying to point fingers or anything...00:35
achiangbut i did try to read the merge man page after someone pointed me at it, and i don't think i would have figured out how to actually drop my change without asking00:36
fullermdachiang: That example line isn't talking about reversing changes, it's talking about a cherrypick merge.00:36
achianghm, perhaps i'm just especially dense. :-/00:36
fullermdIt's about "get the changes from just rev 82 of the other branch", not "reverse rev 82 of this branch"00:37
spivThere have been some small tweaks to the merge help text in trunk.00:37
achiangperhaps an example for "reversing changes" would be helpful then?00:37
fullermdThe 'revert' help actually mentions using merge for reversing stuff.00:38
achiangfullermd: i dunno... i read that help and scratched my head for a while before asking here.00:38
fullermdSecond paragraph of the description.  At least in 2.2 and .dev; I don't have anything earlier right at hand, but I don't think it's all that new.00:39
achiangright, i'm saying i stared at that 2nd paragraph and it didn't make any sense to me.00:39
achiangand it's backwards from what spiv just advised me on00:40
achiangmodulo the fact that i forgot to include "." (my own fault)00:40
fullermdNo, it's the same, it's just using negative revnos rather than positive.00:40
achiangwow.00:41
achiangi had no clue00:41
spivI wonder if that example would be clearer if it didn't use negative revnos (even though saying "undo the 2nd last commit" is probably a pretty common use case).00:42
fullermdI think of it as a good example of the meta- of the online help that makes me grumpy  :p00:43
fullermdIt's sorta quick-ref, except when it's sorta comprehensive-ref, except when it's sorta command-centric-tutorial, except when it's sorta general-tutorial, except when it's sorta buncha-examples, except...00:44
achiangi mean, you could even keep the negative revnos, but just change the wording00:44
achiang"For example, "merge . --revision -2..-3" will remove the changes introduced by second to last commit (-2), without affecting the changes introduced by the last commit (-1)"00:45
achiangor something similar00:45
fullermdWe should use the word 'reverse' there.  'remove' implies the rev is gone, which isn't the case, and 'revert' creates confusion with the revert command.00:45
spivachiang: i.e. just be explicit about the meaning of -2 and -1?  Seems reasonable to me.00:46
fullermdOr we could just add a 'reverse' command.  That would be a nice thin layer on merge...00:46
achiangspiv: yes. i didn't realize those were negative revnos -- i just thought it was some weird convention passed to --revision00:46
spivachiang, fullermd: those tweaks sound good to me.  I'll make them now.00:48
achiangspiv: thanks!00:49
spivfullermd: hmm, actually, I'm ambivalent about the using "reverse"00:49
fullermdI'd half suggest using '-r' instead of '--revision' too.  On the one hand, being explicit and spelt-out is nice, but on the other, who ever actually types --revision?00:49
maxbvila: 2.3b4 now uploaded to bzr-beta-ppa/maverick. I have a feeling it's going to fail the testsuite though, based on local experimentation00:49
spivfullermd: I think it may be as harmful as it is helpful to have yet another term for that added to the text.  Not sure.00:50
fullermdWell, I don't necessarily mean a formal Term, just the choice of verb.  You could call it "constructing the complement", but I'm not sure that would be helpful either  ;)00:51
spivfullermd: sure.  My concern is basically that we should try to use as few different words for the same thing as possible, to avoid confusing new users that don't know all the nuances yet.00:54
fullermdOh, yes.  But I think using 'remove' or 'revert' or the like is worse, because it implies incorrect things.  You're not _removing_ anything, you're creating a new thing on top that's an undo.00:57
fullermdBut I'm half feverish at the moment, so I could be talking in Chinchilla or something for all I know...00:58
achiangthere you go... "to undo..."00:58
spivachiang, fullermd: https://code.launchpad.net/~spiv/bzr/tweak-revert-help/+merge/4303601:06
sqwishyWhen people make pushes to my branch, file permissions seem to get modified under the .bzr directory so that files are owned by whoever made the push. Is that normal or is my configuration wonky?03:36
=== Ursinha-afk is now known as Ursinha
=== Ursinha is now known as Ursinha-afk
jmarsdenI just encountered a bzr crash, I reported it as bug #687266.  Anything else I can usefully do, or can anyone advise how to work around this? Running bzr 2.1.1-1 on Ubuntu 10.04.106:43
ubot5Error: Could not parse data returned by Launchpad: 687266 (https://launchpad.net/bugs/687266)06:43
spivjmarsden: install the bzr-loom plugin06:48
spivThe error ought to be clearer, and say something like "Unrecognised branch format: 'Bazaar-NG Loom branch format 6\n'06:49
spiv" though.06:49
jmarsdenspiv: OK, will do, but why is a plugin needed just to branch something from LP?  Is this a new requirement?06:49
jmarsdenSeems to have worked, thanks.  I'll update the bug report with this workaround.06:51
spivjmarsden: that branch has been created in the loom format06:51
spivSo you'd have to ask the branch owner why they chose to do that.  (Although it is a pretty common plugin, I like it!)06:52
spivIt's not a general requirement for branching from Launchpad.06:52
spivJust for branches in that format :)06:52
jmarsdenAh, OK.  Good :)06:52
spivKind of like how you can't branch from an SVN branch without the bzr-svn plugin ;)06:52
spiv(Although of course LP doesn't host SVN natively)06:53
jmarsdenWell, sure... but crashing because the plugin is not there does not seem like reasonable error handling (or reasonable UI)06:53
spivAgreed, and I just updated the bug to say so :)06:54
jmarsdenI should have refreshed my lp page before adding a comment :)  OK, thanks.06:55
vilagrr why did libssl requires a reboot when upgraded oh why ?07:20
vilamgz: You're reading above my shoulder again... Thanks for fixing the last remaining failures on OSX-10.5 :D07:56
vilamaxb: thanks for updating the beta ppa ! I don't get what you meant by 'fail the testsuite' though ?08:02
=== mthaddon changed the topic of #bzr to: Launchpad down/read-only from 10:00-11:30UTC for a code update | Bazaar version control | try https://answers.launchpad.net/bzr for more help | http://irclogs.ubuntu.copingeling aboutm/ | Patch pilot: vila | 2.3b4 is frozen, installers build time ! (rm vila)
=== rml_ is now known as rml
vilafullermd: ping09:34
vilasheesh, is he *sleeping* ? :D09:35
fullermdNo, just sick.09:36
jelmerg'morning vila, fullermd!09:38
vilajelmer: hey !09:38
vilafullermd: oooh, may be you should... sleep then ? ;)09:38
fullermdI'd love too.  I've tried on and off for the last 18 hours   :|09:39
vilaurgh, bad bad bad09:42
vilabooo for lp going offline ! Yeah for timely notices on my last page refresh !!!09:46
jelmer:)09:48
maxbvila: 2.3b4 is in PPA for maverick I'll do the other series today. it surprised me by passing tests09:55
vilamaxb: great !09:55
maxbbash_completion tests must be being upset by my local environment09:55
vilamaxb: But I still don't get why you're surprised... haaaa09:55
vilamaxb: I almost never see them fail on babune, so yeah, you may want to investigate locally09:56
maxbon the plus side, the ppa is now running all tests, not just --no-plugins09:57
vilamaxb: wow, that's... a very good news :)09:58
vilahttp://babune.ladeuil.net:24842/job/selftest-osx-10.5/ BLUE !10:10
vilamgz: kudos !10:10
=== zyga-afk is now known as zyga
=== r0bby_ is now known as robbyoconnor
mgzyeay, a blue mac. now, why is 10.6 weird...11:25
spivA blue mac?  Aren't those like 15 years old? :P11:26
fullermdThat's why it took so long to get through the tests   :p11:26
vilamgz: so, the doc_generate ones are due to a version too recent (I think)11:26
vilathe bzr_connect_to_bzr_ssh, well, I've stopped complaining about this test :(11:27
mgzthe bp.launchpad ones also look like some kind of installation issue11:27
vilayeah probably, I should look into it and update the osx-installers requirements if that's the case11:28
mgzso, six failures from setup things, and one from ssh hating us.11:28
vilain the later case, it may well be an out of date paramiko (which is weird since we embed a patched one in the osx installer, but may be we don't patch the installed one)11:29
vilawhich in turn would mean *I* patched the 10.5 installed one... I just love untracked admin tweaks...11:30
fullermdYou should get some kinda version control system to keep track of that stuff.11:30
vilagood idea !11:31
vilaThe admins need to be lectured to use it though, which is the hardest part ;)11:31
fullermdSo mass executions it is, then.  I'll warm up my lawnmower.11:32
vilaweird, it seems launchpadlib is installed on 10.6 but not 10.5, so failing to import it on 10.5 masks the lazr.uri import error :)11:34
mgzthat sounds fine, not having it means some things don't run.11:36
mgzbut... launchpadlib should depend on lazr shouldn't it?11:36
vilahrm, no I think the problem is that 10.5 has apython-2.6 installed on top of the system provided 2.5 and the later is used to build installers but the former is used to run the tests11:37
vilathat's *wrong*11:37
fullermdIt's not wrong, it's *agile*!11:37
vilait should use the system-provided one or we're not tested what we should be testing %-)11:38
vilas/tested/testing/11:38
viladamn admins again11:39
vilamgz: launchpadlib shouldn't assume that it has been correctly packaged11:40
mgzhmmp...11:41
vilahmm, I kind of require that babune slaves have bzr installed (if only for bootstrapping), so this issue is naughty: the launchpadlib shipped in the osx installers lacks lazr.uri (all of lazr in fact), so making the tests succeed requires: a valid setup, requiring a valid launchpadlib packaged in the installed bzr...11:47
mgzhm. can you not unselect launchpadlib on install?11:49
mgzsomething to work out with the mac installers at any rate11:49
vilasounds like the fastest workaround, and I should discuss with doxxx why it included an incomplete launchpadlib there (which also requires a bunch of other stuff including simplejson, httplib2, oauth and whatnot)11:50
vilalunch time here, bbl11:51
mgzand I needed to leave ten mins ago.11:52
=== Ursinha-afk is now known as Ursinha
echo-areaI'm not sure this: Does bzrtools add color support for bzr shelve?12:53
echo-areaFor I see colorful diffs after I installed bzrtools yesterday.12:53
vilamgz: even simpler is to make the tests check for the required module: https://code.launchpad.net/~vila/bzr/687315-lazr-uri/+merge/4307913:40
vilamgz: the ssh error on 10.6 is probably the paramiko bug involving ipv6 (it seems there is *no* way to disable v6 on lo0...)14:22
vilamgz: so the alternative is to carry spiv's patch for paramiko in the OSX installers...14:23
=== Ursinha is now known as Ursinha-lunch
mgzI'm not convinced about the launchpadlib dependency thing, it's a deliberate pain to install because of that, if the mac installers work around and break it somehow, I think it's their bug.14:50
mgzHowever, carrying a patched paramiko I thought they did already, and will become increasingly nessersary.14:51
vilamgz: who is 'their' ?14:53
mgzthe mac installer.14:54
vilamgz: yes, paramiko was patched but for the rng warning, not the ipv6 family (bug #579530)14:54
ubot5Launchpad bug 579530 in Bazaar "paramiko does not try all available address families" [Medium,Confirmed] https://launchpad.net/bugs/57953014:54
vilamgz: and the fix would be ?14:54
mgzokay, that one should just be added to the bundle as well.14:54
vilamgz: carrying more dependencies that are useless there ?14:54
mgz^personally, I'd remove launchpadlib, the windows installers don't include it because it's a pain in the bum.14:54
vilamgz: I'm adding the paramiko patch right now14:54
mgzpeople who want to use lp-propose can install seperately.14:55
mgzotherwise, the dependencies need to be correctly bundled.14:55
vilamgz: are you sure it's only for lp-propose ?14:56
mgzand perhaps a couple of other neat little launchpad plugins commands one of the ABs added.14:57
mgzaaron?14:57
mgzyou certainly don't need it to do basic lp interactions.14:58
vilarhaaa doxxx where are you ? :D15:00
vilaand garyvdm... :-/15:00
=== Ursinha-lunch is now known as Ursinha
hazmatis there any way to have bzr pipeline commands execute a hook prior to moving to the prev/next pipe, i'd like to cleanup cached pyc files, as they cause spurious conflicts when stepping through the pipeline16:14
maxbhazmat: erm, why would you have .pyc files under version control? Or do I misunderstand what you mean by conflict?16:25
hazmatmaxb, there not under version control, but if i have them in a directory added in one pipe, and switch to one where the directory doesn't exist, bzr complains that it can remove the directories because their not empty (due to the pyc files), and i have manually resolve the conflict16:26
hazmats/there/their16:26
maxbah, I see16:26
exarkunhazmat: s/their/they're/ first ;)16:27
hazmatexarkun, indeed :-)16:27
exarkunI think I've had a similar problem with looms.  I haven't used pipelines yet.16:27
maxbHmm, so really you need a pre_merge hook16:28
maxbI think the only thing you could do *right now* would be to wrap the switch-pipe command.16:29
maxbA more general solution would be to put a pre_merge hook into bzr core, and then hook that16:30
hazmatmaxb, thanks, i think for now, i'll just hook,  PipelineManager.store_uncommitted to cleanup pyc files, its not elegant but it should solve the problem for me. a pre-merge hook looks like it would nicely as well.16:36
rjekHi.  Is there web-based branch viewer like Loggerhead or bzrweb that won't make me miserable trying to read Python backtraces while trying to work out why they don't work?16:38
Methslaunchpad does well at hiding traces and just telling you it doesn't work when it doesn't work. ;P16:40
vilahazmat: you want to try bzr.transform.orphan_policy=move16:40
vilahazmat: this requires bzr-2.3b216:41
hazmatvila, interesting that would work as well16:42
vilahazmat: .pyc files are the primary target...16:43
achianghello, i have a question regarding bzr bound branches -- i can't actually tell if i want one or not. i'm trying to work with the desktop team's libgtk2.0 branch, so what i've done so far is: bzr branch lp:~desktop-team/gtk/ubuntu ; bzr branch -r <lucid revspec> ubuntu lucid16:53
achiangthat 2nd step created a bound branch for me (which i didn't realize until later)16:53
achiangok, so i go into lucid, make a trivial change to debian/changelog, and then try a debcommit16:53
achiangnow it complains about it being a bound branch16:54
achiangi go to the wiki and read about bound branches16:54
achiangand i can see that i can unbind my lucid branch16:54
achiangbut... honestly, i don't know what the right thing to do here is16:54
achiangthe idea is that i want to backport commits from the upstream 'ubuntu' branch back to this new 'lucid' branch that i created16:55
vilaachiang: the second step didn't create a bound branch, you probably did something else to get it bound16:55
vilaachiang: just 'bzr unbind' it until you need it to be bound16:56
achiangvila: oops, you're right -- i did a bzr checkout, not bzr branch16:56
vilaachiang: *if* you ever need it to be bound16:56
vilaachiang: ha, then 'bzr help reconfigure', I think you want 'bzr reconfigure --tree'16:57
achiangi'm just going to use the bzr branch command16:58
vilaachiang: and 'bzr info' before/after to better understand where you are16:58
vilaoh, if you can restart, then that's better16:58
achiangyes, like i said, i'm just playing around right now and have only made a single trivial change16:58
achiangvila: thanks for the help17:00
vilaachiang: happy to help (TM)17:00
knittlwhy is log on a subdirectory slow as hell?18:55
=== BasicPRO is now known as BasicInWoW
jubei__is there a way to use a specific ssh key that is not ~/.ssh/id_dsa with bazaar ?19:56
bob2yes, configure it in ~/.ssh/config19:58
=== Ursinha is now known as Ursinha-bbl
rjekHi.  Is there web-based branch viewer like Loggerhead or bzrweb that won't make me miserable trying to read Python backtraces while trying to work out why they don't work?21:13
rjekbzrweb works to the extent that it can list the branches correctly, but then I get a meaningless backtrace from it when you try to visit a branch through it.21:14
rjekAnd loggerhead appears to have no documentation for setting it up at all, but you do get a meaningless (to me) backtrace if you try to run the enticingly-named "setup.py"21:14
mgzokay, so bzrlib.osutils.normalizepath is: ancient, undocumented, untested, and cruft-ridden21:49
mgzI'd love to just delete it, but this needs to land on all version 2.0 and up.21:50
al`hi all23:02
al`I seem to be unable to use plugins reliably23:02
al`or, at least, I cannot make work fast-import at all23:03
al`error is: "bzr: ERROR: Unable to import library "fastimport": No module named fastimport"23:03
al`however, ${PYTHONPATH}/bzrlib/plugins definitively contains a directory named "fastimport"23:05
mgzand `bzr plugins` lists it?23:07
spival`: I think the bzr-fastimport plugin relies on a python library called 'fastimport'23:08
spival`: so that may be what it is complaining about.23:08
al`yes23:09
al`modular crap23:09
al`of course, FreeBSD does not package this thing23:10
al`so I'll have to build it locally23:10
maxbI don't think it contains any compiled code, so the build should be trivial23:11
al`still23:11
maxbstill?23:11
al`I should not have to get 15k different module to do what I'm doing23:12
al`that should be all shipped with bazaar, with minimal external dependency23:12
al`hopefully this will be my only contact with bazaar, ie. once this work, I'm going back to git23:12
spivI wonder what al` is doing.23:15
maxbMy guess is installing bzr & bzr-fastimport for a single operation, and getting unjustly annoyed about it23:17
maxberm, wtf, in the topic: "http://irclogs.ubuntu.copingeling aboutm/"23:17
=== maxb changed the topic of #bzr to: Bazaar version control | try https://answers.launchpad.net/bzr for more help | http://irclogs.ubuntu.com/ | Patch pilot: vila | 2.3b4 is frozen, installers build time ! (rm vila)
knittldoes bzr use hardlinks when creating a local clone?23:26
spivknittl: not by default, you can hardlink the files in the tree by passing the --hardlink option though23:49
spivIf you want to save space, you probably want to use a shared repository.  And if you want to save time, a shared repository is probably the best answer there too.  We've found that hardlinking the files in the working tree (which has to check each file to see if it has been modified) tends to be slower than just extracting the files directly out of the repository.23:53
spivShared repository + hardlinked working trees is the maximum space saving though, assuming you want to have multiple working trees on disk simultaneously, rather than one tree that you switch between your different branches.23:55
=== Ursinha-bbl is now known as Ursinha

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