| methods | can i do partial commits ? | 01:40 |
|---|---|---|
| methods | of a file | 01:40 |
| mwhudson | methods: i think there is a plugin, bzr-interactive, that lets you do that | 02:13 |
| * mwhudson afk for a spell | 02:13 | |
| chx | hi. i am trying to find documentation on bzr-git but there is not much. how would you check out http://drupalcode.org/project/drupal.git/log/refs/heads/6.x this branch? | 06:38 |
| chx | nevermind | 06:46 |
| jelmer | 'morning | 09:18 |
| bialix | mgz: are you here? | 09:36 |
| bialix | hi all | 09:36 |
| mgz | morning bialix. | 09:36 |
| * fullermd waves at bialix. | 09:37 | |
| * bialix waves at fullermd :-D | 09:37 | |
| bialix | morning mgz | 09:37 |
| bialix | mgz: can you refresh my memory about standalone bzr.exe: if it don't want to start because of missing msvcrxx.dll then one needs to install redist package from microsoft, right? | 09:38 |
| mgz | yes, though we still need to work out exactly what circumstances that happens in so we can fix it properly | 09:38 |
| mgz | so far, we've had a bug report from someone on win2000, and one from someone on server 200..8? | 09:39 |
| bialix | I have something similar on XP machine | 09:39 |
| mgz | this is unhappy, xp should really work. | 09:39 |
| bialix | what redist package I need? for which VS? 2005 or 2008? | 09:39 |
| bialix | that's not on my own computer< I'm trying to help other man | 09:40 |
| mgz | python 2.6 builds against 2008... I'd need to look up the exact minor version of the dll it needs. | 09:42 |
| mgz | just uninstalled my all-in-one bzr copy annoyingly | 09:42 |
| bialix | on the microsoft site there are simple redist and sp1 | 09:43 |
| bialix | I suspect I need plain one | 09:44 |
| mgz | that's what I remember. | 09:46 |
| mgz | (the normal one, not sp1) | 09:46 |
| bialix_ | ok, I'll try that | 09:47 |
| === bialix_ is now known as bialix | ||
| === Ursinha is now known as Ursinha-afk | ||
| Manikandan1 | bzr: ERROR: Connection error: while sending POST /bazaar/: [Errno 111] Connection refused | 09:59 |
| === bialix_ is now known as bialix | ||
| jelmer | garr, I keep trying to do things only to find out maxb has already done them in the last 15m | 10:02 |
| fullermd | The obvious solution is to sneakily reset his clock... | 10:03 |
| jelmer | I was considering getting up 20 minutes earlier, but I guess that works too :) | 10:03 |
| maxb | :-) | 10:06 |
| maxb | I'm not entirely sure getting up has anything to do with it, considering I'm still in bed | 10:07 |
| vila | LOL | 10:13 |
| * bialix waves ta vila | 10:13 | |
| * vila waves to all :D | 10:14 | |
| jam | hi all | 10:18 |
| jelmer | Moggûh John :) | 10:22 |
| jam | jelmer: huh, it always sounds more like "Morgen", but hey, I'm still trying to figure out the pronounciation of Dutch words. | 10:23 |
| jam | Waalre ==> valgre | 10:23 |
| jam | etc. | 10:23 |
| vila | nexuiz-data still running on jubany... no significant output in logs/nexuiz-data either... weird | 10:23 |
| jam | vila: from how long? | 10:23 |
| vila | since 2011-03-01 20:23 utc | 10:24 |
| jam | anyone have a feeling about how far back I should port 437003 | 10:24 |
| jam | bug 437003 | 10:24 |
| ubot5 | Launchpad bug 437003 in Bazaar "Failure to autopack because of 'missing inventories'" [Critical,Confirmed] https://launchpad.net/bugs/437003 | 10:24 |
| jam | I was thinking 2.2, but I'm open to suggestion | 10:24 |
| vila | spiv's fix was backported to 2.0 no ? | 10:25 |
| jelmer | jam: It varies, this is the Hague dialect | 10:25 |
| === hunger_ is now known as hunger | ||
| vila | jam: is there significant problems if you target 2.0 ? | 10:25 |
| jelmer | it would be nice to get it into 2.0 | 10:25 |
| vila | s/is/are/ | 10:25 |
| jam | vila: my whole world crashes | 10:27 |
| jam | unfortunately | 10:27 |
| jam | but I'll see what I can do :) | 10:27 |
| vila | jam: I think the main issue is that people keep running into it (I think there was yet another report since yesterday), so if we can target 2.0, we are better prepared to deploy it for all users involved (modulo poolie being able to SRU 2.3/2.2/2.1/2.0 ...) | 10:27 |
| vila | jam: huh ? :D | 10:27 |
| jam | I'm not 100% sure how stable the code in this area has been since 2.0, it was quite a while ago... | 10:27 |
| jam | vila: just being silly. I can probably do 2.0 just fine | 10:28 |
| vila | :) | 10:28 |
| jam | since we aren't SRUing back that far by default, it didn't seem important | 10:28 |
| jam | but I can do it | 10:28 |
| vila | I still hope we will be able to SRU 2.0 once we sort out the other ones | 10:28 |
| jam | what still has 2.0 that is under SRU, though? | 10:29 |
| jam | hardy? | 10:29 |
| vila | jam: but if you want to start with 2.2, you could still look at backporting if needed | 10:29 |
| vila | jam: hardy is 1.3, but lucid is 2.1 | 10:30 |
| vila | karmic is 2.0.. so we may never SRU there... | 10:30 |
| maxb | I can't see why we would backport anything further back than 2.1, at this point | 10:35 |
| jam | maxb: because we rock! | 10:35 |
| jam | mostly because that is where the bug started, and we may as well fix it there, rather than backporting | 10:36 |
| maxb | well, if it doesn't incur any extra effort, good | 10:37 |
| fullermd | See, if you'd just gotten jelmer to do it instead, maxb would have had it done 15 minutes ago :p | 10:37 |
| maxb | but, if we don't know of any extant distribution that would make use of it, there may be no point (unless it's essentially free to start fixing in 2.0) | 10:37 |
| jam | maxb: at *this* point, it looks essentially free | 10:38 |
| jam | we can always land and not release | 10:38 |
| jam | or skip landing | 10:38 |
| jam | etc | 10:38 |
| jam | vila: care to discuss the fix. I have 2 options | 10:39 |
| jam | 1) See that we have the inventories in another pack file and stop there | 10:40 |
| jam | 2) Go ahead and pull those inventories, which will also require pulling in the associated chk bytes pages | 10:40 |
| jam | I like the idea of having things all together when reasonable. | 10:40 |
| jam | But it does feel a bit like chasing the rabbit down the hloe | 10:40 |
| jam | hole | 10:40 |
| jam | note that we *don't* pull in the associated texts | 10:41 |
| jam | see line 452 of groupcompress_repo.py | 10:41 |
| jam | # XXX: We don't walk the chk map to determine referenced (file_id, | 10:41 |
| vila | I don't know enough about the details, but I share the feeling that things should stay together | 10:41 |
| vila | jam: if the bug (as it seems) is really confined to autopack (a reproducing test would be nice) then (1) seems to be the way to go no ? | 10:45 |
| vila | jam: or does it also occur in weird stacked scenarios ? | 10:45 |
| jam | vila: it is the minimally invasive form, yes | 10:45 |
| jam | vila: it seems to be *caused* by stacking | 10:45 |
| jam | but not in the "weird stacking scenarios" sense. | 10:46 |
| vila | I ~see | 10:46 |
| jam | In normal fetches, we always send the inventory with the revision, so there would be no reason for them not to be in the same pack file | 10:46 |
| jam | but with stacking, we may send inventories for revisions we don't send | 10:46 |
| jam | also, the suspend/resume features end up creating multiple pack files for one logical pack | 10:46 |
| jam | sorry logical fetch | 10:46 |
| vila | is it a case where some constraints should be relaxed while the logical fetch is processed or do we end up we several pack files ? | 10:48 |
| jam | vila: we end up with several pack files. | 10:48 |
| jam | suspend writes it to disk | 10:48 |
| jam | in the uploads dir | 10:49 |
| jam | resume marks it for final inclusion in the 'packs' directory | 10:49 |
| vila | and they get repacked before inclusion in the 'packs' directory as one pack file ? | 10:49 |
| jam | vila: no | 10:49 |
| jam | they just get renamed into place | 10:49 |
| jam | often the first push is huge | 10:49 |
| jam | and the second is small | 10:49 |
| vila | hmm | 10:54 |
| vila | jam: I don't quite get how we end up with missing inventories when we "may send inventories for revisions we don't send"... | 10:55 |
| vila | and what do you call revisions in this context | 10:56 |
| === vila is now known as vila-afk-lunch | ||
| jam | vila-afk-lunch: still there? | 10:57 |
| jam | ping me when you get back | 10:57 |
| vila-afk-lunch | jam: I will | 10:57 |
| jdobrien | is there a way to change the pull location of a branch you have pushed to LP so bzr pull will just get it from LP without having to specify the location? | 11:00 |
| jam | jdobrien: bzr pull --remember ? | 11:01 |
| jdobrien | jam? how will that changed the pull location | 11:01 |
| jam | jdobrien: bzr pull --remember lp:the-new-place | 11:01 |
| jdobrien | ahh | 11:01 |
| jdobrien | jam thanks! | 11:02 |
| jdobrien | jam I was looking everywhere but there :) | 11:04 |
| jam | jdobrien: happy to help | 11:04 |
| jelmer | would "bzr config push_location lp:the-new-place" also work nowadays? | 11:06 |
| jam | jelmer: I think you need at least an "=" in there | 11:07 |
| jam | it also would end up using 'lp:' each time, and not the expanded url | 11:07 |
| jam | not sure what else | 11:07 |
| jam | would differ | 11:07 |
| jelmer | jam: ah, right | 11:09 |
| jam | so yes, *but* :) | 11:09 |
| jelmer | id=4, tests=34971, failures=2363, skips=3794 | 11:39 |
| jelmer | getting close to that 35k mark :) | 11:39 |
| jelmer | http://samba.org/~jelmer/bzr-alltests.html | 11:46 |
| jam | jelmer: that is running against all plugins, or what? | 11:49 |
| jelmer | jam: Against a lot of the plugins: bisect builddeb builder bzrtools cia cvs dbus email explorer fastimport git grep gtk hg keywords loggerhead loom mtn pipeline pqm qbzr search service svn upload webdav xmloutput | 11:49 |
| jam | jelmer: mtn? I didn't think the models would be close enough to actually be supported. | 11:50 |
| jelmer | jam: the mtn module is as trivial as the bzr-cvs plugin | 11:51 |
| jelmer | ie it just installs a Prober and tells you "please try mtn fast-export and bzr fast-import" if you try to open | 11:51 |
| jam | interesting | 11:52 |
| bob2 | hah | 11:52 |
| === vila-afk-lunch is now known as vila | ||
| vila | jam: back | 12:01 |
| jam | (11:55:42 AM) vila: jam: I don't quite get how we end up with missing inventories when we "may send inventories for revisions we don't send"... | 12:02 |
| jam | we'll start from there | 12:02 |
| jam | Example, Revisions A, B, C, D | 12:02 |
| jam | A B C are in the base repository | 12:02 |
| jam | D is in the target repository | 12:02 |
| jam | "stacked" | 12:03 |
| jam | if D is in the stacked repo, then so is the inventory for C | 12:03 |
| jam | Say we push this somewhere else that only has A B | 12:03 |
| vila | jam: because it's the parent ? | 12:03 |
| jam | vila: right | 12:03 |
| jam | A => B => C => D | 12:03 |
| jam | If we push to somewhere that has A B | 12:04 |
| jam | then we will send D rev+inv, C inv. Then suspend, then resume and send C rev | 12:04 |
| jam | At which point D + C's inventory are in one pack file, but we may do an autopack just involving C's rev pack | 12:04 |
| jam | vila: https://code.launchpad.net/~jameinel/bzr/repack-missing-inventories-437003/+merge/52544 | 12:04 |
| jam | If you want a semi-hackish way of triggering the condition | 12:05 |
| jam | also, vila, did we decide to copy NEWs entries when doing stable updates to multiple releases? | 12:05 |
| jam | I'm always unsure about the state of that | 12:06 |
| vila | we stopped duplicating NEWS entries | 12:06 |
| jam | since we may or may not do releases in sync anymore, I was thinking to copy the entries | 12:06 |
| jam | Notably, 2.0.7 may never see the light of day | 12:06 |
| vila | the RM should ensure that any release includes fixes from lower series | 12:06 |
| jam | I *didn't* like it when I was releasing 3 branches simultaneously every month | 12:07 |
| vila | that is, the fixes known when releasing | 12:07 |
| jam | but I'm pretty sure we've backed off on that | 12:07 |
| vila | yup | 12:07 |
| vila | what I do (as RM these days) is ensuring that lower series have been properly merged up | 12:07 |
| vila | there is one merge that is a bit trickier (can't remember if it's 2.1 or 2.2) where you had to make sure you don't include irrelevant news entries | 12:08 |
| vila | jam: "The test also isn't a permutation test. It might apply vs most Pack based repositories, but I can't guarantee it." | 12:17 |
| vila | jam: how hard would it be to dig a bit more there ? | 12:18 |
| jam | vila: harder than I feel is warranted | 12:18 |
| jam | vila: especially for a backported fix | 12:18 |
| vila | jam: which means 0.92 can still exhibit the bug ? | 12:18 |
| jam | I could possibly be convinced otherwise for newer releases | 12:19 |
| jam | vila: you mean format 0.92? Uses very different logic | 12:19 |
| vila | Oh ! Right | 12:19 |
| jam | doesnt try to match Inventories to Revisions | 12:19 |
| vila | I'd be fine with a permutation tests for 2.4 only | 12:19 |
| vila | hmm | 12:19 |
| vila | so you mean only 2a is really affected then ? | 12:20 |
| jam | vila: by that specific bug? yes | 12:20 |
| vila | jam: great | 12:20 |
| jam | at least, I think so | 12:20 |
| vila | already approved | 12:20 |
| jam | its the only one that really matters, anyway :) | 12:20 |
| jam | the bigger question is what happens when 3a comes out | 12:21 |
| jam | but since that is indefinitely postponed... | 12:21 |
| vila | which means we need some way to express that a test apply to a set of formats | 12:21 |
| vila | jam: if you could put these tests in a dedicated class with a comment explaining the issue that would be awesome to start with | 12:22 |
| vila | well, they already are :) | 12:22 |
| jam | vila: any other comments you want to take back? | 12:22 |
| vila | :) | 12:22 |
| jam | self.fail() seems sufficient to say "this should never happen" | 12:22 |
| vila | yeah, could be | 12:23 |
| vila | yeah, got a bit scared by the size of the test at first | 12:23 |
| jam | understandably | 12:24 |
| jam | I tried to make sure the bulk of things were in "make" helpers | 12:24 |
| jam | so you could clearly see it was complex because of setup | 12:24 |
| jam | not because of assertions | 12:24 |
| vila | I think for such bugs you can't avoid having a complicated setup | 12:24 |
| vila | yup | 12:24 |
| jam | and I made sure to re-use those helpers for some extra testing | 12:24 |
| jam | like real-missing inventories, etc | 12:24 |
| vila | yup, may be my "nice tdd there" didn't express enough congratulations :) | 12:25 |
| vila | but it's a nice piece you put there | 12:25 |
| jam | well, some of it was TAD, but I did make sure to trigger the failure before fixing it. | 12:26 |
| vila | A == After ? | 12:26 |
| vila | D == before then :D | 12:26 |
| jam | Test After Development. I did the first test TDD, I did the "but make sure it still fails when it must" afterwards. | 12:33 |
| jam | especially for bugs, I tend to do close to TDD | 12:34 |
| jam | since I have something I want to reproduce | 12:34 |
| jam | for dev work, it varies, because I don't always know what I'm looking for right away | 12:34 |
| vila | yeah, the usual argument ;) | 12:34 |
| jam | and I've had a lot of trouble trying to work out performance related fixes and testing | 12:34 |
| vila | I don't really buy it because I successfully TDDed on several exploratory works, it means more test refactoring but also gives me more confidence about test coverage | 12:35 |
| vila | jelmer: reagrding bug #713240 | 13:17 |
| ubot5 | Bug 713240 on http://launchpad.net/bugs/713240 is private | 13:17 |
| vila | O_o | 13:18 |
| jelmer | I can't access that page | 13:18 |
| jelmer | ah, neither can ubot5 :) | 13:18 |
| vila | jelmer: regarding bug #731240 | 13:18 |
| ubot5 | Launchpad bug 731240 in Bazaar Hg Plugin "opening over http in dumb mode fails in selftest" [Medium,Triaged] https://launchpad.net/bugs/731240 | 13:18 |
| vila | that's better (first typo after more than a work day and a half... something weird is going on ;) | 13:19 |
| vila | jelmer: our HTTP test server supports range requests, not sure what is going on here | 13:19 |
| jelmer | hmm, that's odd indeed | 13:22 |
| vila | jelmer: and which test is failing ? I can't reproduce locally with bzr-hg trunk and -s bt.per_branch.test_branch | 13:24 |
| vila | jelmer: ha, lazy loading and --start-with interfering ? | 13:25 |
| jelmer | I guess so | 13:26 |
| jelmer | It should be bzrlib.tests.per_branch.test_branch.TestBranch.test_open_containing | 13:26 |
| * vila scratches head, even 'bzr selftest bzrlib.tests.per_branch.test_branch' (aka without -s) | 13:27 | |
| vila | hmm, *really* installing the plugin may help | 13:28 |
| vila | indeed | 13:28 |
| vila | you mean bzrlib.tests.per_branch.test_branch.ChrootedTests.test_open_containing ? | 13:32 |
| jelmer | Ah, yes | 13:34 |
| jelmer | wtf, BzrBranchFormat4 can be used in a metadir? | 14:08 |
| vila | jelmer: bug #731240 is a bug in the bzr HTTP test server ;) | 14:31 |
| ubot5 | Launchpad bug 731240 in Bazaar "opening over http in dumb mode fails in selftest" [Low,In progress] https://launchpad.net/bugs/731240 | 14:31 |
| jelmer | vila: ooh, nice catch :) | 14:32 |
| vila | painful to track down and quite surprising, but as I commented in the bug report, it's still valid | 14:33 |
| vila | well, painful... because I didn't know the bzr-hg and hg internals to be honest | 14:37 |
| jelmer | ah | 14:38 |
| jelmer | jam: do you know if it's intentional that BranchFormat4 also works in meta dirs? | 14:38 |
| jelmer | well, s/works/can be initialized/ | 14:40 |
| === Chex_ is now known as Chex | ||
| vila | jelmer: hehe, look at this (http_server.py line 149): # FIXME: RFC2616 says end is optional and default to file_size | 14:54 |
| vila | *He* knew it ! | 14:55 |
| jam | jelmer: there was a meta-dir version before knits, IIRC | 14:58 |
| jam | so yes, it was intentional | 14:58 |
| jam | I'm pretty sure there was a *very* short lived Weave metadir | 14:58 |
| jelmer | jam: it doesn't have a get_format_string() implementation though | 15:00 |
| jelmer | jam: so it's write-only | 15:00 |
| jam | odd | 15:00 |
| jam | That would be repository Weave anyway | 15:01 |
| jam | the Branch wouldn't matter | 15:01 |
| jam | jelmer: if it is write-only, then there is no reason to support it | 15:01 |
| jam | since it couldn't be read | 15:01 |
| jelmer | ok | 15:03 |
| jelmer | a couple of tests seem to rely on the existing behaviour, but they don't reopen branches after they've been created | 15:03 |
| jam | jelmer: then my guess is that it is an accidental by product of permutations and isn't meant to work | 15:16 |
| jelmer | jam: you mentioned yesterday you were happy to see https://code.launchpad.net/~jelmer/bzr/blackbox-upgrade-formats/+merge/52275 land ? | 15:41 |
| jam | jelmer: yes | 15:42 |
| jam | must have been one of the rejections I didn't catch | 15:42 |
| jam | marked approved | 15:42 |
| jelmer | jam: thanks ! | 15:43 |
| vila | jelmer: NameError: global name 'RemoteHgBranchFormat' is not defined | 15:47 |
| vila | jelmer: can you stop some typo there ? | 15:47 |
| vila | jelmer: or tell me where it's defined ? | 15:47 |
| jelmer | vila: fixed | 15:48 |
| vila | jelmer: in trunk ? | 15:48 |
| vila | indeed | 15:49 |
| vila | then I've got a fix for #713420 | 15:49 |
| vila | pffft | 15:49 |
| vila | bug #731240 (double tyop above, triple if you count the missing 'bug' ;) | 15:50 |
| ubot5 | Launchpad bug 731240 in Bazaar "opening over http in dumb mode fails in selftest" [Low,In progress] https://launchpad.net/bugs/731240 | 15:50 |
| jelmer | vila: hmm? | 16:00 |
| jelmer | ah | 16:00 |
| * jelmer finishes reading through ~2k failing tests | 16:06 | |
| vila | jelmer: here is the mp: https://code.launchpad.net/~vila/bzr/731240-range-parsing/+merge/52573 | 16:06 |
| jelmer | I've filed bugs for all the different issues I've encountered, most failing ones are due to missing VersionedFiles.add_lines() implementations though | 16:07 |
| vila | jelmer: hehe, that's far too much, better fix the most important one first :) | 16:07 |
| jpds | /go #pse | 16:07 |
| jelmer | vila: \o/ I'll have a look | 16:07 |
| vila | jelmer: what ? you need to read them all to find the most important one ? Where is your crystal ball ? | 16:07 |
| jelmer | vila: I wish the test suite could tell us ;-) | 16:08 |
| jelmer | 5 FAILURES (1 severe, 2 important, 2 minor) | 16:09 |
| vila | hmm, good idea ! | 16:09 |
| vila | jelmer: here is the mp for that: https://dreamcode.launchpad.net/~vila/bzr/731240-range-parsing/+merge/52573 | 16:10 |
| vila | err | 16:10 |
| vila | jelmer: here is the mp for that: https://dreamcode.launchpad.net/~vila/bzr/1-dwim/+merge/42 | 16:10 |
| jelmer | hehe | 16:10 |
| jelmer | vila: is "bytes=-" allowed? | 16:14 |
| vila | no, let me test that :) | 16:14 |
| tacone | may any good soul remind me how to create a prefix like lp: ? | 16:14 |
| jelmer | vila: there are two spaces in this line: + self.req_handler = RequestHandler(None, None, None) | 16:14 |
| jelmer | tacone: It's a DirectoryService, you can register a custom one in a bzr plugin | 16:15 |
| vila | jelmer: blame my shaking hand ;) | 16:15 |
| tacone | urgh. do i have to distribute a plugin to all my devs ? :) | 16:16 |
| vila | tacone: better than asking them to *write* one ;) | 16:16 |
| tacone | villa: +1 | 16:16 |
| * tacone mispells nicknames | 16:17 | |
| vila | np | 16:17 |
| tacone | any other way to shorten a sftp long url ? | 16:17 |
| jelmer | tacone: Well, you'll have to get *something* to them | 16:17 |
| vila | tacone: can you elaborate a bit about what kind of aliasing you're after ? | 16:17 |
| jelmer | perhaps bzr-bookmarks can be used for this sort of thing? | 16:17 |
| tacone | maybe, i'm looking it up right now, is it in the repos ? | 16:18 |
| vila | the subject comes back regularly enough that we may investigate how to generalize *something* to the point where you could use it via some configuration | 16:18 |
| vila | tacone: yes... lp:bzr-bookmarks ;) | 16:19 |
| jelmer | tacone: There aren't any packages, if that's what you mean | 16:19 |
| tacone | ok | 16:21 |
| jelmer | who is ADHB? | 16:23 |
| vila | jelmer: abentley | 16:30 |
| === zyga is now known as zyga-afk | ||
| vila | jelmer: what is easier in the hg config API ? | 16:30 |
| abentley | jelmer: Hi. Aaron David Heuer Bentley here. | 16:30 |
| jelmer | abentley: Ah, it's you! | 16:31 |
| vila | Heuer and Bentley, talk about a luxury name ;D | 16:31 |
| jelmer | vila: the API doesn't confuse me in Hg :) | 16:32 |
| abentley | vila: :-) | 16:32 |
| vila | jelmer: haha, ok, but that doesn't tell me much ;) | 16:32 |
| vila | jelmer: as in : I'd like our new config API to be easy for users and devs alike, so all inputs are welcome and... now is a good time ;) | 16:35 |
| tacone | is still Olive the supported GUI ? | 16:39 |
| vila | well, it's hard to say that there is *a* supported GUI | 16:41 |
| jelmer | tacone: qbzr/bzr-explorer are probably a better choice at the moment | 16:42 |
| vila | Olive has been spun off from bzr-gtk, bzr-explorer was intended to support both gtk and qt but has been adopted by the qbzr devs. There seem to be more interest in qbzr than bzr-gtk but both projects are still active, Olive... is a different matter and could certainly benefit from some love... | 16:42 |
| tacone | i'm taking a look to bzr-explorer, thanks | 16:43 |
| vila | jelmer: is https://code.launchpad.net/~jelmer/bzr/weave-plugin/+merge/51301 still reviewable ? (You mentioned BranchFormat4 a bit earlier) | 16:59 |
| jelmer | vila: sortof.. I'm splitting it up further to make it more reviewable | 16:59 |
| vila | jelmer: ok | 16:59 |
| jelmer | vila: review of https://code.launchpad.net/~jelmer/bzr/branch4-no-metadir/+merge/52575 would be great though :) | 17:00 |
| vila | jelmer: I'm on https://code.launchpad.net/~jelmer/bzr/per-repository-vf-reconcile/+merge/52283 | 17:00 |
| jelmer | vila: ah, cool | 17:03 |
| jelmer | vila: I'm not sure I understand your comment on per-repository-vf - the scenarios should be applicable for all files in this directory | 17:04 |
| vila | jelmer: when poolie introduced the .scenarios attribute, the idea was to reduce the distance between the test classes and the parametrization | 17:05 |
| vila | jelmer: in your case, ISTM, the distance is too big | 17:06 |
| vila | jelmer: if the scenarios needs to be shared, I don't have a problem leaving them in __init__, but they can still be referenced from the scenarios attribute | 17:06 |
| vila | jelmer: if they are *not* shared, then I would prefer for them to *not* be in __init__ (for the other submission) | 17:07 |
| vila | jelmer: and when I reviewed the first mp, there was only 1 file in the dir ;) | 17:08 |
| jelmer | vila: What do you refer to by scenarios attribute here specifically? | 17:08 |
| * jelmer suspects there's a concept he's missing here | 17:08 | |
| vila | jelmer: see test_http.py for examples | 17:08 |
| vila | jelmer: yeah, it's possible you didn't see poolie's work there, it still has to be fully deployed | 17:08 |
| jelmer | I inspired my existing code by bzrlib.tests.per_repository | 17:09 |
| vila | jelmer: grep for load_tests_apply_scenarios | 17:09 |
| vila | jelmer: roughly: the idea is to define an attribute on the test class instead of defining a specific load_tests function | 17:10 |
| vila | jelmer: the implementation proposed by poolie has been working pretty well so far | 17:11 |
| jelmer | there will be a lot of scenarios in this case though, in multiple files | 17:11 |
| jelmer | vila: Do you mean just setting "scenarios = all_repository_vf_format_scenarios()" on each of these classes? | 17:14 |
| === deryck is now known as deryck[lunch] | ||
| vila | jelmer: right, I haven't into the details and maybe your proposal is good enough to land, but if you could have a look at this new organization I've got the feeling that the parmetrization could be clearer | 17:14 |
| === charlieS_ is now known as charlieS | ||
| vila | jelmer: may be, may be not, a common base class could work too, I don't remember if you could easily redefine (and enrich) for a daughter class | 17:15 |
| vila | jelmer: as said above, there was only one file for the first mp hence my comment | 17:16 |
| vila | jelmer: for the second mp, if the scenarios are not reused elsewhere then put them in the file that use them | 17:16 |
| jelmer | vila: fair enough | 17:17 |
| vila | jelmer: mainly the scenarios attribute helps putting the scenarios closer to the tests that use them which is the main point of my comments | 17:17 |
| smthomas | does anyone know if it is possible to allow one version controlled branch inside another version controlled branch? I have a folder structure like /modules and /modules/abc in which I would like to keep the abc a separate branch but also commit the entire modules directory for easier deployment on my we server. When I initialize a branch in the modules directory it ignores the other version controlled folder. | 17:20 |
| smthomas | web server* | 17:20 |
| vila | smthomas: you want nested trees which is not implemented yet | 17:21 |
| vila | smthomas: the container branch will ignore the contained branch for now | 17:21 |
| smthomas | vila: ok thanks, any idea when nested trees will be implemented (for future reference)? | 17:22 |
| vila | smthomas: no estimate for now as the bzr devs focus on other features but it's an highly desired feature nevertheless... | 17:24 |
| smthomas | vila: yes I agree with that as I would like to be able to use it, thanks again for the help | 17:24 |
| vila | smthomas: you know about the bzr-upload and the bzr-push-and-update plugins right ? | 17:25 |
| vila | smthomas: the above are for deployment | 17:26 |
| vila | smthomas: but there are also other plugins that try to address the nested trees feature (and I forgot their name, once *again* !) | 17:26 |
| vila | bialix will kill me :-/ | 17:26 |
| smthomas | vila: no I do not, I will have to do some research. I am fairly new to bzr. I have been using it for awhile but not much for deployment yet | 17:27 |
| vila | bzr-externals ? | 17:27 |
| vila | https://launchpad.net/bzr-externals yeah, sounds like it | 17:28 |
| vila | smthomas: you may give it a try too | 17:28 |
| jelmer | vila: what about something like this: http://bazaar.launchpad.net/~jelmer/bzr/per-repository-vf/revision/5688 | 17:30 |
| vila | jelmer: exactly ! | 17:31 |
| vila | err | 17:31 |
| vila | except for the first line in the patch, do you really need to do that ? | 17:31 |
| vila | in tests/__init__.py | 17:32 |
| jelmer | I don't think we would discover that test file otherwise | 17:32 |
| * jelmer tries | 17:32 | |
| jelmer | yeah, without that line we don't run the tests in bzrlib.tests.per_repository_vf.test_repository. Previously the loader.loadTestsFromModuleNames call took care of that | 17:33 |
| vila | then may be just a boilerplate load_tests in __init__ | 17:34 |
| vila | I realize the per_xxx tests are not *that* regular either :-/ | 17:34 |
| vila | oooooh noooo, bazaar-commits@lists.canonical.com thinks I'm spamming ??? | 17:38 |
| jelmer | vila: what about http://bazaar.launchpad.net/~jelmer/bzr/per-repository-vf/revision/5689 ? | 17:38 |
| === r0bby is now known as robbyoconnor | ||
| vila | argh, no, it's even worse ! That's my own ISP !!! | 17:39 |
| jelmer | vila: bazaar-commits@ still exists ? :) | 17:39 |
| vila | jelmer: indeed ! | 17:39 |
| chx | another bzr-git question, how do you update your import of a git repo? | 17:41 |
| jelmer | chx: running bzr git-import again | 17:46 |
| chx | ah! | 17:46 |
| chx | tricky. | 17:46 |
| chx | and bzr dpush in that directory, right? | 17:47 |
| chx | where the shared repo is | 17:47 |
| chx | and finally, how you create a new git branch w/ bzr-git? | 17:48 |
| jelmer | chx: "bzr init" will create a new git branch | 17:50 |
| jelmer | chx: bzr dpush in which directory? | 17:50 |
| chx | jelmer: do you do bzr init in refs/heads? | 17:50 |
| chx | jelmer: i have a directory (shared repo) resulting from bzr-import | 17:50 |
| jelmer | chx: no, just in the root of the repository | 17:50 |
| jelmer | chx: That's a bzr repository, creating something there won't help you get it into git | 17:51 |
| jelmer | chx: What are you trying to do exactly? | 17:52 |
| chx | jelmer: many things :) i try to use bzr-git in place of git... :) | 17:52 |
| === seiflotfy_ is now known as seiflotfy | ||
| chx | jelmer: Drupal moved to git from cvs and i tried git and i am ... well. i'd use bzr-git instead. just there isnt much docs. | 17:52 |
| LeoNerd | Last time I tried bzr-git it didn't have the ability to operate on the bizarre multiple-branches-in-one-place model that git uses | 17:53 |
| jelmer | chx: Contributions in that area would be welcome :) I guess at the moment there are mainly caveats, otherwise it should work like bzr itself | 17:54 |
| jelmer | LeoNerd: it has that capability nowadays, but bzr's UI doesn't actually provide you with the ability to use that yet | 17:54 |
| chx | jelmer: ok i will ask questions and write a blogpost for docs. Deal? | 17:54 |
| jelmer | chx: WFM :) | 17:54 |
| LeoNerd | Ah OK | 17:54 |
| vila | maxb: bananas... | 18:00 |
| maxb | I like bananas. Bananas are good. | 18:01 |
| vila | yeah and testables ! | 18:01 |
| chx | jelmer: http://ex.privatepaste.com/bd99cc1e46 | 18:01 |
| jelmer | chx: the git:// protocol is anonymous, so using that to push will probably not work | 18:02 |
| jelmer | you probably want to push to a git+ssh:// URL, or a local Git repository | 18:02 |
| vila | jelmer: fixreleased+milestone, not fixcommitted, kthksbye :-p | 18:07 |
| jelmer | garrr | 18:12 |
| jelmer | anyway, time to move.. | 18:12 |
| jelmer | chx: I'll be back online in ~30m | 18:12 |
| === deryck[lunch] is now known as deryck | ||
| wart___ | hi folks. i did a bzr branch lp:widelands and a network fail caused me to ctl-c. now when i try bzr branch lp:widelands it won't resume, but says that the directory widelands exists. | 18:33 |
| wart___ | is there a way to resume? | 18:33 |
| wart___ | i tried a couple of things icould think of e.g. cd widelands; bzr pull | 18:33 |
| wart___ | no luck | 18:34 |
| wart___ | it'd stink to rm -rf and start over since i got 200M of the 400M | 18:34 |
| vila | wart___: AFAIK, it stinks | 18:35 |
| maxb | wart___: Hi. Could you pastebin the result of running "find .bzr -ls", to help understand the status of the partial branch? | 18:35 |
| vila | wart___: you should file a bug, I don't think we have one asking for pull to be resumable | 18:36 |
| vila | wart___: alternatively, if you network connection often fail, you could pull by chunks by specifying increasing revnos | 18:37 |
| vila | s/you connection/your connection/ | 18:37 |
| wart___ | maxb: http://pastebin.com/fN1kbTiT | 18:42 |
| wart___ | well this is about the 4th time its done it to me so how do i specify increasing revnos? | 18:42 |
| vila | wart___: bzr branch -r<revno> ; bzr pull -r<revno> | 18:44 |
| maxb | wart___: Ah. As you can probably infer from the lack of large files there, nothing has been stored. | 19:00 |
| jhunt | Hi all - I've made a tweak to lp:~ubuntu-desktop/gdm/ubuntu. I want to push my branch to lp:~jamesodhunt/... somewhere but can't find a path bzr is happy with (either complains about distro series, or just EPERM)...? | 19:54 |
| chx | jelmer: hi | 19:55 |
| === ajmitch_ is now known as ajmitch | ||
| chx | jelmer: that crashed | 19:55 |
| chx | jelmer: http://paste.pocoo.org/show/350399/ | 19:56 |
| jelmer | hi chx | 19:57 |
| chx | jelmer: bzr import git://git.drupal.org/sandbox/chx/1085438.git ; cd 1085438.git/ ; bzr init new-branch ; cd new-branch ; bzr dpush git+ssh://git.drupal.org/sandbox/chx/1085438.git | 19:57 |
| jelmer | chx: Hmm, the repository does not have a HEAD branch? | 19:57 |
| chx | jelmer: the repo is empty | 19:57 |
| jelmer | chx: The repository will need an empty branch first | 19:57 |
| chx | jelmer: so it needs to be git cloned first :( ? | 19:58 |
| jelmer | chx: That's one of the limitations of dpush at the moment | 19:58 |
| jelmer | chx: *Something*, an empty branch is fine first | 19:58 |
| jelmer | s/first/too | 19:58 |
| chx | jelmer: so --- i need to use git first? | 20:00 |
| jelmer | chx: for the moment, yep | 20:00 |
| jelmer | chx: Was I talking with you about bzr-git on answers.lp.net earlier? | 20:01 |
| chx | jelmer: no | 20:03 |
| jelmer | ah, ok | 20:03 |
| jelmer | Let me find the bug # again then | 20:03 |
| jelmer | chx: it's bug 731336 | 20:03 |
| ubot5 | Launchpad bug 731336 in Bazaar "dpush doesn't create new branches" [Medium,Confirmed] https://launchpad.net/bugs/731336 | 20:03 |
| chx | jelmer: http://paste.pocoo.org/show/350426/ | 20:08 |
| chx | jelmer: wait, it does not create new branches at all? ah! | 20:08 |
| chx | jelmer: i see, i see | 20:09 |
| chx | jelmer: so for new branches we need git :/ | 20:09 |
| jelmer | chx: you need to use git on the remote side to create the repository anyway | 20:11 |
| chx | heh that's automated :) | 20:12 |
| jelmer | ah, ok | 20:12 |
| jelmer | wb chx | 20:43 |
| chx | jelmer: so we cant really expect that any time soon 731336 gets solved :( ? | 20:43 |
| jelmer | chx? | 20:45 |
| jelmer | it's on the list of things to fix, though it should be trivial to work around at the moment | 20:45 |
| chx | is it/ | 20:47 |
| chx | how so? | 20:47 |
| jelmer | chx: the first time you work with a git repository, push to a local repository first and then push that using git to the remote server | 20:49 |
| jelmer | chx: so if you're in the bzr branch you'd like to dpush, run something like: | 20:49 |
| jelmer | git init /tmp/tmprepo; bzr init /tmp/tmprepo; cd /tmp/tmprepo && git push HEAD:master git+ssh://..../host | 20:50 |
| jelmer | then later you can just "bzr dpush git+ssh://..." | 20:50 |
| chx | jelmer: sorry for dropping -- so how can be worked around the branch create bug? | 21:07 |
| jelmer | chx: so if you're in the bzr branch you'd like to dpush, run something like: | 21:11 |
| jelmer | git init /tmp/tmprepo; bzr init /tmp/tmprepo; cd /tmp/tmprepo && git push HEAD:master git+ssh://..../host | 21:11 |
| jelmer | then later you can just "bzr dpush git+ssh://..." | 21:11 |
| chx | jelmer: i will try | 21:12 |
| chx | jelmer: it's not that easy to be honest as i start from cloning / bzr-importing a remote repo | 21:12 |
| jelmer | chx, if you start from cloning / bzr-importing a remote repo then this is not necessary | 21:13 |
| chx | jelmer: ?????? | 21:13 |
| jelmer | chx: this is only for creating a new branch | 21:13 |
| chx | jelmer: yes but git repositories contain many branches? | 21:13 |
| chx | jelmer: what i have tried here was to create a new git branch. | 21:13 |
| jelmer | chx: bzr can only address the "current" branch at the moment, see bug 380871 | 21:14 |
| ubot5 | Launchpad bug 380871 in Bazaar "Allow imports of non-master branches when pulling from git repositories" [Medium,In progress] https://launchpad.net/bugs/380871 | 21:14 |
| jelmer | the only way you can work around that is to use a local git repository as a proxy | 21:14 |
| chx | jelmer: this all is extremely confusing | 21:16 |
| jelmer | chx: To summarize: git has multiple branches in a single location | 21:16 |
| chx | jelmer: i barely speak git , it's a major pita that i am now forced to use it, i hoped bzr-git will save my hide but it just adds more compliexity :( | 21:16 |
| chx | jelmer: yes | 21:16 |
| chx | jelmer: i understand git branches (or i hope i do) | 21:17 |
| jelmer | chx: Bazaar only supports a single branch per location at the moment, even with bzr-giut | 21:17 |
| chx | yes i understand that too but i hoped that with shared repository this can be fixed | 21:17 |
| chx | this is http://www.eecs.harvard.edu/~cduan/technical/git/ what i understand of git. | 21:17 |
| jelmer | chx: A shared repository doesn't help in that regard | 21:18 |
| chx | :( | 21:18 |
| jelmer | chx: It sounds like what you really want is a fix for bug 380871 | 21:19 |
| ubot5 | Launchpad bug 380871 in Bazaar "Allow imports of non-master branches when pulling from git repositories" [Medium,In progress] https://launchpad.net/bugs/380871 | 21:19 |
| chx | i have no idea what a no master branch would be | 21:19 |
| chx | i just wanted to create a new git branch from bzr-git :( | 21:19 |
| jelmer | chx: all the git branches except for the HEAD branch are non-master branches :) | 21:19 |
| chx | i see. | 21:20 |
| jelmer | I've updated the title of bug 380871 | 21:20 |
| ubot5 | Launchpad bug 380871 in Bazaar "support for colocated branches" [High,In progress] https://launchpad.net/bugs/380871 | 21:20 |
| chx | i might be an idiot -- but why do we need to colocate them? | 21:20 |
| jelmer | chx: that's what git does | 21:21 |
| chx | it's perfectly fine for me to "unpack" them , say git_repository_name/branches/branchname | 21:21 |
| chx | i mean, even bzr git-import creates a refs dir | 21:21 |
| jelmer | chx: if you do a push to a git repository the branches there are colocated, that's the way it is | 21:21 |
| chx | there yes | 21:22 |
| chx | but can't that be made into a bzr-git internal problem? :) | 21:22 |
| jelmer | chx: Internal problem how? Fixing that is what that bug report is about. | 21:23 |
| jelmer | an alternative would be a "bzr git-push" command that supports a --branch argument to specify the target branch | 21:23 |
| jelmer | but I'd rather focus the effort on fixing this properly | 21:23 |
| chx | on git-import copy the existing branches somwhere (isnt that what refs does ? ) and record somehwere bzr-git specific what happened in the shared repo and on dpush reverse the operation: take every bzr branch and send them as one git repostiroy | 21:24 |
| jelmer | chx: that's at least as much work as fixing that bug I mentioned earlier | 21:25 |
| jelmer | and less elegant of a solution | 21:25 |
| chx | i see | 21:26 |
| chx | i can pick from shooting myself in the left or the right foot | 21:27 |
| jelmer | chx: hmm | 21:27 |
| jelmer | chx: are you running bzr-git from trunk? | 21:27 |
| chx | jelmer: yes | 21:27 |
| chx | i can either use git whch i am not yet capable of or i can use bzr-git where bzr is not yet capable of :) | 21:28 |
| chx | jelmer: and bzr 2.3.0 | 21:29 |
| jelmer | chx: I could give you a very ugly hack to use an environment variable | 21:30 |
| chx | ugly hacks++ | 21:30 |
| jelmer | not now, but ping me tomorrow and I'll have a look | 21:30 |
| chx | sure. | 21:30 |
| chx | thanks, see you later | 21:32 |
| spiv | Good morning. | 23:03 |
| === Ursinha is now known as Ursinha-afk | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!