/srv/irclogs.ubuntu.com/2010/03/31/#ubuntu-reviews.txt

dholbachgood morning08:02
=== BlackZ_ is now known as BlackZ
=== yofel_ is now known as yofel
nigelbok, the basic idea is, we get to see all bugs with patches attached that are open18:51
nigelband the user should be able to select by package or by component (main, universe, etc)18:51
nigelband select status of patch (unreviewed, needs-work, forwarded)18:52
nigelband it should pop out the list of bugs and download the patch18:52
qenseok18:52
nigelbthen if possible display what is the upstream for that project (#wishlist)18:52
qensethat would be neat18:53
nigelbso a reviewer has an easy time doing the review18:53
qensethat's what we want18:53
nigelbpulling source should also be part of this, but we dont need to go into ground control18:54
nigelbsince review is a lot giving back to upstream18:54
qenseYou want to use the Debian watch files or the Vcs-* fields?18:54
nigelbthats where I'm doubtful18:54
nigelbwhen submitting a patch for upstream, it should be against their current trunk18:55
nigelbso someone is better off going to upstream18:55
qensethen you don't want the watch file18:55
nigelbbut sometimes a patch works against ubuntu source and we tweak it for upstream18:55
nigelbso, I'm undecided there18:55
qenseThe reviewer should be presented with an edit mode.18:56
nigelbso a terminal and text editor in one of the tabs18:56
qenseMaybe: first try to autoapply the patch to trunk, if that doesn't work present the user with a clean trunk and ask if he or she wants to apply the patch step-by-step or wants to view the output of the previous autotry18:56
qenseInline text-editor, or embed gedit?18:57
nigelbembed gedit18:57
qensesounds like the best idea to me indeed18:57
nigelbso I'll get the wiki page and mockups over weekend and we can hack code next week?18:58
qenseThat's a testweek for me. :) It would not be very convenient.18:59
nigelbthe week after then18:59
nigelbI need lotsa help with code.  I'm usually better at organizing people to code :D18:59
qensethe test week lasts from next week Wednesday until the Tuesday after that18:59
qensebut I can start helping that week18:59
nigelbok then exactly 2 weeks from now19:00
qenseok! :)19:01
qenseBut I may be reachable in between. If you need some advice you can always mail me.19:01
nigelbsure, will do :)19:02
nigelbI'll be on vacation next week (I just remembered)19:02
qenseJust remembered. :D19:02
qensenigelb: and remember: the reference manual is your friend!19:04
nigelbqense, I remembered that when writing apport hooks :D19:04

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