/srv/irclogs.ubuntu.com/2011/05/03/#bzr.txt

pooliehi all00:19
jelmerhi poolie00:30
pooliehi there jelmer, how are you?00:33
jelmerthanks, had a good weekend, and today I finally got the impression I'm getting to the last of these vf-specific tests that need to be moved00:35
pooliegreat00:35
jelmerWith the new extensions in subunit I'm going to set up a ratchet (?) to get the number down further and see the progress00:36
jelmerpoolie: and how are you? are you on the road yet?00:36
pooliein a few hours00:36
pooliepretty good, just doing all the little chores before travelling00:36
spivHi all.01:19
spivNothing like a mysteriously locked up X to start the morning!01:19
lifeless\i/01:19
=== cinerama_ is now known as cinerama
lifelessjelmer: why does the subunit recipe not use the subunit debian packaging branch ?01:45
jelmerlifeless: that's how jml set it up hystorically, I don't think he has access to the debian branch01:46
lifelessok, changing it01:46
jelmer*historically01:47
lifelessjelmer: https://code.launchpad.net/~testing-cabal/+archive/archive/+buildjob/2517596/+files/buildlog.txt.gz is what got me looking at this01:51
lifelessnote the massive conflicts01:51
lifelesswe should perhaps use nest-part in fact01:51
jelmerlifeless: Did you see I pushed an updated branch earlier tonight?01:51
lifelessyes, and it had commits on it01:52
lifelesswhich suggests its wrong :)01:52
jelmerwhat's wrong about a branch with commits, isn't that the whole point ? :-P01:52
lifelessnot for recipes01:52
lifelessthe point of recipes is to use the packaging data as much as possible01:52
lifelessand fix things upstream01:53
jelmerI merged upstream, I didn't make any additional changes to the upstream code01:53
lifelessjelmer: right, but you needed to merge upstream because the packaging data has previously had changes leading to these conflicts01:53
lifelessno?01:54
jelmerlifeless: right01:55
jelmerI'm not sure why we have those extra changes in the packaging branch01:56
lifelesswell, we don't in my packaging branch01:56
jelmerlifeless: you mean the one on alioth?01:56
lifelessjelmer: there is none on alioth01:57
jelmerbzr info apt:subunit01:57
lifelesslp:~lifeless/debian/sid/subunit/sid01:57
lifelesshymm, there is01:58
lifelessbut thats stale01:58
lifelessjelmer: so looking at that02:01
lifelessI'm unhappy with 0.0.6-202:01
lifelessjelmer: I have not had a single good experience with source format 302:01
lifelessjelmer: why did you change it ?02:02
lifelessjelmer: I mean, I'm glad you're doing stuff there but that particular change is disruptive (as is the dh_python2 change)02:04
lifelessjelmer: it must be very late for you02:04
lifelessjelmer: so I'm happy to talk about this your morning02:04
lifelessjelmer: (and I only now remember you actually stepped up and became maintainer, looking at log of debian/control)02:05
lifelessjelmer: I've put lp:~testing-cabal/debian/sid/subunit/daily-builds up as a stable url so we don't go making new branches every release (because one branch builds for all releases we need to figure out how to do compatibility in one place anyhow)02:09
lifelessjelmer: I really have to run to the chemist, sorry if any of the above is weird/nonsequitor/annoying02:15
pooliehi spiv02:56
spivHi poolie03:01
=== Ursinha is now known as Ursinha-afk
=== Ursinha-afk is now known as Ursinha
jammorning all08:12
jamspiv: if you're still around, any luck with the concurrent repacking issue?08:13
spivjam: https://bugs.launchpad.net/bzr/+bug/772935 temporarily queue-jumped08:45
ubot5Ubuntu bug 772935 in Bazaar "ErrorFromSmartServer: Absent factory for StaticTuple" [High,Confirmed]08:45
jamsure08:45
spivBasically with the concurrent packing one I want to refactor the heck out of pack_repo.py to make the logic much more testable08:46
spiv(without having to resort to e.g. trying to run two threads with precise timing)08:47
spivjam: btw, I've created lp:bzr-repodebug, which collects some of the ad hoc scripts/plugins I've made or seen in the past08:59
spivjam: because it's too hard to find them in random people's +junk months later, but not quite worth making projects for each of them09:00
spivIt's currently got check-chk, chk-used-by, fetch-all-records, fix-missing-keys-for-stacking, mirror-revs-into, repo-has-keys and repo-keys commands09:00
bialixhi guys09:06
bialixC:\work\Bazaar\plugins\format1\__init__.py:13: DeprecationWarning: __builtin__.type.set_default_format was deprecated in version 2.4.0.09:06
bialixworkingtree.WorkingTreeFormat.set_default_format(workingtree.WorkingTreeFormat5())09:06
bialixwhat's wrong?09:06
bialixand why it complains about __builtin__?09:07
bialixthat's with bzr.exe 2.4.b109:07
spivThat message does seem a bit wrong09:08
bialixspiv: the line from my plugin is: workingtree.WorkingTreeFormat.set_default_format(workingtree.WorkingTreeFormat5())09:10
bialixso I should use set_default for 2.4, right?09:17
spivbialix: I think so09:17
spivbialix: probably that deprecation needs to use @deprecated_function instead of @deprecated_method, or deprecated_method needs fixing to handle classmethods better, or something.09:18
spivOr maybe move @classmethod after the @deprecated_method line?  Anyway, that message is clearly a bug.09:19
bialixspiv: should I file a bug report?09:22
=== cbz_ is now known as cbz
=== zyga-afk is now known as zyga
bialixguys, why does bzr 2.4b1 print too much info about plugins on traceback?11:35
bialixis there any knob I can modify to get the old behavior?11:35
bialixBug 77627211:36
ubot5Launchpad bug 776272 in Bazaar "bzr 2.4b1: on traceback bzr prints too much info about plugins" [Critical,Confirmed] https://launchpad.net/bugs/77627211:36
maxbbialix: Try 2.4b2?11:59
bialixno windows installer available yet :-(12:00
lifelessjelmer: hi12:14
jelmerlifeless: hi12:20
lifelessjelmer: sorry I went skewiff your last night12:20
jelmerlifeless: sorry for not responding yesterday12:20
lifelessjelmer: no worries, it was v. late for you12:20
lifelessjelmer: so, I'm glad you are maintaining subunit12:20
lifelessjelmer: I'm worried that in the -2 change we can't build it for CAT in the canonical datacentre anymore12:21
lifelessbecause: source format 3, and dh_python212:21
lifelessI don't know what to do about those. For all the packages I'm still maintainer of I'm basically using source format 1 by choice and the oldest-supported python buildchain.12:22
lifelessjelmer: do you have an ideas about how to deal with that?12:24
jelmerlifeless: I think those changes make sense if we're considering just Debian (e.g. bugs have been filed about upgrading to dh_python2)12:24
jelmerlifeless: I can symphatize with whoever is doing the backporting though, happy to go back for the moment if that helps elsewhere12:25
lifelessjelmer: mmmm, perhaps, if we don't care about debian backports12:25
lifelessjelmer: the CAT archive can't build source package 3, so at least for me using source format 1 is better (and it works fine with bzr maintenance too)12:25
lifelessjelmer: I'm not sure what the dh_python2 backports story is; I've assumed its 'backport debhelper and hope', but I may be wrong12:26
jelmerlifeless: dh_python2/source 3 should work fine in squeeze too IIUC12:26
jelmereither way, I don't particularly mind going back for the moment12:29
lifelessjelmer: squeeze has 2.6.6-3, but -6 was the dh_python landing AFAICT12:29
lifelessjelmer: please, I think that would be good12:29
lifelessjelmer: the source format is the key thing12:30
lifelessjelmer: the CAT dak archive doesn't handle format three at all. Full stop.12:30
jelmerlifeless: 2.6.6-3~ is the minimum version according to the wiki12:30
lifelessjelmer: ah, interesting;12:30
lifelessjelmer: talk to lamont if you want to know the limitations of CAT more12:32
jelmerlifeless: I've been there... :)12:32
jelmerI backported a newer bzr-builder to it a while back12:32
lifelessfun :)12:33
lamontit's lucid+security+updates, and then we try to minimize what else we drag in. packages not in lucid are much easier to bring in, after that it's a question of "who else may break if we pick this up?"12:37
maxbUnless the package does something interesting, backporting a dh7 dh_python2 package is probably just a matter of swapping --with python2 for --with python-support12:38
maxb(and reducing builddeps)12:38
maxbdh_python2 doesn't live in debhelper, it lives in the python-defaults source package (a bit perversely)12:42
jdobrienwhy would launchpad display the message: The name 'u1rest' has been blocked by the Launchpad administrators.12:54
jdobrienit seems to meet the criteria on the project page12:54
jelmerjdobrien: might be that the u1 prefix is blocked?12:55
jelmerjdobrien: #launchpad is probably a better place for this question12:55
jdobrienjelmer, oh sorry12:55
jelmerno problem, it's just that you're more likely to get a useful answer there :)12:56
jelmeror I can speculate wildly, if you prefer12:57
jelmerjam: hi13:49
jelmerjam: Do you know in what situation I could see _LazyGroupCompressFactory._manager being set to None?13:50
vanguardis there some way to always use --strict for commit and push?13:50
jelmerI'm getting a confusing error in get_bytes_as()13:50
jelmervanguard: I just have an alias that aliases "commit" to "commit --strict"13:51
jelmerI think there's also an option that can do it for you, but I forget what it is :)13:51
vanguardjelmer: alias in bashrc?13:52
jelmervanguard: in ~/.bazaar/bazaar.conf - see "bzr alias"13:52
vanguardjelmer: thanks!13:54
Dr_AlI've got a few (fairly elementary) questions about writing some new test code.14:50
Dr_AlThis is related to the bug in setting the parent location with switch -b14:51
Dr_AlBug 51370914:51
ubot5Launchpad bug 513709 in Bazaar ""switch -b" sets incorrect parent for heavyweight checkout" [Low,Confirmed] https://launchpad.net/bugs/51370914:51
Dr_AlI'm starting to write a test harness to demonstrate this behaviour, but my experience with bzrlib is a bit limited.14:52
Dr_AlAccording to the Bazaar Testing Guide: "Use the bzrlib library when setting up tests and when evaluating the side-effects of the command. ".  Therefore, I'm trying to use bzrlib to create a treeless repository with a trunk (or whatever) branch inside it (to match the example in the bug report)14:52
Dr_AlAt the moment, I'm writing the test code in test_switch.py (in the blackbox directory) and I've started by using self.make_repository('repo_lw', shared=True), but I can't see how to specify the tree-less behaviour.14:53
Dr_AlAlso, what's a good way to get used to using bzrlib (e.g. finding out how to create a branch in the new repository and how to check a branch out to a working folder elsewhere?14:54
Dr_AlIt's taking me quite a while to get my head round the different types that are rreturned by the various bzrlib functions.14:54
jelmerhi Dr_al15:00
jelmerDr_Al: tree-less behaviour would be done with repo.set_make_working_trees()15:00
jelmerDr_Al: To test that bug you probably need to add an extra blackbox test for "bzr switch -b" - the other ones are in bzrlib/tests/blackbox/test_switch.py15:01
jelmerThe other tests should hopefully give you a bit of inspiration15:01
Dr_Aljelmer: Thanks: set_make_working_trees looks good.15:03
=== maxb changed the topic of #bzr to: Bazaar version control <http://bazaar.canonical.com> | try https://answers.launchpad.net/bzr for more help | http://irclogs.ubuntu.com/ | Patch pilot: vila
Dr_Aljelmer: I'm already working in test_switch.py as I said.  I've managed to create the repository, but every time I want to do something extra, I seem to be spending an inordinate amount of time searching classes, parent-classes, grand-parent classes etc for a function that might do what I want.  In test_switch.py, most of the functions seem to use make_branch_and_tree, which isn't quite what I want15:05
jelmerDr_Al: I guess it always takes a bit of getting used to15:06
Dr_AlI could (assuming I could extract the path of the repo from the CHKInventoryRepository object that make_repository returns) just use run_bzr to do all the bits and pieces that I want to do, but that seems to be against the spirit of the Testing Guide quote that I posted.15:06
jelmerDr_Al: Using run_bzr is fine in the blackbox tests15:07
jelmerDr_Al: that bit of the testing guide applies to the non-blackbox stuff15:07
Dr_Aljelmer: Really?? It's item 3 on the list of points under the section title "Blackbox (UI) tests"!15:07
Dr_AlMost of the other tests in test_switch seem to do all the preparation with bzrlib and then use run_bzr for the one command that they're testing15:08
jelmerhmm, ok.. that's not my experience in other blackbox commands. perhaps I'm not aware of the best practices in this regard..15:09
Dr_AlFair enough!15:09
jelmerjam, vila; ^15:09
vilajelmer: you're right15:10
vilaDr_Al: ley me check the guide 8-)15:10
Dr_AlRegardless of best practice, is there a good way of interrogating bzrlib?  It seems daft that I'm having to go through classes, parent classes etc ad nauseum to determine the way to do some fairly basic stuff like creating a branch in the repository.15:10
Dr_AlI'm sure I'm missing some really obvious bit of documentation...15:11
Dr_Alvila: Thanks!15:11
jelmerDr_Al: I usually look for other examples in the code that do similar things as I need to - if you need to create a bound branch you can grep for tests that use bound branches (they'll have something like that in their name)15:11
jelmerDr_Al: Why do you need to create a shared repository, is that relevant for that bug?15:12
vilaDr_Al: right, it may be a bit ambiguous in the guide, but the general rule is that for blackbox tests you mainly rely on run_bzr15:13
vanguardhow can I populate a tree via FTP?15:13
vilaDr_Al: UI tests are a bit of a special case as *sometimes* it's easier to tweak directly with bzrlib15:13
vanguardI want to upload my website with a simple `bzr push`15:13
Dr_Aljelmer: The short answer is: I'm not sure.  I'm basing the test case on the way that we (the company at which I work) use Bazaar.  This is with a shared treeless repository on a network drive with feature branches inside it.  We then use heavyweight checkouts and 'switch -b' / 'switch' to pick branches.15:13
vilavanguard: try 'bzr upload' instead from the bzr-upload plugin15:13
Dr_Alvila: Okay, fair enough... I'll do it with run_bzr.  It certainly makes my life easier!15:13
vilaDr_Al: once you submit your proposal, the reviewer may gives you better ideas and hints (if needed), but don't make your life harder *before* that ;)15:14
Dr_AlFor verifying whether the test has passed (i.e. whether the parent is set correctly), is it better to parse 'bzr info' or to use bzrlib to (somehow) open the branch and interrogate the parent location.  My instinct says the latter, but then that may end up being quite complicated!15:15
Dr_Alvila: Can't argue with that!15:15
vilabranch.get_parent_location() (from memory)15:15
vilabah, one of those ...15:16
vilabranch.get_parent()15:16
Dr_Aljelmer: (continuing earlier comment: sorry, got distracted)... therefore, I thought it was more appropriate to make the test case match the behaviour we were looking for...15:16
Dr_Alvila: thank yoU!15:16
Dr_Alvila: Now I just need to open the branch (created with run_bzr), but don't worry, I'll do some digging...15:17
vilathat should be something along bzrlib.branch.Branch.open_containing(path)[0]15:18
vanguardvila: that is super handy, thx!15:18
vilaDr_Al: bzrlib.builtins is the source of all good bits when you're searching for high level usages of bzrlib internals15:18
Dr_Alvila: Great, thanks15:20
Dr_AlI'm (fairly obviously) nowhere near ready to propose this for merging (having only written the test harness), but is there any chance someone could have a look at my test harness code (http://bazaar.launchpad.net/~abudden/bzr/switch_parent_513709/revision/5817) and check that I'm writing code in an acceptable style and that the code looks reasonable at first glance?15:47
vilaDr_Al: rough review:15:53
vilainstead of a _test_xxx method, split it into an assert<Result>(relevant params) and probably a test script (see bzrlib.tests.test_script.py)15:54
vilanever use os.chdir, run_bzr accept a work_dir parameter15:55
Dr_Alvila: Thanks... I'll have a look at that.15:56
vilathe split stuff seems... a bit complex, can't you achieve that with a *single* split ?15:57
vilaha, a single class there...15:58
Dr_AlI'd like to, but I couldn't see an easy way.  split only produces two entries rather than a list (as I was expecting).15:58
vilaDr_Al: you may create a dedicated class for your tests so that they could share a setUp method that will create the needed stuff up to... the run_bzr('checkout'...15:59
vilaDr_Al: so that you create two test_ methods in this new class, one for heavy and the other for light16:00
Dr_AlI considered parent.split(os.path.sep) or parent.split('/'), but I wasn't sure which would be correct for Windows16:00
vilaDr_Al: bzrlib.osutils is your friend there16:00
Dr_AlThat was my original plan actually (dedicated class), but it seemed more natural to use test_switch.  I'll look at refactoring it into its own class tomorrow.16:00
Dr_AlAha... hadn't seen bzrlib.osutils.16:00
Dr_AlThanks vila. I've got to go now, but I may be back asking more (hopefully gradually less stupid) questions tomorrow.16:01
vilaDr_Al: in the end you should end up with two *short* tests that will outline your intent and be trivial to grasp16:01
vilathere is no stupid questions...16:01
vilaDr_Al: oh I see, you copy _test_switch_explicit_nick.... shudder16:02
chriscauser__Hi Everyone. I'm just throwing it out there to see what people think. I've written a small script which I'm using to show bzr branch information at the bash prompt. The code is at http://bazaar.launchpad.net/~chy-causer/bzr/bzr-term-info/view/head:/__main__.py . I know that bzr shell offers similar stuff but I wanted to know if anything similar exists and if not, if there was any need for it16:39
chriscauser__Eagle eyed people may notice that this script was originally hosted by Github, and for that I apologize :$16:40
=== Ursinha is now known as Ursinha-afk
vanguardchriscauser__: looks nice, I build something like that using bash, not that elegant ...17:19
=== Ursinha-afk is now known as Ursinha
chriscauser__vanguard: My initial version was in bash! It was a little too slow if I wanted to check to see if the working tree was dirty though which is why I dropped down to Python17:21
vanguardchriscauser__: I just checked whether there is a .bzr directory. And then I just checked whether `bzr status` had any output17:22
chriscauser__Yeah, that's what I started with but doing it in python gave me a (off the top of my head) ~35-40% speed boost17:23
=== Ursinha is now known as Ursinha-lunch
vanguardchriscauser__: checking for the dir is prtty easy, but the status slow it down really bad, especially on a cold start17:23
chriscauser__vanguard: Yeah :(17:24
vanguardchriscauser__: the only cool thing is that I could easily adapt the script to git, hg, svn … it does all the checks and tells me which SCM is used in this folder :)17:26
chriscauser__vanguard: Ah! Well, my bash prompt contains $(get_bzr_info)$(get_hg_info)$(get_git_info) so it works for all three of them. The only thing is that if a directory is versioned using two version control systems, it's shown twice, but that's a really fringe problem :)17:29
vanguardchriscauser__: using two SCM is a fringe itself I'd say.17:29
vanguardchriscauser__: my script would just show the first one that applied, I guess bzr if it was bzr+git17:30
chriscauser__vanguard: Oh well :) Glad you like it and hope it may be of some use to you in the future. I'll be working on showing conflicts next17:31
=== Ursinha-lunch is now known as Ursinha
vilachriscauser__: You want wt.has_changes()18:17
vilayeah, talking to yourself again...18:17
=== CardinalFang_ is now known as CardinalFang
vilajelmer: loom broken... never mind, you already landed a fix ;) Gee, almost 3 weeks ago even :-D18:38
jelmer :)18:39
achiangwhat is the easiest way to get a diff of all the files in a directory compared to an earlier revision? unified diff is fine, but maybe a gui tool would be useful too18:43
vilaachiang: bzr diff -r<revno>18:44
achiangoh, that's it?18:44
vilayeah18:44
vilait's even 'bzr diff' only to look at your uncommitted changes18:44
achiangpretty easy. :)18:44
vilasee 'bzr diff --help' for more18:45
achiangwell, i'm doing code archaeology, so i don't have uncommited changes per se18:45
achiangvila: what is the name of the gui bzr diff tool?18:45
vilagood, archeology is all about not breaking anything ;)18:45
vilabzr qlog from the qbzr plugin ?18:46
achianghm, ok. i think i have that, thanks18:46
vilabah, bzr qdiff I meant18:47
vilabzr qdiff --help for details18:47
achiangvila: thanks18:47
binary_crayonHello, I would like to install Bazaar Explorer from source. How may I install QBzr as dependency? I am using Mac OS X. Thanks18:56
=== medberry is now known as med_out
=== med_out is now known as medberry
=== Ursinha is now known as Ursinha-afk
=== Ursinha-afk is now known as Ursinha
=== Ursinha is now known as Ursinha-afk
=== Ursinha-afk is now known as Ursinha
ckontrosI pulled a branch, made 1 change and tried to push. Got this: http://paste.ubuntu.com/603012 I'm using bzr from the Natty repo.23:35
ckontrosNot I looked up  "rich-root support" and then tried a "bzr upgrade --rich-root-pack ." Said branch was v.7 and I couldnt go to v.6. SO Im stuck.23:37
ckontrosSo Im stuck. ANy ideas?23:38
glyphckontros: "the natty repo"?  Do you mean "in natty" or "from https://launchpad.net/~bzr/+archive/ppa, and I'm using natty"23:48
ckontrosglyph: BZR from official natty repos.23:48
ckontrosNot PPA.23:50
lifelessckontros: 'bzr info -v' and paste the repository  line please23:50
glyphckontros: OK.23:50
ckontroslifeless: http://paste.ubuntu.com/60302123:52
lifelessckontros: you're already using rick roots23:53
lifelessckontros: note that the rick root pack format is much slower than the 2a format23:53
lifelessckontros: ok, the *remote* branch is the one that is using an old format23:54
lifelesslp:~ubuntustudio-dev/ubuntustudio-default-settings/UbuntuStudio is what needs upgrading23:54
ckontroslifeless: Gotcha.23:54
lifelessckontros: don't pass '--rich-root-pack'23:54
lifelessckontros: you may have a number of related branches to upgrade; the launchpad UI has an 'upgrade this branch' button23:55
lifelessckontros: if you do upgrade them all yoiu should also upgrade your local branch23:55
* ckontros looks23:55
lifelessjust run 'bzr upgrade' in it23:55
ckontrosAfter I pull it clean? (and local?)23:56
lifelesssure23:57
lifelesseach branch of this project needs to be upgraded separately23:57
ckontrosSee, I hit this before and actually tried that 1st thing.23:57
ckontrosSee, something's odd. Im a member of ubuntustudio-dev and it says I cant upload to this branch on the page.  https://code.launchpad.net/~ubuntu-branches/ubuntu/oneiric/ubuntustudio-default-settings/oneiric23:58
ckontrosjames-w looks to have done something last.23:59
lifelessthat branch is owned by ~ubuntu-branches - its the official packaging branch for that packagin - you need package upload rights to push to that branch23:59

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