Re: [PATCH 03/11] centos, test: Add OS info/testcase for CentOS 6.0

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

 



On Sat, Mar 30, 2013 at 11:30:19PM +0100, Fabiano Fidêncio wrote:
> On Mon, Mar 25, 2013 at 9:50 AM, Christophe Fergeau <cfergeau@xxxxxxxxxx>wrote:
> 
> > On Sat, Mar 23, 2013 at 03:09:26PM +0100, Fabiano Fidêncio wrote:
> > > ---
> > >  data/oses/centos.xml.in                            | 22
> > ++++++++++++++++
> > >  .../centos6.0/CentOS-6.0-i386-bin-DVD.iso.txt      | 29
> > ++++++++++++++++++++++
> > >  .../centos6.0/CentOS-6.0-x86_64-bin-DVD1.iso.txt   | 29
> > ++++++++++++++++++++++
> > >  test/test-isodetect.c                              |  6 +++++
> > >  4 files changed, 86 insertions(+)
> > >  create mode 100644
> > test/isodata/centos/centos6.0/CentOS-6.0-i386-bin-DVD.iso.txt
> > >  create mode 100644
> > test/isodata/centos/centos6.0/CentOS-6.0-x86_64-bin-DVD1.iso.txt
> > >
> > > diff --git a/data/oses/centos.xml.in b/data/oses/centos.xml.in
> > > index 3f09dea..bb8e570 100644
> > > --- a/data/oses/centos.xml.in
> > > +++ b/data/oses/centos.xml.in
> > > @@ -16,13 +16,35 @@
> > >          <system-id>LINUX</system-id>
> > >          <volume-id>CentOS_6.0_Final</volume-id>
> > >        </iso>
> > > +      <kernel>isolinux/vmlinuz</kernel>
> > > +      <initrd>isolinux/initrd.img</initrd>
> > >      </media>
> > >      <media arch="x86_64">
> > >        <iso>
> > >          <system-id>LINUX</system-id>
> > >          <volume-id>CentOS_6.0_Final</volume-id>
> > >        </iso>
> > > +      <kernel>isolinux/vmlinuz</kernel>
> > > +      <initrd>isolinux/initrd.img</initrd>
> > >      </media>
> > > +
> > > +    <resources arch="all">
> > > +      <minimum>
> > > +        <n-cpus>1</n-cpus>
> > > +        <ram>536870912</ram>
> > > +      </minimum>
> > > +
> > > +      <recommended>
> > > +        <cpu>400000000</cpu>
> > > +        <ram>1073741824</ram>
> > > +        <storage>9663676416</storage>
> > > +      </recommended>
> > > +    </resources>
> > > +
> > > +    <installer>
> > > +      <script id='http://redhat.com/scripts/rhel/jeos'/>
> > > +      <script id='http://redhat.com/scripts/rhel/desktop'/>
> > > +    </installer>
> >
> > This gets back to a question I had while I was writing the openSUSE
> > patches, should we be duplicating this info, or should this be inherited
> > through the clones/derives-from elements if it's not set?
> >
> 
> IMHO, this info should be inherited through the clones/derives-from
> elements.
> Should I change it? :-)

If you feel like writing a patch for that, sure. Otherwise I guess we can
take this patch as is for now.

Christophe

Attachment: pgpQyVF_NOI2u.pgp
Description: PGP signature

_______________________________________________
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