ladon-dev-team team mailing list archive

[asfaltboy <asfaltboy@xxxxxxxxx>]

asfaltboy has proposed merging lp:~asfaltboy/ladon/fix-ladon-ctl-shebang into lp:ladon.

Requested reviews:
  Ladon Developer (ladon-dev-team)
Related bugs:
  Bug #1187658 in ladon: "ladon-ctl missing Python shebang or extension"
  https://bugs.launchpad.net/ladon/+bug/1187658

For more details, see:
https://code.launchpad.net/~asfaltboy/ladon/fix-ladon-ctl-shebang/+merge/205912

Simple fix of the python shebang which uses the env's declared python. 

Since `setup.py` already sets the "scripts" key, it automatically, when used with a tool such as pip, converts the shebang to the current env's python interpretor full path.
-- 
https://code.launchpad.net/~asfaltboy/ladon/fix-ladon-ctl-shebang/+merge/205912
Your team Ladon Developer is requested to review the proposed merge of lp:~asfaltboy/ladon/fix-ladon-ctl-shebang into lp:ladon.

=== modified file 'frameworks/python/scripts/ladon-ctl'
--- frameworks/python/scripts/ladon-ctl	2011-08-08 23:21:30 +0000
+++ frameworks/python/scripts/ladon-ctl	2014-02-12 08:47:15 +0000
@@ -1,3 +1,4 @@
+#!/usr/bin/env python
 # -*- coding: utf-8 -*-
 
 from os.path import abspath,dirname,basename

=== modified file 'frameworks/python/scripts/ladon2.7ctl'
--- frameworks/python/scripts/ladon2.7ctl	2011-12-11 22:51:56 +0000
+++ frameworks/python/scripts/ladon2.7ctl	2014-02-12 08:47:15 +0000
@@ -1,4 +1,4 @@
-#!/usr/bin/python
+#!/usr/bin/env python
 # -*- coding: utf-8 -*-
 
 from os.path import abspath,dirname,basename