[04:15] -lugito:#lubuntu-devel- [lubuntu-manual] ianorlin opened pull request #39: Add lxrandr (master...master) https://git.io/vp0ay
[04:16] -lugito:#lubuntu-devel- [lubuntu-manual] ianorlin pushed 1 new commit to master: https://git.io/vp0aH
[04:16] -lugito:#lubuntu-devel- lubuntu-manual/master dbb88c4 Lyn Perrine: Add lxrandr (#39)...
[07:59] <lubot2> purplemafia was added by: purplemafia
 Fwd from Linuxgram 🐧: https://youtu.be/h284JgDGC7Y
 positive review 😉
 Ooh. Thanks.
[15:22] <wxl> am i correct that if a .desktop file is in $XDG_DATA_DIRS then lxsession should add it to the database? (assuming a properly formed file)
 @wxl, Go to #lubuntu. XD
[16:00] <wxl> yeah this is actually not a support question
 Fwd from tsimonq2: COSMIC CANIMAL
 Fwd from tsimonq2: That's the codename.
 I'm wrong, actually.
[17:14] <lynorian> I think misreading that as cannibal was quite bad
 Hah.
[17:25] -lugito:#lubuntu-devel- [lubuntu-seeds] lubuntu-lugito created cosmic (+1 new commit): https://git.io/vpEoS
[17:25] -lugito:#lubuntu-devel- lubuntu-seeds/cosmic 99390b0 Adam Conrad: platform.bionic -> platform.cosmic
[17:25] <wxl> cosmic canimal? are you freaking kidding me?
[17:25] <wxl> wth is a canimal????
[17:26] <acheronuk> wxl: c'animal - a placeholder as mark hasn't decided on the animal yet
[17:27] <wxl> cosmic is cool though
[17:27] <wxl> we can make a nice doom metal theme song for it
[17:27] <wxl> https://en.wikipedia.org/wiki/Canimals
[17:27] <acheronuk> both me and tsimonq2 had the same wtf :D
[17:28] <acheronuk> cosmic is nice and short for changelogs as well
[17:33] <genii> !18.10
[17:33] <genii> wxl: Is this something official?
[17:33] <wxl> it's in launchpad, genii
[17:33] <wxl> https://launchpad.net/ubuntu/cosmic
[17:34] <genii> Bleh :-/
[17:34] <genii> s/canimal/capybara
[17:35] <wxl> well as acheronuk said, it's believed that canimal is actually "c-animal" meaning that it's going to be some animal that starts with a c, but it's not announced yet
[17:36]  * genii keeps an eye on sabdfl's blog
[17:48] <acheronuk> Adam (infinity said so)
[18:29] <wxl> @vikingredwolf is this one for you? https://bugs.launchpad.net/ubuntu/+source/lubuntu-default-settings/+bug/1768293
 @wxl, Or you. That's a problem in the config file.
[18:31] <wxl> ok :)
[18:33] <wxl> i thought we fixed the xsession in lxqt @tsimonq2 ??? bug 17682890
[18:33] <wxl> ugh https://bugs.launchpad.net/lubuntu-next/+bug/1768290
 @wxl, The installer is a Ubiquity thing. WON'T FIX.
 @wxl, Errrrr wat
[18:34] <wxl> @tsimonq2: really? @VikingRedwolf seems to suggest otherwise
 Yeah.
 It's hard-coded.
 I read it myself. :P
 Mhm
[18:37] <wxl> @tsimonq2: couldn't we make our own thing? :/
[18:37] <wxl> i mean like kubuntu has their own little thing in ubiquity. we could just copy it and replace with lubuntu, no?
 It can't be hardcoded, many distros use that installer
[18:42] <wxl> @VikingRedwolf do you know where to tweak that?
 But it must be some obscure and obfuscated file
 Let me check
 It's not a thing we can do, wxl.
 I've tried.
 Really? 😢
 It's so much of a clusterheck.
 @tsimonq2, ("clusterheck" heh)
 Then we can't use it. We share repo with kubuntu
 This isn't news, y'all. :P
[18:44] <wxl> to be absolutely sure, you know we're talking about the window name?
[18:44] <wxl> s/name/title/
 Yes.
 I can't believe that we can't customise a single text chain
[18:44] <wxl> well, again, why can't we just duplicate the whole kubuntu bit in ubiquity and re-do it for us?
 That's a big design fault, considering its and independent OpenSource project
 Kubuntu's fault
[18:45] <wxl> it's not kubuntu's fault at all
[18:45] <wxl> anyways what about that lxsession thing, @tsimonq2 ?
 Why other distros can customise it and we can't?
 The Qt frontend depends on KDE libs.
 We can't just copy it.
 Why is it so close in Ubuntus repos?
 So we're stuck with ubiquity? We can run a gtk app though.
 @VikingRedwolf, We're using Calamares!
 I told y'all
 This
 Isnt
 News
 :)
[18:48] <wxl> enough of this. what about the lxsession bug?
 What bug? :)
[18:48] <wxl> rtfsb
[18:48] <wxl> :/
 I don't see an lxsession bug.
 :P
[18:51] <wxl> https://bugs.launchpad.net/lubuntu-next/+bug/1768290
 Oh
 Pfft
 I'll fix that when Lubuntu-default-settings gets fixed up.
[18:52] <wxl> so you can triage that then
 Yep.
 guys, there is a weird behavior in LXTerminal in Lubuntu 18.04
 weird in the sense that when you pick a background color is not rendered correctly
 it's like, lighter
 and there is no teardrop selector for custom colors anymore
 (side effect of the forced GTK 3 port?)
[19:12] <wxl> was it lxterminal that wasa forced to go gtk?
 ¯\_(ツ)_/¯
[19:13] <wxl> did you have this problem before?
 not really
[19:13] <wxl> does that mean no?
 i did not have that problem before
 @wxl, Yes, it was lxterminal.
 as far as i can see, it seems limited to the background color of LXTerminal
 and by the way there is no opacity cursor anymore
[19:16] <wxl> looks like we may need to look into this more, btu chances are we wont' be able to fix it if it's the gtk3 issue
[19:16] <wxl> please file bugs
[19:18] <wxl> i admit i'm having trouble replicating this
[19:20] <wxl> is it just one color?
 all but black
[19:20] <wxl> red seems fine here
[19:21] <wxl> send a screenshot?
 (Photo, 1280x719) https://i.imgur.com/ya5bcCh.jpg
[19:24] <wxl> oh you were saying black is the one that fails?
 no black is ok
 grey are somewhat clearer
 compared to the picker
[19:25] <wxl> what is the color code on that grey cuz again i'm having trouble reproducing
 #555753
 I will try on my other machines and screens
[19:27] <wxl> yeah looks good here
 anyway, there are also no eyedropper color selection and no opacity cursor for the background
[19:27] <wxl> https://share.riseup.net/#m4_NKjyFI33egXBfRjyKxw
[19:28] <wxl> don't mind my transparency XD
 seems definitely a problem of my screen then
 I am actually using a lousy machine now
[19:28] <wxl> you should be able to confirm thta if you try other screens/computers
 I definitely will
[19:31] <wxl> sorry to tell you the problem is in the hardware :( :)
 yeah, I'm observing it for other colors too but the best thing to do is test on other screens
 anyway it's life :)
[19:33] <wxl> that said there still may be a software solution but it's probably driver based
 you know, it might also be an angle of reading issue
[19:34] <wxl> not if it shows up in a screenshot
 like I said, this is a crappy machine, 10 years old
 well, whatever :)
[19:36] <wxl> XD
 thanks for the help!
[19:38] <wxl> np. let us know if anything else pops up
 Yeah, the eyedropper color selection and the opacity cursor for the background are missing
