[07:53] <dholbach> good morning
[07:55] <nigelbabu> morning dholbach
[07:55] <nigelbabu> if you notice, the numbers are less than 1700 today
[07:55] <dholbach> nigelbabu: I just had the thought that we should probably try to avoid to create the perception that "all the patches were integrated"
[07:55] <nigelbabu> I did some review last night
[07:55] <nigelbabu> didn't get you
[07:55] <nigelbabu> you mean the feeling that if something is not in the review queue, its integrated?
[07:55] <dholbach> nigelbabu: eh?
[07:56] <dholbach> instead I think we should say in bold letters XYZ patches were reviewed, and then list individually how many patches are forwarded to upstream, to debian, etc etc
[07:56] <dholbach> if you see what I mean
[07:57] <nigelbabu> yep, agreed.  Next time i ask someone to blog, I'll mention that
[07:58] <dholbach> I'll do it too
[07:58] <dholbach> nigelbabu: what do you think about picking a day each week or maybe every two weeks where we blog the "stats"?
[07:59] <nigelbabu> dholbach: hahaha, I already thought of doing that every monday with alternating folks doing it
[07:59] <nigelbabu> I think I even talked about it in the channel
[07:59] <nigelbabu> dholbach: Great minds think alike :D
[08:00] <dholbach> ok, I'll do monday then
[08:00] <dholbach> maybe we should also do the stats comparing with last week
[08:01] <dholbach> still in the queue: 1789 (-250)
[08:01] <dholbach> or something
[08:03] <nigelbabu> dholbach: yep, I'm also thinking of asking the UWN folks to publish patch stats (not of cleansweep though), just overall ubuntu patches
[08:04] <dholbach> that might work - with a link to the patch reviewers team
[08:05] <nigelbabu> yep, so we can ask to add a line "As always Bug Squad and Reviewers Team need help"
[08:05] <dholbach> yeah
[08:06] <dholbach> nigelbabu: you know what, I'll write a script that'll run every monday and it'll write up what changed since the last run
[08:07] <nigelbabu> dholbach: that works too.  if they can look at a page for stats like people.dholbach.com/patch-weekly-report.txt, would be great!
[08:07] <dholbach> that makes it easier to copy and paste
[08:07] <nigelbabu> You are so full of awesome :)
[08:07] <dholbach> I try :)
[08:08] <dholbach> nigelbabu: https://code.edge.launchpad.net/ubuntu-review-overview/+activereviews
[08:08] <dholbach> vish: did you manage to have a chat with daker?
[08:08] <nigelbabu> dholbach: oh yeah, forgot
[08:08] <dholbach> vish: about http://daniel.holba.ch/review/example.html and http://daniel.holba.ch/review/meter.css
[08:10] <nigelbabu> dholbach: can I say ack and will you merge it in? I'm at work and away from my ssh keys for another 7 hours
[08:10] <dholbach> sure
[08:11] <vish> dholbach: hi.. he was on here for a few mins , and this was what he had come up with > http://imagebin.org/100680
[08:11] <nigelbabu> dholbach: ok, ack :D
[08:11] <vish> dholbach: still needs work
[08:11] <vish> dholbach: also , i told him you prefer it to be a css
[08:11] <dholbach> thanks nigelbabu
[08:12] <vish> why is the world round :(
[08:12] <vish> timezones suck!
[08:12] <dholbach> vish: the problem is that we want people to just have html that doesn't change - we can't have them update the html with actual percentage values if you see what I mean
[08:13] <dholbach> vish: however it's done and which technology is used, I don't care much TBH :)
[08:14] <nigelbabu> dholbach: if you can give changing css, then it works
[08:14] <dholbach> that's what the current branch does
[08:14] <dholbach> if I need to put other changes in there, I'm happy to do that
[08:15] <nigelbabu> then tats all is needed I suppose, since the code in the html doesn't change.  what needs to be done,however, is to correct the html to be a snippet instead of a page
[08:15] <dholbach> vish: but http://imagebin.org/100680 looks GREAT
[08:15] <vish> dholbach: yeah. i'll try to get him here at the same time you are here , would be easier to get it done quick
[08:16] <dholbach> or we do it over mail
[08:16] <vish> dholbach: yeah , it looks nice , we could probably use a little less icons and add counter there
[08:16] <dholbach> dholbach at ubuntu dot com
[08:17] <vish> will send him that
[08:17]  * dholbach hugs you all
[09:00] <bobbo> morning all :)
[09:02] <ansgar> Good afternood :)
[09:05] <dholbach> heya bobbo
[09:06] <bobbo> hey dholbach :)
[09:07] <bobbo> dholbach, I wrote up a progress bog update, like Jono asked
[09:07] <dholbach> yeah, I saw it!
[09:08] <dholbach> good work!
[09:08] <ansgar> The first two days more patches per day have been reviewed than required to reach the goal of having all reviewed by the next Ubuntu release?
[09:08] <dholbach> bobbo: nigelb and I just talked about writing a small script that basically just gets stats and compares them with the stats since the last run, so we can easier keep track of what happened between weeks
[09:08] <dholbach> bobbo: and get that into UWN, etc.
[09:08] <ansgar> If you finish too soon, you could do the same for Debian's BTS ;-)
[09:09] <dholbach> haha
[09:10] <bobbo> dholbach, could it keep it's data in a csv file or similar then just compare it? That sounds do-able
[09:10] <dholbach> bobbo: that's exactly what I'm working on right now :)
[09:10]  * dholbach hugs bobbo
[09:11] <bobbo> dholbach, if you wanna offload it to someone else, I'd gladly help
[09:11] <dholbach> bobbo: I'll poke it just a bit longer and see where I can get it to
[09:12] <bobbo> dholbach, awesome, in that case, off to launchpad for more patches :D
[09:12] <dholbach> ROCK ON
[09:13] <bobbo> dholbach, if I temporarily disappear at any point, I'm not trying to avoid work, I'm getting an insane memory leak in policykit so have to reboot every now and then :)
[09:13] <dholbach> man, that's nuts
[09:13] <dholbach> you should have used maverick only in a VM! :)
[09:13] <dholbach> is there a bug report about it?
[09:14] <bobbo> dholbach, I should probably look for one
[09:16] <bobbo> dholbach, you're too sensible, first bug report I find has a workaround :D
[09:16] <dholbach> bobbo: I lived through a lot of development release pain already :)
[09:54] <dholbach> bobbo, nigelbabu: http://daniel.holba.ch/review/report
[09:55] <dholbach> and less interestingly http://daniel.holba.ch/review/stats.csv
[09:55] <dholbach> nigelbabu: it'll get updated every monday
[09:55] <bobbo> dholbach, that's awesome :D
[09:55] <dholbach> now we just need to tell the UWN team about it
[09:56]  * dholbach goes back to writing talks
[09:59] <dholbach> I'll update it on sunday, that's where the uwn is released
[10:11] <bobbo> woot, just uploaded a patch that went patch-needswork yesterday and the authors responded, the system works :D
[10:12]  * BlackZ hugs bobbo 
[10:26] <vish> @patchcount
[10:27] <vish> oh , no patchbot :p
[10:27] <bobbo> @patchcount
[10:27] <patchbot> bobbo: Total: 2289     Left in Queue: 2059     10.0% complete
[10:27] <bobbo> vish, ^
[10:28] <vish> bad patchbot ;p
[10:28] <hyperair> lol
[10:28] <bobbo> vish, haha, must have closed the terminal window he was running in
[10:29] <hyperair> why no daemonize it?
[10:42] <dholbach> bobbo: did you upload the patch?
[10:43] <bobbo> dholbach, yeah, but I'm forwarding it to Debian too, just a lot of the Debian bugs have been sitting there for years so I wasn't too hopeful for it getting uploaded any time soon
[10:43]  * dholbach nods
[10:43] <dholbach> good work
[10:43] <bobbo> dholbach, when is our training session starting?
[10:44] <dholbach> bobbo: 12 utc
[10:45] <ajmitch> hi people
[10:45] <bobbo> dholbach, okay, so that'll be 1pm my time, I can never get the hang of UTC :D
[10:45] <seb128> could somebody unscribed the reviewers from bug #588772?
[10:45] <ubot4> Launchpad bug 588772 in ubiquity (Ubuntu) "installer crashes when configuring user (affects: 1) (heat: 421)" [Undecided,New] https://launchpad.net/bugs/588772
[10:45] <seb128> unsubscribe
[10:45] <ajmitch> sure
[10:45] <seb128> thanks
[10:45] <dholbach> bobbo: date; date -u
[10:46]  * dholbach adds seb128 to ubuntu-reviewers
[10:46] <ajmitch> removed the subscription now
[10:46] <bobbo> dholbach, thanks :D
[10:46] <seb128> dholbach, yet another team for me now?! ;-)
[10:46]  * ajmitch just adds 12 :)
[10:46] <ajmitch> seb128: you needed another badge on LP
[10:48] <dholbach> seb128: ce n'est pas une équipe ennuyeuse, c'est L'ÉQUIPE
[10:49] <BlackZ> seb128: hey, welcome aboard!
[10:49]  * ajmitch finally expired from ubuntu-{main,universe}-sponsors
[10:49] <BlackZ> \o
[10:49] <seb128> lol
[10:49] <seb128> it's my fault, I should have learnt that every time you touch something around you get tricked into new work
[10:50] <seb128> or responsabilities
[10:50] <seb128> ;-)
[10:50] <ajmitch> especially with dholbach around
[10:50] <dholbach> ajmitch: no no, I make the same mistakes as seb128
[10:50] <BlackZ> lol
[10:50] <dholbach> still didn't learn my lesson either
[10:50] <nigelbabu> dholbach: interesting report!
[10:50] <nigelbabu> seb128: that's how I got sucked into reviewers team in the first place
[10:50] <ajmitch> dholbach: worse, I got convinced to change some code in LP earlier from a discussion in -motu
[10:51] <nigelbabu> made the mistake of saying "I'm bored" in #ubuntu-motu
[10:51] <BlackZ> dholbach: once I have finished to update the mentors documentation, do you want to check the docs for PTC?
[10:51] <seb128> ok, so bug #558841, what do I do with it?
[10:51] <ubot4> Launchpad bug 558841 in indicator-application (Ubuntu Lucid) (and 5 other projects) "bluetooth "devices" menu item not working in bluetooth indicator (affects: 13) (dups: 3) (heat: 78)" [Low,Triaged] https://launchpad.net/bugs/558841
[10:51] <seb128> the patch there is in maverick
[10:51] <dholbach> BlackZ: I'm afraid I don't have the time for that
[10:51] <BlackZ> dholbach: OK
[10:51] <dholbach> BlackZ: I'm not part of the mentoring team any more
[10:52] <seb128> but there is another task on the bug which is open
[10:52] <seb128> should I just drop the patch tag?
[10:52] <seb128> is somebody going to add it back because there is a patch on the bug?
[10:52] <nigelbabu> seb128: just unsubscribe
[10:52] <nigelbabu> if you remove the patch tag, we will get unsubscribed again
[10:52] <seb128> it's still tagged patch
[10:52] <seb128> but that's not really correct
[10:53] <seb128> since that patch has been uploaded
[10:53] <nigelbabu> true, but unsubscribing gets it out of review queue - our work pool
[10:53] <seb128> well, it will come back
[10:53] <nigelbabu> you can add a comment "patch in this bug is accepted, unsubscribing reviewers team"
[10:53] <seb128> when somebody notice it's a bug with a patch and reviewers are not subscribed
[10:54] <seb128> ok
[10:54] <ajmitch> depends if the mass-susbscription script was smart or not
[10:54] <nigelbabu> ajmitch: it is
[10:55] <nigelbabu> brian wrote it, dholbach and I have gone through it with a fine comb, but if you want, you can take a peek.  Its in the ubuntu-qa-tools project on LP
[10:55] <ajmitch> I'll pass, I trust that it'll DTRT :)
[10:56]  * ajmitch has enough other things to fix up
[10:56] <nigelbabu> if it doesn't, you can always propose a merge LD
[10:56] <nigelbabu> :D
[11:14] <bobbo> Bug #150205 <-- How do we deal with that bug?
[11:14] <ubot4> Launchpad bug 150205 in baltix (and 17 other projects) "Make menu items labels more consistent and clear (heat: 10)" [Undecided,Invalid] https://launchpad.net/bugs/150205
[11:15] <daker> hey dholbach http://imagebin.org/100680
[11:15] <dholbach> hey daker
[11:15] <dholbach> daker: yeah, vish showed it to me already - it's great
[11:15] <dholbach> daker: I'm happy to update the css file with the script and everything
[11:16] <dholbach> daker: is there a way so that people can just include it in a webpage and not update the html themselves?
[11:16] <dholbach> http://daniel.holba.ch/review/example.html is what I have right now
[11:16] <dholbach> and http://daniel.holba.ch/review/meter.css (which is automatically generated)
[11:17] <daker> yes there is
[11:18] <daker> give me a minutes
[11:18] <nigelbabu> daker: basically, can we have it as a snippet that people can put up on a website :)
[11:18] <daker> yes
[11:18] <dholbach> awesome
[11:18] <dholbach> take your time :)
[11:21] <daker> dholbach, creat a new .js file
[11:21] <daker> and paste that :
[11:21] <daker> var gadget =\'<style type="text/css" media="all">@import "meter.css";</style>\'+
[11:21] <daker> \'<div class="progress-bar"><div id="sample" class="bar">.</div>\'+
[11:21] <daker> document.write(gadget);
[11:22] <daker> then people have to just to put the js file any where they want like that
[11:23] <dholbach> daker: that's cool - can we get the text in there somehow too? :D
[11:23] <daker> <script type="text/javascript" src="http://daniel.holba.ch/review/widget.js"></script>
[11:24] <daker> you can test
[11:24] <nigelbabu> BlackZ: quite a storm you brewed there on debian-mentors.  Actually, there are a few ubuntu devs who are DDs whom you can poke
[11:25] <nigelbabu> daker: testing
[11:25] <ajmitch> and getting more by the day, it seems
[11:26] <dholbach> daker: http://imagebin.org/100680 has percentage - how do we get that into there? :)
[11:26] <nigelbabu> dholbach: did you create that js file yet?
[11:26] <dholbach> nigelbabu: no
[11:26] <BlackZ> nigelbabu: I don't think they have always the time to sponsor
[11:26] <nigelbabu> dholbach: figures, I get blank page here ;)
[11:26] <BlackZ> also, I know
[11:27] <nigelbabu> BlackZ: I don't remember who it is, but there a few who're always willing to sponsor
[11:27] <nigelbabu> dholbach: can poke me when you do, I'll test here
[11:27] <ajmitch> finding time to sponsor can be a challenge for most of us, I think
[11:27] <dholbach> nigelbabu: daker and I are still figuring something else out
[11:27] <BlackZ> heh ajmitch that's true
[11:27] <nigelbabu> ajmitch: I agree.  Finding time to package is a challenge, so sponsoring can be more time consuming
[11:27] <nigelbabu> dholbach: ah, ok
[11:28] <nigelbabu> dholbach: oh, the %! ok!
[11:28] <daker> dholbach, take a look a here https://code.launchpad.net/~adnane002/+junk/ubuntu-reviews
[11:29] <BlackZ> ajmitch: generally you want to sponsor packages which you're familiar, am I wrong?
[11:29] <dholbach> daker: ok, so I write the .js file like at the end of http://bazaar.launchpad.net/~adnane002/+junk/ubuntu-reviews/annotate/head:/gadget.js.php ?
[11:29] <ajmitch> BlackZ: generally yes, especially when there's specific bits of policy to be familiar with
[11:29] <dholbach> BlackZ: generally it's preferrable if sponsors sponsor stuff they feel comfortable with
[11:30] <daker> dholbach,  are you going to do all the stuff with python ?
[11:30] <dholbach> daker: yes
[11:30] <daker> why ? explain
[11:30] <BlackZ> dholbach: and I'm agree with that, the reason why I send that e-mail is because there are 20+ pending packages (but I mean just who requested sponsor on debian-mentors)
[11:31] <dholbach> daker: because it's very easy for me to use
[11:31] <daker> oki
[11:31] <dholbach> BlackZ: unfortunately that happens
[11:31] <dholbach> daker: is that a problem?
[11:31] <daker> no :)
[11:31] <dholbach> ok, good :)
[11:32] <dholbach> I thought I was missing something
[11:32] <BlackZ> dholbach: I don't want that as a flame as I said in the ml
[11:32] <BlackZ> some DDs thought so
[11:32] <BlackZ> (but I hope not)
[11:32] <dholbach> BlackZ: what are you suggesting should change?
[11:32] <daker> dholbach, file_total = 'http://daniel.holba.ch/review/total.txt';
[11:32] <daker> file_inqeue = 'http://daniel.holba.ch/review/in-the-queue.txt';
[11:32] <BlackZ> dholbach: http://lists.debian.org/debian-mentors/2010/06/msg00160.html
[11:33] <daker> dholbach,  you have to read the content of those files
[11:33] <dholbach> daker: I'll create the .js file from where those values are generated
[11:33] <dholbach> daker: that means I can make the .css static?
[11:34] <daker> no that your python code have to do that
[11:34] <dholbach> ok
[11:34] <dholbach> no problem :)
[11:34] <daker> you have also to regenerate the js file
[11:35] <dholbach> yep
[11:35] <dholbach> will do
[11:35] <dholbach> just a few mins
[11:36] <daker> the js file should like  that :
[11:36] <daker> var gadget =\'<style type="text/css" media="all">@import "meter.css";</style>\'+
[11:36] <daker> \'<div class="progress-bar"><div id="sample" class="bar">.</div>\'+
[11:36] <daker> \'<div id="percentage">the value of the percentage generated with python here %</div>\'+
[11:36] <daker> document.write(gadget);
[11:38] <daker> just to tell you that the widget i made is working & we don't have to write any
[11:38] <daker> just call the js file & it's done
[11:48] <dholbach> cool
[11:48] <dholbach> just a few more ticks
[11:51] <daker> oki
[12:04] <dholbach> daker: http://daniel.holba.ch/review/example.html - good?
[12:05] <dholbach> I wrote my changes to https://code.edge.launchpad.net/~dholbach/ubuntu-review-overview/js
[12:05] <dholbach> and will need to step out for a bit
[12:05] <dholbach> bobbo: I'll be back in time for our session
[12:05] <dholbach> bbl
[12:08] <bobbo> dholbach, okay, I'll go grab some lunch then :)
[12:26] <daker> hey dholbach
[12:26] <daker> open default.css
[12:27] <daker> and change all the "../images/" by "images/"
[12:38] <dholbach> daker: ah ok, will do - thanks
[12:39] <dholbach> daker:  you're a rockstar
[12:39] <dholbach> this is great!
[12:41] <dholbach> daker: beautiful
[12:41] <daker> Oh
[12:41] <dholbach> everybody: http://daniel.holba.ch/review/example.html
[12:41] <dholbach> look at the source of the .html - it's tiny
[12:41] <daker> everyone how want to put the widget on his website
[12:41] <daker> just put that :
[12:42] <daker> <script type="text/javascript" src="http://daniel.holba.ch/review/gadget.js"></script>
[12:42] <bobbo> that's awesome :D
[12:42] <dholbach> daker's the wizard
[12:43] <daker> thanks :)
[12:44] <bobbo> I can't get it to workon my site, I'm not getting any images ...
[12:44] <daker> put your site here
[12:44] <bobbo> maybe the urls in gadget.js have to be absolute so people can link to them
[12:44]  * dholbach just microblogged about the session in a few
[12:44] <bobbo> http://www.bobbo.me.uk <-- On the right hand side
[12:45] <daker> Oh i know :)
[12:45] <dholbach> Packaging Training Session in 15m in #ubuntu-classroom: Operation Cleansweep and the Patch Reviewers Team!
[12:45] <daker> dholbach, we should make some changes
[12:46] <dholbach> daker: ok… what do you have in mind?
[12:46] <daker> you should add everywhere : 'http://daniel.holba.ch/review'
[12:47] <daker> -----------------------------------------
[12:47] <dholbach> daker: do we need to hardcode everything?
[12:48] <daker> yes
[12:48] <daker> here is the js file:
[12:48] <daker> var gadget ='<link rel="stylesheet" href="http://daniel.holba.ch/review/default.css" type="text/css" media="screen">'+
[12:48] <daker> '<div id="badge">'+
[12:48] <daker> '<div id="ubuntu-reviewers-logo"></div>'+
[12:48] <daker> '<div id="ubuntu-logo"></div>'+
[12:48] <daker> '<div id="progress-bar"><div id="bar" style="width:10%">10%</div></div>'+
[12:49] <daker> '<div id="percentage">10%</div>'+
[12:49] <daker> '<div id="ubuntu-review-team"><img src="http://daniel.holba.ch/review/images/3.png"><a href="http://launchpad.net/ubuntu-review" target="_blank">Powered by Ubuntu Review Gadget</a></div>'+
[12:49] <daker> '</div>';
[12:49] <daker> document.write(gadget);
[12:49] <daker> and in the default.css you should replace all the "images/" by "http://daniel.holba.ch/review/images/"
[12:51] <daker> that's all :)
[12:51] <dholbach> that's unfortunate - I thought there was a way to keep it dynamic somehow
[12:52] <daker> ????!!!!
[12:53] <dholbach> so other folks could run the script too and use it on their sites
[12:53] <dholbach> but I have no clue, so I better shut up :)
[12:53] <kazade> dholbach, I've come in halfway through the conversation, but you could try replacing "images/" with "/review/images" to prevent depending on the hostname if I understand correctly
[12:53] <kazade> paths starting with a / are relative to the hostname...
[12:54] <kazade> ignore me if I've misunderstood what's going on :p
[12:54] <dholbach> kazade: I could try that
[12:54] <daker> dholbach, just do what i say then we will see :)
[12:54] <kazade> where is this js file located?
[12:55] <dholbach> kazade:
[12:55] <dholbach>  <script type="text/javascript" src="http://daniel.holba.ch/review/gadget.js"></script>
[12:56] <kazade> ok, so yeah, if you remove the "http://daniel.holba.ch" from every link and retain the leading slash, it will work on any host
[12:56] <daker> kazade, it will not work :)
[12:57] <kazade> how so?
[12:57] <daker> because we don't want people to host the files
[12:57] <kazade> oh right!
[12:57] <kazade> this is what I get for entering a conversation midway
[12:58] <dholbach> daker: why not? :)
[12:58] <kazade> yes, you are right, in that case you need absolute links
[12:58] <bobbo> dholbach, don't want to force people to host the files to show the gadget
[12:58] <dholbach> daker: if people host it when my machine is offline or somebody else runs the script as well I absolutely don't ming
[12:58] <dholbach> bobbo: sure
[12:58] <dholbach> whatever, I can hardcode it too
[12:58] <dholbach> no big deal
[12:59] <kazade> well... I guess you could publish the resources (images, js etc.) on #ubuntuone ... and link to those files...
[12:59] <dholbach> I just thought it'd be nice not to
[12:59] <kazade> then anyone can access it
[12:59] <bobbo> what about sticking a hostname var at the top so it's really easy to modify if anyone else wants to host it?
[12:59] <daker> kazade, in that case yes
[12:59] <dholbach> I'll think about it later on
[12:59] <dholbach> there's a session to be given now
[13:00] <daker> oki
[13:10] <nigelbabu> dholbach, bobbo: good going!
[13:11] <dholbach> bobbo's rocking
[13:11] <nigelbabu> dholbach: we should have a bobbo huggers team now :D
[13:11] <dholbach> totally
[13:15] <nigelbabu> dholbach: ouch, the how to fix docs I intended to write with the whole pbuilder etc is not yet ready :/ sorry.
[13:15] <dholbach> we need to integrate edit-patch in there
[13:15] <dholbach> now that bobbo fixed it up
[13:17] <nigelbabu> dholbach: we can rewrite the whole How to fix to gear towards us.  I'll do it tonight
[13:18] <dholbach> nigelbabu: I think we can probably have a general guide that suits us all
[13:18] <nigelbabu> Lets see what i can come up with :)
[13:24] <dholbach> nigelbabu: have a look at the HowToFix page first - I'm sure it's in there already and just need a few small fixes
[13:25] <dholbach> bobbo: we'll extend edit-patch to just apply the patch and created debian/applied-patches and put it there
[13:25] <bobbo> dholbach, awesome, I'll get that done after the session
[13:26] <dholbach> super
[13:59]  * bobbo waves at karyo 
[14:00] <karyo> how do I do that?
[14:00] <bobbo> "/me"
[14:00]  * karyo waves
[14:01]  * karyo thanks bobbo
[14:03] <nigelbabu> bobbo, dholbach: Awesome session!
[14:04] <dholbach> it's all bobbo's doing :)
[14:04] <nigelbabu> With that, the number of patches hit 1650!
[14:04] <nigelbabu> I should review some patches today to make it < 1600 today
[14:04] <bobbo> nigelb, 1600 is easily possible :P
[14:06] <bobbo> nigelbabu, if we can do it before I go to sleep I can stick it in my daily progress blog, that'd look pretty awesome
[14:06] <nigelbabu> nigelbabu: gimme a few hours to get home and start reviewing
[14:06] <nigelbabu> Also, you'll be around after you're 1 week in canonical too right?
[14:06] <bobbo> nigelbabu, should be, I'll be in school but I don't have any exams or anything so I'll have a ton of free time
[14:07] <nigelbabu> \o/
[14:08] <dholbach> log is up at: https://wiki.ubuntu.com/Packaging/Training/Logs
[14:09]  * dholbach now needs to take care of a few other things
[14:09] <dholbach> bbiab
[14:09] <nigelbabu> later folks, leaving for home
[14:19] <karyo> https://bugs.launchpad.net/ubuntu/+source/poppler/+bug/547225
[14:19] <ubot4> Launchpad bug 547225 in poppler (Ubuntu) (and 1 other project) "Evince fails to properly display this "type" of pdf (affects: 2) (heat: 16)" [Low,Fix committed]
[14:19] <karyo> what tag should I apply?
[14:20] <karyo> seems like the fix originates from upstream
[14:25] <\vish> !test
[14:25] <ubot4> hrm?
[14:31] <bobbo> argh why is it when I'm testing a feature that requires a patchsys I can never find a package that uses one and when I'm testing a feature that requires no patchsys, I can never find one without one
[15:13] <bobbo> dholbach, http://bazaar.launchpad.net/~ubuntu-dev/ubuntu-dev-tools/trunk/revision/657
[15:15] <dholbach> bobbo: awesome - that looks great - you should add a changelog entry
[15:15] <bobbo> dholbach, to debian/changelog?
[15:15] <dholbach> yep
[15:15] <bobbo> it's basically included by what mvo wrote on Monday
[15:15] <dholbach> ah ok
[15:15] <bobbo> he put "support adding existing patches (e.g. from launchpad)"
[15:16] <dholbach> is debian/applied-patches going to be used even if I don't use a patch system at all and don't have an existing patch?
[15:16] <bobbo> no, it's only touched when edit-patch is give an existing patch
[15:17] <dholbach> ah ok
[15:17] <dholbach> wonder if it'd make sense to also do it when there's no patch given
[15:17] <dholbach> probably not
[15:17] <dholbach> … although
[15:17] <dholbach> it might make sense if people care enough to add the patch tagging guidelines
[15:17] <dholbach> but anyway
[15:18] <dholbach> we should probably be good now :D
[15:18] <bobbo> yeah, that seems to be it working pretty well, there's probably some minor stuff I could sort but bash fries my brain :D
[15:18] <dholbach> yeah :)
[15:19] <dholbach> mvo chose it so we could maybe get it  into devscripts at some stage
[15:19] <dholbach> I started it in python
[15:19] <dholbach> and he did it all the legwork to get it into shell
[15:20] <bobbo> I've not worked in shell for a long time, I'm a total Python man at heart :D
[15:20] <dholbach> yeah
[15:20] <dholbach> everything else is a bit of a pain :)
[15:41] <daker> dholbach, i want to see the result :)
[15:42] <dholbach> daker: sorry, I was in what felt like 50000 conversations since we talked
[15:42] <dholbach> I'll have a look at it now
[15:45] <daker> :) oki take your time
[15:46] <dholbach> daker: I hardcoded all of them
[15:46] <dholbach> daker: bobbo.me.uk still doesn't work
[15:48] <daker> well there is something wrong here :
[15:48] <daker> i/* Ubuntu Reviews Gadget */
[15:48] <daker> remove the "i"
[15:49] <dholbach> ugh
[15:50] <dholbach> done, still broken :)
[15:51] <bobbo> I get the same results in a plain test HTML file, so it's not my stylesheets killing it
[15:53] <daker> no dholbach rehardcod all of them
[15:53] <bobbo> dholbach, http://pastebin.com/5nbcV8v6
[15:54] <bobbo> that's them all hardcoded and it works on my system
 var gadget ='<link rel="stylesheet" href="http://daniel.holba.ch/review/default.css" type="text/css" media="screen">'+
 '<div id="badge">'+
 '<div id="ubuntu-reviewers-logo"></div>'+
 '<div id="ubuntu-logo"></div>'+
 '<div id="progress-bar"><div id="bar" style="width:10%">10%</div></div>'+
 '<div id="percentage">10%</div>'+
 '<div id="ubuntu-review-team"><img src="http://daniel.holba.ch/review/images/3.png"><a href="http://launchpad.net/ubuntu-review" target="_blank">Powered by Ubuntu Review Gadget</a></div>'+
 '</div>';
 document.write(gadget);
 and in the default.css you should replace all the "images/" by "http://daniel.holba.ch/review/images/"
 that's all :)
