xpra icon
Bug tracker and wiki

This bug tracker and wiki are being discontinued
please use https://github.com/Xpra-org/xpra instead.

Opened 10 years ago

Closed 10 years ago

Last modified 16 months 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
+    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 10 years ago.

Download all attachments as: .zip

Change History (7)

Changed 10 years ago by James C. McPherson

Attachment: pythonpath.diffs added

comment:1 Changed 10 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 10 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 10 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 10 years ago by Antoine Martin

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

comment:5 Changed 10 years ago by Antoine Martin

Milestone: current0.0.7.x

comment:6 Changed 16 months ago by migration script

this ticket has been moved to: https://github.com/Xpra-org/xpra/issues/79

Note: See TracTickets for help on using tickets.