/srv/irclogs.ubuntu.com/2010/05/05/#ubuntu-reviews.txt

nigelbabuStatus update: 141 bugs :)01:08
micahghow much longer will people be around?01:08
persiauGH.  cHANNEL FAILURE :(01:09
nigelbabutill tomorrow 1000 UTC01:09
persiaAnd it's 0:00 UTC on Patch Day! As the 5th of May begins to cross the Atlantic, we've 141 bugs with patches left in the queue.01:09
persiaThat was supposed to be in *this* channel 10 minutes ago :(01:09
nigelbabuhehe :)01:09
* micahg might be able to do one or two in a couple hours01:10
persiamicahg: The idea being that Patch Day is happening as long as it's 5th May *somewhere* on the planet.  We're not geographically biased :)01:10
persiaCool!01:10
persianigelbabu: Do you happen to have the count that is now 141 from before we started?01:10
nigelbabupersia: 183 or 18501:11
persiaSo we're 23% done.  Thanks.01:12
nigelbabuto think we had gotten down to 100 something a few weeks back and I worried about running out of bugs01:13
persiaheh.  There's always folks submitting new patches.01:13
* micahg needs to make a few SRU patches...01:13
persiaHey mannyv!01:19
nhandlerLooks like I'm up in about 10 minutes01:51
persiaIf you like :)  I thought you weren't scheduled for 0100 UTC until tomorrow.01:54
nhandlerpersia: Oh, you're right :) That works better for me, as I can get ClaseBot finished up for UOW-es ;)01:55
persiaHeh, go for it.  If you've time later, I'm unsure about how well I can cover the 0600-1000 span, but that may be too late for you.01:57
arandFor Bug #538747 is patch-needswork and patch-forwarded-upstream appropriate? (and unsubscribe) -- Patch was sent upstream, where decision was made to delay inclusion until it was fixed up to handle both input and output of SI units.02:10
ubot3Malone bug 538747 in gparted "gparted should use SI prefix instead of IEC prefix as default" [Undecided,New] https://launchpad.net/bugs/53874702:10
ajmitchlooks like the count is slowly coming down02:11
persiaarand: Looks right to me, yeah.02:12
persiaajmitch: slowly, but we're roughly on-trend.02:13
* ajmitch found one where he could do the debian upload & fix other bugs, takes some time :)02:13
arandAh, one must be a prt of reviewers team to unsubscribe them, right? (If anyone would care to on above bug..)02:18
persiaSure.  Done.02:18
arandthanks02:23
micahgso, what are the rules?02:55
micahgnigelbabu: do I need to be added to reviewers to do this?02:57
persiamicahg: You don't.02:57
persiahttps://wiki.ubuntu.com/ReviewersTeam/ReviewGuide are the rules02:58
persiaThere's a link ("query") under workflow that shows the pieces in play (currently 137)02:59
persiamicahg: making sense to you?03:06
micahgpersia: yeah, I'm looking at one now03:06
persiaCool.03:06
micahgpersia: what if another team is supposed to review the patcg?03:11
micahg*patch03:11
persiaDo you have an example?03:11
micahglike security team in the case of my bug03:11
micahgbug 57012803:11
ubot3Malone bug 570128 in firefox-3.5 "apparmor profile blocks Sun Java plugin" [Undecided,Confirmed] https://launchpad.net/bugs/57012803:11
persiaAh, good example.  That's an Ubuntu-specific bug, so we need to push to Ubuntu.  Yeah, coordinate with the security team on that one.03:12
micahgpersia: can you unsub reviewers and I'll sub security?03:12
persiaSure.03:13
persiaDo you know that the security team prefers to receive bugs by subscription?  I'd hate to see that get lost.03:14
micahgyes03:14
micahgin this case03:14
persiaOK.  Just wanted to make sure.  I don't know much about their procedures.03:15
micahgpersia: yeah, I talked to jdstrand about these bugs, he usually jumps on them pretty fast03:16
persiaGreat, then it's just mistargeted.03:17
* ajmitch unsubs -reviewers from bug 570209, should get fixed in debian soonish03:22
ubot3Malone bug 570209 in muine "muine about page links to a domain squatter homepage" [Undecided,New] https://launchpad.net/bugs/57020903:22
persia\o/03:24
ajmitchit needs an FTBFS fix anyway, easiest to do mono stuff in debian & sync it03:24
micahgcan someone unsub reviewers from bug 51261503:40
ubot3Malone bug 512615 in firefox "fonts are incorrectly rendered due to not using system cairo" [High,Fix released] https://launchpad.net/bugs/51261503:40
persiaSure.03:40
persiamicahg: Was that patch ever sent upstream, or shouldn't it be for some reason?03:41
micahgpersia: hmm, good question, I don't think upstream mozilla wants it, they'd prefer upstream cairo to get it03:41
micahgthat's a whole mess03:42
micahgis -reviewers also a placeholder for upstreaming?03:42
persiaThat's primarily what we do, actually.03:43
micahgok, good to know03:43
persiaWe don't presume to understand all the code, but we know how to test, and we know how to push upstream.  So generally we confirm the bug, confirm the patch works, and make sure upsteam can review it to adopt or drop.03:43
persiaIf you want to handle this specific bug differently in your role as part of ubuntu-mozillateam, I'm happy to unsub the reviewers.  If you want to get reviewers unsubbed by acting as a reviewer, it ought be upstreamed first, etc. so it doesn't get lost.03:44
micahgk, leave it there for now, what should really happen is the Ubuntu cairo patch get upstreamed03:45
micahgthen we won't have to patch our source or mozilla's03:45
persiaThen there should be an upstream cairo task on that bug :)03:47
persiaOr is that a different bug?03:47
* persia reads the bug log harder03:47
micahgpersia: different bug03:47
micahgfreedesktop 1030103:48
ubot3Freedesktop bug 10301 in freetype font backend "LCD filtering patch" [Normal,Reopened] http://bugzilla.freedesktop.org/show_bug.cgi?id=1030103:49
persiaRight.  No point in us reviewing that bug then: it's all done.03:49
* persia unsubscribes03:49
micahgit was a hack that we were kinda forced to do since our system cairo is patched and mozilla wants us to use their cairo in the build03:50
persiaUgh, yeah.03:53
micahgif you can any mozilla stuff in teh queue, feel free to ping me03:59
* persia looks quick-like03:59
micahgI didn't see any in the list tonight03:59
micahgsave the one we already took care of03:59
persiaDunno if bug #570812 counts :)03:59
ubot3Malone bug 570812 in chromium-browser "Use the ubuntu startpage by default" [Undecided,Confirmed] https://launchpad.net/bugs/57081203:59
* micahg doesn't know chromium yet :/04:00
persiaI don't see anything else offhand.04:01
micahgk, if something comes up feel free to ping me04:01
micahgshould I idle in here?04:01
persiaIf you like.  If not, if we're pushing something into mozillateam, we'll come find you :)04:02
persiaAnd if you have time, we'd always apprecitate help with reviewing other patches.04:02
micahgk, well, I feel I'm already spread too thin and after UDS, I'm going to have a lot more to do04:03
* micahg needs to learn to recruit and delegate04:04
persiaOK.  Stop by if you find time, or if you can reserve some for the occasional patch days.04:05
micahgk04:06
persiaJust past 3:00 UTC, with 135 patches left to go.  We're 37.1% done!04:12
persiaErr, 27.1% (I can't do math)04:17
ajmitchdoing wel so far04:17
persiaA little below trend, but this is a historically slow time of day.  I think we still have a chance.04:18
ajmitchbug 3179094 looks like nothing patch-worthy, it's just hardware info being collected04:19
ajmitchbug 31709404:19
ubot3Malone bug 317094 in xf86-input-evtouch "meta bug to collect lshal touchscreen info" [Undecided,Incomplete] https://launchpad.net/bugs/31709404:19
ajmitchhm04:19
ajmitchmight help if I look at all 200+ comments04:19
* persia pokes ogra about it04:20
ajmitchit was autotagged as having a patch, nothing useful04:20
ajmitchI'll remove the tag & subscription04:20
* persia removes the patch flag from the evtouch_hal.out attachment04:21
ajmitchah, I didn't spot you could do that04:22
persiaOn the right side, if you click "edit" for the attachments.  When there's a patch, there's a special "patches" box with just patch attachments.04:23
ajmitchright, I'm just having LP timeout again04:24
persiaHey RAOF!04:33
RAOFpersia: Yo!04:33
persiaToday is https://wiki.ubuntu.com/PatchDay so we're following https://wiki.ubuntu.com/ReviewersTeam/ReviewGuide#Workflow04:34
persiaWe're nearly 30% done with the current target query.04:35
RAOFIs that query the one from the Workflow wiki?04:36
persiaYep.04:36
persia4:00 UTC on https://wiki.ubuntu.com/PatchDay !  Only 133 bugs left to review.05:06
* ajmitch imagines that many of the easy ones disappear first05:08
persiaheh05:10
persiaAnyone have an opinion about whether the patch in bug #570908 is a reasonable solution?  makes it unbootstrappable, but it's already bootstrapped, and the changelog can usefully hint how to rebootstrap.05:56
ubot3Malone bug 570908 in vala "Unable to build for PPA when there are patches to vala source" [Undecided,New] https://launchpad.net/bugs/57090805:56
* ajmitch had looked at that, but didn't know what the ramifications of the change would be05:59
persia5:00 UTC, and we're down to 130 bugs, meaning we're 29.8% complete!05:59
persiaajmitch: Yeah.  Looks to me like it's just including the source as well as the built stuff as references, but I'm not sure it belongs upstream: feels like a distribution-bootstrapping-workaround thing to me.06:00
ajmitchworking around the fact that we only have sourceful uploads06:00
ajmitchso not even needed for debian06:00
ajmitch(depending on whether they start throwing away binary uploads)06:01
persiaOh, right.  Ubuntu-specific indeed.06:01
persiaActually, might be needed for Debian, as valac is Arch: any06:02
persia(unless someone constructs an N-architecture .changes file, but that's just madness)06:02
ajmitchwhat, like binary porters do?06:02
ajmitchask lool about it, he's in Uploaders06:03
persiaGood idea :)06:04
ajmitchthough it looks like slomo does most of the vala uploads in debian06:04
persiaI haven't seen slomo around in Ubuntu channels in ages though.06:05
ajmitchnor have I06:05
ajmitchtime for me to get a bus home, I'll be back for more patching action soon06:08
persiaHey NCommander06:09
NCommanderhey persia06:09
persiaIt's https://wiki.ubuntu.com/PatchDay !  We're following https://wiki.ubuntu.com/ReviewersTeam/ReviewGuide#Workflow on submitted patches.06:09
persiaAbout 30% done with our current target, but many hours to go.06:10
micahgpersia: didn't we decide that apport bugs are reported in Ubuntu only or did that change?06:12
persiaOh, my mistake.  I missed that decision.  Feel free to Invalidate the usptream task :)06:13
* micahg searches for email06:13
NCommanderpersia: who's the best person for pointing someone at creating QA tools?06:16
micahgpersia: idr where it is, maybe double check with hggdh in a few hours06:16
persiabug #56979206:17
ubot3Malone bug 569792 in apport "wish, alternative available for apport-bug" [Wishlist,New] https://launchpad.net/bugs/56979206:17
* NCommander notes the retracer explodes if there's an upstream task attached06:17
micahgno, that I know :)06:17
NCommanderwell06:17
NCommanderLP explodes06:17
NCommanderMakes a nice traceback06:17
persiaNCommander: Hrm?  More detail is required, but you might want to ask in #ubuntu-quality to hit the right audience.06:17
persiaHrm.  We're probably going to get in trouble for using ubuntu-qa@lists.ubuntu.com at some point.  That's *supposed* to be the Qatar LoCo list.06:31
NCommanderpersia: maybe move it to ubuntu-quality-assurance, and then give Qatar ubuntu-qa, or maybe just have qatar be ubuntu-loco-qa?06:32
persiaDunno.  Not a battle I choose to fight now.  When the Qatar LoCo team gets active, something will happen.  Anyway, I've unsubscribed the reviewers from 55257506:33
NCommanderpersia: hrm, I can't unsubscribe the reviews team per the wiki.06:33
NCommanderpersia: can you do that with https://bugs.edge.launchpad.net/ubuntu/+source/klibc/+bug/527720 as well? I assigned myself to it as I have a personal interest in fixing it so I'll handle follow ups once the patch is tested06:33
ubot3Malone bug 527720 in klibc "thumb2 porting issues identified: klibc uses mov.*pc" [High,Won't fix]06:33
NCommander...06:33
NCommanderugh06:34
NCommanderI hate when that happens06:34
persiaSure.06:34
NCommanderpersia: https://bugs.edge.launchpad.net/ubuntu/+source/mountall/+bug/537133 - here's another one. mountall is Ubuntu-specific (no separate upstream, no patch to forward)06:36
ubot3Malone bug 537133 in mountall "mountall issues with NFS root filesystem" [Medium,Confirmed]06:36
persiaAnd here some of us thought all the easy ones were done already :)06:37
NCommanderpersia: https://bugs.edge.launchpad.net/ubuntu/+source/software-center/+bug/433838 - another one we can remove, Ubuntu specific, no separate upstream06:38
ubot3Malone bug 433838 in software-center "Use an icon in the location bar" [Low,New]06:38
persianigelbabu: What do you think about thse cases: should they get patch-forwarded-upstream?06:40
macohi06:44
persiaHey maco!06:44
macoim looking at the bugs that are tagged as having patches accepted upstream and going "oh look thingies to sponsor!"06:44
persiaIt's https://wiki.ubuntu.com/PatchDay ! We're following https://wiki.ubuntu.com/ReviewersTeam/ReviewGuide#Workflow on submitted patches.06:44
persiaheh.  That works too :)06:44
macobut umm... is maverick open yet?06:45
persiaAlthough most of them would probably be better served by merge/sync at this pont.06:45
macothere's a merge that needs sponsored06:45
persiahttps://launchpad.net/ubuntu/maverick/+queue?queue_state=1 says you can upload, but it will wait until later to be processed.06:45
persiaThat will be empty from archive-open to archive-freeze06:46
macoyeah i looked at the https://wiki.ubuntu.com/ReviewersTeam/GettingInvolved and clicked the "bug query" link06:46
macoempty?06:46
persiaYes.  UNAPPROVED only ever contains anything when the archive is frozen.06:46
persiaIf the archive is unfrozen, apprvoal is automatic, so it goes to NEW or ACCEPTED directly.06:47
* NCommander fires up a Debian chroot to test a bug06:47
NCommanderpersia: where's the handy forward to BTS script?06:47
macoahh ok06:47
persiaNCommander: submittodebian?06:48
* maco waits for maverick pbuilder to come into existence06:48
macohmm intrepid's gone eh? guess i can nuke that one06:49
macooh apparently i already did06:49
persiaalways good to plan in advance :)06:49
macopersia: are you one of the sru team people?06:50
persiaI'm not.06:51
macomy brain seems unwilling to remember who's on which special teams beyond jdong is on sru team, scottk & riddell on release team, and cjwatson seemingly having every magical power available06:51
persiahttps://launchpad.net/~ubuntu-sru/+members06:52
persiahttps://launchpad.net/~ubuntu-release/+members06:52
persiaetc.06:52
porthosepersia, would you please add me to the Ubuntu-sponsors team :)06:52
persia(this is why we have a team management interface :p )06:52
persiaporthose: Sure.06:52
porthosepersia, thx06:53
macoyeah i know :P ive looked at the sru team member list twice today. cjwatson & jdong were still the only ones that stuck06:53
macoand jdong sticks because ive been using #ubuntuforums and #ubuntuforums-mods to poke him on sru stuff for a long while :P06:54
NCommanderpersia: I thought we just had one for forwarding something to the BTS without a package06:54
NCommanderpersia: basically a bug CCer06:54
persiaI don't know of any.  The BTS doesn't like not having a package, version, etc.06:54
macoso pitti, slangasek, the 2 i know, and 2 i dont know. k.06:54
* ajmitch returns, is the list empty yet?06:55
NCommanderpersia: fair enough. You can unsubscribe https://bugs.edge.launchpad.net/ubuntu/+source/distcc/+bug/51158506:55
ubot3Malone bug 511585 in distcc "Unable to start pump server because of python version mismatch" [Medium,Confirmed]06:55
maco125?06:57
macois that down from the 1800?06:57
macoor is this a subset?06:57
ajmitchsubset06:57
persiaIt's a subset.06:57
macooh ok06:57
persiaDown from 185 when patch day started though, so good progress.06:57
persiaIt's specifially all the new patches that haven't been reviewed since a magic date.06:57
macoi will poke akk with a pointy stick for her to help out06:57
persiaif we run out, we'll move the magic date :)06:58
persiamaco: Thanks!06:58
macoshe's :( about bugs not getting fixed and i told her this is how she could help06:58
ajmitchif we run out, we've done something right06:58
persiaIndeed.  Getting the patches to the folks that can use them is key to getting the bugs fixed.06:58
ajmitcheven if sometimes it's us wearing a different hat06:58
macoajmitch: well part of this is getting the patches to upstream too06:59
macobecause we likely dont want to hold the deltas forever06:59
persiaThe goal of the team is to get the patches to people who can use them.  If people who can use patches join the team and get the patches directly, that also works :)06:59
ajmitchmaco: I know, I'm talking about wearing a debian hat & fixing stuff there, while passing onto upstream :)06:59
macoajmitch: oooh debian hat, i see.07:00
macovery spirally, looks nice on you ;-)07:00
ajmitchgoes with the debian tartan, you know07:00
macoi saw they were getting more debian tartan for debconf this year07:01
macobut the options are kilt or necktie07:01
* ajmitch lives in a city where kilts are sometimes seen on the streets07:01
persiamaco: Kinda, but most of it is about getting the patches *somewhere*: there's too many patches that get missed because nobody understands, etc.07:01
macopretty sure i'd be considered a cross dresser if i wore either of those :P07:01
macopersia: taught akk a couple weeks ago about the magic of the sponsor queue07:02
persiamaco: It's a matter of culture.  Neckties are perfectly acceptable female attire here.07:02
macopersia: sailor suits!07:02
ajmitchkilts are worn as part of girl's school uniforms here, funnily enough07:02
persiaIs one extreme example.07:02
persiahere also.07:02
ajmitchthe joys of living in a city settled by scots :)07:03
macoajmitch: catholic school i went to had kilts in grades 4-8 for girls....or a skort... but they weren't tartan, they were solid navy07:03
persiakilt+necktie+shirt+optional sweater07:03
* ajmitch thinks we're almost off-topic here :)07:04
macoyep07:04
persiaOnly 125 left.  Nearly one-third done!07:04
persiaUnfortunately, I've run against a time-wall, and was scheduled for lots more hours as contact than I could do today (and wasn't able to rearrange things).07:06
persiaIf someone else could answer any questions from new folks for a while, I'd appreciate it.  Extra points if you want to fill some of the blanks at https://wiki.ubuntu.com/PatchDay07:06
persiaAlso, if someone wouldn't mind posting the counts hourly or so: it would be nice to make a graph showing how well we did when PatchDay ends.07:07
macoif launchpadlib wasnt horribly scary, id suggest scripting that and running it hourly07:08
persiaThere is a script.  It doesn't complete within an hour :)07:08
macoO_o07:08
persiaThe script does some other stuff too though, so maybe it's overcomplex :)07:09
macostill 12508:02
dholbachgood morning08:08
ajmitchmorning dholbach08:17
dholbachhey ajmitch08:17
nigelbabuHello folks, second day of patch day today!10:31
nigelbabuWe have 122 open and unreviewed bugs in review queue waiting for review :)10:32
nigelbabuwe need to deal with situation where we're going to go against upstream decisions10:56
=== yofel_ is now known as yofel
nigelbhyperair: you're up next right?12:19
=== nigelb is now known as Guest54184
hyperairyeah12:19
Guest54184sorry, I'm out of power at home12:19
dholbachnigelbabu: how did the session go?12:20
Guest54184dholbach: it was nice, finished fast and too lots of questions12:20
=== Guest54184 is now known as nigelb
nigelbnow, thats better12:21
nigelbdholbach: send most of the development doubts to you ;)12:21
nigelbmaco helped me with questions and thankfully the storm was out before the session12:22
dholbachawesome12:22
nigelbhyperair: do you have the number of bugs handy?12:23
hyperairnigelb: what number?12:23
nigelbhyperair: number of unreviewed bugs12:23
nigelbbdmurray: graphs a bit stale?12:25
nigelbmaco: launchpadlib is not *that* scary12:25
nigelbwell, here we go 121  bugs12:25
nigelbdholbach: thanks for the blog post yday ( I just read it today), perhaps I need to blog today about how much progress we made in 2 days12:40
dholbachnigelb: definitely12:40
hyperairnigelb: no i don't have a number. is it needed?12:45
nigelbhyperair: just post every hour, so we can look at the logs and make a graph for later12:45
hyperairokay12:45
nigelbhelps identify fast hours and slow hours12:45
* nigelb adds blogging to list of things to do12:46
hyperairlooks like my hours are going to be slow >_>12:47
hyperairlaunchpad hates me12:47
hyperairgive it a query, wait 15 minutes for it to load...12:47
hyperairor it just won't load at all.12:48
nigelbouch, give me 15 mins, I'll check back home whts the situation12:50
nigelbif power is back, I'll do the posting for ya12:50
hyperairbah, i can't even get a list of bugs out13:00
hyperair=.="13:00
hyperairokay, it seems there are 121 outstanding13:17
persianigelbabu: How do you mean when we're going against upstream decisions?13:21
nigelbabupersia: no, sorry.  my mistake13:53
hyperairpersia, nigelbabu: what do i do when i encounter a patch that is not relevant? i.e. it works around the issue somewhere else rather than fixing it in the problematic app14:07
persiaexample?14:08
hyperairhttps://bugs.launchpad.net/ubuntu/+source/gnome-themes/+bug/54224014:09
ubot3Malone bug 542240 in gnome-themes "Icons clipped in the indicator applet" [Undecided,New]14:09
hyperairthe patch patches the theme, but i think the issue is in indicator-applet14:09
hyperairas mentioned by the comment i just posted14:09
persiaI'd say "patch-needswork" for that, if you're comfortable saying it's the wrong way to solve the issue.14:10
persiaIf you're unsure, pass it to upstream for a declaration.14:11
nigelbabuI hate a potential patch-rejected bug, but I fear backlash14:11
nigelbabus/hate/have14:11
hyperairheh14:11
hyperairi know that feeling14:11
persiaThat's the nice thing about patch-needswork: it invites discussion.  patch-rejected should only be used when it's confirmed wrong, and there's some other way to do it in process.14:12
hyperair(almost)hourly report: 115 bugs..14:13
nigelbabuThe bug is actually a Won't Fix thing14:13
nigelbabuso we took the middle path, marked it triaged and reported upstream14:13
hyperairlol14:13
nigelbabuupstream removed a feature and people want it back and naturally blame us14:13
hyperairi once had the misfortune of having to argue with someone about a graphical application logging information on the terminal.14:14
nigelbabuouch14:16
nigelbabuI like upstream gnome devs attitude on bug trackers14:16
nigelbabuthe moment you talk about offtopic or blame game on bugs, they will warn you, quite sternly14:16
nigelbabuwhen was the last time we spammed -bugs and -motu?14:17
hyperairheheh14:18
nigelbabuabout time, wanna do the honors hyperair ? ;)14:18
hyperairnigelbabu: what honours?14:19
hyperairi did it 5 minutes ago.14:19
hyperairthe hourly report thing14:19
nigelbabuooh, nice :)14:19
nigelbabuno no, we just pop by and advertise in the -motu and -bugs channels every 4 hours14:19
hyperairo rly14:19
hyperairwhat do i advertise?14:19
nigelbabuJust a reminder, it's Patch Day. Anyone with some time to review patches and help get them in the right places is encouraged to stop by #ubuntu-reviews and help out.14:19
* hyperair copypastas14:20
* nigelbabu dents and tweets and facebooks14:20
hyperairthere we go.14:20
nigelbabuheya geser, Nicke14:21
Nickehello14:22
nigelbabuIt's https://wiki.ubuntu.com/PatchDay  ! We're following https://wiki.ubuntu.com/ReviewersTeam/ReviewGuide#Workflow on submitted patches.14:22
micahgpersia: security team picked up the apparmor bug this morning :)14:50
persiaGreat!14:52
nigelbabuamazing15:02
hyperair112 bugs..15:05
nigelbabuooh, 70 bugs done :)15:06
persia73!15:06
* nigelbabu fails in subtraction15:06
persiaWhich makes us 40% done.15:06
hyperair\o/15:07
persiaWith 21 hours to go, we're still roughly on-target if we can maintain a rate of 6 patches reviewed an hour.15:07
hyperaircool15:07
persiaWhich is only one every ten minutes.  We can do this!15:07
dholbachpersia: how many did you do already? :)15:09
persiaI didn't count.  10 or so.15:09
* dholbach did a few during patch day lite, but just one during patch day15:09
persiaThere's plenty more :)15:09
dholbachyeah15:10
persiaAnd not just the hard ones.  ajmitch and I had thought the easy ones were gone, and NCommander stopped by and processed 3-4 with just bug tag adjustment.15:10
persiaSo I suspect there are other ones that are being well-managed, and just not using our procedure (and which therefore show up as "needing review")15:11
nigelbabuA couple of easy bugs if soeone wants to have a go15:15
nigelbabubug 568402, bug 54034215:15
ubot3Malone bug 568402 in qt4-x11 "Qt displays half width character as full width in some Japanese fonts" [Undecided,New] https://launchpad.net/bugs/56840215:15
ubot3Malone bug 540342 in lintian "Lintian strings scripts cannot handle spaces in path names" [Undecided,New] https://launchpad.net/bugs/54034215:15
* hyperair would be faster if only launchpad responded better15:16
yofelhm, about bug 520919: that's a packaging change request that's waiting for a r-base MIR, and it's already assigned to nixternal, here's no upstream involvement at all15:16
ubot3Malone bug 520919 in kdeedu "Cantor is missing possibility to add R backend" [Wishlist,Triaged] https://launchpad.net/bugs/52091915:17
* persia knocks out 56840215:17
nigelbabuI think the HIG complaiancy patches are okay to get in15:19
nigelbabuJust making the text complaint with upstream HIG, like bug 55521315:20
ubot3Malone bug 555213 in hundredpapercuts "Make printer jobs viewer HIG compliant" [Low,Confirmed] https://launchpad.net/bugs/55521315:20
nigelbabuthoughts?15:20
persiaI used to think that, but seb128 pointed out that 99% of the time upstream was happy to take them, and it's painful to carry the divergence.15:20
persiaSo I'd recommend pushing those upstream, rather than seeking to get them into Ubuntu.15:20
nigelbabuok, so anyone wats an easy bug, piack that one, just send upstream :)15:21
nigelbabuAlso, someone can just forward bug 491940 upstream15:23
ubot3Malone bug 491940 in ltsp "Patch for LTSP clients to properly reboot/shutdown" [Undecided,Confirmed] https://launchpad.net/bugs/49194015:23
nigelbabuso many easy bugs waiting to be picked15:23
bdrunghyperair, nigelbabu: i have to do some work. it would be nice if someone could take the responsibility for the 1700 UTC slot to give me one more hour.15:44
hyperair17 + 8 = 25..15:45
hyperairsure no problem15:45
henne91hi15:47
hyperairhello henne91.15:48
henne91ok I'm pretty new to this15:49
bdrunghyperair: thanks. your day has 25 hours? ;)15:49
nigelbabubdrung: ouch, I have a very early slot15:49
nigelbabuoh well, the rockstar is here :)15:49
nigelbabuwoo hyperair !15:49
hyperairbdrung: it fluctuates between 12 and 48 ;-)15:50
persiahenne91: So we're following https://wiki.ubuntu.com/ReviewersTeam/ReviewGuide#Workflow : just ask if you have any questions, or need an opinion or a hand.15:51
henne91persia: ok thx I'll take a look at that15:51
nigelbabuhyperair: lol15:52
nigelbabu108 bug now :)15:53
persia\o/15:53
persianigelbabu: You'll post a pretty graph to your blog when Patch Day ends, right?15:53
nigelbabuyes :)15:55
nigelbabuoh yeah, blog post to write!15:55
persiaBetter hurry: the 6th of May started 5 hours back, and is moving around the planet at a good clip :)15:57
henne91need to go16:17
nigelbabuhttp://justanothertriager.wordpress.com/2010/05/05/flash-news/16:27
nigelbabudholbach, persia: ^16:27
dholbachnice16:28
dholbachwell done16:28
nigelbabufolks, 8 more bugs and we enter into double digits16:34
qensenigelbabu: There seems to be an inconsistency at <https://wiki.ubuntu.com/ReviewersTeam/ReviewGuide>, the documentation talks about the tags 'patch-forward-upstream' and 'patch-forwarded-upstream'16:39
nigelbabuqense: can correct to forwarded please?16:39
qenseI can do that, yes!16:39
nigelbabuthank you :)16:39
=== yofel_ is now known as yofel
macothis is akk ^17:25
macoakk: this is what we're doing today https://wiki.ubuntu.com/PatchDay17:26
maconobody's announced the count in a few hours. 10417:27
macoakk: 185 bugs were targetted to be reviewed today17:27
macoakk: https://wiki.ubuntu.com/ReviewersTeam/ReviewGuide  <-- thats the step-by-step17:28
akkCool! Is there any synchronization here, like "Is anyone working on bug 12345 already? I want to take it"?17:29
ubot3Malone bug 12345 in isdnutils "isdn does not work, fritz avm (pnp?)" [Medium,Fix released] https://launchpad.net/bugs/1234517:29
akksorry, ubot3 :)17:30
macoheh17:30
macoi'd say pick randomly from the center of the list17:30
macoapparently they set a "magic date" and said "ok we're going to try to get through all patches submitted since $date" and tht turned out to be 18517:31
macojames_w: ping17:31
james_whey maco17:32
macojames_w: can you have lp:debian/sid/meld resync to match current debian? ive got a merge to do17:32
akkCurious what the date was ... I don't see a date in the query there.17:32
james_wmaco: not easily, as LP is currently prevented from importing source packages to its mirror of Debian17:33
james_wyou can see that https://edge.launchpad.net/debian/+source/meld/+publishinghistory is out of date17:34
macojames_w: why?17:34
macoyes i see that its way back at september17:34
macowhat's wrong with lp?17:34
james_wthere's some bug17:34
james_wI don't know17:34
macooh :(17:34
james_wthey are supposed to be looking in to it17:34
james_wI'm trying a workaround17:34
macoi guess as a workaround i could bzr init inside the unpacked debian source package, push that to my-namespace spot in lp, and do the merge that way17:36
macojames_w: would that work? im sure if i knew how to manually merge i could do that too, but...17:38
james_wgive me a few minutes17:38
akkDarn, my gmemusage patch was a year ago so it's not on the list.17:39
macoapparently not17:39
macoakk: you can do something with it anyway :P17:40
akkI probably will start with that. :)17:40
* maco glad crimsun's not in here17:40
macohe'd be lecturing me for not knowing how to do a merge without bzr or MoM17:41
* akk is still wondering when that gimp patch will get pushed into lucid17:41
macoakk: the statusbar one? its in lucid-proposed already17:41
akkRight, but then there was more bug activity that looked like it was moving into actual lucid ... only it didn't. I'm confused what to look for in bug updates.17:41
macopackages spend 2 weeks (i think?) in -proposed so breakage can be caught, then they go to -updates17:41
akkAh! Thanks, I wasn't clear on that.17:42
macosays it hit updates 11 hours ago17:42
macohttps://edge.launchpad.net/ubuntu/+source/gimp17:42
macothe really *awesome* bit about my inability to understand how to merge this package is that the program in the package is a tool for merging17:45
akkheh17:46
james_wmaco: try now17:47
akkNot just merge as in diff -p1 <goo.diff, I assume?17:47
* maco hugs james_w17:48
macoakk: 3-way merge17:49
macoupstream, ubuntu changes, and debian changes17:49
akkOh, yeah, I don't really know how to do that either. Maybe when you fix this package I can use it. :)17:49
macooh i dont know if that package does it, just that that's what i was trying to do17:50
macojames_w: thank you :)17:54
james_wglad it worked17:54
macobzr's being a little bit ridiculous about this merging O_o instead of the >>>>>> ===== <<<<<< lines surrounding just the few lines that changed, it put the entire old file and the entire new file in17:55
maco(if it couldnt resolve something. if it could it was yay)17:55
james_whumm18:03
james_wodd18:03
* hyperair wonders if bdrung has returned18:36
bdrunghyperair: i am back18:50
hyperairyay bdrung19:46
* hyperair wonders where util-linux's upstraem is19:47
hyperairdo they have a bug tracker i can stick patches into?19:47
bdrunghyperair: LKML?19:48
hyperairbdrung: that place sends shivers down my spine ._.19:52
bdrunghttp://www.kernel.org/pub/linux/utils/util-linux/19:53
hyperairhmm19:54
hyperairbut kernel has bugzilla.kernel.org19:55
hyperairoh hey maybe util-linux will be there also..19:55
macojames_w: hello?20:45
macoO_O20:48
macooh. wrong lp page20:49
macodavidek: hi! we're having a patch day today20:49
macohere's the workflow for what we're doing today https://wiki.ubuntu.com/ReviewersTeam/ReviewGuide and the list of bugs to look at is https://bugs.edge.launchpad.net/ubuntu/+bugs?field.subscriber=ubuntu-reviewers&field.tag=patch%20-patch-needswork%20-patch-forwarded-upstream%20-patch-forwarded-debian%20-patch-accepted-upstream%20-patch-accepted-debian%20-patch-rejected-upstream%20-patch-rejected-debian%20-patch-rejected&field.tags_combinator=ALL20:50
davidekhello! that's the reason, i'm here.20:50
maco(goodness that url is looooong)20:54
* hyperair kicks sbuild20:54
hyperairthat's why you're supposed to visit the reviewersteam wiki and just click the link20:54
macoheh21:00
macohey hyperair, how long does it take for lp to notice when you dput something?21:00
bdrungmaco: not long (few minutes)21:01
macook21:01
macoand does the automatically-mark-bug-fixed thing happen when the package hits proposed or updates?21:01
bdrungIIRC when hitting updates21:02
macook21:02
macodavidek: got any questions?21:02
macoor did whomever wrote the wiki page do a good job?21:03
hyperairmaco: 3 minutes or so i think.21:03
bdrungmaco: you should mark dputted SRUs as 'fix committed' because the upload will stuck it the queue and manually accepted21:03
macohmm s/whomever/whoever/21:03
hyperairmaco: updates.21:03
davidekmaco: not yet21:04
macobdrung: same for things uploaded to maverick while frozen?21:04
macowell then again i did commit to bzr so that sounds valid... but i wonder if debcommit means itll mark that automagically21:04
bdrungmaco: yes21:04
macoim gonna wait a few minutes and see if the debcommit/bzr mark-uploaded/bzr push combination does it automagic21:05
bdrungif i am right, mark-uploaded just adds a tag21:06
* maco pokes bzr-builddeb with a Hobbsee's pointy stick21:16
maco-S -- -sa? grr21:17
hyperairlol21:17
hyperairi use aliases21:17
macoreading bzr-builddeb --help i thought the --split thing was gonna do it for me magic-like, but nope21:18
* maco goes to add note to wiki page21:18
hyperairbzr always gives yyou that impression.21:18
hyperairuse git!21:18
imbrandonmaco: lol@Hobbsee's stick, havent seen that thing fly arround in a while21:30
ajmitchmorning21:31
imbrandonheya ajmitch21:33
* keffie_jayx missed hobbsee's stick of doom :(21:40
bdrungmaco: what's  Hobbsee's pointy stick?21:56
macobdrung: a stick (that i have the impression she's sharpened) that she uses to poke people/things on irc21:57
bdrungough, that probably hurts21:57
* ajmitch hasn't really seen hobbsee around for awhile22:02
macoajmitch: i think school's been kicking her22:08
ajmitchquite probably22:10
ajmitchshe needs to kick back22:10
* nigelbabu is here!22:11
nigelbabuaw no, 1 hour early22:12
bdrungnigelbabu: but you are here, right?22:14
nigelbabuyes :)22:14
nigelbabu@now22:14
bdrungnigelbabu: then i can take a shower and you will be there for answering questions22:16
nigelbabubdrung: hehe, go ahead :)22:17
bdrungnigelbabu: it has been quite quiet the last hours (= not much to do for me)22:18
* ajmitch will try & ask some annoying questions then22:19
nigelbabuhahah22:23
nigelbabuyou'll and up answering yourself22:23
nigelbabus/and/end22:23
nigelbabuits 3 am here22:23
ajmitch9:24 here :)22:24
macothe count is currently 10322:24
macoi think akk worked on forwarding patches she's submitted to lp that weren't on today's list22:25
macospeak of the devil22:25
akkSorry, busy day.22:26
nigelbabudid we go below 100 at any time?22:26
* ajmitch spots one to unsubscribe22:27
maconigelbabu: i dont think so22:30
macoi spent a /lot/ of time doing a fairly simple merge because i kept forgetting how to use things like bzr and bzr-buildpackage22:30
macojames_w got me straightened out and then i fixed the wikipage22:31
ajmitchmaco: what needed changed?22:31
ajmitch& yeah, LP not updating its debian info is problematic for merging22:32
macoajmitch: i got to the end of the "how to do a merge" and went "ok...so i push to lp and then...? how do i upload" so yay for the uploading page, added a link to that... and then added a bit thats like oh by the way, you need bzr-buildpackage -S -- -sa -vFOO22:32
macowhen doing a merge from debian22:32
ajmitchwhich page was this on?22:33
* ajmitch needs to refresh his knowledge of this stuff :)22:33
macohttps://wiki.ubuntu.com/DistributedDevelopment/Documentation/Merging22:33
macoi learned about bzr mark-uploaded today22:34
ajmitchtoo much magic :)22:34
macojames_w: idea for bzr mark-uploaded: change bug status to "fix committed"22:36
kklimondaheh, every time I try using bzr magic to maintain some package the results are terrible and I go back to the packaging branches..22:36
macojames_w: actually... no this is more of a lp thing. lp should go "oh look thingy in queue! *fix committed*"22:37
ajmitchI think that's in progress now22:37
nigelbabuyaay! 9922:37
ajmitchbug 31843922:39
ubot3Malone bug 318439 in launchpad-code "bzr --fixes lp:xxxx doesn't change the status of the bug" [High,Triaged] https://launchpad.net/bugs/31843922:39
nigelbabuit does22:40
nigelbabuajmitch: it makes it fix comitted22:40
ajmitchok, that's useful then22:41
* ajmitch thought that was still WIP, shows how much I use branches for packages22:41
nigelbabuwe are now at 96!22:42
* ajmitch hopes to see it at 0 by lunchtime22:43
nigelbabuanyone knows what eucalyptus is about?22:50
gesernigelbabu: does it work for all kind of branches or only for upstream branches or packaging branches?22:50
nigelbabugeser: ive seen it work for individual branches of a project22:51
nigelbabuI dunno for ubuntu development branches, lemme check22:51
ajmitchnigelbabu: eucalyptus implements something like amazon's EC2, it's what runs the cloud stuff in ubuntu22:51
nigelbabuajmitch: sh, patch in it, dunno what to do :/22:52
ajmitchsee if someone in the server team is on it22:54
nigelbabugeser: no, ubuntu development branches it doesnt happen22:54
nigelbabuajmitch: smoser?22:58
nigelbabuah yes, server team22:58
bdrungnigelbabu: we need a tag for "don't touch this patch, i can handle it myself"22:58
nigelbabubdrung: yeah we need one22:59
bdrungfor bugs like #36952523:00
ajmitchagreed, I've come across a few bugs like that23:00
nigelbabulol, i just played there didnt I?23:00
ajmitchyes23:00
ajmitchor bugs like bug 27647223:01
ubot3Malone bug 276472 in samba "cp -p on CIFS mount does not preserve permissions and returns a permission denied error" [Medium,In progress] https://launchpad.net/bugs/27647223:01
ajmitchhandled by a kernel team member23:01
nigelbabubdrung: easiest way, just unsubscribe review team without removing patcch tag23:01
ajmitchdown to 9423:02
nigelbabumake that 9323:02
bdrungnigelbabu: k, that would work, but then i have to wait until the script subscribed the review team23:03
nigelbabubdrung: if there is a patch tag, we won't be subscribed again23:05
bdrungnigelbabu: so adding the patch and adding the patch tag would work?23:06
ajmitchwhen poatches originate from upstream, I guess I should unsubscribe the review team?23:07
nigelbabubdrung: yep23:07
nigelbabuajmitch: patch-accepted-upstream?23:07
ajmitchvery much accepted, more patch-originated-upstream23:08
nigelbabuyeah, use that tag so peole who want to package can spot it23:08
ajmitchyeah I was just checking the debian changelog, fix is already in lucid (from debian)23:09
ajmitchit's a debdiff for karmic-proposed23:09
nigelbabuoh, SRU23:09
ajmitchyep, and it's hardly a critical one23:09
nigelbabuI'dsay poke someone in sru team to accept or reject23:10
ajmitchyes, I was about to23:10
nigelbabuajmitch: always a step ahead :)23:10
ajmitchspecifically jdong, since the bug/patch submitter is from someone he knows, and he's subscribed23:11
nigelbabuoh yaay! 91 bugs to go!23:11
nigelbabubdmurray: are the graphs a bit stale? is it because you're working on updating the script?23:13
nigelbabu89! lets move it folks :)23:18
bdmurraynigelbabu: can you show me what you mean by stale?23:20
nigelbabuhttp://qa.ubuntu.com/reports/patches/23:23
nigelbabubdmurray: last update date is 04/2823:23
nigelbabuajmitch: you folks were talking about the vala bug yesterday?23:27
nigelbabuwhat was the final call on that one?23:27
ajmitchI was talking with persia about it, I don't know if he talked to lool23:34
nigelbabuhm23:35
ajmitchI didn't follow up further23:35
* nigelbabu really wants to see 0!23:35
nigelbabupersia: can you followup on the vala bug? since I dunno what to ask ;)23:37
nigelbabu88 bugs, we're dropping constantly here :)23:44
yofelanother one done23:46
nigelbabuyaay!23:47
nigelbabuwhere is everyone? akk? maco?23:48
akkSorry, I'm lurking but I don't think I'll have time to do anything today.23:49
nigelbabuakk: ah, no problem :)23:49
akkAnd if I did I'd be awfully tempted to start with my year-old patch that's too old to be in your list.23:49
* ajmitch can't really do much more for a little while23:51
* ajmitch wishes he'd got stuff into karmic-proposed long before lucid release23:52
ajmitchhaving people still available to test stuff would be nice23:52
nigelbabuakk: you can get it into our list23:53
nigelbabuajmitch: yeah, definitely23:53

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