[patch icedtea/openjdk] print more accurate information about the distribution

Matthias Klose doko at ubuntu.com
Wed Mar 12 06:58:59 PDT 2008


Mark Wielaard schrieb:
> Hi Matthias,
> 
> On Tue, 2008-03-11 at 17:27 +0100, Matthias Klose wrote:
>> Ubuntu doesn't use (and therefore doesn't change) /etc/debian_version, but uses
>> lsb_release for more accurate information about the running distribution.  Allow
>> to read /etc/lsb-release (which has a list of key/value pairs before trying to
>> read /etc/debian_version.
>>
>> However the file /etc/lsb-release is not standardized, forwarded this to
>> lsb-discuss at lists.linux-foundation.org.
>>
>> ok to checkin?
> 
> Is this any different from the last post?
> http://mail.openjdk.java.net/pipermail/distro-pkg-dev/2008-March/001317.html
> Could you look at the comments made to that one?
> http://mail.openjdk.java.net/pipermail/distro-pkg-dev/2008-March/001328.html

Please find applied the patch including the bits to apply the patch.

> Shouldn't this be a while loop to guard against short reads?

Well, then the function fails and then tries to read /etc/debian_version; as
said the format of this file is not specified, and on this particular
implementation it is not longer than 500 chars. I didn't get yet feedback from
the lsb-discuss ML.

  Matthias
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: print-lsb.diff
Url: http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20080312/552bdd2b/print-lsb.diff 


More information about the distro-pkg-dev mailing list