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

Fabiano Fidêncio fidencio at redhat.com
Mon Feb 18 07:58:52 UTC 2019


On Thu, Feb 14, 2019 at 6:28 PM Cole Robinson <crobinso at redhat.com> wrote:

> On 12/21/18 9:09 AM, Fabiano Fidêncio wrote:
> > Signed-off-by: Fabiano Fidêncio <fidencio at redhat.com>
> > ---
> >  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 at redhat.com>
>
> - Cole
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/libosinfo/attachments/20190218/b9f1e219/attachment.htm>


More information about the Libosinfo mailing list