[20:06] <wxl> that i can confirm. but that may be the way it is
 losing features while upgrading... it's a bit backwards 😭
[20:08] <wxl> understood
[20:09] <wxl> unfortunately upstream lxde is relatively poorly supported
[20:09] <wxl> and this is an example of a major problem
[20:09] <wxl> the whole thing is gtk2 only
 Ok
[20:09] <wxl> the only reason lxterminal is gtk3 is because of debian policy about terminals
[20:09] <wxl> basically it was that or get removed from the archive
[20:10] <wxl> which would have meant no lxterminal at all XD
 This means that switching to LXQT might be a workaround, right?
[20:10] <wxl> yes
[20:11] <wxl> there should be an actual released lubuntu lxqt image this cycle
 (Document) http://vps.tsimonq2.net//file_3980.mp4
[20:11] <wxl> hm that's a 404?
 Pardon?
 Oh you are using a relay so maybe you can't see gifs?
 Sorry
 I should fix that. :P
 Or, wxl does have SSH access...
[20:20] <wxl> no thank you
[20:20] <wxl> you're in charge of supporting telegram
 Fiiiiiiine
 Lol
 https://twitter.com/tsimonq2/status/991440777200001024
[22:22] -lugito:#lubuntu-devel- [lubuntu-manual] ianorlin opened pull request #40: fix warnings for bad index and duplicate file.  (master...master) https://git.io/vpukM
[22:23] -lugito:#lubuntu-devel- [lubuntu-manual] ianorlin closed pull request #40: fix warnings for bad index and duplicate file.  (master...master) https://git.io/vpukM
[22:27] <wxl> is there a better screensaver for lxqt?
[22:28] <tsimonq2> wxl: Ask in #lubuntu. XD
[22:28] <wxl> not a support question
[22:29] <wxl> also do you know of calamres being crashy? install issues?
[22:29] <tsimonq2> Nope.
[22:30] <tsimonq2> Although my testing has been somewhat limited.
[22:30] <tsimonq2> So I'm curious to see if there's any issues that are reproducible.
[22:31] <tsimonq2> wxl: Aaaaaaaaaaaactually...
[22:32] <tsimonq2> Fritz did something weird.
[22:32] <tsimonq2> He tried a Ubiquity install then when that crashed did a Calamares install... (I think...)
[22:32] <wxl> meaning the selection of it in the menu?
[22:32] <tsimonq2> Sooo the target was still mounted
[22:32] <tsimonq2> Causing literally everything to lolwat
[22:32] <tsimonq2> Just a hunch.
[22:32] <tsimonq2> But it should be investigated.


[01:19] <rubio> hello
[01:20] <rubio> anyone here???
[12:41] <phillw> rafaellaguna: where are you hiding ?


