| exarkun | Done, https://bugs.launchpad.net/bzr/+bug/587692 | 00:06 |
|---|---|---|
| ubot5 | Launchpad bug 587692 in Bazaar "Sometimes "bzr version-info" doesn't complete (affected: 1, heat: 0)" [Undecided,New] | 00:06 |
| igc1 | morning all | 00:49 |
| === igc1 is now known as igc | ||
| poolie | hi all, igc | 00:51 |
| spiv | Good morning. | 01:03 |
| poolie | hi there spiv | 01:20 |
| poolie | what fabulous adventures will you have today? | 01:22 |
| spiv | Finishing the patch(es) for allowing bzr-builder to merge just debian/ dirs from parallel imports, I hope! | 01:26 |
| poolie | nice | 01:27 |
| lifeless | back | 02:51 |
| poolie | hi there lifeless | 03:13 |
| lifeless | hiya | 03:19 |
| poolie | hi | 03:21 |
| poolie | re high bugs | 03:21 |
| poolie | if you want to strike while the iron's hot and fix it, feel free | 03:22 |
| poolie | but it's not as critical as some others | 03:22 |
| duffyd | poolie: hi | 03:23 |
| duffyd | poolie: was just talking to lifeless on #nzpug re. finding a good hotel in brisbane | 03:23 |
| duffyd | poolie: he thought you might have some suggestions :) | 03:23 |
| lifeless | poolie: I really feel that if we allow things to slip, they stay slipped; the broken windows theory of code maintenance | 03:23 |
| lifeless | poolie: in particular I was expecting/hoping parthm would pick it up with the priority I'd set it, as he's slowly getting deeper and deeper in. | 03:24 |
| lifeless | duffyd: also wikitravel if you haven't looked there already ? | 03:24 |
| duffyd | lifeless: oh ta | 03:24 |
| poolie | re broken windows, i sympathize but the important thing is to fix them, making all bugs high won't help | 03:26 |
| lifeless | thats true | 03:26 |
| lifeless | I feel that regressions should be equal to the highest priority open bug, in essence | 03:27 |
| lifeless | I know you don't feel that way, and I don't understand why, yet. | 03:27 |
| poolie | lifeless, if "regression" means "used to work, now doesn't" | 03:30 |
| poolie | then, i agree it's an interesting thing to track | 03:31 |
| poolie | i don't think it jumps to the top of the queue | 03:31 |
| poolie | if somebody picks up a copy of bzr today and hits say 5 bugs | 03:31 |
| poolie | the fact that one of them was once working and the other was not is not super important | 03:31 |
| poolie | in a sense if you have too many of them, then it seems that your quality process is not very controlled | 03:32 |
| lifeless | thats true too | 03:32 |
| poolie | either you're lacking tests or people are careless | 03:32 |
| lifeless | Doing them first, always, should either be a small time fraction (and ok) or a burden and thus make it clear you have a problem earlier in the process | 03:33 |
| lifeless | I think its these two things that keep me coming back to this and not accepting the 'at any point in time, what is the most useful improvement to work on' argument | 03:33 |
| lifeless | I think thats a great argument when choosing new stuff to make good | 03:33 |
| lifeless | but not for choosing how to deal with things that we've permitted to slip [through whatever reason] | 03:34 |
| poolie | i think saying 'we will throttle the amount of new work by not starting any while there are regressions open' would be ok | 03:34 |
| lifeless | I expect that to add more latency to addressing regressions than I'd like us to achieve. | 03:36 |
| lifeless | anyone got python2.4 handy ? | 04:24 |
| spiv | I seem to still have it installed. | 04:26 |
| lifeless | can you do from email import Message ? | 04:29 |
| spiv | Yep. | 04:30 |
| lifeless | cool, thanks | 04:30 |
| anders_ | hi guys. | 04:41 |
| anders_ | I want to use cherrypicking, but I hope it can merge with history information. How can I do? | 04:42 |
| poolie | sorry, we don't do that at the moment | 04:42 |
| anders_ | Why not? | 04:43 |
| poolie | sorry, what's you question precisely? | 04:43 |
| anders_ | I think it is a valuable function. | 04:44 |
| poolie | i agree | 04:44 |
| anders_ | Is there any plans to implement it? | 04:44 |
| poolie | at the moment bzr doesn't track history for cherrypicks | 04:44 |
| poolie | we'd like to implement it | 04:44 |
| poolie | there is a bug you can subscribe to | 04:44 |
| anders_ | Poolie, thank you. | 04:46 |
| lifeless | poolie: so you suggested tagging regressions; did I miss ail about that ? | 04:50 |
| poolie | i didn't send mail about it yet | 04:50 |
| lifeless | kk | 04:50 |
| poolie | you can | 04:50 |
| lifeless | ok | 04:50 |
| poolie | i think the issue of where they should be prioritized is interesting but also one i don't want to spend much time on atm | 04:51 |
| poolie | weapons free to fix them on demand | 04:51 |
| lifeless | :P | 04:51 |
| poolie | by which i mean, as they come up | 04:51 |
| lifeless | I'm interested in that, but not in chasing it if noone else is; so I won't ask you to spend much time on the priority discussion, but I will kick one off on the list, unless thats a concern for you | 04:52 |
| lifeless | poolie: you expressed concerns about dirstate2's complexity | 04:56 |
| lifeless | poolie: can you be a little more detailed ? | 04:56 |
| poolie | i did- | 04:56 |
| poolie | not right now but i do want to go back and reply in that thread | 04:56 |
| lifeless | ok | 04:56 |
| poolie | i'm not totally trolling in asking | 04:56 |
| poolie | why fix this now rather than fixing regressions? | 04:57 |
| poolie | most of the bugs in dirstate have been there since it was first done | 04:57 |
| lifeless | its my fault its like this | 04:57 |
| lifeless | ok, thats a shallow answer | 04:57 |
| lifeless | uhm | 04:57 |
| poolie | no, it's a reasonable answer | 04:57 |
| lifeless | on regressions, I have a complex of feelings there | 04:58 |
| lifeless | tied into the cost of accepting contributions I guess | 04:58 |
| poolie | it's complex for me too | 05:00 |
| poolie | for me at least marking them is a start | 05:00 |
| lifeless | dirstate2 has a few interesting things | 05:01 |
| lifeless | - if its not ready when we do 3, we don't get it for X more years | 05:02 |
| lifeless | (per Mark and changing formats) | 05:02 |
| lifeless | - it affects a great number of users in varying ways, and is, I think, causing bzr explorer and tortoise-bzr users particular pain on windows | 05:02 |
| lifeless | - I really want emacs folk to be happy | 05:03 |
| poolie | i'd really like to see it fixed too | 05:04 |
| poolie | i was using gannotate the other day and it bit me | 05:04 |
| poolie | if we can finish it soon for 2.2 that would be good | 05:05 |
| poolie | i guess i'm mostly concerned about being able to deal with the fallout | 05:05 |
| poolie | - getting proper review for it, not rubberstamps | 05:05 |
| lifeless | oh, and one more thing - its in-stock inventory at the moment | 05:05 |
| poolie | - and whether it takes too much attention away from udd and other kinds of bugs | 05:05 |
| lifeless | something I abhor | 05:05 |
| poolie | mm | 05:10 |
| poolie | i don't want the fix for it to stuff more things into the pipeline than will fit, or to cause it to be overloaded as we deal with fallout | 05:10 |
| poolie | otherwise, i agree | 05:10 |
| lifeless | agreed | 05:10 |
| lifeless | Note that dirstate2 is a very specific thing | 05:10 |
| lifeless | its not a new dirstate serialiser, for instance | 05:10 |
| lifeless | John would like to see more work done on what-is-stored-where, and so would I, but I think that that sort of thing is very high risk. | 05:11 |
| lifeless | OTOH doing that might make the overall design simpler. So we should talk next Monday briefly about that. | 05:11 |
| wgrant | So, mgiuca is trying to sign the contributor agreement, but the link on http://www.canonical.com/contributors is broken. | 05:15 |
| lifeless | ugh | 05:15 |
| poolie | oops | 05:16 |
| poolie | well, it's a very attractive 404 | 05:16 |
| poolie | though not really the attitude you'd expect mark to take :) | 05:16 |
| lifeless | I'll file a bug | 05:16 |
| poolie | k | 05:17 |
| poolie | https://bugs.edge.launchpad.net/ubuntu-website-content/+filebug | 05:17 |
| lifeless | yes | 05:17 |
| wgrant | The latest copy I have is of 2.4, but 2.5 appears to be the real latest. Is there another copy of that floating around somewhere? | 05:19 |
| lifeless | http://www.canonical.com/files/Canonical%20Contributor%20Agreement,%20ver%202.5.pdf gives a 403 | 05:19 |
| poolie | i have 2.5 here | 05:19 |
| lifeless | bug 587755 | 05:19 |
| ubot5 | Launchpad bug 587755 in ubuntu-website-content "Contributor Agreement PDF link on http://www.canonical.com/contributors is broken (affected: 1, heat: 0)" [Undecided,New] https://launchpad.net/bugs/587755 | 05:19 |
| poolie | wgrant, http://sourcefrog.net/tmp/1y/Canonical%20Contributor%20Agreement,%20ver%202.5.pdf | 05:20 |
| wgrant | poolie: Pointed him to it. Thanks. | 05:21 |
| frion | so I run bzr branch lp:dcplusplus, download the patch posted at http://launchpadlibrarian.net/49406269/plugins.patch , and when I run "bzr patch plugins.patch" I get "bzr: ERROR: Error invoking patch: Invalid argument" | 05:38 |
| frion | this is with bzr 2.1.1; why might this be happening? | 05:38 |
| poolie | frion, hm, there's no other details about which argument was invalid? | 05:39 |
| poolie | maybe something in ~/.bzr.log? | 05:39 |
| frion | the only Google result for anything similar is http://www.mail-archive.com/leo-editor@googlegroups.com/msg06411.html and that sort of gives up on the actual cause (it suggests using an external patch tool) | 05:39 |
| frion | oh, didn't check, but no, not displayed, regardless of being in verbose mode or no | 05:40 |
| poolie | is this on linux? | 05:40 |
| frion | Windows | 05:40 |
| parthm | hello. i noticed that the imports in bzrlib.status and bzrlib.ignores are not lazy. is this intentional or just that no one has gotten around to doing it? i benchmarked (time) 'bzr st' with/without lazy and performance seems the same. | 05:41 |
| frion | Looking for .bzr.log actually, now. | 05:41 |
| spiv | frion: bzr --version should tell you | 05:42 |
| frion | spiv: ah, yup | 05:43 |
| spiv | frion: or just repeat the command with -Derror to force it to show the full traceback, i.e. "bzr -Derror patch plugins.patch" | 05:43 |
| frion | got the traceback already in the log | 05:43 |
| frion | http://pastebin.com/4NMfXjLF | 05:44 |
| frion | Still no obvious information on which argument is invalid, as far as I can tell | 05:45 |
| poolie | frion is this on linux? | 05:47 |
| poolie | sorry, | 05:47 |
| poolie | just saw your message | 05:47 |
| poolie | parthm, not sure off hand | 05:47 |
| poolie | if there's no comment explaining why they have to be non-lazy then there's probably no good reason | 05:48 |
| poolie | try adding stuff to test_import_tarrif | 05:48 |
| spiv | frion: it seems that trying to start a 'patch' subprocess is failing, perhaps simply because you don't have an executable called 'patch' on your PATH? | 05:49 |
| parthm | poolie: thanks. will look at it. if there are no problems i will make it lazy as i am doing some imports. | 05:49 |
| spiv | parthm, poolie: or perhaps because there's no observed benefit to making them lazy? | 05:50 |
| poolie | right | 05:50 |
| frion | ah, that's not internal to the standalone release? the first "patch" in the path seems to be in the strawberry Perl directory, though I don't know if that's a sufficient GNU patch | 05:50 |
| poolie | parthm, also, perhaps some can just be removed altogether | 05:50 |
| poolie | which is better | 05:50 |
| spiv | mwhudson's branch of pyflakes can spot unused imports (even when lazy_imports are used) | 05:51 |
| parthm | poolie: will check that too. | 05:51 |
| frion | ah that's also the only "patch" in the path | 05:51 |
| spiv | (FWIW, pyflakes doesn't warn about any unused imports for status.py or ignores.py for me) | 05:52 |
| spiv | I'm a little surprised that bzrtool's patch command calls out to an external patch tool, I would have thought bzrlib has most of that functionality builtin already. | 05:54 |
| frion | and without that in the path, it fails with some 'can't invoke patch' message, so obviously it's trying and failing and for whatever reason Perl's patch.exe isn't sufficient | 05:54 |
| frion | it does seem to be buildable from bzr's core functionality | 05:54 |
| poolie | lifeless, reading your diff now | 05:57 |
| poolie | for dirstate2 | 05:57 |
| lifeless | poolie: thanks | 05:59 |
| lifeless | I'm about to cook dinner | 05:59 |
| lifeless | then I'll be back for a bit | 05:59 |
| parthm | For a test case I need to put junk into $BZR_HOME/ignore. This seems to fail http://pastebin.com/V2VC2cBk | 06:04 |
| parthm | Am I doing something wrong? | 06:05 |
| poolie | i don't think so, but perhaps this is not a testcaseintempdir and needs to be? | 06:05 |
| parthm | poolie: This is TestCaseWithTransport. I checked the hierarchy, it derives from TestCaseInTempDir | 06:06 |
| poolie | sorry i don't know then | 06:07 |
| poolie | ah, maybe the code that creates ~/.bazaar has not run? | 06:07 |
| spiv | parthm: where are you doing this? | 06:08 |
| parthm | spiv: bzrlib.tests.blackbox.test_status.TestStatus | 06:08 |
| spiv | parthm: I mean, at what point in the test? In a test method, or overriding setUp, or? | 06:09 |
| parthm | spiv: oh. Its a test method. | 06:09 |
| parthm | I want to add junk into .bazaar/ignore and then run_bzr | 06:09 |
| spiv | Ok, so that seems reasonable. So I guess there's no .bazaar set up for soem reason, did something change there recently? | 06:10 |
| spiv | Hmm, I'm probably getting confused with the whoami change. | 06:11 |
| parthm | spiv: looking at tests/__init__.py there does seem to be code but it doesn't seem to be working. http://pastebin.com/RxVxpJz1 | 06:11 |
| parthm | spiv: whoami change touch EMAIL env variable and nothing else so I don't think it should matter. | 06:12 |
| spiv | parthm: I think perhaps you have a wrong assumption about when .bazaar/ignores is created | 06:13 |
| spiv | parthm: look at get_user_ignores() | 06:13 |
| spiv | parthm: it has the code that does the "try open that file, if it doesn't exist create it" | 06:13 |
| parthm | spiv: I am opening it in write more but the creation is failing ... http://pastebin.com/V2VC2cBk ... so looks like some dir is missing in the hierarchy | 06:14 |
| spiv | parthm: "in the write more"? | 06:14 |
| parthm | spiv: sorry. I meant, write mode for the test case. I don't care about existing content. Just want to add 1 junk entry. | 06:15 |
| spiv | parthm: use the APIs in bzrlib.ignores, don't write the file directly | 06:15 |
| spiv | parthm: compare how test_ignore works | 06:16 |
| parthm | spiv: good idea. will try that. | 06:16 |
| spiv | Note that it calls get_user_ignores, or _set_user_ignores | 06:16 |
| parthm | spiv: ok. | 06:16 |
| spiv | And _set_user_ignores explicitly does "ensure_config_dir_exists" | 06:16 |
| spiv | Or perhaps add_unique_user_ignores would be more appropriate in your case. | 06:18 |
| spiv | Anyway, you're running into trouble because you're bypassing the APIs in bzrlib.ignores which take care of this stuff for you :) | 06:18 |
| parthm | spiv: yes. that was it. add_unique_user_ignores takes care of it. thanks :) | 06:20 |
| poolie | lifeless, ok, dirstate2 comments | 07:09 |
| poolie | .. sent | 07:09 |
| lifeless | thanks | 07:09 |
| poolie | it's not inherently scary :) | 07:09 |
| poolie | there are both some substantial comments and some on the description | 07:09 |
| zpp | hi everyone... I'm looking for someone that is well versed in the bzr-svn plugin for some help with bug hunting :) | 07:09 |
| poolie | oh i meant to say thanks so: 'thanks' | 07:09 |
| poolie | lest all the criticism seem too much | 07:10 |
| poolie | zpp, jelmer's the expert; he'll probably be on in a bit | 07:12 |
| poolie | you can just ask though | 07:12 |
| lifeless | poolie: I appreciate the thorough examination | 07:14 |
| poolie | i appreciate the code and documentation, and the openness to feedback | 07:15 |
| lifeless | poolie: some bits I do disagree with (I don't want hashes in the dir with working tree config files etc, experience tells us folk facing too-many-files apply big hammers | 07:15 |
| lifeless | figuring out what makes things correct given fs nastiness is really the key bit, I think | 07:16 |
| poolie | i think so too | 07:16 |
| poolie | i had a go at listing adequately pessimistic assumptions | 07:16 |
| lifeless | I've starred your post for next pass at the code | 07:16 |
| poolie | i probably should have added that you can barely count on ordering in the normal case | 07:16 |
| lifeless | by which I mean, i won't reply till then | 07:17 |
| poolie | even without crashes network clients may be incoherent | 07:17 |
| poolie | np | 07:17 |
| lifeless | but I have read it | 07:17 |
| poolie | i may have a go at removing write-during-read | 07:17 |
| lifeless | yes | 07:17 |
| poolie | it may suck | 07:17 |
| lifeless | OTOH bad-network-protocols are bad | 07:17 |
| poolie | mm | 07:17 |
| lifeless | the problem | 07:17 |
| poolie | there's another alternative which is to just say "sorry, no" | 07:18 |
| lifeless | with write-during-read is that we aim to have our operations take < 3 seconds | 07:18 |
| poolie | and keep relying on OS locks for writing | 07:18 |
| lifeless | that would still not work on NFS, for instance. | 07:18 |
| poolie | sorry i don't follow you | 07:18 |
| lifeless | I will note that the pack repo format, which I built using an earlier version of the same algorithm has been pretty darn reliable | 07:18 |
| lifeless | sorry, changed horses | 07:19 |
| zpp | poolie: sorry... just got back :)... thanks for that | 07:40 |
| zpp | Im attempting to create a remote branch with a bound local repo & checkout and keep getting this error when I push data to the remote branch: ErrorFromSmartServer: Error received from smart server: ('error', 'Internal check failed: Cannot add revision(s) to repository: missing chk node(s) for id_to_entry maps') | 07:42 |
| vila | hi all | 07:43 |
| lifeless | ok | 07:44 |
| lifeless | EOD | 07:44 |
| lifeless | poolie: ^ | 07:44 |
| poolie | hi there vila, cheerio lifeless | 07:45 |
| spiv | zpp: ooh, interesting | 07:49 |
| spiv | zpp: I guess that's probably https://bugs.edge.launchpad.net/bzr/+bug/485601 | 07:50 |
| ubot5 | Launchpad bug 485601 in Bazaar "missing chk node(s) for id_to_entry maps (affected: 2, heat: 12)" [Medium,Incomplete] | 07:50 |
| spiv | zpp: that bug seems to be stalled on a way for bzr devs to reproduce it, though | 07:51 |
| spiv | zpp: I don't suppose your repositories are public? :) | 07:51 |
| spiv | It's interesting that bzr-svn always seems to be involved. | 07:52 |
| mohitranka | Hi... | 08:00 |
| mohitranka | Is it possible to give access to bzr without creating a user account at the system? | 08:00 |
| === radoe_ is now known as radoe | ||
| poolie | mohitranka, over ssh? | 08:07 |
| mohitranka | Poolie, yes | 08:08 |
| poolie | see http://doc.bazaar.canonical.com/bzr.dev/en/admin-guide/security.html#authentication | 08:11 |
| zpp | spiv: yep I think so too... it's causing us a lot of issues so i'm hunting it now :) | 08:18 |
| zpp | spiv: I actually believe it might be a race condition somehow... | 08:18 |
| zpp | spiv: if you see jelmer before i do... can you let him know i'm hanging around? | 08:33 |
| zpp | i have a fun way of reproducing the issue now it seems | 08:33 |
| spiv | zpp: great! | 08:37 |
| igc | night all | 08:37 |
| zpp | spiv: unfortunately it's all closed source stuff so i can't gie direct access to the repo | 08:37 |
| zpp | spiv: but it looks like under some conditions multiple svn checkins get identical bzr rev numbers... | 08:37 |
| spiv | zpp: Hmm! | 08:38 |
| zpp | spiv: and these checkins then appear in svn...but the files never appear (or the log entry) in a bzr clone of the svn repo | 08:38 |
| spiv | zpp: anyway, please post about it on the bug report. That will make sure interested people see it, and will help pool knowledge about what's going on | 08:38 |
| zpp | spiv: yep... definitely will do :) | 08:38 |
| bialix | hi all | 09:04 |
| bialix | bonjour vila | 09:04 |
| bialix | hi poolie | 09:05 |
| vila | bialix: privet Sacha :) | 09:05 |
| bialix | privet :-) | 09:05 |
| zpp | jelmer: are you around atm? | 09:05 |
| bialix | vila, at uds you asking me and Gary about adding new feature to qbzr | 09:06 |
| bialix | vila: about checking branches and their relations with their parents/submit branches | 09:06 |
| vila | bialix: yeeeees.... | 09:06 |
| vila | and ? | 09:06 |
| bialix | vila: can you file a bug report please so we don't forget it? | 09:06 |
| vila | I won't forget it (in fact I badly needed it again when my HD died), | 09:07 |
| vila | but the last discussions left me with the idea that I could start with command-line plugin | 09:08 |
| vila | if only to explore a bit what is needed there and provide low-level stuff for qbzr | 09:09 |
| vila | or any other GUI :-P | 09:09 |
| jelmer | zpp: hi | 09:10 |
| vila | bialix: I think I saw you continuing on your idea about branches in the same repo and being able to add/remove them interactively to qlog, which is also very good and complementary | 09:10 |
| zpp | jelmer: hi.... I'm currently chasing on bug 485601 atm... would you have a moment to lend a quick hand? I have a theory... but am not sure how to reproduce atm | 09:11 |
| ubot5 | Launchpad bug 485601 in Bazaar "missing chk node(s) for id_to_entry maps (affected: 2, heat: 12)" [Medium,Incomplete] https://launchpad.net/bugs/485601 | 09:11 |
| bialix | vila: ok, just head up. I recall that discussion this morning and decided to ping you anyway | 09:12 |
| vila | bialix: thanks, but don't worry, I won't forget this one | 09:13 |
| vila | bialix: it just keeps haunting me every other day | 09:13 |
| bialix | hmm, why people said that my summary about uds is very good? I've just written my personal impressions. I can't say for other: what they did, what they talking about | 09:14 |
| bialix | vila: at least summary mail to qbzr ML will be good to have, maybe somebody else has similar ideas and willing to implement ;-) | 09:15 |
| === nlisgo_ is now known as nlisgo | ||
| vila | bialix: sure, I'll try to do that as soon as I recover from the fallouts of my HD death | 09:18 |
| bialix | oh, sorry | 09:19 |
| zpp | jelmer: I believe there is a race condition in bzr-svn that can cause revisions to not be picked up by bzr from the svn repo | 09:20 |
| vila | bialix: what ??? You're the one who killed my HD ? | 09:37 |
| vila | :-) | 09:37 |
| bialix | sorry for bother you while you in such problem | 09:37 |
| fullermd | It's not dead, it's just in Vegas for the weekend. | 09:42 |
| jelmer | zpp: there is an open bug report about an issue like that | 10:12 |
| ignas | hi, I am trying to install bzr on a mac, but seem to be unable to download the dmg file | 10:12 |
| zpp | jelmer: I'm just finishing a script to reproduce it reliably, and a potential work around. I'll file a new bug and you can mark as dup if it's the same :) | 10:13 |
| ignas | I keep getting not-found error | 10:13 |
| ignas | from launchpad | 10:13 |
| vila | fullermd: you know the funny thing about that ? I'm pretty convinced now, that the OS tried to inform me about the problem via syslog looong ago (months), but always failed to do so because... the HD had parked its heads and was refusing all requests :) | 10:15 |
| jelmer | zpp: cool,t hanks | 10:15 |
| vila | fullermd: observing syslogd consuming 100% CPU is what led to me think about this scenario | 10:16 |
| vila | syslogs 100% and no disk IOs ? Wtf ?? | 10:16 |
| fullermd | vila: Well, that's why you have the OTHER hard drive in the mirror, right? | 10:19 |
| * fullermd hugs his ZFS close. | 10:20 | |
| vila | haaa, interesting solution... I still haven't played with raid so far... | 10:20 |
| * fullermd twitches. | 10:21 | |
| fullermd | I haven't built a system in more'n a decade without it. I'm not paranoid; I know for a FACT that hardware it out to get me. I don't trust hard drives as far as I can swallow them. | 10:22 |
| vila | That wasn't an option with that box anyway, I had to give it for repair as I couldn't access the single HD there (without removing the LCD and having to play with thermal glue or horrors like that) | 10:23 |
| vila | HD failures are still rare enough for me that I can cure them on a case-by-case basis, I may have to reconsider though :) | 10:25 |
| bialix | ignas: try later maybe | 10:27 |
| ignas | bialix, I thought broken link maybe ;) | 10:29 |
| bialix | which one? | 10:30 |
| bialix | if there is broken liunk then it could be a bug in lp | 10:30 |
| vila | bialix: no worries, you couldn't know :) | 10:34 |
| bialix | I'm don't | 10:39 |
| mgz | hey bialix, why did you put your diff filenames branch back to WIP? Looked good to land to me. | 11:36 |
| bialix | mgz: heya! | 12:10 |
| bialix | after some discussion I'd like to change mbcs to terminal_encoding and add --path-encoding command-line option | 12:11 |
| vila | bialix: I think it would be better to land what has been approved first. A further (smaller) submission will be easier to discuss | 13:35 |
| vila | (Just a general rule, I didn't follow closely enough to know if it was approved or if some tweaks were required) | 13:36 |
| mgz | It was basically approved, and yeah, further improvements could land on top. | 13:39 |
| bialix | then at least I should change from mbcs to terminal_encoding | 13:57 |
| bialix | mgz, vila: ^ | 13:57 |
| vila | bialix: yup, sounds fine | 14:00 |
| mgz | hmph, the problem with using terminal_encoding is you need to be sure it's actually ending up on the terminal (and in which case, you'd be better using unicode) | 14:04 |
| mgz | mbcs is at least still a reasonable option if it's going into a pipe or file | 14:04 |
| parthm | lifeless: ping | 14:09 |
| mgz | he should be sleeping, unless he's not liking his bed again. | 14:10 |
| parthm | mgz: yes .. just looked up the time in google ... 1am. i really should start taking timezones into account :) | 14:11 |
| bialix | mgz: not really. mbcs is not good choice for piping | 14:20 |
| mgz | depends where the pipe goes, which we can't know. | 14:20 |
| bialix | mgz: with terminal encoding user can at least change that encoding with chcp command | 14:20 |
| bialix | if I stick with mbcs then... | 14:21 |
| mgz | if it passes through some byte-manipulating program and ends up on the terminal anyway, we'd want terminal encoding | 14:21 |
| mgz | if it ends up in a file, we'd want mbcs (or utf8 perhaps) | 14:21 |
| bialix | mgz: I think the main use case: show me diff on terminal (or paging it via more|less) | 14:21 |
| bialix | mgz: if we want utf8, then I need --path-encoding option | 14:21 |
| mgz | if it's going into a program for processing, lots of apps assume their input is in mbcs | 14:22 |
| bialix | really | 14:22 |
| mgz | yeah, the paging is the main breakage case | 14:22 |
| mgz | more really screws things. | 14:22 |
| bialix | vila, mgz: that's why I want to implement --path-encoding for landing | 14:22 |
| mgz | but bialix, the current behaviour is a output with mixed encodings, and different from what I get if I use --diff-options=something | 14:23 |
| mgz | so, the patch as it exists is an improvement | 14:23 |
| mgz | (I happen to be in a locale where the terminal encoding equals the base encoding, so I'm a little biased :) | 14:24 |
| bialix | yes, but I've asked people in ru_bzr, and consensus seems to be using terminal_encoding by default | 14:24 |
| vila | bialix: I thought poolie wanted to do some infra work that could *also* handle --path-encoding ? | 14:24 |
| vila | bialix: so that would be cleaner in a separate submission | 14:25 |
| bialix | infra? | 14:25 |
| mgz | yeah bialix, but they're biased because they have a different terminal encoding ;) | 14:25 |
| bialix | of course | 14:25 |
| mgz | I'd prefer we did printing to the terminal "right" and solved the paging thing seperately | 14:26 |
| mgz | (there's no reason to encode to print to the terminal in windows, it's unicode anyway) | 14:27 |
| mgz | (it's just pipes that are the problem) | 14:27 |
| bialix | mgz: you know that we don't print on terminal buffer | 14:35 |
| mgz | not terribly hard to change that though, if it'd actually help things | 14:36 |
| bialix | yep, but... | 14:37 |
| vila | bialix: infrastructure to be able to -opath-encoding=xxx without the need to define --path-encoding, or something along those lines | 14:46 |
| bialix | vila: I'm not surte it will be so easy | 14:59 |
| vila | bialix: which is why it's better as a separate submission :) | 15:01 |
| bialix | OKAY I'M GIVE UP | 15:02 |
| mgz | ehehhee | 15:03 |
| mgz | we've worn him down! | 15:03 |
| mgz | vila: two things I'd like to bug you about if you've got the time | 15:05 |
| vila | bialix: :-) My feeling was that you and poolie had different pov on that, I don't want your actual submission to stay in the queue while it's sorted out, better land the already agreed upon stuff, that was my point since the beginning :) | 15:06 |
| vila | mgz: don't ask to ask, ask :) | 15:06 |
| mgz | it takes time to type! | 15:06 |
| vila | oh sorry :) | 15:06 |
| mgz | vila: 1 - I still don't quite get what state bugs should be ending up in when closed, some people have been setting milestones on things but I don't see any option to do that on launchpad or if I should be doing this | 15:07 |
| mgz | ack, visitors have just arrived, I'll have to be quick | 15:07 |
| GaryvdM | Hi bialix | 15:07 |
| vila | 1) when a patch has landed is should be 'Fix released', before that it's 'In Progress' is someone is working on it, including waiting for review | 15:08 |
| vila | mgz: we don't use 'Fix Committed' anymore | 15:08 |
| GaryvdM | I guess that you were looking the 0.14.7 release | 15:08 |
| GaryvdM | Hi all | 15:08 |
| mgz | vila: 2 - if I want to create a branch, with no tree, for a test, so that I can then try and branch from it as if it were a remote repo, what's the right mechanism for that? | 15:09 |
| vila | mgz: it's weird that you don't see milestones, it's the last column (Affects, Status, Importance, Assigned to, Milestone) | 15:09 |
| vila | mgz: branch --no-tree (IIRC) | 15:09 |
| vila | 'reconfigure --branch' may work too, but it may also be under the shared-repo control, I don't remember the exact rules there | 15:11 |
| mgz | right, but if I want to do that for a test | 15:12 |
| mgz | there must be a way not through run_bzr right? | 15:12 |
| mgz | (I'm thinking specifically of testing filenames that can't be created on a windows filesystem, then trying to create a working tree from that branch) | 15:13 |
| vila | bzrlib.branchbuilder.py, sorry I didn't realize you wanted that for tests | 15:13 |
| mgz | thanks, will look for examples on that later | 15:14 |
| vila | mgz: there are more and more examples using it as it's a good way to avoid blackbox tests | 15:14 |
| mgz | seemed like it might be the right thing but slightly low-level and weird, what with having to make the magic root entries and stuff | 15:15 |
| * mgz displays ignorance of how bzr actually *works* | 15:16 | |
| mgz | back to 1 again, eg bug 581298 | 15:16 |
| ubot5 | Launchpad bug 581298 in Bazaar "bt.test__walkdirs_win32.TestWin32Finder.test_dir fails without ever being run (affected: 1, heat: 0)" [Undecided,Fix released] https://launchpad.net/bugs/581298 | 15:16 |
| mgz | should I be setting a milestone there? | 15:17 |
| mgz | because launchpad gives me no box for doing it (it lists importance, but doesn't let me change it) | 15:17 |
| vila | mgz: 1) we set milestone when the submission lands 2) looks like you're not in the right group :-/ | 15:18 |
| mgz | okay, so should set fixed released and the milestone at the same time? | 15:18 |
| vila | you *should* be able to set the importance | 15:19 |
| vila | yup | 15:19 |
| mgz | okay, thanks! | 15:19 |
| vila | mgz: meh, you're part of the contributot agreement group and nothing else it appears, try joining some bazaar group :) | 15:20 |
| vila | mgz: and be ready to receive some related mail or double-check your subscriptions ;) | 15:21 |
| vila | mgz: see http://launchpad.net/~bzr to join | 15:23 |
| bialix | hi GaryvdM ! | 15:23 |
| bialix | GaryvdM: yes, I've looked at 0.14.7 | 15:23 |
| bialix | vila: btw, http://launchpad.net/~bzr said the team is restricted, and there is no easy way to say "I'd like to be a member"? | 15:25 |
| mgz | right, I was just about to say that. | 15:25 |
| mgz | okay, lunch for me. | 15:26 |
| bialix | bon app | 15:27 |
| GaryvdM | bialix: I'm quite sure that the 2.1 requirement was only introduced in 0.19b1. See http://bazaar.launchpad.net/~qbzr-dev/qbzr/trunk2a/annotate/head:/NEWS.txt#L58 | 15:27 |
| vila | bialix, mgz : Hmm, but reading this page: 'Membership is granted to people who've already been active on the list' so presumably poolie or lifeless can add people there | 15:28 |
| GaryvdM | http://bazaar.launchpad.net/~qbzr-dev/qbzr/trunk2a/annotate/head:/NEWS.txt#L44 (better context) | 15:28 |
| bialix | my coworker have troubles to run 0.18 on bzr 2.0 | 15:28 |
| bialix | I don't have a details, but the only one answer for 2.0 was to use qbzr 0.14 | 15:29 |
| GaryvdM | bialix: Please ask him to log a bug. | 15:29 |
| bialix | will try | 15:31 |
| GaryvdM | bialix: If we don't fix the bug, then we should may be have more strict checks. (e.g. we could have checks in 0.14.7 with both minimum and max checks) | 15:31 |
| * bialix is ok with that | 15:31 | |
| bialix | we can use bzrlib builtins check then, right? | 15:32 |
| GaryvdM | blacklisting in bzr would also be nice... | 15:32 |
| GaryvdM | bialix: Yes | 15:32 |
| bialix | hehe, we finally found use case for that | 15:32 |
| GaryvdM | bialix: There is a fix for a bug in 0.18. We just don't have a milestone... | 15:33 |
| GaryvdM | bug 580798 | 15:34 |
| ubot5 | Launchpad bug 580798 in QBzr 0.18 "treewidget shows wrong tree (affected: 1, heat: 6)" [High,Confirmed] https://launchpad.net/bugs/580798 | 15:34 |
| bialix | hmm, I don't see a new revisions in 0.18 after 0.18.6 | 15:34 |
| GaryvdM | bialix: The rev not yet in 0.18, but it was a daggy fix, so it will be really fast to merg. | 15:35 |
| GaryvdM | *merge | 15:35 |
| * GaryvdM does that now... | 15:35 | |
| GaryvdM | bialix: also bug 546843 | 15:36 |
| ubot5 | Launchpad bug 546843 in QBzr 0.18 "bzr qlog broken with new 2.2.0 bzrlib API (affected: 1, heat: 6)" [Undecided,New] https://launchpad.net/bugs/546843 | 15:36 |
| GaryvdM | That description is misleading. I'm going to change that... | 15:37 |
| bialix | I'd like to have those bugfixes in 0.18, so I can planning next release. can you add next milestone? | 15:37 |
| GaryvdM | bialix will do. | 15:37 |
| GaryvdM | bug 546843 - better now... | 15:38 |
| ubot5 | Launchpad bug 546843 in QBzr 0.18 "bzr qlog breaks when bzr-svn incomparable with bzrlib (affected: 1, heat: 6)" [Undecided,New] https://launchpad.net/bugs/546843 | 15:38 |
| bialix | I'm usually adding milestone when there is something to put into release | 15:38 |
| bialix | ok | 15:38 |
| GaryvdM | * incompatible... | 15:38 |
| GaryvdM | ok | 15:38 |
| GaryvdM | bialix: Are the qlog labels better now? | 15:40 |
| * bialix pulling new revisions | 15:42 | |
| bialix | labels still is not uniques, but there are indeed tooltips. | 15:43 |
| bialix | using tooltip is workaround | 15:44 |
| * bialix still is not happy | 15:45 | |
| bialix | I will try to look into this in next days | 15:45 |
| GaryvdM | bialix: Would you mind If I uncommited a rev in lp:qbzr/0.14 commited in the last few min? | 15:59 |
| GaryvdM | I did it :~ | 16:02 |
| bialix | bbl | 16:24 |
| === Adys_ is now known as Adys | ||
| sili | I have a shared repository where people do checkouts, commits, and also add new branches. Am I understanding correctly that a post-commit email hook cannot be installed on the repository, but a plugin should be installed on each bzr client? | 17:09 |
| Raim | you could run a cronjob with bzr-email-notifier, which is able to scan for new branches | 17:16 |
| sili | Raim: I think this will work; thanks. | 17:37 |
| sili | Does anyone know if copying a bzr shared repository with rsync is a bad idea? | 17:38 |
| sili | the copy will never be written to. | 17:38 |
| GaryvdM | sili: I don't know. | 18:04 |
| GaryvdM | sili: but there use to be a bzr rsync plugin | 18:04 |
| GaryvdM | sili: It has been deprecated in faviour of just using bzr to pull. | 18:05 |
| GaryvdM | sili__ ^ | 18:06 |
| GaryvdM | bbl | 18:17 |
| MvG | lifeless: Care to re-review https://code.launchpad.net/~gagern/bzr/bug560030-drop-zsh-completion/+merge/25955 ? Addressed your fixing request. | 18:42 |
| lifeless | MvG: thanks. The (gagern) isn't needed now, we auto-add that when sending it | 18:49 |
| lifeless | moin moin everyone | 18:50 |
| MvG | lifeless: Shall I strip it then? | 18:55 |
| MvG | Oh, I see you did. | 18:55 |
| lifeless | :> | 18:57 |
| * MvG is glad to have two out of four open bzr merge requests ready to land. | 19:00 | |
| Guest5926 | hi everyone, i'm jesse | 19:08 |
| Guest5926 | i'm thinking of using bazaar to record the history of wikipedia entries | 19:10 |
| Guest5926 | so i'm thinking of doing a sort of revision mapping | 19:10 |
| Guest5926 | are there any tutorials or dev doc i can refer to? | 19:10 |
| MvG | lifeless: thx | 20:03 |
| === kosipov is now known as kostja_osipov | ||
| lifeless | anyone else notice that commit templates add lots of vws ? | 20:33 |
| lifeless | jam: ping | 20:52 |
| jam | lifeless: officially I'm on vacation today | 20:53 |
| lifeless | jam: I'll be very quick | 20:53 |
| jam | what's up? | 20:53 |
| lifeless | I'd rathe rhav ethe Patch pilot week of the 21st, before vila inserted himself | 20:53 |
| lifeless | can I swap that and the 28th with you | 20:53 |
| jam | sure | 20:53 |
| lifeless | thanks | 20:54 |
| lifeless | now go repair your base :P | 20:54 |
| vila | :) | 20:54 |
| mgz | lifeless: "During the week I got tired of Martin[gz]'s patches" - ha, wrong move if you were tired of them! | 20:59 |
| mgz | (think I provided what was needed via email, yell if there's anything else I need to do) | 21:00 |
| lifeless | mgz: I pinged the rt ticket to say you'd added your key to launchpad | 21:03 |
| lifeless | mgz: tired-of-manually-processing | 21:03 |
| lifeless | not tired of the contributions. | 21:03 |
| mgz | I know, just teasing :) | 21:03 |
| lifeless | :P | 21:03 |
| MvG | Lifeless: Shall I change the status of bug #560030 to committed or released? | 21:26 |
| ubot5 | Launchpad bug 560030 in Bazaar "Shell completion scripts outdated (affected: 1, heat: 6)" [Medium,Confirmed] https://launchpad.net/bugs/560030 | 21:26 |
| lifeless | MvG: if the branch landed, released | 21:32 |
| lifeless | and set the milestone to 2.2b4 | 21:32 |
| MvG | Seems I can't set milestones. | 21:32 |
| lifeless | ah | 21:32 |
| lifeless | bzr-qa can, I think | 21:33 |
| lifeless | mgz: reviewed your testtools change | 21:33 |
| MvG | Do you want to set both, or shall I set the status without the milestone? | 21:33 |
| lifeless | I've done it | 21:33 |
| MvG | thx | 21:33 |
| lifeless | no probs | 21:34 |
| lifeless | thank you for doing all the hard work | 21:34 |
| MvG | Oh! LP has the arrows the wrong way round in my rendering: {OLD_VALUE} <- {NEW_VALUE} instead of {OLD_VALUE} -> {NEW_VALUE}. Is it just me? | 21:35 |
| lifeless | rendering where ? | 21:35 |
| MvG | At the bottom of the bug report, where the changes are listed after the comments. | 21:35 |
| MvG | Three boxes, for status, milestone and assignee, and all of them the same way round. | 21:36 |
| MvG | Initial change after comment 1 the same as well. | 21:36 |
| MvG | Hmmm... but the HTML reads → which in theory should be correct. | 21:37 |
| * MvG scratches his head | 21:37 | |
| lifeless | milestone: none → 2.2b4 | 21:39 |
| lifeless | old -> new | 21:40 |
| lifeless | what locale are you using ? | 21:40 |
| lifeless | text rendering ? | 21:40 |
| lifeless | are you using a RTL text layout or something ? | 21:40 |
| MvG | de_DE, but LC_MESSAGES=C, Firefox, Gentoo, would really like to know what font it used. | 21:40 |
| lifeless | please file a bug | 21:40 |
| lifeless | this would really confuse people *badly* | 21:41 |
| MvG | Copy & Paste displays correct glyph, so it's FF or some aspect of my font system, not LP. | 21:43 |
| lifeless | ok | 21:44 |
| MvG | Added Add-On to analyze problem, restarted firefox -> problem gone. | 21:46 |
| lifeless | hah | 21:47 |
| mgz | lifeless: thanks! all the comments seem reasonable, I'd like to bug you a bit more about how to write the tests but will reply in the review later | 21:49 |
| lifeless | absolutely | 21:49 |
| mgz | before that, for the *other* patch, about stripping tracebacks | 21:49 |
| lifeless | yes | 21:49 |
| mgz | would the debugging-testtools-itself concern be resolved by having the base TestCase for testtools-itself tests override the module __unittest variable back to False resolve it? | 21:50 |
| lifeless | you want to see me cry don't you | 21:51 |
| mgz | well, I'm not sure which debugging methods you're most worried about, having testtools-itself tests displaying full tracebacks seems a reasonable requirement | 21:52 |
| lifeless | I was teasing | 21:52 |
| lifeless | :P | 21:52 |
| mgz | :P | 21:52 |
| lifeless | seriously though | 21:52 |
| lifeless | the __unittest variable is a terrible hack, IMNSHO | 21:52 |
| mgz | this is not untrue, it is however, a hack that works. | 21:53 |
| lifeless | Here's what I'd like: | 21:53 |
| mgz | there are a couple of other options | 21:53 |
| lifeless | - unit-under-test frames are shown | 21:53 |
| lifeless | - infrastructure frames are not shown | 21:53 |
| lifeless | - when the same *line* is present in both sets, its still only shown for the unit-under-test frame | 21:53 |
| lifeless | - if we have two threads executing tests, they don't break each other | 21:54 |
| lifeless | - infrastructure frames from users of testtools are also not shown | 21:54 |
| lifeless | fin | 21:54 |
| lifeless | mgz: now, if this is too hard, we can look at doing something temporary | 21:57 |
| mgz | I'm still not clear from that how we differentiate | 21:58 |
| mgz | the callback-happiness of testtools is also problematic, can hop in and out of infrastructure code more than once | 21:58 |
| lifeless | hmm, I'm not sure that it does | 21:58 |
| lifeless | runtest and TestCase are both infrastructure | 21:58 |
| mgz | (but the old way also had issues, failures inside infrastructure code (for instance not removing a dir in cleanup) tended to be stripped of traceback completely) | 21:59 |
| lifeless | right | 21:59 |
| lifeless | I think its crucial that we err on too-much-info | 21:59 |
| lifeless | but I'd be delighted to reduce the error margin | 21:59 |
| mgz | so, I can recode TestResult._exc_info_to_string to do... something not TestResult._is_relevant_tb_level related (which 'd remove some magic from the other patch as a side effect), the question is what | 22:01 |
| mgz | testtools has enough levels of nested calls that it's annoying not to strip *something* even in the fail-inside-cleanup case | 22:03 |
| mgz | it's not twisted-bad, but it's still pretty unreadable | 22:03 |
| mgz | okay, I need to go off and cook now, but will think about it some more and try a couple of approaches | 22:05 |
| lifeless | feel free to just brainstorm | 22:05 |
| lifeless | I'm happy to put considerable effort into getting a good answer | 22:06 |
| mtaylor | lifeless: heyho... | 22:32 |
| mtaylor | lifeless: any idea what this means: bzr: ERROR: Tree transform is malformed [('unversioned executability', 'new-130')] | 22:32 |
| lifeless | yes | 22:36 |
| mtaylor | lifeless: awesome! | 22:36 |
| mtaylor | lifeless: (I was trying to do a merge-upstream) | 22:36 |
| lifeless | it means something is changing the +x bit on a file which is deleted / not added | 22:36 |
| lifeless | please file a bug | 22:36 |
| mtaylor | ok. will do | 22:36 |
| mtaylor | lifeless: on bzr-builddeb? or on bazaar? | 22:38 |
| lifeless | yes | 22:38 |
| lifeless | (I don't know where the bug lies, so choose one) | 22:38 |
| mtaylor | done | 22:40 |
| rryan | Hey all, anybody know the bazaar equivalent of `hg outgoing' or `hg incoming' ? | 23:02 |
| rryan | I searched a little bit but didnt find anything. | 23:04 |
| mtaylor | rryan: what do they do? | 23:05 |
| jelmer | rryan: bzr missing | 23:05 |
| jelmer | rryan: it does both "hg outgoing" and "hg incoming" | 23:05 |
| rryan | jelmer : thanks! | 23:06 |
| rryan | mtaylor : outgoing tells you which local changesets would be pushed if you pushed, and incoming does the opposite | 23:06 |
| mtaylor | rryan: gotcha. well, jelmer was quicker to answer than me - but yeah, missing will fix you up :) | 23:08 |
| mtaylor | lifeless: I figured out the cause at least - in the tree I had moved a file, license.py, to internal/licensing.py and then added a new file called license.py | 23:12 |
| mtaylor | lifeless: this caused merge upstream to become confused | 23:12 |
| mtaylor | lifeless: if I re-expressed that in the upstream bzr tree as adding a new file internal/licesnsing.py and modifying license.py with the new contents - all worked fine | 23:13 |
| lifeless | mtaylor: please add to the bug | 23:13 |
| lifeless | mtaylor: probably tarball import related | 23:13 |
| mtaylor | yup | 23:13 |
| lifeless | bbs, <-> doctors | 23:25 |
| poolie | hi all | 23:49 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!