src/qfriendslol.pyw hashbang needlessly set to python2.6

Bug #471252 reported by Wouter van Heyst
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenFilepad
Fix Committed
Critical
Toni Ruottu

Bug Description

The 0.1.1 release of qfriendslol src/qfriendslol.pyw starts out like this:

    #!/usr/bin/python2.6

As I run Debian Lenny which does, alas, not contain python2.6, this does not work for me.
Changing it to python2.5 seems to run fine though.

I would recommend chaging the first line to:

    #!/usr/bin/env python

to ensure a much wider range of python interpreters can pick up the task. If there really is need for a specific newer python and you don't want a SyntaxError to cryptically communicate that, 'sys.version_info' can be of help. Or see the evil reinvoke hack that bzr uses.

Related branches

Changed in qfriendslol:
assignee: nobody → Toni Ruottu (toni-ruottu)
status: New → Confirmed
Revision history for this message
Sami Saada (samitheberber) wrote :

If I remember correctly, Toni said something about developing at first with python 2.6, because all maintainer had it and he also had python 3. If I'm correct, python 3 is backward compatible in most of case and I agree you opinion. However I let Toni to decide what to do.

Changed in qfriendslol:
importance: Undecided → Critical
milestone: none → 0.1.2
Revision history for this message
Toni Ruottu (toni-ruottu) wrote :

I wasn't aware of a sys.version_info, but checking for correct version with that and throwing a pop up "You have python 3.0 installed. QFriendsLOL is only able to run on Python versions 2.5 - 2.6".

What will happen, if user has both Python 2.6 and Python 3.0 installed, while /usr/bin/python links to 3.0?
Won't use of "#!/usr/bin/env python" then break our software?

Changed in qfriendslol:
assignee: Toni Ruottu (toni-ruottu) → nobody
Revision history for this message
Sami Saada (samitheberber) wrote :

There is also /usr/bin/python2 which might be the correct one for this?

Changed in qfriendslol:
assignee: nobody → Toni Ruottu (toni-ruottu)
status: Confirmed → Fix Committed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.