MoinMoin: MoinMoinChat/Logs/moin-dev/2008-06-23

2008-06-23T00:30:01  <ThomasWaldmann> http://master17.moinmo.in/HelpOnMacros TOC broken? 1.6 shows different behaviour.
2008-06-23T01:32:47  <TheSheep> dreimark: I did this for a friend once: http://sheep.art.pl/misc/gallery.html
2008-06-23T02:21:52  <ThomasWaldmann> waldi: no commits since 10 days
2008-06-23T02:22:35  <ThomasWaldmann> TheSheep: for pawel: no commits since 7 days
2008-06-23T02:24:50  <ThomasWaldmann> some students need to significantly speed up and work far more regularly
2008-06-23T02:30:46  <TheSheep> ThomasWaldmann: I just got an e-mail from him, I need to get him to commit more frequently
2008-06-23T02:36:47  <ThomasWaldmann> TheSheep: btw, if you want to do some theme reorg in 1.8, add to that theme / 1.8 todo page
2008-06-23T02:38:21  <TheSheep> you mean http://moinmo.in/MoinMoinTodo/Release 1.8 ?
2008-06-23T02:42:03  <ThomasWaldmann> yes, and that feature req page
2008-06-23T02:46:05  <TheSheep> http://moinmo.in/FeatureRequests/ThemeChange ?
2008-06-23T02:47:30  <ThomasWaldmann> http://moinmo.in/FeatureRequests/ThemeApi
2008-06-23T02:49:30  <TheSheep> ah, internals :)
2008-06-23T02:49:57  <TheSheep> http://moin.sheep.art.pl/ <-- I wonder it this would be possible :)
2008-06-23T02:52:55  <ThomasWaldmann> not much to see there
2008-06-23T02:56:18  <ThomasWaldmann> hehe, switches logos when touched :)
2008-06-23T02:56:40  * TheSheep switched the logo
2008-06-23T02:56:52  <ThomasWaldmann> ah
2008-06-23T02:57:21  <TheSheep> teh changes are small
2008-06-23T02:57:38  <TheSheep> lines, spacing, colors
2008-06-23T02:58:28  <TheSheep> it mostly changes the rendering when css is disabled
2008-06-23T02:58:39  <ThomasWaldmann> well, it looks nice except that I still prefer the page title below the login/settings/search
2008-06-23T03:01:01  <ThomasWaldmann> hehe, funny how your header stuff moves when reducing width
2008-06-23T03:03:31  <ThomasWaldmann> TheSheep: can you rearrange it?
2008-06-23T03:04:51  <TheSheep> yes
2008-06-23T03:05:49  * ThomasWaldmann tries opera
2008-06-23T03:06:44  <ThomasWaldmann> looks much better for O.
2008-06-23T03:08:09  <ThomasWaldmann> that scales width cool now :)
2008-06-23T03:09:43  <ThomasWaldmann> failed search icons are broken
2008-06-23T03:09:58  <ThomasWaldmann> http://moin.sheep.art.pl/static/modernized/img/icon-error.png
2008-06-23T03:10:08  <ThomasWaldmann> http://moin.sheep.art.pl/static/modernized/img/idea.png
2008-06-23T03:10:48  <TheSheep> yeah, I didn't copy the smilies
2008-06-23T03:10:57  <TheSheep> I was also thinking about using the tango smilies
2008-06-23T03:11:29  * ThomasWaldmann tries with the nokia mobile browser
2008-06-23T03:12:06  <TheSheep> now, smilies copied
2008-06-23T03:12:53  * TheSheep removes the acls
2008-06-23T03:14:13  <ThomasWaldmann> the logo has a blue bar in the bg when viewed with the mobile browser
2008-06-23T03:14:30  <TheSheep> transparent png
2008-06-23T03:15:53  <ThomasWaldmann> the search box and buttons also look bigger somehow
2008-06-23T03:17:46  <ThomasWaldmann> what do you think of title/text buttons below search field, right aligned?
2008-06-23T03:17:50  <TheSheep> I didn't style them, they should appear as the system default
2008-06-23T03:18:19  <TheSheep> ThomasWaldmann: it wouldn't look like a search box anymore, would it?
2008-06-23T03:18:48  <TheSheep> but I think I have an idea
2008-06-23T03:19:15  <ThomasWaldmann> (that's about the only thing not fitting on mobile width)
2008-06-23T03:19:39  <ThomasWaldmann> (the action menu is also hanging a bit outside to the right)
2008-06-23T03:20:25  <TheSheep> now...
2008-06-23T03:20:40  <TheSheep> they should drop below the search box if there is limited space
2008-06-23T03:23:46  <ThomasWaldmann> yes, but now the search box also jumped more to the right
2008-06-23T03:24:25  <TheSheep> I set max--width: 40%
2008-06-23T03:24:37  <ThomasWaldmann> (more height for login?)
2008-06-23T03:25:19  <TheSheep> now, removed max-width
2008-06-23T03:25:32  <TheSheep> I will also remove nowrap from the actions bar
2008-06-23T03:26:41  <ThomasWaldmann> that completely changed looks on the small screen
2008-06-23T03:27:04  <TheSheep> oh, to better or worse?
2008-06-23T03:27:28  <ThomasWaldmann> looking worse, but maybe better usable
2008-06-23T03:28:10  <ThomasWaldmann> much empty space right of logo now
2008-06-23T03:28:21  <TheSheep> 12 pixels...
2008-06-23T03:28:42  <TheSheep> I geuss I should set it in ems
2008-06-23T03:28:47  <ThomasWaldmann> most space right of it is clear
2008-06-23T03:29:00  <ThomasWaldmann> login is above logo now
2008-06-23T03:30:00  <ThomasWaldmann> with opera at the width when login jumps above the logo
2008-06-23T03:30:15  <ThomasWaldmann> that's about what the mobile displays
2008-06-23T03:30:48  <TheSheep> how about now?
2008-06-23T03:32:14  <ThomasWaldmann> no change on mobile
2008-06-23T03:32:44  <TheSheep> it wraps differently in ff now
2008-06-23T03:33:09  <ThomasWaldmann> i guess the height of login should be equal as search field/buttons
2008-06-23T03:33:22  <ThomasWaldmann> to avoid strange rendering
2008-06-23T03:33:38  <TheSheep> hmm... hard to do, I don't know the size of search buttons
2008-06-23T03:33:43  <ThomasWaldmann> (not the font height, just the box height)
2008-06-23T03:33:50  <TheSheep> it's different on every platform
2008-06-23T03:33:55  <TheSheep> I could force it, but...
2008-06-23T03:34:41  <ThomasWaldmann> the problem now is that search box collides with title text when shrinking width
2008-06-23T03:35:16  <ThomasWaldmann> (although there is enough space a bit below so it doesnt need to collide)
2008-06-23T03:35:54  <ThomasWaldmann> but because login is consuming less height than search box, title and search collide
2008-06-23T03:37:21  <ThomasWaldmann> besides that, it's perfect :)
2008-06-23T03:39:11  <ThomasWaldmann> ah, it is maybe also caused a bit by page name having more height than interwiki name
2008-06-23T03:39:40  <TheSheep> how about now?
2008-06-23T03:40:54  <ThomasWaldmann> behaves almost correct, but has a broad white box in opera
2008-06-23T03:41:30  <TheSheep> yeah, I added the backgrounds to see the boxes better
2008-06-23T03:41:33  <TheSheep> ignore them
2008-06-23T03:41:46  <ThomasWaldmann> and when the search box collides with login link, the login link is render at same place as the interwiki link, garbling chars
2008-06-23T03:42:02  <TheSheep> hmrm :/
2008-06-23T03:42:26  <TheSheep> try now
2008-06-23T03:43:05  <TheSheep> now this is pretty obnoxious :/
2008-06-23T03:43:41  <ThomasWaldmann> login sometimes garbled with interwiki
2008-06-23T03:43:57  <ThomasWaldmann> (try opera and shrink width)
2008-06-23T03:44:41  <TheSheep> now, I think I solved it
2008-06-23T03:44:51  <ThomasWaldmann> but scaling already is good
2008-06-23T03:45:54  <TheSheep> and it even looks and works the same in msie6
2008-06-23T03:46:44  <ThomasWaldmann> search box seems to collide with big H of Home
2008-06-23T03:47:37  <TheSheep> in which browser?
2008-06-23T03:48:18  <ThomasWaldmann> opera
2008-06-23T03:48:39  <TheSheep> still?
2008-06-23T03:48:57  <ThomasWaldmann> same for ff3
2008-06-23T03:49:34  <TheSheep> I don't see it
2008-06-23T03:50:23  <ThomasWaldmann> if the left border of search box reaches the H of Home, it pushes it down
2008-06-23T03:50:36  <TheSheep> yes
2008-06-23T03:50:58  <TheSheep> that's because the buttons wrap
2008-06-23T03:52:04  <ThomasWaldmann> and short before that moment, vspacing between Home and search box is too small
2008-06-23T03:52:39  <TheSheep> ok, that can be fixed :)
2008-06-23T03:52:41  <TheSheep> now?
2008-06-23T03:54:23  <ThomasWaldmann> better
2008-06-23T03:54:43  <ThomasWaldmann> can you try some pix more, just to see if it fixes the H collision?
2008-06-23T03:55:19  <TheSheep> doubled it
2008-06-23T03:55:49  <ThomasWaldmann> no, seems unrelated
2008-06-23T03:56:10  * TheSheep disables the annoing button wrapping
2008-06-23T03:57:05  <TheSheep> I can't see the collision, there is at leas 8px gap between them
2008-06-23T03:57:33  <ThomasWaldmann> that's what i mean
2008-06-23T03:57:43  <ThomasWaldmann> just like repulsion rather :)
2008-06-23T03:58:04  <TheSheep> you want that gap smaller, larger, gone or what? :)
2008-06-23T03:59:13  <TheSheep> .oO( personally I vote for the "on what" )
2008-06-23T03:59:22  <TheSheep> or even
2008-06-23T03:59:30  <ThomasWaldmann> looks ok now
2008-06-23T04:00:13  * ThomasWaldmann misses the dowhatiwant css attr
2008-06-23T04:01:02  <TheSheep> msie6 woyuld set it to "onlywhenIfeellike" permanently anyways
2008-06-23T04:02:11  <ThomasWaldmann> no RC for me
2008-06-23T04:03:49  <TheSheep> argh, that's the install on which I made the acl experiments
2008-06-23T04:04:26  <TheSheep> now you should have rights for everything
2008-06-23T04:04:27  <ThomasWaldmann> new underlay is better :)
2008-06-23T04:04:55  <TheSheep> I didn't adjust the icon sizes for the new icons
2008-06-23T04:05:44  <ThomasWaldmann> i see it :)
2008-06-23T04:07:07  <ThomasWaldmann> ok, I think we replace modern py and css with that
2008-06-23T04:07:22  <ThomasWaldmann> still unsure about the icons
2008-06-23T04:07:28  <TheSheep> yeah
2008-06-23T04:08:07  <ThomasWaldmann> the old icons look like crap, but they are easy to recognize :)
2008-06-23T04:08:41  <TheSheep> icon sizes fixed
2008-06-23T04:09:03  <TheSheep> the icons in RC were kinda selected in a hurry
2008-06-23T04:11:22  <ThomasWaldmann> icons make sense, mostly
2008-06-23T04:11:38  <ThomasWaldmann> the bookmark diff and the info is a bit strange
2008-06-23T04:12:55  <ThomasWaldmann> is there an icons showing 3 papersheets (for the bookmark diff)?
2008-06-23T04:13:46  <TheSheep> no, but there is a bookmark icon...
2008-06-23T04:14:21  <ThomasWaldmann> btw, these icons will be harder to target because they have less width
2008-06-23T04:16:49  <ThomasWaldmann> ok, i guess i will retry sleeping
2008-06-23T04:17:37  <ThomasWaldmann> thanks for that theme, feel free to integrate into 1.8, i guess we can fix leftover issues there
2008-06-23T04:17:59  <ThomasWaldmann> maybe keep modern for now so we can easily compare
2008-06-23T04:18:46  <ThomasWaldmann> gn
2008-06-23T04:28:16  <TheSheep> good night
2008-06-23T04:28:24  <TheSheep> at least the new icons don't have i18n issues
2008-06-23T08:24:18  <ThomasWaldmann> moin
2008-06-23T08:25:50  <ThomasWaldmann> TheSheep: yeah
2008-06-23T09:54:11  <xorAxAx> moin melita
2008-06-23T10:19:36  <gizmach> moin xorAxAx
2008-06-23T10:29:43  <dreimark> moin gizmach
2008-06-23T10:29:52  <dreimark> moin to everywhere
2008-06-23T10:29:56  <gizmach> dreimark: hi hi
2008-06-23T10:35:55  <dreimark> gizmach: can I see today a commit ?
2008-06-23T10:36:14  <gizmach> I tkink you can
2008-06-23T10:36:16  <gizmach> :)
2008-06-23T10:36:35  <gizmach> before midnight
2008-06-23T10:37:04  <dreimark> perfect
2008-06-23T10:37:16  <dreimark> htg bbl
2008-06-23T10:37:21  <zenhase> moin
2008-06-23T10:38:12  <zenhase> ThomasWaldmann: ok, will have a look (@ global error handler / mimetype)
2008-06-23T10:46:35  <CIA-52> Bastian Blank <bblank@thinkmo.de> default * 3669:595c2f17ddf7 1.8-dom-bblank/MoinMoin/converter2/ (_tests/test_creole_in.py creole_in.py):
2008-06-23T10:46:35  <CIA-52> Add Creole input converter.
2008-06-23T10:46:35  <CIA-52> * MoinMoin/converter2/creole_in.py: Copy from MoinMoin/parser/_creole.py.
2008-06-23T10:46:35  <CIA-52> * MoinMoin/converter2/_tests/test_creole_in.py: New file.
2008-06-23T10:46:38  <CIA-52> Bastian Blank <bblank@thinkmo.de> default * 3670:5b87592cd696 1.8-dom-bblank/MoinMoin/converter2/creole_in.py:
2008-06-23T10:46:40  <CIA-52> Creole input converter - Update API
2008-06-23T10:46:42  <CIA-52> MoinMoin/converter2/creole_in.py
2008-06-23T10:46:44  <CIA-52> - Rename Parser to Converter.
2008-06-23T10:46:46  <CIA-52> - Use __call_ for invocation.
2008-06-23T10:46:48  <CIA-52> Bastian Blank <bblank@thinkmo.de> default * 3671:4a853f852e0b 1.8-dom-bblank/MoinMoin/converter2/creole_in.py:
2008-06-23T10:46:51  <CIA-52> Creole input converter - Use stack and support heading
2008-06-23T10:46:53  <CIA-52> MoinMoin/converter2/creole_in.py
2008-06-23T10:46:55  <CIA-52> - Use stack instead of current node, we have no parent pointer.
2008-06-23T10:46:57  <CIA-52> - Make _upto pop from the stack.
2008-06-23T10:46:59  <CIA-52> - Port headings to new interface.
2008-06-23T11:39:17  <CIA-52> Bastian Blank <bblank@thinkmo.de> default * 3672:8826b86b2024 1.8-dom-bblank/MoinMoin/converter2/ (_tests/test_creole_in.py creole_in.py):
2008-06-23T11:39:17  <CIA-52> Creole input converter - Support plain text
2008-06-23T11:39:17  <CIA-52> * MoinMoin/converter2/creole_in.py: Port inline text support to new interface.
2008-06-23T11:39:17  <CIA-52> * MoinMoin/converter2/_tests/test_creole_in.py: Add testcase for plain text.
2008-06-23T11:39:19  <CIA-52> Bastian Blank <bblank@thinkmo.de> default * 3673:b382ace6a7e6 1.8-dom-bblank/MoinMoin/converter2/creole_in.py:
2008-06-23T11:39:22  <CIA-52> Creole input converter - Move duplicated code into methods.
2008-06-23T11:39:24  <CIA-52> MoinMoin/converter2/creole_in.py
2008-06-23T11:39:26  <CIA-52> - Add new stack manipulation functions.
2008-06-23T11:39:28  <CIA-52> - Rename _upto to _stack_pop_name to match other names.
2008-06-23T11:39:30  <CIA-52> Bastian Blank <bblank@thinkmo.de> default * 3674:8bb21ba6315f 1.8-dom-bblank/MoinMoin/converter2/creole_in.py:
2008-06-23T11:39:33  <CIA-52> Creole input converter - Never clean stack completely
2008-06-23T11:39:35  <CIA-52> MoinMoin/converter2/creole_in.py:
2008-06-23T11:39:37  <CIA-52> Fix stack pop to work as documented and never try to clean it completely.
2008-06-23T11:39:39  <CIA-52> Bastian Blank <bblank@thinkmo.de> default * 3675:fb39acd1393f 1.8-dom-bblank/MoinMoin/converter2/ (_tests/test_creole_in.py creole_in.py):
2008-06-23T11:39:42  <CIA-52> Creole input converter - Support emphasis and strong
2008-06-23T11:39:48  <CIA-52> * MoinMoin/converter2/creole_in.py:
2008-06-23T11:39:50  <CIA-52>  Port emphasis and strong support to new interface.
2008-06-23T11:39:52  <CIA-52> * MoinMoin/converter2/_tests/test_creole_in.py: Add testcase.
2008-06-23T11:39:54  <CIA-52> Bastian Blank <bblank@thinkmo.de> default * 3676:290e493d9296 1.8-dom-bblank/MoinMoin/converter2/creole_in.py:
2008-06-23T11:39:57  <CIA-52> Creole input converter - Remove extra space after inline text
2008-06-23T11:39:59  <CIA-52> MoinMoin/converter2/creole_in.py:
2008-06-23T11:40:03  <CIA-52> Remove extra space after all inline text. It seems to not break.
2008-06-23T11:56:07  <dreimark> TheSheep: mouse over effect controls very nice
2008-06-23T12:25:17  <dreimark> ThomasWaldmann: yeah TOC 1.   1. looks broken
2008-06-23T12:25:26  <dreimark> bbl
2008-06-23T12:39:32  <CIA-52> Christopher Denter <moin GUESSWHAT the DASH space DASH station ROUNDTHING com default * 4048:777536feaa82 1.8-storage-cdenter/MoinMoin/storage/ (3 files in 3 dirs): storage: Adding test for MemoryBackend. Implementing trivial Metadata methods. Metadata locking is next. Small changeset in order to synchronize my machines.
2008-06-23T12:50:14  <PawelPacana> moin
2008-06-23T13:06:28  *** irc.freenode.net sets mode: +o ChanServ
2008-06-23T13:11:59  <dennda> johill: Do we want users tomanually lock items in order to alter their metadata?
2008-06-23T13:12:28  <dennda> I wonder if we can't do that implicitly
2008-06-23T13:20:17  <ThomasWaldmann> fpletz: btw, the result list ordering is one of the remaining problems when using xapian search
2008-06-23T13:21:04  <ThomasWaldmann> fpletz_: after I removed that sort = None statement, it got a bit better, but still far from perfect
2008-06-23T13:25:33  <CIA-52> Christopher Denter <moin GUESSWHAT the DASH space DASH station ROUNDTHING com default * 4049:24e02f23a3ea 1.8-storage-cdenter/MoinMoin/storage/ (__init__.py backends/memory.py):
2008-06-23T13:25:33  <CIA-52> storage: (MemoryBackend): Implementing locking for Item metadata on the
2008-06-23T13:25:33  <CIA-52> MemoryBackend. Fixing bug in the abstract backend. You can now read and write
2008-06-23T13:25:33  <CIA-52> Item metadata with explicit locking. I wonder if we can do that implicitly,
2008-06-23T13:25:35  <CIA-52> though.
2008-06-23T13:26:11  <xorAxAx> gizmach: what was so funny? :)
2008-06-23T13:27:06  <gizmach> xorAxAx: well It was a bit stranged that in one moment I looked and was noone on the chanel
2008-06-23T13:27:27  <xorAxAx> gizmach: just we both? :)
2008-06-23T13:27:54  <gizmach> no, you were the first to loged in
2008-06-23T13:28:01  <gizmach> and no I'm not crazy
2008-06-23T13:28:08  <xorAxAx> hm? :)
2008-06-23T13:28:11  <xorAxAx> it was a netsplit
2008-06-23T13:29:00  <xorAxAx> you are the sanest girl from croatia that i know :)
2008-06-23T13:29:14  <zenhase> how many do you know?
2008-06-23T13:29:17  <xorAxAx> 1
2008-06-23T13:29:30  <zenhase> then this was not a very nice statement :)
2008-06-23T13:29:32  <dennda> which makes you the most insane croatian girl, too
2008-06-23T13:29:41  <dennda> (which he knows)
2008-06-23T13:29:42  <zenhase> dennda: hrhr
2008-06-23T13:29:47  <xorAxAx> zenhase: why? it just says nothing about her
2008-06-23T13:30:00  <gizmach> http://code.bulix.org/q8avn6-67239
2008-06-23T13:30:03  <xorAxAx> i was just wondering why she wondered why she would be crazy
2008-06-23T13:30:32  <gizmach> xorAxAx: it's ok
2008-06-23T13:30:34  <xorAxAx> gizmach: ah, i was the first to join
2008-06-23T13:30:43  <gizmach> yes
2008-06-23T13:31:11  <gizmach> dennda: well I don't mind
2008-06-23T13:31:19  <xorAxAx> hmm, and you didnt see my cloak
2008-06-23T13:31:24  <TheSheep> ThomasWaldmann: I was wondering, can I just merge this repo in with 1.8? http://sheep.art.pl/devel/modernized/
2008-06-23T13:31:31  <dennda> :)
2008-06-23T13:31:31  <xorAxAx> so i was naked when joining :)
2008-06-23T13:31:44  <gizmach> :))
2008-06-23T13:32:09  <TheSheep> the commit messages are probably wrong out of context :(
2008-06-23T13:47:57  <ThomasWaldmann> TheSheep: i am not sure we can directly merge, because this is not a moin repo, it just contains the theme
2008-06-23T13:49:42  <ThomasWaldmann> if noone has a better idea, one maybe could just condense the summary lines from hg log into some CHANGES entry about modernized theme
2008-06-23T13:53:14  <TheSheep> ThomasWaldmann: so I should just copy it in?
2008-06-23T14:02:57  <waldi> yeah, new creole parser glued with html output (for now static) and working[tm]
2008-06-23T14:03:36  <TheSheep> waldi: cool!
2008-06-23T14:04:41  <waldi> okay, it still dies with attribute errors on not changed parts of the parser
2008-06-23T14:06:10  <TheSheep> waldi: oh, you copied the whole parser :/
2008-06-23T14:07:36  <waldi> yes, why not?
2008-06-23T14:09:07  <TheSheep> waldi: once you are at it, you can replace these re.sub's with re.finditers
2008-06-23T14:09:16  <TheSheep> waldi: I was about to do it, but never got to it
2008-06-23T14:09:55  <waldi> sure, but for now I want to make it working
2008-06-23T14:10:07  <ThomasWaldmann> waldi: cool :)
2008-06-23T14:10:33  <TheSheep> right, sorry
2008-06-23T14:10:55  <ThomasWaldmann> waldi: btw, I asked myself if maybe attribute names that are valid python identifiers could be useful
2008-06-23T14:11:23  <ThomasWaldmann> TheSheep: maybe wait for a comment of xorAxAx / johill
2008-06-23T14:11:44  <waldi> ThomasWaldmann: what do you mean?
2008-06-23T14:11:47  <xorAxAx> about merging? :)
2008-06-23T14:12:08  <ThomasWaldmann> yes
2008-06-23T14:12:09  <waldi> TheSheep: what is that "bloglike_lines" about?
2008-06-23T14:12:41  <xorAxAx> TheSheep: ask #mercurial
2008-06-23T14:12:48  <xorAxAx> thats where the best practices reside
2008-06-23T14:13:22  <waldi> (the subsystem maintainers of linux upstream usualy uses rebase instead of merge ...)
2008-06-23T14:40:36  <TheSheep> waldi: when designing Creole, there was a strong force to preserve any newlines (instead of treating them as whitespace), just like (some) blogs do. To avoid conflicts and still keep Creole sane, it ended up as an option (that hopefully nobody will ever enable)
2008-06-23T14:41:31  <waldi> hmm, newline is a whitespace
2008-06-23T14:42:09  <TheSheep> they wanted to convert it to <br>s
2008-06-23T14:43:02  <TheSheep> I know it sucks
2008-06-23T14:43:14  <TheSheep> I consider it my personal failure
2008-06-23T14:43:30  <waldi> hmm, the creole site explicitely say that newlines are treated as whitespace
2008-06-23T14:45:20  <TheSheep> unless that option is enabled :)
2008-06-23T14:45:50  <TheSheep> I doubt anybody will ever use it
2008-06-23T14:46:16  <TheSheep> but I implemented it to be fair
2008-06-23T14:50:14  <zenhase> data_underlay_dir "/Users/flowhase/Projects/moin-gsoc/wiki/underlay" does not exist, or has incorrect ownership or permissions.
2008-06-23T14:50:17  <zenhase> hmm ...
2008-06-23T14:50:39  <zenhase> on py.test :o
2008-06-23T14:51:08  <waldi> mkdir wiki/underlay/pages -p
2008-06-23T14:51:26  <waldi> (and hope that it won't check for specific pagenames)
2008-06-23T14:52:02  <zenhase> hmm i don't know ;)
2008-06-23T14:52:25  <zenhase> the tests should run 'on checkout' :)
2008-06-23T14:53:10  <ThomasWaldmann> zenhase: tar xf underlay.tar
2008-06-23T14:53:23  <ThomasWaldmann> zenhase: cd ..
2008-06-23T14:53:31  <ThomasWaldmann> zenhase: make pagepacks
2008-06-23T14:58:25  <zenhase> hmm, i don't remember that i had to do this last year for the tests to work :o
2008-06-23T15:01:00  <zenhase> ok now it works
2008-06-23T15:01:13  <zenhase> (or better throws the errors i did expect from py.test)
2008-06-23T16:28:08  <TheSheep> PawelPacana: good news :)
2008-06-23T16:28:25  <TheSheep> PawelPacana: they implemented commits without a workingdir in mercurial
2008-06-23T16:28:55  <PawelPacana> TheSheep: splendid :)
2008-06-23T16:29:13  <dennda> Which means?
2008-06-23T16:29:16  <TheSheep> PawelPacana: also, please commit even the smallest changes you do, and do a push at least once a day
2008-06-23T16:29:51  <TheSheep> dennda: we can get rid of some tricks we had to use
2008-06-23T16:30:03  <TheSheep> dennda: actually, PawelPacana can ;)
2008-06-23T16:30:20  <PawelPacana> dennda: we dont need to write custom extension to achieve this now
2008-06-23T16:30:51  <PawelPacana> however, i'm not sure yet, until i see it
2008-06-23T16:30:58  <dennda> PawelPacana: You can check the API now and play with it
2008-06-23T16:31:08  <dennda> Maybe that helps you to get a better understanding of how it works
2008-06-23T16:31:29  <dennda> There is a basic implementation which I am currently working on, in memory.py see the MemoryBackend
2008-06-23T16:31:40  <dennda> It is near-complete
2008-06-23T16:32:05  <PawelPacana> dennda: cool, i`m keeping track constantly an merging your repo
2008-06-23T16:32:26  <CIA-52> Bastian Blank <bblank@thinkmo.de> default * 3677:62a417919a6d 1.8-dom-bblank/MoinMoin/converter2/ (_tests/test_creole_in.py creole_in.py):
2008-06-23T16:32:26  <CIA-52> Creole input converter - Check for page as top level element, fix wrong tests
2008-06-23T16:32:26  <CIA-52> * MoinMoin/converter2/creole_in.py: Check for page as top level element.
2008-06-23T16:32:26  <CIA-52> * MoinMoin/converter2/_tests/test_creole_in.py:
2008-06-23T16:32:28  <CIA-52>  Check for emphasis and strong is correctly enclosed in a paragraph.
2008-06-23T16:32:29  <dennda> Great, that means I don't have to do that :)
2008-06-23T16:32:30  <CIA-52> Bastian Blank <bblank@thinkmo.de> default * 3678:f591afce7709 1.8-dom-bblank/MoinMoin/converter2/ (_tests/test_creole_in.py creole_in.py):
2008-06-23T16:32:33  <CIA-52> Creole input converter - Support line break
2008-06-23T16:32:35  <CIA-52> * MoinMoin/converter2/creole_in.py: Port line break support to new interface.
2008-06-23T16:32:37  <CIA-52> * MoinMoin/converter2/_tests/test_creole_in.py: Add testcase.
2008-06-23T16:32:37  <TheSheep> duck! ;)
2008-06-23T16:32:41  <CIA-52> Bastian Blank <bblank@thinkmo.de> default * 3679:9570a5ce3dba 1.8-dom-bblank/MoinMoin/converter2/creole_in.py:
2008-06-23T16:32:44  <CIA-52> Creole input converter - Remove unused code
2008-06-23T16:32:46  <CIA-52> MoinMoin/converter2/creole_in.py: Remove DocNode class.
2008-06-23T16:32:48  <CIA-52> Bastian Blank <bblank@thinkmo.de> default * 3680:9d3e1d3d8df9 1.8-dom-bblank/MoinMoin/converter2/creole_in.py: Update copyright entries.
2008-06-23T16:32:53  <CIA-52> Bastian Blank <bblank@thinkmo.de> default * 3681:093c482e02e1 1.8-dom-bblank/MoinMoin/converter2/ (_tests/test_creole_in.py creole_in.py): (log message trimmed)
2008-06-23T16:32:54  <PawelPacana> we have to write more generic testcases for backends in general
2008-06-23T16:32:56  <CIA-52> Creole input converter - Support url, cleanup
2008-06-23T16:32:58  <CIA-52> * MoinMoin/converter2/creole_in.py
2008-06-23T16:33:00  <CIA-52>  - Port url support to new interface.
2008-06-23T16:33:02  <CIA-52>  - Remove namespace attribute from Converter.
2008-06-23T16:33:04  <CIA-52> * MoinMoin/converter2/_tests/test_creole_in.py
2008-06-23T16:33:07  <dennda> PawelPacana: Yes
2008-06-23T16:33:08  <CIA-52>  - Add testcases for url.
2008-06-23T16:33:10  <CIA-52> Bastian Blank <bblank@thinkmo.de> default * 3682:027ed3409367 1.8-dom-bblank/MoinMoin/converter2/ (_tests/test_creole_in.py creole_in.py):
2008-06-23T16:33:13  <CIA-52> Creole input converter - Support link
2008-06-23T16:33:15  <CIA-52> * MoinMoin/converter2/creole_in.py: Port link support to new interface.
2008-06-23T16:33:17  <CIA-52> * MoinMoin/converter2/_tests/test_creole_in.py: Add testcases.
2008-06-23T16:33:23  <dennda> Would be great to collaborate there
2008-06-23T16:33:25  <dennda> If youw ant
2008-06-23T16:33:41  <dennda> Maybe some abstract Test Classes that specific backends can use and override
2008-06-23T16:33:49  <dennda> Have you seen my tests? They are not complete yet either
2008-06-23T16:34:52  <PawelPacana> yes
2008-06-23T16:35:00  <TheSheep> dennda: aren't you supposed to write the tests first? ;)
2008-06-23T16:35:01  <PawelPacana> i was writing test for my backend
2008-06-23T16:35:20  <dennda> TheSheep: Maybe I am, I got it on my todo list
2008-06-23T16:35:41  <dennda> But I think PawelPacana needs to use that then to create specific test cases for the hg backend
2008-06-23T16:35:43  <PawelPacana> however it came to me that almost similar tests for every backend is pointless
2008-06-23T16:36:27  <PawelPacana> so i`ll try to commit some generic one for both of us
2008-06-23T16:36:56  <PawelPacana> and then we`ll subclass specific flavours
2008-06-23T16:37:19  <PawelPacana> TheSheep: where did you get the news?
2008-06-23T16:37:25  <dennda> Yah I got * write more and better tests and * make moin use the new API on my list
2008-06-23T16:37:37  <dennda> Especially the second point will be interesting for both of us
2008-06-23T16:37:37  <TheSheep> PawelPacana: ronny told me on #moin
2008-06-23T16:37:57  <dennda> you can then test your backend using moin :)
2008-06-23T16:38:34  <PawelPacana> dennda: that would be crash-test at this point ;)
2008-06-23T16:45:49  <PawelPacana> TheSheep: is this going to be released soon? i assume it lives in unstable dev repo by now
2008-06-23T16:46:08  <TheSheep> PawelPacana: no idea, I guess you should ask on #mercurial :)
2008-06-23T16:46:44  <PawelPacana> yeah, thought you have some secret info ;)
2008-06-23T16:47:19  <TheSheep> no secrets
2008-06-23T17:00:26  <ThomasWaldmann> waldi: after removing that extra space, does multiline text still get joined correctly into float text?
2008-06-23T17:01:01  <ThomasWaldmann> (the \n is maybe already taken away)
2008-06-23T17:05:27  <waldi> ThomasWaldmann: no, it is not. so that is the reason for this, \n is not used correctly
2008-06-23T17:09:04  <ThomasWaldmann> waldi: and please add a test for correctly and badly nested strong/emphasis
2008-06-23T17:12:55  * ThomasWaldmann looks at pidfile stuff
2008-06-23T17:18:45  <waldi> yeah
2008-06-23T17:20:49  <waldi> creole does not specify what happens with badly nested strong/emphasis, yeah
2008-06-23T17:22:07  <waldi> err, wrong paper. it is not allowed
2008-06-23T17:29:28  <TheSheep> it does specify -- it closes both on first closing tag
2008-06-23T17:37:37  <waldi> TheSheep: it does? i see only "Unacceptable: **//bold italics**//
2008-06-23T17:38:11  <waldi> which I would parse as either UB (undefined behaviour) or forbidden
2008-06-23T17:45:07  <TheSheep> oh?
2008-06-23T17:45:19  <waldi> (http://wikicreole.org/wiki/Creole1.0)
2008-06-23T17:48:01  <TheSheep> I can't find it, but there is a general rule that closing a block closes all markup within it -- ie. markup doesn't "bleed"
2008-06-23T17:48:37  <waldi> search for "Bold Italics"
2008-06-23T17:48:59  <TheSheep> http://wikicreole.org/wiki/Talk.BoldAndItalics <-- this is my own comment at the bottom
2008-06-23T17:56:36  <CIA-52> Bastian Blank <bblank@thinkmo.de> default * 3683:bdb93d760c85 1.8-dom-bblank/MoinMoin/converter2/_tests/test_creole_in.py: Tests for Creole input converter - Add several testcases for emphasis and headings
2008-06-23T17:56:37  <CIA-52> Bastian Blank <bblank@thinkmo.de> default * 3684:28b6ff57c29d 1.8-dom-bblank/MoinMoin/converter2/creole_in.py: Creole input converter - Use re.finditer instead of re.sub
2008-06-23T18:05:06  <waldi> TheSheep: hmm, the _replace function can be written as "getattr(self, '_%s_repl' % match.lastgroup)(match.groupdict())"
2008-06-23T18:05:34  <waldi> (to make it maybe faster and remove some code)
2008-06-23T18:06:23  <CIA-52> Thomas Waldmann <tw AT waldmann-edv DOT de> default * 3736:441a4136db05 1.7/MoinMoin/ (script/server/standalone.py server/daemon.py): standalone server: fix --pidfile option
2008-06-23T18:06:29  * TheSheep checks what .lastgroup does
2008-06-23T18:07:29  <waldi> TheSheep: http://paste.debian.net/7359/
2008-06-23T18:07:48  <TheSheep> waldi: it might fail if the lastgroup is not the one you want, for example when it puts some hlper group in there...
2008-06-23T18:08:32  <TheSheep> oh, it takes the outer group?
2008-06-23T18:08:38  <TheSheep> hmm... this may work :)
2008-06-23T18:08:39  <waldi> yes, this is the last one
2008-06-23T18:09:19  <waldi> at least it does not fail yet
2008-06-23T18:09:40  <TheSheep> the concentration of "magic" in this single line is disturbing though :)
2008-06-23T18:10:14  <TheSheep> waldi: btw, it might be better to use re.finditer instead of re.sub and _replace
2008-06-23T18:10:19  <waldi> sure
2008-06-23T18:10:28  <waldi> already partially replaced
2008-06-23T18:10:58  <TheSheep> I also do another magical thing: pass **dict as the param
2008-06-23T18:11:06  <TheSheep> after removing all empty groups
2008-06-23T18:11:20  <waldi> hmm
2008-06-23T18:11:25  <TheSheep> this way you can have def link_repl(link, target, text=""):
2008-06-23T18:12:10  <TheSheep> and it raises an error if some non-optional group is missing
2008-06-23T18:12:36  <TheSheep> http://sheep.art.pl/devel/dandelion/file/5151f10d67af/dandelion/parser.py <-- line 66
2008-06-23T18:13:17  <waldi> the whole "_code_head_repl = _code_repl" can go away
2008-06-23T18:13:33  <TheSheep> yeah
2008-06-23T18:14:46  <ThomasWaldmann> johill: http://moinmo.in/MoinMoinBugs/1.7SearchForOrexprNotWorking can you look at it?
2008-06-23T18:45:33  <waldi> okay, I hope I fixed the multiline problem. just make the re also match \n for the text case
2008-06-23T18:54:47  <CIA-52> Bastian Blank <bblank@thinkmo.de> default * 3685:de73dfacba65 1.8-dom-bblank/MoinMoin/converter2/creole_in.py:
2008-06-23T18:54:47  <CIA-52> Creole input converter - Only apply complete match
2008-06-23T18:54:47  <CIA-52> MoinMoin/converter2/creole_in.py
2008-06-23T18:54:47  <CIA-52> - Use match.lastgroup for function lookup. It always lists the outer matched
2008-06-23T18:54:48  <CIA-52>  group.
2008-06-23T18:54:50  <CIA-52> - Remove not longer necessary aliases for several group names.
2008-06-23T18:54:54  <CIA-52> Bastian Blank <bblank@thinkmo.de> default * 3686:6002fa958ff9 1.8-dom-bblank/MoinMoin/converter2/ (_tests/test_creole_in.py creole_in.py):
2008-06-23T18:54:57  <CIA-52> Creole input converter - Fix multiline text
2008-06-23T18:54:59  <CIA-52> * MoinMoin/converter2/creole_in.py:
2008-06-23T18:55:01  <CIA-52>  Also accept newline at the beginning of inline text.
2008-06-23T18:55:03  <CIA-52> * MoinMoin/converter2/_tests/test_creole_in.py:
2008-06-23T18:55:05  <CIA-52>  Add testcases for newline in several locations.
2008-06-23T19:18:52  <waldi> hrm, no, does not work
2008-06-23T19:26:44  <CIA-52> Bastian Blank <bblank@thinkmo.de> default * 3687:850aecea6bb2 1.8-dom-bblank/MoinMoin/converter2/ (_tests/test_html_out.py html_out.py):
2008-06-23T19:26:44  <CIA-52> HTML output converter - Support emphasis and strong
2008-06-23T19:26:44  <CIA-52> * MoinMoin/converter2/html_out.py
2008-06-23T19:26:44  <CIA-52>  - Add methods for emphasis and strong.
2008-06-23T19:26:44  <CIA-52>  - Fix exception.
2008-06-23T19:26:46  <CIA-52> * MoinMoin/converter2/_tests/test_html_out.py: Add testcases.
2008-06-23T21:18:37  <gizmach> I will commit like this
2008-06-23T22:21:32  <CIA-52> MelitaMihaljevic default * 3681:b9acbb0c1c8e 1.8-ldapgroups-mmihaljevic/MoinMoin/auth/ldap_login.py: added lsuer method for getting user informations from LDAP, added memberOFGroup method for getting all groups that a user is a member of
2008-06-23T22:25:53  <dreimark> gizmach: that l var is a PEP8 violation if you work later on on that class again please rename it to a better one
2008-06-23T22:26:10  <gizmach> dreimark: will do
2008-06-23T22:27:29  <dreimark>  def luser(self, filterstr, ldap_object, attrs=[getattr(self, attr) for attr in [ .... ) as default
2008-06-23T22:27:58  <dreimark> then you can call it with NONE or any other attrs
2008-06-23T22:28:05  <dreimark> None
2008-06-23T22:28:43  <dreimark> gizmach:
2008-06-23T22:29:09  <gizmach> ah yes it didn't have to be the if
2008-06-23T22:29:16  <gizmach> will remove it now
2008-06-23T22:30:00  <CIA-52> MelitaMihaljevic default * 3682:33b4a35b6cfe 1.8-ldapgroups-mmihaljevic/MoinMoin/auth/_tests/test_ldap_login.py: added testMemberOFGroup for testing memberOfGroup method in ldap_login
2008-06-23T22:31:10  <gizmach> dreimark: but I need to leave if
2008-06-23T22:31:25  <gizmach> if not it will take the attrs that are defined in the class
2008-06-23T22:31:33  <gizmach> or not
2008-06-23T22:31:48  <gizmach> ah I see
2008-06-23T22:31:55  <gizmach> forget what I aked
2008-06-23T22:31:58  <dreimark> gizmach: in the first hunk you missed to add   filterstr.encode(coding) for the call  lusers = self.luser(filterstr, l, attrs=True)
2008-06-23T22:32:04  <gizmach> :)
2008-06-23T22:40:48  <ThomasWaldmann> gizmach: why do you call luser twice?
2008-06-23T22:41:37  <gizmach> well, I could make to called that once
2008-06-23T22:41:40  <gizmach> ThomasWaldmann:
2008-06-23T22:41:45  <gizmach> wasn't thinking
2008-06-23T22:43:41  <ThomasWaldmann> and please use more sane attr names
2008-06-23T22:44:15  <ThomasWaldmann> call a thing like what it really is, not like something else what it is not
2008-06-23T22:44:40  <gizmach> ok
2008-06-23T22:45:33  <ThomasWaldmann> e.g. luser() is wrongly named
2008-06-23T22:45:57  <gizmach> :)
2008-06-23T22:45:58  <ThomasWaldmann> user_filter same
2008-06-23T22:46:49  <ThomasWaldmann> lusers same
2008-06-23T22:47:39  <dreimark> btw. not gizmach fault old code has had this naming convention
2008-06-23T22:48:01  <gizmach> but I could change it for my purpose
2008-06-23T22:48:18  <gizmach> and will fix that l
2008-06-23T22:48:49  <dreimark> yeah s/could/must/ :)
2008-06-23T22:48:58  <ThomasWaldmann> I guess the old code dealt with users :)
2008-06-23T22:49:36  <ThomasWaldmann> gizmach: and the transformation from found groups to moin-usable group names has to be checked
2008-06-23T22:49:53  <ThomasWaldmann> maybe that needs to be a configurable function, ask the ldap geeks
2008-06-23T22:50:21  <gizmach> ThomasWaldmann: ok I will
2008-06-23T22:50:25  <gizmach> dreimark: ;)
2008-06-23T22:51:42  <ThomasWaldmann> and you need a .decode(...) for the groups
2008-06-23T22:52:42  <gizmach> yes but I was thinking to put it in refactored wikidicts
2008-06-23T22:52:58  <ThomasWaldmann> that is lowlevel code
2008-06-23T22:53:18  <ThomasWaldmann> decode early, encode late
2008-06-23T22:54:06  <gizmach> ok
2008-06-23T22:56:55  <gizmach> but first few hours of sleep
2008-06-23T22:57:19  <ThomasWaldmann> |-)
2008-06-23T22:59:44  <dreimark> good night gizmach
2008-06-23T23:00:36  <gizmach> goodn night all
2008-06-23T23:01:10  <ThomasWaldmann> gizmach: Each result tuple is of the form (dn, attrs), where dn is a string containing the DN (distinguished name) of the entry, and attrs is a dictionary containing the attributes associated with the entry.
2008-06-23T23:01:55  <waldi> (requested attributes)
2008-06-23T23:02:00  <ThomasWaldmann> (so if you searched for FOOs, foo_dn is the thing you get with [0][0])
2008-06-23T23:02:30  <ThomasWaldmann> waldi: yeah
2008-06-23T23:02:55  <waldi> and always specify the needed attributes. or the server will hate you
2008-06-23T23:03:06  <ThomasWaldmann> gizmach: gn, dream of well-commented, easily readable code :D
2008-06-23T23:21:43  <xorAxAx> hmm, the 1.7 search seems useless. try searching for theme market on the moin wiki
2008-06-23T23:25:14  <dreimark> not useless but somehow too much output
2008-06-23T23:25:50  <xorAxAx> wrong order
2008-06-23T23:27:51  <ThomasWaldmann> yeah, result ordering could be better
2008-06-23T23:28:14  <ThomasWaldmann> (btw it was worse a few weeks ago X)
2008-06-23T23:28:34  <xorAxAx> and i dont think that anybody wants to download an image from the search results
2008-06-23T23:28:44  <xorAxAx> (so it should use the view operation and not the get one)
2008-06-23T23:28:59  <dreimark> I think i would prefer something similiar to a contents (TOC on pages) where you see subitems below an item
2008-06-23T23:29:35  <dreimark> curretnly subitems are shon in the result on the same level as items