/srv/irclogs.ubuntu.com/2008/12/24/#bzr.txt

=== joshuablount is now known as joshuaAFKblount
* LeoNerd unsubs from the mailing list00:18
LeoNerdThis is just waaaaay too busy for me to keep up with :/00:18
jelmerLeoNerd, hmm? I thought it was actually pretty quiet lately..00:50
pooliejml, hi?02:11
* igc lunch03:01
jmlpoolie: hi03:17
jmlpoolie: I'm not at work today.03:17
pooliehi04:09
pooliejml, i just realized that04:09
poolieenjoy your holidays04:10
=== mw is now known as mw|out
DBOis it possible to make one bzr branch just eat all the files of another bzr branch that has no common ancestor04:19
poolieDBO, 'bzr merge -r 0.. OTHER_BRANCH'04:47
DBOpoolie, slightly too late =(04:47
DBOjust did it all by hand04:47
pooliesorry04:50
bobesponjahi07:17
bobesponjais there a way to do a partial checkout of a branch?07:17
igcbobesponja: not yet. See http://bazaar-vcs.org/FilteredViews for the latest on progress towards this feature.07:21
bobesponjaok thanks07:36
=== Tobbe|autoaway is now known as Tobbe
igctime for my xmas holiday/break08:04
igcsee you all next year08:05
=== Tobbe is now known as Tobbe|autoaway
=== Tobbe|autoaway is now known as Tobbe
=== thekorn_ is now known as thekorn
CaMasonHi guys.. I just tried to so a bzr branch over sftp on Vista x64, and I got a BSOD :o14:40
CaMasonif I do a branch on my linux machine, then copy that over to windows, will it work14:41
CaMason?14:41
Jc2kxD15:22
jelmerJc2k, hi15:55
=== Tobbe is now known as Tobbe|autoaway
=== Hydrogen is now known as MerryChristmas
jambzr co16:48
jammt16:48
rexbronhello, is it possible to change the dpkg-buildpackage command that --source uses in bzr-builddeb?16:49
rexbronie. via an environment var or something?16:49
LarstiQrexbron: could you give some more context? What are you trying to do?16:50
jelmerrexbron, yes, I think it's mentioned in the bzr-builddeb docs (can't remember off the top of my head)16:52
rexbronLarstiQ: in the bzr-bd man page is states that --source defaults to "dpkg-buildpackage -rfakeroot -us -uc -S". Looking closer, I see it mentions that it can use config files, but no reference is made as to 1) where those config files should be/are and 2) the format those config files should take16:53
* LarstiQ looks at bd docs16:54
LarstiQrexbron: I'd assume regular bazaar configuration mechanisms16:54
* rexbron found the README :P16:55
LarstiQrexbron: the README states, .bzr-builddeb/local.conf in the package directory, ~/.bazaar/builddeb.conf and .bzr-builddeb/default.conf in the package directory16:55
LarstiQright :)16:56
rexbron199     The command used to build a source package if the ``--short`` or ``-S``16:59
rexbron200 69.1.7     options are used.16:59
rexbronhmm, is --short supposed to be --source?16:59
james_wrexbron: it is, yes17:04
james_whttp://jameswestby.net/bzr/builddeb/user_manual/configuration.html17:05
rexbronjames_w: what would be the best way of adding additonal export profiles? ie. my builder command for the archives would be different from the one I would use for revu.17:06
rexbronis it possible to do so from the .conf files?17:06
james_wnot currently17:06
james_wI want to add that sort of ability, but I've not come up with a good way to do it yet17:06
rexbronjames_w: is this a planned feature already? Shall I draw up a spec?17:07
james_wif you have a good idea of how it should work I would love to hear it17:07
rexbronjames_w: just thinking here, the same way that we can pull the value of builder from config file, perhaps it would be possible to also define a way to have bzr-bd associate command-line switches17:09
rexbronsomething along the lines of if --revu is passed and is in a .conf file, use this builder string17:09
james_wnot sure about that17:10
james_wyou could have "--builder-profile revu" which makes it look up "revu-builder" in the configuration17:11
rexbronand a short alias for brevity :)17:11
james_w:-)17:11
rexbronthat would simpify things for me and strikes me as something that people will want as Ubuntu moves towards bzr and bzr-bd as the primary tools for handeling packages17:12
james_wprobably17:15
james_wwe also need a way to easily add e.g. "-sa" to the build17:15
james_wseparate "profiles" to do that seems like a bit of overkill17:15
rexbronjames_w: profiles would be an extendable way to impliment that, while seeming like overkill would prevent this sort of problem from ocuring further down the line. I would sugget that either bzr-bd or another package in ubuntu ship a set of commonly used profiles.17:17
james_wyup17:17
rexbronjames_w: so now it all comes down to getting someone motivated to do it ;)17:20
james_wheh :-)17:21
rexbronjames_w: I'll take a stab at it, but I am a real amature17:30
james_wit shouldn't be too hard17:30
james_wfamous last words :-)17:30
james_wthat would be great though, thanks17:30
rexbronjames_w: I am really wondering what the best way to go about this is17:57
=== Tobbe|autoaway is now known as Tobbe
rexbronjames_w: I am wondering if implementing something like the way hooks work...18:03
rexbronwould be the best way to go18:03
james_wI don't see how18:04
rexbronjames_w: basically specifying a section like [PROFILES]18:15
james_wah18:16
james_wyeah, that could work18:16
rexbronand either specifying profile = revu18:17
rexbronbuilder = ..18:17
rexbronand so on an so forth18:17
rexbronor whether it would be better to merge the profile name into the variable18:17
james_wrevu = debuild --whatever18:18
james_wwould work18:18
james_wor it could go in to the [BUILDDEB] section as "revu-builder"18:18
rexbronjames_w: or better might be rather than have a section called profiles, just use the profile name there instead18:21
james_wrexbron: yeah, I agree18:23
dashanybody know who I could talk to about bzr-builddeb? :)18:26
james_wheh18:26
dashaha18:26
james_whello dash18:26
dashthat name looks familiar. :)18:26
dashso, i'm adding debian packaging to a project I developed in bzr18:27
dashit uses autotools18:27
dashi don't keep 'configure' and other generated stuff in the repo, just the sources (configure.ac, Makefile.am)18:28
dashis there a way to get bzr-builddeb to run autoreconf before starting dpkg-buildpackage?18:28
dashi could added it to debian/rules, but I wouldn't want to run it if building from a tarball18:28
dashany ideas? :)18:32
james_wdash: http://jameswestby.net/bzr/builddeb/user_manual/hooks.html18:47
dashaha, hooks18:47
dashi should have read that. :)18:48
dashthat does the trick18:50
dashjames_w: thank you. hope your Christmas is full of merges and conflict-free. :)18:51
james_wheh. thanks :-)18:53
rexbronjames_w: I think I just got it to work :D19:07
james_wcool19:07
Jc2kjelmer: hi19:31
jelmerJc2k, hi!19:43
jelmerJc2k, I've added a dul-fetch-pack command that seems to work19:43
Jc2kcool!19:43
jelmerJc2k: Your upload-pack command doesn't seem tested yet in the server?19:45
* jelmer now needs to figure out how the pack filename is determined19:49
Jc2kpack-sha1ofpack.pack19:49
Jc2k?19:49
* Jc2k is on phone19:49
jelmerJc2k, it's a sha1, but I'm not quite sure of what yet. It's not the sha1 of the pack file or its index at least19:52
Jc2ki just assumed it was the sha1 of the pack (minus the 20 byte sha footer) :(19:54
Arjenjelmer: I think you can find the answer in pack-write.c (if you're talking about Git packfiles)19:54
Jc2kjelmer: am gonna pull the git backend out of the daemon and into dulwich/server.py (planning to add dul-receive-pack and dul-upload-pack)19:56
jelmerArjen: That assumes the sha1 is already calculated elsewhere (it uses sha1 in struct packed_git)19:56
Jc2kunless there is a better place19:56
jelmerJc2k, you mean dul-receive-pack and dul-upload-pack as separate binaries ? :-/19:57
Jc2kjelmer: just for testing purposes - they'll basically be little stubs..19:57
jelmerJc2k, ah, ok19:58
Jc2kjelmer: then i can plumb them directly to git with git-fetch-pack --exec=dul-upload-pack or whatever the syntax is]19:58
rexbronInteresting error: https://code.edge.launchpad.net/~rexbron/bzr-builddeb/trunk.profiles19:58
jelmerJc2k: So dul-daemon will still be using the internal code?19:58
rexbronsomething about an internal check failing19:58
Jc2kjelmer: yeah, it becomes about 3 lines importing TCPGitServer and GitBackend and then calling TCPGitServer.serve_forever19:59
jelmerJc2k: Ah, makes sense19:59
rexbronInternal check failed: Newly created pack file <bzrlib.repofmt.pack_repo.NewPack object at 0x2def190> has delta references to items not in its repository: {'inventories': [('jw+debian@jameswestby.net-20081202164232-17kqr0tsvqy3ebyi',)], 'texts': [('config.py-20060816235838-wk9rk801k2fdaihy-3', 'jw+debian@jameswestby.net-20080529233852-6p0p3xd8198fx20t'), ('readme-20060917150309-xbse5n20p61fw62z-1', 'jw+debian@jameswestby.net-2008091017002019:59
rexbron-6helfl8ag97p4sgt')]}19:59
jelmerrexbron, is this a stacked branch?20:00
Jc2kjelmer: don't worry, im not going to start doing forky things :p20:00
rexbronjelmer: yes, but this is only LP's side only20:00
rexbrons/only//20:00
jelmerrexbron, looks like a stacking bug to me, there were some issues like this fixed recently20:03
rexbronjelmer: would it be a client or server side bug?20:03
jelmerrexbron, no idea, sorry20:03
james_wI think that was a bug fixed in a recent client20:06
* rexbron goes to see if it hit jaunty20:06
jelmerJc2k, found it20:10
jelmerJc2k, it's the sha1 over the object sha1s in the pack20:10
rexbronjames_w: would deleting the branch and re-pushing it potentially avoid the error20:10
james_wrexbron: yes, if you have a fixed client20:11
LarstiQjelmer: so then you know not to fetch the pack if you have all objects, or what?20:13
jelmerLarstiQ, I think the main intent is that it would be unique20:13
jelmerAnd deterministic20:14
=== phinze_ is now known as phinze
LarstiQhmm20:14
Jc2kit means lots of people can push at once and not collide20:15
LarstiQJc2k: ah yes, and if the pack is already there, no need to push it, since all your object are there20:16
Jc2kwell, it doesnt care about the pack20:16
Jc2kjust the objects20:17
LarstiQhow would people collide then if not for the naming?20:17
Jc2kthe objects would be duplicated :)20:18
Jc2kgit just checks all pack indexes until it finds a match, so objects can be duplicated until the next git repack20:18
LarstiQah20:18
rexbronI just noticed that the bzr PPA does not build packages for Jaunty, would this be possible?20:19
LarstiQ~/bzr/+archive has loggerhead-1.10 for jaunty20:22
LarstiQrexbron: https://edge.launchpad.net/~bzr-nightly-ppa/+archive though20:22
rexbronLarstiQ: thanks20:22
asabiljelmer: I just checked out dulwich, and ran the unit tests, it seems like 1 test case is failing20:26
asabilis that a known issue ?20:26
=== asac_ is now known as asac
Jc2kjelmer: pushed dul-upload-pack and dul-receive-pack, untested atm (on a mini mini laptop which is making me want to cry)20:45
Jc2kjelmer: what are you thinking about re tests and the server code?20:45
* Jc2k wants tests v v v v much20:45
* LarstiQ offers jc2k his 3 kilo laptop in exchange for the mini laptop20:46
* Jc2k has an XPS GEN2 at home, so declines :p20:49
jelmerasabil, yeah, that's right. I haven't checkout out what's causing it yet20:50
jelmerJc2k, Yeah, we need heavy testing20:50
* Jc2k agrees20:50
Jc2ki wanted to test it directly against the client side git components20:50
jelmerhmm, looks like the delta resolution can use some more work. It works, it's just not very fast20:59
Jc2kalso pushed some protocol doc. its not much atm, but adds a bit more to upstream docs.21:01
rexbronbzr-bd review request: https://code.edge.launchpad.net/~rexbron/bzr-builddeb/trunk.profiles/+merge/253521:08
=== pbor is now known as pbor|out
=== fawek is now known as fawek|away
jelmerrexbron, this seems pretty similar to "source-builder" setting in the bzr-builddeb config?21:13
rexbronjelmer: what if you have multiple targets for your source builds?21:13
jelmerrexbron, what do you mean with multiple targets specifically?21:13
rexbronjelmer: I need to use different dpkg-buildpackage options to upload to REVU as I would for the main ubuntu archives.21:14
rexbronfor example, revu always requires a full-source upload21:14
jelmerrexbron, ah, ok21:15
jelmerrexbron, that makes more sense21:15
rexbronjames_w and I discussed this earlier today and came to the conclusion that this is more extendable solution21:16
jelmerJc2k, hmm, if I send "have" lines the server suddenly hangs up on me21:37
Jc2kjelmer: i havent tested that yet - got distracted by the corrupt packs21:41
Peng_jelmer: Not to try to rush you, but is anything going to become of 308353, or am I gonna be stuck on bzr-svn 0.4 or something?21:42
Peng_Err, bug 30835321:42
ubottuLaunchpad bug 308353 in bzr-svn "[0.5] KeyError in old_inventory when branching Lighttpd 1.4" [Medium,Triaged] https://launchpad.net/bugs/30835321:42
Jc2kjelmer: will have a look when i've wrapped my mums xmas present!21:43
Peng_Like I said, I'm not trying to rush you. I'd just like to know it's somewhere on your todo list.21:43
jelmerPeng_, I *may* get to it in the next two days, otherwise it'll probably be in one of the first days of the new year21:45
Peng_jelmer: Ok. Thank you. :)21:46
asabiljelmer: concerning the test failure in dulwich, I think the test case is wrong22:00
asabilthe ordering of the expected result seems reverted22:00
asabils/revert/invert/22:00
Jc2kjelmer: hang up with no error, or something else?22:00
jelmerJc2k, never mind that, I've already fixed it22:01
jelmerasabil, as I said, I haven't really looked into it yet22:01
jelmerasabil, patches welcome, though :-)22:01
asabiljelmer: yeah sure, I will try to push a branch when I get more confident about my checks22:02
jelmerah, I know why the delta resolving is slow22:10
jelmerit's in an infinite loop :-P22:10
Jc2k:p22:10
Jc2kjelmer: do you have any thoughts on testing server.py?22:12
Jc2kif not im going to knock up something that should work, but im not sure where it lands on the dirty scale22:12
jelmerJc2k, well, some ad-hoc tests won't hurt22:13
jelmerJc2k, It would also be nice to test the client against the server22:13
jelmerbut it's also good to test outside of that, to make sure they're not both broken :-)22:13
Jc2k:]22:13
Jc2ki was thinking dummy backend + pre-recorded streams to replay at client22:14
Jc2k**SERVER22:14
Jc2kidef send_fn(data): sys.stdout.write(data) sys.stdout.flush()22:14
Jc2koops22:14
Jc2kdamn tiny laptop22:14
Jc2kis the client pushed somewhere?22:15
Jc2ki merged lp:jelmer22:15
Jc2k/dul/trunk22:15
jelmerhttp://people.samba.org/bzr/jelmer/dulwich/trunk22:15
jelmerJc2k, ^22:17
* Jc2k merges, commits, pushes22:17
jelmerJc2k: hardcoding some streams doesn't seem to ugly to me22:18
jelmerthe main issue would be the readability, and the size of the tests22:18
Jc2ki was going to reuse the pkt line stuff from server.py22:19
Jc2kso it would be a bit less eww..22:19
Jc2ki wanted to try integrating dul-* foo into the git test suite if its not too bad..22:19
Jc2ki think we can factor some of client and server code out into protocol.py ?22:21
Jc2klike the pkt_line stuff and sideband stuff.22:22
Jc2kand possbily capabilities22:22
=== cprov is now known as cprov-out
jelmerJc2k, yeah, I think that would be useful22:48
Jc2kjelmer: just pushed a protocol.py and made client and server use it22:49
Jc2kim a bit drunk and theres no coverage so YMMV :)22:49
Jc2k*test22:49
Jc2kjelmer: going to write some tests that poke the server stuff via protocol.py22:51
Jc2kjelmer: how about an object with 2 read/write pairs22:57
jelmerJc2k: :-)22:57
jelmerJc2k, btw, do you have any objection to having all new code being GPLv2+ like bzrlib rather than GPLv2?22:57
Jc2knone at all22:57
Jc2kthough i wrote some of this on work time so i should probably check..22:58
jelmerJc2k: k22:59
Jc2ktests: i dont want to enter blocking hell by trying to test my server directly, so was thinking about a buffer object where i created an expected conversation (a buffer to read from, and a buffer off what to expect the server to send)22:59
Jc2kTHEN invoke the server handler with a read_fn and write_fn that just operate on the buffer22:59
Jc2kexploding if the output differs.23:00
jelmerthat doesn't ensure that a packet in the read buffer comes before one in the write buffer and vice versa though?23:00
Jc2kblocking hell it is, i guess :o23:01
Jc2kor i could make my test case yield23:01
Jc2ki dont have enough brain for that tonight..23:02
Jc2kthe conversation builder idea could be modified to be a single buffer of tuples ('r', 'foo'), ('w', 'bar'). the yieldy approach would just remove the buffer and be a pull instead of a push23:04
Jc2kall so messy.23:05
Jc2ki might just wait to see how you test the client :P23:06
jelmerheh, ok23:10
jelmercycling time, back in ~45 min23:10
Jc2kjelmer: moved the send_cmd stuff as AFAIK its tcp/ip specific - so now the GitClient class should work for git+ssh:// access for when i start jabbing things with the git tests23:37

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