/srv/irclogs.ubuntu.com/2009/02/18/#launchpad-meeting.txt

=== ursula_ is now known as Ursinha
=== ursula_ is now known as Ursinha
=== salgado-afk is now known as salgado
barry#startmeeting15:00
MootBotMeeting started at 09:00. The chair is barry.15:00
MootBotCommands Available: [TOPIC], [IDEA], [ACTION], [AGREED], [LINK], [VOTE]15:00
barryhello everyone and welcome to this week's ameu reviewers meeting.  who's here today?15:00
bigjoolsme15:00
gary_posterme15:00
barrywe have a number of sprinters who probably won't make it15:00
bacme15:01
barryso today's agenda will be pretty light15:01
gmbme15:01
salgadome!15:01
barryallenap: sinzui cprov BjornT ping15:02
BjornTme15:02
cprovme15:02
BjornTbarry: allenap is off sick today15:02
gmbbarry: allenap has the lurgi15:03
barryBjornT: bummer :(15:03
barry[TOPIC] agenda15:03
MootBotNew Topic:  agenda15:03
barry * Roll call15:03
barry * Action items15:03
barry * Mentoring update15:03
barry * Peanut gallery (anything not on the agenda)15:03
barry[TOPIC] action items15:03
MootBotNew Topic:  action items15:03
barry * abentley to experiment on storm base class15:03
sinzuime15:04
barrydunno where abentley is but he did work on a branch, which i semi-reviewed15:04
barryit looked pretty good.  stub did a semi-review too and between us i think we had some good suggestions15:04
barryjml has the official review, though i'll pick it up on friday if it gets that far15:04
barryi think it'll make a nice convenient base class for our storm classes15:05
barryanyway, we'll see how that goes15:05
barry * barry to add `field_id` to coding guideline15:05
barrythat came up at the asiapac meeting.   preference is to use field_id instead of fieldID for foreign key references15:06
barryiow, the attribute name in the storm class15:06
barry * barry to add `pretty()` functions to reviewers docs15:06
barrynot done15:06
barry * flacoste to work on API reviewer cheat sheet15:06
barryi suspect, not done15:06
gary_poster:-)15:06
salgadohe said something about that item last week, didn't he?15:07
salgadoor was it his other item?15:07
barryit was that one15:07
barryhe's changing his name to flacoste-hoover because he sucks :)15:08
* barry thinks he has to do the same15:08
salgadoFeb 11 10:43:45 <flacoste>      if it's not done next week, i change my15:08
salgadoname to flacoste_hoover15:08
barryright :)15:08
barry[TOPIC]  * Mentoring update15:08
MootBotNew Topic:   * Mentoring update15:08
barryany feedback from mentors or mentats?15:08
barryokay!15:09
barry[TOPIC]  * Peanut gallery (anything not on the agenda)15:10
MootBotNew Topic:   * Peanut gallery (anything not on the agenda)15:10
barryyou guys have anything?15:10
gary_posterWhy does MootBot suck?15:10
gary_posterOr, ore directly, why don't we have logs? :-)15:10
gary_posters/ore/more/15:10
barrygary_poster: and why do i continue to use it?15:11
gary_poster:-)15:11
gary_posterIt would be nice to have logs15:11
bigjoolsregarding the patching we do to get around circular imports for the API work, I recommend that we get some helper functions written.15:11
barrybigjools: i think that's the way to go.  my previous experiment failed15:11
barrybigjools: would you like to take a crack at that?15:11
rockstarbigjools, have you seen interfaces/_schema_circular_imports15:11
bigjoolsrockstar: I haven't15:11
rockstarbigjools, that's where the code team has started putting their circular import avoidance patching.15:12
gmbWhat happened to lazy_import? Or doesn't that work for schema declarations?15:12
rockstargmb, I don't think lazy_import would help in this situation.15:12
bigjoolsbarry: I can try, if I find the time!15:12
bigjoolsrockstar: that's not what I had in miund15:13
barryrockstar: +1 for moving them all to the same place15:13
gmbrockstar: Ah, right15:13
rockstarbarry, it makes my life easier, because I know where the declarations are.15:13
barryrockstar: yeah, grepping them out was 3/4 of the pain of my experiment15:13
bigjoolsI am talking about helper funcs that save you from working out WTF you need to patch15:13
barrythough i hate the module name :)15:13
bigjoolsI'd rather they were kept in the interface file15:13
rockstarbigjools, ah, I can't think of a use for that.  I usually know what I need to patch as I do it.15:14
barrybigjools: if the helpers were easily grepped for, that's fine15:14
rockstarbarry, it can be changed.  It was thumper's idea, and can change pretty easily.15:14
bigjoolsI think something like patch_returns_collection_type(func, type)15:14
bigjoolsand patch_param_type(func, param, type)15:15
bigjoolsmake sense?15:15
barrysomething like that.  maybe get 'circular' and 'import' in the names?15:16
barrye.g. close_circular_import(...) ?15:16
bigjoolsso_we_end_up_with_a_really_long_help_function_name() ? :)15:16
sinzuispin_wheels_make_smoke15:16
barrybigjools: in the fine tradition of python, keep ugly things ugly :)15:17
barrybigjools: anyway, take a crack at it and i'll happily review it!15:17
bigjoolsokidoki15:17
barry[ACTION] bigjools to work on helper functions for circular imports in interfaces15:17
MootBotACTION received:  bigjools to work on helper functions for circular imports in interfaces15:17
barrybigjools: thanks!15:17
bigjoolsmy pleasure15:17
barryanything else?15:17
bigjoolsone more thing15:18
barrysure thing15:18
bigjoolsI chatted to Bjorn this morning about extending the sourcedeps rsync filter that rocketfuel-get uses15:18
bigjoolsso it ignores */.bzr/*15:18
bigjoolsmy motiviation being that I am fed up of downloading the huge packs files when rsync runs after a bzr sourcedep update15:19
bigjoolsdoes this sound good or bad?15:19
bigjoolsideally I'd like to bzr pull the source deps ...15:19
gary_posterbuildbot has had issues with pulling bzr this weekend15:20
gary_posterit pulls sourcedeps to update normally15:20
gary_posterbut bzr did something funky with their branch15:20
gary_posterso it needed a merge (or an rsync)15:20
gary_posterso rsync is safer when that kind of situation arises15:21
bigjoolssounds like it's not a problem to be too worried about?15:21
barrydon't we eventually want to move to ppas for all that stuff anyway?15:21
gary_postersure, once in a month and a half15:21
bigjoolshow does everyone else sync sourcedeps?15:21
bigjoolsbarry: +115:21
barrybigjools: rocketfuel-get (rsync)15:22
gary_postera combination of ppas and buildout eggs15:22
barrygary_poster: yes yes yes yes yes yes yes yes yes15:22
gary_posterwell, source15:22
bacrf-get15:22
bigjoolsor "sourcecode" I should say15:22
barrygary_poster: you coming to our nags head sprint? :)15:22
gary_posterbarry: not that I know of :-) (first I heard of it)15:23
barrygary_poster: 120 hours of drinkin' and hackin'15:23
gary_posterheh, cool15:23
bacgary_poster: drive on down!15:23
gary_poster:-)15:23
bigjoolsanyway we don't need to decide now, I just wanted to discuss and see what people thought because I really am fed up with downloading 100+Mb of data when sourcecode/bzr is updated15:23
barrybigjools: thanks for putting this on the radar15:23
bigjoolsBjorn's objection to not syncing the .bzr dirs was that people might want to branch off it15:24
bigjoolsnp15:25
bigjoolsEOF15:25
gary_posterfor most people that will happen rarely enough that making a fresh branch shouldn't be a problem15:25
gary_posterIME15:25
barrycool.  anything else on this or other topic?15:25
bigjoolsmy thoughts too - branch off the official source, not our sourcecode deps :)15:25
barrysounds like we're done!15:27
barry#endmeeting15:27
MootBotMeeting finished at 09:27.15:27
barrythanks everyone15:27
bigjoolscheers15:27
gary_posterthanks15:27
=== salgado is now known as salgado-lunch
=== salgado-lunch is now known as salgado
=== mrevell is now known as mrevell-dinner
=== bac is now known as fakestevejobs
=== fakestevejobs is now known as bac
=== salgado is now known as salgado-afk

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