1 2013-02-12T00:42:43 *** RogerHaase
2 2013-02-12T00:46:24 <moinBot> http://hg.moinmo.in/moin/2.0/rev/10e20c8a2c29 2013-02-12 Thomas Waldmann <tw AT waldmann-edv DOT de> replace literal strings by constants from constants.keys
3 2013-02-12T01:15:27 <moinBot> http://hg.moinmo.in/moin/2.0/rev/ff71ef20d363 2013-02-12 Thomas Waldmann <tw AT waldmann-edv DOT de> replace more string literals by constants from constants.itemtypes/contenttypes
4 2013-02-12T02:03:05 *** ronny
5 2013-02-12T02:04:34 * ThomasWaldmann applied for a opensource license for MoinMoin development. The license might be for *all* "project members" if I understood it correctly (minus the people doing commercial support/development).
6 2013-02-12T02:04:55 <ThomasWaldmann> ^ for pycharm.
7 2013-02-12T02:11:09 *** xiaq
8 2013-02-12T03:12:01 *** dwcramer
9 2013-02-12T03:17:08 <xiaq> oops
10 2013-02-12T03:17:21 <xiaq> i took a look at failing tests, turns out to be all PEP8 failures
11 2013-02-12T03:18:15 <xiaq> somehow moin's pep8 specs were not respected
12 2013-02-12T03:18:25 <xiaq> ThomasWaldmann: where is the pep8 configuration located?
13 2013-02-12T03:18:55 <xiaq> oh, setup.cfg
14 2013-02-12T03:20:26 <xiaq> aha, turned out not my fault :) i was missing a changeset that fixed pep8 conf
15 2013-02-12T03:21:10 * xiaq still gets a pep8 failure after pulling
16 2013-02-12T03:21:31 <xiaq> /home/xiaq/src/moin2/MoinMoin/apps/frontend/test.py:9:1: E302 expected 2 blank lines, found 1
17 2013-02-12T03:21:34 <xiaq> @frontend.route('/+test')
18 2013-02-12T03:21:34 <moinBot> xiaq: Error: "frontend.route('/+test')" is not a valid command.
19 2013-02-12T03:21:37 <xiaq> ^
20 2013-02-12T03:30:12 <xiaq> ThomasWaldmann: new PR, plz check :)
21 2013-02-12T03:54:22 <xiaq> spy: any progress with the commenting system you started on gsoc? :)
22 2013-02-12T03:54:39 <xiaq> if not i'll attempt it
23 2013-02-12T03:55:06 <xiaq> (since it's crucial to the ticket system too)
24 2013-02-12T04:53:14 *** xiaq
25 2013-02-12T05:18:04 *** dwcramer
26 2013-02-12T05:31:57 *** dwcramer
27 2013-02-12T06:20:34 *** dwcramer
28 2013-02-12T08:05:23 *** ronny
29 2013-02-12T08:44:49 *** grzywacz
30 2013-02-12T08:49:06 <ronny> moin
31 2013-02-12T09:10:07 *** xiaq
32 2013-02-12T09:50:24 *** greg_f
33 2013-02-12T09:54:46 <ThomasWaldmann> moin
34 2013-02-12T09:56:32 <ThomasWaldmann> xiaq: maybe you could first look at open/close/... status, that's even more essential
35 2013-02-12T10:02:15 <ThomasWaldmann> http://google-opensource.blogspot.de/2013/02/flip-bits-not-burgers-google-summer-of.html
36 2013-02-12T10:03:45 <xiaq> ThomasWaldmann: already in progress, expect a CR soon :)
37 2013-02-12T10:03:59 <xiaq> ThomasWaldmann: was just about to share that gsoc2013 was announced :)
38 2013-02-12T10:14:56 <moinBot> http://hg.moinmo.in/moin/2.0/rev/490c13d072f2 2013-02-12 Cheer Xiao remove Submit widget
39 2013-02-12T10:23:33 <ThomasWaldmann> xiaq: btw, maybe don't work on completely new features right now (like commenting system)
40 2013-02-12T10:24:24 <ThomasWaldmann> such stuff can make SOC projects maybe, while fixing existing code and doing smaller stuff maybe can't
41 2013-02-12T10:34:03 <xiaq> ThomasWaldmann: ok. with namespaces branch merged is it ok to do some big cleanups on templates, themes, etc.?
42 2013-02-12T10:34:30 <xiaq> ThomasWaldmann: will i be eligible as student developer in this year's gsoc? :)
43 2013-02-12T10:35:55 <ThomasWaldmann> if you are still a student: likely. in case of doubt, read their rules. :D
44 2013-02-12T10:36:56 <ThomasWaldmann> in case not: we also need mentors ;)
45 2013-02-12T10:38:54 <xiaq> ThomasWaldmann: yes still a student. afaik some projects accept exclusively new developers, while some projects are more casual about freshmanship. seems a grey zone of gsoc rules. so i suppose it's up to you :)
46 2013-02-12T10:38:56 <ThomasWaldmann> xiaq: about doing bigger changes: there is still gae branch open (and I don't like to merge it as it requires a unreleased whoosh)
47 2013-02-12T10:40:37 <ThomasWaldmann> i guess i don't have a preference for new developers. any developer who makes a good application and looks like being able to really do it is welcome.
48 2013-02-12T10:41:13 <xiaq> ThomasWaldmann: ok. new cr: https://codereview.appspot.com/7313083
49 2013-02-12T10:43:31 <xiaq> ThomasWaldmann: and: https://codereview.appspot.com/7310085
50 2013-02-12T10:50:57 * ThomasWaldmann LOLed. Just got a mail a vamp picture and "DIE MICROSOFT" written below it. Strange: it was from a Microsoft SW Distributor.
51 2013-02-12T10:51:09 <ThomasWaldmann> mail with a*
52 2013-02-12T10:52:17 * ThomasWaldmann ... then scrolled down a little and it read "DIE MICROSOFT\nTRILOGIE".
53 2013-02-12T11:06:31 <xiaq> ThomasWaldmann: PR for previous 2 patches sent
54 2013-02-12T11:10:50 <xiaq> ThomasWaldmann: https://codereview.appspot.com/7303077 https://codereview.appspot.com/7308089/
55 2013-02-12T11:16:09 <dreimark> moin
56 2013-02-12T11:19:23 <dreimark> ThomasWaldmann: we got today some mail from carol
57 2013-02-12T11:20:00 <dreimark> ehem no yesterday
58 2013-02-12T11:24:44 *** xiaq
59 2013-02-12T11:25:12 <ThomasWaldmann> hi dreimark, see my 3rd line from today ;)
60 2013-02-12T11:32:44 *** grzywacz
61 2013-02-12T11:35:43 <moinBot> http://hg.moinmo.in/moin/2.0/rev/ff77f6d8da38 2013-02-12 Cheer Xiao substitute forms.render_button with forms.render_submit
62 2013-02-12T11:35:44 <moinBot> http://hg.moinmo.in/moin/2.0/rev/22441b6366e6 2013-02-12 Cheer Xiao improve use of is_new in Ticket.do_modify a bit
63 2013-02-12T12:17:25 *** xiaq
64 2013-02-12T12:20:33 <xiaq> ThomasWaldmann: for ticket status, i think the approach taken by github and bitbucket actually makes sense
65 2013-02-12T12:20:45 <xiaq> which seems to be the trend now :)
66 2013-02-12T12:20:57 <xiaq> i.e. they only distinguish open and closed
67 2013-02-12T12:21:59 <ThomasWaldmann> i never knew whether there was a real consensus about what new and open means
68 2013-02-12T12:22:22 <ThomasWaldmann> and yes, much can be just done by a comment
69 2013-02-12T12:22:34 <xiaq> my negative experience with status other than open/closed is that 1) one can often forget to update the status 2) they are not well-defined, and consequently 3) sometimes 2 or more values seem appropriate, sometimes no value captures current status
70 2013-02-12T12:23:00 <xiaq> so i conclude that its use is limited and brings more trouble than the problem it tries to solve (if any! :)
71 2013-02-12T12:23:05 <ThomasWaldmann> but be careful when looking at github. we've already seen stuff in their issue tracker (like tags) that doesn't really scale that way.
72 2013-02-12T12:24:22 <ThomasWaldmann> ok, so maybe drop status in same cs as improving "closed"? should it be "closed" or "open"?
73 2013-02-12T12:25:15 <xiaq> ThomasWaldmann: ah, i forgot to remove 'status' - will do that
74 2013-02-12T12:25:23 <ThomasWaldmann> xiaq: https://bitbucket.org/thomaswaldmann/moin-2.0/issue/74/mad-url-redesign-item-lookup-idea btw, any comment?
75 2013-02-12T12:25:41 <xiaq> ThomasWaldmann: i
76 2013-02-12T12:25:48 <xiaq> ThomasWaldmann: i'm in favor of "closed"
77 2013-02-12T12:26:08 * ThomasWaldmann tried yesterday to work on deleted (nameless) items, but there is no way yet, as most "addressing" depends on having a name
78 2013-02-12T12:26:42 <xiaq> "open" can be used as either a verb or adjective, "closed" doesn't suffer from this ambiguity. :)
79 2013-02-12T12:27:12 <ThomasWaldmann> :)
80 2013-02-12T12:28:07 <xiaq> it can also be convinient to match initial value with default value since None can be treated like False
81 2013-02-12T12:28:07 <ThomasWaldmann> did we already solve the "X is working on it" information for issues
82 2013-02-12T12:28:31 <ThomasWaldmann> (other than putting a comment)
83 2013-02-12T12:28:58 <xiaq> ThomasWaldmann: "Assigned To" is supposed to do that
84 2013-02-12T12:29:42 <ThomasWaldmann> ok.
85 2013-02-12T12:30:05 <ThomasWaldmann> 12:25 ThomasWaldmann$ xiaq: https://bitbucket.org/thomaswaldmann/moin-2.0/issue/74/mad-url-redesign-item-lookup-idea btw, any comment?
86 2013-02-12T12:30:32 * xiaq looks
87 2013-02-12T12:42:44 <xiaq> ThomasWaldmann: wikiname:ns1:ns2:virtns:name looks fine to me
88 2013-02-12T12:44:37 <xiaq> ThomasWaldmann: how is namespace structured - can it be nested to arbitrary levels?
89 2013-02-12T12:44:39 <ThomasWaldmann> maybe I'll try that in the namespaces branch. will likely be a rather big refactoring
90 2013-02-12T12:44:45 <ThomasWaldmann> yes
91 2013-02-12T12:45:15 <ThomasWaldmann> currently it is just a string stored into NAMESPACE metadata
92 2013-02-12T12:46:02 <xiaq> ThomasWaldmann: is there ui to place an item in a namespace? creating "a:b" just creates an item named "a:b".
93 2013-02-12T12:46:20 <xiaq> ThomasWaldmann: are namespaces preconfigured or can be created on the ui?
94 2013-02-12T12:46:30 <ThomasWaldmann> and potentially also used for routing to some specific backend
95 2013-02-12T12:56:43 <ThomasWaldmann> currently there isn't much ui at all and it has to be configured
96 2013-02-12T13:06:40 <ThomasWaldmann> first thing i'll do with it likely will be using the userprofiles namespace, to get rid of the account profiles in the standard namespace
97 2013-02-12T13:06:43 <ThomasWaldmann> brb
98 2013-02-12T13:36:47 <xiaq> ThomasWaldmann: i thought a bit and changed my mind. :) https://bitbucket.org/thomaswaldmann/moin-2.0/issue/74/mad-url-redesign-item-lookup-idea
99 2013-02-12T13:47:05 <ThomasWaldmann> xiaq: there is another ticket about that and yes, some parts are of course optional
100 2013-02-12T13:47:41 <ThomasWaldmann> you can have a look at the interwiki tests about how it behaves currently
101 2013-02-12T13:49:58 <ThomasWaldmann> and basically there isn't much difference between "look up X in virtns of itemids" and "look up X using it as itemid"
102 2013-02-12T13:53:52 <ThomasWaldmann> hmm. for the nesting we have to consider that
103 2013-02-12T13:54:01 <xiaq> ThomasWaldmann: which is the another ticket?
104 2013-02-12T13:56:08 <ThomasWaldmann> https://bitbucket.org/thomaswaldmann/moin-2.0/issue/81/store-a-list-of-names-into-meta-name rather big one from when that namespaces / multi names stuff began...
105 2013-02-12T13:56:39 *** dave_largo
106 2013-02-12T14:27:09 *** dwcramer
107 2013-02-12T14:41:48 *** dwcramer
108 2013-02-12T14:51:23 *** RogerHaase
109 2013-02-12T15:14:27 *** nightolo
110 2013-02-12T15:16:04 *** nightolo
111 2013-02-12T15:21:02 *** nightolo
112 2013-02-12T15:22:06 *** nightolo
113 2013-02-12T15:37:50 *** dwcramer
114 2013-02-12T18:10:27 <moinBot> http://hg.moinmo.in/moin/2.0/rev/38e79cd2b320 2013-02-12 Thomas Waldmann <tw AT waldmann-edv DOT de> use more constants: ITEMTYPE_DEFAULT
115 2013-02-12T18:11:34 <ronny> ThomasWaldmann, what is moinbot using?
116 2013-02-12T18:27:27 *** ronny
117 2013-02-12T18:27:43 *** ronny
118 2013-02-12T18:44:43 *** greg_f
119 2013-02-12T18:56:18 *** dwcramer
120 2013-02-12T19:11:22 *** xiaq
121 2013-02-12T19:32:32 <sunu> moin
122 2013-02-12T19:33:45 <sunu> searching for an issue in the tracker to attack .. :)
123 2013-02-12T20:08:55 *** ThomasWaldmann
124 2013-02-12T20:09:27 *** ThomasWaldmann
125 2013-02-12T20:23:00 <ThomasWaldmann> sunu: lots of stuff there. :)
126 2013-02-12T20:25:18 <moinBot> http://hg.moinmo.in/moin/2.0/rev/d06e56e20b91 2013-02-12 Thomas Waldmann <tw AT waldmann-edv DOT de> use more constants: ANON for anonymous
127 2013-02-12T20:37:51 <ThomasWaldmann> ronny: some stuff suggested by xiaq (code for a hook)
128 2013-02-12T20:38:39 <ronny> i see
129 2013-02-12T20:39:13 <ronny> (im setting up a bot for pylib, pytestn tox and the other stuff i work on with holger
130 2013-02-12T20:41:08 * ThomasWaldmann just noticed i can't use pytest-pep8 1.04 as it requires pytest >= 2.3beta-whatever
131 2013-02-12T20:45:23 <ronny> hmm, tell in pylib please
132 2013-02-12T20:54:50 <ThomasWaldmann> well, we'll upgrad to 2.3 at some time. just a bigger effort (as from 1 to 2 ...)
133 2013-02-12T21:07:41 <ronny> ThomasWaldmann, i might kill the pytest-pep8/flakes/codechecks things for pytest-flint if flint takes off like i expect it to
134 2013-02-12T21:10:24 <ThomasWaldmann> don't kill it as long as there's no proven replacement :) we need it!
135 2013-02-12T21:12:57 <ronny> ThomasWaldmann, thats why its a might
136 2013-02-12T21:16:02 <ThomasWaldmann> btw, if you have time, look at tickets related to "namespaces" branch
137 2013-02-12T21:16:22 <ThomasWaldmann> things get rather "interesting"
138 2013-02-12T21:17:22 <ronny> ThomasWaldmann, atm im working on my cv - its really a pain to write those but i have to get one done
139 2013-02-12T21:55:18 *** StrangeLoop
140 2013-02-12T22:05:43 *** dave_largo
141 2013-02-12T22:08:31 *** StrangeLoop
142 2013-02-12T22:23:38 <dreimark> ThomasWaldmann: for deleted(nameless) items we could set the id as name (name == id)
143 2013-02-12T22:36:47 <ThomasWaldmann> how's that helpful?
144 2013-02-12T23:04:38 <dreimark> ThomasWaldmann: is that known http://rn0.ru/show/rGlltWCZkyYzAj5OFH6K/
145 2013-02-12T23:05:03 <dreimark> this was a rm -r wiki/*
146 2013-02-12T23:05:56 <dreimark> have the serialized items installed and index builded
147 2013-02-12T23:10:39 * dreimark ignore cookie was not removed
148 2013-02-12T23:14:30 *** RogerHaase
149 2013-02-12T23:35:54 <dreimark> ThomasWaldmann: we need only to set name[0] to '' and name[1] could be the old name
150 2013-02-12T23:42:22 <dreimark> gn