1 2013-07-11T01:07:09 *** dav1d
2 2013-07-11T01:07:16 *** Clopinou
3 2013-07-11T01:10:29 *** dav1dde
4 2013-07-11T01:48:30 *** ashutosh_singla1
5 2013-07-11T01:48:45 *** ashutosh1461
6 2013-07-11T02:24:45 *** ashutosh_singla1
7 2013-07-11T06:47:31 *** greg_f
8 2013-07-11T07:45:43 <ananasova> moin
9 2013-07-11T07:45:55 <dreimark> moin
10 2013-07-11T08:48:44 *** rciorba
11 2013-07-11T08:54:54 <rciorba> moin
12 2013-07-11T09:17:07 *** xorRaxRax
13 2013-07-11T09:17:48 *** xorRaxRax
14 2013-07-11T10:53:57 *** brunomartin
15 2013-07-11T11:06:16 *** brunomartin
16 2013-07-11T11:08:18 <ThomasWaldmann> moin
17 2013-07-11T11:58:18 * ThomasWaldmann updated ashutosh's etherpad and wonders why he isn't using ssh/screen/irssi to stay connected.
18 2013-07-11T12:25:18 <ThomasWaldmann> waldi: ananasova: btw, if we have the lists unified in one place in the revision and split it into 2 index fields at indexing time, also the code that uses the whoosh document as cache to avoid storage needs to be considered
19 2013-07-11T12:25:44 <ThomasWaldmann> as long as we don't use the same key/field name, it shouldn't be an issue, though
20 2013-07-11T12:59:39 <ThomasWaldmann> "avoid storage access"*
21 2013-07-11T13:00:27 <ThomasWaldmann> df
22 2013-07-11T13:00:32 <ThomasWaldmann> oops :D
23 2013-07-11T13:41:37 <xiaq> eSyr: https://codereview.appspot.com/10889044/ plz review this! :)
24 2013-07-11T13:41:40 <xiaq> ThomasWaldmann: ^
25 2013-07-11T15:11:15 *** brunomartin
26 2013-07-11T15:23:05 *** brunomartin
27 2013-07-11T15:37:26 *** dave_largo
28 2013-07-11T16:04:12 *** brunomartin
29 2013-07-11T16:07:03 *** rciorba
30 2013-07-11T16:35:03 <ThomasWaldmann> xiaq: done
31 2013-07-11T16:39:53 *** ashutosh1461
32 2013-07-11T16:41:35 * ashutosh1461 is hoping to finalize some work today :)
33 2013-07-11T16:49:35 <ashutosh1461> ThomasWaldmann, updated + replied to your comments https://codereview.appspot.com/10842043/ and replied to your comments here also, https://codereview.appspot.com/10973043/
34 2013-07-11T17:48:37 *** greg_f
35 2013-07-11T17:56:03 *** sharky93_
36 2013-07-11T18:07:40 <ananasova> i have a blocker: can't get the app config at logging init time.
37 2013-07-11T18:09:04 <ananasova> i am adding a section to the logging config and passing some arguments to the custom email handler
38 2013-07-11T18:09:23 <ananasova> those arguments are taken from the config, like admin emails
39 2013-07-11T18:10:55 <ananasova> one way doing is picking up the app config inside the handler and initializing admin emails right there, but it violates the encapsulation, and looks like a dirty hack
40 2013-07-11T18:11:15 <ananasova> ThomasWaldmann: maybe you have some pointers on this?
41 2013-07-11T18:44:52 *** sharky93_
42 2013-07-11T18:49:14 <sharky93> moin
43 2013-07-11T18:49:24 * sharky93 is back online! :)
44 2013-07-11T18:49:33 <dreimark> re
45 2013-07-11T18:49:56 <sharky93> TheSheep: btw, i had shared this one too @ http://www.youtube.com/watch?v=KWJCBrSCNf8 , along with the screenshot
46 2013-07-11T19:01:23 <ThomasWaldmann> ananasova: logging needs to be initialized first, before other stuff happens
47 2013-07-11T19:13:18 <ananasova> ThomasWaldmann: so there is no way of getting access to the config?
48 2013-07-11T19:19:30 *** MaikuMori_
49 2013-07-11T19:25:51 <TheSheep> sharky93: I want to see the html :)
50 2013-07-11T19:26:16 <TheSheep> sharky93: by the way, I think we should follow that advice from that article about less
51 2013-07-11T19:26:40 <TheSheep> sharky93: instead of adding bootstrap classes to moin's html, re-use moin's classes wherever possible
52 2013-07-11T19:30:58 *** MaikuMori
53 2013-07-11T19:30:59 *** ronny
54 2013-07-11T19:31:53 <sharky93> TheSheep: how about doing that in the second revision .. first we just get it working and then in a different patchset do that .. ?
55 2013-07-11T19:33:21 *** ronny
56 2013-07-11T19:37:01 <sharky93> TheSheep: i'll post these soon for review .. btw the other search code, you had a look yet ?
57 2013-07-11T19:38:33 <TheSheep> sharky93: argh, I forgot, sorry
58 2013-07-11T19:38:48 <TheSheep> sharky93: do you have the links handy?
59 2013-07-11T19:40:07 <sharky93> TheSheep: hmm, just a second
60 2013-07-11T19:40:53 <sharky93> so here goes. 1 @ https://codereview.appspot.com/10976043/
61 2013-07-11T19:41:01 <sharky93> 2 @ https://codereview.appspot.com/10935045/
62 2013-07-11T19:41:08 <sharky93> 3 @ https://codereview.appspot.com/10971043/
63 2013-07-11T19:41:11 <sharky93> TheSheep: ^
64 2013-07-11T19:43:09 <TheSheep> thanks
65 2013-07-11T19:47:21 *** ashutosh_singla1
66 2013-07-11T19:52:41 *** ashutosh1461
67 2013-07-11T19:56:27 * ThomasWaldmann looks at ashutosh_singla1's changes
68 2013-07-11T20:17:18 *** ashutosh_singla1
69 2013-07-11T20:26:22 * ananasova sees that after the latest sync there are some pep8 test failures in the MoinMoin/config/default.py... and not only
70 2013-07-11T20:33:29 *** BlindBanana
71 2013-07-11T20:35:10 <ashutosh1461> ThomasWaldmann, are you done ?
72 2013-07-11T20:36:41 <ananasova> waldi: please review when free - https://codereview.appspot.com/11139044/
73 2013-07-11T20:36:45 <sharky93> ThomasWaldmann: if you could merge the patch for broken modify that would be great :) .. its annoying :)
74 2013-07-11T20:37:52 *** BlindBanana
75 2013-07-11T20:38:45 <ThomasWaldmann> ashutosh1461: not yet, got distracted...
76 2013-07-11T20:41:03 <ThomasWaldmann> sharky93: done
77 2013-07-11T21:16:20 *** dave_largo
78 2013-07-11T21:39:36 <ThomasWaldmann> (sorry for delays, but there is a rather interesting discussion on tv with jacob appelbaum, lobo and some other people about nsa, surveillance, etc.)
79 2013-07-11T21:48:53 <ananasova> wow, you have a tv
80 2013-07-11T21:52:34 <ThomasWaldmann> well, the interesting stuff on tv is rather rare ...
81 2013-07-11T22:43:53 * ashutosh1461 in replying to tw comments on https://codereview.appspot.com/10842043/
82 2013-07-11T22:52:27 <ashutosh1461> ThomasWaldmann, replied here https://codereview.appspot.com/10842043/
83 2013-07-11T23:01:55 <ThomasWaldmann> ashutosh1461: replied there
84 2013-07-11T23:13:29 <ashutosh1461> ThomasWaldmann, Thanks and please review https://codereview.appspot.com/10915044/ and see comments @ https://codereview.appspot.com/10973043/ when free :)
85 2013-07-11T23:25:31 * sharky93 goes to bed after a tiring 24hr journey..
86 2013-07-11T23:25:33 <sharky93> gn
87 2013-07-11T23:38:24 <ThomasWaldmann> ashutosh1461: done
88 2013-07-11T23:38:26 <ThomasWaldmann> and gn :)
89 2013-07-11T23:48:19 <ashutosh1461> gn
90 2013-07-11T23:48:19 *** ashutosh1461
91