Code review request for Linux version id fix (6986195)

Omair Majid omajid at redhat.com
Mon Sep 16 10:07:26 PDT 2013


Hi Gerald,

On 09/16/2013 12:52 PM, Gerald Thornbrugh wrote:
> Sorry about the delay of my response.

No problem :)

> I chose to place the /etc/SuSE-release file in front of the
> /etc/os-release file because the
> SuSE-release file was more clear and concise than the os-release file.

I see. Yes, that does make a lot of sense.

> I propose the following change:
> 
> 2181 // Searching system-release (Red Hat) and os-release (other
> linuxes) are a
> 2182 // next to last resort.  The os-release file is a new standard that
> contains
> 2183 // distribution information and the os-release file seems to be an
> old standard
> 2184 // that has been  replaced by the lsb-release and os-release files.


There seems to be a thinko here: "The os-release file is a ... and the
os-release file seems to be ... that has been replaced by .. os-release"

If you mean there are two different 'standards' for os-release, maybe
it's better to say that.

Thanks,
Omair

-- 
PGP Key: 66484681 (http://pgp.mit.edu/)
Fingerprint = F072 555B 0A17 3957 4E95  0056 F286 F14F 6648 4681


More information about the hotspot-runtime-dev mailing list