[00:35] <ubotu> New bug: #213121 in blueprint "Launchpad.net fakes from address in bug updates " [Undecided,New] https://launchpad.net/bugs/213121
[00:55] <daveux> Hi, I've created two projects, do and do-plugins, and launchpad will not let me make do-pluings a subproject of do. What gives?
[00:56] <mwhudson_> projects can only be members of project groups
[00:56] <daveux> So I need to make a project group?
[00:57] <daveux> Where can I do that?
[00:58] <mwhudson_> you have to get an admin to do it, ask a question at https://answers.edge.launchpad.net/launchpad
[00:58] <daveux> Ok, thanks.
[00:58] <daveux> jcastro, can you hook me up?
[02:31] <xtknight> i can't figure out what i'm supposed to put in my .dput.cf.  i can't find my exact id.  URLs show xt-knight but it shows "Home > xtknight".  which one is it?
[03:20] <LaserJock> do PPAs still fail on the first upload?
[03:21] <LaserJock> xtknight: it'd be the on in the urls
[03:22] <xtknight> thx
[03:22] <xtknight> ppas fail on first upload?  i dont know i uploaded a pkg for the first time today and it worked.  but my first three got rejected
[03:22] <xtknight> rejected because of my mistakes
[03:24] <LaserJock> I see
[03:25] <Hobbsee> LaserJock: yeah.  the bug got a milestone, now
[03:25] <LaserJock> Hobbsee: is there a way to get give-backs in PPAs?
[03:25] <LaserJock> :-)
[03:26] <RAOF> LaserJock: Yes; you press the "retry build" button :)
[03:26] <Hobbsee> what RAOF sais
[03:26] <Hobbsee> -d
[03:26] <LaserJock> RAOF: if it's not my PPA? :-)
[03:27] <RAOF> LaserJock: Well... ask nicely for team access? :)
[03:27] <LaserJock> it's not a team PPA
[03:27] <LaserJock> just some guys
[03:28] <LaserJock> ah well, maybe I'll send him an email
[03:29] <LaserJock> I'm guessing it'll take a while for a suitable LP admin to wake up
[03:31] <Hobbsee> LaserJock: URL?
[03:31] <LaserJock> https://edge.launchpad.net/~cezpi4y02/+archive
[03:32] <steven__> Hi launchpadders
[03:32] <steven__> Is there an eclipse plugin for bzr
[03:34] <Hobbsee> LaserJock: given back
[03:35] <lifeless> steven__: no; there is a bzr plugin for eclipse though :P
[03:36] <lifeless> LaserJock: you can always upload his package to your ppa
[03:36] <lifeless> LaserJock: :)
[03:36] <steven__> lifeless, GOOD :P
[04:01] <ubotu> New bug: #207558 in bzr ""bzr branch http://bazaar.launchpad.net/...." fails with bzr.dev >= r3309" [Critical,Fix released] https://launchpad.net/bugs/207558
[04:01] <spiv> ubotu: this channel wants to know about the launchpad-bazaar status of that bug, not the bazaar status :P
[08:31] <carlos> morning
[10:09] <Kmos> could someone remove this entry - https://edge.launchpad.net/ubuntu/+mirror/ftp.dei.uc.pt-archive2 ?
[10:10] <Kmos> i own it.. but it's an duplicate
[10:16] <bokey> kiko: hi :) could you help me out sort out http://launchpad.net/~ubuntu-np. basically this person's email is bouncing and nobody from ubuntu nepal community knows him as well.
[10:17] <bokey> we need to maintain Point of contact information there and also some localisation projects.
[10:17] <bokey> thanks!
[10:18] <Matthai> hi, https://launchpad.net/ubuntu/+filebug is not working for me
[10:18] <Fujitsu> !doesn't work
[10:18] <ubotu> Doesn't work is a strong statement. Does it sit on the couch all day? Does it want more money? Is it on IRC all the time? Please be specific! Examples of what doesn't work tend to help too.
[10:19] <Fujitsu> bokey: Please ask a question at the answers URL in the topic.
[10:19] <Matthai> it says "Please try again", a tried after several minutes and still nothing
[10:20] <Fujitsu> Does it have an oops number, or is it something more sinister?
[10:21] <Ng> Matthai: could you try again please?
[10:21] <Matthai> OK, now it is working
[10:21] <Matthai> tnx
[10:21] <Ng> bokey: unless I'm being blind, kiko isn't in here atm
[10:50] <ubotu> New bug: #213257 in malone "Annoyingly readds CVEs when mentioned in comments" [Undecided,New] https://launchpad.net/bugs/213257
[10:55] <Ng> SteveA: about?
[10:57] <SteveA> hi Ng
[11:06] <ubotu> New bug: #213261 in malone/1.2 "Allow specifying the notification level for structural subscriptions" [Undecided,New] https://launchpad.net/bugs/213261
[11:12] <thumper> Fujitsu: how do you know what to prod ubotu with?
[11:12] <Hobbsee> thumper: magic.
[11:12] <Hobbsee> thumper: when's the rollout this month?
[11:12]  * thumper looks
[11:12] <Hobbsee> please don't tell me the 24th.
[11:13] <thumper> Hobbsee: Wed 30th
[11:13] <Hobbsee> oh good
[11:13] <Hobbsee> thumper: what did you want with ubotu?
[11:13] <Fujitsu> thumper: Lots of #ubuntu.
[11:13] <thumper> Hobbsee: I was wondering as Fujitsu mentioned !doesn't work
[11:13] <Hobbsee> ahhhhh
[11:13] <Hobbsee> yeah, #ubuntu and such will do that.
[11:13] <Fujitsu> 'tis a good factoid.
[11:14] <Hobbsee> indeed.
[11:14] <Hobbsee> Fujitsu: i recommend a good, strong drink, btw.
[11:14] <Fujitsu> Hobbsee: Um?
[11:15] <Hobbsee> Fujitsu: you'll need it.  more rubbish is coming.  (speaking of #ubuntu and friends)
[11:15] <Fujitsu> BjornT: Why does #213261 only apply to structural subscriptions?
[11:15] <Fujitsu> Ah.
[11:16] <BjornT> Fujitsu: because it's easier :) it should be done for normal bug subscriptions as well, but we'll do that as a separate task.
[11:53] <luisbg> when I try to push to a branch in bzr...
[11:54] <luisbg> Unable to obtain lock lp--1227763540:///lock
[11:54] <luisbg> held by<EMAIL_ADDRESS>on host vostok [process #25367]
[11:54] <luisbg> :(
[11:54] <luisbg> how do I unlock?
[12:04] <Ng> luisbg: when i hit that recently, I was advised to bzr break-lock the branch
[12:05] <luisbg> Ng, yes
[12:05] <luisbg> I fixed it with bzr break-lock branch_url
[14:31] <ubotu> New bug: #213345 in launchpad "code that logs requests per thread leaving too many open files" [Undecided,New] https://launchpad.net/bugs/213345
[14:52] <cjwatson> Hi, I need to get https://translations.launchpad.net/ubuntu/hardy/+source/debian-installer/+pots/debian-installer updated with the new template from http://people.ubuntu.com/~cjwatson/installer-po/template.pot
[14:52] <cjwatson> this is quite urgent - I thought it had been done ages ago, but it apparently hasn't, and the non-language-pack translation deadline for 8.04 is approaching and will probably have to be extended on account of this
[14:52] <cjwatson> would it be possible for me to get permission to do that upload myself?
[14:54] <cjwatson> carlos: ^-- if you're around ...
[14:55] <carlos> cjwatson: hi, I was planning to do it today
[14:55] <carlos> and yes, I did an update for Hardy a while ago, but it may be more than a month ago
[14:56] <cjwatson> there are missing strings that were added to that template no later than 28 February
[14:56] <cjwatson> we really need to automate this :-(
[15:01] <emgent> heya
[15:01] <carlos> cjwatson: jeroen did a script that should be a good base to develop a cron script for that task
[15:02] <carlos> cjwatson: i will talk with jtv and danilos about it tomorrow to see whether we could push for that as soon as possible...
[15:02]  * carlos -> out. back later!
[15:04] <cjwatson> carlos: thanks
[15:13] <mc__> I told my teacher today about launchpad and he is considering it for use at school. Is there some information/publicity/introducory material besides the tour? ( german preferred)
[15:15] <matsubara> mrevell: ^
[15:16] <mrevell> thanks matsubara
[15:16] <mrevell> Hi mc__
[15:16] <mc__> hi mrevell
[15:16] <mrevell> mc__: The help wiki is a good place to get a feel for what Launchpad does: https://help.launchpad.net/
[15:17] <mrevell> mc__: I'm afraid we don't have anything in German, though :(
[15:19] <mc__> No problem,thank you anyways :)
[15:25] <ubotu> New bug: #213366 in soyuz "OOPS in +builds page passing a list as the query string" [Undecided,New] https://launchpad.net/bugs/213366
[15:26] <mc__> Is there a way I cann announce bugs like that from my own project in my own channel?
[15:48] <hexmode> mc__: if you find out, let me know...
[15:48] <hexmode> I think you can just do an RSS IRC bot
[15:54] <mc__> hexmode: where to get the RSS from?
[15:55] <kiko-afk> mc__, from launchpad.
[15:55] <thatch> mc__: i.e. http://feeds.launchpad.net/ubuntu/+source/trac/latest-bugs.atom
[15:56] <thatch> when viewing the bug list, it's in the html as an alternate
[15:56] <mc__> thank you
[15:57] <thatch> I was wondering how you were typing while afk, kiko :)
[16:00] <kiko> thatch, I have a brainwave sensor that I sometimes use when my wrists hurt. it's a bit imprecise though. when you see any typos from me that's what it is <wink>
[16:04] <thatch> kiko: I have a room full of monkeys and typewriters to handle my typing while afk, but luckily they don't have Internet...
[16:47] <AnAnt> Hello
[16:52]  * sabdfl would hate to have everything he thinks, show up on screen
[16:56] <ubotu> New bug: #213414 in rosetta "template updates should only update date_updated if there are message changes" [Undecided,New] https://launchpad.net/bugs/213414
[16:57] <AnAnt> Is that a bug: when I upload a package to PPA, then re-upload it after changing the orig tarball , the upload gets rejected because md5sum check
[16:58] <AnAnt> of course I done a : debuild after changing the orig tarball
[17:21] <ubotu> New bug: #213430 in blueprint ""Request feedback" should send an e-mail to the person who is asked" [Undecided,New] https://launchpad.net/bugs/213430
[17:24] <daveux> Hey, I'm a happy Launchpad user and I'd like to read some explanations of the decisions to keep some parts of Launchpad closed.  Where can I find this information?
[17:28] <matsubara> daveux: https://help.launchpad.net/FAQ#head-34295746b9c12bbe42eee4a9bd5e2656306fd796
[17:30] <daveux> Thank you.
[17:40] <Anonymo> there's something wrong somewhere
[17:40] <Anonymo> cause i've been redirected to ##launchpad
[17:41] <Anonymo> Please try again
[17:41] <Anonymo> Sorry, there was a problem connecting to the Launchpad server.
[17:41] <Anonymo> Try reloading this page in a minute or two. If the problem persists, let us know in the #launchpad IRC channel on Freenode.
[17:41] <Anonymo> Thanks for your patience.
[17:41] <Anonymo> irc://chat.freenode.net/#launchpad <--- should be without #
[17:43] <andrea-bs> Anonymo: where did you get this error?
[17:43] <Anonymo> https://launchpad.net/kusaba/
[17:43] <AnAnt_> Is that a bug: when I upload a package to PPA, then re-upload it after changing the orig tarball , the upload gets rejected because md5sum check
[17:43] <Anonymo> i get it sometime
[17:44] <AnAnt_> of course I done a : debuild after changing the orig tarball
[17:44] <andrea-bs> Anonymo: only with that page?
[17:45] <andrea-bs> AnAnt_: which are the version numbers of the new package and the old one?
[17:45] <Anonymo> yup
[17:45] <AnAnt_> andrea-bs: old: 0.7.0~beta1-2~ppa2
[17:45] <AnAnt_> andrea-bs: new: 0.7.0~beta1-2~ppa3
[17:46] <AnAnt_> oh
[17:46] <AnAnt_> the package is zekr
[17:46] <andrea-bs> AnAnt_: to upload a new orig you need to publish a new version (i.e. 0.7.1 or 0.8.0)
[17:46] <AnAnt_> andrea-bs: ok
[17:47] <AnAnt_> andrea-bs: so that's not a bug
[17:47] <AnAnt_> ?
[17:47] <andrea-bs> no, this is the Ubuntu policy
[17:47] <AnAnt_> ok
[17:47] <AnAnt_> thanks
[17:48] <AnAnt_> andrea-bs: ok, there is an SVN import requested last week that didn't get attempted yet
[17:48] <andrea-bs> Anonymo: it looks like a real bug, I can't help you :(
[17:48] <Anonymo> kthxbye
[17:48] <AnAnt_> andrea-bs: I sent an email to launchpad-users about it, but the mail didn't get through yet
[17:49] <andrea-bs> AnAnt_: I'm not an administrator, only a contributor, sorry but I can't help you :(
[17:49] <AnAnt_> ok
[17:49] <andrea-bs> AnAnt_: maybe ask on launchpad answers
[18:30] <ubotu> New bug: #213466 in launchpad "Packages missing from PPA listing (missing prev/next buttons?)" [Undecided,New] https://launchpad.net/bugs/213466
[21:21] <ubotu> New bug: #213551 in launchpad "ppa search on package name fails to produce ppa containing said package name" [Undecided,New] https://launchpad.net/bugs/213551
[22:24] <nxvl> i have just install my system and i'm having a problem with launchpad styles
[22:25] <nxvl> the content font is huge
[22:33] <gmb> nxvl: It's a known issue with the way that FF3 renders fonts in some cases.
[22:33] <gmb> nxvl: There's a workaround for it; hang on, let me find it...
[22:34] <gmb> nxvl: Bug 191791
[22:34] <ubotu> Launchpad bug 191791 in firefox-3.0 "Firefox-3.0 zoomed images and webpages (dup-of: 175904)" [Low,Incomplete] https://launchpad.net/bugs/191791
[22:34] <ubotu> Launchpad bug 175904 in firefox-3.0 "Firefox-3.0 window moves to current workspace" [Medium,Confirmed] https://launchpad.net/bugs/175904 - Assigned to Alexander Sack (asac)
[22:35] <gmb> nxvl: The workaround is here: https://bugs.edge.launchpad.net/ubuntu/+source/firefox-3.0/+bug/191791/comments/19
[22:35] <ubotu> Launchpad bug 191791 in firefox-3.0 "Firefox-3.0 zoomed images and webpages (dup-of: 175904)" [Low,Incomplete]
[22:35] <ubotu> Launchpad bug 175904 in firefox-3.0 "Firefox-3.0 window moves to current workspace" [Medium,Confirmed]  - Assigned to Alexander Sack (asac)
[22:38] <emgent> hello, someone can review my python-launchpad-bugs support in anteater tool ?
[22:39] <emgent> https://code.edge.launchpad.net/~ubuntu-whitehat/ubuntu-whitehat-project/uwht.dev
[22:58] <Rinchen> emgent, awesome thanks
[23:03] <nxvl> gmb: thanks i will take a look
[23:06] <nxvl> gmb: it doesn't work :(
[23:08] <gmb> nxvl: Hmm. Best post a comment to the bug to explain the situation; someone there might be able to help.
[23:09] <nxvl> gmb: i file a bug, i will mark it as duplicate
[23:09] <gmb> nxvl: Cool.
[23:10] <nxvl> gmb: i think i have a different problem, now that i see the screnshot
[23:12] <nxvl> gmb: https://bugs.edge.launchpad.net/launchpad/+bug/126993/comments/13
[23:12] <ubotu> Launchpad bug 126993 in launchpad "bug body text font size recently changed to be annoyingly large" [Medium,Fix released]  - Assigned to Matthew Paul Thomas (mpt)
[23:12] <nxvl> it does't zoom every text, just the content
[23:12] <nxvl> i see the menus like they should be
[23:20] <ubotu> New bug: #213642 in shipit "Make look of shipit pages more similar to ubuntu.com" [High,Confirmed] https://launchpad.net/bugs/213642
[23:29] <emgent> hello Rinchen :)
[23:29] <Rinchen> hi emgent.  I can't review your branch but it looks interesting
[23:30] <Rinchen> emgent, I've also let SteveA know it's there in case he was interested.
[23:30] <emgent> heheh :)
[23:31] <ubotu> New bug: #213649 in launchpad "Update menu API to support having a page be the current one across multiple contexts" [High,Triaged] https://launchpad.net/bugs/213649
[23:31] <emgent> Rinchen: it's good project
[23:31] <emgent> we will start it quickly
[23:33] <emgent> more info https://edge.launchpad.net/~ubuntu-whitehat
[23:36] <ubotu> New bug: #213651 in launchpad "Menu-less person pages" [High,Confirmed] https://launchpad.net/bugs/213651
[23:36] <ubotu> New bug: #213652 in launchpad "Menu-less project pages" [High,Confirmed] https://launchpad.net/bugs/213652
[23:36] <ubotu> New bug: #213654 in launchpad "Menu-less distribution pages" [Undecided,New] https://launchpad.net/bugs/213654


[06:51] <sorinello> hello. is there any xfce app to add a system wide proxy in xubuntu ? or do I have to manually edit /etc/environment ?
[06:53] <flocculant> not that I know of
[06:55] <sorinello> just that... editing /etc/environment and rebooting the machine does not work
[06:58] <sorinello> all the examples I find are like: http_proxy=http://myproxyserver.com:80
[06:58] <sorinello> dunny if my proxy server is to be accessed using http
[06:59] <flocculant> sorinello: no idea - pretty much something that should be in #x or if no-one answers there then #ubuntu
[16:05] <flocculant> bluesabre: seems that our terminal is broken by https://github.com/gnunn1/tilix/issues/916
[16:06] <flocculant> more words in xfce-dev
[16:09] <akxwi-dave> bugger
[16:10] <akxwi-dave> thats a big mess up
[16:24] <flocculant> well not broken by that gtihub issue itself - the change in packages does it :)
[16:37] <flocculant> now I'm not even sure where I got the updated terminal package from ...
[16:42] <flocculant> downgraded terminal for the moment
[16:42] <flocculant> that's driving me up the wall :)
[17:04] <flocculant> bluesabre: comments re terminal issue in https://irclogs.ubuntu.com/2017/05/16/%23xubuntu.html#t16:46
[17:04] <flocculant> from jbicha
[17:33] <cyphermox> wxl: you had asked about ip, IIRC. I just found out about 'ip -br addr' and 'ip -br link', if it helps :)
[17:42] <jbicha> cyphermox: could we get --color by default like:  ip a --color
[17:49] <cyphermox> jbicha: please file a bug, but I think we might as well leave that to users to alias as necessary
[17:49] <cyphermox> colors typically don't work all that great on some architectures, and some servers.
[19:44] <Unit193> flocculant: Did you file a bug in vte for that?  I don't see it.
[19:44] <Unit193> Would be nice if it wasn't prematurely updated, or even if it wasn't in main. :P
[19:44] <flocculant> Unit193: didn't do anything but install 0.8.4 so I could use it
[19:44] <Unit193> Ah.
[19:46] <flocculant> Unit193: I can do it now though if you want - report bug to what? terminal or something else?
[19:48] <flocculant> vte2.91 I assume?
[19:49] <flocculant> oh - so that doesn't work
[19:49] <flocculant> *shrug*
[19:52] <Unit193> flocculant: I thought you had already, if you haven't than no issue.
[19:53] <Unit193> (I was looking at
[19:53] <Unit193> https://bugs.launchpad.net/~flocculant/+reportedbugs?field.searchtext=&search=Search&field.status%3Alist=NEW&field.status%3Alist=OPINION&field.status%3Alist=INVALID&field.status%3Alist=WONTFIX&field.status%3Alist=CONFIRMED&field.status%3Alist=TRIAGED&field.status%3Alist=INPROGRESS&field.status%3Alist=FIXCOMMITTED&field.status%3Alist=INCOMPLETE_WITH_RESPONSE&field.status%3Alist=INCOMPLETE_WITHOUT_RES
[19:53] <Unit193> PONSE&assignee_option=any&field.assignee=&field.bug_commenter=&field.subscriber=&field.structural_subscriber=&field.tag=&field.tags_combinator=ANY&field.status_upstream-empty-marker=1&field.has_cve.used=&field.omit_dupes.used=&field.omit_dupes=on&field.affects_me.used=&field.has_patch.used=&field.has_branches.used=&field.has_branches=on&field.has_no_branches.used=&field.has_no_branches=on&field.has_
[19:53] <Unit193> blueprints.used=&field.has_blueprints=on&field.has_no_blueprints.used=&field.has_no_blueprints=on&orderby=-id&start=0 in case someone already incorrectly marked it invalid.)
[19:53] <Unit193> Erm....
[19:53] <Unit193> Wow.
[19:53] <flocculant> what?
[19:54] <flocculant> I'd have told you if I reported something - you didn't need to ping me with my reported bugs :D
[19:55] <Unit193> Right, that's what I was looking at and didn't see it. :P
[19:55] <flocculant> :D
[19:55] <Unit193> And, url is tooooo loooong.
[19:55] <flocculant> indeed it is :)
[19:55] <flocculant> anyway - so do you want me to report it? and to what package and I'll do it now :)
[19:56] <Unit193> Nah, we'll just wait on LP 1636666
[19:57] <flocculant> but do something with terminal?
[19:57] <flocculant> in the meantime
[19:57] <Unit193> Building now, want to test before I push?
[19:57] <flocculant> I can yup
[19:58] <flocculant> point me at it
[20:05] <jbicha> Unit193: it's not a bug in vte2.91
[20:44] <Unit193> jbicha: https://launchpad.net/ubuntu/+source/vte2.91/0.48.2-0ubuntu1 would disagree with you, but not my point anyway...
[20:46] <jbicha> yes, but it's intentional
[20:47] <Unit193> Sure, but it's an intentional bug that breaks other things, thus still a bug in vte.
[20:49] <jbicha> all you have to do is patch xfce-terminal to not assume that vte 1.48 uses pcre2, change the check to vte 1.99 or whatever made-up number you want instead
[20:49] <Unit193> That's what I did...
[20:50] <jbicha> we'll eventually switch vte2.91 to pcre2 but the Security Team would rather that we postpone that for a while
[20:51] <jbicha> pcre2 got a very high showing in https://opensource.googleblog.com/2017/05/oss-fuzz-five-months-later-and.html :(
[21:00] <jbicha> https://people.canonical.com/~ubuntu-archive/transitions/html/pcre2-main.html


