/srv/irclogs.ubuntu.com/2005/12/28/#launchpad.txt

lifelessspiv: reviews for you please12:19
lifelessspiv: they are sqlobeject which I know you know12:19
spivlifeless: Ok.12:21
lifelessthanks!12:25
=== ilmari [i=ilmari@ritchie.ping.uio.no] has joined #launchpad
=== ilmari [i=ilmari@ritchie.ping.uio.no] has left #launchpad []
lifelessspiv: reviewed your branch12:43
=== poningru [n=poningru@pool-71-243-228-54.tampfl.fios.verizon.net] has joined #launchpad
spivThanks12:45
lifelessI admire your time travel machine12:45
=== doko_ [n=doko@dslb-084-059-064-159.pools.arcor-ip.net] has joined #launchpad
=== mpt [n=mpt@210-55-36-159.dialup.xtra.co.nz] has joined #launchpad
mptGoooooooooooooooood afternoon Launchpadders!01:26
lifelessmpt: hey01:28
lifelessmpt: back in .nz ?01:28
lifelessmpt: so the branch I was asking about is still listed in 'baz' format. You need to convert its entry in pending reviews01:28
mptlifeless, I realize that, which is why I was asking if pqm was off chinstrap yet01:29
mptI suppose I could convert just that branch and not my others01:30
lifelessmpt: yes, pqm is off chinstrap01:30
mptokie01:30
mptand yes, I'm back in NZ, and partly recovered from jetlag01:32
=== salgado [n=salgado@201-27-6-196.dsl.telesp.net.br] has joined #launchpad
=== stub yawns
mpthi salgado 01:54
lifelessmorning stub 01:55
salgadoyo mpt01:56
salgadohi lifeless 01:57
salgadolifeless: I replied to your email a few minutes ago01:57
lifelessthanks01:59
mptmpt@chinstrap:/home/warthogs/archives/mpt $ /home/warthogs/source/bzr.integration/bzr baz-import ./launchpad/2005-09-menus ../mpt\@canonical.com/launchpad--menus--0509 ../rocketfuel02:05
mptbzr: ERROR: pybaz.errors.NamespaceError: invalid archive name: ../mpt@canonical.com/launchpad--menus--050902:05
mpt  at /home/pqm/source/pybaz/pybaz/_builtin.py line 74402:05
mpt  in __init__02:05
=== mdke_ is now known as mdke
lifelessmpt: no ../02:08
lifelessmpt: baz archive names are not paths per se02:08
lifelessexcept when they are02:09
lifelessand they're not here :[02:09
mptlifeless, I get the same error removing either or both of the ../s02:20
=== mpt clearly needs more sleep
jameshmpt: you probably want "bzr baz-import-branch ../launchpad/2005-09-menus mpt@canonical.com/launchpad--menus--0509 ../rocketfuel"02:22
jameshto import a single branch02:22
mptoh, I got it now I think02:22
mpthmm, no02:23
=== tambaqui [n=patricia@200-208-63-72-mns.cpe.vivax.com.br] has joined #launchpad
jameshactually, remove that first "../" bit02:23
mptwell done jamesh :-)02:24
mptI missed the -branch bit02:24
salgadolifeless: would you like to talk about that refactoring now?02:24
mptbzr: ERROR: bzrlib.plugins.bzrtools.baz_import.NotPreviousImport: /home/warthogs/archives/mpt/launchpad/2005-09-menus is not the location of a previous import.02:25
stubarn't you on holidays yet salgado?02:25
salgadostub: not yet. it starts on the 26th02:26
=== stub thought 22nd
salgadohmmm. maybe the note I added to the PendingReviews is wrong?02:27
stubno - I just can't read02:29
jameshstub: if you've got a bit of time, could you do a quick review on https://chinstrap.ubuntu.com/~jamesh/pending-reviews/jamesh/launchpad/ErrorReportManagement/full-diff?02:29
stubor am confused02:29
stubSure02:29
jameshit's mostly deletes :)02:30
stubIf ErrorData.count was a property, you could have used the new sort() form avoiding a lambda. But I wouldn't bother changing it.02:35
lifelesssalgado: in 5 minutes if thats ok02:40
lifelessmpt: is it a new dir, an empty dir, or something else ?02:40
lifelessjblack: daf: LaunchpadHackingFAQ - can one of you update the bazaar instructions there to bzr ?02:41
lifeless(but let me finish my stuff first !)02:41
stubjamesh: It will need a command line argument limiting the number of days in the past the report is generated for. I can imagine we will acumulate months or years worth of exceptions on chinstrap for histerical reasons, but we really are only interested in reporting on the last week.02:45
stubjamesh: Otherwise r=stub02:45
jameshstub: you pass the yyyy-mm-dd directories as arguments to the script02:49
jameshstub: so if I only pass the last week of dates, it will generate a report on the last week's errors02:50
jamesh2005-12-* will give a december report02:50
lifelesssalgado: ok02:50
stubWe can't really cron that02:50
jameshthat's a point.02:51
jameshI think the reports have been done manually up til now02:51
salgadolifeless: after all the changes I've done, what we have left are DistributionMirrorEditView.doSchemaValidation() and DistributionMirrorAddView.doSchemaValidation() which are exactly the same code02:52
salgadobut one is in distribution.py and the other is in distributionmirror.py02:52
lifelessbut both are DistributionMirror*.doSchemaValidation02:52
salgadobecause the contexts of each view are not the same --one is a DistributionMirror and the other is a Distribution02:53
lifelesswhy are not both forms in distributionmirror.py ?02:53
salgadoI usually place all views that share the same context in the same file02:53
lifelessI can tell you what my instinct says to do here02:53
lifelessbut I suspect we need stevea to weigh in02:54
lifelessthey are both called 'Di..Mirror?View'02:54
lifelessso I would expect both to have a DistributionMirror as a context. the Add would get a NullDistributionMirror which it would only use to grab the Distribution object from, the other would have a real DistributionMirror02:55
lifelessI think that Mark did something like that for new objects02:55
lifelesscreate-on-the-fly.02:55
salgadohmmm. I never heard of it02:55
lifelessbut if we dont have that facility handy, then I would still put them both in distributionmirror.py02:55
lifelessand the schema validation does not look at context, so its fine.02:56
lifelessif however that goes against current layout requirements (it may)02:56
lifelessthen I would extract the schema validation logic to a new object - a 'schema'02:56
lifelessand in both forms call 'DistributionMirrorSchema.validate(form_fields)'02:57
lifelessin doSchemaValidation02:57
lifelessBjornT: around? does z3 have something like this built-in ?02:57
salgadothat could even live in interfaces/distributionmirror.py, I guess02:58
lifelesscould do. I don't know the z3 idioms well enough to really comment there.02:59
salgadoor maybe interfaces/validators.py. 02:59
salgadoyes, that seems to be the right place02:59
lifelessthat sounds more like it to me02:59
salgadoI can even move it there and keep the two view classes into different files, in case I'm right and we should actually place views that share the same context in the same file03:00
salgadomaybe spiv can help us here?03:00
lifelessspiv: ^^03:00
stubThere is a schema validation example in lib/zope/schema/README03:04
stub.txt03:04
stubStuff under interface is just interface, and schemas are extensions of interfaces. Interface validation (IFoo.providedBy(obj) ) doesn't do enough03:06
=== stub wonders wtf they want to validate the object's __dict__ rather than the object, which screws properties
salgadoeh?03:08
stubIgnore all that - the README.txt is broken and science fiction (at least in our Z3 release)03:09
stubOh... lifeless was using 'schema' in a different sense, which is naughty because it is a loaded term in Z3 development03:11
lifelessstub: sorry03:15
salgadolifeless: what about https://chinstrap.ubuntu.com/~dsilvers/paste/file8YXEAN.html in interfaces/validation.py?03:17
lifelessfor now, yes03:17
lifelessit can land like that,but I really want to get stevea to eyeball the approach03:18
salgadoI don't plan to land it now, so I can talk to steve this morning03:18
lifelessok03:18
lifelesspersonally I'd have done a class03:18
lifelesswith a validate method03:18
lifelessbut thats because I am allergic to context-free-contextual-methods03:19
=== gneuman [n=agboa@200-158-86-38.dsl.telesp.net.br] has joined #launchpad
spivlifeless, salgado: Hmm, what can I do for you?03:20
salgadospiv: first question: should we always try to keep view classes that share the same context in the same file or this is not important?03:20
dilysMerge to devel/launchpad: more ErrorReportManagement updates, r=stub (r2937: James Henstridge)03:21
spivWell, the important thing is that it should be easy to find code.03:21
spivGrouping view classes by context object is a good way to do that, but I'm not sure that it's an absolute rule.03:22
spivWhat's the alternative you're thinking of?03:22
salgadospiv: the view name. for instance, I have DistributionMirrorAddView and DistributionMirrorEditView03:24
salgadothey don't have the same context --one has a Distribution as context while the other has a DistributionMirror03:24
spivThe existing convention for FooAddView is that they are kept in browser/foo.py03:25
spivE.g. BugAttachmentAddView's context is a Bug, but it's in bugattachment.py03:26
spivWhich is where I first thought to look, without really thinking about it very hard :)03:26
lifelessahha!03:26
lifelessmy instinct was right :)03:26
spivSo I think both DistributionMirrorAddView and DistributionMirrorEditView would belong in browser/distributionmirror.py03:27
salgadospiv: the same is valid for the zcml of the page that uses the view? (in our case the zcml for the page that uses FooBarAddView would be in bar.zcml although the context is a Foo)03:28
spivsalgado: The existing convention for zcml is a bit different, it seems.03:31
spivSo e.g. BugAttachmentAddView is in bug.zcml, not bugattachment.zcml.  The ZCML seems to be strictly grouped by context object.03:31
spivI'm not sure if this was a conscious decision or not, but I'd be consistent with what we have, and talk to Steve and/or mail the list if you think it's worth changing the groupings of things in browser/ and zcml/ to be consistent.03:32
salgadoI don't think it's worth changing, but I'll talk to Steve and try to get this formalized somewhere. I think this is something important to have03:33
lifelessagreed03:33
salgadoanyway, for now, I agree with you, we should be consistent with what we have03:33
=== salgado needs to sleep now. see you guys in a while
=== salgado is now known as salgado-zzz
spivsalgado-zzz: FWIW, I agree, I think it's fine as-is too.  It would be good to have this formalised, possibly in the README.txts in the relevant directories.03:41
=== mpt [n=mpt@210-55-36-130.dialup.xtra.co.nz] has joined #launchpad
=== mdke [n=matt@ubuntu/member/mdke] has joined #launchpad
=== bradb [n=bradb@S01060014bf4ac6aa.wp.shawcable.net] has joined #launchpad
=== Mez [n=Mez@ubuntu/member/mez] has joined #launchpad
=== Wolever [n=wolever@HSE-Montreal-ppp133816.qc.sympatico.ca] has joined #launchpad
WoleverWould this be the right place to come for help with Launchpad problems?04:58
mdkeyes04:59
WoleverAllright, well, I had an account with the old Ubu ShipIt, then they switched over to Launchpad and my account did not work anymore.  I just tried to create a new account (with the same e-mail address) and I still can't login (if I click the forgot password link, it tells me that my account details have not been found)05:00
stubWolever: What email address did you try to register on Launchpad with?05:10
Woleverwolever@ftml.net05:20
stubWolever: No record of that in the system. I suspect you didn't successfully complete the registration form or something got lost in transmission.05:23
Woleverstub: Hrm, stange.  I'll try registering again and see if that fixes anything.05:24
stubThere are several other @ftml.net addresses in there, but none resemble yours05:24
WoleverAh, ok...05:24
WoleverThis is looking better... The first time I registered it didn't log me in afterwards =\05:26
WoleverHorra! It works05:26
WoleverThanks for the help.05:28
Woleverhttp://www.bitpress.com/atari/prog.mov05:44
Wolevererr05:44
Woleverwrong window =\05:44
=== mpt [n=mpt@210-54-127-78.dialup.xtra.co.nz] has joined #launchpad
=== stub [n=stub@costello.z3u.com] has joined #launchpad
=== marcolu [n=marcolu@61.151.243.154] has joined #launchpad
=== fabbione [n=fabbione@port49.ds1-van.adsl.cybercity.dk] has joined #launchpad
=== stub [n=stub@costello.z3u.com] has joined #launchpad
dilysMerge to devel/launchpad: [trivial]  Update pgrestore.py to restore PostgreSQL 7.4 dumps under PostgreSQL 8.0 (r2938)07:21
SteveAhi08:01
jameshhi SteveA 08:06
SteveAhello jamesh 08:08
SteveAi just replied to your review of mpt's capitalization branch08:08
SteveAi want to keep all source code files in launchpad in ascii08:08
SteveAthat should be part of the coding standards08:09
jameshSteveA: how about page tests?08:10
jameshsome of his changes were to switch from straight to curly quotes in the page test output08:11
jameshI'm not sure how you'd handle that differently though08:11
SteveAthese cause some problems08:22
SteveAcomparing non-ascii in the pagetest output has caused problems for kiko before08:22
lifelessSteveA: what prevents us using utf8? (I presume that ascii is a means to an end not an end itself)08:22
SteveAi think we ended up doing   output = http(....) ; '\{curlyquote} blah blach {curlyquote}' in output; True08:23
SteveAlifeless: if we start using utf-8 in source files, then we get various problems.  characters start appearing in source files that aren't what they appear to be at first glance.08:24
SteveAsuch as quotes that aren't the quote character08:24
SteveAoutput of things like diffs that pass through various systems gets mangled08:25
SteveAit's not like this is a new thing.  the coding standard for launchpad has always been ascii-only source files.08:26
lifelessI was just curious08:26
SteveAif we want to change this, we should do so in a well-considered way, and not just because mpt wants it for curly quotes08:26
SteveAor scott wants it for the copyright-c-circle symbol08:26
lifelessas, AFAIK, our entire toolchain these days is utf8 safe08:26
SteveAi remain sceptical, but i'd like to be proved wrong.08:27
lifelessI don't particularly care myself ;).08:27
SteveAbut, even so, my point about having characters that don't look as they appear to be stands08:27
SteveAonce i redid some tests that daniel had written08:27
lifelessif we have time at some point, get can look into testing the toolchain we care about08:27
SteveAthey were testing that something took appropriate unicode input and did the right thing with it08:28
SteveAthese tests were in utf-8 encoded source files08:28
SteveAusing, in the source file, various different unicode features such as combining diacritics08:28
lifelessdo you want an ascii lint enforced on commit to pqm ?08:28
SteveAso that when reading the tests, even in a utf-8 capable editor, it wasn't clear what the input was08:29
SteveAnor how to maintain it, unless you're a utf-editor-jockey08:29
SteveAi don't think we need an ascii lint.  having it in the coding standards is enough08:29
SteveAand there may be exceptional cases were we want to use it08:30
SteveAan example would be a module written externally that we want in the 'contrib' package08:30
lifelessI think its the sort of thing that people forget08:30
lifelessand doing what we have done before, with a list of exceptions, and a check, would provide a small safety net with documentation.08:31
SteveAdo you think ascii lint would be a good thing?08:31
lifelessyes. It lowers the things people can get wrong by mistake.08:31
SteveAthen sure, go ahead08:31
lifelessI'll open a bug08:32
lifelessI think its a low-priority thing08:32
SteveAok08:34
lifelessbug 606508:34
UbugtuMalone bug #6065: non-ascii files should be checked for before pqm commits In: launchpad (upstream), Severity: Normal, Assigned to: Nobody, Status: New https://launchpad.net/bugs/606508:34
=== xester good morning
=== tomatox [n=tomatox1@a84-231-196-83.elisa-laajakaista.fi] has joined #launchpad
jameshspiv: ping?08:58
spivjamesh: pong08:58
jameshspiv: was the selectOne() usage really that weird?  It seemed pretty natural to me08:59
spivjamesh: Well, it seemed weird to me.  :)  But, I'm not sure it really is that weird.09:00
spivjamesh: Which is why I suggested asking a few other people and seeing what they think.09:00
spivI'm happy to readjust my brain if it makes sense to other people.09:00
stubI believe Unicode aware editors are free to change the normalization if they feel like it, which will break tests doing byte comparisons.09:09
=== mpt [n=mpt@210-55-161-137.dialup.xtra.co.nz] has joined #launchpad
stubBug 515609:13
UbugtuMalone bug #5156: baz-import fails with AttributeError In: bzr (upstream), Severity: Normal, Assigned to: Nobody, Status: New https://launchpad.net/bugs/515609:13
stublifeless: If baz-import bzr or bzrtools?09:14
jameshbzrtools, iirc09:17
stubBah - no bzrtools product in Launchpad anyway09:19
jameshcreate one then :)09:20
stubProbably better in bzr anyway - most likely it is a bzr version thing. Just thinking of reasons it might now have been triaged for three weeks (I could, but that would be rude).09:23
jameshstub: what's your opinion on using selectOne() with a query that would return multiple results but you've used limit=1?09:28
stubFine09:28
stubActually I'll qualify that. Fine if you have used an ORDER BY clause guaranteeing what result will be retrieved. Or tests will blow up.09:29
jameshsure.09:31
SteveAi think that a limit of 1 or no limit is all that makes sense09:58
SteveAthe case where there's a limit of 1 and an orderby clause is really saying selectFirst09:59
SteveAso maybe it is confusing to use selectOne for this09:59
SteveAselectFirst would have a limit of 1 hardcoded, and would require an orderby09:59
SteveAand selectOne can remain simpler10:00
=== SteveA wonders what stub and jamesh and spiv think
=== stub shrugs
stubI don't see much gain having selectFirst around - may be more precise, but adds another method to remember when to use.10:02
SteveAi'm reading spiv's original code review of jamesh's changes.  spiv said that it took him a while to see what was going on with using selectOne to get the first result.10:04
stubselectOne we introduced because select(...)[0]  was a common idiom and hid bugs10:04
SteveAi think selectOne's specific semantics should be for cases where the application says there should only ever be one or zero results10:04
SteveAselecting the first only doesn't fall into this usecase, except in one edge-case10:05
stubwhich is exactly what is happening when you have LIMIT 1 in your query, which is a common idiom10:05
stub(eg. "SELECT foo FROM bar ORDER BY foo LIMIT 1" is a common way of retreieving the smallest foo)10:05
SteveAi think application code reads more clearly using selectFirst than selectOne with a limit.10:06
SteveAi'm interested to hear other opinions.10:06
stubI'm -0 anyway. Not particularly fussed10:06
SteveAi guess selectOneOnly is what i mean10:07
SteveAdaf: hello, around?10:08
jameshSteveA: the cases I used selectOne with limit were cases of "get the smallest" or "get the newest" or similar10:08
jamesh(well, the code was already doing that10:08
stubI think selectFirst would be correct, which simply does assert orderBy is not None and  returns selectOne (limit=1)  if you want something like this.10:08
jameshspiv: does selectFirst() with stub's semantics sound good?10:09
jameshI agree that the only values of limit that make sense for a select-with-single-result function are None and 110:09
SteveAi'd expect selectOne to have the options of limit=2 or limit=1, but no others.10:09
SteveAbecause, if you get 2, then it's an error10:10
jameshlimit 2?10:10
jameshah.10:10
SteveAso, rather than putting 'limit' into selectOne10:10
stubHmm... that actually could be a good optimization on the PostgreSQL end...10:10
SteveAthe semantic is "error_on_multiple_results=True" as the default we're using10:10
SteveAi'm surprised we're not using limit=2 already for selectOne10:11
SteveAso, i suppose my concern is this:10:11
SteveAright now, we use selectOne in a way where we expect it to give an error if we get 2 or more results10:11
stub(or maybe not - it is a rare case where PostgreSQL doesn't already know only one result will be returned due to UNIQUE indexes)10:11
stub(but it wouldn't hurt)10:12
SteveAand passing in limit=1 is changing that "error on multiple results" behaviour by telling postgres to only sent a maximum of one result10:12
SteveAi feel uncomfortable about that because it is mixing up the layers10:12
SteveAalthough it does get the job done10:12
SteveAand that unfomfortableness is what led me to suggest selectFirst10:13
SteveAwhich would never give that "more than one result" error10:13
stubIt is all perfectly clear if you look at it from the SQL end ;)10:13
dafSteveA: hi10:17
jameshso, selectFirst() then?10:17
SteveAdaf: want to do more rosetta bug stuff on voip today?10:20
SteveAjamesh: i'd like to get spiv's opinion, as he originally brought it up10:20
=== koke [n=koke@ubuntu/member/koke] has joined #launchpad
dafSteveA: I do want to, but I've left my headset in Cambridge10:21
SteveAdarn.  we can use irc in any case10:22
SteveAor even one-way skype ;-)10:22
daf:)10:22
dafI'm going to grab some breakfast quickly10:24
dafhow about now + 15?10:24
stubI've got a failure in xx-shipit-login.txt. Looks like login is succeeding, but the Z3 principals are not being changed causing launchbag.user to still be None10:24
stubHmm... probably my fault10:25
=== stub steps through further
SteveAdaf: ok10:28
stubWhat listens for CookieAuthLoggedInEvent ?10:33
stubfound it10:36
=== beyond [n=beyond@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
=== mpt [n=mpt@210-55-43-236.dialup.xtra.co.nz] has joined #launchpad
dafSteveA: ready when you are10:45
=== Seveas [n=seveas@ubuntu/member/seveas] has joined #launchpad
=== Nafallo_away is now known as Nafallo
dafjordi: hi, are you around?11:07
stubSteveA: What should be handling CookieAuthPrincipalIdentifiedEvent?11:13
jordidaf: pong11:15
dafjordi: hi, I've just assigned a bug to you about iso-codes for breezy being unavailable in Rosetta11:16
SteveAstub: can't remember.  i'd need to look through the code...11:16
dafjordi: I'm hoping you'll know what to do with it11:17
jordilet me have a look11:17
dafI think we should encourage people to use the support tracker for this stuff, but I'm not sure how to do that11:17
jordidaf: but that's probably carlos' court11:17
stubA CookieAuthPrincipalIdentifiedEvent is being published, and seems to be handled by authentication.py's handle method. This method happily ignores event.principal, tries to autheticate, and fails. 11:18
stubI suspect there is another handler that should be being invoked11:18
jameshstub: staging.ubuntu.com is giving a Bad Gateway error.  Is that a known problem?11:19
stubjamesh: As I mentioned in the meeting yesterday, expect staging to be down for the next few days. Postgres7.4-8 migration stuff11:20
jameshokay11:20
spivjamesh: Sorry, I was at dinner.11:21
spivjamesh: Yes, selectFirst sounds good to me.11:21
SteveAstub: what is emiting the event?11:23
SteveAstub: can we not have a nice page saying "staging is down because..." ?11:24
stubSteveA: I havn't even got the rt job for that done on production and you want it on staging?11:26
SteveAi want the moon on a string11:27
stubShipitLoginView is a subclass of LoginOrRegister. LoginOrRegister.process_form is not logging in, causing this for fail. 11:28
stubIt happily authenticates the user, and calls logInPerson() in login.py, giving a valid principal11:29
SteveAwhen we have a request where we have a cookie login form11:30
SteveAwhen we have a request where we have the cookie login form POST, rather11:30
SteveAthen we have what is initially an anonymous request11:30
SteveAit is handled by the view, which interprets the form, checks the details, and sets session data11:30
SteveAfrom that point on, we also want the user to be logged in11:31
stubYup. All that is working.11:31
stubBut they arn't logged in11:31
SteveAso, that's when the CAPIEvent is sent11:31
SteveAi think that's meant to log them in... the handler for that11:31
stubIt isn't senting a PrincipalIdentified event, it is sending a CookieAuthLoggedInEvent11:31
stubWhich is handled by authentication.py's handle() function11:32
SteveAbut just in shipit?11:32
stubThe page test that is failing is a shipit one11:33
stubxx-shipit-login.txt11:33
stubBut logInPerson() always sends a CookkieAuthLoggedInEvent11:34
stubWhich is odd, because logInPerson has been passed a principal to log in, but doesn't propogate it making me think that the wrong event is being published?11:37
stubI see... then the authenticate method is invokved, basic auth fails, falls through to cookie auth, which first tries to pull the auth data out of the session that we just shoved in there.11:38
stubWhich is where my code changes might start causing problems11:40
SteveAaha11:40
stubLooks like a bug in the new session machinery11:47
=== stub tries to write a test case to reproduce the loss of stickyness
=== jinty [n=jinty@205.134.224.215] has joined #launchpad
stubIf you construct your session data storage area, set stuff, and then construct it again in the same request you won't see your changes.11:49
stubAhh... but maybe only when you don't have a session cookie set.11:54
SteveAit would be nice to somewhere somehow detect refusal of the session cookie11:57
SteveAso we can say "screw you hippie" 11:57
SteveAand later on, offer basic auth instead11:57
dafmpt: yo11:59
SteveABjornT: hi12:03
SteveAhttps://launchpad.net/products/rosetta/+bug/249712:03
UbugtuError: I cannot access this bug12:03
BjornThi SteveA 12:04
SteveAcan we do better at stripping GPG stuff?12:04
dafthe "This is an OpenPGP/MIME..." thing gets in the way of reading the bug description12:05
SteveAthere's also the signature .asc12:05
SteveAwhich isn't all that important12:05
BjornTyes we can. 12:06
BjornTthe "This is an OpenPGP/MIME" stuff shouldn't appear in new comments12:06
SteveAcool12:06
BjornTthe signature.asc is easy to get rid off. currently IMessage.contents include all the message's parts, when it should only include text/plain parts12:07
BjornTi'm not sure if we have a use case for the current IMessage.contents, but we could add an IMessage.text_contents, that would contain only the text parts of the message.12:10
SteveAsounds good12:10
BjornTthat should get rid of most GPG stuff. inline signatures will still appear in comments, though. we could get rid of those as well, but it's a little bit trickier.12:10
BjornTthere should be a bug open for this12:11
BjornTbug 2653, i'll see if i will have time to fix it today12:12
UbugtuMalone bug #2653: Strip signature.asc attachments from e-mailed comments and bug reports In: malone (upstream), Severity: Normal, Assigned to: Bjrn Tillenius, Status: Accepted https://launchpad.net/bugs/265312:12
=== mpt [n=mpt@210-55-161-149.dialup.xtra.co.nz] has joined #launchpad
SteveABjornT: https://launchpad.net/products/rosetta/+bug/293412:18
UbugtuError: I cannot access this bug12:18
SteveAi just assigned it to stub12:18
SteveAbut, the original email from stub attempted to set him as the assignee12:18
SteveAso, maybe there's a bug somewhere/12:19
SteveA?12:19
BjornTsort of. the 'assignee' command is a part of the 'affects' command, and have to appear right after it. this will be fixed in the next rollout, though.12:20
=== Seveas [n=seveas@ubuntu/member/seveas] has joined #launchpad
=== niemeyer [n=niemeyer@201.23.160.12] has joined #launchpad
=== matsubara [n=matsubar@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
matsubaragood morning!12:30
matsubarampt: Hey ho! how was your trip?12:34
=== beyond is now known as beyon-out
daflifeless: perhaps the baz stuff in the hacking faq should go to a saeparate page12:46
=== cprov [n=cprov@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
dafand be converted to bzr stuff12:47
dafhmm: https://launchpad.net/products/husna12:49
=== niemeyer [n=niemeyer@201.23.160.13] has joined #launchpad
=== salgado [n=salgado@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
mptmatsubara, tiring01:09
matsubarampt: You forgot/left over one of your packs here. A green one.01:15
matsubarampt: and 2 white blankets01:16
salgadoSteveA, around?01:17
salgadolifeless, have you seen the whole diff of the mirror-management branch?01:17
=== daf notes that https://launchpad.net/products/malone/+filebug has floated to the top of Epiphany's frequently-used URLs list
SteveAsalgado: yes, although having lunch somewhat01:27
SteveAmpt: i've been talking with daf about improving certain rosetta UI01:28
mptmatsubara, I left behind the green bag deliberately, because the vicious So Carlos footpaths broke one of its wheels01:31
mptmatsubara, I left behind the sheets deliberately too -- they're no use to me01:32
mptSteveA, which?01:33
=== mpt really should be asleep
SteveAhttps://wiki.launchpad.canonical.com/FindProgramsToTranslate01:33
SteveAalthough, the launchpad entry needs updating in that spec01:33
SteveAthis will make things more focused, and also remove a bunch of timeout errors01:34
jblacklifleess: sure01:37
jblacklifeless: i'll d that with the rocketful stuff I want to finish today01:37
=== daf wonders what "vinculate" means
dafit sounds like a perfectly plausible word01:38
dafbut I think it may be cromulent01:38
daf(https://launchpad.net/products/malone/+bug/5251)01:38
UbugtuMalone bug #5251: Bugs not vinculated In: malone (upstream), Severity: Normal, Assigned to: Nobody, Status: New https://launchpad.net/bugs/525101:38
mptI noticed that earlier01:39
=== thierry [n=thierry@modemcable084.72-131-66.mc.videotron.ca] has joined #launchpad
SteveAgoogle matches 609 vincluateds01:42
dafaha, I think it means "to link" in Spanish01:45
mptSteveA, I like the focus on workflow, but probably it could do with some simplification01:46
mptMost importantly, I doubt that translators are going to start by going from launchpad.net/ to launchpad.net/distros/ubuntu01:46
dafmm, that bit is a bit dodgy01:46
dafbut Ubuntu is a pretty big link on the front page01:47
dafs/big/prominent/01:47
mptAt least if I was wanting to translate, I would click the link that says "Translations"01:47
daftrue01:47
SteveAi think many people want to translate a particular favourite program01:47
mptespecially if I didn't realize that Launchpad hosted distributions other than Ubuntu01:47
mptso I didn't realize I *could* even go into the Ubuntu context, let alone that it was a good idea to do so01:48
dafthe "Translateble APplications" table: the "application" column seems redundant01:49
daf(https://launchpad.net/rosetta)01:49
mptyes01:49
mptthey're url-names01:50
dafsource package names, perhaps?01:51
dafor bianries01:51
=== gneuman [n=gneuman@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
dilysMerge to devel/launchpad: [r=Bjornt]  fixing bug # 6040, bug # 4810, bug # 3530 and bug # 3531. Fixing minor on buildd infrastructure (bad zipped tempfile name), Soyuz/Buildd UI improvements (build state selector, common templates and widgets, etc). (r2939: Celso Providelo)02:06
=== bradb [n=bradb@S01060014bf4ac6aa.wp.shawcable.net] has joined #launchpad
=== heyko [n=heyko@tor/session/x-da76544f3f148247] has joined #launchpad
salgadoSteveA, on of lifeless' emails for the review of my mirror-management branch asks for your thoughts. have you seen that email?02:37
salgadoSteveA, Subject: Re: Urgent review request attn: stevea02:37
SteveAi have it flagged, but not properly read02:37
salgadoanother thing I'd like to talk to you is about defining a policy for where we should place view classes02:38
=== cprov cprov-away
salgadoI thought we should always keep view classes that share the same context in the same file, but this is not always true02:38
salgadofor instance, things like BugAttachmentAddView are defined in bugattachment.py, although it's context is a Bug02:40
salgadoon the other hand, the zcml for the page that uses BugAttachmentAddView is in bug.zcml02:40
salgadoSteveA, do you think it's worth having a policy for this, and have it formalized on the wiki?02:41
SteveAsalgado: i'm working with daf on bug triage at the moment.  i'd like to talk about this with you later, when i can give my attention to it02:41
salgadoSteveA, sure02:42
=== thierry is now known as thierry_eating
jblackstevea: ping02:47
SteveAhello jblack 02:47
jblackMind if I bother you privately for a moment?02:48
SteveAdaf: can i take 5 for a chat with jblack?02:49
dafcertainly02:49
bradbSteveA: Can you add me to your queue too please? :) Questions about Bugzilla => Malone transition and implementing keywords.02:57
=== Seveas [n=seveas@ubuntu/member/seveas] has joined #launchpad
SteveAbradb: can we talk later on, after daf has finished work?03:01
bradbSteveA: Sure. What time is that, UTC?03:02
SteveAit's 2 UTC now03:04
SteveAdaf: how much longer are you working today?03:04
dafgood question03:05
dafstill another 3 hours or so, I think03:06
dafso perhaps you should talk to Brad now03:06
=== bradb is happy to do whatever works better for SteveA/daf
SteveAbrad should be around for a while i think03:07
bradbA long while, yes.03:07
SteveAso let's continue looking through rosetta bugs03:07
SteveAand i'll talk with brad later on03:07
dafdepends if it's blocking you or not03:07
dafok03:07
bradbsounds good. /me fades into the background03:07
=== GoRoDeK [n=gorodek@p5083FFA4.dip.t-dialin.net] has joined #launchpad
=== thierry_eating is now known as thierry
=== GoRoDeK [n=gorodek@p5083FFA4.dip.t-dialin.net] has joined #launchpad
SteveAbradb: want to chat now?  daf's taking a food break03:58
bradbSteveA: Sure.03:59
=== tomatox [n=tomatox1@a84-231-196-83.elisa-laajakaista.fi] has joined #launchpad
=== matsubara is now known as matsubara-lunch
SteveAdaf?04:37
dafSteve04:39
SteveAhow's your lunch going?04:39
dafit's finished04:39
SteveAcool04:39
SteveAmore bugs?04:39
dafyep04:39
=== matsubara-lunch is now known as matsubara
sivangdaf: lunch now ? :)05:04
=== sivang thinks it more of a dinner time
dafa late lunch05:06
dafdefinitely too early for dinner05:06
sivangdaf: probably :-D05:07
sivangjblack: around?05:07
sivangbtw, who of the lp folks are also off between 25-1jan ?05:08
dafalmost all of us, I think05:13
=== beyon-out is now known as beyond
=== Ubugtu [n=bugbot@ubuntu/member/seveas] has joined #launchpad
=== Ubugtu [n=bugbot@ubuntu/member/seveas] has joined #launchpad
=== Ubugtu [n=bugbot@ubuntu/member/seveas] has joined #launchpad
salgadoSteveA, do you have some time to talk about that issue now?05:59
SteveAsalgado: i'm still working with daf06:00
SteveAcan i answer the basic question, and we can have the discussion later?06:00
salgadosure06:00
SteveAwhat is the basic question?06:01
salgadoSteveA, if a view class is defined in browser/foo.py, then a page that uses that view should be declared in zcml/foo.zcml. do we have a policy like this or something similar?06:02
SteveAthe overall policy is that we put things where someone familiar with the launchpad codebase would expect to find them06:03
SteveAnow, there's an interesting case for adding things06:10
SteveAbecause the context for a view that adds things is something else06:11
SteveAsuch views to do with adding things should be grouped with the things being added, in general.06:11
salgadosometimes I find it confusing when looking for one of the add views in the thing's zcml file06:13
salgadobecause right now, these views are in the context's zcml file06:13
salgadoand when I'm trying to find the view that's used in a given page, I first grep for the page name inside zcml/*, get the filenames that match and then look into that filename inside browser/06:15
salgadobut when it's an add view, I won't find it there06:16
salgadoanyway, this is no big deal. I can live with it as it is, easily06:16
SteveAsalgado: add it as a launchpad hackers' FAQ, without an answer06:17
SteveAor with "steve should answer this!"06:17
salgadoSteveA, what I really want is your input on that email lifeless sent. I need to land that mirror-management branch before I leave. so, I'd appreciate if you could reply to that today06:18
=== salgado adds that question to the FAQ
SteveAsalgado: in the email, i don't really know what is being asked06:22
SteveAwe don't need to pay a lot of attention to handling form submissions that haven't come from the application06:22
SteveAor have come from an older version of an application06:22
SteveAso long as they won't do damaging things06:23
salgadothat won't do any damage if the form comes from an older version of the app06:24
salgadoneither if it doesn't come from the app at all06:24
SteveAthen, there is no problem.  the form handler must be polite for any data it receives from the current form06:25
SteveAbut it can give an AssertionError or something if it receives data not from the app06:25
=== Wolever [n=wolever@HSE-Montreal-ppp133816.qc.sympatico.ca] has left #launchpad []
=== eruin [n=eruin@unaffiliated/eruin] has joined #launchpad
=== bradb & # lunch
sivangdaf: have you updated the official languages in .il for rosetta ? Is that what you were pinging me about?07:01
dafer, did I ping you?07:05
dafI can't remember07:05
sivangdaf: yeah, you pinged me 2 days ago IIRC07:07
sivangdaf: there was a guy "nakee" opening a bug report about official langs in rosetta for .il07:07
dafthat was probably it07:07
dafhe said "aramic" -- do you think he meant "arabic" or "aramaic"?07:07
sivanghe was talking about aramic07:09
sivangno idea why he mentioned it07:09
sivangit's an ancient language that's not spoken anymore07:09
sivanghe also said english is an official language, while I belive it's not - as per my comment and wikipedia links on the bug report07:10
sivangso it's only arabic and hebrew07:10
dafwell, he said "Yes acording to dry law Israel gave up english as a formal language."07:10
LarstiQ"dry law"?07:11
dafbut at any rate, I don't think any English spoken in Israel would be sufficiently different to merit its own code07:11
dafI was wondering if perhaps "aramic" is a typo07:12
daffor "arabic"07:12
dafin English, there's no such word "aramic", only "aramaic"07:12
LarstiQpossibly, I think it is rather aramaic though07:12
LarstiQdaf: I would have translated it to aramic myself too, before you mentioned aramaic07:13
dafWikipedia says Aramaic is still spoken in Israel07:13
=== tambaqui [n=patricia@200-208-53-77-mns.cpe.vivax.com.br] has joined #launchpad
=== Ubugtu [n=bugbot@ubuntu/member/seveas] has joined #launchpad
=== Ubugtu [n=bugbot@ubuntu/member/seveas] has joined #launchpad
=== Ubugtu [n=bugbot@ubuntu/member/seveas] has joined #launchpad
=== Ubugtu [n=bugbot@ubuntu/member/seveas] has joined #launchpad
Seveassorry for ubugtu's flooding - testing a new version of the Malone plugin and something is wrong :)08:14
LarstiQheh08:14
dilysMerge to devel/launchpad: [r=Bjornt]  fixing bug # 6040, bug # 4810, bug # 3530 and bug # 3531. Fixing minor on buildd infrastructure (bad zipped tempfile name), Soyuz/Buildd UI improvements (build state selector, common templates and widgets, etc). (r2940)08:22
=== Ubugtu [n=bugbot@ubuntu/member/seveas] has joined #launchpad
LarstiQSeveas: ah, working again?08:59
Seveastesting now (the error was during starting/exiting, which is really annoying)09:00
=== Ubugtu [n=bugbot@ubuntu/member/seveas] has joined #launchpad
=== siretart [i=siretart@ubuntu/member/siretart] has left #launchpad []
=== Ubugtu [n=bugbot@ubuntu/member/seveas] has joined #launchpad
Seveasok, now it's working again, with shiny new code I'm no longer embarrased of :)09:11
LarstiQheh09:12
Seveasthe old plugin was a collection of dirty hacks09:13
Seveasthe new one is nice, less hackish code09:13
matsubaraSeveas: is dilys code available?09:25
SeveasI don't run dilys 09:25
Seveasbut I guess that even when it's available it's of little use since it's quite tied into launchpad, which is not yet open source09:26
matsubaraSeveas: you run ubugtu, right?09:26
Seveasyes09:27
matsubaraI see, and i think daf runs dilys. I got confused. So, Is ubugtu code available?09:28
Seveasnot yet09:28
Seveasbut Ubugtu is a supybot, so it's only my plugin that is not yet available.09:29
SeveasIt wil be when I finish the debbugs part09:29
matsubaraSeveas: interesting. I'll sure take a look at it when you release. :)09:36
Seveasping me again in a few hours, it should be done by then :)09:37
matsubaraSeveas: I'll probably not be here. But we'll look for it after the holidays.09:42
Seveashappy holidays :)09:49
Seveaspartial results09:51
Seveasdebian bug 30012309:51
UbugtuDebian bug 300123: "alt-. gives wrong signal to zsh" Package: xterm, Version: 4.3.0.dfsg.1-8;, Severity: important, Maintainer: Debian X Strike Force  http://bugs.debian.org/30012309:51
Seveasso that already works, about 30 minutes max until completion :)09:51
bradbUsing browser:generalform, the IBugTask.product field, and its SinglePopupWidget, the field is always blank when the form gets rendered. Any ideas?09:54
=== Ubugtu [n=bugbot@ubuntu/member/seveas] has joined #launchpad
bradbAh, I think I know why. GFV uses setUpWidgets and not setUpEditWidgets10:09
Seveasmatsubara, http://bugbot.ubuntulinux.nl10:12
Seveasjoin #supybot10:13
Seveasehm right, that should be with an /10:13
bradbUsing setUpEditWidgets seems to fix this problem, w00t.10:13
=== lucasvo [n=lucasvo@www.wservices.ch] has left #launchpad []
=== poningru [n=poningru@pool-71-243-246-145.tampfl.fios.verizon.net] has joined #launchpad
matsubaraSeveas: thanks, will take a close look later. 10:37
matsubaraTo all you launchpadders a Merry Christmas and a Happy New Year! See you all on the next year. Take care.10:38
Seveas!config channel Plugins.Bugtracker.snarfTarget malone10:48
UbugtuThe operation succeeded.10:48
dilysMerge to devel/launchpad: [trivial]  Update tsearch2 dictionaries when migrating a database from PostgreSQL 7.4 (r2942: Stuart Bishop)10:52
=== BjornT [n=bjorn@212.247.111.13] has joined #launchpad
=== GoRoDeK [n=gorodek@p5083F1B4.dip.t-dialin.net] has joined #launchpad

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