Code review request for Linux version id fix (6986195)

Gerald Thornbrugh gerald.thornbrugh at oracle.com
Mon Sep 16 10:35:28 PDT 2013


Hi Omair,

Thanks for the review!

Jerry
> On 09/16/2013 01:31 PM, Gerald Thornbrugh wrote:
>> OK, one more time:
>>
>> 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 system-release file seems to be
>> an old standard
>> 2184 // that has been replaced by the lsb-release and os-release files.
>>
> Looks good to me. Thanks for updating this!
>
> Cheers,
> Omair
>



More information about the serviceability-dev mailing list