[Libosinfo] [PATCH 06/10] Fix SLE derives from information

[Charles Arnold]

>>> On 12/13/2016 at 05:04 AM, "Daniel P. Berrange" <berrange at redhat.com> wrote: 
> On Mon, Dec 12, 2016 at 03:43:06PM -0700, Charles Arnold wrote:
> 
>> diff --git a/data/os/suse.com/sled-11.xml.in 
> b/data/os/suse.com/sled-11.xml.in
>> index 589de8a..39e3f8c 100644
>> --- a/data/os/suse.com/sled-11.xml.in
>> +++ b/data/os/suse.com/sled-11.xml.in
>> @@ -8,7 +8,8 @@
>>      <_vendor>SUSE</_vendor>
>>      <family>linux</family>
>>      <distro>sled</distro>
>> -    <upgrades id="http://suse.com/sled/10"/>
>> +    <upgrades id="http://suse.com/sled/10.4"/>
>> +    <derives-from id="http://suse.com/sled/10.4"/>
> 
> There's a bug here - there's no file defining sled 10.4 currently. Did you
> forget to add that file ?

Oops. Omitted an entire patch containing the SLE 10 distros.

> 
>> diff --git a/data/os/suse.com/sles-11.xml.in 
> b/data/os/suse.com/sles-11.xml.in
>> index a031c6b..d982195 100644
>> --- a/data/os/suse.com/sles-11.xml.in
>> +++ b/data/os/suse.com/sles-11.xml.in
>> @@ -8,7 +8,8 @@
>>      <_vendor>SUSE</_vendor>
>>      <family>linux</family>
>>      <distro>sles</distro>
>> -    <upgrades id="http://suse.com/sles/10"/>
>> +    <upgrades id="http://suse.com/sles/10.4"/>
>> +    <derives-from id="http://suse.com/sles/10.4"/>
> 
> Same issue here

I will post a couples patches to add the missing SLE 10 distros
and point this patch back to 10.4.

Thanks,

- Charles 

> 
> 
> Regards,
> Daniel
> -- 
> |: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ 
> :|
> |: http://libvirt.org              -o-             http://virt-manager.org 
> :|
> |: http://entangle-photo.org       -o-    http://search.cpan.org/~danberr/ 
> :|