intrigeri has proposed merging lp:~intrigeri/apparmor/utils-keep-shebang into
lp:apparmor.
Requested reviews:
AppArmor Developers (apparmor-dev)
For more details, see:
https://code.launchpad.net/~intrigeri/apparmor/utils-keep-shebang/+merge/332636
This patch by Adam Conrad <[email protected]> was added to the Ubuntu
packaging in the "Patches backported from upstream fix submissions" section, so
I assume it was meant to be merged upstream at some point?
I'm not sure I understand what's the rationale behind this change, the patch
doesn't really tell.
And If we want this upstream, I guess we'll want to drop the "if False" thing.
So at this stage, this MR is more a request for info from the Ubuntu folks than
a real "please apply upstream".
--
Your team AppArmor Developers is requested to review the proposed merge of
lp:~intrigeri/apparmor/utils-keep-shebang into lp:apparmor.
=== modified file 'utils/python-tools-setup.py'
--- utils/python-tools-setup.py 2015-01-13 21:03:11 +0000
+++ utils/python-tools-setup.py 2017-10-23 14:28:31 +0000
@@ -43,7 +43,7 @@
f = prefix + s
# If we have a defined python version, use it instead of the system
# default
- if 'PYTHON' in os.environ:
+ if False:
lines = open(os.path.basename(s)).readlines()
lines[0] = '#! /usr/bin/env %s\n' % os.environ['PYTHON']
open(f, 'w').write("".join(lines))
--
AppArmor mailing list
[email protected]
Modify settings or unsubscribe at:
https://lists.ubuntu.com/mailman/listinfo/apparmor
