/srv/irclogs.ubuntu.com/2007/10/19/#bzr.txt

thumpermorning bzr dudes00:00
igcmorning all00:05
jelmerjam-laptop: there's no guarantee though that a revision with multiple parents contains only merges00:52
lifelessjelmer: what do you mean?00:52
jelmerlifeless: we were talking about rebasing "merge revisions"00:53
jelmerand how you could skip them during rebase if the rhs parent has already been merged in the branch you're rebasing00:53
lifelessjam-laptop: ping00:54
lifelessjelmer: surely its a topo sort of the revisions and rebase them all; merges too00:54
jelmerlifeless: rebasing a commit with a rhs parent that has already been merged is most likely going to cause conflicts00:56
jelmerhttps://bugs.edge.launchpad.net/bzr-rebase/+bug/12674300:57
ubotuNew bug: #154119 in bzr "Repository.item_keys_introduced_by should not lock Repository" [Medium,Triaged] https://launchpad.net/bugs/15411901:05
=== kiko is now known as kiko-afk
ubotuNew bug: #154129 in bzr "pack does not optimise layout" [Undecided,New] https://launchpad.net/bugs/15412901:41
lifelessigc: poolie: first round of fixen pushed01:59
lifelessjam-laptop: ping02:27
lifelesshi igc02:41
igchi lifeless02:41
igcworking thru pack_repo.py today FYI02:42
lifelesscool02:55
jeremy_cwill bazar launch a merge tool for me?02:59
lifelessdo you mean a gui conflict-resolving tool ?03:00
jeremy_clifeless, yes, such as Meld on Linux, or TortoiseMerge on windows?03:01
lifeless / per-file merge tool ? e.g. meld ?03:01
lifelessYes, there are some plugins that do this03:01
jeremy_cI'll look in the plugin section on the wiki. thanks.03:01
lifelessnp03:01
poolieok i've read the whole patch03:45
pooliei'm going to trim out the boring parts and post my comments03:45
=== kiko-afk is now known as kiko-zzz
lifelesswoot03:48
pooliemm?03:48
lifelessyour having read the entire patch03:50
lifelessI've just replied to all the reviews so far03:50
lifelessand actioned nearly everything I agreed with03:50
lifelessspiv: ping03:50
spivlifeless: pong03:50
lifelesstheres a gem in my patch for you03:50
lifelessinsert_data_stream03:50
lifelesswhere ith da docstring03:50
spivlifeless: heh, just read that a few seconds ago.03:50
lifelessi386: also where were you last night? release party one block from atlassian?03:51
spivlifeless: I'll write one and send it to the list.  Thanks for the poke.03:51
i386lifeless: asleep!03:51
larsl`Is there a way to get 'bzr send' to use a readable patch format?03:51
i386Ive been drinking heavily for the last two weeks and Ive been kinda sick03:52
lifelesslarsl`: it's got a human copy of the patch by default03:52
lifelesslarsl`: if you're not seeing that, what parameters are you giving it ?03:52
lifelessi386: :(03:52
lifelessi386: we put a few back for you :O03:52
larsl`lifeless: No other copy here. I just do 'bzr send --mail-to some@address'03:52
lifelesslarsl`: that should fire up your mail client03:53
lifelesslarsl`: with an attached patch03:53
larsl`lifeless: It does.03:53
lifelesslarsl`: the attached patch should be human readable with a base64 tail03:53
larsl`But the patch looks like a block of uuencoded something. There is no readable text.03:53
lifelesslarsl`: thats strange; is this open - can you pastebin the patch for me to look at ?03:54
larsl`Sure, it's just a test.03:54
larsl`lifeless: This is the contents of the sole attachment in the mail: http://rafb.net/p/MFCGVY79.html03:55
larsl`...oh. I need to commit locally before sending.03:57
larsl`That was just an empty patch.03:57
lifeless:)03:58
lifelessI was just finishing extracting the data to examine03:58
* igc food04:31
larsl`Trying to push a branch to a FTP server, but I keep getting the error message "FTP temporary error: 451 /bzr/bzzr/.bzr/repository/inventory.knit: Append/Restart not permitted, try again. Retrying."05:12
larsl`"bzr: ERROR: exceptions.AttributeError: 'FtpTransport' object has no attribute 'get_credentials'"05:12
lifelessoh hmm05:12
larsl`It seems to do something, because there are directories added on the FTP server, but I can't checkout.05:12
lifelessOur FTP maintainer is probably asleep right now05:13
lifelessvila: ^05:13
lifelessanyhow, the knit repository format requires APPE/REST be available, and they appear to be disabled on your server.05:13
larsl`Hm.05:13
lifelessthe get_credentials thing is a bug05:14
lifelessbut probably only occuring as a side effect05:14
larsl`Would any of the other formats work?05:15
lifelesspacks should05:16
lifelessthey are currently being reviewed, should be merged later today05:17
larsl`Ah. I'm using 0.91.0 from Debian Testing.05:17
abentleylifeless: Would weaves require APPE/REST?05:23
lifelessabentley: yes, for the history file in the branch object05:27
abentleyHow clever of me to have removed that requirement.05:28
=== thumper is now known as thumper-office
=== thumper_laptop is now known as thumper
spivlifeless: here's a docstring for Repository.insert_data_stream: http://rafb.net/p/9bcOyO41.html.  Any comments or should I just merge it?05:36
lifelessI think its too vague05:37
lifelessyou say e.g. bencoded05:37
lifelessis it or isn't it?05:37
lifelessis it allowed to vary by repo05:37
lifelessis there a version marker?05:38
lifelessor is it self delimited by a version?05:38
lifelesswhy why why basically05:38
spivlifeless: I was afraid you'd ask the hard questions :)05:38
spivI'm not sure of the answers to a lot of them to be honest.05:38
lifelessthere's a list05:39
lifelessI hear smart people subscribe to it05:39
larsl`So none of the current repository formats would work on a FTP server that doesn't allow Append/Restart?05:39
spivI'll start a thread there, then.05:40
ubotuNew bug: #154173 in bzr "pack reconcile does not correct for the fix_text_parents problem" [Undecided,New] https://launchpad.net/bugs/15417305:40
lifelesslarsl`: sorry, no :(05:43
larsl`I guess there is no write support for HTTP?05:44
lifelessthere is a webdav plgin05:46
lifelessplugin05:46
lifelessand there is bzr+http too05:47
lifelessthere is also SFTP05:47
lifelesswhich is more secure than FTP05:47
larsl`Yeah, but FTP and HTTP is all I have for now.05:48
larsl`What's bzr+http? Is there any documentation for it? Google only finds code.05:49
lifelesstry googling for 'serving bazaar with fastcgi'05:51
larsl`Seems to be read-only.05:52
lifelessor http://doc.bazaar-vcs.org/05:52
lifelesslatest, user guide,05:52
lifelessif you set auth up, then you can make it read write05:53
gotgenesCan I create a patch just using 'bzr -r REVNO.. > mypatchfile' ?05:54
lifelesssure05:55
lifelessthat will go from revno to your current tree05:55
gotgenesand then the person I send that patch to just does 'patch -p0 < mypatchfile' right?05:55
larsl`Hm. I don't think I have mod_python either.05:55
lifelessgotgenes: if they are not using bzr yes05:56
gotgenesgotgenes: they aren't05:56
lifelessgotgenes: if you and they are both using bzr you are much better off committing and using 'bzr send'05:56
gotgenesha05:56
gotgeneslifeless: ah, that's good to know!05:56
lifelessbug 15420407:04
lifelessbug #15420407:04
ubotuLaunchpad bug 154204 in bzr "revision specs do not lock branches appropriately." [Critical,Triaged] https://launchpad.net/bugs/15420407:06
ubotuNew bug: #154204 in bzr "revision specs do not lock branches appropriately." [Critical,Triaged] https://launchpad.net/bugs/15420407:11
igcheading off a bit earlier today to catch up with some people07:29
igcmight be back later tonight07:29
igcif not, have a good weekend all07:29
spivigc: me too.  Have a good weekend.07:30
igcthanks spiv07:31
vilajelmer: pong (looks like being in the same TZ doesn't help us to get in touch ;-)08:25
fullermdTZ?  You mean those things some weird people actually obey?  ;>08:26
vilafullermd: yeah, especially children you insensitive clod !08:26
vila:)08:27
fullermdAh, you just plow a few double espressos into 'em, they'll adjust.08:27
vilafullermd: obviously you don't know my daughters, they are already too much and too often awake for our taste :)08:37
vilathey leave us sleep sunday mornings since a few years though...08:38
fullermdSee, if you could just get them to use some of that awake time for bzr hacking...08:38
vilawhy do you think there are so many bugs in my submissions ?08:39
fullermdI see.  Kids these days...   no QC.08:40
vilaSheesh, and don't start me about jam on the keyboard...08:41
fullermdHah.  My first thought to that was "Doesn't John have his own keyboard?"   :p08:41
zerokmorning :)09:23
sabdflhey all09:40
sabdfllifeless: nearly, nearly there?09:42
KinnisonGood moring bzr hackers09:54
ubotuNew bug: #154259 in bzr "traceback on temporary ftp error" [Undecided,Confirmed] https://launchpad.net/bugs/15425910:16
fullermdI wonder if that error has EVER been temporary...10:18
=== zerok is now known as zerok|away
=== zerok|away is now known as zerok
lifelesssabdfl: very very close10:33
=== Mez is now known as Mez|Away
=== Mez|Away is now known as Mez
=== Philosof is now known as Olberd
OlberdHi. I'm trying to install bzrsvn in windows and can't make it work11:10
OlberdI have installed the python-svn requirement, but apparently not so that bzrsvn can locate it11:11
Olberdor could it be that the win32 package linked to on bzrsvn's homepage isn't new enough?11:13
Olberdjelmer, maybe you know?11:13
jelmerOlberd, Did you install the python-subversion bindings linked fomr the wiki page?11:13
Olberdyes, if you mean http://bazaar-vcs.org/BzrSvn11:15
ubotuNew bug: #154283 in bzr "indexerror in Knit._get_components_positions pulling in pack repo" [Undecided,New] https://launchpad.net/bugs/15428311:15
jelmerOlberd: What exactly doesn't work?11:19
Olberdjelmer, it says in bzrsvn's readme that the svn should be at least 1.5. But the link on the wiki has the version 1.4.3.11:19
jelmeryes, but the .exe linked from the wiki has got the required patches backported11:19
OlberdWhen I try to run bzr bzrsvn prints: "no python bindings for subversion installed..."11:19
jelmercan you start python and run "import svn" ?11:23
Olberdyes, no errors11:24
jelmeris the same python interpreter used by bzr ?11:26
jelmeror did you perhaps install a bzr that included its own python?11:26
OlberdI used the standalone 0.91 windows installer11:28
jelmeryou probably need the python-based installer11:30
jelmerI'll add a link to the wiki page11:30
Olberdok, I'll try using the other installer11:33
vilajelmer: pong (looks like being in the same TZ doesn't help us to get in touch ;-)11:43
jelmervila: :-)11:46
jelmervila: What exactly did you with your comment to bug 141105?11:46
vilayou mean mean ?11:47
jelmerI consider the svn+ prefix a hack, ideally bazaar should just handle SSL certificates properly11:47
vilahrm11:48
vilajelmer: feel free to reopen the bug then11:49
vilawhat I meant was that bzr now gives a proper error message instead of KeyError: 7711:49
jelmerah, ok11:49
vilaHandling SSL certificates is on my TODO list, python 2.6 will provides an ssl module for that11:50
vilaproblem solved :)11:50
jelmer:-)11:50
vilakidding, there is a module that can be installed for python 2.[45], I'm looking into integrating it into bzr or making bzr depend on it11:51
jelmerAh, I was confusing this bug with bug 8208611:51
jelmersorry11:51
vilanp, better two brains than none on any subject :)11:51
vilathat module will allow us to also have a proper https server, but there is a bit of work to get there11:51
vilaand then, we can just deprecate pycurl and drink some champagne :)11:52
jelmerYeah, that'd be nice11:54
ryanakcaIs there a way to make a useable diff? I know 'bzr diff', but you can't really apply === modified file 'template.htm' (properties changed)11:56
Kinnisonbundles11:56
Kinnisonthey're more what you want11:57
KinnisonI think11:57
ryanakcathanks :)11:57
KinnisonSpecifically the 'bzr send' command I think11:57
Kinnisonaah or the 'bzr bundle-revisions' command may be of more use11:59
ryanakca:)12:01
ryanakcaKinnison: any idea for http://pastebin.ca/742241?12:04
Kinnisonodd12:04
Kinnisonmake the bundle --no-patch?12:04
Kinnisondoes that work?12:05
* ryanakca tries12:08
ryanakcaKinnison: bzr: ERROR: no such option: --no-patch12:08
jelmerOlberd, any luck?12:11
Kinnisonryanakca: odd, because it's there for m12:14
Kinnison+e12:14
ryanakcahmmm...12:14
ryanakcawhat version?12:14
ryanakca0.90.0?12:15
ryanakcaKinnison: hmm... I'll be back in 8 hours or so, school. See you ;)12:21
danigmhi, i have a question. When two developers push in one branch, and each developer has his own local branch, the log is overwrite. There is a method for push, or something similar that log well?12:49
danigmi mean, i push, and in the log appear danigm has changed ..., and when other developer push, must appear only his changes, not his branch log12:50
jelmerdanigm, not sure I'm following12:51
danigmwhen i change something in my own branch, the log say that i make, and i merge with the main branch, and log it as "merged with main branch". And then push my branch, and the log of main branch dissapear, and appear my own log. I upload my own branch, the main branch is dessapear, is merged.12:54
danigmI think it's possible having two locals branch, my branch, and the main branch. When I want to publish my chages in main branch, I should merge that with mine, and push main12:57
datoI *think* he wants append_only = True, lemme re-read12:58
danigmbut I need to branch from main, everytime that i want to push my changes12:59
datooh, he was spanish13:02
=== mrevell is now known as mrevel-lunch
Olberdjelmer, other error now13:12
Olberdit simply says: unable to load plugin u'bzrsvn' from u'.../plugins'13:15
OlberdI've run 'bzr selftest'13:15
OlberdIt returns error: ... no module named svn.transport13:15
jelmerthe plugin has to be named 'svn'13:15
Olberdok13:16
jelmerrather than bzrsvn13:16
Olberddoes it say that anywhere?13:16
jelmerit's only necessary for the testsuite afaik13:16
Olberdok13:17
Olberdwell, not only for the selftest13:18
OlberdRenaming it solved my problems it seems13:19
=== mrevel-lunch is now known as mrevell
=== kiko-zzz is now known as kiko
=== mvo_ is now known as mvo
mrevellHey - what version was pushing using bzr+ssh introduced? According to the release notes, I think it might 0.11 but then it says smart server didn't come until bzr 0.16.15:01
quicksilverbzr+ssh isn't smart server15:06
quicksilverbzr+ssh is just using ssh isn't it?15:06
mrevellAh right, thanks.15:06
radixquicksilver: no, it's the smart server15:20
radixquicksilver: You can't do much with "just ssh" :)15:21
radixbzr+ssh connects on ssh and immediately starts up a smart server on the other side to handle the connection15:21
=== cypherbios is now known as cypherbios|lunch
=== cypherbios is now known as cypherbios|lunch
quicksilverradix: You could. You use use ssh and then shell out to commands like 'cat' and 'chmod' to build/alter the repo.15:28
quicksilverradix: I have used other systems which work like that.15:29
quicksilverbut fair enough if that isn't what it does :)15:29
radixquicksilver: that's "ssh and cat and chmod", not "just ssh" :P15:29
radixbut that's enough pedantry from me :-)15:30
=== cprov is now known as cprov-lunch
=== cypherbios|lunch is now known as cypherbios
=== kiko is now known as kiko-fud
=== Mez is now known as Mez|Away
=== kiko-fud is now known as kiko
james_wWhen using VersionedFile.add_lines() the parents argument is the previous revision-ids in which the file was modified, rather than the parents of the current revision, is that correct?17:38
james_wso 3 revisions, with revids rev1, rev2, rev3, and a file a that is created in rev1 and modified in rev3.17:38
james_wYou would add lines for rev1 with no parents, and then rev3 with rev1 as parent, with nothing for rev2, is that correct?17:39
jam-laptopjames_w: Correct17:43
jam-laptop(It is part of setting the per file graph, and also making sure the delta is built against an existing revision)17:43
james_wjam-laptop: thanks.17:43
james_wjam-laptop: also, any suggestion for what the tree root id could be for bzr-git. I am going with hash(path) for the ids, but this means that all trees would have the same root. This doesn't seem ideal, but I'm not sure what the impact would be.17:44
jam-laptopjames_w: At the moment, I think all bzr trees use 'TREE_ROOT'17:45
jam-laptopwhich is also not ideal17:45
jam-laptopbut it turns out that older versions of bzr didn't handle other values properly17:45
jam-laptopso we had to wait for a format bump to introduce unique ids17:45
james_wit's unambiguous though :)17:45
jam-laptopI believe Knit3 uses unique ids17:46
james_wyeah, it does for what I have seen17:46
jam-laptop(And bzr since about 0.14 or so is able to handle it)17:46
jam-laptopactually, maybe all the way back to 0.1017:46
jam-laptopand it is just 0.8 that can't17:46
jam-laptopanyway17:46
jam-laptopThe *reason* to have unique tree roots17:47
jam-laptopis so that if you merge 2 trees into 117:47
jam-laptopthen when you do further merges17:47
jam-laptopit knows where to put files that show up in the root dir17:47
jam-laptopI'm guessing Git doesn't have any concept of project id17:47
jam-laptopetc17:47
jam-laptopso I would just go with hash('')17:47
jam-laptopand live with it17:47
jam-laptopyou *could* use TREE_ROOT17:47
jam-laptopif you wanted17:48
james_wI'll go with the hash, thanks.17:48
james_wI was wanting to make it unique, as git supports submodules, and so we should map them on to nested trees at some point, and it seems non-unique would get in the way.17:49
james_wbut they don't want to be too unique :)17:49
jam-laptopwell, if you made them random17:49
jam-laptopthen 2 people using bzr-git couldn't interoperate17:50
jam-laptopwe'll have to handle the nested tree case anyway for our own data17:50
jam-laptopsince we already have all trees with non-unique root ids17:50
jam-laptopplus bzr-git is going to have issues with all the same path directories looking alike anyway17:51
jam-laptop(so having 'src/' is going to collide with every other project that has a src/)17:51
james_wah, of course. I'll just use that solution when you find it.17:51
jam-laptopof course17:51
jam-laptopif you had a way to come up with a unique tree root17:52
jam-laptopyou could incorporate that into all the other directories17:52
james_wyeah, one solution could be to have a project-id passed to bzr-git that is just introduced in to the hash.17:52
jam-laptopbut I can't think of a way to do it17:52
james_wbut that would require an extra argument added to branch, checkout, init etc.17:52
jam-laptopif you wanted to be minorly evil17:53
jam-laptopyou could double hash the first commit17:53
jam-laptop(take the commit's hash, and hash it)17:53
jam-laptopthat would most likely be unique17:53
jam-laptopif a bit odd17:55
jam-laptopand might have problems after joining projects17:55
jam-laptopsince then you have multiple rev1 nodes17:55
jam-laptop(you could sort them and just always pick the 1st one)17:55
james_wyeah, using the root commit as the root id did occur to me. You would get collisions occasionaly, but they would be far less.17:56
jam-laptopcertainly far less than using the same value for everything :)17:56
james_wThere are going to be a massive number of fork+execs to make this work it seems.17:59
jam-laptopwell, that is how git is written, right?18:09
Keybukwing-commander upstart% bzr bundle18:11
KeybukUsing saved location: bzr+ssh://keybuk@bazaar.launchpad.net/%7Ekeybuk/upstart/main/18:11
Keybukbzr: ERROR: exceptions.AttributeError: 'RemoteRepository' object has no attribute '_make_parents_provider'18:11
Keybuk^ d'oh18:11
jelmerKeybuk: IIRC that's a known bug18:13
jelmerhigh on the list for 0.9218:13
Keybukok18:25
=== cprov-lunch is now known as cprov-out
=== p4tux is now known as YBAM
=== YBAM is now known as p4tux
=== p4tux is now known as help
=== help is now known as p4tux
jam-laptopspiv: I still haven't heard the status of chunked encoding: http://bundlebuggy.aaronbentley.com/request/%3C20070903172153.GC22003@steerpike.home.puzzling.org%3E20:04
jam-laptopI think Martin pinged you on it the other day, but I didn't see a response20:04
ryanakcabzr merge foo.bundle : any idea for http://pastebin.ca/742241?20:56
AnMasterbzr+ssh push protocol seems to ignore ~/.ssh/config21:08
AnMasterbecause it doesn't try to connect on the port I set it should in ~/.ssh/config21:08
AnMasterso pushing doesn't work as server use non-standard port21:09
AnMastercan anyone confirm this?21:09
AnMasterthis bug*21:09
AnMasterryanakca, tabs/spaces mixup?21:10
ryanakcaAnMaster: yes, but why should bzr crash (other than python not liking mixtures of tabs/spaces)21:11
AnMasterryanakca, no idea21:11
AnMasterany idea about my problem?21:11
poolfoolAnMaster: https://bugs.launchpad.net/debian/+source/bzr/+bug/14671521:30
ubotuLaunchpad bug 146715 in bzr "bzr+ssh broken for non standard ports." [High,Fix committed]21:30
AnMasteryes just found it a bit before21:30
poolfoolAnMaster: Sorry didn't take the time to read it ... I was just lurking.21:31
AnMasterno problem21:31
jam-laptopryanakca: it seems to be doing the merge using a patch21:36
jam-laptopnot a bundle21:36
jam-laptopand it is requiring the source to be an exact match21:37
jam-laptopThe traceback is probably because that code path isn't used yet21:37
jam-laptopI can't say I've seen it before21:37
jam-laptopI could be wrong21:38
jam-laptopAnMaster: known bug21:38
jam-laptopIt has been fixed in bzr.dev21:38
jam-laptopso it will be fixed in 0.9221:38
jam-laptopryanakca: The "crash" is probably because we would rather fail safe than continue, so we would at least give an error that the patch does not match21:39
jam-laptopIt probably should be a simple refusal21:39
jam-laptopand not a traceback21:39
ryanakcajam-laptop: so, how to I make the requiring source an exact match?21:42
ryanakcajam-laptop: bzr revert?21:42
lifelessmoin21:44
jam-laptophi LarstiQ21:45
jam-laptoplifeless:21:45
jam-laptop^^21:45
jam-laptopcome to think of it, I haven't seen LarstiQ in a while21:45
jam-laptopryanakca: it sounds like your bundle was munged21:45
jam-laptopand it is trying to ensure that the bundle you merge is the same as the patch you see21:45
jam-laptop(so that you don't look at the preview and approve that, but the actual merge merges something else)21:45
jam-laptopryanakca: so it is *possible* that you could just edit the bundle to change the lines back to what it thinks they should be.21:46
jam-laptop(if they are now spaces, try making them tabs, etc)21:46
ryanakcaAha... I see.21:46
ryanakcajam-laptop: recreating the bundle fixed it... methinks it has something to do with copy-pasting.21:47
jam-laptopthat would do it21:47
jam-laptopif it had tabs in it21:47
jam-laptopand you copy pasted it from a terminal21:47
jam-laptopthat would probably turn it into spaces21:47
jam-laptopbzr send -o foo.patch is your friend :)21:47
jam-laptop(As would be bzr send --mail-to=joe@foo.com)21:48
ryanakca:)21:48
ryanakcaOr just a plain old wget (since the bzr branch is in /var/www ) :D21:49
ryanakcajam-laptop: thanks!21:49
jam-laptopryanakca: no problem21:50
lifelessryanakca: well, if the branch is wgettable, just 'bzr merge' :)22:02
lifelesslaters22:03
ryanakcalifeless: oh yeah, hehe :)22:04
gotgenesI can't connect to my server's svn repo using bzr-svn because it's through https with a self-signed certificate. Is there a workaround?23:43

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