Drop prebang in files that don't need one

Bug #404372 reported by Julian Aloofi
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Entertainer Media Center
Fix Released
Medium
Samuel Buffet

Bug Description

In the following files there is a #!/usrbin/env :
/entertainerlib/backend/backend_server.py
/entertainerlib/utils/preferences_dialog.py
/entertainerlib/utils/content_management_dialog.py
/entertainerlib/frontend/translation_setup.py

However these files are libraries and don't need to be executable. I'm trying to build packages for Fedora, could you please drop the prebang, as it's not working nicely with RPM packages.
Thanks in advance :D

Linux Distribution: Fedora 11
Processor (32/64-bit): 32-bit distro
Python Version: Python 2.6 (r26:66714, Jun 8 2009, 16:07:26)
Python Traceback: None

Tags: 0.4.2

Related branches

Julian Aloofi (nailuj24)
tags: added: 0.4.2
Revision history for this message
Samuel Buffet (samuel-buffet) wrote :

I can confirm that in trunk also (except the first file backend_server.py).
This is indeed useless.

Will fix that tonight.

Thanks,

Changed in entertainer:
status: New → Triaged
importance: Undecided → Medium
assignee: nobody → Samuel Buffet (samuel-buffet)
milestone: none → entertainer-0.5
Changed in entertainer:
status: Triaged → In Progress
Revision history for this message
Samuel Buffet (samuel-buffet) wrote :

Julian Aloofi hi.

Trunk has been fixed.

Thanks for the bug report and thanks for taking care of the RPM package.

Samuel-

Changed in entertainer:
status: In Progress → Fix Committed
Matt Layman (mblayman)
Changed in entertainer:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers