Issue1552

<script language="jython" setbeans="false">
<![CDATA[
import sys
for x in sys.path:
    print x
]]>
</script>

This is a problem in other places that create an embedded jython via the Java API (new PythonInterpreter()), like modjy and likely the jsr223 support as well

The new PythonInterpreter doesn't import site like the command line does. Alan Kennedy and I agreed that we should change this (mostly for modjy's sake) and create a flag to disable the behavior. CPython already acts this way when you create an embedded interpreter via its C API

Unfortunately I forgot to change this before the 2.5 release. It's a fairly significant change so it'll have to wait for 2.6

Deferring this breaking change to 2.7

Alan, Philip: was this done for 2.7? Can this be closed?

AFAIK it has not been done yet

This issue is about 'import'ing site into all embedded interpreters, by default, which does not happen right now, and should.

Back when the issue was reported, I put special coding into modjy to handle this.

http://hg.python.org/jython/rev/933c4138dad6

Now that we're changing major versions (2.5 to 2.7 is major), it's time to put that right, and remove that support from modjy and push it down into PySystemState.

Question is, how do we enable the user to switch it off, akin to the "-S" command line option?

I think we need a new registry property for this, the default value of which causes loading of site.py.

Possibilities include

A: "python.site", a brand new property, which defaults to "true".
B: "python.options.site", a brand new property, which defaults to "true".


I will undertake the relevant work.

When this is complete, this is a fairly major change that will affect all users who embed jython, a very substantial part of our user base, I believe.

So we'll have to get our messaging right on this.

There's an outstanding pull request https://bitbucket.org/jython/jython/pull-request/15/ensure-that-the-site-module-is-imported/diff that we should apply. Note that it does not yet also change org.python.core.Options such that this default can be changed. Per other such option settings, I would suggest we use python.options.importSite as the actual setting name.

Should be part of beta 4 at the latest

So the PR is fine, we just need to add some testing to get it into trunk. We will control with the importSite option.

Fixed as of http://hg.python.org/jython/rev/6d3e8e67af1d