Jakub Wilk <jw...@debian.org> writes:

> Hi Dmitrijs,
>
>>+@@ -145,7 +145,7 @@
>>+             line = line.rstrip("\n")
>>+             if fileType == "sh":
>>+                 if line == "#PYTHONPATH":
>>+-                    script.append('PYTHONPATH="%s:$PYTHONPATH"' % 
>>(install_lib,))
>>++                    script.append('PYTHONPATH="%s:+:$PYTHONPATH"' % 
>>(install_lib,))
>>+                 elif line == "#PATH":
>>+                     script.append('PATH="%s:$PATH"' % (os.path.join(base, 
>>"bin"),))
>>+                 else:
>
> This patch looks very suspicious. Have you actually tested it?
>
> Besides, as far as I can see, /usr/bin/caldavd doesn't need PYTHONPATH 
> mangling
> at all.
Now to answer your question =))))

No it doesn't seem to need 1st PYTHONPATH mangling, but I think it does need the
second one:

export PYTHONPATH=`${python} -c 'import sys; print 
"/usr/lib/twisted-calendarserver/lib/python%s/site-packages" % 
(sys.version[:3])'`

But this one is "secure" way to set it.

Attachment: pgpjx2BUfOCh4.pgp
Description: PGP signature

Reply via email to