/srv/irclogs.ubuntu.com/2005/08/17/#launchpad.txt

mpt<h2 style="margin-top: 0; padding-top: 0;">OMG WHAT A HACK</h2>, even12:05
bradbcheck now; the actions portlet ends up really high up12:06
bradbour other pages don't line up the actions portlet with the page label (or whatever you want to call that bit), they line up with the bit that appears in the body12:07
bradber, wait12:07
bradbSWEET12:08
bradbthat worked, thanks12:08
mptbradb: It shouldn't affect the branch's review/merge in any way, but it will be interesting trying to come up with meaningful text for that "General Bugs" heading 12:11
mptIs "Untargeted" wrong?12:11
bradbUntargeted is right12:11
mptThen why does bug 3 show up in "Fix wanted for Sarge" as well?12:12
bradbmpt: because bug #3 is targeted to be fixed in Sarge12:12
mptWell it can't be both targeted and untargeted12:12
mptI understand what's going on in the underlying model, I just don't know how to explain it :-)12:13
bradbyeah, it's a bit tricky12:14
=== Mez [n=Mez@cpc2-lich4-3-0-cust115.brhm.cable.ntl.com] has joined #launchpad
=== mpt curses the inventors of the Insert key, the Caps Lock key, and the Num Lock key
=== mpt [n=mpt@200-171-140-32.dsl.telesp.net.br] has left #launchpad []
=== jinty [n=jinty@205.134.224.215] has left #launchpad ["Leaving"]
=== WaterSevenUb [n=WaterSev@bl5-220-53.dsl.telepac.pt] has joined #launchpad
=== Mez [n=Mez@cpc2-lich4-3-0-cust115.brhm.cable.ntl.com] has joined #launchpad
=== stub [n=stub@203-217-37-199.dyn.iinet.net.au] has joined #launchpad
stubspiv: ping02:55
spivstub: pong02:56
stubIn current head, a person can have multiple wikinames per wiki, including the ubuntu one.02:57
stubThis currently occurs when person merging happens02:57
stubI was wondering how this would affect the moin integration02:58
jameshprobably depends on whether the authserver uses selectOne :)02:58
stubI need to determine if this is a bug or a feature ;)02:58
spivThe authserver will just grab which ever row happens to be returned first, I think.02:59
spivjamesh: The authserver doesn't use SQLObject atm, so it doesn't :)02:59
spivThere really ought to only be one wikiname for the "ubuntu" wiki.02:59
spiv(Which really means in a strict sense that it ought to be column on Person...)03:00
stubHow will it affect subscriptions etc? Will it depend on what wikiname I log in as? Or will preferences be stored against the one returned by the authserver?03:00
spivThere's some confusion here.03:00
spivSo:03:00
spiv * it doesn't affect subscriptions at all.03:00
spiv * you don't log in as a wikiname (you log in with your launchpad login, like everywhere else)03:01
spiv * launchpad-integrated moin uses the Person id for keeping track of preferences and things, the wikiname is purely for display.03:01
spivThe wikiname is only used as the thing to display in RecentChanges and the like.03:02
stubI though 'launchpad-login' meant 'your person.name, or one of your validated email addresses, or your wikiname' at the moment?03:02
spivNo, just the first two.03:02
spiv(Although LP itself only accepts validated email addresses -- we should make this consistent)03:03
stubok. So we can consider this a feature if we want, and the only glitch might be that the authserver may not consistantly return the same wikiname for a peron03:03
stub(which would be a trivial ORDER BY fix)03:04
spiv(I'd have been just as happy for the "wikiname" for our wikis to be the Person.name, but people seem to expect otherwise003:04
stubIn hindsight you might be right there.03:04
spivWell, Wikipedia/Wikitravel have heaps of happy users with names that don't conform to WikiCaps.03:05
spivIn fact, so did our old zwiki, which used person.name :)03:05
jameshspiv: would it be difficult to get launchpad integrated moin to accept the wiki name too?03:05
spivjamesh: Not at all -- but the existing inconsistency bugs me, and I don't like the idea of making it worse.03:05
jameshokay03:05
stubwhich inconsistency do you refer to?03:06
spivPurely from a UI standpoint -- having a unified login that then behaves differently seems a bit silly :)03:06
jameshwikipedia doesn't do auto linking of StudlyCaps, so having non StudlyCaps wiki names isn't as big a deal03:06
jameshspiv: I was suggesting allowing either an email address or the wiki name03:07
spivstub: https://launchpad.net/ accepting different logins to the wikis (and anything else using the authserver)03:07
jameshspiv: that way it'll work for people who see it as "launchpad login", and for people who see it as "moin login"03:07
spivstub: Saying "Use your Launchpad login, only different" on the login form would be silly ;)03:07
spivjamesh: It's pretty explicitly a Launchpad login.03:07
spivTo the extent that to create a wiki account it says "Go here and sign up with Launchpad."03:08
stubI see. Yes, launchpad and authserver should use the same 'username' definition. No reason for them to be different. No reason they can't all use any of the unique keys associated with a person (name, emails, wikinames)03:08
stubI'll open a bug on that03:08
spivYeah, I'm happy for it to include wikiname, so long as the web app does too :)03:09
stubSo allowing multiple wikinames for our internal wikis is utterly pointless?03:11
spivjamesh: Yeah.  Sometimes I think people confuse auto-linking StudlyCaps as being an integral part of being a wiki, when the important part is just that every page is editable.  (linking should be easy, of course, but ["some foo"]  isn't significantly harder than SomeFoo).03:11
jameshemail addresses are distinguishable from launchpad usernames and wiki names03:11
spivstub: Yeah.  If only one of them is ever going to be displayed in page metadata, there's not much point.03:11
jameshbut launchpad user names aren't necessarily distinguishable from wiki names03:12
jameshwhich would be a problem03:12
spivWell, lp user names must be lower case, iirc.03:12
jameshyou'd only want to allow one of those at most as a username03:12
spivAnd by convention wikinames have caps in them, but that's not enforced...03:12
stubI think StudlyCaps is pretty important - it makes wiki pages trivially linkable, less likely to get typos and nicer urls03:13
jameshokay.  So if we require a leading capital on the Ubuntu wiki name then they are definitely distinguishable03:14
spivstub: Wikipedia disproves that, I think.  I may depend somewhat on the audience and activeness of the wiki, though.03:14
spivs/I may/It may/03:15
stubwikipedia solves the same issues in otherways. it controls the wikinames and urls that get generated. It doesn't rely on someone typing ["This is a wiki page"]  to generate a new document.03:16
spivIt does?  I wasn't aware of any differences -- you can link to pages that don't exist, and if you visit a page that doesn't exist, you're prompted to create it if you like.03:17
spiv(I'm more familiar with Wikitravel than Wikipedia, but they both run Mediawiki)03:17
jameshstub: you can create pages in wikipedia the same way, iirc -- link to a non-existant page, or just type in the URL03:18
stuboh... you are probably right. It uses a different algorithm to generate urls though (whitespace is replaced with _, which I think is non standard (and preferable))03:18
spivOh, right.  Yeah :)03:18
jameshthat and requiring special syntax to create links03:19
jamesh(note that moin 1.3 also converts spaces to underscores in page names)03:19
spivStudlyCaps is already special syntax, really.03:19
stubheh... they still get occasional sucky urls though ;) http://en.wikipedia.org/wiki/Union_County%2C_New_Jersey03:20
=== spiv just remembered the dumbest wiki page name he ever saw -- "ClickHere".
=== rbelem is now known as rbelem-zzz
=== camilotelles [n=Camilo@201.32.223.192] has joined #launchpad
=== robitaille [n=daniel@d154-5-117-228.bchsia.telus.net] has joined #launchpad
=== nlghtcrawler [n=nlghtcra@24-119-154-145.cpe.cableone.net] has joined #launchpad
nlghtcrawlerhey guys07:39
=== robitaille [n=daniel@d154-5-117-228.bchsia.telus.net] has joined #launchpad
dilysMerge to rocketfuel@canonical.com/arch-pqm--main--0: r=spiv merge in public branch and pqm status improvements (patch-26: robert.collins@canonical.com, pasc@redellipse.net)09:02
=== ddaa [n=ddaa@ordo.xlii.org] has joined #launchpad
=== jinty [n=jinty@205.134.224.215] has joined #launchpad
=== carlos [n=carlos@243.Red-83-47-24.pooles.rima-tde.net] has joined #launchpad
carlosmorning10:18
=== asmodai [n=asmodai@sakko.demon.nl] has joined #Launchpad
spivcarlos: Hey.10:23
carlosspiv, hi10:23
spivcarlos: Can you find out if that rosetta-export-queue.py on production is current, so we can track down that error?10:24
carlossure10:25
spiv(I'm very tired though, so input from me may have to wait until Monday)10:25
carlosspiv, Monday is holiday here so I will not be back until Tuesday10:26
spivHmm :/10:27
spivWell, get me as much info as possible so I can look into it myself on Monday, I guess.10:27
carlosI'm updating my "dist" directory to see the patchsets on production10:28
carlosI think I would only give you the "yes, production code has the fix" or "no, production code is lacking the fix"10:28
spivCan you give me a brief description of how I might be able to test or reproduce this?  I'm not hugely familiar with that export code.10:30
spivI'll resort to just reading the code and guessing if I have to :)10:30
carlosspiv, to reproduce exactly the same situation10:31
carlosrequest the download of the same po file from different users10:31
carlosand then execute the export script10:31
carlosthe first will work10:31
carlosand will upload that file into librarian10:31
carlosthe second request will try to get the file from librarian and will fail10:32
spivExcellent.  Thanks!10:32
carloshttp://localhost:8086/products/evolution/+series/main/+pots/evolution-2.2/es/+export10:32
carlosspiv, you can request the export from that URL10:32
=== spiv mails a reminder to himself
carlosspiv, and the cronscript you need to execute is rosetta-export-queue.py from the cronscripts directory10:33
carlosspiv, I can do it if you want10:33
spivAlready done it ;)10:34
spivThanks.10:34
carlosnp10:36
carlosstub, which is current production branch?  rocketfuel@canonical.com/launchpad--production--1.26 ?10:37
carlosspiv, if production is 1.26, it has already the fix10:48
=== _Rappy_ [n=hunt-pre@dsl-253-122.monet.no] has joined #launchpad
jordicarlos: any quick idea of how you'd remove strings from a po file that are not included in a second pofile?10:56
jordimsgcmp tells me whch, but I can't generate a file based on that.10:56
=== Mez [n=Mez@cpc2-lich4-3-0-cust115.brhm.cable.ntl.com] has joined #launchpad
carlosjordi, msgmerge?10:56
carlosjordi, take the second pofile as a .pot file10:57
carlosand merge it10:57
jordicarlos: doesn't work, it gives me a greatly fuzzied file10:57
carlosjordi, no idea then, sorry11:00
carlosjordi, perhaps you could use msgfilter11:00
carlosto remove all translations11:00
carlosso you get a .pot file11:00
carlosand you can merge that11:00
jordihmm11:03
jordilet's try that one11:03
jordiyou mean msgattrib=11:05
stubcarlos: stuart.bishop@canonical.com/launchpad--production--1.27 (it never got tagged across to Rocketfuel)11:20
carlosjordi, msgfilter lets you filter translations11:22
carlosstub, oh, ok11:22
=== jinty [n=jinty@205.134.224.215] has joined #launchpad
carlosspiv, ok, confirmed, production has the commit between the librarian upload and the next request11:27
spivcarlos: Ok, I'll dig further.  Thanks!11:50
carlosspiv, you are welcome11:51
carlosspiv, as a workaround, should I catch that exception and do a full export instead?11:52
carlosor do you think you will fix it soon?11:52
spivI'm not sure off the top of my head, I need to read the code and get a little more context.11:53
spivcarlos/jordi: http://mail.zope.org/pipermail/zope3-dev/2005-August/015197.html11:56
carlosspiv, ok11:57
carlosspiv, the workaround is easy so If you don't have a clue on Tuesday I will implement it and ask to include it with the production update11:57
spivOk :)11:58
carlosspiv, that's what I call a useful bug report....11:58
spivcarlos: I know :/11:59
=== carlos answers
spivStill, a prompt response will hopefully give a positive impression, and turn it into something useful.11:59
spivInteresting also that they chose to ask the Zope community for help rather than Launchpad.12:00
carlosanswered12:07
=== WaterSevenUb [n=WaterSev@bl5-48-21.dsl.telepac.pt] has joined #launchpad
MezI cant seem to login to launcyhpad and stay logged in12:33
KinnisonMez: the cookie is short-lived12:34
KinnisonMez: Or perhaps you're refusing cookies from us entirely12:34
MezKinnison, It's letting me log in now12:35
Mezweird12:35
MezKinnison, it was saying I was logged in, then I was trying to click on my name in the top right and it was logging me out :D12:35
Kinnisonvery odd12:35
Mezmmhmm12:36
Mezworks now12:36
KinnisonSteveA: ping12:46
dilysMerge to rocketfuel@canonical.com/launchpad--devel--0: [trivial]  Feedback and tweaks to data fix script (patch-2250: stuart.bishop@canonical.com)12:54
SteveAKinnison: 01:02
KinnisonSteveA: see your SMS inbox01:02
SteveAthx01:03
=== Mez [n=Mez@cpc2-lich4-3-0-cust115.brhm.cable.ntl.com] has joined #launchpad
=== Mez [n=Mez@cpc2-lich4-3-0-cust115.brhm.cable.ntl.com] has joined #launchpad
SteveAspiv: re ian bicking's blog, can you tell jblack where it is?01:17
SteveAthis would be a good place for jblack to chip in about the supermirror01:17
spivjblack, SteveA: http://blog.ianbicking.org/01:19
SteveAi replied to the list.  can you reply with that url?01:19
spivOk.01:19
spivHmm, he's just posted another entry.01:20
spivDone.01:22
SteveAspiv: is ian's blog widely read?01:23
spivYes, quite.01:23
spive.g. Havoc Pennington linked to him (on this topic).01:23
=== Mez [n=Mez@cpc2-lich4-3-0-cust115.brhm.cable.ntl.com] has joined #launchpad
spivHe's fairly well known, because of his work on SQLObject and WSGI.01:24
=== salgado [n=salgado@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
salgadoyo SteveA. any news about the review of my branches?01:44
SteveAsalgado: i'm 1/2 way through01:45
SteveAlooking good.  i need to check that the login stuff works properly with the rest of the infrastructure.01:45
salgadoSteveA, great. thank you. :)01:46
=== allee [n=ach@dialin-212-144-129-149.arcor-ip.net] has joined #launchpad
alleeHi02:10
alleeI'm plaing with malone. When Debian BTS has already a bug registered. I tried 'Indicate Bug Occurs in Distribution'02:28
alleebut found no way to specify debian (select always returns nothing)02:29
=== j^ [n=j@195.169.149.104] has joined #launchpad
=== carlos -> lunch
=== mpt [n=mpt@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
=== WaterSevenUb [n=WaterSev@bl5-44-226.dsl.telepac.pt] has joined #launchpad
bradballee: hi.03:03
alleebradb: hi03:03
bradballee: you're right, we don't seem to have imported details about debian into Launchpad yet.03:03
alleebradb: 'k03:03
salgadoSteveA, how do I get rid of the security proxy so I can append()/insert() items in a list?03:04
alleeanother thing I've not found yet: is there a way I can subscribe cc/watch to some pkgs (not only bugs).03:05
bradballee: not yet, but we just finished spec'ing such functionality last week and the week before at a developer summit in brazil03:05
bradballee: https://wiki.launchpad.canonical.com/PackageSubscriptions03:06
alleebradb: that's great news.  Malone has many features I missed03:06
SteveAsalgado: i think this is in the hackers faq, but in case it isn't, the best way is to use list(L) to make a mutable copy of the list03:06
alleebradb: about indicate distro:  Does it submit a but report to this BTS too?  this was not clear to me  (because there is no bug# to specify)03:08
allees/but report/bug report/03:08
KinnisonIf anyone needs anything from me, catch me in the next 10 minutes03:08
bradballee: no. malone can watch the status of bugs in external bug trackers, but not push bugs into the upstream bugtracker itself (unless, of course, upstream is using Malone. :)03:10
bradbpushing the bugs upstream would encourage people to file dups, IMHO03:10
alleeThat's okay. I added a watch already.  To make this clear I would suggest to ask for distro/package/bugID on this page.  At least I would not have worried that malone may submit a bts report03:12
alleebradb: Thx.  Malone is pretty usable and nice.  I'll recommend it to others ;)03:14
=== mpt falls out of his chair
bradballee: that's an interesting idea about the "indicate also exists in..." for a distro. we'll ponder that.03:14
mptbradb: Shall I make you up a template for the merged page?03:14
mptor would it be auto-generated?03:15
bradbmpt: don't worry about it; forms like that are easy enough html/css wise that i can do them :P03:16
mptSo, just a JPEG then? :-)03:16
mptsalgado: I got an error in make run03:18
mptpsql: FATAL:  user "mpt" does not exist03:18
mpt* Add the following to /etc/postgresql/postgresql.conf:03:18
mpt    search_path='$user,public,ts2'03:18
bradbmpt: if you want to do it, go ahead. i've got a huge amount of work to do before i get anywhere near it, so...03:19
salgadompt, can you try that again now?03:20
mptpsql:trusted.sql:13: ERROR:  permission denied for language plpythonu03:20
mptpsql:trusted.sql:24: ERROR:  must be owner of function valid_name03:20
mptsalgado: and a dozen other pairs of errors of the same type03:20
salgadompt, again? ;)03:21
mptnow we're further ...03:21
=== Kinnison [n=dsilvers@haddenham.pepperfish.net] has joined #launchpad
mpt* Creating launchpad_dev03:23
mptdropdb: database removal failed: ERROR:  database "launchpad_dev" is being accessed by other users03:23
mptsalgado: I'm not sure that problem's fatal, though03:23
mptcreatedb: database creation failed: ERROR:  database "launchpad_dev" already exists03:23
mpt* All done03:23
=== mpt tries running
stubThat should be fatal03:24
mptok, it runs, but the schema still doesn't match the code03:24
salgadompt, we're going to share the same database for some time. this is definitely not a good idea, but I don't have time to investigate how to make the db creation scripts read the configuration files03:24
mptSo I need to update this branch, then03:25
mptok, no big deal, thanks03:25
salgadostub, how much work do you think it is to get the db-creation scripts (make -C database/schema, in fact) to read the config files and create the proper databases03:25
stubsalgado: Bits would need to be rewritten in Python. It wouldn't help though as the test database names are not configured in the config file - they are well known names.03:27
stubsalgado: So it would also involve trawling for launchpad_ftest_template and launchpad_ftest (I think) and fixing those references.03:28
salgadompt, then we'll probably have some considerable annoyances soon. :-(03:28
stubsalgado: You can't run multiple postmasters? Might be easier - just give developers their own ports03:29
mptsalgado: How do you and kiko and cprov normally work together, then?03:29
bradbmpt: where did the attachment go that was pointed to from https://wiki.launchpad.canonical.com/MaloneSearchResults ?03:29
salgadompt, they work in their laptops. I'm the only one using anthem's postmaster (the one you use now)03:29
mptbradb: lpwiki, probably03:29
bradbISTR that might have been the spec where we used a long-and-ugly URL to link to the attachment, and those URLs are subject to change, AFAIK03:29
mpthmm, no03:30
mpt"Upload new attachment" means it was never uploaded03:30
salgadostub, that seems to be the best thing to do. will try it03:30
bradbwe didn't upload it03:30
mptusing a long and ugly URL would give you a broken-image icon03:30
bradbwe linked to it with some long-and-ugly URL pointing somewhere else, instead of uploading it into the wiki for that specific spec and using the normal "attach:filename" syntax, IIRC03:30
bradbor not. i dunno. but it's not there now. where do i find it?03:31
mptNo, because if we'd done that, there'd be a broken-image icon rather than "Upload new attachment"03:33
mptWell, there's still the version in Montreal200503:33
bradbn/m, i found the file03:33
mpthttps://wiki.launchpad.canonical.com/Montreal2005?action=AttachFile&do=get&target=2-search-results-list.jpg03:34
mptbash: refuel: command not found03:35
bradbright, i *knew* there was a long-and-ugly-subject-to-changing URL somewhere :)03:35
mptbut I know what you mean, I did draw a newer version during the sprint03:35
bradbargh, the file that i thought it would be (per the name) isn't what i expected it to be03:36
salgadostub, I guess it's a good idea to use a different $PGDATA, or there's no need for it?03:37
bradbmpt: where can i get the photo of the newest design?03:37
mptbradb: I don't know, sorry03:38
stubsalgado: You have to - you won't be able to start multiple postmasters using the same one03:38
mptbradb: I'll draw it again if you like03:38
salgadostub, how do I setup a new PGDATA? what files are initially needed there?03:38
stubinitdb 03:40
stubinitdb -E UNICODE --no-locale --pgdata=/var/whatever/foo03:41
stubsalgado: You can probably share the launchpad.conf etc. by creating symlinks to /etc/postgresql (see the default PGDATA for an example - default Debian setup does it this way)03:44
salgadoyes, it looks like initdb did this for me03:45
mptsalgado: dude, you don't have refuel installed :-)03:46
salgadostub, "postmaster -d 5 -D /var/lib/postgres/data/mpt -p 5431" seems to run fine, but is not listening on port 5431. am I missing something?03:50
stubAdd a -i in there too03:51
salgadoah, cool. ta03:53
=== niemeyer [n=niemeyer@brutus.conectiva.com.br] has joined #launchpad
bradbspiv: ping04:00
niemeyerGreetings!04:00
mptbradb: https://wiki.launchpad.canonical.com/SimplifyingMalone04:02
bradbdaringly sane04:03
bradbone of my favourite bits of that would be to de-amateurize the way comments are displayed04:04
bradbSteveA: I want to declare a constant for the list of unresolved bugtask statuses. It would look like: BUGTASK_UNRESOLVED_STATUSES = [BugTaskStatus.NEW, BugTaskStatus.ACCEPTED] . Do we have a module for that?04:09
bradbif not, should i just create a canonical.launchpad.constants?04:10
=== WaterSevenUb [n=WaterSev@bl5-134-192.dsl.telepac.pt] has joined #launchpad
mptddaa: Had time to look at the samba/ubuntu-doc bug yet?04:13
mptgreat, baz crashes on normal merging *and* star-merging04:25
bradbspiv: ping04:39
mptddaa/jblack: baz is crashing on every merge, and it's not because it's running out of memory ... help?05:02
=== Seveas [n=seveas@seveas.demon.nl] has joined #launchpad
bradbmpt: Do we have a CSS class to underline links in portlets? Should "requires _login_" be underlined in a portlet?05:05
mptbradb: No, portlets are explicitly excluded from the "links should look like links" part of launchpad.css05:05
bradbeven for a "requires _login_" link?05:05
bradb(like the "Assigned to me (requires _login_)" option)05:06
mptbradb: Yes, portlets are explicitly excluded from the "links should look like links" part of launchpad.css, and CSS is not yet powerful enough to examine the text of nodes05:06
mptbradb: "(requires login)" looks bad anyway ... Can you instead link to a generic URL that asks you to log in, then redirects you to your own list of assigned bugs?05:07
mptThen you'd only need to follow one link, not two 05:08
bradbhm, good point, i'll file a bug on this05:08
mptWe appear to already have this, bradb, https://launchpad.net/malone/assigned05:09
=== jeprubio [n=jeprubio@80-102-225-170.bcn2.dialup.uni2.es] has joined #launchpad
bradbquite trolling :)05:10
bradbquit, even05:10
jeprubiohello05:10
mptexcept that after logging in, instead of redirecting me to https://launchpad.net/people/mpt/+assignedbugs or whatever, it says "Sorry, you don't have permission to access this page"05:10
mptI'm not trolling, I'd forgotten about that page until after I suggested it05:10
=== mpt thought that bug was marked fixed
bradbThat page is not equivalent to the "Assigned to me" bug list link05:11
bradb(not to mention that /malone/assigned just sucks in overall)05:11
bradbs/in //05:12
mptWhat is it supposed to do?05:13
mptIt looks like it's supposed to be equivalent to "Assigned to me", because when I went over its template I ended up reporting https://launchpad.net/malone/bugs/136705:13
jeprubioI have found a bug detecting a Sis sound card and a web who fixes it, I think it could help for other people who have the same problem, I'm going to include the link in the wiki but I'm sure it should be better that ubuntu fixes it05:13
mptjeprubio: At the moment, you should report a bug like that in http://bugzilla.ubuntu.com/05:14
SteveAbradb: constants are part of the interfaces05:14
bradb/malone/assigned is context-free, lives at the wrong URL (and has thus been reimplemented in FOAF), has a totally inconsistent look and feel to the other bug listings (and to all other lp pages), etc.05:14
SteveAbradb: our standard for constants is that they live inside a class's namespace, and are importable from interfaces.05:14
jeprubiompt ok, thanks05:15
SteveAif this is more of a one-off then perhaps just declare it in interfaces/bugtask.py05:15
SteveAand make it available from interfaces.05:15
SteveAthere is no need for a special new place for these things.05:16
bradbi.e. IBugTask.UNRESOLVED_STATUSES?05:16
bradber, no, i guess you meant module-level05:16
SteveAyes05:18
SteveAbut, it needs to be UNRESOLVED_BUGTASK_STATUSES05:18
SteveAas it will be importable from interface05:18
mptbradb: Well, a context-free URL to begin with is what you want when someone's not logged in yet, and it already has part of the right behavior (making you log in first) ... I don't know how much sense it would make to modify it to redirect you to your own +assignedbugs as opposed to implementing that from scratch, though.05:18
bradbmpt: why do you want a context-free URL when someone's not logged in but they're already in a context?05:19
SteveAi don't like the idea of pages that themselves change totally according to who is logged in.  i prefer the idea of pages that redirect to a page that is related to a person05:19
SteveAso, for example,05:20
SteveAi go do +my-own-assigned-bugs-k-thx-bye05:20
SteveAand it redirects to /person/stevea/+assigned05:20
SteveAthis is a good model, because i can then give /person/stevea/+assigned to other people05:20
SteveAand they can see what i see05:20
mptbradb: Oh, *that* kind of context-free05:20
bradbmpt: the fact that /malone/assigned makes you login is purely accidental, btw. :)05:20
mptbradb: I thought you meant "all bugs assigned to me"05:20
=== robitaille [n=daniel@d154-5-117-228.bchsia.telus.net] has joined #launchpad
mptSteveA: Sure, I absolutely agree, we just need an URL to put in the portlet that will (a) make you log in and (b) direct you to your personal URL, such that we don't need to cruft up the portlet with "this isn't a link because you need to __log in__ first"05:22
=== jamesh [n=james@203-59-251-126.dyn.iinet.net.au] has joined #launchpad
bradbmpt: Just to be clear then, when looking at, say, /products/malone/+bugs and not logged in, should there be a "Show Assigned to Me" link which, when clicked, will prompt you to login?05:35
bradbSteveA: I'm about to finish a response to a review on a critical Malone landing (sp bug listing), but spiv, the reviewer, appears to be sleeping. Is it possible that you might be able to pick up on this and look at my response based on his comments and tell me if I can merge it?05:37
mptbradb: "Assigned to me", yes05:38
bradbok, thanks05:38
SteveAbradb: i have a salgado review to finish, and then some other stuff planned.  i may be able to look at it later, but i think spiv should be the one to finish the process he and you started.05:39
SteveAyou're not going to get the changes into the standard rollout anyway05:39
mptbradb: All those buglists should have their own <h4>Bug lists</h4> or similar, so that they're separated from the actions and therefore don't all need to start with "Show"05:39
bradbI put them in their own portlet, but the sab moved them into the actions portlet in .br, IIRC05:40
SteveAum05:41
SteveAthe actions portlet should not be called "actions"05:41
SteveAand so it need not have everything be verbalized05:42
mptbradb: well, if you're repeating the same word(s) more than ~2 times it's a good sign you're using the wrong presentation05:42
bradbmpt: preaching, choir, etc.05:42
mptWell, I could remove all the <h4>actions:</h4> right now if someone could help me get baz working ;-)05:44
mptjblack?05:46
jblackI'm here, but in a conversation. what's up?05:46
mptjblack: In two different branches, I get "baz: uncaught exception: -1:(unable to fork for patch)" when merging rocketfuel into them05:47
mptfor both normal merge and star-merge05:47
mptand it's not lack of memory, consumption reaches only about 60% before the failure05:47
jblackTry the --star-merge argument05:49
mptYes, that's what I mean by "both normal merge and star-merge"05:50
jblackOh, unable to fork for patch.05:52
jblackFor some reason, when baz is calling exec on patch, its not finding it, can't get a process for it, there's not enough memory for patch, etc.05:52
jblacktypically, because you're actually missing patch. Its seen most often on bsd systems that haven't install gdiff/gpatch. Maybe you're missing diff3.05:53
salgadojblack, no way, he uses the same system I'm using05:53
mptmpt@deadsea:~/ubuntu/launchpad$ which patch05:53
mpt/usr/bin/patch05:53
=== Keybuk [n=scott@syndicate.netsplit.com] has joined #launchpad
=== WaterSevenUb [n=WaterSev@bl5-133-44.dsl.telepac.pt] has joined #launchpad
mptjblack: It gets as far as applying 62 revisions out of 80 before it gives up.05:54
mpt(assuming each "." is one revision)05:55
jblacksalgado: Taht's what that error is. baz either can't fork, or it can't call patch. 05:55
jblackWhy don't you try stracing it, and seeing what the last 20 calls or so were. 05:55
jblack(well, also, patch could be dying with an error somehow)05:55
mptsalgado, on anthem it doesn't crash06:01
salgadompt, then your problem is lack of memory. that's why I have 1G06:01
jblackYeah. Memory could do it. 06:02
mptfnord.06:02
mptok, thanks for your time jblack06:02
bradb"your problem", i.e. baz's problem turned your problem :)06:02
mptHas baz reduced memory consumption in the past month?06:02
mptThis version is from July 9th06:02
bradbI haven't noticed any improvement06:03
jblackbradb-bbl: Yeah, its baz's problem that it eats all that memory.06:20
jblackBut a problem in any tool is a problem to the designer as well, as his tool can't do the job he needs.06:21
jblackmpt: Yeah, its had various memory fixes over the months. 06:21
dilysMerge to rocketfuel@canonical.com/launchpad--devel--0: [r=spiv]  Change all DOAP to Registry, including urls (patch-2251: morgan.collett@canonical.com)06:27
dilysMerge to rocketfuel@canonical.com/launchpad--devel--0: [r=spiv]  MaloneSourcePackageBugListing implementation (patch-2252: brad.bollenbach@canonical.com)06:58
=== mohameth [n=mohameth@p549D665F.dip.t-dialin.net] has joined #launchpad
mohamethhelo :)07:58
mohamethsomebody who`s working on Gourmet Recipe Manager here?07:59
carlosmohameth, I'm not working on it, but anyway, what do you need?08:03
=== Seveas [n=seveas@seveas.demon.nl] has joined #launchpad
=== asmodai_ [n=asmodai@sakko.demon.nl] has joined #Launchpad
=== asmodai_ is now known as asmodai
mohamethcarlos, just a suggestion08:06
mohamethbut i will tell it by mail08:07
carlosok08:07
=== asmodai_ [n=asmodai@sakko.demon.nl] has joined #Launchpad
=== asmodai_ is now known as asmodai
=== Znarl [i=karl@bb-82-108-14-161.ukonline.co.uk] has joined #launchpad
mptbradb: http://wiki.launchpad.canonical.com/MaloneSearchResults08:48
bradbI'm just preparing to land distro release targeting now. I'll take a closer look in about an hour.08:50
ddaampt: did not have time to look at it (I had three days off)09:02
ddaaIf the amount of nagging I get about it is any measure, I guess that has to be pretty important. I'll start learning about the problem early next week (before finishing my current task).09:03
ddaaThat means tuesday, since monday is a national holiday.09:03
mptthanks ddaa09:04
ddaahopefully the reviews will start to flow again, the cscvs backlog is already uncomfortably large.09:05
ddaaand any non-trivial work I have to do involves very large chunks since it's overall a pep8 calamity.09:06
=== mpt [n=mpt@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
=== ddaa hears the Voyager theme downstairs
bradbmpt: for DR targeting, if a release is already targeted, so it's checkbox is disabled, should any part of "[/]  sarge mozilla-firefox (New, Unassigned)" be linkified and if so, what and pointing where?09:23
mptbradb: I haven't seen the page we're talking about, but I guess it would probably be linked to all the open bugs targeted to that DR package09:29
=== madduck [n=madduck@debian/developer/madduck] has joined #launchpad
madduckhi dudes.09:31
madduckjust tried to attach a file to https://launchpad.net/malone/bugs/1755/09:31
madduckand it gave me a system error09:31
bradbmpt: http://69.70.209.33:8086/distros/debian/sarge/+bugs/1/+target09:31
madducki should be reporting a new bug but i am pressed for time.09:31
madduckso please, if someone could check it out?09:31
bradbBjornT: ^^09:31
mptmadduck: If telling someone a bug on IRC is faster than reporting it in Malone, there's something wrong with Malone :-)09:32
madduckmpt: yeah dude09:32
mpt(er, something other than the bug you're experiencing in the first place)09:33
bradbmpt: It's faster to report it in Malone, but human nature jumps to IRC first09:33
bradbmpt: which is why i'm campaigning for a bot that speaks Malonese09:33
bradbof course, reporting bugs to a bot increases the likelihood of dups, without some extra special feeping creaturism09:34
mptmadduck: Is it specific to the file? Can you attach a different file ok?09:34
madduckmpt: lemme check09:35
mptbradb: Are you a pagetest expert?09:35
mpt:-)09:35
bradbmpt: sort of09:35
madduckmpt: yeah, fstab attached fine.09:35
mptmadduck: Okay, you'll need to make public somehow the file that wouldn't attach, so people can poke it with a stick and work out why Malone doesn't like it09:36
madduckmpt: it's wget of the link i posted in the comment.09:36
bradbmpt: right, i'm going to go with what i've got re: DR targeting and you can tweak it later if you want09:37
madduckmpt: i also checked "patch" on the failing file, not on fstab09:38
mptooo09:39
mptworksforme without marking it as a patch09:40
mptBjornT: Is there a test for attaching a patch?09:40
bradbBjornT said there was something important missing from his attachments implementation. maybe this is what he was referring to.09:40
madduckmpt: i thought so too.09:40
madduckbecause the patch is actually malformed.09:40
madduckrafb.net/paste malforms it.09:40
mptbradb: Yes, as I said, link to the list of open bugs for that package09:41
mptbradb: And soon I'll have ready a nice package icon09:41
bradbok09:43
mptbradb: Ok, so in a pagetest instead of doing the print http(...) ... thing, I do output = http(...)09:45
mptand then things like09:46
mpt>>> 'Your subscription to this bounty has been updated' in output09:46
mptTrue09:46
mptBut they all fail because "TypeError: iterable argument required"09:46
spivmpt: do 'foo' in output.getBody()09:46
mptI'm guessing I shouldn't be using the word "in", but what should I be using instead?09:46
mptah, thanks spiv09:47
spivmpt: the result of http(...) isn't actually a string, it's a richer object.09:47
spivIt happens to behave like one when you print it, for convenience.09:47
mpt>>> do 'Your subscription to this bounty has been updated' in output.getBody()09:47
mptTrue09:47
spiv"do"?09:48
mptthat's what you said ...09:48
spiv(There are some examples of this in lib/canonical/launchpad/pagetests/standalone/xx-notfound-head.txt)09:48
mptoh, ok09:48
spivmpt: English :P09:48
spivPlus, I shouldn't be awake ;)09:49
mptxx-notfound-head.txt doesn't use "in" at all09:49
spivOh, hmm.  It does use getBody though.09:51
spivxx-login-and-join-links.txt and xx-productseries-reset-to-test.txt do.09:52
mptgreat, thanks09:53
BjornTmadduck: by looking at the error logs it looks like you tried to upload an empty file. maybe your web browser couldn't read the contents of the file? anyway, there should be a nice error message there, I'll look into it.09:53
=== mpt wonders if he gets karma from uploading madduck's patch
BjornTmpt: there are tests for attaching patches/non-patches, but not for attaching an empty file09:54
madduckBjornT: that's weird, but thanks for clearing it up.09:54
madduckmpt: you get karma. :)09:54
madduckand wow. there is no such file.09:54
madducki am confused now.09:54
madducki wgot it, then x-pasted the name to the browser09:55
mptOld-fashioned Web browsers that let you enter non-existent paths, ptui09:55
madduckfirefox snapshot :)09:55
madducksorry to have stirred things up09:55
BjornTmadduck: it's ok, it's still a bug :) it shouldn't produce a system error09:56
madducktrue09:56
madduckyou guys are using plone2.1, right?09:56
spivbradb: FWIW, I'm happy with you merging that branch now (which I see you already did)10:05
spivbradb: Don't forget to remove it from PendingReviews though!10:05
bradbspiv: right, I'll remove it now, thanks10:06
BjornTbradb: do you have any idea of why the bug notifications get weirdly wrapped?10:11
bradbspiv: sorry if I was a bit hasty about merging, but I felt like I addressed your points fairly directly and, in the two cases where i wasn't 100% sure, i asked the relevant people for their advice (i.e. SteveA re: constants and mpt re: the "Assigned to me" link)10:12
bradbBjornT: probably. I just want to finish up this DR release targeting landing before doing anything else. after that, i'll take a look.10:13
spivbradb: No, it's fine, you addressed the points.  I'm perfectly happy :)10:15
bradbcool, thanks10:15
BjornTbradb: ok, cool. if you feel you don't have time, i could do it, though.10:16
bradbBjornT: i'll have a 30 sec look to see if i can offer a quick solution10:19
bradbsalgado: you've got mail!10:20
bradbBjornT: hm, DocWrapper doesn't seem to deal with newlines in paragraphs well10:28
bradbe.g. even:10:28
bradbfoo10:28
bradbbar10:28
bradbdoesn't wrap to one line10:28
BjornTwell, i'd expect that to be two lines. the problem seems to be, if you give it something that's already properly wrapped, it will wrap it some more...10:31
bradbWhat do you mean by "properly wrapped?"10:32
bradbPresumably you mean a line already fewer than 80 chars10:33
BjornTexactly. i just submitted a bug via email, with all lines less than 72 chars. in the notification, some lines had been wrapped anyway.10:34
bradbRight, do you want to look into this then?10:36
bradbsalgado: around?10:36
BjornTbradb: sure10:37
bradbthanks10:37
bradbBjornT: You might want to start by writing some science fiction in doc/textformatting.txt, where this module is tested10:37
BjornTbradb: yeah, i was planning to do that.10:39
mptAnyone: Where can I find docs or an example on how to use the new "logged in as person Foo" syntax for pagetests?10:39
mptIt's not in the LaunchpadHackingFAQ or LaunchpadPageTests10:40
spivmpt: You mean the still-unmerged feature from  steve.alexander@c.c/launchpad--unittest-authentication--0?10:41
mptah, crap10:41
mptprobably :-)10:41
spivhttps://chinstrap.ubuntu.com/~jamesh/pending-reviews/steve.alexander@canonical.com/launchpad--unittest-authentication--0/filtered-diff10:41
spivI'm not sure why it isn't merged; it's been merge-conditional since 2005-06-01, apparently.10:42
mptwell, failing that, how do I turn a known sampledata name and password into the magic "Authorization: Basic Zm9vLmJhckBjYW5vbmljYWwuY29tOnRlc3Q="-style string?10:43
bradbmpt: why do you want to do that instead of using login()?10:44
mptbradb: What's login()?10:45
bradbcanonical.launchpad.ftests.login, IIRC10:46
bradblogin(email_address) Just Works10:46
BjornTmpt: try using: Authorization: Basic test@canonical.com:test10:46
bradbmpt: btw, is salgado around there?10:47
mptUm, I think so10:47
mptone moment10:47
mptbradb, he's gone out, but reportedly will return soon10:48
bradbok, thanks10:48
bradbn/m what i said about login() btw; i was thinking doctests10:49
bradbactually, i was /really/ thinking of getting DR targeting landed, but anyway...10:49
mptsorry for the interruption :-)10:49
=== mpt has been trying to get this decruftification landed all day
bradbmpt: is anyone working on google-style list results? if not, i'll start that now (will unfortunately have to put of BjornT's review of the bugtask assignee widget for the moment, i guess)10:50
mptthanks BjornT10:50
mptbradb: If you mean MaloneSearchResults ;-), I finished the Design section of that spec a couple of hours ago10:51
bradbyeah, i saw that10:52
=== mpt doesn't understand this obsession with Google, honestly...
mptI could do the HTML part of it easily enough10:52
bradbRationale -- "The table format is hard to scan..."?10:52
mptyes, because stuff is all spaced out10:53
bradbThe table format is *easier* to scan than a google-style listing could ever hope to be. At least things are lined up. :)10:53
mptthe gap between columns is much wider than the normal gap between words.10:53
bradbThe rationale is that there's too much info to show about a task without exploding off the end of the pages.10:53
BjornTmpt: sorry, what i said won't work, our copy of zope3 is too old. you can look in some other pagetest, though, and copy the header from there.10:54
mptBjornT: I don't know who the authorization is for unless it's explicitly mentioned in the pagetest10:55
mptbut thanks10:55
BjornTspiv: i think the reason SteveA's branch isn't merged, is that the functionality already exists upstream10:55
bradbmpt: so, noone's working on implementing MSR right now, right?10:57
mptbradb: Not as far as I know10:57
spivBjornT: Hmm.  How hard is it to backport that feature from upstream?10:57
bradbok, i'm bazzing my way to an MSR branch right now...10:57
=== yota [n=seb@home.nmeos.net] has joined #launchpad
yotahi10:58
mptI need (a) the magic string for someone who's not Foo Bar, and (b) the name of that person10:58
yotaany rosetta admin here ?10:58
mptyota: Sorry, they're all sick/asleep at the moment ... Did you have a problem or a suggestion?10:59
yotaproblem, I cannot upload a translation file11:00
mptWhat happens when you try?11:00
yotaa message appears on screen and that all11:01
mptWhat does the message say?11:01
yotaoups sorry11:02
yotathx you for you upload ...11:02
mpt:-)11:02
mptYou were expecting to see something else instead, perhaps?11:02
mpt(Maybe we should be showing something more obvious)11:02
yotasure, an update :)11:03
bradbERROR: Thank you for your upload.11:03
BjornTspiv: it shouldn't be that hard. i think it's only one extra function, and one modification to another function. although isn't the plan to upgrade our version of zope3 soon?11:03
mptbradb, shush11:03
spivBjornT: I'm not sure, but this has been an outstanding problem for a long time, and it's frustrating that we both have a branch to solve it, and later upstream solves it, but we aren't using either.11:04
spivWho's responsible for updating our zope3?11:04
BjornTspiv: yeah i know. and i actually think that upstream had this before we even tried to fix it...11:05
bradbIsn't the intent for us to wait to upgrade to 3.1 when it's officially released?11:08
mptyota: Does the message say something about the import process taking a while?11:10
BjornTwell, it'd be best to upgrade before that, to ensure that the functionality we use still is supported11:11
mptyota: Ah, you get an orange box, which looks like an error message11:11
yotampt: message say that translation content will appear  in few minutes. After few uploads, no update11:12
SteveAhi11:12
mptbradb: Bad news11:13
bradbmpt: kiko's already working on it?11:13
bradbor salgado's not coming back?11:14
mptbradb: salgado's back from the doctor, but his eyes are no-worky so he can't read anything you say11:14
bradbouch! :/11:14
mptand they'll be like that for the next day or two11:14
mptSo he says to e-mail him11:14
mpt(he managed to find Ubuntu's "Log Out" menu item)11:14
bradbpoor guy11:15
spivSteveA: Btw, the other place I regularly see Ian Bicking's blog entries is linked from Daily Python-URL (http://www.pythonware.com/daily/)11:19
SteveAmpt: if salgado is there, can you tell him that i reviewed his smallfixes--4 branch.  it's good.  a couple of small changes, and he'll be good to merge it.11:29
mptSteveA: He can't read code and he's just gone home for the weekend11:33
SteveAokay11:33
SteveAjblack: ping11:33
jblackpong11:39
jblacksteva: pong11:40
mptAHA!!!11:42
=== mpt figures out why his pagetest is failing
SteveAjblack: i guess i didn't understand what i was asking for when we talked about the bzr demo tarball.11:42
SteveAi've had to bzr init and bzr add in the various branches to get them to work11:43
jblackOh, no, that's my fault. 11:43
SteveAand of course, i'd forgotten there's no bzr switch yet11:43
jblackI forgot to init them.11:43
SteveAso, a multi-branch demo doesn't make much sense11:43
jblackYeah. bzr isn't usable in any real sense.11:43
SteveAi think your instructions are a good start11:43
SteveAin that they show how easy it is to branch, and commit and diff11:44
jblackIts about good enough for local stuff and to play with some of the early merging stuff, but thats about it. :( 11:44
jblackI'll put up a better 8/12 with the tools initted. sorry for the botch11:44
SteveAno woorries.  please go through the instructions from a fresh download to check they make sense and work :-)11:53
jblackYeah. I did with 8-11. The changes to 8-12 were so minor, that I didn't check. 11:56
jblackI'm walking through them now with 8-12-211:57

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