Sextant

Merge lp:~patrickas/ensoft-sextant/add_dev_script into lp:ensoft-sextant

  1. add_dev_script
  2. Merge into whiteline
Proposed by Patrick Stevens
Status: Merged
Approved by: ChrisD
Approved revision: 15
Merged at revision: 15
Proposed branch: lp:~patrickas/ensoft-sextant/add_dev_script
Merge into: lp:ensoft-sextant
Diff against target: 15 lines (+11/-0)
1 file modified
src/sextant.sh (+11/-0)
To merge this branch: bzr merge lp:~patrickas/ensoft-sextant/add_dev_script
Reviewer Review Type Date Requested Status
ChrisD Approve
Review via email: mp+233022@code.launchpad.net

Commit message

Add a script in src to allow execution of Sextant as a developer.

Description of the change

Add a script in src to allow execution of Sextant as a developer. It sets the PythonPath to be the right thing (whereas bin/sextant didn't necessarily).

To post a comment you must log in.
Revision history for this message
Patrick Stevens (patrickas) wrote :

Should I then remove the PythonPath-setting line from bin/sextant? That line reads:

export PYTHONPATH=$PYTHONPATH:$DIR/../src

which is probably not something we want in there if Pip has installed us.

Revision history for this message
ChrisD (gingerchris) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== added file 'src/sextant.sh'
2--- src/sextant.sh 1970-01-01 00:00:00 +0000
3+++ src/sextant.sh 2014-09-02 10:15:46 +0000
4@@ -0,0 +1,11 @@
5+#!/bin/bash
6+
7+DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )" && pwd )"
8+
9+export PYTHONPATH=${DIR}:${PYTHONPATH}
10+
11+# Allow people to specify which Python to use, by setting SEXTANT_PYTHON
12+# environment variable
13+export SEXTANT_PYTHON=${SEXTANT_PYTHON:=python}
14+
15+$SEXTANT_PYTHON -m sextant $@

Subscribers

People subscribed via source and target branches