[12:27] <oSoMoN> hi all
[12:28] <oSoMoN> I had a libreoffice build going in a PPA of mine, which I saw failing towards the end of the build due to a unit test failure, and when I went back to it to get the full log I saw that the build had just been restarted under my feet
[12:28] <oSoMoN> how is that possible?
[16:14] <cjwatson> oSoMoN: Can happen if your build crashes the builder.
[16:15] <oSoMoN> cjwatson, I guess in that case there's no way to recover the build log until the builder crashed, right?
[16:15] <cjwatson> Correct.
[16:15] <oSoMoN> right, so I guess it makes sense to trigger a new build
[16:16] <oSoMoN> it's kind of a pain for libreoffice and similar packages that take forever to build though
[16:16] <cjwatson> Such is life, I'm afraid.
[16:16] <oSoMoN> fair enough
[16:16] <oSoMoN> thanks for the explanation!
[16:17] <cjwatson> (For completeness, it can also happen if LP loses connectivity to the builder for long enough, or some other similar issue.)
[16:59] <rbasak> I'm expecting lp.git_repositories.getByPath to be callable, but instead I get a KeyError: http://paste.ubuntu.com/25904265/
[16:59] <rbasak> What am I missing?
[17:00] <rbasak> Using python3-launchpadlib    1.10.3-3ubuntu0.1
[17:00] <rbasak> On Xenial.
[17:00] <rbasak> len(lp.git_repositories) is 0, and next(iter()) on it gives me an immediate StopIteration.
[17:01] <rbasak> len(lp.distributions) says 37, so presumably something works.
[17:02] <rbasak> My paste is wrong. It was "import launchpadlib.launchpad" not just "import launchpadlib".
[17:03] <rbasak> (but I still have the problem)
[17:23] <rbasak> I think I figured it out.
[17:24] <rbasak> version='devel'. Not just 'devel'.
[17:24] <rbasak> nacc: ^
[17:24] <rbasak> (FYI)
[17:37] <nacc> rbasak: ah sure
[18:10] <cjwatson> Yep.  (Also, the reason len(lp.git_repositories) is 0 is that iterating over all Git repositories in LP isn't a thing we consider sensible to do on the webservice, but the collection has to exist as a place for all-Git-repositories-related methods to live.)
[18:13] <nacc> cjwatson: is there a sensible way for me to go from a user or team to all git repositories for that user/team? Or do I need to do an iteration over all the git repositories? In essence, I want to iterate the list of current usd-import-team git repositories so I can see what source packages are already imported and determine what their default git repo s
[18:20] <cjwatson> We could export something for that if needed, although we might want to come up with something more targeted; iterating over 20k+ or whatever entries isn't going to be particularly quick.
[18:21] <nacc> rbasak: --^ maybe we're thinking about it wrong, though; given that we have the phasing constants, whitelist and blacklist available, I can figure out the list of source packages we 'expect' to ahve imported currently (in theory, this is a one-time operation anyways) and we can iterate any of those with usd-import-team repositories?
[22:16] <Saviq> is it possible with launchpadlib to find if a person is an indirect member of a team? I can only see https://launchpad.net/+apidoc/devel.html#team_membership mentions a ParticipationTable, but can't find it anywhere else
[22:20] <Saviq> also found mention of "IPerson.inTeam", but bug #377579 :/
[22:23] <Saviq> or do we have a better way to check if someone's signed the CLA, or is employed by Canonical, to check if their contributions can be appropriated correctly? maybe a webservice of some kind?
[22:25] <cjwatson> The best way I can think of OTTOMH (inefficient but I believe it works) is to get the target team and iterate over its .participants
[22:29] <cjwatson> I'm not totally sure why the answer to that bug isn't to just put the method on the team rather than the participant, though
[22:34] <cjwatson> I guess that wouldn't help with private subteams


[04:06] <StevenK> thumper: Hey, can you look at db-add-ifp-job again, when you have a tick?
[04:14] <thumper> StevenK: ok
[04:49] <jtv> mwhudson: maybe you can review this urgent but near-trivial fix?  https://code.edge.launchpad.net/~jtv/launchpad/bug-631259/+merge/34651
[04:49] <jtv> (Some would call it trivial but the term is overused)
[05:05] <jtv> mwhudson: you move quietly but surely; hadn't noticed your approval.  Thanks!
[05:05] <jtv> I'm just adding a test actually.  :)
[05:08] <mwhudson> jtv: heh
[05:08] <mwhudson> yeah
[05:08] <mwhudson> oops
[05:10] <jtv> I don't think it matters much since I just restored something that's been in production for years, and the tests have to pass since we just removed everything that relies on this field, but it looks more proper
[08:23] <henninge> jtv: I have to admit I was not really familiar with the fact that UPDATEs could be fed their values from SELECTs. cool
[08:23] <henninge> jtv: What happens if a SELECT returns more than one row?
[08:24] <jtv> There shouldn't be any SELECTs… did I make a mistake?
[08:24] <jtv> Nope, there are no SELECTs.
[08:24] <henninge> jtv: sorry, I am talking about anything after FROM ...
[08:26] <henninge> jtv: like the second query (for ideal cases) where the condition could be true for more than one POFILE.id (and it's not unlikely I think.)
[08:27] <jtv> henninge: picks an arbitrary one.
[08:27] <henninge> Ah, good for us ;_)
[08:28] <jtv> May even do multiple updates of the same record, but frankly I just don't care.  The result is still the same, apart from performance.
[08:28] <henninge> jtv: I know we don't care, I was more worried about it breaking.
[08:28] <henninge> the query, I mean
[08:28] <jtv> It'll work—I did it this way to keep the query simple to reduce risk of mistakes.
[08:30] <henninge> jtv: I think you should leave the condition explicitly in the last two queries.
[08:30] <jtv> henninge: I don't understand
[08:31] <henninge> you know, POTemplate.iscurrent IS NOT TRUE and TranslationTemplateItem.sequence = 0
[08:32] <henninge> jtv: I am just thinking that will make it more transparent to others that will read these.
[08:32] <jtv> henninge: that only introduces more complexity and so more danger of mistakes.  The condition "TranslationMessage.pofile" takes care of it.
[08:33] <henninge> jtv: ok, it was just a thought.
[08:33] <henninge> jtv: r=me
[08:34] <jtv> henninge: great, thanks.
[08:34] <jtv> Now to get my branch in somehow.  :-(
[08:34] <henninge_> jtv: as you can tell, I did not do anything about it yet ...
[08:35] <henninge_> but I heard your thanks.
[08:35] <jtv> henninge_: I was thanking you for the review of my queries
[09:20] <lifeless> henninge_: hi
[09:20] <lifeless> https://code.edge.launchpad.net/~lifeless/launchpad/private-librarian/+merge/31020
[09:28] <henninge> lifeless: I am sorry but I will have to refuse this one.
[09:28] <lifeless> henninge: ok
[09:28] <lifeless> henninge: may I ask why ?
[09:29] <henninge> lifeless: besides it being oversized it would take me all day to get into it and I think Stuart can do that much more efficiently.
[09:29] <lifeless> fair enough
[09:29] <henninge> sorry
[09:29] <lifeless> thats fine
[09:29] <lifeless> stub: ping
[09:29] <stub> lifeless: pong
[09:30] <lifeless> stub: the librarian rework is ready for another review
[09:30] <stub> k
[09:30] <lifeless> stub: code & db & all
[09:32] <lifeless> stub: I've just pushed up 9588 making the hostname check a little cleaner and fixing the build.
[09:32] <stub> k
[09:32] <stub> There are still changes in session.sql that should be in launchpad_session.sql
[09:33] <stub> This is all Launchpad specific stuff
[09:33] <lifeless> I must have missed that
[09:33] <lifeless> moving around
[09:34] <lifeless> pushing
[09:34] <stub> Putting session_store() in sqlbase is naughty cause it should really be in lp.*mumble*. But I do that too because foundations has never really been clear on where its stuff should live.
[09:35] <lifeless> I figured consolidated it into a function anywhere was better than making another callsite with the full spelling.
[09:37] <stub> re: the comment in MixedFileAliasView, you might want to consider opening bugs on the tech-debt work (remove the old code one we are sure things work with the feature flag on, consolidate it into LibraryFileAliasView). The bug can then be referenced as an XXX:
[09:38] <stub> Your XXX just below that is naughty as it isn't referencing a bug. If we need to fix it, open a bug. If not, it isn't an XXX but a comment.
[09:39] <stub> contecxts is not spelled that way
[09:41] <stub> I think assert is fine over 404, as the view should never be called for deleted files and we want bad links like that to show up on OOPS reports.
[09:41] <lifeless> I'll use 395960 for the xx bugs here
[09:41] <lifeless> all the same thing, its not done till its done
[09:42] <lifeless> the xxx policy of needing a bug, is, IMCO a bit noddy, still, linked it to 395960
[09:42] <lifeless> re the assert
[09:42] <lifeless> if someone opens a page with an attachment
[09:42] <lifeless> and someone else deletes the attachment
[09:42] <lifeless> then the user clicks on the link
[09:43] <lifeless> we'll trigger an assert for a normal situation; that makes no sense to me.
[09:44] <lifeless> I've put that usecase in.
[09:49] <stub> line 239 of the diff is engrish
[09:49] <lifeless> +from hashlib import md5?
[09:50] <stub> +    LibraryFileAlias directly is OK as at that point the status of the file239+    is know.
[09:50]  * stub reloads the diff
[09:52] <stub> ProxiedLibraryFileAlias should be rolled into LibraryFileAlias eventually?
[09:52] <lifeless> I think it should be renamed
[09:52] <lifeless> LibraryFileAliasReference
[09:53] <lifeless> not sure about roll-in or not
[09:53] <lifeless> Proxied... dates to sept last year
[09:54] <stub> lifeless: Your database patch uses 'timestamp with time zone' but your storm class is using UtcDateTime. I think your DB patch should be 'timestamp without time zone' to match the storm class and our existing timestamp columns.
[09:56] <stub> I don't think Storm uses notNull in the column definitions? IIRC that is SQLObject specific.
[09:58] <stub> lifeless: I don't understand why a 16 character hex string is preferable to str(random.random())
[10:00] <lifeless> stub: running make schema with the tz change
[10:01] <stub> If you want to make them less guessable, md5(baseline + str(time.time()) + url + thread.get_ident())
[10:01] <stub> I suspect the store.commit() is unnecessary as I think that connection is running in autocommit mode. I haven't double checked though.
[10:02] <lifeless> stub: it seemed nicer
[10:02] <stub> So the only purpose it serves it to issue a redundant call to the database.
[10:02] <lifeless> the store.commit won't hit the db will it? no changes - noop ?
[10:02] <stub> I guess the alternative is flush
[10:03] <stub> lifeless: Hmm... I'm not sure. I guess Storm will know its a noop and noop it.
[10:03] <lifeless> anyhow, its there as belt and bracers, can definitely take it out
[10:10] <stub> lifeless: Any reason we are only using multidomain stuff for restricted files?
[10:11] <lifeless> yes
[10:11] <lifeless> dns lookups
[10:11] <lifeless> librarian files are used for branding images
[10:11] <lifeless> we don't want to cause dns lookups *and* SSL handshaking for every one of those
[10:12] <stub> ok. I was thinking it might be desirable to trick browsers to load things in parallel, like team emblems.
[10:14] <lifeless> three-way handshake + dns will swamp the benefits there
[10:14] <lifeless> what we should do is have a spite generator sort of thing
[10:14] <lifeless> we can revisit in future obviously
[10:14] <stub> What happens now if a test module doesn't have a test_suite() function?
[10:14] <lifeless> the tests are loaded via TestLoader().loadTestsFromModule
[10:15] <jtv> henninge: could you review this testfix branch? https://code.edge.launchpad.net/~jtv/launchpad/testfix/+merge/34663
[10:15] <lifeless> i.e. it DTRT if the test_suite was just doing loadTestsFromName
[10:15] <stub> Ok. I think I added test_suite to the template not to long ago :)
[10:16] <lifeless> :P
[10:19] <jtv> henninge: the diff has appeared
[10:19] <henninge> jtv: yes, I am on it
[10:19] <jtv> thanks
[10:22] <stub> lifeless: So +    def getAlias(self, aliasid, token, path): accepts path as a parameter, and then proceeds to pretend it is a URL.
[10:22] <stub> lifeless: I think you need to make your mind up and change the parameter to URL and tweak the documentation so it matches reality.
[10:23] <lifeless> its a path only as per your previous comments
[10:23] <lifeless> I will update the table definition
[10:35] <stub> Imports on diff line 1407 are badly formatted (layers.py)
[10:37] <stub> Annoying that Librarian stuff has spread from just canonical.librarian to canonical.launchpad.database.librarian and other files
[10:37] <stub> lifeless: But done. Nothing major I can see.
[10:38] <lifeless> thanks
[10:43] <jtv> henninge: any progress with my branch?
[10:43] <henninge> jtv: sorry, distracted.
[10:43] <jtv> henninge: not a good thing while reviewing a testfix!  :)
[10:43] <henninge> jtv: I was just looking closer at one thing.
[10:43] <henninge> sorry
[10:44] <henninge> jtv: r=me
[10:44] <jtv> thanks
[10:45] <jtv> It's in pqm.  I'm taking a break for urgent non-work stuff.
[10:46] <jtv> gah
[10:47] <jtv> gmb, I'm not sure but I _think_ the PQM error message says I need RC to land this testfix: https://code.edge.launchpad.net/~jtv/launchpad/testfix/+merge/34663
[10:49] <gmb> jtv, Landing it on devel, perchance?
[10:50] <jtv> gmb: yes
[10:50] <jtv> that's the build it fixes
[10:50] <gmb> jtv, Right. Yes, you need RC. And you have it.
[10:50] <gmb> :)
[10:50] <jtv> gmb: many thanks!
[10:50] <jtv> …on the way.
[10:51] <gmb> #\0/
[19:58] <gary_poster> hm.  no-one is here to review my poor little MP https://code.edge.launchpad.net/~gary/launchpad/bug627442/+merge/34701 :-/
[20:04] <lifeless> looking
[20:08] <gary_poster> thank yoU!
[20:09] <lifeless> gary_poster: needs an rc stamp
[20:09] <lifeless> also I need to sit on thumper and convince him to ok the use of queues.
[20:10] <gary_poster> lifeless, thanks.  will ping gmb.
[20:10] <lifeless> if he's not around, any other TL will do
[20:10] <lifeless> or we can get someone else to do the code review and I'll make my stamp a release-critical
[20:10] <lifeless> (lp only allows one type of review per reviewer at the moment, ce la BUG
[21:04] <thumper> lifeless, gary_poster: it is ok for someone to give an rc and a review
[21:04] <thumper> it is just the landing script that is not smart enough
[21:06] <lifeless> thumper: it would be nice if lp could store it though
[21:26] <lifeless> gary_poster: hi
[21:26] <gary_poster> lifeless, hi
[21:27] <lifeless> I've fixed the remaining test failure in my librarian MP, and need the commits-for-that to be reviewed ;)
[21:27] <gary_poster> heh, ok, where do I look?
[21:27] <lifeless> https://code.edge.launchpad.net/~lifeless/launchpad/private-librarian/+merge/31020 is the MP
[21:27] <lifeless> I'll get you specific commit links in a sec
[21:28] <gary_poster> k thanks
[21:28] <lifeless> (lp is just scanning it)
[21:29] <lifeless> http://bazaar.launchpad.net/~lifeless/launchpad/private-librarian/revision/9594 was abels fix to a test script I've reviewed this, so strictly it doesn't need more eyeballs. OTOH eyeballs are valuable ;)
[21:30] <lifeless> http://bazaar.launchpad.net/~lifeless/launchpad/private-librarian/revision/9595 I found a missing/unclear thing in feature flags which I just tweaked.
[21:30] <lifeless> http://bazaar.launchpad.net/~lifeless/launchpad/private-librarian/revision/9596
[21:30] <lifeless> http://bazaar.launchpad.net/~lifeless/launchpad/private-librarian/revision/9597
[21:30] <lifeless> http://bazaar.launchpad.net/~lifeless/launchpad/private-librarian/revision/9598
[21:31] <lifeless> they are all tiny, just split up for clarity
[21:32] <lifeless> however, loggerhead looks to be on the way down
[21:32] <lifeless> so perhaps just pull lp:~lifeless/launchpad/private-librarian and do 'bzr log -p | less and look at the top 4 patches
[21:34] <lifeless> gary_poster: ^
[21:34] <gary_poster> ack, looking
[21:36] <lifeless> thanks
[21:40] <gary_poster> lifeless, IIUC, the current thought is that we want to make sure that .annotations is reliably on every request Launchpad code sees, and then we can do away with other thread locals that are compensating for that lack.  Right?  If so, +1
[21:40] <lifeless> gary_poster: yes
[21:40] <gary_poster> cool
[21:40] <lifeless> gary_poster: we need to extend the concept of annotatable things to scripts
[21:40] <lifeless> gary_poster: thats the sticking pointy bit
[21:41] <gary_poster> scripts, or the requests that scripts use?
[21:41] <lifeless> well, scripts don't used requests today.
[21:41] <gary_poster> "interactions"
[21:41] <gary_poster> no, "participations"
[21:41] <lifeless> yes, participation is the leverage point we should use to sort this mess out
[21:41] <gary_poster> cool, agreed
[21:42] <lifeless> right now scripts with with a special security policy
[21:42] <lifeless> and a single noddy participation
[21:42] <gary_poster> heh
[21:42] <lifeless> what they need to do is switch participations when doing work-on-behalf-of-something-else
[21:42] <gary_poster> absolutely
[21:42] <lifeless> the special security policy says 'anything goes'
[21:43] <gary_poster> ...is that special security policy desirable, or legacy?  Would a more careful policy buy us something worthwhile?
[21:44] <lifeless> legasheee
[21:44] <lifeless> basically, it went something like this
[21:45] <lifeless> early scripts used different domain objects - e.g. sql
[21:45] <lifeless> that was bad
[21:45] <lifeless> so bits of zope were stubbed up enough to use the same sqlobject objects
[21:45] <lifeless> 'zopeless'
[21:45] <lifeless> that had issues
[21:45] <lifeless> so more was included
[21:45] <lifeless> 'parseZCMLForScripts' or whatever
[21:45] <lifeless> anyhow, its all grown organically.
[21:46] <gary_poster> Gotcha.  yeah, t happens.
[21:46] <gary_poster> from a review perspective, is any other test code currently doing the dance that you do in librarian.txt to work around bug 631884?  If so, should it be a test helper?  If not, never mind.
[21:46] <_mup_> Bug #631884: feature flags get out of sync easily <Launchpad Foundations:New> <https://launchpad.net/bugs/631884>
[21:46] <lifeless> so we should IMO use the same security framework and policies regardless of context.
[21:46] <gary_poster> Sounds great to me.
[21:46] <lifeless> gary_poster: no other code is doing it yet.
[21:46] <gary_poster> ok
[21:47] <lifeless> next one along can make it a helper - or we may have the bug fixed to not need it earlier.
[21:47] <gary_poster> I have a had a card to try to audit the security we have in views and see what we can do to rip it out
[21:47] <gary_poster> it is a problem
[21:47] <gary_poster> i have not gotten around to it
[21:47] <lifeless> what sort of stuff are you referring to there?
[21:48] <lifeless> things like stashing 'this is ok' directly into the permission cache?
[21:49] <gary_poster> like, direct calls in views to checkPermission, where it is effectively an actual security protection.  I have rarely been in the situation in which this made sense, but canWrite and canRead are safer when you need it, and I'm pretty sure we are abusing these sorts of things.
[21:49] <lifeless> ah yes
[21:49] <lifeless> they seem ugly to me
[21:49] <gary_poster> This makes our automated security worse than nothing in some cases
[21:49] <gary_poster> Because it lulls into a false sense of security
[21:49] <gary_poster> for the webservice for instance
[21:50] <lifeless> I'd like to get us really using zope permissions as properly as possible.
[21:50] <gary_poster> but also across domain knowledge, or even across forgetfullness
[21:50] <lifeless> *if* they aren't sufficient/performant-enough we can talk about what to do, but we can't really assess that while stuff is wedged in every which way
[21:50] <lifeless> gary_poster: I agree completely
[21:51] <gary_poster> cool
[21:51] <gary_poster> back to review ;-)
[21:59] <gary_poster> lifeless: typo "acces" in lib/canonical/launchpad/database/librarian.py : """Return the token path used for authorising acces to url.""" Everything else looks good.
[21:59] <lifeless> gary_poster: thanks!
[22:00] <gary_poster> np
[23:41] <lifeless> I need a) a TL to release-critical and b) a code review for this 5-liner https://code.edge.launchpad.net/~lifeless/launchpad/oops/+merge/34709
[23:44] <lifeless> thumper: mwhudson: I pick thee ^
[23:44] <thumper> lifeless: done
[23:44] <thumper> rc anyway
[23:44] <thumper> you could submit with rc and r = me if you like
[23:52]  * lifeless does that
[23:52] <lifeless> damn the torpedoes!


