Debian
python-defaults package

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

  1. Squeeze (6.0)
  2. bug-632020
  3. Merge into squeeze
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
1=== modified file 'debian/python-minimal.postinst'
2--- debian/python-minimal.postinst 2010-07-11 17:08:19 +0000
3+++ debian/python-minimal.postinst 2010-09-06 22:41:03 +0000
4@@ -1,6 +1,6 @@
5 #! /bin/sh
6 set -e
7
8-python -m compileall /usr/share/python/ >/dev/null
9+/usr/bin/python -m compileall /usr/share/python/ >/dev/null
10
11 #DEBHELPER#

Subscribers

People subscribed via source and target branches