/srv/irclogs.ubuntu.com/2009/08/07/#bzr.txt

igcmorning all00:04
igchi jelmer00:04
SamBam I right in thinking that bzr selftest -E foo is supposed to put 'foo' in bzrlib.debug.debug_flags ?00:09
=== cprov is now known as cprov-afk
poolielifeless: i see we have an Inventory and an InventoryCommon class00:24
pooliewhich i think is the antipattern we were talking about a while ago wrt RepositoryBase00:24
lifelessno quite00:24
lifelessInventoryCommon is the base; Inventory is really MutableInventory, or InMemoryInventory00:25
lifelessbut there were enough uses of it that renaming would have been traumatic00:25
pooliewhy is that different?00:25
lifelesswe've moved a lot of stuff to Common00:25
lifelessCHKInventory is immutable00:25
pooliei could equally say that Repository is really VFSBasedRepository00:25
SamBhooray for immutability00:26
SamBpoolie: do you have some other kind?00:26
lifelesspoolie: Repository long ago lost concrete code00:26
poolieRemoteRepository00:26
lifelessits not a leaf00:26
lifelessRemoteRepository is a leaf, as is Inventory, and CHKInventory00:26
poolieargh00:26
lifelesspoolie: I could be wrong; but thats why I think the cases are different00:27
* SamB wishes for a black board00:27
pooliei should be more precise00:27
pooliethe issue is: things that sound like they should be base classes, actually are not00:27
lifelesspoolie: oh right. Inventory is a really misleading name00:28
lifelessI would love to renamed it to (say) InMemoryInventory00:28
lifelesss/renamed/rename/00:28
SamBhow about rename but leave an alias behind?00:28
pooliewe could get there from here by either - inserting a new class called *Base or *Common00:28
SamBwith a comment that it's there to avoid pain00:28
poolieor, changing the existing one00:28
poolieSamB: it's too hard wrt our api compatibility00:28
pooliei'm hoping that introducing different api break windows may make it easier00:29
lifelessSamB: do, or do not :).00:29
poolieit may still be traumatic even just within bzrlib though00:29
SamBwell, at least that way people would see the right name if they repr'd it00:29
pooliebtw re https://code.launchpad.net/~mbp/bzr/403523-status-crash/+merge/973900:30
pooliedo you object to me putting has_filename in the base class?00:30
lifelessso, to rename it, we could do it, use a factory as Inventory, which can issue deprecationwarning00:30
poolieanyhow, concretely00:31
pooliethe thing to do is change RemoteRepository to inherit from Repository and see if anything breaks00:31
poolieor if anything should go into a LocalRepository00:31
poolieor some better name00:31
lifelessyes00:31
lifelessI'd just change RR to inherit from R and see what the fallout is00:31
lifeless+1 on that change00:32
lifelesshas_filename is a little ugly though00:32
igclifeless: that usertest run on OOo with your pending patches got done last night ...00:39
igclifeless: 73.9 -> 1.8 seconds for selective commit00:39
poolie! :)00:39
lifelesswell, thats better than it was00:39
lifelesshows full commit?00:39
igclifeless: and also interesting is ...00:39
lifeless[drum roll please]00:40
igcsend -o dropping from 332 seconds to 69.6 seconds00:40
lifelessin my branch?00:40
igclifeless: where 70 seconds still sucks, fwiw00:40
lifelessthat would be the path2id improvement showing up00:41
lifelessigc: whats full tree commit in OOo?00:41
igc1.8 seconds00:41
lifelessok, so we're still dominated by something not addressed00:42
lifelessI'd love an lsprof of both incremental and full commits of a single file00:42
lifelessI suspect dirstate load/save00:42
igcshall do00:42
igclifeless: if it is, I have a patch for that - still in review maybe00:43
poolieigc, i assume that send thing is with jam's ptach?00:43
lifelesspoolie: I improved a code path send uses as well00:43
igcpoolie: the 332 seconds is with bzr.dev rev 4593 - pretty recent00:45
igcpoolie: but looking deeper, I'm not sure whether my 'pending' branch with robert's patches has it or not00:45
pooliespiv, does my last comment on bug 405251 make any sense to you? :)00:48
ubottuLaunchpad bug 405251 in bzr "Huge data transfers/bad performance OVERALL" [Undecided,Incomplete] https://launchpad.net/bugs/40525100:48
thumperdoes anyone use bzr with windows and launchpad here?00:54
thumperhttps://answers.edge.launchpad.net/launchpad/+question/7926100:54
thumperthe question here is trying to work out why it isn't working00:54
sidneithumper: occasionally00:54
thumperAFAICT it should work00:54
thumperso I'm guessing it has to do with getting bzr on windows to use his putty key00:55
thumperand I have no idea how to check that00:55
spivpoolie: "I wonder if this is related to binary modules not being built?" ?00:55
spivOh, there's a newer one now.00:56
pooliemm, maybe, or actually to something being stuck or inefficient in hpss protocols00:56
spivIt wasn't there when I first loaded the page.  email lag I suppose.00:56
pooliebit of a wild-ass-guess00:56
sidneithumper: the steps he outlined seem correct generally. unless he messed up on step 5.00:57
thumpersidnei: is there any way for him to check?00:57
sidneistep 5 from the launchpad wiki, that is, the one that has a explicit warning about where it can get messed up00:57
lifelessigc: there are a few commits from trunk missing00:57
sidneithumper: when you upload the ssh key to launchpad does it get used right away, or there's a time needed for it to sync somewhere?00:59
lifelessigc: its missing 458801:00
lifelesswhich changes bundles as well01:00
lifelessigc: I'll merge trunk now01:00
igclifeless: thanks. My 'pending' trunk is usually trunk + merges of stuff I want to benchmark but ...01:01
igclifeless: I'd prefer to just benchmark you branch if it's up to date with trunk01:01
igcs/you/your/01:01
thumpersidnei: I don't know01:01
lifelesssidnei: its usable immediately, modulo db lag01:03
sidneithumper: that's the only thing that comes to mind: he uploaded the key and expected it to work right away, but the key isn't immediately put in place on the server side. but i don't know if that is the case or not.01:03
thumperhmm..01:03
lifelesssidnei: the ssh daemon isn't openssh, so it just asks the db for the key, rather than waiting for it to be put on disk01:03
lifelessigc: pushed01:03
igclifeless: is iter-changes-partial-parents enough yet? Or do I still need to follow that up with a merge from selective-file-commit?01:12
lifelessyou need both01:13
poolieigc, any comments on kiko's vm mail?01:24
igcpoolie: if we want continuous testing following a commit, we need it up most of the time I suspect01:32
poolieyeah01:32
igcpoolie: perhaps a time out of a few hours would work though and ...01:33
pooliethat's about $100/month, it shouldn't break the bank01:33
igcmight result in it being down over weekend periods in particular01:33
igcpoolie: if we go the mega-vs-simple installer route, ...01:34
igcpoolie: then the testing only needs the standard simple installer or easy-install01:34
igcpoolie: building the mega-installer will happen rarely01:34
sidneihey igc, poolie01:42
sidneinot sure if vila let you guys know, but we have builds being run by kerguelen now01:42
lifelessexcellent01:43
lifelesssidnei: have you seen hudson?01:43
sidneimatter of fact, i did01:43
mwhudson...01:43
sidnei:P01:43
lifelessmwhudson: change your highlight rules ;)01:43
mwhudsonlifeless: this one is in my brain, sadly01:44
lifelessmwhudson: I hear electroshock is much more refined these days01:44
mwhudsonheh heh01:44
kikopoolie, igc: I'm outta that circuit now -- I have done what I can :-)01:45
kikopoolie, igc: again, for me bzr2 is the priority -- this is just something nice to have happen01:45
pooliekiko, thanks for connecting the circuit01:57
pooliea good 2.0 is the priority but getting good windows builds for it will help make it a good release01:57
spivHmm, test_pack_repository really ought to be renamed to per_pack_repository or per_repository_pack I think.02:08
spivpoolie: actually, that's something you might want to look at while looking at running tests with 2a as the default, test_pack_repository doesn't include 2a in its scenario list!02:12
spiv(also, the way the scenario definition duplicates a bunch of format strings etc rather than just grabbing them off the format objects is weird, I think.)02:13
poolie:/02:15
pooliecould you file a bug assigned to me?02:15
spivSure.02:16
jfroyjelmer: is there some guide on using dpush?02:24
lifelesspoolie: review sent02:30
lifelessbah02:31
lifelessspiv: review sent02:31
pooliefor the delta branch?02:32
lifelessyes02:32
lifelessabout 6 hours of reading and thinking02:32
krisives1Anyone know how I can get `bzr log` to output time and date?02:36
lifelessit does by default?02:36
krisives1It outputs date, but no timestamp02:37
lifelesstimestamp: Fri 2009-08-07 01:03:20 +010002:37
lifelessis what I see02:37
lifelesswhat are you seeing?02:37
krisives1Wait, nevermind02:37
krisives1I am using `bzr log --short`02:37
krisives1Hmm, I want the short but with the timestamp02:37
krisives1I suppose I can parse02:38
spivlifeless: thanks!02:42
krisives1I know this isn't SQL02:42
krisives1Anyone know a good way to swap 2 primary keys?02:42
lifelesslunch02:43
lifelessif you need me, ring me, I'll be offline02:43
krisives1cheers lifeless02:44
=== lamont` is now known as lamont
* igc lunch03:59
emmajaneigc, noooo. no lunch!04:14
emmajane:)04:14
=== abentley1 is now known as abentley
spivOof, reviews replied to.06:36
poolielifeless: are you back?06:38
poolieepic review there06:40
spivYes, "epic" was exactly the word I was just thinking of :)06:40
spivMore epic than expected, really.06:41
spivThere are a few points still not 100% resolved, but I think it's probably good enough to land, and the last few nits can happen in a followup landing, I think.06:41
* spiv makes a coffee06:42
spivYeah, with the aid of this coffee, I'm pretty comfy with the state of this branch, so I'll pqm-submit it now and send the last few fixes later.06:53
vilahi all07:25
vilapoolie: I'd like to answer that mail about the win32 build machines but I miss the meaning of some TLAs: AWS, AMI ?07:27
bob2amazon webservices, amazon machine something (disk image)?07:29
vilaha right07:30
pooliehello vila07:47
vilapoolie: hey !07:49
pooliehi07:49
poolieshall we talk about that?07:49
pooliei was just going to send mail07:50
poolieor maybe i should send mail and then we  should talk07:50
vilapoolie: same here, mail nearly godd to be sent from here07:50
poolievila what do you mean by "as long as we can be freed from the nightly ones"07:57
vilagrr, freed "from the risk of regressions"07:58
vilawow, this sentence makes no sense at all, sorry :-/07:59
vilaI meant to say: If the nightly builds gave us the benefit of catching the regressions sooner, the release builds should be hassle free and as such can be done by humans07:59
poolieok, right08:00
poolieat least kicked off by a human08:00
vilapoolie: 'kicked off' as in 'in a freshly started VM' ? Yes08:03
vilaor any other valid host08:03
spivHeh, "make check" fails on my branch due to "make docs" not liking the markup for the -DIDS:always debug flag.08:05
pooliespiv maybe eventually debug_flags should be a dict08:06
pooliei'd kind of agree for now it should be IDS_always but whatever08:07
vilaspiv: 'IDS:' made it kind of obvious that the flags were exclusive... too bad :)08:12
lifelessvila: bob2: amazon web services, amazon machine image08:14
spivI could take lifeless' suggestion and just not document them...08:16
vilaspiv: >-/08:17
poolieno, they're fine08:20
poolielifeless: in the context of https://code.edge.launchpad.net/~mbp/bzr/403523-status-crash/+merge/973908:52
pooliewhat's the difference between test_inv and per_inventory?08:53
poolieit looks like they want to be the same thing?08:53
lifelessI think they do08:53
lifelessexcept08:53
lifelessperhaps08:53
lifelessper_inventory wants to be per_mutable_inventory08:53
lifelessand the tests at the top of test_inv - the parameterised ones, a small part of the file08:54
lifelesswant to be per_immutable_inventory08:54
lifelessor something along those lines of separation08:54
poolieok08:56
pooliethen i might:08:56
pooliefile a bug for this08:56
poolierearrangement;08:56
pooliestick a test for has_filename into test_inv08:56
poolieand merge my change08:56
lifelessyup08:56
lifelessper_inventory only tests Inventory today08:57
* igc dinner09:46
spivlifeless: btw, if I just delete that if block from the per_interrepo fetch test the tests still pass.10:14
spivlifeless: so whatever bug I had that prompted me to add it in is gone.10:14
lifeless\o/10:15
spivAnother problem solved by deleting some code!10:15
Kinnisonhuzzah10:27
lifelessspiv: uhm10:27
lifelessspiv: you should delete the if statement and undent the block, I think.10:27
spivlifeless: right, that's what I meant.10:27
lifelessphew10:28
spivrun the assertions unconditionally.10:28
lifelessI thought that on first read10:28
spivSorry about the false alarm(s) :)10:28
lifelessthen I read what you wrote.10:28
spivYeah, my bad.10:28
lifeless:)10:28
OllieRHey, do symlinks work in bzr?12:04
fullermdIn what sense?  bzr sill store and version symlinks, sure.12:07
vilasidnei: ping12:33
=== cprov-afk is now known as cprov
jelmerjfroy: not really, it's basically just like push13:15
jelmerigc: hi14:10
igchi jelmer14:32
jelmerigc: I've proposed a merge for some foreign branch tests, I was wondering if you could comment on the general approach I've taken, so I know if I'm on the right track before doing a lot more of these tests.14:38
jelmerigc: (the merge proposal is at https://code.edge.launchpad.net/~jelmer/bzr/foreign-tests/+merge/9642)14:39
igcjelmer: I won't have time in coming days sorry, Maybe vila or jam could offer an opinion? If not, I'll take a look mid-to-late next week14:40
igcjelmer: sorry about that but I just have too much other stuff on my plate currently :-(14:40
jelmerigc: No worries14:41
igcnight all14:43
jelmerg'night igc14:43
dgoulethey people!, with bzrlib, is it better to use bzrlib.commit or bzrlib.builtins.cmd_commit() for a working tree?14:57
dgouletanybody?15:00
viladgoulet: cmd_commit is very close to the command line but may be harder to use than the command line itself15:03
vilabzrlib.commit may be harder to understand if you're not familiar with the bzr internals15:04
jelmerdgoulet: any reason for not using WorkingTree.commit()? That invokes bzrlib.commit15:04
jelmermoin vila15:04
vilajelmer: hi15:04
dgouletno not at all, actually i'm getting deeper into bzrlib day after day and I'm wondering the best way to commit a working tree15:05
dgouleti'm using cmd_commit right now but i was thinking that there might be a better way15:05
viladgoulet: you should have a look at the commit tests and decide from there what suit your needs15:07
viladgoulet: try 'bzr selftest commit --list' to get a taste15:08
vilabut presumably jelemer is right, you awnt to use wt.commit()15:08
dgoulethmm ok I look into that, very nice15:09
dgouletanother question while i'm here15:11
dgouleti have a branch with NO tree (only .bzr)15:11
dgouletwhat's the best way of getting that into a working tree to work on whitout a cmd_branch() ?15:12
jelmerdgoulet: BzrDir.open(path).create_workingtree()15:15
meuserjAfter upgrading bzr to 1.17, my integration with Redmine suddenly stopped working.  Did a little diving into Redmine to see what command is causing the issue, and bzr is indeed crashing on the command which redmine uses: http://pastebin.com/d58052ae215:15
meuserjany fixes or workarounds available?15:16
jelmermeuserj: please file a bug report, this looks like a regression in bzr15:18
meuserjjelmer: a little more experimentation shows that it is the revno:-1 option which is causing the crash15:28
=== BasicPRO is now known as BasicOSX
meuserjah, and it only happens when I use bzr://.. other remote protocols, including bzr+ssh seem to work.15:35
jelmermeuserj: ah, odd15:57
jelmermeuserj: definitely seems like a bug then15:57
jelmermeuserj: the fact that it only occurs with bzr:// probably explains why nobody has noticed it before15:57
meuserjjelmer: yeah I assumed that bzr:// isn't used often.. the only reason I use it is that the machine hosting Redmine is different than the machine hosting the central repository, and it's easier to use the bzr server than try to set up ssh keys for the web server user.15:59
dgouletfrom a working tree, if I want to revert a specific file to a lower revision, how can I make that happen because i'm having no result from wt.revert()... ?16:06
meuserjjelmer: wow.. yet another variable.. it only happens with repositories served via bzr-server, not individual branches16:13
manueehi everyone, brand new user here... silly question: if i do bzr ignore dirname, is everything under that directory ignored recursively?16:21
Takdgoulet: I'm using RevisionTree.revert()16:21
vxnickmanuee: yes16:21
manueeow awesome16:21
manueei was wondering because olive lists the files under those dirs as unknown instead of ignored16:21
manueeso i was geting confused16:21
vxnickah :)16:22
manueethanks vxnick =)16:22
dgouletyess that's what I though but when I try to get the RevisionTree from my WorkingTree, it always tell me that NoSuchRevision...16:22
dgouleti'm doing something bad but I can,t figure it out16:22
manueeim planing on geting my drupal development under bazaar (soloing)16:22
Takwait, my bad16:22
dgouletfrom a RevisionSpec, how can I get the revision id ?16:22
jelmerdgoulet: the as_revision_id() method IIRC16:23
TakI'm getting the rev from the revision tree, then using WorkingTree.revert() with that rev16:23
dgoulet:| how?16:23
Takone sec, let me pastebin some code16:23
manueeive got my repository setup here onmy local development machine, the next step would be to set it up on the production server,... any guides/howtos/tips on how to go about it so ican push changes to my server from my local machine?16:23
manueei know its a broad question... perhaps i should study the manual a bit more16:24
manuee:X16:24
vxnickmanuee: depends if you wanna use commit/update or push/pull16:24
vxnickI think the manual has a few examples which you might find helpful16:24
manueeyeh i should probably re-read that see if i figure things out16:25
manueeow16:25
manueeserver doesnt have bzr installed :X16:25
vxnickif you want to use push/pull you don't need bzr installed on the server16:25
manueeooow nice16:26
vxnickyou can do bzr push sftp://you@server.com/path/to/repo16:26
manueepush only uploads stuf ?16:26
dgouletTak: thanks!16:26
manueei rtfm vxnick16:26
Takhttp://paste2.org/p/36691216:26
manueeheheh bbiab16:26
Takthat's what I'm doing, which may not be The Right Way16:26
Takerr, remove that [0] from the first line16:27
dgouleti figure ;) i'm trying it real quick16:27
manueereading http://bazaar-vcs.org/BazaarForWebDevs atm--- it mentions to use the bzr-push-and-update plugin16:27
vxnickmanuee: only if you have a remote working tree, which you prob don't need16:27
manueeah right16:28
manueeye only my local box will be having vc16:28
manueethanks vxnick16:28
vxnickno probs16:28
manueeim tryin to just have a simple setup, im very new to vcs :X16:29
manuee=)16:29
manueeback to reading (getin excited)16:29
vxnickmanuee: have you made the repo on your local machine already?16:30
manueehumm yes16:30
manueei ran bzr innit16:30
manueeignored everything but the custom code i want on vc16:30
manueeand added what i wanted16:30
vxnickcommitted?16:30
manueei committed the first yes16:30
manueei havent created a branch though16:30
manueenot sure how that goes :X16:30
vxnickdo this:16:30
vxnickcheck the manual for bzr push - this will upload your repo to your server. If you want to get a copy of it on other machines, you use bzr branch on the URL16:31
manueebzr info returns Standalone tree (format: pack-0.92)16:32
vxnickso when you make changes in future, commit then push again (you don't need to specify the server details each time), then others can use bzr pull to grab your changes16:32
manueeok bzr help push --- reading...16:32
dgouletTak: it's working very well!16:34
dgouletthanks a lot! I appreciate it!!16:34
Taknp16:35
manueeow ok, bzr remove actualy deletes the files16:35
manueelol good go know16:35
vxnickyou can use --keep to keep them16:36
manueeah awesome16:36
manueeim thinkin im only going to vc the theme for now16:36
=== abentley1 is now known as abentley
vxnickyou may as well do the whole thing, makes life easier :)16:36
vxnickand bzr revert should bring back those deleted files, if you want them16:37
manueeah heheh16:38
manueevery handy16:40
manueeok moment of truth16:46
manueeim using bzr push --creat-prefix sftp:/blabla --use-existing-dir16:47
manueei hope it puts things into its place :X16:47
vxnick--use-existing-dir seems redundant as --create-prefix implies that you're pushing to a path that might not already exist16:48
vxnickbut don't worry about it16:48
manueei didnt use it the first time and it got me an error saying it doesnt have a valid .bzr directory16:49
vxnickah, ok16:50
manuee"Created new branch."16:51
manueelooks like it worked no?16:51
vxnickyou can test by branching it to another directory and making sure it pulls all the files16:51
manueehumm k i dont understand the whole branching deal it looks like16:52
vxnickbranching means to copy the repository to another location16:53
=== deryck is now known as deryck[lunch]
sohail|laptopgetting a wierd problem. I can use putty to connect to my host but when I do bzr+ssh, I get bzr: ERROR: socket.error: Socket is closed17:09
sohail|laptopthe host that I am trying to bzr+ssh to is a mac17:09
manueehmmm k it isnt working17:09
manueei made a small change to a file to test, commited the change, ran bzr push, which looked ok, but the file didnt get updated on the server17:10
sohail|laptophmm, sftp works17:10
sohail|laptopoh well17:10
vxnickmanuee: the files don't actually end up on the server, just the contents of .bzr17:11
manueeow17:11
manueehumm so im missing a step then?17:11
vxnickif you branch it though you'll see them :)17:11
vxnicknope you've done it correctly17:11
vxnickif you actually want the files to be uploaded to the server, look at the bzr-upload plugin17:12
manueeow ok17:15
manueethanks vxnick17:15
=== kiko is now known as kiko-fud
vxnickno prob :)17:15
manueehttp://bazaar-vcs.org/BazaarUploadForWebDev =))17:17
manuee"No uploaded revision id found, switching to full upload"17:18
vxnickthat's fine17:18
manueeah its cause its the first time right17:19
vxnickyeah17:19
manueeoki17:19
manueew00t its working17:24
manueei dont htink i actualy need to push anything right17:24
manueesince im not versioning files over there17:25
vxnickyou don't need to push it if your'e the only one using the repo17:27
manueeah k, i think im starting to understand things17:29
manueeso to remove what i did using push, i just delete the .bzr directory on the server, and all is like before there17:30
vxnickyep17:30
manueeits nice it only adds it at one location, cvs is crazy like that (which is what the drupal proyect is under sigh)17:30
vxnickhehe17:30
manueei can barely get my head around cvs to maintain a couple of proyects i have on there17:30
manueeeverytime i have to look up the docs lol17:31
vxnickluckily bzr is a lot simpler :)17:31
manueeno doubt17:31
manueethanks a ton for your help & patience vxnick17:31
manueeyou helped a lot17:31
vxnicka pleasure17:32
=== beuno is now known as beuno-lunch
=== vxnick is now known as vxnick-AFK
=== deryck[lunch] is now known as deryck
SamBjelmer: so ... I pushed up some code to log SVN editor activity on lp:~naesten/bzr-svn/355143-replay/ ... I added it to PathStrippingDirectoryEditor and PathStrippingEditor. Not sure if that was the best place, thogh... what do you think?18:28
* SamB wonders how to set a PDB breakpoint in code he can't/shouldn't edit without using the PDB shell ...18:31
LarstiQSamB: edit code; import ipdb; ipdb.set_trace() ?18:35
SamBLarstiQ: I hesitate to do that to a module in the globally-installed standard library ;-)18:38
LarstiQSamB: be fearless! ;)18:39
LarstiQSamB: how about starting it with the pdb command then?18:39
LarstiQyou can set a break, and then continue18:39
SamBI guess I could do that18:40
SamBsomehow I'd gotten it into my head that that would not work ;-)18:40
SamBsay, where is debugging of bzr documented?18:40
LarstiQSamB: HACKING I'd think18:42
LarstiQSamB: you're aware of BZR_PDB and C-\ ?18:42
SamBLarstiQ: yeah18:43
=== beuno-lunch is now known as beuno
SamBbut it might be a good idea to remind folks that they can still use plain-old pdb?18:43
LarstiQsure18:43
LarstiQand the import pdb; pdb.set_trace() routine18:44
SamBLarstiQ: though the C-\ trick I picked up only by seeing sigquit listed somewhere related to BZR_PDB, and I think I found out that C-\ is the usual way to send one from wikipedia ...18:44
LarstiQSamB: doh18:45
SamBmaybe there should be an entry in the online help too?18:45
SamBit could even just say to look in some particular other place, for all I care18:45
LarstiQSamB: it is mentioned in HACKING under Debugging18:45
LarstiQSamB: right, good idea18:46
SamBoh, speaking of the online help ...18:47
SamBhttps://bugs.launchpad.net/bzr/+bug/40898318:48
ubottuLaunchpad bug 408983 in bzr "need a "bzr apropos" command to search the topics available through "bzr help"" [Undecided,New]18:48
SamBLarstiQ: does that not seem like a good idea?18:51
SamBlike, practically essential even?18:51
LarstiQSamB: not sure about essential, but I think it is a good idea too.18:52
jelmerthis is why I don't like the online help system in bzr, we basically end up implementing our own documentation reader in the end :-(18:52
SamBjelmer: true18:52
LarstiQjelmer: I'm fine with hooking up with `info`18:53
jelmerSamB: Thanks, I'll have a look18:53
jelmersorry for answering in the wrong order :-)18:53
SamBjelmer: I'm not at all sure I've taken the right approach with that tracing, which is why I'm asking you to look at it18:54
SamBthe more I think about it, the more I feel like it might be better to create a set of wrapper classes that, when the flag is enabled, wrap around the actual editors for tracing purposes18:55
jelmerSamB: right, that's what we do for transports18:56
jelmerSamB: see MutteringTransport18:56
SamBthat was actually my first idea18:56
SamB;-)18:56
SamBjelmer: so do you think I should go ahead and do that?18:57
SamBjelmer: and if so, where should I put them?18:57
SamB(and I really wish bzr had a concept of "interfaces", if only for documentation purposes ...)18:58
SamBjelmer: hmm ... what module is that in?19:00
SamBI found something a bit different-looking in bzrlib.transport.trace...19:00
* SamB wishes bzr selftest could abbreviate test names like 'bzrlib.plugins.' => 'bp.'19:04
=== vxnick-AFK is now known as vxnick
SamBhmm, would it be allowable for bzrlib to override unittest.TestResult._exc_info_to_string() ?19:20
jelmerSamB: sure, just put them in fetch.py19:23
jelmerSamB: technically, this is a subvertpy API19:24
SamBjelmer: point19:24
SamBI realized that pretty soon after mentioning that19:24
SamBbut, if bzr used interfaces as defined by some standard and/or popular third-party library, subvertpy could too, couldn't it?19:25
SamBif only for documentation purposes?19:25
SamBjelmer: hmm, is bzr-svn considered to have a public API?19:28
=== ja1 is now known as jam
=== deryck_ is now known as deryck
jelmerSamB: other than the layout stuff, not really20:32
jelmerSamB: also, subvertpy._ra.Editor documents the Editor interface (just committed)20:35
* SamB wishes one or the other of the python modes would highlight local variables somehow ...20:53
SamBheck, even highlighting variables name and attribute names differently would be helpful ...20:54
SamBjelmer: committed where exactly ?20:57
RenatoSilvahow to rename tags?21:24
SamBdoh. committed to subvertpy, duh ;-P ...21:32
fullermdRenatoSilva: No way, the name _is_ the tag.  But you can make a new tag with a different name pointing at the same place.21:37
meuserjThe topic says that Launchpad is now open source... but I don't see anything about where to get it on launchpad.net...21:41
RenatoSilvafullermd: ok21:41
=== JaredWigmore is now known as JaredW
beunomeuserj, code.launchpad.net/launchpad21:44
meuserjduh.. it make sense that they would host their own card.21:45
meuserjerr code21:45
RenatoSilvais there a way to search for a text that was added or removed in some commit? I have a new line which introduced a bug, and I want to remove it, but before I want to know why that line was inserted. So I want to find the commit which inserted that change. Any idea?22:24
fullermdannotate will tell you when a line was last changed (which may or may not be the insertion; you can step backward if it's not)22:26
fullermdRemoved is obviously harder.22:26
SamBfullermd: I think he was hoping for a pickaxe22:32
SamBwhich would be seriously usefull, really22:33
fullermdI've got a splitting maul, is that close enough?22:37
SamBprobably not22:39
lifelessRenatoSilva: bzr search can do that22:51
RenatoSilvait's a plugin, right?22:52
lifelessyes22:52
lifelesshowever, gannotate is probably best22:53
RenatoSilvais there a way to at least show a log including the diffs? Then I could use grep...22:53
lifelessas it can step through full lines22:53
lifelesslog -p22:53
RenatoSilvaYeah, I just read this in the help, thanks!22:54
RenatoSilvalifeless: it was commit 1, and I discovered the reason it was added anyway..23:00
RenatoSilvathanks23:00
SamBjelmer: ... who is supposed to call close on an Editor ?23:04
jelmerSamB: whoever is reporting the changes23:40
SamBjelmer: well, I don't see it getting called on my tracing editors ...23:41
NoldorinLarstiQ: heya23:46
NoldorinLarstiQ: any news on the chmod issue?23:47

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