[03:23] <ajmitch> hads: I'd like to go, maybe I'll have to see if I can stay at my sister's place :)
[03:24]  * ibeardslee just had the word from above .. it'd have to be under my own steam
[03:24] <ajmitch> shame
[03:25] <ibeardslee> aye
[03:25] <ajmitch> ticket+accomodation+travel+days off work tends to add up a bit
[03:26]  * ibeardslee really needs to find more interesting things to talk about
[03:28] <ojwb> possible tip - osdc last year was in canberra and from wellington i found it was much cheaper (and actually quicker) to fly to sydney and get the bus which runs from sydney airport to central canberra
[03:28] <ojwb> i saw something suggesting there might be direct flights by lca though
[03:29] <ojwb> which would at least make that a slower option
[03:32] <ajmitch> ojwb: yeah I've taken the bus from sydney airport before, it often works out pretty cheap
[03:33] <ojwb> the one annoying feature is it's cheaper if you say exactly which bus you'll be on, which is a bit hard to predict with customs, etc
[03:33] <ajmitch> unless you want to wait around for some time
[03:33]  * ajmitch has always found australian customs to be fairly quick
[03:34] <ojwb> yeah, usually isn't bad, but it'd be annoying to have to buy a whole new ticket
[03:35] <ajmitch> do you have a NZ passport?
[03:36] <ajmitch> in the last few years it's been made quicker with the automation, you don't really need to talk to a customs person
[03:36] <ojwb> no, uk one
[03:36] <ojwb> not quite been here long enough to get an nz one
[03:45] <hads> It does add up, ticket price is only a moderate percentage.
[03:45] <hads> I've missed the last two though so wasn't missing a third.
[05:35] <G> ajmitch: Customs is only quicker etc, if you have absolutely nothing to declare, living rurally throws that out of the water so to speak
[05:37] <G> 30-60 minutes in customs hall at Brisbane last time (includes the queue when I arrived in school holidays morning), vs straight through the MAF checkpoint at Auckland at 11:30pm with quite a few others
[05:59] <ojwb> MAF seemed very slack coming in a few months back
[05:59] <ojwb> no interest in walking boots at all
[08:39] <G> ojwb: I have my entry to Australia routine down good "yeah, I live on a farm, but nothing I've got in my bag has been on the farmland, only the driveway" they umm and ahhh for a minute then decide to just put me through the dog queue
[19:13] <ibeardslee> morning
[19:20] <Atamira> morning
[19:20] <Atamira> raining up here
[19:24] <ibeardslee> supposed to be raining here this morning
[19:24] <ibeardslee> haven't seen it yet
[19:28] <ojwb> morning
[19:39] <mwhudson> morning
[19:48] <ajmitch> morning
[20:25] <chilts> morning
[20:27] <ojwb> every time I go to report a bug in ubuntu I find it harder to find the link
[20:28] <ojwb> aha - https://bugs.launchpad.net/ubuntu/+filebug/?no-redirect
[23:19] <thomi> lifeless: you around? Got a minute to answer some questions about testtools & unicode?
[23:21] <thomi> I'm trying to figure out what I have to do to make testtools work with an assertion message that contains unicode characters, currently I get this: http://pastebin.ubuntu.com/1259064/
[23:22] <lifeless> thomi: ok, so thats not testools thats blowing up :)
[23:22] <lifeless> UnicodeDecodeError: 'ascii' codec can't decode byte 0xa1 in position 425: ordinal not in range(128)
[23:23] <lifeless> note that its a *decode* error,
[23:23] <lifeless> but the line of code referenced is doing an *encode* operation.
[23:23] <thomi> ... huh
[23:24] <lifeless> thomi: you have a bytestring (type(foo) is str), which is non-ascii, and is being passed in to a thing that is expecting a string (type(foo) is unicode))
[23:24] <thomi> hmmm
[23:24] <lifeless> thomi: I bet you that if you put a debug statement there, or run under pdb or something
[23:24] <lifeless>  you'll find text is a str, not unicode
[23:25] <thomi> hmmmm
[23:25] <lifeless> thomi: now, testtools can deal with arbitrary binary data, but you're in the text attachment codepath
[23:25] <thomi> ahhhh
[23:25] <lifeless> because backtraces are strings, not bytestrings (normally)
[23:25] <thomi> the penny drops!
[23:25] <lifeless> thomi: note that if you do this: raise MyException(u"foo\x8")
[23:25] <lifeless> under python2.x
[23:26] <lifeless> you'll fuck things up royally
[23:26] <thomi> yeah, I'm attaching content using text_content
[23:26] <thomi> ...which, n this test, will have odd unicode characters in it, and I'm using bytestrings :(
[23:26] <lifeless> because python2 's __unicode__ and __str__ for Exception is broken.
[23:26] <lifeless> thomi: so, either attach them as non-text
[23:27] <lifeless> thomi: or attach them as text_content with actual strings, not bytestrings
[23:27] <thomi> this is python 2, so my understanding is that there's just str and unicode, right?
[23:28] <lifeless> effectively
[23:29] <lifeless> there is a bytes type in 2.7, but its just an alias to str
[23:29] <lifeless> so its not terribly useful in catching confused types early.
[23:29] <lifeless> thomi: can you paste the attaching code?
[23:30] <thomi> sure, I was using this:
[23:30] <thomi> stdout, stderr = process.communicate()
[23:30] <thomi> self.addDetail('stdout', text_content(stdout))
[23:30] <thomi> where 'process' is a subprocess.Popen object
[23:47] <lifeless> ah yes, so thats not going to work so well :)
[23:47] <lifeless> as it could be anything ;>
[23:48] <lifeless> you can either decode stdout
[23:48] <lifeless> if you know, for instance, that its utf8
[23:48] <lifeless> stdout = stdout.decode('utf8')
[23:48] <lifeless> then it will be text that you're adding
[23:49] <lifeless> or you can use the lower level layers to add what you've got with a more conservative encoding (e.g. cp1280 or whatever it is that accepts 256 bytes as legitimate, never crashes.
[23:50] <lifeless> thomi: e.g.
[23:52] <lifeless> self.addDetail('stdout', Content(ContentType('text', 'plain', {'charset': 'iso-8859-1'}), lambda:[stdout]))
[23:55] <thomi> lifeless: cool, that works - thanks :)
[23:56] <lifeless> thomi: np