Greetings,

I am sponsoring the following code review for Jerry Thornbrugh.
Since this code review request is going to two aliases, please make
sure you include BOTH aliases in your replies.

Dan



OpenJDK bug:

6986195 correctly identify Ubuntu as the operating system in crash
               report instead of "Debian"
http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=6986195

Internal (for now) bug:

JDK-6986195 correctly identify Ubuntu as the operating system in crash
                      report instead of "Debian"
https://bugs.openjdk.java.net/browse/JDK-6986195


External Webrev:

http://cr.openjdk.java.net/~dcubed/for_gthornbr/6986195-webrev/

Internal Webrev:

http://javaweb.us.oracle.com/~gthornbr/6986195-webrev/


Background:

The current Linux print_distro_info() function incorrectly reports an Ubuntu OS as a Debian OS because it does not look for the Ubuntu release information file (/etc/lsb-release) and instead finds the Debian release information file (/etc/debian_version). Because of this
Ubuntu releases are identified as Debian releases.

Here are some other issues that I have found:

The "lsb-release" file is the new Linux standard so it is now in the list.

Some Debian OS's have very poor /etc/debian_version files that report something like "wheesy/sid". Now a "Debian " in printed before the contents of the /etc/debian_version file is printed for better
identification.

Some Linuxes based on Redhat have their own "xxx-release" files. Because of this "xxx-release" files for Redhat based linuxes should be looked for before the "redhat-release" file.

Since Redhat has a "lsb-release" file that is not very descriptive the "redhat-release" file needs
to be looked for before the "lsb-release" file.

I also added the "system-release" and "os-release" files since they are an old standard.

I have complete some testing on all major Linux distributions and some minor ones.
The differences I have found between the new and old algorithm are below:

Ubuntu Linux Mint 15

Before (using /etc/debian_version):
.
.
.
---------------  S Y S T E M  ---------------

OS:wheezy/sid
.
.
.

After (using /etc/lsb-release):
.
.
.
---------------  S Y S T E M  ---------------

OS:DISTRIB_ID=LinuxMint
DISTRIB_RELEASE=15
DISTRIB_CODENAME=olivia
DISTRIB_DESCRIPTION="Linux Mint 15 Olivia"
.
.
.



Oracle Linux

Before (using /etc/redhat-release)
.
.
.
---------------  S Y S T E M  ---------------

OS:Red Hat Enterprise Linux Server release 6.3 (Santiago)
.
.
.

After (using /etc/oracle-release)
.
.
.
---------------  S Y S T E M  ---------------

OS:Oracle Linux Server release 6.3
.
.
.


Reply via email to