Merge lp:~mwhudson/debian/squeeze/python-defaults/bug-632020 into lp:debian/squeeze/python-defaults

Proposed by Michael Hudson-Doyle
Status: Merged
Merge reported by: Michael Hudson-Doyle
Merged at revision: not available
Proposed branch: lp:~mwhudson/debian/squeeze/python-defaults/bug-632020
Merge into: lp:debian/squeeze/python-defaults
Diff against target: 11 lines (+1/-1)
1 file modified
debian/python-minimal.postinst (+1/-1)
To merge this branch: bzr merge lp:~mwhudson/debian/squeeze/python-defaults/bug-632020
Reviewer Review Type Date Requested Status
Ubuntu branches Pending
Review via email: mp+34708@code.launchpad.net

Description of the change

Hi, as described in the linked bug report, using a bare "python" in a postinst script is risky.

To post a comment you must log in.
Revision history for this message
Stefano Rivera (stefanor) wrote :

Any reason why this is filed against a Debian branch (which is read-only)? Can you not propose it as a patch in Debian's BTS?

Revision history for this message
Michael Hudson-Doyle (mwhudson) wrote :

I did, and it got fixed (although not in the way I suggested, oh well)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/python-minimal.postinst'
--- debian/python-minimal.postinst 2010-07-11 17:08:19 +0000
+++ debian/python-minimal.postinst 2010-09-06 22:41:03 +0000
@@ -1,6 +1,6 @@
1#! /bin/sh1#! /bin/sh
2set -e2set -e
33
4python -m compileall /usr/share/python/ >/dev/null4/usr/bin/python -m compileall /usr/share/python/ >/dev/null
55
6#DEBHELPER#6#DEBHELPER#

Subscribers

People subscribed via source and target branches