Re: [PATCH 06/10] Fix SLE derives from information

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



>>> On 12/13/2016 at 05:04 AM, "Daniel P. Berrange" <berrange@xxxxxxxxxx> 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/ 
> :|



_______________________________________________
Libosinfo mailing list
Libosinfo@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libosinfo



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Fedora Users]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]

  Powered by Linux