MoinMoin: MoinMoinChat/Logs/moin-dev/2008-11-22

2008-11-22T09:11:48  <dreimark> moin
2008-11-22T10:49:30  <dreimark> bbl
2008-11-22T11:48:44  <ThomasWaldmann>   moin
2008-11-22T16:30:05  * ThomasWaldmann debugs Load action
2008-11-22T16:57:14  <CIA-58> Thomas Waldmann <tw AT waldmann-edv DOT de> default * 4430:97ecf28b5969 1.9/MoinMoin/action/Load.py: fix Load action to use WSGI/werkzeug API correctly
2008-11-22T19:58:38  <dreimark> re
2008-11-22T20:04:20  <ThomasWaldmann> hi dreimark
2008-11-22T20:04:53  <dreimark> I see meanwhile one bug is fixed
2008-11-22T20:07:25  <ThomasWaldmann> the pygments problem is strange
2008-11-22T20:11:39  <TheSheep> ThomasWaldmann: what is the pygments problem?
2008-11-22T20:12:18  <dreimark> TheSheep: MoinMoinBugs/1.9Requires_pygments.lexers
2008-11-22T20:12:44  <dreimark> it has a problem with import if i deploy with cgi
2008-11-22T20:12:45  <TheSheep> dreimark: old version of pygments?
2008-11-22T20:12:58  <dreimark> which is required.
2008-11-22T20:13:04  <dreimark> Is this already documented?
2008-11-22T20:13:19  <TheSheep> no, I worked with current, and that's what attached to moin
2008-11-22T20:13:36  <TheSheep> which version do you have?
2008-11-22T20:14:30  <dreimark> 0.9-2
2008-11-22T20:14:50  <TheSheep> 0.10-1 here
2008-11-22T20:14:57  <dreimark> TheSheep: it works in standalone and wsgi
2008-11-22T20:15:14  <TheSheep> dreimark: probably because it's using the one attached to moin then
2008-11-22T20:15:38  <dreimark> yeah, that is a good explaination
2008-11-22T20:16:06  * dreimark tries to update
2008-11-22T20:16:09  <TheSheep> many webservers don't set environemnt for cgi scripts, things like path and pythonpath
2008-11-22T20:16:21  <TheSheep> and cwd
2008-11-22T20:17:17  <dreimark> and but then we better patch flup because that is also in the moin path
2008-11-22T20:17:24  <dreimark> -and
2008-11-22T20:17:51  <TheSheep> or make the moin.cgi add apropriate path to sys.path
2008-11-22T20:17:54  <dreimark> no
2008-11-22T20:17:58  <dreimark> sorry
2008-11-22T20:18:03  <dreimark> MoinMoin/parser/highlight.py
2008-11-22T20:18:17  <dreimark> calls import pygments
2008-11-22T20:20:29  <dreimark> TheSheep: highlight does not import the moin version of pygments
2008-11-22T20:21:02  <TheSheep> dreimark: I thought the support dir is added to the python path?
2008-11-22T20:22:28  <dreimark> I don't thinks so. on many other places we import similiar to from MoinMoin.support.....
2008-11-22T20:22:52  <TheSheep> ah, then it's my mistake
2008-11-22T20:29:29  <dreimark> ok should I wait or should i try the Load fix in my environment
2008-11-22T20:30:49  <dreimark> which deployments do we currently cover by test setups for 1.9?
2008-11-22T20:31:08  * dreimark has currently standalone, cgi and cgi with http_auth
2008-11-22T20:32:49  <dreimark> test19 is by mod_wsgi
2008-11-22T20:35:21  <ThomasWaldmann> dreimark: iirc there is some place that adds MoinMoin.support to sys.path. new in 1.9.
2008-11-22T20:37:43  <dreimark> then this does not work. me dislikes currently to change all the from MoinMoin.support ... lines
2008-11-22T20:38:18  <dreimark> I would prefer this refactoring after 1.9 and a clean situation now.
2008-11-22T20:40:43  <ThomasWaldmann> dreimark: maybe you can find out why it doesn't work for cgi
2008-11-22T20:40:55  <ThomasWaldmann> moin should behave same for all serving methods
2008-11-22T20:43:22  * dreimark is looking already
2008-11-22T20:50:58  * ThomasWaldmann looks at the session delete problem
2008-11-22T21:54:14  <dreimark> sys.path is correct using cgi and standalone
2008-11-22T22:26:25  <dreimark> ha
2008-11-22T22:26:31  <dreimark> TheSheep: ThomasWaldmann got it
2008-11-22T22:26:48  <ThomasWaldmann> ?
2008-11-22T22:27:01  <dreimark> lexers is missing from built
2008-11-22T22:27:09  <dreimark> so it is not installed
2008-11-22T22:28:21  <dreimark> all subdirs of pygments are missing
2008-11-22T22:28:51  <ThomasWaldmann> setup.py likely needs some more fixing then
2008-11-22T22:29:43  <dreimark> manually copying theses dirs fixed it
2008-11-22T22:31:55  * dreimark fixes setup.py
2008-11-22T22:32:40  <TheSheep> one of these days I'm going to learn to use setuptools
2008-11-22T22:33:36  <TheSheep> dreimark: so it was breaking *because* it used the provided pygments?
2008-11-22T22:33:38  * dreimark will get to lear to do always http://moinmo.in/GoogleSoc2008/HowToTest
2008-11-22T22:34:05  <dreimark> ThomasWaldmann: we have an attachment spammer on MM
2008-11-22T22:35:21  <dreimark> TheSheep: yes my local version has no lexers
2008-11-22T22:35:29  <dreimark> hmm it has
2008-11-22T22:36:02  <dreimark> but it tells it can't import
2008-11-22T22:37:33  <dreimark> I think it fails because it does not go to the fall back version
2008-11-22T22:37:59  <dreimark> it founds pygments and tries to load
2008-11-22T22:49:46  <CIA-58> Reimar Bauer <rb.proj AT googlemail DOT com> default * 4431:436048b821a5 1.9/setup.py: setup: fixed missing pygments subdirs
2008-11-22T23:09:23  <dreimark> it snows here
2008-11-22T23:10:48  <TheSheep> yeah, here too
2008-11-22T23:12:17  <dreimark> looks like I better go tom. by train to Aachen.
2008-11-22T23:12:37  <dreimark> mostly at the first winter day they drive crazy
2008-11-22T23:23:17  <dreimark> hmm, do we drop fcgi ? I don't see a server script
2008-11-22T23:31:15  <dreimark> ThomasWaldmann: ^
2008-11-22T23:34:51  <ThomasWaldmann> it is done by flup (hopefully)
2008-11-22T23:39:51  <dreimark> seems it is not implemented it has: CGI, SCGI, AJP
2008-11-22T23:40:28  <dreimark> aha, it has for cgi also server_types flup.server.fcgi
2008-11-22T23:40:51  * dreimark will setup tom. a wiki for that too