thumpersinzui: there are foundational bugs about trailing whitespace00:03
thumpersinzui: all I want is a ui=sinzui :)00:03
thumpersinzui: I'll look into the whitespace issue00:03
thumpersinzui: it appears that the multiline editor adds some carriage returns at the end for some reason00:04
thumperyet to be determined00:04
thumpermwhudson: got any comments on the pics above?00:07
sinzuithumper: The UI is r=me. You may want to verify the Comments IField is strippedText* and then you want to verify the setter does the cleaning instead of the widget. I had to do something similar in other Ajaxed forms00:11
thumpersinzui: ok, ta00:11
mwhudsonthumper: looks nice00:14
* mwhudson afk for lunch00:14
sinzuithumper: the StrippedTextLine is flawed. StrippedTextWidget is doing the work. The comment's IField type must work like c.l.field.URIField00:16
thumperanyone know our windmill xpath expressions?02:07
thumpermwhudson: you around for a pre-impl call?03:04
mwhudsonthumper: sure03:04
wgrantGnargh @ people who don't check security declarations before they export objects.08:03
lifelessgoing to ile a bug?08:13
wgrantJust done so now.08:14
wgrantbug #45883308:14
adeuringgood morning08:18
wgrantbigjools: Morning... Want to have a look at bug #458833?09:23
bigjoolswgrant: have you tried to rocketfuel-branch today?09:52
wgrantbigjools: No, I don't normally use it.09:53
* wgrant tries09:53
bigjoolsI got an error when it builds the wadl - "ImportError: No module named apt_pkg"09:53
wgrantI know why that is.09:54
bigjoolsmissing __init_ ?09:54
wgrantThere's a new python-apt in primary.09:54
wgrantWhich supersedes the one in launchpad/ppa which supports python2.4.09:54
wgrantmaxb usually updates his, which then gets copied into launchpad/ppa, but it's easy enough for anybody to do it.09:55
wgrantFor now, you can just downgrade.09:55
jmlrockstar, bzr-removable09:55
gmballenap: I've a question about utilities/ec2 if you've the time.10:35
allenapgmb: Sure.10:35
gmballenap: What Python should ec2 be using? It breaks with 2.4 because of the PQM issue and it breaks with the system python because of something to do with paramiko (don't have a traceback to hand at the moment).10:37
gmballenap: It does work with python 2.5 but I figured we were hoping to skip that and go straight to 2.6 with the rest of LP.10:37
allenapgmb: It'll work with either 2.5 or 2.6.10:38
gmballenap: Oh, okay. My paramiko error must've been caused by something else. Cool, thanks.10:39
allenapgmb: The utilities/ec2 script is a bit broken right now. See "Problems with utilities/ec2 test -s "..." on Jaunty (and possibly on Karmic too)?" on launchpad-dev@.10:39
gmballenap: Yeah, test -s is what broke for me. Using the version in an old branch of stable worked for me though.10:40
wgrantIs a separate 2.6 branch going to be maintained for 6 months?10:40
wgrantISTM that neither trunk can be moved to 2.6 before Lucid.10:40
allenapwgrant: Because 2.6 isn't on hardy?10:43
wgrantallenap: Right.10:43
mwhudsonwgrant: there is A Plan10:44
mwhudsonwhich involves a ppa10:44
allenapwgrant: I don't think a little thing like that will stop them :)10:44
wgrantmwhudson: Urgh.10:44
deryckMorning, all11:02
maxbbigjools: hmm..... but how did you get that newer python-apt installed?!  the requirements of launchpad-developer-dependencies should have forced you to stick with the old version until the PPA was updated!11:04
* bigjools shrugs11:04
maxbMy updating just now has presented me it as a held-back package (which is what prompts me to go and update it11:05
=== henninge_ is now known as henninge
thumperderyck: hey, you might want to look at my lazr-js branch too11:49
deryckthumper, ok, will do.  do you do any work on this patching problem?  Or just fixes to get your stuff working?11:51
thumperderyck: the lazr-js branch is all css styling to use a class rather than an id11:51
thumperderyck: mwhudson has looked at it11:52
thumperderyck: but i was wondering about replacing rather than adding css styles11:52
thumperderyck: in my branch multiline edit branch I add the lazr-multiline-edit class to the bugs page too11:52
thumperderyck: in preparation for the lazr-js fix11:52
deryckthumper, ok, cool.  thanks for making this better. :)11:53
thumperderyck: np, I'm off now, it is almost midnight11:53
deryckcool, enjoy the weekend.11:56
=== matsubara-afk is now known as matsubara
=== mrevell is now known as mrevell-lunch
bigjoolsmaxb: did you figure out why your PPA package wants to get upgraded to karmic's?13:13
maxbIt doesn't, for me :-)13:14
bigjoolsmaxb: I downgraded and it wants to upgrade again for me, is there any debugging I can do for you?13:14
maxbWhat wants to upgrade it? apt-get ? aptitude? update-manager?13:15
bigjoolstried the first 2 and they both want to13:15
maxbThis makes no sense13:18
bigjoolsaptitude says: python-apt [ ->]13:18
maxbAfter upgrading to vanilla karmic, the dependencies of launchpad-developer-dependencies would be violated13:18
leonardrallenap, when you are around would you check and make sure that bug 451158 is fixed in the version of launchpad i just released?13:18
mupBug #451158: AttributeError: 'Credentials' object has no attribute 'authorizeSession' <launchpadlib :Fix Released> <https://launchpad.net/bugs/451158>13:18
maxbbigjools: you do have lp-dev-deps installed?13:18
* bigjools facepams13:19
bigjoolsfacepalms, too13:19
allenapleonardr: Okay.13:19
bigjoolsguess what!13:19
bigjoolsthe karmic upgrade must have removed it :(13:19
=== mrevell-lunch is now known as mrevell
* maxb disappears for lunch13:21
bigjoolsmaxb: and now it works, thanks13:21
allenapleonardr: I get this now: http://pastebin.ubuntu.com/299769/13:44
allenapleonardr: I got the Python console by: bzr branch trunk boot && cd boot && virtualenv --no-site-packages . && bin/python bootstrap.py && bin/buildout && bin/py13:44
leonardri tested get_token_and_login but not login_with13:49
leonardri think i need to create a fake launchpad so i can write tests for these methods13:52
allenapleonardr: get_token_and_login() works fine.13:52
allenapleonardr: Something for a quiet afternoon perhaps ;)13:52
leonardryeah, i think it's just called something other than 'credentials' now13:52
leonardrallenap, it looks like the problem is that the credentials object is no longer set as 'credentials' on the service root14:05
allenapleonardr: Is is somewhere else, or is this a pita to fix?14:07
leonardrit's a bit of a pita14:07
leonardri'll probably end up doing another lazr.restfulclient release that sets it14:08
leonardri bet it would currently be available as ._browser.authorizer, but it makes more sense just to set it again14:09
leonardrallenap: would you try your code in conjunction with https://code.edge.launchpad.net/~leonardr/lazr.restfulclient/set-credentials ?14:26
leonardrand also review that branch? (i'm writing mp now)14:26
allenapleonardr: Okay.14:27
gmbOkay, can someone help my poor, tired brain: Why would bug 438985 occur in a web environment but not in the harness or in a test?14:28
mupBug #438985: Trying to make myself as bug supervisor of my project oopses <oops> <Launchpad Bugs:In Progress by gmb> <https://launchpad.net/bugs/438985>14:28
gmbOh, wait.14:28
gmbI know why.14:28
gmbIt's a DB object thing isn't it?14:28
gmbOkay, so, my next question: What's the best way for me to ensure that I've got two differnt objects that refer to the same row in the database?14:29
gmballenap: Do you know? ISTR you having similar problems a while back (I could, of course, be making this up).14:29
allenapgmb: Why do you want to ensure that you have two different objects for the same row?14:31
gmballenap: So that I can write a regression test for the above bug.14:31
allenapgmb: (a is not b and a.id == b.id)14:31
gmballenap: Right, that's what I need.14:31
gmbIs there a simple way to do it that you know of?14:31
allenapgmb: Oh, and check the table.14:32
allenapgmb: I don't really understand. If you have two objects for the same row from the same store that's a bug in Storm isn't it?14:32
gmballenap: Okay, I'm confused... I don't need to check that they're different, I need them to actually *be* different. How do I create two different object referring to the same DB row?14:32
gmballenap: Well, it might be, but the problem is that we're comparing two Person objects using 'is' instead of '==', and it breaks in the web environment but not in tests or harness (because the object stays the same between queries)14:33
gmballenap: Fixing the bug is easy. Writing the test to make sure it doesn't happen again isn't.14:33
allenapgmb: Ah, can't you just instantiate them and then set their properties? Or, if you don't need them to actually work, you could use a Snapshot.14:33
gmballenap: Ah, interesting idea. Yes, I could cheat like that, I guess :).14:34
gmb\0/ for hacks14:34
gmballenap: Thanks, I'll try that.14:34
allenapgmb: If the test is just to prevent someone from using "is" again, then it sounds like it's not worth it.14:35
gmballenap: Would you be happy for that to land, as a reviewer?14:35
sinzuibarry: stand-up14:35
gmbIt's a bit of a bugger of a thing to detect, so I'm inclined to at least try to get the test to fail.14:36
allenapgmb: I guess I don't really know how this bug plays out in the code, so I'll keep quiet from here on :)14:39
gmballenap: Fair enough. Suffice it to say it's a PITA.14:39
barrysinzui: just you and me? :)14:42
allenapleonardr: That took me a while ("now, how do I actually use an unreleased library here?") but in the end, it works fine.14:48
leonardrallenap: sorry, i could have helped oyu with that14:48
allenapleonardr: I found HackingLazrLibraries which was a good thing to read anyway.14:50
leonardrallenap: can i remind you to vote on the mp?14:51
allenapleonardr: Just done.14:52
sinzuibarry: look at lib/canonical/__init__.py14:57
sinzuibarry: I think you can filter all Set, SHA, and MD5 from there.14:57
barrysinzui: ah, very interesting.  thanks for the link.  one problem is that this won't filter anything for the twisted applications that run in a subprocess.  but i think i have hacked tachandler to do that.14:59
sinzuibarry: I think you can. There used to be hacks in the tac files for error loging14:59
sinzuiEdwinGrubbs: what is the status of bug 43650715:02
mupBug #436507: Distribution series source package page has no links to distribution source package page <post-3-ui-cleanup> <Launchpad Registry:In Progress by edwin-grubbs> <https://launchpad.net/bugs/436507>15:02
EdwinGrubbssinzui: I didn't work on it. It just needs a couple of minor changes before it can be put in review.15:02
sinzuiEdwinGrubbs: the timeout bug is more import15:03
EdwinGrubbssinzui: if I don't work on the timeout, I might get karmic retribution15:04
barryjml: ping15:17
jmlbarry, I'm on a call right now -- how can I help?15:17
barryjml: just ping me when you're free, i'd like some suggestions on how to handle a tricky twisted problem15:18
jmlbarry, ooh, fun.15:18
jmlbarry, will do.15:18
barryjml: thanks!  yes, "fun" :)15:18
=== ursula is now known as Ursinha
bachi beuno15:30
beunohi bac15:32
jmlleonardr, lazr.restfulclient's setup.py says it uses zope.interface, but zope.interface isn't used in the code15:33
leonardrjml: it's used by lazr.restful, which is a testing dependency of lazr.restfulclient15:33
leonardryou might be able to remove it from setup.py but it will still be downloaded and used15:33
jmlleonardr, ahh ok.15:34
jmlleonardr, I'm just trying to figure out what extra dependencies bzr would need if it started to use launchpadlib15:34
adeuringhenninge: r=me for your branch from this morning15:38
henningeadeuring: thank you15:54
adeuringhenninge: welcome :)15:54
barryjml: reping (if you're free)16:16
jmlbarry, like a bird on a wire16:18
jmlbarry, what's up?16:18
barryjml: maybe -> #launchpad-sprint?16:18
=== salgado is now known as salgado-lunch
BjornT_gmb: did you make any progress on bug 438985?16:25
mupBug #438985: Trying to make myself as bug supervisor of my project oopses <oops> <Launchpad Bugs:In Progress by gmb> <https://launchpad.net/bugs/438985>16:25
gmbBjornT_: I've got a fix playing in ec2 now.16:26
BjornT_gmb: i mainly wanted to know if you figured out what the problem was16:27
henningeadeuring: when approving merge proposals, it would be great if you could set the review type to "code". The new "ec2 land" command looks for that to find the reviewer for the the branch. Danke.16:28
gmbBjornT_: wgrant already did that: we were comparing two Person objects with "is" instead of "==" in the check to see whether one could subscribe the other. So even if you were subscribing yourself the comparison returned False, hence the OOPS.16:28
adeuringhenninge: thanks for the head-up. I wasn't aware to this16:28
gmbBjornT_: The problem I had was writing a suitable regression test, but allenap helped me figure that out.16:29
henningeadeuring: np16:29
henningeadeuring: I wasn't either until it just refused to work for me ... ;-)16:29
BjornT_gmb: do you know why you had different objects there? what does the test look like?16:31
gmbBjornT_: No, I don't. I'll paste a diff of the test for you, hang on...16:31
BjornT_gmb: ok. did you try passing in a security wrapped object to the method? that should do it; i'm quite sure that is want is happening.16:32
gmbBjornT_: Actually, no I didn't. The test looks like this: http://pastebin.ubuntu.com/299839/.16:33
gmbBjornT_: The security-wrapped object would probably be a better test, actually. What do you think?16:34
gmbBjornT_: Actually, I wonder, looking at it now - is there a chance that that test could pass when it shouldn't? It failed for me every time without the fix, but perhaps I'm being overly optimistic about it.16:35
BjornT_gmb: makePerson() is buggy. it strips the security proxy from the created person16:36
BjornT_gmb: there is no rationale for stripping it (except for modifying some attributes), so i would try fixing it. if that fails, you should add an XXX, explaining why you get the created person from the db again.16:38
barryabentley: ping16:38
gmbBjornT_: Okay, will do. Thanks.16:38
abentleybarry: pong16:38
barryabentley: hi.  i'm having a problem with 'bzr shelve somefile'.  it asks me whether i want to shelve it, i say yes, but then nothing gets put on the shelve and the change in the working tree doesn't get backed out.  any idea what's going on?16:39
barryabentley: bzr shelve --all works fine though (but it's too coarse for what i need to do now)16:39
abentleybarry: And does "bzr shelve" work if you shelve only somefile?16:40
barryabentley: no16:40
james_wbarry: do you press "y" twice in the case where it doesn't work?16:40
abentleybarry: I'm not sure why you would get that.16:40
james_wal-maisan: we just tested your permission change on testing and it works, thanks16:41
barryjames_w: it only asks me once, so i only hit 'y' once16:41
james_wbarry: does it show a diff?16:41
abentleybarry: That's very odd.16:41
barryjames_w: it does!16:41
barryabentley: yeah16:41
abentleybarry: It should always confirm the overall shelve operation.16:41
barryabentley: i never get that last confirmation16:42
barryabentley: unless i use --all16:42
barryShelve? [yNfq?]y16:42
barrySelected changes:16:42
barry M  lib/canonical/launchpad/daemons/tachandler.py16:42
barry 16:42
barrythen it exits16:42
barry% bzr shelve --list16:43
barryNo shelved changes.16:43
abentleybarry: Are you hitting lowercase y?16:43
barryabentley: yep16:43
abentleyDoes it say "no changes to shelve" before it quits?16:43
barryabentley: nope16:44
barryoh wow16:44
barryabentley, james_w: um, if i run it in a gnome-terminal instead of an emacs shell buffer, i get the confirmation message!16:44
barryand when i give the second 'y', it works16:45
barryjames_w: shall i file a bug on bzr?16:45
abentleybarry: Please do.16:45
barryabentley, james_w thanks!16:45
abentleybarry: I'm not sure-- it could be caused by the way prompts are written/overwritten with \r.16:46
barryabentley: very possibly so16:46
abentleybarry: Or it could be the way we retrieve a single character.16:46
abentleybarry: If you want to play with it, it's in bzrlib.shelf_ui.Shelver.prompt16:48
barryabentley: thanks.  my stack is deep right now so i'll just report the bug and move on ;)16:48
abentleybarry: Understood.  Wouldn't want you to get a recursion traceback.16:49
barryabentley: MemoryError16:49
barryabentley, james_w: https://bugs.edge.launchpad.net/bzr/+bug/45918516:51
mupBug #459185: 'bzr shelve' with confirmation fails in an emacs shell buffer <Bazaar:New> <https://launchpad.net/bugs/459185>16:51
=== matsubara is now known as matsubara-lunch
=== danilo__ is now known as danilos
=== salgado-lunch is now known as salgado
=== beuno is now known as beuno-lunch
=== gary_poster is now known as gary-lunch
=== dpm is now known as dpm-afk
=== matsubara-lunch is now known as matsubara
=== gary-lunch is now known as gary_poster
=== beuno-lunch is now known as beuno
rockstarabentley, could you give me hints on where to put pdb trace to find this hidden exception: http://pastebin.ubuntu.com/299933/18:48
abentleyrockstar: I could, but I've got a good guess at the cause: you haven't configured an oops directory.18:49
rockstarabentley, you mean error_dir ?18:51
abentleyrockstar: possibly18:53
rockstarabentley, looks like it, but it doesn't appear to be set in any of the other jobs in configs/schema-lazr.conf18:54
abentleyrockstar: True.18:54
rockstarabentley, configs/testrunner/launchpad-lazr.conf ?18:55
rockstarIt looks like that's what I need.  Thanks.18:55
abentleyrockstar: The error isn't accessible to you because the test is running a script.  Run the script yourself in order for pdb to work.18:55
rockstarabentley, yeah, but it didn't fail when I just ran the script.18:56
abentleyrockstar: Probably you didn't set the CONFIG environment.18:56
rockstarabentley, maybe.  Adding it to the testrunner configs made my test work.18:57
abentleyrockstar: Great.18:57
rockstarabentley, this job script stuff is quite nice once you get all the intricacies of it.18:57
abentleyThanks.  It could probably do with some reworking, because I put it together gradually.18:58
abentleyrockstar: And the oops reporter should scream if you try to configure it with None as the error_dir.18:58
rockstarabentley, yeah, probably.18:59
rockstarabentley, chat?19:13
abentleyrockstar: sure.19:13
abentleyrockstar: Where?19:15
rockstarabentley, GTalk19:16
=== salgado is now known as salgado-afk
=== matsubara is now known as matsubara-afk
=== MTeck-ricer is now known as MTecknology
mzzis it normal for rocketfuel-get to complain about "Not a branch" twice (once for shipit, once for canonical-identity-provider)?22:23
wgrantmzz: Yes. Those two components remain proprietary.22:26
mzzah, ok. I didn't notice anything breaking, just making sure I wasn't accidentally running a script normal people don't use22:26
wgrantThat script will die if one of the essential components fails to download22:27
mzzso! I'm trying to fix bug 456764. Iiuc I need to teach lazr.restful how to convert a timedelta to json first. I figured I'd just do that separately, build an egg with the version bumped, put it in lp-sourcedeps, and tell launchpad to use it (versions.cfg?). But to do that I need a python 2.4 setuptools.22:57
mupBug #456764: please expose builder queue state through the api <api> <ppa> <Soyuz:Triaged> <https://launchpad.net/bugs/456764>22:57
mzzquestions: is this at all sane, and is there a python-setuptools for python 2.4 somewhere?22:57
mzzor hmm, I guess I could be lazy and just let ez_setup grab setuptools22:59

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