xpra icon
Bug tracker and wiki

Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#79 closed enhancement (invalid)

removing need to set PYTHONPATH in the shell

Reported by: James C. McPherson Owned by: Antoine Martin
Priority: minor Milestone: 0.0.7.x
Component: client Version:
Keywords: Cc:

Description (last modified by Antoine Martin)

I've got xpra running on a development build of Solaris 11. Getting started
took a while because I forgot to set PYTHONPATH before invoking the xpra script.

To remove the need for this, I added the following lines to it:

-import sys
+import sys, os
+sys.path.insert(0,
+    os.path.realpath(os.path.join(os.path.dirname(__file__), "../lib/python")))

Full patches for each invocation script are attached.

Attachments (1)

pythonpath.diffs (1.2 KB) - added by James C. McPherson 9 years ago.

Download all attachments as: .zip

Change History (6)

Changed 9 years ago by James C. McPherson

Attachment: pythonpath.diffs added

comment:1 Changed 9 years ago by James C. McPherson

ahem ... that should by "../lib/python2.6" rather than "../lib/python". Unless you create a symlink during the installation process.

comment:2 Changed 9 years ago by Antoine Martin

Description: modified (diff)
Resolution: invalid
Status: newclosed

Sorry, but you should be installing Xpra system-wide, in which case it will be in your PYTHONPATH:

python make_constants_pxi.py wimpiggy/lowlevel/constants.txt wimpiggy/lowlevel/constants.pxi
CFLAGS=-O0 python setup.py install

If you chose not to do that, then obviously you will need to adjust PYTHONPATH to match your environment, changing the code is not an acceptable solution.

comment:3 Changed 9 years ago by James C. McPherson

I note that sometimes on multiuser machines where you do not have root access, you do need to make that change, or write a wrapper script which sets PYTHONPATH for you. *shrug* The patch was offered in the hope that it might help somebody who came across a similar issue.

comment:4 Changed 9 years ago by Antoine Martin

r495 adds an INSTALL file with (hopefully) clearer information.

comment:5 Changed 9 years ago by Antoine Martin

Milestone: current0.0.7.x
Note: See TracTickets for help on using tickets.