[15:54] <dholbach> (script ran in the meantime)
[15:54] <bobbo> 11% wooot
[15:55] <daker> listen
[15:55] <daker> put that line :
[15:55] <daker> v
[15:55] <daker> <link rel="stylesheet" href="http://daniel.holba.ch/review/default.css" type="text/css" media="screen">
[15:57] <dholbach> yeeehaw
[15:58] <bobbo> got it working on my site
[15:58] <bobbo> daker and the uber-bar of review awesomeness get a mention in the daily update blog
[15:58] <dholbach> bobbo: *I* got it working on your site - your 0wned!
[15:58] <dholbach> ;-)
[15:59]  * dholbach shuts up - it's 30°C in here, I can't think straight
[15:59] <daker>  '<div id="ubuntu-review-team"><img src="http://daniel.holba.ch/review/images/3.png"><a href="http://launchpad.net/~ubuntu-review" target="_blank">Powered by Ubuntu Review Gadget</a></div>'+
[15:59] <dholbach> thanks a bunch daker and bobbo
[15:59] <daker> add this
[16:00] <daker> missing the '~'
[16:01] <bobbo> dholbach, do you want me to put it in the wiki for you?
[16:02] <dholbach> daker: maybe link to https://wiki.ubuntu.com/ReviewersTeam instead?
[16:02] <dholbach> bobbo: that'd be sweet
[16:02] <daker> oki as you like
[16:02] <dholbach> super
[16:02] <daker> just to correct the link
[16:02] <dholbach> yep
[16:04] <daker> other thing we can add later is to put on the left the number of patches or something like that
[16:04] <dholbach> fixed now
[16:04] <dholbach> at least on http://daniel.holba.ch/review/example.html
[16:04] <dholbach> on bobbo's site it's cached
[16:04] <dholbach> this looks GREAT
[16:04] <dholbach> thanks a bunch daker
[16:04] <dholbach> thanks bobbo
[16:05]  * bobbo isn't using dholbach's code yet, still using my own to grab it
[16:05] <dholbach> ah ok
[16:05] <daker> \o/
[16:05] <daker> no problem :)
[16:06] <bobbo> woot, using dholbach's code works :D
[16:06] <bobbo> dholbach, for the wiki, how often will that script run to update the numbers?
[16:07] <dholbach> every 20 minutes
[16:07] <bobbo> epic
[16:13] <vish> woot! daker
[16:14] <daker> \o/
[16:14] <bobbo> that's it on the wiki, too
[16:15] <vish> bobbo: dholbach: what do you think of having the "patches reviewed" as in done on the top left ?
[16:15] <dholbach> vish: I have no opinion - if you want to change and think it looks better: sure
[16:15] <bobbo> vish, if it could fit in without making it look too full/messy, sounds awesome
[16:15] <vish> the number would keep rising and look insane :D
[16:16] <vish> daker: ^  probably the same font size as the % would do ..?  "XXX Reviewed"
[16:17] <daker> bobbo, where ?
[16:17] <daker> yes
[16:17] <bobbo> daker, https://wiki.ubuntu.com/OperationCleansweep
[16:19] <daker> reviews = total - in queue ?
[16:20] <vish> daker: what are all the numbers we are getting now?
[16:20] <vish> from the scripts
[16:21] <vish> daker: nvm , the total - in queue == done was what i meant :D
[16:21] <dholbach> well done guys
[16:23] <daker> dholbach, the .js file is generated with python ?
[16:23] <dholbach> daker: yes
[16:24] <dholbach> let me show you
[16:24] <dholbach> http://bazaar.launchpad.net/~ubuntu-reviewers/ubuntu-review-overview/trunk/annotate/head:/countdown.py
[16:24] <dholbach> line 9 to 23
[16:26] <dholbach> ok my friends - I call it a day - see you tomorrow!
[16:41] <nigelb> cyphermox: poke
[16:58] <bobbo> @patchcount
[16:58] <patchbot> bobbo: Total: 2280     Left in Queue: 2016     11.6% complete
[16:59] <seb128> the count is weird
[16:59] <nigelb> ugh, I didn't go through dholbach's code
[16:59] <nigelb> its higher than what it should be
[16:59] <seb128> or it doesn't match the path to be reviewed list in the topic
[16:59] <nigelb> seb128: working on it now :)
[17:00] <seb128> ok
[17:00] <cyphermox> nigelb: poke
[17:01] <nigelb> cyphermox: I was wondering if you'd like to take a class at ubuntu user days
[17:02] <bobbo> I'm going to call it a day here, have a good evening everyone :)
[17:03] <nigelb> bobbo: night
[17:03] <cyphermox> nigelb: did you already have a specific subject in mind?
[17:03] <nigelb> cyphermox: " Choosing hardware that works with Ubuntu
[17:03] <cyphermox> oooh
[17:04] <nigelb> cyphermox: like it? ;)
[17:04] <cyphermox> yep
[17:04] <cyphermox> how did you know I'd be interested?
[17:04] <nigelb> I know you're working on hardware certification at canonical ;)
[17:05] <nigelb> You did tell me last time I talked to you.  I remember things. :)
[17:05]  * cyphermox is starting to wonder just how much nigelb knows about him ;)
[17:05] <cyphermox> ah, i see ;)
[17:05]  * nigelb just has good memory :)
[17:05] <nigelb> ok, so the schedule is here https://wiki.ubuntu.com/UserDaysTeam/lucid
[17:05] <nigelb> is 0200 UTC okay with you? or do you want to go in between?
[17:06] <cyphermox> let me translate that
[17:07] <cyphermox> 0200 UTC is reasonable
[17:07] <nigelb> ok, I'll put you down for 0200 :)
[17:07] <nigelb> thanks for helping out, the event is on 10th July
[17:07] <cyphermox> that time ends up being on July 11th though, no?
[17:08] <cyphermox> (UTC)
[17:08] <nigelb> yeah
[17:08] <nigelb> but it should still be 12th for you
[17:08] <cyphermox> yes, still the 10th
[17:09] <nigelb> err 10th, yeah
[17:27] <bobbo> http://bobbo.me.uk/index.php/planet-ubuntu/operation-cleansweep-day-four-update <-- Day 4 update
[17:53] <daker> \o/
[17:56] <nigelb> \o/
[18:01] <nigelb> @patchcount
[18:01] <patchbot> nigelb: Total: 2279     Left in Queue: 2014     11.6% complete