Re: [osinfo-db PATCH 8/9] centos: Add "centos7" as short-id

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

 





On Thu, Feb 14, 2019 at 6:28 PM Cole Robinson <crobinso@xxxxxxxxxx> wrote:
On 12/21/18 9:09 AM, Fabiano Fidêncio wrote:
> Signed-off-by: Fabiano Fidêncio <fidencio@xxxxxxxxxx>
> ---
>  data/os/centos.org/centos-7.0.xml.in | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/data/os/centos.org/centos-7.0.xml.in b/data/os/centos.org/centos-7.0.xml.in
> index 8b3531e..cc583b9 100644
> --- a/data/os/centos.org/centos-7.0.xml.in
> +++ b/data/os/centos.org/centos-7.0.xml.in
> @@ -3,6 +3,7 @@
>       See http://www.gnu.org/licenses/ for a copy of the license text -->
>    <os id="http://centos.org/centos/7.0">
>      <short-id>centos7.0</short-id>
> +    <short-id>centos7</short-id>
>      <_name>CentOS 7.0</_name>
>      <version>7.0</version>
>      <_vendor>CentOS</_vendor>
>

Huh I didn't know we could do that, but I see we do the same for debian
distros, so it makes sense


Cole, I will end up dropping this patch.
While we can do that (as in our schema allows that), there's no code on libosinfo that would properly deal with a second short-id.

So, for now, I'll not go for this patch.
 
Reviewed-by: Cole Robinson <crobinso@xxxxxxxxxx>

- Cole
_______________________________________________
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