1 2010-12-12T00:15:52 <GCIBot> ClaimRequested: http://www.google-melange.com/gci/task/show/google/gci2010/moinmoin/t129037964696 (find a failing test, determine problem, fix it (difficulty: easy) 3/3)
2 2010-12-12T00:17:37 *** Roboraider
3 2010-12-12T00:47:57 <GCIBot> ClaimRequested: http://www.google-melange.com/gci/task/show/google/gci2010/moinmoin/t129037964696 (find a failing test, determine problem, fix it (difficulty: easy) 3/3)
4 2010-12-12T01:11:40 *** m4k3r
5 2010-12-12T01:17:55 *** Roboraider
6 2010-12-12T01:18:07 <GCIBot> ClaimRequested: http://www.google-melange.com/gci/task/show/google/gci2010/moinmoin/t129037964696 (find a failing test, determine problem, fix it (difficulty: easy) 3/3)
7 2010-12-12T01:32:04 *** grzywacz
8 2010-12-12T01:34:57 <ThomasWaldmann> re
9 2010-12-12T02:39:17 *** rfw
10 2010-12-12T02:39:30 *** rfw
11 2010-12-12T02:39:30 *** rfw
12 2010-12-12T02:58:58 <jbills> dreimark: The tabs would be because I copied it from code that was already indented
13 2010-12-12T05:43:55 *** jbills
14 2010-12-12T07:50:11 <Roboraider> is anyone here?
15 2010-12-12T08:39:10 *** Roboraider
16 2010-12-12T08:47:28 *** kursor
17 2010-12-12T08:49:14 *** kursor
18 2010-12-12T08:49:46 *** kursor
19 2010-12-12T08:57:23 *** m4k3r
20 2010-12-12T09:34:57 *** rfw
21 2010-12-12T09:43:32 *** rfw
22 2010-12-12T10:10:22 <m4k3r> How can I obtain values setted with http://my.url/index.html?foo=bar&baz=spam (in this case, foo and baz)?
23 2010-12-12T10:23:24 *** rfw
24 2010-12-12T10:32:55 *** grzywacz
25 2010-12-12T11:02:14 <dreimark> m4k3r: key = request.form.get('key', '')
26 2010-12-12T11:02:34 <m4k3r> dreimark: thanks!
27 2010-12-12T11:05:01 * dreimark breakfas
28 2010-12-12T11:05:02 <dreimark> t
29 2010-12-12T11:10:46 <ThomasWaldmann> moin
30 2010-12-12T11:11:36 <ThomasWaldmann> m4k3r: request.args
31 2010-12-12T11:30:15 <m4k3r> Oh Jesus. Seems that ContinueLogin() is called _before_ oid.after_login. So the authentication method is called before the user really logged in with his openid. This means flask-openid is incompatible with MoinMoin.
32 2010-12-12T11:32:57 <m4k3r> Since my task deadline is tomorrow, I'm going to leave it. I wish I had more time, but school took me the most of this week. I will write a comment explaining why flask-openid is incompatible with MoinMoin, and then submit a patch with my current work.
33 2010-12-12T11:41:45 <ThomasWaldmann> you could have a look at how moin 1.9 did it
34 2010-12-12T11:45:40 <m4k3r> ThomasWaldmann: yeah, but I have one day left :\
35 2010-12-12T11:46:36 <ThomasWaldmann> we can extend the deadline, if you like
36 2010-12-12T11:50:03 <m4k3r> Ok, let me take a look at moinmoin1.9's openid auth. But previously I'll write a comment about my current work.
37 2010-12-12T11:53:39 <dreimark> m4k3r: we can extend the time at the moment we do a review
38 2010-12-12T12:15:07 <m4k3r> dreimark: I've submitted a snippet of MoinMoin's logging output proving what I said, and the link to the last changest in my bitbucket repo.
39 2010-12-12T12:28:43 *** skylamer`
40 2010-12-12T12:35:04 <dreimark> aa_: can you have a look too ^^
41 2010-12-12T12:35:11 <dreimark> if possible
42 2010-12-12T13:11:22 *** greg_f
43 2010-12-12T13:24:33 *** Samm
44 2010-12-12T13:35:53 *** Samm
45 2010-12-12T13:38:53 *** Samm
46 2010-12-12T15:02:20 *** kursor
47 2010-12-12T15:15:25 <aa_> dreimark: sure, at what?
48 2010-12-12T15:35:34 *** Samm
49 2010-12-12T16:19:21 *** maker__
50 2010-12-12T16:21:48 *** m4k3r
51 2010-12-12T16:43:22 <GCIBot> ClaimRequested: http://www.google-melange.com/gci/task/show/google/gci2010/moinmoin/t129202088118 (update moin 1.9 translation 1/3)
52 2010-12-12T17:13:34 <GCIBot> ClaimRequested: http://www.google-melange.com/gci/task/show/google/gci2010/moinmoin/t129202088118 (update moin 1.9 translation 1/3)
53 2010-12-12T17:37:17 <skylamer`> U can write me when U come Thomas
54 2010-12-12T17:41:27 *** maker__
55 2010-12-12T17:43:05 *** maker__
56 2010-12-12T17:43:49 <TheSheep> U is not a word
57 2010-12-12T17:44:00 <GCIBot> ClaimRequested: http://www.google-melange.com/gci/task/show/google/gci2010/moinmoin/t129202088118 (update moin 1.9 translation 1/3)
58 2010-12-12T17:46:58 *** maker__
59 2010-12-12T17:47:34 *** maker__
60 2010-12-12T18:16:24 *** greg_f
61 2010-12-12T18:22:49 <skylamer`> Hello again ThomasWaldmann!
62 2010-12-12T18:22:55 <ThomasWaldmann> skylamer`: and just ask what you want to really know
63 2010-12-12T18:23:06 * ThomasWaldmann is mostly busy currently
64 2010-12-12T18:23:09 <skylamer`> want to show u some pics
65 2010-12-12T18:23:19 <ThomasWaldmann> but there are other mentors here who can look at it also
66 2010-12-12T18:23:47 <skylamer`> i dont know
67 2010-12-12T18:23:50 <skylamer`> who's the others?
68 2010-12-12T18:23:53 *** jbills
69 2010-12-12T18:34:16 *** KenuR
70 2010-12-12T18:34:42 <KenuR> hello, I claimed this: http://www.google-melange.com/gci/task/show/google/gci2010/moinmoin/t129202088118
71 2010-12-12T18:34:52 <KenuR> on gci and i have a question
72 2010-12-12T18:35:10 <KenuR> how do I edit the page if it is immutable?
73 2010-12-12T18:36:20 <TheSheep> obviously you don't
74 2010-12-12T18:36:44 <KenuR> so how do I fix the translation?
75 2010-12-12T18:37:02 <TheSheep> KenuR: http://moinmo.in/EditingOnMoinMaster
76 2010-12-12T18:41:29 *** jbills
77 2010-12-12T18:45:01 *** jbills
78 2010-12-12T18:45:33 <CIA-62> Thomas Waldmann <tw AT waldmann-edv DOT de> default * 5750:021c1f6d3272 1.9/MoinMoin/ (3 files in 3 dirs): (log message trimmed)
79 2010-12-12T18:45:33 <CIA-62> experimental queued indexing support to work around memory leak
80 2010-12-12T18:45:33 <CIA-62> you can use this like this:
81 2010-12-12T18:45:33 <CIA-62> #!/bin/sh
82 2010-12-12T18:45:33 <CIA-62> # put all pages into indexer queue:
83 2010-12-12T18:45:34 <CIA-62> python MoinMoin/script/moin.py index build --mode=makequeue
84 2010-12-12T18:45:34 <CIA-62> # take <count> pages out of the indexer queue and index them,
85 2010-12-12T18:50:06 <KenuR> it says that I have to add myself to MoinPagesEditorGroup, so should I just write myself in?
86 2010-12-12T18:51:49 *** maker__
87 2010-12-12T18:52:07 *** maker__
88 2010-12-12T18:56:29 <skylamer`> anyone heere?
89 2010-12-12T18:59:36 <TheSheep> skylamer`: no
90 2010-12-12T18:59:43 <skylamer`> ok
91 2010-12-12T19:02:34 *** jbills
92 2010-12-12T19:44:14 *** KenuR
93 2010-12-12T19:50:31 *** rfw
94 2010-12-12T19:50:31 *** rfw
95 2010-12-12T20:02:12 <dreimark> re
96 2010-12-12T20:02:18 *** jbills
97 2010-12-12T20:04:05 *** m4k3r
98 2010-12-12T20:04:08 *** maker__
99 2010-12-12T20:07:16 <skylamer`> dreimark
100 2010-12-12T20:07:17 <skylamer`> hello
101 2010-12-12T20:15:33 <dreimark> moin
102 2010-12-12T20:16:48 <skylamer`> m0inmoin
103 2010-12-12T20:16:52 <skylamer`> r u mentor in g c i ?
104 2010-12-12T20:17:00 <dreimark> i am
105 2010-12-12T20:17:07 <skylamer`> ok then
106 2010-12-12T20:17:13 <skylamer`> proceed to privat ?
107 2010-12-12T20:17:20 <dreimark> no
108 2010-12-12T20:17:26 <jbills> dreimark: Will pytest work on windows?
109 2010-12-12T20:17:33 <dreimark> because then noone can read it
110 2010-12-12T20:17:42 <dreimark> jbills: it works
111 2010-12-12T20:17:50 <jbills> thanks
112 2010-12-12T20:17:55 <dreimark> i have no idea why it shouldn't
113 2010-12-12T20:18:04 <dreimark> ronny: do you know ^^
114 2010-12-12T20:19:55 <jbills> dreimark: I does not work for me
115 2010-12-12T20:21:34 <dreimark> hmm, http://agiletesting.blogspot.com/2005/01/python-unit-testing-part-3-pytest-tool.html
116 2010-12-12T20:21:46 <dreimark> add D:\Python25\Lib\site-packages\py\bin\win32\
117 2010-12-12T20:21:46 <dreimark> to your path,
118 2010-12-12T20:21:47 <dreimark> rename py.test.cmd to something like py_test.cmd
119 2010-12-12T20:22:01 <dreimark> from that post, i would say it runs
120 2010-12-12T20:22:11 <dreimark> ronny: may know better
121 2010-12-12T20:24:59 <ronny> jbills: i would need a better error descriptions than 'does not work'
122 2010-12-12T20:25:21 <jbills> Just could not find it found it now
123 2010-12-12T20:25:29 <ronny> jbills: in case of doubt, running the pytest file in the repo with python will run a pytest version that is known to work
124 2010-12-12T20:27:10 <dreimark> skylamer`: which task do you work on?
125 2010-12-12T20:27:50 <jbills> dreimark: THe py folder is not in site-packages. I have python 2.6
126 2010-12-12T20:28:48 <dreimark> if you can't run the one in moin-2.0-dev dir
127 2010-12-12T20:28:52 <skylamer`> for
128 2010-12-12T20:28:53 <dreimark> you need to install it
129 2010-12-12T20:29:04 <skylamer`> i tell u in priv
130 2010-12-12T20:29:17 <dreimark> skylamer`: give us the url
131 2010-12-12T20:29:20 <dreimark> here
132 2010-12-12T20:30:16 <skylamer`> http://www.google-melange.com/gci/task/show/google/gci2010/moinmoin/t129202087976
133 2010-12-12T20:30:43 <jbills> dreimark: Install what?
134 2010-12-12T20:34:02 <dreimark> jbills: http://pylib.org/
135 2010-12-12T20:34:09 <jbills> thank you
136 2010-12-12T20:34:12 <dreimark> http://pytest.org/
137 2010-12-12T20:34:24 <dreimark> runs on Posix/Windows, Python 2.4-3.2, PyPy and Jython
138 2010-12-12T20:34:46 <dreimark> 2.0 may be a bit too new
139 2010-12-12T20:35:11 <dreimark> jbills: try something below e.g. 1.3 - 1.4
140 2010-12-12T20:35:58 <dreimark> skylamer`: the moin_h_bg is has nice edges but i am not sure if transparent is a very good idea
141 2010-12-12T20:36:06 <dreimark> s/is//
142 2010-12-12T20:38:47 <dreimark> skylamer`: the background of the TableOfContents in MoinMoin.png should also change
143 2010-12-12T20:38:53 <dreimark> it stays grey now
144 2010-12-12T20:39:39 <skylamer`> yes its grey and above have radial lighting
145 2010-12-12T20:40:59 <dreimark> the clouds above should then maybe some thunderstorm clouds
146 2010-12-12T20:41:09 *** Roboraider
147 2010-12-12T20:41:36 <skylamer`> :D)
148 2010-12-12T20:41:40 <dreimark> you have not to keep those, you are free to change them too
149 2010-12-12T20:50:45 <jbills> when I try to easy install pytest it gives me this error:http://paste.pocoo.org/show/304030/
150 2010-12-12T20:57:20 <Roboraider> dreimark: could you please look at my explanation for my fix? http://paste.pocoo.org/show/304034/
151 2010-12-12T20:57:50 <Roboraider> heres the task that I'm solving: http://www.google-melange.com/gci/task/show/google/gci2010/moinmoin/t129037964696#c11001
152 2010-12-12T20:59:21 <dreimark> jbills: did you tried -U
153 2010-12-12T20:59:30 <jbills> yes
154 2010-12-12T20:59:40 <jbills> none worked
155 2010-12-12T21:01:16 <ronny> jbills: please try python pytest in the moin checkout
156 2010-12-12T21:01:19 <dreimark> give me the line you used
157 2010-12-12T21:01:26 *** skylamer`
158 2010-12-12T21:01:41 <ronny> (that file is a single file version of pytest which is known to work)
159 2010-12-12T21:02:43 <dreimark> Roboraider: this is not patch
160 2010-12-12T21:02:54 <Roboraider> Yeah
161 2010-12-12T21:02:55 <jbills> C:\Python26\Scripts\easy_install install pytest I have also tried it with -U
162 2010-12-12T21:03:13 <Roboraider> it is my explanation for my fix
163 2010-12-12T21:03:19 <dreimark> i don't like to manually compare
164 2010-12-12T21:04:12 <dreimark> Roboraider: why do you used backend._index?
165 2010-12-12T21:04:19 <dreimark> an internal method
166 2010-12-12T21:04:35 <dreimark> ThomasWaldmann: wants that answered
167 2010-12-12T21:05:24 <Roboraider> I used backend._index because I'm trying to avoid using the history() found in indexing.py
168 2010-12-12T21:05:52 <Roboraider> The get_item("first") is where it is failing
169 2010-12-12T21:07:02 <ThomasWaldmann> the goal is not to tweak the test somehow so it does not fail, but to look what the test is intended to test and find out why exactly that does not work
170 2010-12-12T21:07:34 <ThomasWaldmann> and then fix the root cause at the right place (which may or may not be the test)
171 2010-12-12T21:08:13 <Roboraider> I've found it out, but I'm unsure of changing the history(name) routine in indexing.py because of its potential impact on the rest of the program.
172 2010-12-12T21:09:57 <Roboraider> Should I change the history routine in indexing.py?
173 2010-12-12T21:11:53 <ThomasWaldmann> i can't look at details right now, but you could write a precise diagnosis of the problem
174 2010-12-12T21:13:38 <dreimark> jbills: may be windows needs an extension for the file pytest try if you can run it as pytest.cmd
175 2010-12-12T21:14:07 <jbills> Its a problem with easy_install
176 2010-12-12T21:14:23 <dreimark> this is a different problem
177 2010-12-12T21:14:26 *** rfw
178 2010-12-12T21:14:32 <dreimark> there is a pytest in the moin source
179 2010-12-12T21:14:40 <dreimark> try to get that running first
180 2010-12-12T21:14:44 *** rfw
181 2010-12-12T21:17:26 <dreimark> jbills: on the python channel one mentioned to use pip
182 2010-12-12T21:17:42 <dreimark> pip install py.test
183 2010-12-12T21:17:47 <jbills> Where is pip stored?
184 2010-12-12T21:23:15 <dreimark> jbills: easy_install -U pytest <- was that the line you tried ?
185 2010-12-12T21:23:40 <dreimark> because the error in your pastebin means name not found
186 2010-12-12T21:24:04 <jbills> I have tried that. this error: Processing pytest
187 2010-12-12T21:24:04 <jbills> error: Not a recognized archive type: pytest
188 2010-12-12T21:28:07 <dreimark> jbills: look at the one in the moin top level dir
189 2010-12-12T21:28:19 <dreimark> try to run that, there is no installation needed for
190 2010-12-12T21:28:51 <dreimark> this one http://hg.moinmo.in/moin/2.0-dev/file/fda490a80728/pytest
191 2010-12-12T21:29:18 <dreimark> go to the dir try python pytest
192 2010-12-12T21:29:34 <jbills> thanks
193 2010-12-12T21:29:47 <dreimark> you have that file locally on the same place
194 2010-12-12T21:30:11 <dreimark> http://hg.moinmo.in/moin/2.0-dev/file/fda490a80728/
195 2010-12-12T21:36:02 *** Roboraider
196 2010-12-12T21:40:19 <dreimark> jbills: if this didn't work download the archiv.zip of pytest and install it directly from that archiv by easy_install archiv.zip
197 2010-12-12T21:40:49 <jbills> dreimark: i got this error from pytest ERROR: file not found: C:\Users\josiah\Downloads\moin-main\moin-2.0-dev - Copy\M
198 2010-12-12T21:40:50 <jbills> oinMoin\_test\test_sourcecode.py
199 2010-12-12T21:42:36 <dreimark> does it when you run python pytest run
200 2010-12-12T21:42:41 <dreimark> it runs then all tests
201 2010-12-12T21:45:08 <dreimark> jbills: you can set -k to test_sourcecode if it runs tests
202 2010-12-12T21:47:35 *** Samm
203 2010-12-12T21:49:53 <jbills> dreimark: What does this mean: W191 indentation contains tabs
204 2010-12-12T21:51:46 <TheSheep> jbills: you used tab characters where you should have used spaces
205 2010-12-12T21:52:10 <TheSheep> jbills: configure your editor to use spaces
206 2010-12-12T21:52:16 <jbills> ok
207 2010-12-12T21:52:20 <jbills> Idle?
208 2010-12-12T21:53:38 <jbills> How would you do this with idle?
209 2010-12-12T21:54:17 <TheSheep> how would I know?
210 2010-12-12T21:54:30 <TheSheep> I can tell you how to do it with vim
211 2010-12-12T21:57:18 <ronny> jbills: you did call easy_install pytest in the moinmoin checkout, didnt you? (cause then the file named pytest in there confuses easy_install
212 2010-12-12T21:57:39 <ronny> currently the best way to do a system install of pytest that works with moinmoin is easy_install py==1.3
213 2010-12-12T21:57:42 <jbills> ronny: I just used moinmoins eay install
214 2010-12-12T21:57:54 <jbills> I mean pytest
215 2010-12-12T21:58:04 <ronny> yes, that one works well
216 2010-12-12T21:58:12 <ronny> (its a single file version of py.test 1.3.4)
217 2010-12-12T21:58:40 *** m4k3r
218 2010-12-12T22:18:03 <ThomasWaldmann> re
219 2010-12-12T22:41:04 <jbills> dreimark: Ok fixed all the errors in MY code. There is no difference. My output still has paragraph marks in it.
220 2010-12-12T22:43:30 <ThomasWaldmann> what are "paragraphs marks"?
221 2010-12-12T22:44:21 <jbills> ΒΆ
222 2010-12-12T22:45:54 <ThomasWaldmann> white question marks on black background?
223 2010-12-12T22:46:21 <ThomasWaldmann> and what exactly is the problem?
224 2010-12-12T22:53:46 <jbills> Let me show you a screenshot. http://dropmocks.com/mM-oR
225 2010-12-12T22:59:48 <ThomasWaldmann> these are the mouseover anchors, they should not be in the TOC
226 2010-12-12T23:00:08 <jbills> huh?
227 2010-12-12T23:01:26 <ThomasWaldmann> at least they look like that
228 2010-12-12T23:01:48 <jbills> the paragraph symbols?
229 2010-12-12T23:01:53 <ThomasWaldmann> yes
230 2010-12-12T23:02:29 <jbills> what is it in my source code that causes them?
231 2010-12-12T23:03:41 <jbills> http://paste.pocoo.org/show/304077/
232 2010-12-12T23:08:09 <ThomasWaldmann> text = ''.join(elem.itertext()) i guess it is this
233 2010-12-12T23:08:27 <jbills> but how.
234 2010-12-12T23:08:59 <jbills> It must be a problem with the version I downloaded. It worked fine in a older version of moin2
235 2010-12-12T23:09:12 <ThomasWaldmann> because the headline has some text and an anchor link (and the text of the anchor link is that paragraph symbol)
236 2010-12-12T23:09:58 <ThomasWaldmann> introducing those mouseover anchor links has been another gci task
237 2010-12-12T23:10:15 <jbills> so those are supposed to be there?
238 2010-12-12T23:10:25 <ThomasWaldmann> if you just want the headline text, don't user itertext
239 2010-12-12T23:10:51 <jbills> just elem?
240 2010-12-12T23:11:27 <ThomasWaldmann> find out :) maybe elem? elem[0]?
241 2010-12-12T23:11:43 <jbills> ok
242 2010-12-12T23:15:17 <jbills> It was elem[0]
243 2010-12-12T23:15:20 <jbills> :)
244 2010-12-12T23:15:25 <jbills> YAY!!!!!
245 2010-12-12T23:16:29 <ThomasWaldmann> please put a comment there why it is elem[0]
246 2010-12-12T23:28:13 <dreimark> jbills: count = count+1
247 2010-12-12T23:28:19 <dreimark> not PEP8 conform
248 2010-12-12T23:28:32 <dreimark> and it can also be written as count += 1
249 2010-12-12T23:29:07 <dreimark> and likly if something like that is used there is a better solution
250 2010-12-12T23:29:20 <jbills> I have changed it to that now
251 2010-12-12T23:35:54 <dreimark> ThomasWaldmann: see comment at http://www.google-melange.com/gci/task/show/google/gci2010/moinmoin/t129037964696
252 2010-12-12T23:40:26 *** franklin_
253 2010-12-12T23:42:45 *** franklin
254
