[Libosinfo] [PATCH 2/2] fedora: Add F22 info

[Zeeshan Ali (Khattak)]

On Wed, May 27, 2015 at 4:15 PM, Zeeshan Ali (Khattak)
<zeeshanak at gnome.org> wrote:
> On Wed, May 27, 2015 at 3:38 PM, Christophe Fergeau <cfergeau at redhat.com> wrote:
>> On Wed, May 27, 2015 at 12:35:48AM +0100, Zeeshan Ali (Khattak) wrote:
>>> This time around, we also recommend RAM and storage to some nice sane
>>> values.
>>>
>>> Test data for each media is included.
>>> ---
>>>  data/oses/fedora.xml.in                            | 174 ++++++++++++++++++++-
>>>  .../Fedora-Live-Workstation-i686-22-3.iso.txt      |  29 ++++
>>>  .../Fedora-Live-Workstation-x86_64-22-3.iso.txt    |  29 ++++
>>>  .../fedora22/Fedora-Server-DVD-i386-22.iso.txt     |  29 ++++
>>>  .../fedora22/Fedora-Server-DVD-x86_64-22.iso.txt   |  29 ++++
>>>  .../fedora22/Fedora-Server-netinst-i386-22.iso.txt |  29 ++++
>>>  .../Fedora-Server-netinst-x86_64-22.iso.txt        |  29 ++++
>>>  .../Fedora-Workstation-netinst-i386-22.iso.txt     |  29 ++++
>>>  .../Fedora-Workstation-netinst-x86_64-22.iso.txt   |  29 ++++
>>>  9 files changed, 404 insertions(+), 2 deletions(-)
>>>  create mode 100644 test/isodata/fedora/fedora22/Fedora-Live-Workstation-i686-22-3.iso.txt
>>>  create mode 100644 test/isodata/fedora/fedora22/Fedora-Live-Workstation-x86_64-22-3.iso.txt
>>>  create mode 100644 test/isodata/fedora/fedora22/Fedora-Server-DVD-i386-22.iso.txt
>>>  create mode 100644 test/isodata/fedora/fedora22/Fedora-Server-DVD-x86_64-22.iso.txt
>>>  create mode 100644 test/isodata/fedora/fedora22/Fedora-Server-netinst-i386-22.iso.txt
>>>  create mode 100644 test/isodata/fedora/fedora22/Fedora-Server-netinst-x86_64-22.iso.txt
>>>  create mode 100644 test/isodata/fedora/fedora22/Fedora-Workstation-netinst-i386-22.iso.txt
>>>  create mode 100644 test/isodata/fedora/fedora22/Fedora-Workstation-netinst-x86_64-22.iso.txt
>>>
>>> diff --git a/data/oses/fedora.xml.in b/data/oses/fedora.xml.in
>>> index 2e4b898..2b6e994 100644
>>> --- a/data/oses/fedora.xml.in
>>> +++ b/data/oses/fedora.xml.in
>>> @@ -4081,6 +4081,171 @@
>>>      </installer>
>>>    </os>
>>>
>>> +  <os id="http://fedoraproject.org/fedora/22">
>>> +    <short-id>fedora22</short-id>
>>> +    <_name>Fedora 22</_name>
>>> +    <version>22</version>
>>> +    <_vendor>Fedora Project</_vendor>
>>> +    <family>linux</family>
>>> +    <distro>fedora</distro>
>>> +    <upgrades id="http://fedoraproject.org/fedora/21"/>
>>> +    <derives-from id="http://fedoraproject.org/fedora/21"/>
>>> +
>>> +    <release-date>2015-05-26</release-date>
>>> +
>>> +    <variant id="workstation">
>>> +      <_name>Fedora 22 Workstation</_name>
>>> +    </variant>
>>> +    <variant id="workstation-netinst">
>>> +      <_name>Fedora 22</_name>
>>> +    </variant>
>>
>> Why not "Fedora 22 Workstation" here?
>
> Hmm.. Yeah I guess it makes sense now that there are different netinstaller ISO.
>
>>> +    <!-- Network Installer !-->
>>> +    <media arch="i686">
>>> +      <variant id="workstation-netinst"/>
>>
>> Looks like this would be much better expressed as
>> <media arch="i686" installer="true" netinst="true">
>>   <variant id="workstation"/>
>> </media>
>> or something like that.
>
> I think you are correct. Wonder why didn't think of that. probably
> cause I had been thinking in term so associating installer to
> particular variants.
>
> Having said that, I don't think this is so much better than the the
> solution we already merged that it'd be worth delaying releases for
> (which were due day before yesterday). We'd want "netinst" param (or
> "require-internet") param on media anyway though to let apps know this
> media needs internet to work.

Just realized, we won't need to change/modify any API to convert these
entries to what you suggested. We'll just need a bit of new API.

-- 
Regards,

Zeeshan Ali (Khattak)
________________________________________
Befriend GNOME: http://www.gnome.org/friends/