sensors.py sys_get_distrib_name() is broken

Bug #264856 reported by Przemysław Firszt
2
Affects Status Importance Assigned to Milestone
Universal Applets
Fix Committed
Undecided
Przemysław Firszt

Bug Description

sensory.py lines 236/237

  elif os.path.exists('/etc/lsb-release'):
  return 'Debian ' + str(commands.getoutput('cat /etc/debian_version'))

I'm sure checking if /etc/lsb-release exist is not enough to assume that it's a debian. Needs to be fixed by somebody with a debian.

Revision history for this message
Natan Yellin (aantny) wrote :

/etc/debian_version can be used to check if a distro is debian or a debian derivative. (E.g. on ubuntu, the file contains 'lenny/sid')

Revision history for this message
Przemysław Firszt (pefi) wrote :

Bug fixed, see rev. 497. Aantn, thanks for the information!

Changed in universal-applets:
assignee: nobody → pefi
status: New → 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.