/usr/bin/gwibber-accounts has bogus #! line

Bug #988385 reported by Barry Warsaw
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Gwibber
Fix Released
Low
Unassigned
gwibber (Ubuntu)
Fix Committed
Low
Unassigned
Precise
Fix Released
Low
Unassigned

Bug Description

% head -n 1 /usr/bin/gwibber-accounts
#!/usr/bin/env python

This is destined to break. It is never a good idea for production code to use /usr/bin/env. Instead, it should use #!/usr/bin/python just like gwibber-service does.

Revision history for this message
Barry Warsaw (barry) wrote :

Note that it's okay for the *development* branch to use /usr/bin/env, in fact, it's preferable so that it's easier to develop against alternative Python's. Once deployed though, it must use /usr/bin/python.

Changed in gwibber:
assignee: nobody → Ken VanDine (ken-vandine)
importance: Undecided → Low
milestone: none → 3.4.1
status: New → Confirmed
Bilal Shahid (s9iper1)
Changed in gwibber (Ubuntu):
status: New → Triaged
importance: Undecided → Low
Changed in gwibber:
status: Confirmed → Fix Committed
Changed in gwibber:
status: Fix Committed → Fix Released
assignee: Ken VanDine (ken-vandine) → nobody
Revision history for this message
Martin Pitt (pitti) wrote : Please test proposed package

Hello Barry, or anyone else affected,

Accepted gwibber into precise-proposed. The package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

Changed in gwibber (Ubuntu Precise):
status: Triaged → Fix Committed
tags: added: verification-needed
Revision history for this message
Sebastien Bacher (seb128) wrote :

Confirming that the new version uses the correct interpreter line

tags: added: verification-done
removed: verification-needed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package gwibber - 3.4.1-0ubuntu1

---------------
gwibber (3.4.1-0ubuntu1) precise-proposed; urgency=low

  * New stable release
    - fallback to displaying full sender's name if the nickname is empty, even
      if show-fullname is false (LP: #985801)
    - catch GLib.GError instead of GLib.Error, fixes crasher (LP: #940182)
    - Specify /usr/bin/python in gwibber-accounts instead of relying on
      env (LP: #988385)
    - lens: use category icons that match the rest of unity. This only affects
      the unity-lens-gwibber package in universe.
  * debian/patches/*.patch
    - dropped, merged upstream
  * debian/control
    - Added depends on gir1.2-wnck-3.0, gwibber-accounts needs it (LP: #957272)
 -- Ken VanDine <email address hidden> Wed, 25 Apr 2012 16:22:20 -0400

Changed in gwibber (Ubuntu Precise):
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

Remote bug watches

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