ReviewerMeeting20090715

Not logged in - Log In / Register

ReviewerMeeting20090715

summary

logs

AMEU

Jul 15 10:00:17 <barry> #startmeeting
Jul 15 10:00:18 <MootBot>       Meeting started at 09:00. The chair is barry.
Jul 15 10:00:18 <MootBot>       Commands Available: [TOPIC], [IDEA], [ACTION], [AGREED], [LINK], [VOTE]
Jul 15 10:00:20 *       deryck (n=deryck@samba/team/deryck) has joined #launchpad-meeting
Jul 15 10:00:30 <barry> hello everyone and welcome to this week's ameu reviewer's meeting.  who's here today?
Jul 15 10:00:32 <bigjools>      me
Jul 15 10:00:35 <EdwinGrubbs>   me
Jul 15 10:00:35 <sinzui>        me
Jul 15 10:00:37 <deryck>        me
Jul 15 10:00:38 <mars>  me
Jul 15 10:00:38 <henninge>      me
Jul 15 10:00:51 <noodles775>    me
Jul 15 10:01:00 <henninge>      jtv sends his apologies but he does need to sleep sometime ... ;-)
Jul 15 10:01:29 <intellectronica>       me
Jul 15 10:01:43 <danilos>       me
Jul 15 10:01:54 <barry> adeuring: ping
Jul 15 10:02:02 <barry> allenap: ping
Jul 15 10:02:03 <adeuring>      whoops, me
Jul 15 10:02:09 <barry> bac: ping
Jul 15 10:02:10 <leonardr>      me
Jul 15 10:02:18 <barry> BjornT: ping
Jul 15 10:02:19 <allenap>       barry: I'm on holiday this week.
Jul 15 10:02:34 <barry> allenap: ah!  lucky you :)
Jul 15 10:02:43 <barry> cprov: ping
Jul 15 10:03:08 <barry> flacoste: ping
Jul 15 10:03:15 <barry> gary_poster: ping
Jul 15 10:03:22 <barry> gmb: ping
Jul 15 10:03:41 <barry> rockstar: ping
Jul 15 10:03:45 <barry> salgado: ping
Jul 15 10:03:55 <barry> [TOPIC] agenda
Jul 15 10:04:31 *       Disconnected ().
**** ENDING LOGGING AT Wed Jul 15 10:04:31 2009

**** BEGIN LOGGING AT Wed Jul 15 10:04:43 2009

