/srv/irclogs.ubuntu.com/2008/08/13/#bzr.txt

lifelessjam: yes, thats my thoughts too00:00
lifelessjam: want to work up a patch, or shall I?00:00
lifelessthe packing heuristics still bother the heck out of me00:01
jamobviously I won't have time to do so today... :)00:01
lifeless:P00:01
jam"packing heuristics"?00:01
lifelessI'm still looking at this critical bug today I expect00:01
lifelesskeys to nodes00:01
lifelessthe gzip magic00:01
jamah, that part00:01
jamwell, we can punt on it00:01
pooliejam, spiv, hi00:01
jambut it is fairly useful00:01
lifelessjam: its needed I think00:02
jamhi poolie00:02
lifelessjam: but its gets very bad on bzr-search indices00:02
jamIIRC it shrunk them by a significant amount00:02
jamlifeless: how long are your bzr-search keys?00:02
lifelessjam: tiny00:02
lifelessjam: 4, 5 bytes00:02
jamthe pack work was certainly tuned around 60-byte keys, and often 120 bytes as tuples00:02
lifelessthe keys are document ids00:03
lifelesse.g. '1'00:03
lifelesswell, ('1',)00:03
spivGood morning.00:03
pooliejam, merging from BB urls was working for me on monday00:03
jampoolie: it seems to be a BB thing00:03
jamas BB is giving a 200 OK, Status: 404 results00:03
mwhudsonbeuno: hm you said something about making the breadcrumb stuff less inventory specific00:03
pooliemake up your mind :)00:03
mwhudsonbeuno: did you get to that?00:03
jamso something is confused about the return code00:03
mwhudsonspiv: hello00:03
jamI don't know HTTP well enough to know if "Status: " is supposed to overwride00:04
beunomwhudson, no, I didn't. I left that for after we fixed the root issue00:04
lifelessjam: so, inventories00:04
mwhudsonbeuno: hmm00:04
lifelessjam: I mailed,  you replied - is it paused now ?00:04
lifelesshi poolie00:05
jamlifeless: today was painting the deck redux00:05
jamI want to follow up00:05
spivmwhudson: hi00:07
lifelessjam: ok00:08
lifelessjam: I'd like that00:09
mwhudsonbeuno: http://bazaar.launchpad.net/~mwhudson/loggerhead/directory_breadcrumbs/revision/20900:22
beunomwhudson, I knew you'd find a simpe way around it  :)00:23
mwhudsonit's a bit of a hack, but then hey! it's loggerhead00:23
mwhudsonat least it's a localized hack00:24
beunohahaha00:24
aquariusI'm a bit confused by bzr init and init-repo. I'd like to store my home folder on my laptop in bzr, with a centralised repo on a remote sftp server (so I can check out from there to other machines).00:24
james_whey aquarius00:25
james_wyou probably don't need init-repo in that case00:25
aquariusah. I'm not clear what the difference between init-repo and init is...00:25
james_wyou may want it on the server if you are going to have branches "desktop" and "laptop" that are slightly different and you merge between them00:26
beunomwhudson, so you think it's ready for merging, or do you want me to have another look at it?00:26
james_wbut I doubt you'll be making feature branches on your laptop to try out experimental new settings00:26
james_whey besonen_mobile__00:26
aquariusso can I just, on the laptop, do "bzr init sftp://server; bzr checkout sftp://server $HOME" ?00:26
james_whey beuno00:26
beunohey james_w, how are you?00:26
mwhudsonbeuno: which?  directory_breadcrumbs? no00:26
james_wgood thanks beuno, you?00:26
james_wenjoying work?00:26
aquariusjames_w: for the moment, assume that I want all checkouts to be identical on every machine. (I may add cleverness later. :))00:27
james_waquarius: that will work in principle, though it would probably fail as $HOME already exists00:27
beunojames_w, good good, enjoying it very much, yes  :)00:27
james_waquarius: you probably want to create the branch locally and then push to the server00:27
james_wbeuno: cool :-)00:27
aquariusjames_w: OK...see, now I'm confused again. How do I do that, then? :)00:28
james_waquarius: "cd $HOME; bzr init; bzr add <files you want to version>; bzr commit; bzr push sftp://server/" or similar00:28
james_waquarius: plain "bzr add" is probably not what you want, so I would avoid that.00:29
james_wjudicious use of "bzr ignore" will probably help you avoid problems00:29
aquariusjames_w: excellent. A few questions. can I just do plain "bzr add" if I already have a .bzrignore file that correctly ignores the stuff I want to ignore?00:29
james_wunless you actually aim to version ~/Videos/ and ~/Music/00:29
aquariussecond: does sftp://server/whatever/I/put have to exist before I push to it, or will push create it if it doesn't exist?00:30
james_waquarius: yeah, if you've set up ignores then bzr add won't add them. If you "bzr add <something you've got in .bzrignore>" then it will be added00:30
aquarius*nod* win00:30
james_waquarius: no, it doesn't have to exist, but if sftp://server/whatever/I doesn't exist then you will need --create-prefix00:31
aquariusthird: once I've done all that, on machine2 can I just "cd $HOME; bzr checkout sftp://wherever/ ." ?00:31
aquariusfourthly: imagine that I'd tried to do all this a couple of weeks ago and got it wrong (this is why i have a .bzrignore file). If I just remove $HOME/.bzr then all my previous screwups are eliminated, yes? bzr doesn't keep my info anywhere else magic?00:32
james_win principle, but again, I think it will fail due to . existing, though it's probably worth a try00:32
aquarius*nod*00:32
james_wa workaround is to checkout in to a tempdir, then move .bzr from that tempdir to $HOME00:33
james_wyou will also need to move any files not already on the machine from the tempdir00:33
james_wjust running "bzr diff" will tell you what is missing and modified00:33
aquariusfinal question (you've been really helpful!): if I want to work on other projects hosted in bzr (say, a project currently in launchpad) and I grab a branch into $HOME/Projects or similar -- is this going to screw up because I'm putting one bzr project inside another?00:33
james_wyou can add anything else you like of course00:33
james_w4) there's no secret hidden stuff, so removing .bzr should be enough00:34
james_wthe exception would be if you had run "bzr init-repo ." in / or /home, which is very unlikely00:34
aquariusnah, I didn't do that :)00:35
james_wyou can check "bzr info" output before removing the old .bzr to make sure00:35
james_wfinally, no that's not a problem.00:35
james_wlet me find you a reference on what init-repo actually is00:35
james_waquarius: does http://doc.bazaar-vcs.org/latest/en/user-reference/bzr_man.html#repositories explain what "init-repo" is for sufficiently?00:40
aquariusjames_w: sorta, yeah. So if I'm not doing multiple branches then I don't need init-repo at all.00:41
james_waquarius: yeah, that's the main point for you in this case00:42
aquariusright, ok. makes a bit more sense now, I think :)00:42
james_wbut if you do work on a project and use multiple branches for it, then shared repositories (in the bzr sense) are a big win00:42
aquarius*nod* I can imagine.00:44
aquariusif I don't create a shared repo now (on my sftp server) and later wish I had created one after all, can I take the existing pushed-to repo and "upgrade" it to be a shared one?00:45
james_waquarius: yup, see the reconfigure command00:46
aquariuscoolness. so, it sounds to me like doing it the simple way first is the best plan and then being more complex later if I need to will work :)00:46
james_waquarius: I've never done it myself, so I can't give you the recipe, but if you do come to need it then swing back here and someone will be able to help00:47
markhlifeless: fyi, that samba issue yesterday turns out to be fairly non-deterministic - it randomly fails and works without the additional gc.collect().  Would it be useful to keep it added and to try and determine if it ever fails *with* it?00:49
lifelessmarkh: well its not a 'solution'00:50
lifelessmarkh: but if we can track down an actual cause - ftw00:50
markhyeah, hence me not knowing if that avenue is worth spending more time on...00:51
markhI figure than in the short-term, if we can get everything working correctly on native windows (ie, no attempts to move/delete open files), then we can be fairly confident we aren't leaking such handles in a major way.  Which led to me looking at those test failures yesterday I submitted a patch for00:53
lifelessright00:58
beunomwhudson, bundle branch up for review. Is pickscrape's branch ready?  Do you want me to merge it?01:19
mwhudsonbeuno: it should have breadcrumbs on all pages01:19
pickscrapebeuno: once the branch root problem is solved I need to apply it to a few other pages before it should be merged01:19
mwhudsonand the last 'crumb' shouldn't be a link01:20
beunopickscrape, lp:~mwhudson/loggerhead/directory_breadcrumbs01:20
beunoit's fixed01:20
mwhudsonand we should think about how it interacts with the --user stuff01:20
mwhudsonbut perhaps that's less important01:20
beunook, so a few more things to chew before01:20
* beuno is going office -> home bbiab01:21
pickscrapemwhudson: excellent, thanks!01:21
pickscrapeI'll get to spreading that feature out to other pages as soon as I can01:22
mwhudsonpickscrape: cool01:26
mwhudsonpickscrape: please try to find some way of extracting the common code, not copy+paste :)01:26
ToksyuryelThat seems like bad advice, except the last one.01:56
mwhudsonToksyuryel: ?01:59
Toksyuryelkomputes's quitline02:00
mwhudsonoh02:00
* mwhudson has mental filters for those02:00
Toksyuryelheh02:00
pooliemarkh, it would be interesting to do a test transport decorator that forbids moving/deleting open files02:04
markhyeah, that would be cool.  How would it work?  monkeypatching?02:05
lifelesswouldn't help with tree transform stuff02:05
lifelessbut I don't think we actually do that; I think its more reference leaks and the like due to slack gc02:06
markhI think most of the problems I found last night were caused by exceptions keeping files alive slightly longer than normal.02:07
pooliethat's true it won't particularly help with tree transform02:07
markhand most of these were fairly close to the "surface" of the API - so its only a problem in practice for a single process doing multiple bzr "commands" before terminating - eg, the test suite.02:07
pooliei guess you could either monkeypatch or otherwise intercept file open/close02:08
markhie, I haven't yet found a problem a user of normal 'bzr' would see.02:08
poolieoh i see02:08
markhthere is actually one strange bit of code in a decorator that might be responsible for the exception causing the cycle - lemme find it02:10
markhdecorators._fast_needs_read_lock() - it has a try/except/raise construct where a try/finally appears more logical.  Keeping the traceback alive in a local variable is potentially a cycle that wouldn't otherwise exist.02:12
pooliei was thinking of something similar but a bit different, which is catching on unix problems that might break windows02:13
markhthe fact the name includes "_fast" though makes we worry someone has benchmarked things or something and found that to be faster02:13
pooliespecifically relating to transport/control file stuff02:13
markhyes, I think that would be beneficial02:13
pooliesee for example fakevfat.py02:13
lifelessmarkh: the fast refers to docstrings02:13
markhsimilarly for locks ;)  Was there some conclusion to that?02:13
lifelessmarkh: and the like02:13
lifelessmarkh: did you read the dirstate RFC I posted a few days back02:13
markhlifeless: regarding the ability to avoid locks in many cases?02:14
lifelessyes02:14
poolieit seems to me we want to run most tests with the lowest-common-denominator transport...02:14
markhlifeless: it sounds good to me, but I have to admit the conversation was a little too far over my head to make any meaningful contribution :s02:15
lifelessI think we should run the entirely in memory - but we can make that as low as we want02:15
jmlhmm. running bzr selftest --no-plugins only drops about 700 tests compared to the full suite for me.02:15
* jml should write more tests for his plugins02:16
markhpoolie: fakevfat does seem to be the way to approach it02:17
markhthey even list that as the first feature to be added ;)02:17
pooliei don't know if there are any interesting behaviours in ntfs that wouldn't occur on vfat02:18
poolielifeless, i'd think we should gradually go to running on vfat+memory02:18
poolieit would be interesting to profile how many tests use which transport class02:19
markhnot many we use.  ntfs has recently grown links02:19
lifelesspoolie: ntfs won't do things worse for us than vfat02:20
markhbut - back to the locks in the short term.  Many tests on windows are failing due to an assumption locks are not exclusive - any ideas about how to handle that in the short term?  Just leave them failing?02:20
lifelesspoolie: it simply has additional capabilities we could use if we wanted [but generally explorer dies in the arse if you use them, so we shouldn't]02:20
lifelessmarkh: its in the definition of behaviour for that tree type; I don't know that there is much we can do02:21
pooliesure02:21
poolielifeless, i was thinking of things like unicode handling02:21
poolieit may not differ meaningfully to us02:21
markhwe could arrange for them to be either skipped or known failures if it was worth the bother, and if it wasn't going to stay that way for the long term :)02:21
poolieat any rate the code that's affected should mostly not use transports02:21
lifelessvfat is the same as NTFS for unicode to all intents and purposes -02:21
pooliemarkh, so one approach here is to avoid having two objects addressing the same file02:22
lifelesspoolie: so, tree rules, whats the outcome there?02:22
pooliewe're seeing the conflict aaiu because we have that aliasing02:22
markhI'm still not sure of the impact of the locks - do_merge is being called by the test suite with the same dir - are there real user scenarios that would hit this?02:23
lifelessmarkh: yes02:24
lifelessmarkh: I answered that yesterday FWIW02:24
markhoops02:24
* markh scrolls back...02:24
poolielifeless, well, i think we should still do what i said: remove the code that reads it02:24
poolieunless someone proposed an alternative?02:24
lifelessits a real bug, it causes problems and its not amenable to changes without changing the format, because of old bzr clients02:25
poolielifeless, re _buffer_all, would those repos have also OOM'd in 1.5?02:25
lifelesspoolie: no, just hadn't got the go-ahead02:25
lifelesspoolie: yes, but they won't in 1.602:25
lifelessunless we start triggering _buffer_all02:25
pooliebut at the price of getting substantially slower for things that currently _do_ work02:26
lifelessyes02:26
lifelessconstraints and accomodations again02:26
poolielifeless, i had in mind to make the api just something like _readahead_hint('per_file_graph')02:28
poolieor 'per_file_log'02:29
pooliethen those repos in this release can buffer everything, but there's no implied promise it's doing so02:29
jmllifeless: btw, I was looking around the test scenario stuff. It seems that objects in the scenario dict are shared between tests?02:29
lifelessjml: bzr's scenarios?02:30
lifelessjml: the parameterisation stuff ?02:30
pooliemarkh, re do_merge, i didn't see lifeless's specific answer but it seems possible02:30
pooliebut, i would think that locking the same object twice should not be a problem- it will just increment the internal counter02:31
markhI looked and failed to find his specific answer too :)02:31
pooliethe problem we want to avoid is having two objects pointing to the same file02:31
markh2 different objects involved IIRC02:31
lifelessmarkh: it wasn't as detailed as I just gave02:31
poolieif you can do that, there should be no problem with locks being exclusive or not02:31
lifelessthe case it occurs for users is when the working tree is two of the coordinates in a merge operation02:31
lifelessone readonly one write, it will self-exclude on windows02:31
=== kiko is now known as kiko-zzz
poolieif this is only happening in tests not in real use this may be a problem of, say02:32
pooliean api that implicitly opens a wt when it should allow the object to be passed in02:32
pooliejml, in the scenario they're just directly assigned to the test object02:32
pooliethey're typically strings or valueobjects02:32
poolieif you pass a mutable object then yes, it may cause contamination between tests02:33
lifelessjml: we could deep copy to actively prevent bugs02:33
pooliewhich would be bad ;-)02:33
lifelessjml: but we've never had mutated objects there to day02:33
jmlright.02:33
poolielifeless, that kinda sounds like an accomodation ;-)02:33
jmlI guess you guys use format specifiers and the like.02:33
lifelesspoolie: python doesn't have 'const' :)02:33
pooliei mean it's pythonic to have mutable objects and tell people to use them carefully02:33
lifelesspoolie: its also slow02:33
lifelessand many other things; we accomodate all the time02:34
poolielifeless, slow to copy?02:34
lifelesspoolie: just slow :>02:34
poolieanyhow there's no guarantee that deep copy would work correctly on arbitrary objects02:34
lifelessactually there is02:34
poolieoh you're agreeing?02:34
mwhudsonsockets, for example :)02:34
lifelessthere is a deep copy protocol02:34
* markh wonders why the merge code can't detect the same tree is being used and not bother taking the second read lock...02:34
jmlso, I'm thinking that I want to do something a lot like what TestScenarioApplier does, except have it take a scenario *factory*02:35
lifelessjml: pass the factory in as the scenario02:35
lifelessjml: ?02:35
jmllifeless: right, that's what I'll do in the short term.02:35
lifelessjml: whats the use case?02:35
pooliejml, for instance, just put a callable as the scenario parameter02:35
poolieor a class02:35
poolie(which is a callable of course)02:35
lifelesswe pass in factories all the time02:35
jmlpoolie: right, that's what I'll do in the short term.02:35
pooliejml, this would be worth documenting though02:36
lifelessso I'm keen to understand the use case that makes you think of this as a short term solution02:36
lifelessrather than the right solution02:36
pooliewant to send a one-para patch to the hacking file?02:36
jmlpoolie: maybe a bit later -- after I've successfully used it.02:36
pooliemarkh, having two objects and not locking one of them would be risky02:36
pooliealso probably futile as the object might implicitly lock itself02:37
pooliehowever, saying 'oh these should be the same object i'll fix them up' would be,02:37
jmllifeless: it's short term because in the slightly longer term I want to try doing things my way and see if it's better :)02:37
lifelessjml: there are many varied examples in bzrlib; I'd be delighted to help you choose a good one to cult from02:37
poolieum, maybe not optimal but worth looking at02:37
pooliejml, lifeless, i was wondering whether it is really tasteful to have the scenario written into arbitrary test attributes02:37
poolierather than .scenario02:37
lifelessjml: so we started with a fully general thing that was called per function if I remember correctly02:38
pooliefor instance02:38
markhpoolie: but if its an oslock, its only purpose is to keep others out isn't it?  And as we work correctly when the locks aren't exclusive, I don't see much risk.  But I'm sure I'm just blinkered :)02:38
lifelesspoolie: I think its tasteful; you can always have a scenario that puts everything on .scenario02:38
* jml disagrees02:38
jmlbut taste is a matter of... taste.02:38
lifelessjml: which statement do you disagree with ?02:39
markhI can imagine all the decorators failing though...02:39
pooliemarkh, you might get two in-memory objects that are out of sync with each other02:39
lifelessmarkh: no, they are exclusive02:39
pooliebut, presumably it already has to deal with this on unix02:39
jmllifeless: I disagree that it's tasteful.02:39
lifelessjml: to write to any attribute?02:39
markhyeah - that's what I'm a little lost with still...02:39
jmllifeless: yeah.02:39
lifelessmarkh: so, on unix they are exclusive *cross process*02:40
pooliemarkh, btw, today's my last day here, do you want to catch up?02:40
lifelessmarkh: its a special case that it doesn't exclude the same process02:40
pooliei do have a bunch to do02:40
markhlifeless: right - so in process you could theoretically detect and avoid 2 locks on the same file?02:40
lifelessmarkh: not locking would allow two bzr processes to stamp on each other on windows *as well* as allowing a single-process to behave as bzr on linux does02:40
markhbut I'm only saying not to lock when we already have a write lock02:41
lifelessmarkh: I guess you could reinstate the global dict we used to have02:41
lifelessmarkh: but IIRC we took it out cause it caused other bugs02:41
markhbut why not just compare paths, and if we already have a write lock on the same path, just don't take the read lock.  Could theoretiucally be all detected in do_merge02:41
poolielifeless, do you agree that the general approach is to try to have one in-memory object corresponding to each real-world or fs object?02:42
lifelessjml: I think forcing the core code to only write to one attribute is not treating callers like adults; and it doesn't stop people having all the issues arbitrary attributes have in micro02:42
lifelesspoolie: no, I don't agree with that02:42
poolieok02:42
markhit would just solve the problem for a single "Merger" object, that that seems the only real use case that its a real problem with02:42
lifelesspoolie: I think its good when we do that but aliasing can trivially stop that working, so we need to be correct regardless02:42
lifelesspoolie: consider z: and f: mapped to the same SMB drive, for instance.02:43
pooliesure02:43
lifelessjml: I will agree that as a convention having stuff on .scenario is good for documentation and predictability; but thats up a layer in the 'userspace' of the facility02:43
lifelessmarkh: I think we should just fix the underlying bug and issue a new recommended format for windows02:44
jmllifeless: well, actually, I think that the nicest way would be to call a method on the test.02:45
lifelessjml: to set stuff?02:45
jmllifeless: yes.02:45
lifelessjml: kind of like _setattr_ but different?02:45
jmllifeless: kind of like setScenario(scenario)02:45
lifelessjml: I think that would be great; I also want python unittest compatibility02:46
lifelessjml: but doing a getattr for setScenario first would be fine IMO02:46
jmllifeless: of course, then the way you set attributes would have to match whatever the default implementation of setScenario does.02:46
pooliejml, that seems pretty close to just poking into .scenario_* then letting setUp() run...02:47
poolielifeless, yeah i was thinking of .scenario* just as a convention not a requirement02:48
jmlpoolie: well, the difference is that the test decides what to do with the scenario.02:48
lifelessjml: yes, but hopefully both are in the same codebase02:48
jmllifeless: I've got use cases for doing otherwise.02:49
lifelessjml: you're making me guess here02:49
pooliei guess in my experience it's often just storing them02:49
pooliebut your cases would be interesting02:49
jmllifeless: bzr plugins02:49
jmllifeless: like bzr-loom02:50
pooliejml, out of curiousity, did addCleanup go from bzrlib to pyunit3k?02:50
jmlwait, wrong example02:50
jmlpoolie: no.02:50
markhlifeless: I'm not sure exactly what the bug is?  I got the impression from yesterday that they were *intended* to be exclusive?02:50
jmlpoolie: I implemented addCleanup using test-driven development for Twisted and then re-implemented it with TDD as a patch to Python02:50
jmlpoolie: the one in pyunit3k is from the latter.02:50
lifelessmarkh: WT4 is designed with a built in flaw; its a design bug we need to fix02:50
markhand that would impact the recommended default format for windows?02:51
jmlpoolie: the addCleanup in bzrlib isn't tested, per se.02:51
lifelessmarkh: for everyone02:51
markhor you mean we'd end up with a wt5?02:51
markhright02:51
pooliereally?02:51
lifelesswt502:51
jmlpoolie: really no tests.02:51
lifelessmarkh: 'bzr commit' -> opens editor02:52
lifelessin the editor, !bzr diff, or similar02:52
lifeless-> FAIL02:52
markhright - you mean implement your RFC?02:52
lifelessyes02:52
markhgotchya - cool02:52
=== RAOF_ is now known as RAOF
markhwhich will still leave us with the longer term problem of how to disable the existing tests once this new wt comes out - the old tests aren't going away...02:54
jmlpoolie: the Twisted one allows cleanups to return Deferreds and ensures they are run in serial, fwiw.02:54
markhdisable on windows02:54
markhbut that can wait ;)02:54
jmlpoolie: and as such is way more complex than any other implementation I've seen.02:54
pooliejml, i meant did the idea come from bzrlib or was it independent reinvention02:54
jmlpoolie: oh, the idea is from bzrlib.02:55
lifelessmarkh: we'd add a feature02:55
lifelessmarkh: and requireFeature those tests02:55
markhahh - nice clue - thanks!02:55
jmlpoolie: in fact, I don't know of anyone else who's done it apart from bzrlib and myself.02:55
lifelessthe feature would be something like02:55
lifelessBrokenInProcessLocksFeature02:56
lifelessjml: I think that bzrlib is one of the drivers of testing facilities in unittest tests in python02:56
poolielet's use a more specific word than 'Broken'02:56
lifelessjml: other than e.g. nose which seems half cool half terminally confused02:57
lifelesspoolie: :)02:57
lifelesspoolie: I tried to think of one, to represent what linux os locks do02:57
lifelesspoolie: but nothing other than 'fucked' and 'broken' came to mind02:57
lifelessso I went polite02:57
poolie<stephane> don't goof off too much02:57
poolie^^ at me laughing at you02:57
markh:)02:57
lifeless:)02:57
jmllifeless: well...02:57
jmllifeless: Twisted and Zope have both explored new testing territory.02:58
poolieProcessWideFileLocks02:58
jmllifeless: Zope, for example, has definitively shown that layers are a terrible idea.02:58
pooliejml, congratulations on the user reviews of the code review feature!02:58
jmllifeless: that's progress.02:58
pooliemarkh, so do you need anything from me before i go?02:59
poolie(at about 5pm today)02:59
poolieit looked like the questions about packaging were fairly well answered?02:59
markhpoolie: yeah, thanks.  Over that period I ill have binaries available and I'm not sure what I should do with them exactly :)  Current plan is to mail the mailing list and host them on that starship server I have access to.  Anything else I should do with them while you are gone?03:00
poolieyou can put them up as file downloads on launchpad?03:01
poolies//can't you/03:01
poolieyou might need to join the right team there...03:01
markhyeah - will that give them much visibility?03:01
pooliehttps://edge.launchpad.net/bzr/+download03:01
markhyeah - that's what I'm thinking - I'm still a total newb with launchpad03:02
poolieand then link them from the /Download page on the main site03:02
pooliedo you get an 'add download file' link there?03:02
markhthe fact mailing lists were attached to teams is what threw me before03:02
lifelessjml: :)03:02
lifelessjml: I meant positive directions03:02
lifelessjml: we're still on for sunday?03:02
pooliemarkh, yeah that's one of those possibly too clever ideas03:02
jmllifeless: I thought it was Saturday03:02
lifelessjml: ah yes, it is03:02
lifelessI was thinking with my wow-addict-hat for a second03:03
lifelessjml: btw, can you log in sometime and hit return on the stuff in your mailbox ;)03:03
markhpoolie: no "add download file" link03:03
jmllifeless: to which machine?03:03
pooliewhat's your lp userid?03:04
lifelesscaelstraz03:04
markhmhammond03:04
poolielifeless, jml, i assume you wouldn't object to me adding him to ~bzr?03:04
lifelessnot at all03:04
lifelessthough perhaps you meant jam & abently there :)03:05
jmllifeless: oh, right. I guess so.03:05
pooliemarkh, try again?03:06
thumperis there any easy way to check the revno of the public branch?  like `bzr revno -d :public`?03:07
* thumper waves hands around03:07
spivthumper: "bzr revno :public"03:08
thumperspiv: thanks#03:08
markhpoolie: awesome, thanks!  You now have my permission to enjoy yourself for a couple of weeks on that BMW riding north! :)03:09
poolie:-)03:09
jmlspiv: bzr://rhino.local/addCleanup-args03:13
thumperabentley: I don't suppose bzrtools has a command that does the equivalent of "branch X into shared repo, then do a lightweight checkout" ?03:14
abentleythumper: Sounds like cbranch.03:14
thumperabentley: can I tell cbranch to --hardlink from a different working tree though?03:14
thumperabentley: normally I go `bzr cbranch local-a local-b` and that's exactly what I want for that use case03:15
abentleythumper: Yes.  --files-from03:15
thumperabentley: but if I want to go `bzr cbranch lp:~a/b/c` that works?03:15
abentleythumper: Yes, you can cbranch from any location that bzr understands.03:16
thumperabentley: you rock!03:16
abentleybtw, you need to supply *both* --hardlink and --files-from if you want that.03:16
thumperabentley: gotcha03:17
thumperabentley: I have cbranch aliased to `cbranch --lightweight --hardlink`03:17
abentleyMe too.03:17
abentleythumper: If you forget to supply --files-from, you can use "bzr link-tree" to hardlink after the fact.03:19
spivHmm, "bzr diff -r submit:" is including the changes in the common ancestor revision in the diff.03:19
spivThat's weird.03:19
thumperabentley: and that converts copies to hard links?03:19
abentleythumper: Yes.03:19
thumpercool03:20
thumperis link-tree in bzrtools or trunk?03:20
thumpers/trunk/core/03:20
lifelesspoolie: there is a squid meetup in sydney while you are away03:20
lifelesspoolie: ok if I pop down there for a bit? not sure how much time I want to spend; its monday and tuesday next week03:21
spivAh, it's because my mirror of bzr.dev is older than the branch I'm making a diff of.03:24
lifelessbeuno: around still?03:27
poolielifeless, np03:34
pickscrapemwhudson/beuno: ping04:14
beunolifeless, pong04:34
beunopickscrape, pong04:34
lifelessbeuno: hi04:37
beunohi lifeless04:37
lifelessbeuno: uhm, what was it, oh yeah - does loggerhead have a way to get favicon.ico from a branch ?04:37
lifelessbeuno: wondering if I should file a bug for that04:37
pickscrapebeuno: hi. I was looking to move the breadcrumbs markup out into a shared place. It looks like metal is what I'm after04:38
beunolifeless, I've seen your comments on the bug.  What do you mean "from a branch"?  from the actual tree?04:38
beunopickscrape, yes, or in macros.pt04:38
pickscrapeThere is a template there already called 'macros.pt', but it contains a full-HTML macro and it doesn't like it if I try to define any outside of the html element04:38
beunobut that will just make rockstar move them to a separate tempalte later on04:38
pickscrapeAh, he's unifying or something like that isn't he04:39
lifelessbeuno: I mean from the branch, so for non-launchpad installs of loggerhead04:39
beunohe's spliting things up so it's more themeable04:39
beunolifeless, currently LH gets it's favicon statically from it's own dir04:40
beunoyou would want to add a favico.ico to each dir of each branch?04:40
pickscrapefavicon just gets served from root doesn't it?04:41
lifelessbeuno: sure, why not :)04:41
lifelesspickscrape: I think documents can override per-document04:41
beunopickscrape, I can help you place it in the templates, if you get the python code in place to spit out the content04:42
beunolifeless, no reason why not, it would make projects more personalized. Although I wonder how many people will go through the trouble of creating one per project/branch04:42
pickscrapebeuno: I'm actually keen to learn how this works myself. I've been playing around and have actually got it working from a separate macros template, though I'm sure that's not ideal04:43
beunolifeless, I'm all for it, just not on the top of the list right now. I agree with you  :)04:43
lifelessbeuno: well, if it was (say) .favicon.ico, I'd happy put one in most of my projects04:43
beunopickscrape, separate macros template?  embeded in each template?04:44
pickscrapeI created a breadcrumbs.pt file in the same location as macros.pt, and also had to add a line to loggerhead/templatefunctions.py to get it to recognise it04:45
beunopickscrape, that sounds correct  :)04:45
rockstarpickscrape, the "unify" was because the branch was started with a different idea then what actually came out of it.  :)04:46
taconefavicon can be specified in <head> as well04:46
pickscrapeIs that the approach that rockstar will be taking?04:46
beunopickscrape, yes, that's why it sounds right  :)04:46
rockstarYes, absolutely.04:46
rockstarThe idea is that someone might want to put the breadcrumbs somewhere else.04:46
pickscrapeOh sweet. I was concerned that calling it 'breadcrumbs' was being a bit too granular.04:46
rockstarSo they should just have a template function that generates the  breadcrumbs, and they can call that wherever they like.04:47
pickscrapeYes, that makes sense04:47
pickscrapeIn that case it doesn't sound like I'll be conflicting with rockstar as much as I was afraid I might be.04:48
rockstarbeuno, are you gonna merge my branch before or after 1.6?04:48
beunorockstar, not at all, ASAP04:48
beunoso, before04:48
rockstarCool.04:49
beunoanything that's laying around and intersting goes in. I just don't want more features, or it will be like bzr's 1.604:49
* beuno ducks04:49
rockstar:)04:51
lifelessbeuno: so I should file a bug?04:54
beunolifeless, haven't you already?  Where have you been ping-ponging with poolie about this then?04:56
lifelessbazaar-launchpad, different use case04:56
lifelessalmost entirely unrelated in fact :)04:57
* beuno looks04:58
beunoah, right. I pictured it in my head on how I'd implement it, so I ended up thinking it was on LH  :p04:58
beunolifeless, sure, go ahead04:58
pickscrapenight all05:00
lifelessdone05:00
thumperbeuno, rockstar: I've commented on the review, and I think they should be looked at before merging05:00
mwhudsonlifeless: oh, i thought your bug report was about loggerhead too :)05:01
beunothumper, just saw it and replied. I fully agree with most of it. Does answering by email automagically work?05:01
thumperbeuno: it should do :-)05:01
beunopickscrape, g'night05:01
beunothumper, it did, yay!   Now I like reviews better05:01
mwhudsonoh, it was05:01
thumperbeuno: :-)05:02
thumperbeuno: you can thank abentley05:02
* beuno thanks abentley05:02
abentleybeuno: just doing my job05:03
beunoabentley, still worth thanking you  :)05:03
beunohm, LP's sends me back the email I sent as a review05:06
* beuno takes back 1/8th of his thank you from abentley 05:06
abentleybeuno: Mailing lists generally do that.05:07
beunoabentley, isn't it disabled on most of em?05:07
lifelessbeuno: no05:08
* beuno can't think of any lists he's on that work that way05:08
abentleybazaar@lists.canonical.com, bzr-gtk@lists.canonical.com05:09
beunothe bzr ml doesn't send me back my own emails AFAIK05:10
abentleybeuno: It sends me mine.05:10
mwhudsonit's a per-user option in mailman05:10
spivMailman lists let subscribers opt out of having their own messages sent back to them.05:10
beuno(I'm not actually complaining anymore, just truthfully curious now)05:10
spivBy default those lists will send them back, I believe.05:10
abentleybeuno: I find it essential to have the mail sent back to me so that I can read mail threads I've participated it.05:11
abentleyparticipated *in*05:11
beunohm, I wonder if it's gmail "doing stuff for me" then05:11
lifelessabentley: speaking of this05:13
lifelessabentley: I find I miss reviews when folk use BB's web interface because it doesn't mail me05:13
lifelessabentley: how do you feel about reimplementing mailmain in turbogears/05:13
lifeless?05:14
beunoROFL05:14
lifelessbeuno: gmail hides N>1 instances of a mail with the same id05:14
abentleyYou miss reviews because they aren't addressed to you personally?05:15
lifelessyes05:15
beunolifeless, ah, ok. So I've been protected from reality up to today.  Why doesn't this apply to LP then?05:15
lifelessI send in a patch, someone reviews on the web, I notice a few days later when I'm doing a list-scan or BB web UI check05:15
AfC"I've been protected from reality". Heh. I'm going to steal that.05:15
lifelessbeuno: LP sends its mails with a different id05:15
lifelessbeuno: rather than as a bounce05:15
abentleylifeless: I am too busy reimplementing Mailman in launchpad to reimplement it in TurboGears :-)05:16
abentleybeuno: This is arguably a bug.05:16
beunoabentley, yeah, it at least felt odd to my normal workflow.  Mind if I file it then?05:17
abentleyNot at all.  It should have had the same id.05:17
lifelessabentley: is it actually the same content?05:17
abentleylifeless: More or less.  May have a header prepended or a footer appended.05:18
lifelessthen I agree05:18
lifelessFWTW05:19
beuno\o/05:19
lifelessthumper: hi05:20
beunoabentley, I found bug #24056405:21
lifelessI keep asking this05:21
lifelesshttps://bugs.edge.launchpad.net/bzr/+bug/205416/+addbranch05:21
lifelesscan we make the whiteboard be shorter so I don't have to scroll on that page?05:21
beunofixing that would fix this too, wouldn't it?05:21
beunohttps://bugs.edge.launchpad.net/launchpad-bazaar/+bug/24056405:21
lifelessthumper: if you say yes I'll file a bug05:21
lifelessthumper: also, at least for me, the common case is 'fix available' not 'work in progress', so could we change the default too ?05:21
thumperlifeless: please file a bug, and assign it to rockstar05:23
thumperlifeless: I know you've been asking, but I've been implementing something else you asked for ;-)05:23
rockstarYes please.  Assign all bugs to me.05:23
rockstar:)05:23
thumperrockstar: something for you to do :)05:23
rockstarthumper, job security  :)05:24
thumper:)05:24
mwhudsonwatch out, at current rates our team will run out of work ... in 2120 or so05:25
lifelessthumper: rockstar whats your lp id ?05:27
rockstarrockstar05:27
rockstar:)05:27
rockstarmwhudson, after my talk on LP and bzr tonight, people were telling me all about the bugs they've discovered, as if I was the bug tracker myself.  :)05:28
mwhudsonyeah, people do that05:28
beunoabentley, so, still file a bug inspite of https://bugs.edge.launchpad.net/launchpad-bazaar/+bug/240564 ?05:33
beunoit's the last thing I'm doing before *really* going to bed05:33
abentleybeuno: Yes.  This bug is "The message id isn't being preserved when handling incoming mail"05:34
abentley240564 is "Only code review messages are threaded, not other messages about the merge proposal".05:35
beunoabentley, thanks, filed as #25749405:36
beunoI'm really off now, g'night everybody05:37
abentleybeuno: g'night.05:38
lifelessgnight beuno05:39
Peng_This is late, but good night.05:44
abentleylifeless: If Bundle Buggy CC'ed the submitter, would that stop you from missing reviews?05:49
lifelessabentley: yes indeed05:53
abentleylifeless: I've got Mailman configured not to mail me duplicates.  So I think if it sent a message to bazaar@lists.canonical.com that was cc'ed to me, Mailman would not forward it to me.05:55
abentleyThat would mean we've covered both cases without having to introduce any per-user settings on BB.05:56
lifelessabentley: cool06:40
lifelessspiv: bb:tweak on the testing docs btw06:45
lifelessspiv: I think I forgot to say06:45
spivlifeless: oh cool, thanks06:45
lifelessspiv:07:08
lifelessif I could ask for a review, my InterTree + remove tweaks would be a lovely pair to do07:08
spivlifeless: I'll take a look.07:10
poolielifeless: hm this might have been faster if i'd re-read your rfc before we talked for so long07:12
poolieit's good07:12
lifelesspoolie: :)07:12
lifelesspoolie: thats good to hear07:13
pooliehm07:21
poolieso it's basically like being split by directory, except not necessarily by directory07:21
pooliei'm not sure if explaining that way is going to cause more or less confusion07:22
lifelesssplit by directory achieves some of the goals07:22
lifelessbut conceptually its what a lot of people reach for first07:22
poolieright07:22
lifelessI think its important that we explain why that isn't sufficient; and also why its good so that we can use the good bits07:24
lifelessthe good bits are having a canonical form, being updatable in < tree07:26
lifelessthe bits that are not sufficient are being == dir size, and pathological cases have such a strong habit of turning up07:28
lifeless(also not mapping fileids -> paths at all well)07:29
pooliethat last one may be a challenge for many designs07:29
lifelessif its too hard its too hard07:30
lifelessbut trivially consider three things07:30
lifelessa path->id index07:30
lifelessa id->path index07:30
lifelesscontent07:31
pooliesure07:32
lifelessmaking the indices outside the core data is possiblty doable07:33
lifelessand another thing to consider07:33
lifelessif we put them outside and use a parallel design to implement the index, it might make the 'core' smaller but is it simpler07:37
lifelessright now we have no path<->id indices, instead we derive them on th efly07:39
lifelesswe could do that too07:40
lifelessas long as there was an index on (say) basename<->fileid, and fileid->content07:40
lifelessthen path-> content is 'basename) -> fileid set, repeat lookup on fileid->content until a parent  mismatches07:46
pooliehm07:56
pooliei do feel very excited about trying to write such a thing07:57
poolie:/07:57
lifeless:)07:57
lifelessthats a good sign07:57
lifelessit indicates the design sounds attractive07:57
pooliehm08:13
pooliei want to talk more about how to prototype it but really should not08:13
lifeless:>08:14
lifelessthanks for the feedbackm its was good08:16
lifelessI have replied08:16
=== mark1 is now known as markh
pooliei'd be inclined to start by prototyping something that takes an in-memory inventory and writes it out in a new form08:16
lifelessyes, that would give you a canonical form08:16
lifelessthe converter has to do that anyhow :P08:17
pooliesue08:19
pooliesure*08:19
pooliei mean that might be an interesting place to start08:19
lifelessI am inclined to start by crisping-up the interfaces for diff etc08:19
lifelessso that we pass inventory deltas around more08:19
lifelessrather than upcasting dirstate to an inventory08:20
lifelessso diff -r -3 internally becomes08:20
lifelessa call to diff -3..-1 updated by -1..current:08:20
lifelesspoolie: if you do want to chat, I am around for 30 more minutes, raid is at 608:23
pooliei have some more mail to send and then need to clean up here08:23
pooliesadly08:23
pooliei kind of wish i was disconnecting with a laptop and less interruptions08:23
lifelessheh08:24
lifelesswould be less of a holiday wouldn't it :)08:28
robstahi09:11
robstais it possible to push a sequence of bzr commits to an svn repo as a single commit?09:12
james_wrobsta: hi, it's not as far as I know, you need to do a squash merge first.10:02
lifelessrobsta: do you want to discard the history, or to make it look like a single merge?10:04
robstamake it look like a single commit in svn10:05
lifelessrobsta: bzr checkout svn:/// fo10:10
lifelesscd fo10:10
lifelessbzr merge <your branch>10:10
lifelessbzr commit10:10
lifelessthat will result in a single commit to svn10:10
lifelessyou'll want the latest bzr-svn which has a raft of bugfixes10:11
robstaok, so i need two branches10:11
lifelessjelmer announced it monday or something10:11
robstafair enough10:11
* robsta wondering how to best leverage the combination of gnome's svn and bzr-playground10:14
lifelessspiv: ping10:18
rjekIncase it's not been reported before; http://rafb.net/p/cqMCzd70.html10:51
rjek(bzr explosion when doing bzr vis with no DISPLAY on 1.3.1)10:51
lifelessfun10:52
lifelessplease file a bug?10:52
rjekSorry, I don't have an account on Launchpad.10:52
lifelessoh, ok10:53
lifelessuhm, perhaps a mail to the bzr-gtk list?10:53
lifelessyou don't need to subscribe, moderator can let it through10:53
rjekRight, OK.  I'll do that.10:53
rjekDo you happen to know its address, or an URL of a page that describes it?10:54
lifelesshttp://bazaar-vcs.org/bzr-gtk10:54
rjekTa.10:54
rjeksent!10:56
rjekThanks.10:56
Peng_BB down?12:04
Peng_Oh, good, just slow.12:08
=== AnMaster_ is now known as AnMaster
markhI think I'm still missing something conceptually to do with merging. Let's say i have 3 branches, 'bzr.dev', and 2 'work' branches 'bzr.1' and 'bzr.2'.  I periodically update bzr.dev then pull from 'bzr.dev' into 'bzr.1' - but in the meantime I also make a number of changes in bzr.2, then 'bzr merge' them into bzr.1 and commit them (into bzr.1).  When a do 'bzr log' of bzr.1, all I see is a single 'merge' commit, and not the individual comm12:57
james_wmarkh: even with "bzr log --long" (the default, but you may have it aliased)?12:59
markhjames_w: yeah13:00
james_wand I assume you didn't do "bzr revert --forget-merges" at any point?13:01
james_wand the merge wasn't a cherry pick?13:01
james_wi.e. it was just "bzr merge ../bzr.2" not "bzr merge -rx..y ../bzr.2" ?13:01
markhactually, when I first started looking at this, the initial merges were '-r x..y' - but even then I'd expect to see the logs for r..y?  I'm experimenting now with trying to stitch 2 related branches together without a '-r' though...13:04
=== kiko-zzz is now known as kiko
james_wif you use "-rx..y" then it's a cherry pick, and so all of the commits will be collapsed13:04
markh(for x..y)13:04
james_w-ry or no -r won't do that13:04
james_wcould you pastebin a transcript showing what you did, and the resulting log?13:05
markhthis is all kinda "post-mortem" - I do need to try some controlled experiments (or find clear docs :)13:07
markhyeah, merge without -r does what I expect.  So maybe I'm confused about how to make 2 branches "re-converge", so a simple 'bzr merge' keeps them up to date even when they previously diverged.  IIUC, 'bzr pull --overwrite' will lose history too.  I'm still too confused to phrase a reasonable question though :-S13:24
james_wyeah, pull --overwrite is the wrong thing13:26
james_wif you "bzr merge" one way you can "bzr pull" the other way afterwards, then the branches should be mirrors13:26
james_wif you run "bzr pull" and it succeeds then the two branches are mirrors13:26
james_w"bzr merge" allows you to bring in the changes from another branch regardless of whether a pull would succeed13:27
markhbut once I've done a merge, how do I then get a pull to work again in the future once things are back together?13:30
markheg, I checked in locally, then sent a bundle which was merged to bzr.dev - but in the meantime, I've had to "bzr merge" from bzr.dev.  But now it was merged upstream, what do I do?13:30
markhstart a new branch?13:30
markh(but in the meantime I've also merged various other branches in - although everything in bzr.dev is now in my branch)13:31
markheg, I've 3 outstanding merge requests with bundle-buggy - all these are already in my branch that I'm otherwise keeping up-to-date with bzr.dev.  hence my general confusion :)13:32
awilkinsHere's a thing ; the smart server requires the dumb server to work13:34
awilkinsOn modern webserver ; fine, on IIS 6 ; annoying13:34
awilkinsHow much work would it be to get the smart server to just serve the files that normally come from the dumb server?13:35
Peng_The smart server already provided VFS access, so I think it should be *possible*.13:36
Peng_The ideal solution is for the smart server to do everything through RPCs.13:36
awilkinsIIS7 works fine13:36
awilkinsIIS7 has a way of catching /smart links though13:36
awilkinsIIS6 only likes directing things with dotted file extensions to ISAPI modules though13:37
markhyeah - merging into a new tree seems to be much closer to what I expect.13:41
markhit avoids the need to cherry pick.  I think trying to stick to an initial "base tree" that has diverged from public branch is the problem13:41
markh(and it seems I also get to learn about "criss-cross merging" :)13:43
james_wmarkh: if you are merging multiple things in to a branch, including bzr.dev then you won't be able to resynchronise to bzr.dev, but you can keep merging it13:43
markhi've got myself into the state where the only merge that works is to cherry-pick - which collapses the merges as we mentioned.13:46
james_whmm, I'm not sure why that would happen13:46
markhI think I'm looking for the "--oh-yeah-i-know-i-merged-something-silly-before-but-please-just-reconcile-things" option ;)13:47
james_wheh :-)13:47
james_wdoes the command say that you are not allowed to do it, or are you hitting an error?13:47
james_wthe only way I know of to prevent merge working is to have no common ancestor, which shouldn't be the case here13:48
markhit actually just ended up merging more than I wanted.  things got very confusing when I ended with with both 1.6 and .dev branches13:48
james_whmm, interesting13:49
markhso I've got patches I'm submitting against bzr.dev, but still applying locally to a 1.6 based branch13:49
markhstarting with a clean, new branch up the upstream, then merging in the various ".work" branches seems to be working fine.13:50
markhI think I just got myself working on the "rhs" of a branch, not the left.13:50
markh(or something like that :)13:50
markhs/branch up the upstream/branch *of* the upstream/13:50
markhI've now got 6 branches based, at some stage in the past, on bzr.dev13:51
markheach with their own changes, each less or more important13:52
markhso each now targetting either .dev or both .dev and 1613:52
=== abentley1 is now known as abentley
=== abentley1 is now known as abentley
=== abentley1 is now known as abentley
ToraninHow do you get a proper log of bzr selftest results?  Piping to tee seems to truncate the test names to 80 characters wide, which is not really useful.15:04
jammarkh: I might be able to add a bit more to the locking discussion if you are around15:06
epsyhi, i've just filed a bug, and i'm not sure what priority to give it15:16
epsyit affects storage(dirstate), blocks me from using the repository, but is something that could only happen in a very specific setup15:17
epsywell, i think it could only happen in a very specific setup, that is15:17
james_wepsy: I'll give it a priority if I triage it15:18
epsyok15:18
james_wepsy: drop the number here and I can do it now15:18
epsyhttps://bugs.launchpad.net/bzr/+bug/25759915:18
ubottuLaunchpad bug 257599 in bzr "exceptions.AssertionError: no parent entry" [Undecided,New]15:18
epsyheh, nice15:18
james_wToranin: are you sure it's not what you are using to view the log that is truncating?15:19
james_wepsy: that's a dupe I think, let me find it15:19
Toraninjames_w: fairly sure but I'll try it again -- I'm just doing (more or less) bzr selftest -v | tee testlog15:20
epsywas the other report resolved?15:20
james_wepsy: yeah, it will be fixed in 1.615:20
epsyis there any way i can fix my repo or should i trash it?15:20
Toraninoutput looks like:15:20
Toraninrunning 14456 tests...15:20
Toranin...hon2.5/site-packages/bzrlib/doc/api/branch.txt)   OK                1294ms15:20
james_wepsy: there's a way to fix it15:22
epsyphew :)15:22
james_wbug 15043815:23
ubottuLaunchpad bug 150438 in bzr "corrupt inventory deltas are not detected by dirstate" [High,Fix released] https://launchpad.net/bugs/15043815:23
Toranin"COLUMNS=$COLUMNS bzr selftest -v | tee testlog" <-- this widens it out to my terminal width in the log at least.  218 ought to be enough for most test names, I think15:23
james_wepsy: I think it's "mkdir dir; bzr add dir; bzr st; bzr rm dir"15:23
james_wwhere "dir" is the directory that was removed that it is complaining about15:24
epsythe way to reproduce it or the way to fix it?15:24
james_wthe way to fix it15:24
epsyok15:24
epsyin my case, it's images/avatars/ that i have to add again, right?15:26
epsyGreat, it works! thank you very much :)15:29
james_wno problem15:39
=== kiko is now known as kiko-phone
jonnydeelifeless: Just wanted to let you know it seems like you have found a bugfix for https://bugs.launchpad.net/bzr/+bug/255656 :)15:58
ubottuLaunchpad bug 255656 in bzr ""bzr: ERROR: [Errno 22] Invalid argument" when "bzr pack" is executed manually or when "autopack" is triggered on a repository located on a windows network share" [Undecided,New]15:58
jonnydeeThank you very very much!!!!15:58
=== kiko-phone is now known as kiko
jambeuno: ping16:29
beunojam, pong16:29
jambeuno: were you the one that helped me make deb releases?16:29
jammarkh: ping16:31
beunojam, yeap. You ready to release?16:31
jamNot quite yet, but I just wanted you on hand because it makes my life easier :)16:31
jamI'll *probably* do a 1.6rc2 today if at all possible16:31
* Peng_ predicts 1.6rc7 will be released on October 24.16:33
jamPeng_: :) Martin is gone so I have release powers now16:33
jamAnd I'm planning on forcing it out the door.16:33
Peng_I'll believe it when I see it. :P16:33
pickscrapeWasn't this a long release in order to get something in for the gnome project? Was it successful?16:33
Peng_Are you sure you don't want to wait for groupcompress to be merged?16:34
* Peng_ hides.16:34
jam:)16:34
beunojam, I'm here for the next 8 hours or so  :)   probably more16:34
cody-somervillebeuno, :-)16:34
jamanyone know what markh's hours are? He seems to be only online late at night or early morning CST time, which seems a bit odd for someone in the US16:34
Peng_jam: I'm in the U.S.16:35
Peng_Maybe he's just crazy like me.16:35
jamPeng_: you're online late morning :)16:35
Peng_Yeah, my schedule is slipping.16:35
beunojam, isn't he in australia?16:35
james_wyeah, that's what I thought16:36
jamWell, according to this: http://www.swfwmd.state.fl.us/about/staff/hammond.html he is in Florida (yes I know that's not him :)16:36
jamIt was never clear to me where he lived, you could be completely right.16:37
jamgreat, on non win32 WindowsError isn't defined, so you can't just blindly put it in your exception handler... :(16:38
jamah, but WindowsError is a subclass of OSError16:39
beunojam, his LP page says australia: https://edge.launchpad.net/~mhammond16:40
LarstiQjam: he's in .au16:51
jamLarstiQ: you know, for how active you are  in #bzr, I'm surprised I don't see you at all on the mailing list :)16:52
LarstiQjam: that's scheduled for september16:54
LarstiQjam: I have been reading some mail again for starters.16:54
jamwhat's in Sept? You start school again ?16:54
LarstiQstop school actually.16:54
jamabentley: ping17:19
abentleyjam: pong17:19
jamabentley: In the interests of getting 1.6rc2 out today, I want to make sure I've evaluated everything that people want to land17:20
jamAnd I'd like a bit more info why your patch is necessary for 1.617:20
abentleyjam: It fixes readability of "info" for branches in the new formats.17:21
jamwell, I'm getting something even weirder here17:22
jamusing bzr.dev17:22
jamI get a traceback with17:22
jambzrlib.errors.UnknownFormatError: Unknown branch format: 'Bazaar Branch Format 7 (needs bzr 1.6)\n'17:22
jamthough that might be my 'nick' script17:23
abentleyjam: I can't reproduce that with bzr.dev17:24
jamabentley: And this looks perfectly readable: http://rafb.net/p/Or1iue87.html17:24
jamThe indentation is a little odd, I guess17:25
jambut not unreadable17:25
abentleyjam: It gets away with it because it comes last.  If both branch and working tree formats were multi-line descriptions, it would be very hard to parse.17:27
abentleyBut it's always been a single line in the past, and I believe that's what people want and expect.17:27
jamtrue, though arguably info-v should be updated to support multiple lines.17:27
jamI kind of like the more verbose description17:28
jamI'm willing to merge what you have17:28
jamI'm just not sure I agree 100%17:28
abentleyjam: It's a very poor answer to the question "What is my repository format?"17:28
abentleyjam: additionally, the docstring for the RichRoot variant is flat-out wrong.17:29
jamhmm... I find it a pretty good response17:29
jamI agree that RichRoot is just wrong.17:29
abentleyjam: Not in my opinion.  It's not the reply I'd want on IRC, or when discussing things in email.17:30
jamabentley: well *I* would want "RepositoryFormatKnitPack5" as that fully encapsulates it ,and is short for IRC, etc.17:31
jamI don't think that is suitable for "bzr info" though17:31
jamso I'm not sure if that is the best test17:31
abentleyIt's a fine reply to "What does my repository format support, etc"17:31
abentleyjam: So how can people find out what their repo format is, if not through "info"?17:32
jamabentley: well plain "bzr info" gives: Shared repository with trees (format: pack-0.92)17:32
jamIf we were going to do it *right*17:32
jamthen we should be using the short names for them.17:32
jamfor some definition of "right" because of the permutation problem17:33
abentleyIt shouldn't say pack-0.9217:33
jamabentley: that was a different repo17:33
jamShared repository with trees (format: 1.6)17:33
jamIf you prefer17:33
jamI'm really tending on the "get 1.6 released" already and postponing everything that isn't an actual regression.17:34
jamBut if you feel strongly, I'll merge it17:34
abentleyMany are unhappy with using the short strings.17:35
abentleyEspecially when you're talking about a shared repo or checkout where it could be one of many.17:35
abentleyThe current docstrings are unacceptable to me because they don't say "RepositoryFormatKnitPack5" or something like that anywhere, and because they make false statements.17:36
abentleyI don't insist on fixing the multi-line thing for 1.6.17:37
abentleyThough of course, the simplest thing to do would be to apply my patch.17:39
jamabentley: so you *do* feel like the strings need to be updated for 1.617:39
jamI'll probably just merge your patch17:39
jamyou know, I wish Thunderbird wouldn't include BB emails when I search by your name17:40
jamI think it is because bb's email address includes yourname.com17:41
abentleyYeah.  Sorry.17:41
jamIf I use your full name with a space, then BB is filtered out17:41
awilkinsjam: Would your proposed consideration of kind == 'file' to kind == None as a content change mean that the deltetion of a file would show up as a revision it participated in, in the log (desirable, IMHO)17:44
jamawilkins: something else entirely17:45
jambzr log -v already does that17:45
jamJust that bzr log -v file doesn't IIRC17:45
awilkinsSomeone was asking about it last week and this reminded me17:46
jamabentley: any chance you could look at: http://bundlebuggy.aaronbentley.com/project/bzr/request/%3C48A307FB.7060905%40arbash-meinel.com%3E17:46
jamI think it is the last patch which is specifically blocking 1.6rc217:46
jamThough you may not be comfortable reviewing it17:46
abentleyjam: I don't think I have time today.17:47
jamIt's rather small17:47
jamjust win32 specific17:47
awilkinsHe wanted to know which revision a file(not in present inventory) was deleted in ; it's easy enough to extend bzr log to support --file-id, but then the log alas, doesn't show the deletion :-(17:48
* awilkins would review it if his vote carried any weight17:48
jamawilkins: I'm well versed in the problem, and know why it is an issue17:48
jamawilkins: all reviewers carry some weight17:48
awilkinsBut if I voite, BB ignoreds me, right ?17:48
jamBB probably does17:48
jamThat doesn't mean *I* would :)17:48
abentleyjam: I'm not supposed to be working on bzr during paid hours, and I'm in the middle of reviewing other stuff, and I have plans this evening.17:49
jamI'm surprised you aren't supposed to do any bzr work, given that you are in launchpad-bazaar, but sure17:49
jamI mean, you *did* just submit a patch because it effected LP :)17:50
jamWe just don't have any other reviewers in this TZ17:50
jamUnless jelmer is awake ...17:50
awilkinsI can work on it during paid hours (with a clean conscience) as long as it benefits my project17:50
awilkinsI can work on it (with a dirty conscience) anyway :-P17:50
abentleyjam: It doesn't affect LP.  I've already overridden those strings for LP.17:51
jamawilkins: :)17:52
jamabentley: so I should make the request in #bzrlp, right ? :)17:52
abentleyjam: I don't think anyone there is awake and in my chain-of-command.17:54
jamno, I just mean then it appears as a bzrlp request17:55
jamWe've taken longer discussing it than it would have taken you to review it17:55
jamDon't worry about me, I'll find someone else17:55
awilkinsPulling patches via URLs is very convenient17:57
jamawilkins: I agree18:00
awilkinsjam: Well, it makes the test pass ; but I see a few other places where ENOTDIR is used18:00
jamawilkins: sure, I'm just looking for a *regression* fix18:00
jamnot a full solution to all problems18:00
jamFor a release, I want it to not be worse than it was in the last release.18:01
jamAnd we *really* need to get a release out the door18:01
awilkinsI suppose it does lower the footprint18:01
awilkinsjam: I agree about the release, building my own is a PITA (and worrying, if I've patched it off the main line)18:04
awilkinsThis will fix my last issue with 1.6 (all my other patches are in)18:04
jambeuno: ping18:07
jonnydeejam: hi, regarding the bug https://bugs.launchpad.net/bzr/+bug/255656 -- what do you think? will my nomination for release 1.6 be considered?18:08
ubottuLaunchpad bug 255656 in bzr ""bzr: ERROR: [Errno 22] Invalid argument" when "bzr pack" is executed manually or when "autopack" is triggered on a repository located on a windows network share" [Undecided,New]18:08
beunojam, pong18:08
jamjonnydee: well, the fix robert proposed isn't really an official fix, more of a workaround to see what the bug was18:08
jamas such, I'm hesitant to make it into 1.6 without a real fix and test (as I'm trying to get 1.6 pushed out the door)18:09
jamI would be surprised if it didn't land in 1.7, though18:09
jonnydeeso I will have to copy with this bug in release 1.6, too?18:09
jamjonnydee: 1.5 performs the same, right?18:09
jonnydeewell, I'm not sure...18:10
jonnydeesorry, I meant "cope"18:10
jamjonnydee: I'm really trying to keep changes to 1.6 as regressions only18:10
jamwe have slipped 3 months because of delays18:11
jonnydeeyes, I understand...18:11
jamwhen we get back into a regular release18:11
jam1.7 will come out within a couple weeks18:11
jamso no big "must get it right now" issues18:11
jamjonnydee: so... if it is a *regression* you have a chance otherwise, not really18:11
jambeuno: any chance you could look at: http://bundlebuggy.aaronbentley.com/project/bzr/request/%3C48A307FB.7060905%40arbash-meinel.com%3E18:11
jamI figure 2 non-core people looking over the code for obvious flaws is enough for me to merge it18:12
jamand then I'll be ready for 1.6rc218:12
jonnydeehow can I support to get it to a *regression*18:12
jamjonnydee: If 1.5 *works* and 1.6 *doesn't* that is a regression18:13
beunojam, sure, looking18:13
jonnydeeok, I see18:13
jamjonnydee: which is why I'm trying to get the "rm" patch merged18:13
jamand not the 10 other things that are pending review in BB that I've written :)18:14
=== mw is now known as mw|brb
Toraninjam: FWIW, the patch looks sane to me, presuming your comments on Python's behavior are correct18:16
Toraninhowever, I've not read or worked on the context code at all18:16
jamToranin: thanks, we have some strong hints that it is correct because we've been using a variant in workingtree_4.py18:17
Toraninmy only concern would be that you strip the comment explaining WHY the exception is okay in the first use case18:18
awilkinsjam: I't OK on Python25 but for a (related/unrelated?) reason I am running into an error on Python2418:18
jamawilkins: what specific test are you running?18:19
awilkinsjam: Is pywin32 supposed to be a cast-iron requirement for bzr on windows?18:19
jonnydeeanyway, thanks for your help! :) You do great work!!! :)18:19
awilkinsjam: I'm running test_switch, which is where I found it at first18:19
jamawilkins: are you python extensions rebuilt?18:20
awilkinsjam: nope, hold on18:20
jamI get: NotImplementedError: We must have one of fcntl, pywin32, or ctypes available to support OS locking.18:20
awilkinsawilkins: But I think it's because my Python24 lacks pywin3218:20
jambut that just means I need to install pywin3218:20
jamwhich sounds the same as yours :)18:20
beunojam, sent my thumbs up to the list18:20
jambeuno: thanks18:20
awilkinsDoes gmane work well for list posting?18:22
* awilkins has just used it to post approval18:22
awilkinsjam: That means that pywin32 is required to run the test suite?18:23
jamawilkins: We either need ctypes or pywin3218:23
jamPython2.5 bundles ctypes directly18:23
awilkinsAh, yes18:23
awilkinsHey, Python 2.6 !?18:25
jambeuno: ok, well, I'll merge this patch, and one more to get the version strings to show 1.6rc218:25
jamAnd then I'll be ready for some packaging.18:25
jamThis won't be 1.6 final, so that people can respond to my comments in the 1.6 thread18:26
beunosounds good18:26
=== kiko is now known as kiko-fud
=== mark1 is now known as markh
datoheh, persona non gratis18:31
awilkinsBugger, I meant "grata" didn't I....18:31
dato:)18:33
pickscrapeInterestingly, bzr 1.6_rc1 is already available in the bzr gentoo overlay :)18:33
LarstiQjam: unless it's python2.5 on win64 of course, sigh18:34
pickscrapeGah, ignore me. You're talking about rc2 aren't you. :)18:34
jampickscrape: yep18:35
jamLarstiQ: yeah, but we probably have a raft of issues with p2.5 on win64 anyway18:35
=== mw|brb is now known as mw
jamAnd nobody has stepped up to complain yet :)18:35
pickscrapeIs anyone aware of a problem deleting a symlink that points to a directory that doesn't exist?18:35
awilkinsThey tried to get me to buy Vista64 at my friendly retailer18:36
rockstarbeuno, lp:~rockstar/loggerhead/unify-templates18:36
awilkinsI laughed in their face and burned their warehouse down18:36
Peng_abentley: ping18:36
rockstarI've moved the search and rss out of the menu18:36
pickscrapeIt gives me: bzr: ERROR: Not a branch: "/path/to/symlink/target/".18:36
Peng_abentley: I just got a mailer daemon error from a message sent to you (just a CC on a mailing list post). time limit exceeded: "procmail -a "$EXTENSION""18:36
jamawilkins: well, I got your email from gmane, so it seems to work fine18:37
* beuno looks at rockstar's branch18:38
LarstiQjam: as does everybody else, yes.18:39
jamLarstiQ: especially with the Py_ssize_t stuff18:40
beunorockstar, all changes look good to me18:49
beunoI'm happy to merge, unless you want to wait for mwhudson and thumper to re-review18:50
pickscrapeIs it worth me raising a new bug about the fact that bzr rm <dangling_symlink> doesn't work, or it is amply covered by the existing bugs that deal with the same problem with other commands?18:50
beunopickscrape, I'd say it's worth it. If it turns out to be covered, it'll just get marked as duplicate18:52
beunorockstar, btw, if you ever want to test search, you just need to install the bzr-search plugin, and run 'bzr index' on the branch18:52
=== mw is now known as mw|lunch
=== kiko-fud is now known as kiko-phone
rockstarbeuno, I've addressed everything thumper had in the review.  I haven't seen a review from mwhudson yet.20:15
rockstarAlthough, sometimes it takes him a few days.20:15
beunorockstar, I'll merge. mwhudson can yell at me later.  I think 2 reviews is more than adecuate20:16
TheEricHow does one reconcile without using ssh?20:16
rockstarbeuno, :)20:16
beunoTheEric, bzr reconcile sftp://...20:16
TheErick, I'm going to try that approach to reconcile our current app as the ssh method evidently doesn't work.20:17
beunoTheEric, good luck  :)20:18
TheEricfail. Paraminko needs to be installed.. Bah.20:18
beunoTheEric, it should be installed by default.  You're on windows, right?20:19
Manfrei had the same problem yesterday, gave up since pycrypto needed vs2003..i only have 2005/200820:19
Manfrebeuno, it's not installed by default for python 2.5 installer and not for 1.6 bzr iinstaller20:19
* beuno pokes markh 20:20
beunoI don't know much about windows anymore. Mark may be awake already20:21
=== kiko-phone is now known as kiko
TheEricGotta love university bandwidth. Downloads around 1000kb/s20:23
beunorockstar, merged20:24
rockstarbeuno, great, thanks!20:24
=== sm is now known as sm-afk
Peng_Ooh.20:35
=== mw|lunch is now known as mw
Peng_beuno / rockstar: The feed button on /changes is broken (not the <link> tag, just the regular old <a> next to the search box).20:38
Peng_<a href="Exception: name 'url' is not defined" title="RSS feed for /my/branch">20:38
rockstarPeng_, ah, thanks.20:38
rockstarbeuno, I'll fix it in the existing branch, and you can just merge it again.  Sound good?20:38
beunorockstar, yeap20:39
beunoPeng_, as always, you rock  :)20:39
Peng_:)20:39
=== Toranin1 is now known as Toranin
jambeuno: There is a release candidate tarball that has been uploaded.20:53
jamI'm sending the announcement now20:53
Peng_Whee.20:53
rockstarbeuno, the fix is ready for merging.20:54
=== jam changed the topic of #bzr to: Bazaar version control system | http://bazaar-vcs.org | please test 1.6rc2! / http://irclogs.ubuntu.com/ | http://planet.bazaar-vcs.org/ | http://blogs.mysql.com/kaj/2008/06/19/version-control-thanks-bitkeeper-welcome-bazaar/
beunojam, I'll upload packages then.20:54
beunorockstar, cool, I'll merge20:54
rockstarI also checked to make sure that nothing else is broken.  :)20:54
beunorockstar, pushed. Sorry I missed that in the review21:00
rockstarbeuno, yea, I missed it as well.  It happens.21:00
beunojam, when do you expect 1.6 final to be out?  this week?21:00
jambeuno: ASAP, within a week if nobody complains about the release21:02
jammarkh: I would like you to make full-fledged installers if you can. If not, I can put together some bzr-lite ones (I won't be including tbzr, etc.)21:03
beunojam, because it may be worth uploading 1.6rc2 hardy to the ~bzr PPA too, since there's no bzr release on it now21:03
jambeuno: can't we just copy the one from gutsy?21:03
jamI'm not a big wizard on the ppa side of things21:03
jambut I thought the gutsy and hardy package were identical21:03
beunojam, copying packages breaks stuff. I haven't gotten them to work. Ever.21:04
beunothere are theories21:04
beunoI can try it again21:04
beunolast I heard, copyong gutsy -> hardy should be fine21:04
beunobut it tends to not work21:04
beunooh, and we can't have 1.521:04
beunosince we already uploaded 1.6b2 before21:04
jam"can't have 1.5" ? ouch21:04
beunoso it's 1.6b3+21:05
beunoyeah21:05
pickscrapeThat seem like quite a limitation to the PPA system.21:05
jamwell, in that case I'd do 1.6rc2, but still crappy21:05
jampickscrape: you *can* have old versions, but it won't recommend them21:05
jamif something newer has been uploaded and then subsequently removed21:05
beunopickscrape, it's apt, not ppa21:05
jambeuno: so I believe you *can* explicitly request 1.521:05
beunojam, ok, cool. I'll upload to both21:05
jambeuno: only for hardy though on ~bzr21:05
pickscrapeEven if you never installed the previously uploaded version?21:06
jambeuno: and don't forget the bzr1.6~rc1 :)21:06
beunojam, there may be stuff in place on PPA to prevent that21:06
beunojam, yes, and yes  :)21:06
beunoI'll run it by you before uploading21:06
jampickscrape: if you use the 'any' filter you can see it: https://edge.launchpad.net/~bzr/+archive?field.name_filter=&field.status_filter=any21:07
jamthere is a bazaar1.5 there21:07
jamIt has status "superseded" though, and it is superseded by the deleted package :(21:07
jambeuno: are you sure Superseded isn't only a ppa thing?21:07
=== meteoroid_ is now known as meteoroid
pickscrapeBut isn't that status defined on the PPA side, or would changing it break apt?21:08
beunojam, yeah, it works that way everywhere else21:08
james_wbeuno: you can re-upload 1.521:08
jamjames_w: but isn't it superseded by something newer?21:09
jamjames_w: I mean, the 1.5 package is *still there*21:09
james_wjust give it a version of "1.6b3+really.1.5-1" or similar21:09
jamjames_w: evil21:09
beunojames_w, right, dato proposed that, but it seemed like too much of a hack21:09
pickscrapebut crafty :)21:09
james_wthere's plenty of precedent21:09
beunoand, well, 1.6 has been "almost" out for a while now  :)21:10
jamjames_w: with the downside that 1.6b3 will be "upgraded" to 1.5, right?21:10
james_wyeah, if you had the ~bzr ppa, but not the ~bzr-beta21:10
jambeuno: well, I'm planning on pushing really hard to see that it does21:10
james_wthat's normally the intention when you do that21:10
jamThere was really only 1 thing that Martin had on his list that I'm skipping21:11
jambut I have to wait for AU time for everyone to respond to my plans21:11
jambeuno: are you going to be around in 48 hours? (My friday)21:11
beunojam, I can't upload to ~bzr-beta-ppa, I haven't been approved21:12
beunojam, yeah, I'll be here friday21:12
TheEricwell, that reconcile failed with the sftp://21:12
TheErichttp://pastebin.com/m65dc142321:13
jambeuno: well, we'll have to see if *I've* been approved :)21:13
jamI think martin added me a while back21:13
jamif you can put them somewhere, I'll upload21:13
beunojam, yeah, you're approved. Once I upload succesfully to ~bzr, I'll place em somewhere so you can upload them to ~beta21:14
TheEricHere's the .bzr.log file - http://pastebin.com/m13f08af1 to go with that failed reconcile.21:17
jambeuno: once you upload it, I can actually just copy it directly to ~bzr-beta-ppa using LP's copy package feature21:18
beunojam, for hardy you can, not the rest21:19
jambeuno: can't I copy ~bzr/hardy => ~bzr-beta-ppa/hardy21:20
jamand then copy /hardy => /gutsy, etc ?21:20
jamJust not dapper21:20
beunojam, nope. It broke every single time we did that with poolie21:20
jamwell, that's pretty crummy21:20
jamjames_w: what's up with that ?21:20
beunoOTOH, it's rumoured that uploading to, say, feisty, and then copying "upwards", works21:20
jambeuno: was it just a dependencies thing?21:21
james_wbeuno: what broke?21:21
jamand does it change if you check the "copy binaries" flag?21:21
james_wif it's just a source copy and you are re-uploading the same source then I would assume it works, but I am probably missing something21:21
beunojames_w, dependencies21:22
beunoif you re-upload, and it re-builds, it's fine21:22
beunoif you copy, it's not21:22
james_wyeah, they are generated at build time, so I would have thought that a source copy would cause a rebuild and that would work21:22
jamjames_w: so you think that copying without checking the "copy binary" should work?21:24
james_wthat's what I thought, but it hasn't worked previously21:25
beunojam, bzr_1.6~rc2-1~bazaar1~hardy121:26
jambeuno: I thought we weren't doing the ~hardy1 thing ?21:26
beunojam, I'm not, but poolie kept on doing it21:26
jamJust plain ~bazaar121:26
beunoyeah, I like that more21:27
jamas long as there is precedence, I guess21:27
james_wyou should to provide an upgrade path21:27
jamjames_w: but then you have to edit it for each one21:27
james_wjam: yup, it's a pain21:27
jamto do ~intrepid1, ~gutsy1, etc.21:27
jamjames_w: and how does *that* work with copying?21:27
beunojam, it copies ~release121:27
james_wdon't copy if you are doing that I think21:27
beunowhich is confusing  :)21:28
jambeuno: so if you copy ~hardy1 => gutsy, it changes that value?21:28
miracle2kam I crazy or is there now way to link to the latest revision of a file in the launchpad browser?21:28
beunojam, nope21:28
awilkinsmarkh: Are you win32 release builder?21:28
beunomiracle2k, where are you looking?21:28
miracle2kbeuno: say I want to link to the latest README.txt from: http://bazaar.launchpad.net/~loggerhead-team/loggerhead/trunk/files21:29
beunojam, I'd say, leave ~hardy1, etc, don't copy, and have a talk with poolie when he gets back from vacation21:29
jambeuno: um, then I have to re-upload everything21:30
beunomiracle2k, ah, I don't think we have a direct link for that now21:30
jammiracle2k: There is a secret revision "HEAD" that you can use, just a sec21:31
beunojam, your call. I'll do either21:31
jambeuno: I'll just upload it, but I worry because I've never *actually* successfully built and uploaded packages21:31
jamway too much hassle to do the N-way release thing21:32
jambeuno: this gives me an "internal server error": http://bazaar.launchpad.net/~loggerhead-team/loggerhead/trunk/annotate/head?file_id=readme.txt-20061211064337-5lpgd5knbgm7jrgz-221:32
jamI thought that was the right way21:32
beunojam, http://bazaar.launchpad.net/~loggerhead-team/loggerhead/trunk/annotate/head:?file_id=readme.txt-20061211064337-5lpgd5knbgm7jrgz-221:32
jamah 'head:'21:33
jamthe ':'21:33
jammiracle2k: ^^21:33
miracle2kjam: thanks a lot!21:33
miracle2kthe extra stuff in the file_id just refers to the branch itself I assume?21:34
beunomiracle2k, it's the unique ID for that file21:34
beunowe will probably address that in the next release21:35
jambeuno: wasn't there a path= that you could use?21:35
jammiracle2k: so using a file_id, means that if you rename the file, it will follow the rename21:35
beunojam, no, I don't think you can use paths ATM21:35
jambeuno: so, you have the package available, or it has been uploaded?21:35
jambeuno: hmm.. I was pretty sure it used to work, but not that *I* really care21:36
beunojam, uploading hardy to ~bzr21:36
miracle2kjam: I get that, but I guess I am confused by the fact that the filename itself is included as well. but nvm, I'll have a closer look at the bzr internals at some point21:36
beunojam, I've never used the path bit, and I can't think of any part of the code that would do it. That said, Loggerhead still has some unexplored places, so anything's possible21:37
jammiracle2k: when you 'bzr add' a file, we generate a filename that starts with the path21:38
jamafter renames, etc, it keeps the original id21:38
jambeuno: I just thought someone posted a link using head: and path= as a "nice" url that wasn't exposed anywhere.21:39
jamYou just had to know it existed.21:39
jamBut I would guess you have as much info about LH as anyone anymore21:39
beunoI remember the head:, because we use bzr's magic underneath.  path escapes me, but, again, it wouldn't shock me that it's hidden somewhere. mwhudson is good with those things21:40
beunojam, [PPA bzr] Accepted: bzr 1.6~rc2-1~bazaar1~hardy1 (source)21:40
jambeuno: well the obvious "path=README.txt" doesn't work21:41
jamcertainly it is just a "path2id()" away21:41
beunojam, maybe we can get a LP admin to approve me in ~beta instead of us uploading/downloading, etc?21:42
Peng_Yeah, the feed thing is fixed. :)21:43
Peng_Thanks.21:43
jambeuno: yeah, poolie added lifeless and I as "Members" but not "Administrators" so we can't approve you.21:44
jamI started the copy to ~bzr-beta-ppa21:44
jamWhich seems successful and seems to want to build binaries for it21:44
beunojam, wanna try copying to intrepid/gutsy/feisty?21:45
jamjames_w: can you mark a package "Needs building" ?21:45
jambeuno: I can certainly try it, right?21:45
jamworst case we just upload them directly?21:45
beunoyeap, with -2 in 'em21:45
james_wjam: don't think so, it should happen automatically if you didn't copy binaries21:45
jamjames_w: Well, I just tried to copy ~hardy1 => intrepid, and it gave me:21:46
jamThe following source cannot be copied: bzr 1.6~rc2-1~bazaar1~hardy1 in hardy (same version already building in the destination archive for Hardy)21:46
jamjames_w: It seems the "series" field isn't being respected?21:46
jamMaybe it is being overloaded by the ~hardy1 ?21:46
beunohrm21:47
beunomaybe PPA's don't allow the same package in 2 different places?21:47
jambeuno: well, that negates the *point* of the copy feature, right?21:47
=== salimander is now known as Mercenary
=== Mercenary is now known as Suigintou
jamI mean, if you can't actually copy to another "Destination Series" then you really can't do anything21:47
jamother than between PPAs21:47
jambut certainly they are trying to give the option21:48
beunojam, unless it's for a different release, but yeah, I can't get my head around copying in PPA21:48
jambeuno: and, of course, there is "expected behavior" and there is "bugs"21:48
beunoexactly  :)21:49
jamThat's why I'm trying to poke at james_w to see what category this sort of thing falls under21:49
jamhttps://help.launchpad.net/PPA?action=show&redirect=PPAQuickStart#Copying%20packages21:49
jamIs pretty vague21:49
james_wI've never done a package copy, I agree with your expectation of this though jam21:50
jamReally, it just says what you should expect on the page21:50
james_wit sounds like it's trying hardy->hardy21:50
jamand not really what you should expect it to actually do21:50
james_wmaybe #launchpad ?21:51
jamjames_w: it does indeed, but I'm definitely changing the "Destination Series"21:51
jamtrying there21:58
[reed]How do I kill a stuck lock on a bzr repo? Trying to commit to a local repo, and it says there's a lock. The pid it claims the lock belongs to doesn't exist (it did the first time, but it's been killed since then).22:02
awilkinsbzr break-lock22:03
[reed]thanks!22:04
beunoOMG, jam names his PC after pokemons!22:13
LarstiQbeuno: you didn't know? :)22:15
* LarstiQ fondly remembers jigglypuff22:15
beunono, I should pay more attention22:16
Peng_I didn't know. Nice. :)22:18
=== kiko is now known as kiko-afk
Stavroshello22:29
Stavrosis there a bzr repo that has the current version of bzrtools?22:29
Stavrosright now i can install either the latest bzr version or bzrtools :/22:30
lifelessjam: ^ I think perhaps you uploaded a beta to stalle ppa?22:30
beunolifeless, I did. There was no bzr for hardy22:31
beunoso, I thought, better 1.6rc2 than nothing22:31
beunoof course, this breaks bzrtools22:31
beuno*sigh*22:31
beunoand, good morning lifeless   :)22:32
Stavrosis there a solution for the bzrtools problem?22:32
beunoStavros, you can uninstall it, and install from source22:32
Stavrosbeuno: but that way it won't be tracked22:32
beunoStavros, right. The other option is wait until someone has time to upload bzrtools 1.6  (is that even released?)22:33
Stavrosah :/22:34
beunosorry about that22:34
Peng_You could install bzrtools from source, and delete it once an update is available.22:35
=== meteoroid_ is now known as meteoroid
=== meteoroid is now known as meteoroid_
=== meteoroid_ is now known as meteoroid
jambeuno, LarstiQ: Actually after Super Smash Brothers22:45
jamSamus == Metroid lady22:45
jamJigglypuff was my old mac22:45
jamlifeless: Kiko on #launchpad determined that you *can't* use the copy feature of LP to copy between distributions within the same archive22:46
jambecause it can't rename the package22:46
beunojam, right, right, smash brothers  :)22:46
jamSo it would get naming conflicts.22:46
beunojam, btw, uploading for dapper now, that should be the last of it22:47
jambeuno: SSB with sudden death mode, and all Jigglypuff characters was very entertaining22:47
lifelessjam: AIUI we shouldn't ever be using copy22:47
jamlifeless: well, copy is mentioned in the docs22:47
jambecause it would make our lives a lot easier22:47
lifelessour docs or launchpads?22:47
jamand you *can* copy the package from ~bzr-beta-ppa => ~bzr22:47
jamlifeless: ours22:47
jamand launchpad hints that you can use it22:47
jambut gives very little as to what to expect.22:47
lifelessright, but it lies :)22:47
jamLP will *let* you say that you want to copy X package to the same archive22:48
jambut that never works22:48
jamunless maybe you copy the binaries22:48
jambut then you likely have problems with deps22:48
lifelessjam: full test run of .bzrrules disabling code is running now22:52
jamlifeless: well, you also have to convince at least one other person that it is *correct*, as I don't really think it is.22:52
lifelessjam: I've convinced three others22:53
lifelessjam: Ian, Jelmer, and Martin22:53
lifelessthough to be pedantic, Jelmer came preconvinced22:53
lifelessand if I could find this damn email rant from Aaron back in the day I'd have more evidence22:53
strkbazaar thinks I hold a lock, but I've no bzr process running22:55
strkany way to further debug the issue ?22:55
lifelessstrk: we don't use oslocks, so thats possible22:55
lifelessstrk: what does bzr break-lock say about the lock ?22:55
jamlifeless: If you've convinced Ian, I'm okay with it. I think it falls into the "we keep saying this isn't the right way, but we provide no better way so we have nothing".22:55
strkprompts me to break it, said yes22:56
strksaid it was hold by me (my email)22:57
strkone hour, 30 mins22:57
strkit was after a merge from trunk22:57
lifelessjam: I've just reread Ian's mail to me, I think I had agreement there is an issue.22:58
lifelessjam: I'm forwarding it to you22:58
lifelessstrk: something must have hard-interrupted bzr22:59
lifelessstrk: e.g. SIGKILL or SIGHUP22:59
strkpossible23:00
lifelessstrk: we can't use OSLocks over the network; so we don't use them for data structures that are network accessible23:01
beunojam, all debs uploaded and published23:01
jamlifeless: I'm also responding with the last post I saw from igc on the mailing list23:02
jamlifeless: or two SIGINTs at the right time (one while we're cleaning up is sufficient)23:02
jambeuno: thanks a lot!!23:02
beunojam, my pleasure. You should milk this as much as possible, before I actually get a manager assigned and have something specific to do  :)23:03
lifelessjam: did you have more feedback on the inventory docs? I think I have enough to recast it as a tighter set of needs if you don't23:04
jamlifeless: I have an email for you I'm finishing, but right now I have to pick up my son from daycare.23:05
lifelessjam: go then23:05
lifelessjam: think you'll get it to me today?23:05
thumpercould I get someone who knows about reconcile to take a look at https://bugs.edge.launchpad.net/launchpad-bazaar/+bug/257340 please?23:06
ubottuLaunchpad bug 257340 in launchpad-bazaar "Revision History set to 0 following push" [Undecided,New]23:06
thumperit is beyond me23:06
lifelessjam: on the rules stuff23:07
lifelessjam: I had a conflict of interests in my head back at the start of the yea23:07
lifeless*year*23:07
lifelessas a community guy I wanted to flat veto designs involved versioned user files23:07
lifelessas a canonical guy I wanted to see features land as fast as possible23:08
lifelessI've wrestled with myself and come to the conclusion that landing the wrong thing isn't right for anyone23:08
lifelesswhich is why I'm engaging on this topic again23:08
luksI wonder how do you want to handle historical data if you don't version .bzrrules23:15
=== sam__ is now known as samatwork
samatworkdoes anyone know how to fix a bzr: ERROR: not well-formed (invalid token) error?23:20
samatworkI've looked all over for an error that explains it and I know that many have had it, but I didn't see any clear-cut solution23:20
james_wsamatwork: hi, I've never seen that before, do you get a full backtrace, or just that?23:21
samatworkjust that23:21
samatworkI'm getting it on all clients that are referencing a central repository23:21
samatworkI fibbed it it also gives a line and column number, but I have no idea what this is referencing23:22
samatworkcan't find anything at launchpad either23:23
james_wcould you please run with "-Derror" as an extra command line argument to get a backtrace, and then pastebin that?23:24
james_wubottu: paste23:24
ubottupastebin is a service to post multiple-lined texts so you don't flood the channel. The Ubuntu pastebin is at http://paste.ubuntu.com (make sure you give us the URL for your paste - see also the channel topic)23:24
samatworkhere you go .... http://paste.ubuntu.com/37261/23:27
james_wsamatwork: what about "bzr -Derror update"?23:27
lifelessluks: its not about historical data; its about decoupling representation and function23:28
james_wif it's the same then the problem is on the remote side, I think23:28
lifelessluks: if you want to to a fix for squid 3.0, on windows, you need EOL translation23:28
lifelessluks: but the 3.0 revision doesn't have .bzrrules23:28
lifelessluks: so I think having .bzrrules in the users tree is fine as a presentation layer today23:30
lifelessluks: but I want us to retain the ability to change representation in the future; without requiring a 'dump and reload' approach23:31
lifelessluks: think about CVS 'b' flags - they were not per-branch, there was no 'propogation' needed23:31
samatworkjames_w: http://paste.ubuntu.com/37263/23:32
samatworklooks like it has something to do with an unexpected inventory format23:32
james_wit looks like corruption of the inventory on disk23:33
samatworkit must be on the side of the repository though since all clients point to it return the same message23:34
samatworkany thoughts on how to fix this short of recreating the repository?23:34
james_wI'm not sure23:34
james_wlifeless: any hints for samatwork?23:35
jamlifeless: my response has been emailed23:36
lifelessjames_w: I'd use repository.get_inventory_xml to get the raw xml and look at it23:36
jamlifeless: I understand your concern, but I also feel you have at least 2 major things on your plate for 1.723:36
lifelesssamatwork: are you using bzr-svn ?23:36
jambringing in a 3rd is not really going to happen23:36
samatworklifeless: nope just straight bzr23:37
lifelessjam: If it needs to happen to avoid a mistake, it needs to happen23:37
lifelessjam: its a lot less stressful for me to do that than to see .bzrrules go in as-is23:37
jamI also have to say that the last messages from IGC don't look like agreement23:37
lifelessit will make nested-trees harder, join-tree and split-tree, it won't solve the usability problems the folk that I know that want EOL stuff on windows have23:38
jamThey pretty much say "we need something, and this is the path of least resistance"23:38
jamwhich is where I fall23:38
jamI fully understand your conflict of interest23:38
jamAnd feel the same way23:38
lifelessif I had resolved it previously, I would have bb:vetoed this code23:38
jamI probably am more on the "frustrated by not having things like nested trees"23:38
jamfor *years* now23:38
jamin any form23:38
lifelessfull ack23:38
jamand taking what, >1 year to get any real in-core tags23:38
jamand we still have non-optimal ones23:38
lifelessnote that git doesn't version its tags23:39
jamI guess sort of like the OpenGL 3.0 release... wait a year, and still get only something incrementally better.23:39
lifelessjam: that is because noone is spending time on these things, not because its a particularly hard problem23:39
jamlifeless: well, we had several designs that were possible23:40
jamand it was fundamentally a trade-off23:40
jambetween complexity and handling all cases23:40
jamIn git, moving a tag is a bad deal23:40
jamwhich it pretty much is in bzr as well23:40
jamtrying to get it to propagate23:40
jamand deleting anything doesn't work23:40
jam(well, it deletes locally, but never propagates, etc)23:42
jamdifferent topic... With LP, if I upgraded from Knit => pack-0.92 when  I upgrade from pack-0.92 => 1.6 won't that cause a problem with the backup.bzr directory?23:43
jamIs there a simple way to delete that?23:43
jam(Other than going in with abentley's hitchiker and doing a sftp deltree)23:43
samatworkthanks for the help everyone... I have to run23:44
lifelessjam: they are just refs in git, and their ref management is quite sophisticated23:45
jamlifeless: specifically, change a tag and pull into the next repo23:46
jamit won't propagate23:46
jamwell, maybe just merge23:46
jamI think if you force a pull it will overwrite23:46
jamAnd I don't know about delete23:46
lifelessjam: I think you need hitchhiker23:46
lifelessjam: I'd file a bug on lp :P23:47
jamlifeless: I'm not doing it right now, I'm just upgrading a Knit repo23:47
jamand realizing we'll probably have a problem later23:47
ben__Hi! I have a question on how to clean up a repository e.g.:23:51
ben__bzr init-repository foobar23:51
ben__bzr init foobar/trunk23:51
ben__bzr branch foobar/trunk/ foobar/feature23:51
ben__dd if=/dev/urandom of=foobar/feature/random count=1000023:51
ben__bzr add foobar/feature23:51
ben__bzr ci foobar/feature23:51
ben__du -sh foobar/.bzr23:51
ben__5,0M  foobar/.bzr/23:51
ben__how do I delete the branch feature out of the whole project?23:51
Peng_ben__: You pretty much don't.23:51
ben__theere23:52
ben__there is no way?23:52
james_wben__: you mean you want to discard the branch, and free up the used disk space?23:53
ben__yes23:53
ben__the docu says i just have to rm foobar/feature23:53
james_wyeah, but that doesn't free up the space23:54
ben__but then I have the wasted space in .bzr23:54
ben__yes23:54
james_wthere's no simple command to do this at the moment23:54
ben__is there already a bugreport (featurerequest) for this?23:54
james_wwhat you basically have to do is create a new repo, branch the branches you want in to it, and then delete the old one and move the new one in to place23:54
james_wI'm not sure23:55
ben__yes, that solution came into my mind, but it is not very nice…23:55
james_wno, it's not great23:56
james_wthere's an experimental plugin to do it in one go if you want to test that23:56
ben__yes, of course ;))23:56
ben__but i didn't found one doing this at the homepage…23:57

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