1 2012-05-28T00:22:17 <spy> ThomasWaldmann: ether
2 2012-05-28T01:09:46 *** dwcramer
3 2012-05-28T01:11:01 *** spy
4 2012-05-28T03:17:21 *** dwcramer
5 2012-05-28T03:42:09 *** sudo_dirk
6 2012-05-28T03:57:53 *** sudo_dirk
7 2012-05-28T05:52:07 *** dwcramer
8 2012-05-28T06:05:10 *** dwcramer
9 2012-05-28T06:57:51 *** xiaq
10 2012-05-28T07:18:48 *** progmanos1
11 2012-05-28T07:45:24 *** dwcramer
12 2012-05-28T08:54:33 <dreimark> eSyr-ng: http://moinmo.in/MoinMoinBugs/MigrationIgnores intermap.txt
13 2012-05-28T08:56:08 <eSyr-ng> the most complicated part that there's no good solution :(
14 2012-05-28T10:46:35 *** spy
15 2012-05-28T11:15:09 *** greg_f
16 2012-05-28T12:11:55 *** jaiditya
17 2012-05-28T12:54:01 <ThomasWaldmann> jaiditya: please add links to your calendar pages for your review submissions
18 2012-05-28T12:54:24 <ThomasWaldmann> (i did not see any "review submission" here)
19 2012-05-28T13:04:30 <ThomasWaldmann> breton: xiaq: MaikuMori: add a repository link to your project wiki page, please.
20 2012-05-28T13:06:05 <ThomasWaldmann> xiaq: MaikuMori: i do not see ANY diary updates for your projects, what's up?
21 2012-05-28T13:06:24 <MaikuMori> I hope to fix that this evening.
22 2012-05-28T13:06:51 <ThomasWaldmann> that stuff has to be done every work day, not some days later
23 2012-05-28T14:22:41 <jaiditya> ThomasWaldmann: links to changesets
24 2012-05-28T14:22:57 <jaiditya> changeset#1 http://codereview.appspot.com/6258046/
25 2012-05-28T14:22:58 <jaiditya> changeset#2 http://codereview.appspot.com/6250055/
26 2012-05-28T14:22:58 <jaiditya> changeset#3 http://codereview.appspot.com/6255051/
27 2012-05-28T14:22:58 <jaiditya> changeset#4 http://codereview.appspot.com/6250062/
28 2012-05-28T14:22:58 <jaiditya> changeset#5 http://codereview.appspot.com/6256053/
29 2012-05-28T14:22:58 <jaiditya> changeset#6 http://codereview.appspot.com/6259045/
30 2012-05-28T14:22:58 <jaiditya> changeset#7 http://codereview.appspot.com/6243062/
31 2012-05-28T14:22:58 <jaiditya> changeset#8 http://codereview.appspot.com/6243063/
32 2012-05-28T14:22:59 <jaiditya> changeset#9 http://codereview.appspot.com/6242063/
33 2012-05-28T14:30:16 <ronny> jaiditya: if its that many you might want to use a pastebin
34 2012-05-28T14:36:56 <jaiditya> ronny: usually they are not many at once, I upload them one by one as I proceed.
35 2012-05-28T14:37:06 <eSyr-ng> ThomasWaldmann: http://codereview.appspot.com/6256061
36 2012-05-28T14:59:27 *** DasIch_
37 2012-05-28T15:02:08 *** DasIch
38 2012-05-28T15:02:08 *** DasIch_
39 2012-05-28T15:09:22 *** progmanos1
40 2012-05-28T15:32:08 *** kanha_
41 2012-05-28T15:32:45 *** kanha
42 2012-05-28T15:43:43 *** progmanos1
43 2012-05-28T15:56:51 *** brunomartin
44 2012-05-28T16:12:05 *** dwcramer
45 2012-05-28T16:56:33 <dreimark> jaiditya: he asked for adding to the calendar pages ;)
46 2012-05-28T17:47:05 *** dwcramer
47 2012-05-28T18:20:48 <ThomasWaldmann> eSyr-ng: review done
48 2012-05-28T18:22:34 <eSyr-ng> ThomasWaldmann: well, copy_underlay_page is first what i tried and it didn't work because underlay page has rev 99999999 and this evaluates exists to False and attachment is added to underlay page.
49 2012-05-28T18:23:22 <eSyr-ng> maybe we should fix rev for pages copied from underlay somehow.
50 2012-05-28T18:25:16 *** dwcramer
51 2012-05-28T18:31:28 <ThomasWaldmann> jaiditya: you are use codereview in a suboptimal way
52 2012-05-28T18:31:39 <ThomasWaldmann> using*
53 2012-05-28T18:32:01 <ThomasWaldmann> waldi: did you already tell that?
54 2012-05-28T18:36:57 <ThomasWaldmann> usually, if review topic is still the same and you are just uploading the next iteration of it, you use -i ISSUE and ISSUE is the same issue number as the previous iteration
55 2012-05-28T18:46:28 <ThomasWaldmann> did somebody see "inducer" here?
56 2012-05-28T18:46:44 <jaiditya> ThomasWaldmann: I didn't knew that.
57 2012-05-28T18:48:06 * eSyr-ng thinks whether should he mention [[CodeReview]] or not
58 2012-05-28T18:50:40 <ThomasWaldmann> jaiditya: now you know :)
59 2012-05-28T18:50:56 <jaiditya> yup :)
60 2012-05-28T18:51:06 <ThomasWaldmann> also, try to avoid too many too little codereviews.
61 2012-05-28T18:52:00 <jaiditya> ok.
62 2012-05-28T18:52:08 <ThomasWaldmann> try to completely do one simple thing. e.g. if you work on replacing colour literals by names, try to completely do that, so you don't need multiple codereviews just for that.
63 2012-05-28T18:54:03 <ThomasWaldmann> this also applies to commits. a "clean" commit is doing one precise thing, completely, avoiding mixing with other stuff that one usually finds while doing that (take notes somewhere).
64 2012-05-28T18:55:09 <jaiditya> ok. Is there any by which now I can combine my commits ?
65 2012-05-28T18:56:02 <ThomasWaldmann> i usually use patches and edit them, but there are also more sophisticated tools, i think esyr can tell about that.
66 2012-05-28T18:57:24 <ThomasWaldmann> (and I usually try to really focus on ONE thing and doing it completely, so such "patch surgery" is reduced to a minimum)
67 2012-05-28T18:57:58 <jaiditya> ok I'll ask him. N when's the next meeting. I havn't seen any updates about it till now ?
68 2012-05-28T18:58:39 * ThomasWaldmann looks at the doodle
69 2012-05-28T18:59:46 <ThomasWaldmann> 5 people failed to update their availability (or didn't put "upd." so ony can clearly see that they did)
70 2012-05-28T19:00:41 <ThomasWaldmann> breton: xiaq: spy: waldi: eSyr-ng: DasIch: ^^ (6 even)
71 2012-05-28T19:02:14 <ThomasWaldmann> currently it looks like sat 13:00 utc, but as long as we are not finished with updating, that might change
72 2012-05-28T19:02:27 <jaiditya> ok.
73 2012-05-28T19:04:35 <spy> ThomasWaldmann: updated as soon as you said it early
74 2012-05-28T19:06:40 *** dwcramer
75 2012-05-28T19:09:16 <ThomasWaldmann> ah, ok
76 2012-05-28T19:13:26 <ThomasWaldmann> i got a pull request for a markdown converter btw (which seems to work by calling some existing markdown-to-html conversion code and then making moin-dom node from the html nodes)
77 2012-05-28T19:13:31 <ThomasWaldmann> waldi: ^^
78 2012-05-28T19:16:33 <waldi> well
79 2012-05-28T19:23:44 <DasIch> 13utc on a saturday that is early
80 2012-05-28T19:35:02 <ThomasWaldmann> :D
81 2012-05-28T19:36:07 * ThomasWaldmann set MOINLOGGINGCONF to a custom logging_local.conf
82 2012-05-28T19:42:56 *** greg_f
83 2012-05-28T19:50:20 <ThomasWaldmann> oops, unexpected "<!DOCTYPE html>" in the middle of the html output...
84 2012-05-28T19:55:04 *** inducer
85 2012-05-28T19:55:49 <inducer> hi there, i'm the guy that submitted the markdown parser, https://bitbucket.org/thomaswaldmann/moin-2.0/pull-request/36/add-markdown-input-parser
86 2012-05-28T19:56:21 <inducer> thomas asked me to come here to get feedback
87 2012-05-28T19:56:47 <ThomasWaldmann> ah, hi inducer
88 2012-05-28T19:57:00 <inducer> hi
89 2012-05-28T19:57:13 <inducer> right, so--what's the story?
90 2012-05-28T19:57:14 <inducer> :)
91 2012-05-28T19:57:18 <ThomasWaldmann> did you work with codereview yet?
92 2012-05-28T19:57:42 <ThomasWaldmann> codereview.appspot.com
93 2012-05-28T19:57:43 <inducer> codereview-the-site, no
94 2012-05-28T19:57:55 <inducer> let me check
95 2012-05-28T19:58:15 <ThomasWaldmann> pastebin on steroids :) you just need a google account and upload.py
96 2012-05-28T19:59:08 <inducer> gah, google sign-in
97 2012-05-28T20:01:39 *** dwcramer
98 2012-05-28T20:02:54 <ThomasWaldmann> no google account?
99 2012-05-28T20:07:44 <ThomasWaldmann> btw, http://mathema.tician.de/ is down?
100 2012-05-28T20:09:17 <inducer> apt-get thought it to be wise to shut down apache while upgrading stuff
101 2012-05-28T20:09:18 <inducer> lovely
102 2012-05-28T20:09:25 <inducer> thanks for pointing this out
103 2012-05-28T20:09:27 <inducer> :)
104 2012-05-28T20:09:36 <inducer> should be back up momentairly
105 2012-05-28T20:10:43 <ThomasWaldmann> ;)
106 2012-05-28T20:11:19 <inducer> "upload.py -- -r 1361 -r 1362" says "No output from ['hg', 'status', '-C', '--rev', '739d6964866a', 'MoinMoin/constants/contenttypes.py']
107 2012-05-28T20:15:26 * ThomasWaldmann usually just invokes upload.py (before committing)
108 2012-05-28T20:15:43 <inducer> the web interface says "patch set contains no recognizable patches" about http://tiker.net/tmp/moin2-add-markdown.diff
109 2012-05-28T20:15:54 <inducer> looks perfectly recognizable to me
110 2012-05-28T20:16:03 <inducer> this is totally going great. :)
111 2012-05-28T20:16:41 <inducer> good thing i'm not about to have my memorial day mood be spoiled by computers acting up. :)
112 2012-05-28T20:17:11 *** dennda__
113 2012-05-28T20:17:34 <inducer> that whole "before committing" is a bit too late now...
114 2012-05-28T20:17:52 <ThomasWaldmann> it also works afterwards, but you have to find out how :)
115 2012-05-28T20:17:57 *** dennda_
116 2012-05-28T20:18:33 <ThomasWaldmann> great, moin says audio/wave, firefox wants audio/wav, and /etc/mime.types has audio/x-wav
117 2012-05-28T20:34:23 <ThomasWaldmann> hmm, somehow chrome and ff do not play wav.
118 2012-05-28T20:36:02 <eSyr-ng> are they should?
119 2012-05-28T20:36:37 * eSyr-ng thought the only thing that can be played is vorbis
120 2012-05-28T20:37:51 <ThomasWaldmann> somehow i thought wav is maybe also simple and old enough to get played, but maybe i am wrong...
121 2012-05-28T20:38:11 <ThomasWaldmann> also, ff accept header has audio/wav and audio/*
122 2012-05-28T20:38:30 <inducer> https://codereview.appspot.com/6243064
123 2012-05-28T20:38:34 <inducer> only took half an hour
124 2012-05-28T20:38:37 <inducer> god, i hate hg
125 2012-05-28T20:38:41 <ThomasWaldmann> \o/
126 2012-05-28T20:38:55 <eSyr-ng> hehe
127 2012-05-28T20:39:01 * ThomasWaldmann likes hg :P
128 2012-05-28T20:39:20 <inducer> that's too bad. git's going to put you out of business. better get over it now. :)
129 2012-05-28T20:40:44 <eSyr-ng> ThomasWaldmann: well, at first, try to repeat git add -i - patch - edit in hg :)
130 2012-05-28T20:41:28 <inducer> (btw, i got this to work by un-committing, applying the patch, and then running upload.py)
131 2012-05-28T20:43:13 <inducer> while i'm here: i recall reading somewhere that moin2 has some widget that makes it semi-usable as a blog (with an rss feed, perhaps?). where do i need to look?
132 2012-05-28T20:44:15 <inducer> and another one: why don't you guys simply ditch the current 'modernized' atrocity and simply use something simple based on the twitter bootstrap as a theme?
133 2012-05-28T20:44:17 <ThomasWaldmann> inducer: that is a SOC 2012 project :) and has just begun.
134 2012-05-28T20:44:25 <inducer> oh, good
135 2012-05-28T20:44:35 <inducer> can't wait to nuke my drupal install
136 2012-05-28T20:44:41 <ThomasWaldmann> and there is also a theme project and a rudimentary foobar theme.
137 2012-05-28T20:45:47 <ThomasWaldmann> (and in general: we need more people caring for design/theme stuff who are good at such stuff)
138 2012-05-28T20:46:35 <inducer> that's the beauty of bootstrap--it looks acceptable even without me caring too damn much. :)
139 2012-05-28T20:53:56 <ThomasWaldmann> review done
140 2012-05-28T20:54:34 <ThomasWaldmann> btw, use -i SAMEID if you update the codereview item
141 2012-05-28T21:02:33 <inducer> k, i'll reply in a bit
142 2012-05-28T21:43:15 *** sudo_dirk
143 2012-05-28T22:17:44 <inducer> btw, how insane would it be to put my public wiki onto moin2 right now? (as you know, i'm ok with doing a bit of hacking--but i'd need it to be more robust than, say, a house of cards in high wind. also, is the storage format guaranteed forward-compatible?)
144 2012-05-28T22:21:20 <ronny> inducer: i'd wait for the merge/branch + network sync stuff to be finished (for metadata)
145 2012-05-28T22:22:09 <ThomasWaldmann> it's not robust yet
146 2012-05-28T22:22:34 <ThomasWaldmann> so rather play with it locally
147 2012-05-28T22:23:02 <inducer> ok, thanks for the assessment'
148 2012-05-28T22:23:05 <ronny> ThomasWaldmann: imm under the impression, that the main item to finish is the branch/merge thing
149 2012-05-28T22:23:22 <ronny> or are there more items tha would affect data formats
150 2012-05-28T22:23:47 <ThomasWaldmann> the namespaces branch is also waiting for finishing/merging (since ~6m)
151 2012-05-28T22:24:00 <ronny> i see
152 2012-05-28T22:28:11 <eSyr-ng> ThomasWaldmann: well, it's still not clear for me, what problem with mount-based storage.
153 2012-05-28T22:48:03 <ThomasWaldmann> namespaces is simpler and can intergrate better with interwiki
154 2012-05-28T22:49:45 <ThomasWaldmann> but that was not the problem, just a side effect
155 2012-05-28T22:51:09 <ThomasWaldmann> in that branch, we also have multiple names per revision
156 2012-05-28T22:51:41 <ThomasWaldmann> so, basically, if storage place depends on the name, it gets rather complicated. same for acl, btw (hier. acl)
157 2012-05-28T22:53:15 <eSyr-ng> well, in POSIX acl is also hierarchical.
158 2012-05-28T22:54:49 <ThomasWaldmann> but directories there are real hierarchies, we don't do that in moin
159 2012-05-28T22:56:16 *** MattMaker
160 2012-05-28T23:08:07 <jaiditya> ThomasWaldmann: why same two logos of moinmoin present in global static's logos folder ? (named moinmoin.png and moinmoin_alpha.png)
161 2012-05-28T23:08:40 <jaiditya> moinmoin.png has a bad unnecessary white border with it
162 2012-05-28T23:11:00 <ThomasWaldmann> moinmoin_alpha.png: PNG image data, 64 x 64, 8-bit/color RGBA, non-interlaced
163 2012-05-28T23:11:00 <ThomasWaldmann> moinmoin.png: PNG image data, 64 x 64, 8-bit colormap, non-interlaced
164 2012-05-28T23:11:41 <ThomasWaldmann> iirc there were browser issues with the _alpha one
165 2012-05-28T23:15:50 <jaiditya> ok...
166 2012-05-28T23:17:55 <eSyr-ng> ThomasWaldmann: the last browser which had issues with png alpha was ie7 iirc.
167 2012-05-28T23:19:02 <eSyr-ng> (and most png alpha channgel issues in ie can be fixed with setting of proper DX filter)
168 2012-05-28T23:19:25 <eSyr-ng> but this is waaay too 2000-ies.
169 2012-05-28T23:31:38 <ThomasWaldmann> DX?
170 2012-05-28T23:36:20 <jaiditya> DirectX filter ?
171 2012-05-28T23:38:49 <jaiditya> DX filters http://msdn.microsoft.com/en-us/library/ie/hh801215(v=vs.85).aspx
172 2012-05-28T23:44:31 <CIA-146> Thomas Waldmann <tw AT waldmann-edv DOT de> default * 1362:4078dfe5d337 2.0/MoinMoin/converter/ (_wiki_macro.py macro.py moinwiki_in.py pygments_in.py): macro/parser/pygments converter: improve debug logging
173 2012-05-28T23:57:22 *** jaiditya
174