Jul 15 10:04:43 *       Now talking on #launchpad-meeting
Jul 15 10:04:43 *       Topic for #launchpad-meeting is:  Launchpad Meeting Grounds | Channel logs: http://irclogs.ubuntu.com/ | Meeting logs and agenda: https://dev.launchpad.net/MeetingAgenda
Jul 15 10:04:43 *       Topic for #launchpad-meeting set by matsubara at Thu Apr 30 11:14:51 2009
Jul 15 10:04:43 *       #launchpad-meeting :https://launchpad.net
Jul 15 10:04:50 *       barry hates irc
Jul 15 10:04:58 <barry> i'm just going to assume everyone here :)
Jul 15 10:05:04 <barry> [TOPIC] agenda
Jul 15 10:05:09 <MootBot>       New Topic:  agenda
Jul 15 10:05:19 <barry>  * Roll call
Jul 15 10:05:19 <barry>  * Action items
Jul 15 10:05:19 <barry>  * Mentoring update
Jul 15 10:05:19 <barry>  * Including js script tags conditionally
Jul 15 10:05:19 <barry>  * ReST headers in doctests -- gmb
Jul 15 10:05:19 <barry>  * Permission explosions -- flacoste
Jul 15 10:05:19 <barry>  * Peanut gallery (anything not on the agenda)
Jul 15 10:05:23 *       Received a CTCP VERSION from freenode-connect
Jul 15 10:05:27 <barry> [TOPIC] action items
Jul 15 10:05:33 <MootBot>       New Topic:  action items
Jul 15 10:05:38 <barry>  * gary_poster to take importfascist and rSP() discussion to ml
Jul 15 10:05:56 <gary_poster>   barry: I will do that in the next week or we will take it off the list.
Jul 15 10:06:00 <gary_poster>   my apologies
Jul 15 10:06:09 <barry> gary_poster: no worries.  thanks
Jul 15 10:06:17 <barry>  * intellectronica to email list about higher JS branch limits
Jul 15 10:06:42 <intellectronica>       i did
Jul 15 10:06:48 <barry> intellectronica: yep, thanks!
Jul 15 10:06:56 <barry> [TOPIC] mentoring update
Jul 15 10:06:57 <MootBot>       New Topic:  mentoring update
Jul 15 10:07:06 <barry> any feedback from our mentors or mentats?
Jul 15 10:08:01 <barry> guess not :)
Jul 15 10:08:04 <deryck>        it's been going fine for me, not that that's much feedback. :)
Jul 15 10:08:22 <barry> deryck: great to hear
Jul 15 10:08:30 <barry> [TOPIC]  * Including js script tags conditionally
Jul 15 10:08:31 <MootBot>       New Topic:   * Including js script tags conditionally
Jul 15 10:08:42 <barry> sorry, i forget who added this one
Jul 15 10:08:56 <deryck>        barry, I did, per a review you did of my branch.
Jul 15 10:09:09 <barry> deryck: right!  it's been one of those weeks ;)
Jul 15 10:09:20 <barry> deryck: take it away deryck the orchestra leader!
Jul 15 10:09:42 <deryck>        This is Just as a reminder to all that we should only include script tags when in devmode, so that we don't load js twice.
Jul 15 10:10:01 <deryck>        scripts are concat'ed into launchpad.js, so be careful of this, and watch for it in reviews.
Jul 15 10:10:05 *       abentley (n=abentley@CPE001a7046c8f9-CM001e6b233d5a.cpe.net.cable.rogers.com) has joined #launchpad-meeting
Jul 15 10:10:20 <abentley>      me (sorry I'm late)
Jul 15 10:10:29 <barry> deryck: can you describe the tal you used to load that conditionally?
Jul 15 10:10:35 <gmb>   me
Jul 15 10:11:09 <deryck>        barry, just a <tal:devmode condition="devmode"> with the scripts in that block.
Jul 15 10:11:15 <flacoste>      deryck: can it be added to the JS wiki page?
Jul 15 10:11:21 <sinzui>        The yahoo calendar widget is not in the compressed yui file, which does require a conditional in the head-epilogue.
Jul 15 10:11:27 <deryck>        flacoste, sure, I can do that.
Jul 15 10:11:45 <barry> deryck: thanks
Jul 15 10:11:54 <deryck>        np :)
Jul 15 10:12:03 <sinzui>        Is it possible to get that script into something that is away "on"
Jul 15 10:12:09 <flacoste>      sinzui: can't we just add it?
Jul 15 10:13:03 <sinzui>        flacoste: I was tempted to do so last night. I'm not sure if the separation is historical or YUI2. I think I need a pre-imp call to understand the issue
Jul 15 10:13:20 <flacoste>      sinzui: let's have a pre-impl call later
Jul 15 10:13:27 <sinzui>        fab
Jul 15 10:13:43 <barry> cool.  anything else on this topic?
Jul 15 10:14:07 <barry> [TOPIC]  * ReST headers in doctests -- gmb
Jul 15 10:14:07 <MootBot>       New Topic:   * ReST headers in doctests -- gmb
Jul 15 10:14:12 <gmb>   Ah.
Jul 15 10:14:26 <gmb>   So, this came up on Monday.
Jul 15 10:14:47 <gmb>   First there was some confusion about whether we should use ReST or Moin style headers in  doctests.
Jul 15 10:14:54 <barry> rest
Jul 15 10:15:10 <gmb>   Then, when bigjools and I had confirmed we should be using ReST, no one could remember why we had changed back to ReST from Moin
Jul 15 10:15:15 <barry> but existing pages can be changed as you go, low priority
Jul 15 10:15:18 <gmb>   (having gone from ReST to Moin some time before)
Jul 15 10:15:36 <sinzui>        lp-dev-util/formatdoctest -p path/to/tests/*.txt will fix many. Edwin pointed out though the script uses the headers I use from gedit doctests. we can tweak the script to use gary's headers
Jul 15 10:15:41 <gmb>   barry: Right, but the point is that people are still confused about this, and the reasons for the change don't seem to have been communicated effectively.
Jul 15 10:15:48 <barry> gmb: we're trying to standardize on rest since that's basically what the python community has standardized on
Jul 15 10:15:52 <flacoste>      well, they were
Jul 15 10:15:54 <flacoste>      but
Jul 15 10:16:03 <flacoste>      we didn't follow-through
Jul 15 10:16:12 <flacoste>      the idea is that there are no tools to process Moin syntax
Jul 15 10:16:19 <flacoste>      and we aren't really using Moin syntax at all anyway
Jul 15 10:16:36 <gary_poster>   I guess.  trying to make it casual and low-pain means that people didn't notice or remember
Jul 15 10:16:46 <gary_poster>   that is, I guess we didn't follow through
Jul 15 10:17:04 <flacoste>      the idea was to have a sphinx easy setup to browse all the doctests
Jul 15 10:17:05 <sinzui>        I think salgado or bac have test cleanup listed among their objectives. rs=sinzui to run the script and land the branch
Jul 15 10:17:06 <flacoste>      as a start
Jul 15 10:17:14 <flacoste>      and then start organizing it
Jul 15 10:17:27 <flacoste>      that's what we didn't follow-through on
Jul 15 10:17:31 <gary_poster>   that test cleanup sounds cool
Jul 15 10:17:41 <gary_poster>   I see
Jul 15 10:18:09 <salgado>       me
Jul 15 10:18:26 <barry> it would also be nice if we could start using rest in our wikis :)
Jul 15 10:18:35 <sinzui>        salgado: are you late or volunteering to update all tests to ReST headers
Jul 15 10:18:47 <gary_poster>   or both! :-)
Jul 15 10:18:52 <salgado>       just late!
Jul 15 10:19:08 <barry> flacoste, sinzui  can we put some action items on the list for this?  or is it all just so background/low priority that we won't get to it any time soon?
Jul 15 10:19:23 <abentley>      barry: We can start using rest in our wikis.
Jul 15 10:19:52 <barry> abentley: really?  well, that would be cool.  also in our merge proposal cover letters :)
Jul 15 10:20:16 <abentley>      barry: Sure, moin has supported rest for years.
Jul 15 10:20:26 <flacoste>      barry: yeah, it's not really a priority at the moment
Jul 15 10:20:33 <sinzui>        barry: I can do it within the next 3 weeks. I will create a bug about it since I think we want to codify the header style in the script
Jul 15 10:21:09 <flacoste>      standardizing on what the python community uses is sufficient in itself imho
Jul 15 10:21:14 <barry> flacoste, sinzui, abentley cool.  so, gmb does that answer the question? :)
Jul 15 10:21:23 <flacoste>      especially since it means that's what we are using in the lazr packages
Jul 15 10:21:27 <flacoste>      because of PyPi
Jul 15 10:21:42 <gmb>   barry: Yes, thank you. I'll update the style guid to include some details as to the whys and wherefores.
Jul 15 10:21:48 <gmb>   *guide
Jul 15 10:21:54 <barry> gmb: awesome, thanks
Jul 15 10:21:58 <abentley>      barry: Just specify #FORMAT rst  at the top.
Jul 15 10:22:08 <barry> [ACTION] gmb to update style guides to clarify why we're standardizing on rest
Jul 15 10:22:09 <MootBot>       ACTION received:  gmb to update style guides to clarify why we're standardizing on rest
Jul 15 10:22:23 <sinzui>        BTW I had a merge conflict into lazr.restful because the headers in the tests are different
Jul 15 10:22:26 <barry> abentley: rock.  i will try that when i update the minutes of this meeting :)
Jul 15 10:22:45 <barry> gmb: thanks, anything else on this topic?
Jul 15 10:22:53 <gmb>   Nope.
Jul 15 10:22:55 <gmb>   That's all.
Jul 15 10:23:12 <barry> [TOPIC]  * Permission explosions -- flacoste
Jul 15 10:23:13 <MootBot>       New Topic:   * Permission explosions -- flacoste
Jul 15 10:23:22 <flacoste>      ok
Jul 15 10:23:31 <flacoste>      this is a reminder
Jul 15 10:23:44 <flacoste>      we should try to minimize the number of special permissions
Jul 15 10:23:59 <flacoste>      so in reviews, everytime you see somebody define a new permission
Jul 15 10:24:07 <flacoste>      you should ask why one of the regular one isn't used
Jul 15 10:24:12 <flacoste>      the model is
Jul 15 10:24:22 <flacoste>      View -> Append -> Edit -> Admin
Jul 15 10:24:41 <flacoste>      there is also the Moderate permission which sits above Append at the level of Edit
Jul 15 10:24:52 <flacoste>      those are our core permissions
Jul 15 10:25:19 <flacoste>      now, if you look at permissions.zcml, you'll see that there are a lot of other special permissions
Jul 15 10:25:38 <flacoste>      sometimes these are unvoidable, but many times they are not
Jul 15 10:25:57 <flacoste>      for example, the ProjectReview special permission was introduced recentely
Jul 15 10:26:07 <flacoste>      i think there is no reason why launchpad.Moderate couldn't have been used there
Jul 15 10:26:28 <flacoste>      so as reviewer, be though on the introduction of new permission
Jul 15 10:26:36 <barry> flacoste: do we have any plans to do an audit and clean up of those special permissions?
Jul 15 10:26:51 <bac>   flacoste: i'll follow up with you to clean that one up
Jul 15 10:27:11 <flacoste>      yeah, i think we'll need to do an audit at some point and see if we can remove some of those
Jul 15 10:27:30 <flacoste>      but starting with a lock-down will make the audit easier :-)
Jul 15 10:27:34 <barry> flacoste: cool
Jul 15 10:27:53 <flacoste>      i'll update the reviewers checklist with this reminder
Jul 15 10:28:01 <flacoste>      not sure if that page is still checked often though
Jul 15 10:28:04 <barry> flacoste: also, what is the root cause for adding new special permissions?  is it because we don't (as a group) have a good understanding of how the existing permissions work?
Jul 15 10:28:27 <flacoste>      some of that
Jul 15 10:28:46 <flacoste>      but also, the way permissions are mapped to roles is kind of ad-hoc
Jul 15 10:28:56 <flacoste>      (through the IAuthorization interface)
Jul 15 10:29:34 <flacoste>      you have to inspect code to understand who get to do what
Jul 15 10:29:36 <flacoste>      where
Jul 15 10:30:09 <barry> flacoste: which isn't really feasible when you're working out a new feature
Jul 15 10:30:14 <mars>  flacoste, so, would centralizing the permissions and capabilities of each make them easier to use?
Jul 15 10:30:32 <flacoste>      they are already kind of centralised
Jul 15 10:30:33 <flacoste>      in one file
Jul 15 10:30:48 <flacoste>      but that's not the way things work
Jul 15 10:30:57 <flacoste>      permission are granted on a per-context level
Jul 15 10:31:05 <flacoste>      that's why it's possible to not define new ones
Jul 15 10:31:15 <flacoste>      but reuse the semantics of existing ones
Jul 15 10:31:40 <flacoste>      for example, i think the Moderate permission is really only defined on the Question object
Jul 15 10:32:06 <flacoste>      which means it's easy to reuse the semantic for project review
Jul 15 10:33:20 <barry> flacoste: maybe a brief overview (largely what you wrote above) would be useful in the dev wiki?  could you add that when you update the reviewer docs?
Jul 15 10:33:29 <flacoste>      sure
Jul 15 10:33:34 <barry> flacoste: thanks!
Jul 15 10:33:59 <barry> [ACTION] flacoste to update reviewer docs and dev wiki re: permission implosion
Jul 15 10:34:00 <MootBot>       ACTION received:  flacoste to update reviewer docs and dev wiki re: permission implosion
Jul 15 10:34:06 <flacoste>      explosion!
Jul 15 10:34:13 <flacoste>      proliferation
Jul 15 10:34:16 <flacoste>      whatever
Jul 15 10:34:18 <flacoste>      eot
Jul 15 10:34:18 <barry> :)
Jul 15 10:34:23 <barry> flacoste: thanks, great topic
Jul 15 10:34:32 <barry> [TOPIC]  * Peanut gallery (anything not on the agenda)
Jul 15 10:34:33 <MootBot>       New Topic:   * Peanut gallery (anything not on the agenda)
Jul 15 10:34:44 <barry> okay, the floor is open, does anybody have anything else?
Jul 15 10:34:46 <intellectronica>       i wanted to talk briefly about generating js
Jul 15 10:34:56 <barry> intellectronica: go ahead
Jul 15 10:35:15 <intellectronica>       from time to time you'll need to generate javascript code and values to embed in a page
Jul 15 10:35:39 <intellectronica>       say, when you want to initialise a widget or call a function using values you have when you render the page
Jul 15 10:35:52 <intellectronica>       two important things to remember:
Jul 15 10:36:35 <intellectronica>       1. avoid using zpt for doing that. it's clumsy and hard to maintain. do the work in the view code in python, expose it as a property and include it in the template
Jul 15 10:37:20 <intellectronica>       2. always use simplejson to serialize python values into javascript code. it's so easy to do, and you have no chance of getting better result by preparing the string yourself
Jul 15 10:37:32 <intellectronica>       that's it
Jul 15 10:37:47 <barry> intellectronica: thanks very much.  i learned this lesson the hard way (and thanks for your patch :)
Jul 15 10:38:21 <mars>  intellectronica, is there a specific pattern we have to follow to get the <script> tag and it's contents in there from the view?  Because zpt tries to "help" with script tags...
Jul 15 10:38:26 <sinzui>        I am tempted to try a 3rd way, the view reads the initialization script, adds the setup values and exposes it as a property so that the JS is not in the python
Jul 15 10:38:46 <noodles775>    sinzui: +1
Jul 15 10:39:12 <intellectronica>       mars: <script tal:content="structure view/js_code" /> should do the trick, i think
Jul 15 10:39:14 <barry> sinzui: i'm not sure i understand, can you clarify?
Jul 15 10:39:20 <intellectronica>       sinzui: please explain more. i don't understand
Jul 15 10:39:41 <sinzui>        I am hesitant to start this because I want to ensure there are sane values in the initialization script for YUI-test
Jul 15 10:39:45 <noodles775>    I'm finding it hard to see a case where you'd need to generate js, rather than the js initialising from classes/content on the page (but just point me where)
Jul 15 10:40:28 <barry> sinzui: if you'd like to defer to next week or the ml, that's cool
Jul 15 10:40:44 <intellectronica>       noodles775: yes, that's another way to do that. still you have to put these values in the page
Jul 15 10:41:09 <noodles775>    intellectronica: yes, but that can be done semantically (so the page represents the state that the JS initialises from)
Jul 15 10:41:35 <sinzui>        intellectronica: I will follow up on the list. I just want the keep js and python and html separated.
Jul 15 10:41:49 <intellectronica>       noodles775: i don't understand. i think you guys are ahead of me
Jul 15 10:42:02 <intellectronica>       sinzui: cool, i look forward
Jul 15 10:42:02 <barry> sinzui: it's the intersection of those three axes of evil where things get interesting
Jul 15 10:42:19 <noodles775>    intellectronica: no, I probably just haven't seen an example where it's required. I'll watch for the mailing list discussion too :)
Jul 15 10:42:28 <sinzui>        interesting? you mean "knives out".
Jul 15 10:42:31 <mars>  sinzui, I would love to see a more in-depth statement of the problem, issues, solution, and how it might address them on the list
Jul 15 10:42:32 <barry> :)
Jul 15 10:42:52 <barry> sinzui:  thanks.  we have 3 minutes left so moving that to the ml would be great
Jul 15 10:43:50 <barry> any last minute items?  otherwise, we're done
Jul 15 10:43:55 <barry> 5
Jul 15 10:44:00 <barry> 4
Jul 15 10:44:04 <barry> 3
Jul 15 10:44:07 <barry> 2
Jul 15 10:44:10 <barry> 1
Jul 15 10:44:12 <barry> #endmeeting

AsiaPac

Jul 15 18:30:20 <barry> #startmeeting
Jul 15 18:30:21 <MootBot>       Meeting started at 17:30. The chair is barry.
Jul 15 18:30:21 <MootBot>       Commands Available: [TOPIC], [IDEA], [ACTION], [AGREED], [LINK], [VOTE]
Jul 15 18:30:27 <barry> jml, thumper, mwhudson ping
Jul 15 18:30:30 <jml>   hi
Jul 15 18:30:35 <mwhudson>      hi!
Jul 15 18:30:39 <thumper>       jo
Jul 15 18:31:11 <barry> how are y'all doing?  earthquake + tsunami didn't bother you did it?
Jul 15 18:31:19 <thumper>       didn't even feel it
Jul 15 18:31:38 <mwhudson>      given the magnitude it seems to have caused amazingly little damage
Jul 15 18:31:53 <barry> good to hear!
Jul 15 18:31:55 <mwhudson>      and only a baby tsunami
Jul 15 18:32:23 <barry> i had a friend from pittsburgh who moved to san fran describe his first earthquake
Jul 15 18:32:56 <barry> said, "they always tell you never run outside".  when it first hit, he said he found himself outside before he even knew what was going on ;)
Jul 15 18:33:09 <mwhudson>      (the epicentre really is in the ass-end of nowhere too, that helps)
Jul 15 18:33:35 <jml>   the epicentre is in Palmy? :P
Jul 15 18:33:59 <mwhudson>      hah
Jul 15 18:34:08 <barry> inside joke? :)
Jul 15 18:34:13 <barry> anyway...
Jul 15 18:34:14 <mwhudson>      i live in palmy
Jul 15 18:34:17 <barry> oh :)
Jul 15 18:34:20 <mwhudson>      it's not the most exciting place in the universe
Jul 15 18:34:31 <mwhudson>      indeed, anyway...
Jul 15 18:34:46 <barry> do you guys have burning issues, or do you want me to start with a review of ameu?
Jul 15 18:35:07 <jml>   review please
Jul 15 18:35:07 <mwhudson>      review of ameu sounds good
Jul 15 18:35:07 *       mars has quit (Remote closed the connection)
Jul 15 18:35:12 <jml>   all issues have been safely coated in asbestos
Jul 15 18:35:32 <barry> lessee...
Jul 15 18:35:58 <barry> deryck brought up an issue from a review about only loading javascript files in devmode, since in prod they're always included in launchpad.js
Jul 15 18:36:18 <barry> he'll write about that in the js wiki page
Jul 15 18:36:53 <barry> gmb brought up a question about doctest format, moin or rest, but since you guys all hate doctests, i guess that one doesn't matter :)
Jul 15 18:37:11 <mwhudson>      well
Jul 15 18:37:33 <mwhudson>      if our documentation is going to be written in some markup
Jul 15 18:37:39 <mwhudson>      it would be nice if this was used somehow
Jul 15 18:37:49 *       mars (n=mars@canonical/launchpad/mars) has joined #launchpad-meeting
Jul 15 18:37:54 <barry> mwhudson: we're rest all the way baby!
Jul 15 18:38:09 <barry> but existing doctests can be updated as we go
Jul 15 18:38:09 <mwhudson>      i don't really have an opinion on moin vs rest, pick one and stick to it though
Jul 15 18:38:27 <barry> definitely rest, for a host of reasons outlined in the detailed minutes
Jul 15 18:38:41 <barry> gmb will update the relevant wiki pages
Jul 15 18:38:50 <mwhudson>      if we setup a sphinx-y type thing somewhere and actually use it, it might encourage us to make our doctests better documentation
Jul 15 18:38:52 <barry> i've also filed an rt to get our dev wiki to support rest
Jul 15 18:38:52 <mwhudson>      (gasp!)
Jul 15 18:39:07 <barry> mwhudson: +1 is not plussy enough
Jul 15 18:39:34 <jml>   barry, if Launchpad gets a wiki, do you anticipate *it* supporting rest?
Jul 15 18:39:41 <barry> jml: i do
Jul 15 18:39:46 <barry> at least
Jul 15 18:39:54 <thumper>       we're going to hand craft a wiki aren't we?
Jul 15 18:40:07 <thumper>       I thought we were going to build one on bzrlib
Jul 15 18:40:22 <barry> thumper: that's been on the wishlist for at least 2 years
Jul 15 18:40:26 <jml>   like someone has already done with Hg :(
Jul 15 18:40:28 <thumper>       right
Jul 15 18:40:32 <barry> :(
Jul 15 18:40:59 <jml>   gosh that was disappointing watching that lightning talk at europython and thinking "we've had this idea for at least two years now"
Jul 15 18:41:01 <jml>   anyway
Jul 15 18:41:12 <barry> last review item: flacoste brought up the issue of "permission explosion", the tendency to create too many special permissions
Jul 15 18:41:29 <barry> instead of reusing the core ones
Jul 15 18:41:38 <thumper>       barry: for example?
Jul 15 18:41:50 <barry> thumper: let me get my notes...
Jul 15 18:42:12 <jml>   barry, presumably flacoste noticed this trend by inspecting our highly auditable zcml or perhaps looking at a permission control panel? :P
Jul 15 18:42:52 <barry> Jul 15 10:23:44 <flacoste>      we should try to minimize the number of special permissions
Jul 15 18:42:52 <barry> Jul 15 10:23:59 <flacoste>      so in reviews, everytime you see somebody define a new permission
Jul 15 18:42:52 <barry> Jul 15 10:24:07 <flacoste>      you should ask why one of the regular one isn't used
Jul 15 18:42:52 <barry> Jul 15 10:24:12 <flacoste>      the model is
Jul 15 18:42:52 <barry> Jul 15 10:24:22 <flacoste>      View -> Append -> Edit -> Admin
Jul 15 18:42:52 <barry> Jul 15 10:24:41 <flacoste>      there is also the Moderate permission which sits above Append at the level of Edit
Jul 15 18:42:52 <barry> Jul 15 10:24:52 <flacoste>      those are our core permissions
Jul 15 18:42:52 <barry> Jul 15 10:25:19 <flacoste>      now, if you look at permissions.zcml, you'll see that there are a lot of other special permissions
Jul 15 18:42:52 <barry> Jul 15 10:25:38 <flacoste>      sometimes these are unvoidable, but many times they are not
Jul 15 18:42:52 <barry> Jul 15 10:25:57 <flacoste>      for example, the ProjectReview special permission was introduced recentely
Jul 15 18:42:52 <barry> Jul 15 10:26:07 <flacoste>      i think there is no reason why launchpad.Moderate couldn't have been used there
Jul 15 18:42:52 <barry> Jul 15 10:26:28 <flacoste>      so as reviewer, be though on the introduction of new permission
Jul 15 18:42:54 <jml>   (sorry, I think I prepared the coffee extra bitter this morning).
Jul 15 18:43:17 <thumper>       we have an Append permission?
Jul 15 18:43:25 <jml>   yeah.
Jul 15 18:44:03 <thumper>       huh
Jul 15 18:44:06 <thumper>       didn't know that
Jul 15 18:44:20 <mwhudson>      i was more or less aware of this point
Jul 15 18:44:22 <barry> flacoste will update the relevant wiki pages with details
Jul 15 18:44:23 <jml>   neither did I.
Jul 15 18:44:36 <jml>   barry, and how would we *find* the relevant wiki pages?
Jul 15 18:44:42 <mwhudson>      i haven't reviewed or written any code to do with permissions in ages though
Jul 15 18:44:46 <barry> jml: they should be on the dev wiki <wink>
Jul 15 18:45:02 <jml>   well that narrows it down :\
Jul 15 18:45:10 <barry> foundations also wants to audit and simplify our permissions
Jul 15 18:45:14 <barry> jml: yeah
Jul 15 18:45:37 <barry> anyway, that's all i have from ameu.  what's on your mind?
Jul 15 18:45:38 <mwhudson>      well at least there's less obsolete crap on the dev wiki than launchpad.c.c
Jul 15 18:45:50 <jml>   yeah. that's a very good thing.
Jul 15 18:46:13 <thumper>       I don't have anything just now
Jul 15 18:46:16 <mwhudson>      nothing in particular
Jul 15 18:46:25 <thumper>       apart from a general ORM annoyance
Jul 15 18:46:28 <barry> we need some interns to garden our wikis :)
Jul 15 18:46:32 <thumper>       w.r.t. permissions
Jul 15 18:46:37 <jml>   nothing from me, apart from general build process annoyance :)
Jul 15 18:46:42 <mwhudson>      the usual trepidation about how we're all supposed to be focused on a not-yet designed ui
Jul 15 18:46:47 <jml>   barry, with respect, no.
Jul 15 18:47:10 <barry> jml: you don't like interns?
Jul 15 18:47:47 <jml>   barry, we need a little bit of bootstrapping to put some kind of structure in place, and we need developers to be dissatisfied with simply getting an answer to their question.
Jul 15 18:48:55 <barry> jml: yeah.  but wiki editing is pretty low on people's list of things to do
Jul 15 18:49:37 <mwhudson>      there's something about wiki editing that always infuriates me
Jul 15 18:49:56 <jml>   barry, it's not wiki editing, it's working in a distributed team.
Jul 15 18:49:56 <mwhudson>      "edit this section" like media wiki would help, i think
Jul 15 18:49:56 <barry> mwhudson: the fact that textarea sucks?
Jul 15 18:50:04 <mwhudson>      barry: partly yeah
Jul 15 18:50:07 <jml>   mwhudson, +1
Jul 15 18:50:30 <barry> that would be nice.  it's all text does make things a bit better (not great though)
Jul 15 18:51:09 <barry> well, that's all i have anyway
Jul 15 18:51:20 <jml>   likewise.
Jul 15 18:51:35 <mwhudson>      if we end up with sphinx for our documentation, we should use that more and the wiki less maybe>
Jul 15 18:51:36 <mwhudson>      ?
Jul 15 18:51:43 <mwhudson>      but enough directionless rambling on this topic
Jul 15 18:51:46 *       mwhudson is done
Jul 15 18:51:57 <barry> mwhudson:  possibly.  i'm digging sphinx a lot lately
Jul 15 18:52:16 <barry> cool.  i guess we're done.  thanks guys
Jul 15 18:52:18 <barry> #endmeeting

ReviewerMeeting20090715 (last edited 2009-07-15 22:54:59 by barry)