phpsysinfo doesn't detect Ubuntu

Bug #2399 reported by Fabio Marzocca on 2005-09-18
Affects Status Importance Assigned to Milestone
phpsysinfo (Ubuntu)
Adrien Cunin

Bug Description

When running phpsysinfo on Ubuntu, in the Sistem Vital description the Distro Name is reported as Debian unstable/testing and debian's icon is displayed... This is because application is gathering the distro name from the file /etc/debian_version, and in ubuntu that file contains "unstable/testing". I had to manually patch the file /usr/share/phpsysinfo/includes/os/ replacing Debian with Ubuntu Breezy, and loading the correct icon..

Changed in phpsysinfo:
assignee: nobody → motu
Stephan Rügamer (sruegamer) wrote :

which version of php? 4 or 5?



Fabio Marzocca (thesaltydog) wrote :


php is version 4, but the bug is not related to php. As I reported, the code inside the program looks for the distro name into /etc/debian_version and that file contains "unstable/testing" instead of "05.10" or "Breezy".

Andrew Mitchell (ajmitch) wrote :

Which means that the more reliable way might be to use lsb_release

Fabio Marzocca (thesaltydog) wrote :

Yor're right, that program line should be changed

Vassilis Pandis (pandisv) wrote :

The program appears to have changed. Taking a look at distros.ini, it seems that /etc/lsb-release is used and the distro reported is "Ubunto".

Vassilis Pandis (pandisv) wrote :

I'm attaching a debdiff that PROBABLY fixes the issue. I am not sure it does, since I can't test it.

Changed in phpsysinfo:
assignee: motu → motureviewers
Vassilis Pandis (pandisv) wrote :

I tested it and it doesn't. Do NOT upload please.

Changed in phpsysinfo:
assignee: motureviewers → motu
Changed in phpsysinfo:
status: Unconfirmed → Confirmed
Vassilis Pandis (pandisv) wrote :

There's a patch upstream. I haven't tested it.

Vassilis Pandis (pandisv) wrote :

Upstream have committed fix to CVS head. This has not been fixed in 2.5.2's release.

Changed in phpsysinfo:
status: Confirmed → Fix Committed
Adrien Cunin (adri2000) on 2007-01-14
Changed in phpsysinfo:
assignee: motu → adri2000
Adrien Cunin (adri2000) wrote :

phpsysinfo (2.5.2-0ubuntu1) feisty; urgency=low

  * New upstream release
  * debian/patches:
     - configfile.diff, hddtemp-config.diff, samsung-hd-fix.diff: updated
  * Detect Ubuntu properly (Closes: Malone #2399)
     - Added detect-ubuntu.patch from upstream CVS
     - Converted upstream gif Ubuntu icon to xpm to ship it in the diff.gz
     - Build-Depends on imagemagick to convert the xpm to gif in debian/rules

 -- Adrien Cunin <email address hidden> Sun, 14 Jan 2007 16:00:58 +0100

Changed in phpsysinfo:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Duplicates of this bug

Other bug subscribers