Re: [PATCH osinfo-db] dragonflybsd : Add DragonFlyBSD data

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

 



Le 20/07/2018 à 12:43, Fabiano Fidêncio a écrit :
> On Thu, Jul 19, 2018 at 5:00 PM, Pierre-Alain TORET
> <pierre-alain.toret@xxxxxxxxxxxxxx> wrote:
>> On 19/07/2018 16:54, Fabiano Fidêncio wrote:
>>> On Thu, Jul 19, 2018 at 3:13 PM, Fabiano Fidêncio <fabiano@xxxxxxxxxxxx> wrote:
>>>> On Thu, Jul 19, 2018 at 3:08 PM, Pierre-Alain TORET
>>>> <pierre-alain.toret@xxxxxxxxxxxxxx> wrote:
>>>>> On 19/07/2018 14:41, Fabiano Fidêncio wrote:
>>>>>> On Thu, Jul 19, 2018 at 2:14 PM, Pierre-Alain TORET
>>>>>> <pierre-alain.toret@xxxxxxxxxxxxxx> wrote:
>>>>>>> On 19/07/2018 13:30, Fabiano Fidêncio wrote:
>>>>>>>> On Thu, Jul 19, 2018 at 10:40 AM, Pierre-Alain TORET
>>>>>>>> <pierre-alain.toret@xxxxxxxxxxxxxx> wrote:
>>>>>>>>> On 19/07/2018 10:02, Fabiano Fidêncio wrote:
>>>>>>>>>> On Thu, Jul 19, 2018 at 9:47 AM, Pierre-Alain TORET
>>>>>>>>>> <pierre-alain.toret@xxxxxxxxxxxxxx> wrote:
>>>>>>>>>>> Hello Fabiano,
>>>>>>>>>>>
>>>>>>>>>>> On 19/07/2018 07:59, Fabiano Fidêncio wrote:
>>>>>>>>>>>> Pierre-Alain,
>>>>>>>>>>>>
>>>>>>>>>>>> On Wed, Jun 6, 2018 at 10:54 PM, Pierre-Alain TORET
>>>>>>>>>>>> <pierre-alain.toret@xxxxxxxxxxxxxx> wrote:
>>>>>>>>>>>>> Hello,
>>>>>>>>>>>>>
>>>>>>>>>>>>> [...]
>>>>>>>>>>>>>
>>>>>>>>>>>>> Signed-off-by: Pierre-Alain TORET <pierre-alain.toret@xxxxxxxxxxxxxx>
>>>>>>>>>>>>
>>>>>>>>>>>> For some reason I'm not able to apply your patches atop of git master.
>>>>>>>>>>>> It may be some issue on my mail client breaking the patches lines in
>>>>>>>>>>>> an weird way.
>>>>>>>>>>>>
>>>>>>>>>>>> Would you have those patches on a git repo that you could point me to?
>>>>>>>>>>>> I'd like to review and have them merged sooner than later.
>>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>> Actually I cleaned my repos just 2 days ago, this is really bad luck.
>>>>>>>>>>> I tried and can't apply the patch myself neither so it seems there's an
>>>>>>>>>>> issue with what I sent.
>>>>>>>>>>> I guess I will have to make a new patch by hand, but this will take a
>>>>>>>>>>> little time, except if someone has a better solution.
>>>>>>>>>>
>>>>>>>>>> Okay. :-/
>>>>>>>>>
>>>>>>>>> Ah actually I found the .patch files generated at that time, so I
>>>>>>>>> re-used them and pushed on my gitlab fork of the projects :
>>>>>>>>> https://gitlab.com/daftaupe/libosinfo/tree/dfly
>>>>>>>>
>>>>>>>> This repo seems to be closed and I can't access it.
>>>>>>>>
>>>>>>>
>>>>>>> Sorry this repo was in private mode, and the other in public mode, I
>>>>>>> don't why as I just forked both of them.
>>>>>>> Anyway you should have access now.
>>>>>>>
>>>>>>>>> https://gitlab.com/daftaupe/osinfo-db/tree/dfly/
>>>>>>>>
>>>>>>>> This one is fine!
>>>>>>>>
>>>>>>>>>
>>>>>>>>> I added release 5.2.2 while at it :)
>>>>>>>>>
>>>>>>>>> Sorry for the noise.
>>>>>>>>>>
>>>>>>>>>> I'll go and prepare a release later Today with what we have merged and
>>>>>>>>>> this series will be in the next one.
>>>>>>>>>> Hopefully it's not a big problem for you.
>>>>>>>>>>
>>>>>>>
>>>>>>> Sorry, I didn't answer to that. It's totally fine.
>>>>>>
>>>>>> Okay, I've checked your patches.
>>>>>>
>>>>>> There are 2 changes that *have* *to* be merged into your patches:
>>>>>>
>>>>>> [fidencio@machado osinfo-db]$ git diff
>>>>>> diff --git a/data/os/dragonflybsd.org/dragonflybsd-4.6.1.xml.in
>>>>>> b/data/os/dragonflybsd.org/dragonflybsd-4.6.1.xml.in
>>>>>> index fdf0972..dac23f3 100644
>>>>>> --- a/data/os/dragonflybsd.org/dragonflybsd-4.6.1.xml.in
>>>>>> +++ b/data/os/dragonflybsd.org/dragonflybsd-4.6.1.xml.in
>>>>>> @@ -1,14 +1,14 @@
>>>>>>     <libosinfo version="0.0.1">
>>>>>>     <!-- Licensed under the GNU General Public License version 2 or later.
>>>>>>          See http://www.gnu.org/licenses/ for a copy of the license text -->
>>>>>> -  <os id="http://dragonflybsd.org/dragonflybsd/4.8.0";>
>>>>>> -    <short-id>dragonflybsd4.8.0</short-id>
>>>>>> -    <_name>DragonFlyBSD 4.8.0</_name>
>>>>>> -    <version>4.8.0</version>
>>>>>> +  <os id="http://dragonflybsd.org/dragonflybsd/4.6.1";>
>>>>>> +    <short-id>dragonflybsd4.6.1</short-id>
>>>>>> +    <_name>DragonFlyBSD 4.6.1</_name>
>>>>>> +    <version>4.6.1</version>
>>>>>>         <family>dragonflybsd</family>
>>>>>>         <distro>dragonflybsd</distro>
>>>>>>         <_vendor>DragonFlyBSD Project</_vendor>
>>>>>> -    <upgrades id="http://dragonflybsd.org/dragonflybsd/4.6.2"/>
>>>>>> +    <upgrades id="http://dragonflybsd.org/dragonflybsd/4.6.0"/>
>>>>>>
>>>>>>         <release-date>2017-03-27</release-date>
>>>>>>       </os>
>>>>>> diff --git a/data/os/dragonflybsd.org/dragonflybsd-5.0.0.xml.in
>>>>>> b/data/os/dragonflybsd.org/dragonflybsd-5.0.0.xml.in
>>>>>> index 07e587f..4a675dc 100644
>>>>>> --- a/data/os/dragonflybsd.org/dragonflybsd-5.0.0.xml.in
>>>>>> +++ b/data/os/dragonflybsd.org/dragonflybsd-5.0.0.xml.in
>>>>>> @@ -17,7 +17,7 @@
>>>>>>           <iso>
>>>>>>                <system-id>DragonFly</system-id>
>>>>>>                <volume-id>DragonFly v5.0.0</volume-id>
>>>>>> -           <volume-size>346052608</volume-size>
>>>>>> +           <volume-size>789700608</volume-size>
>>>>>>           </iso>
>>>>>>         </media>
>>>>>>       </os>
>>>>>>
>>>>>>
>>>>>> There's also another change that I'd like to do before merging those,
>>>>>> if you agree and its related to
>>>>>> https://www.redhat.com/archives/libosinfo/2018-July/msg00032.html
>>>>>>
>>>>>> Basically, all other distros but openbsd are using single quotes for
>>>>>> the arch entry.
>>>>>>
>>>>>> If you agree, I'll merge those changes into your patches before
>>>>>> pushing them upstream.
>>>>>>
>>>>>
>>>>> Agreed with all, Fabiano, I've pushed (force so it stays as one commit)
>>>>> on the repo accordingly (I've fixed the release date for 4.6.1 as well
>>>>> and a few tabs instead of spaces).
>>>>
>>>> Right. I'll do a second round of review.
>>>> Thanks!
>>>
>>> Pierre-Alain,
>>>
>>> You don't have to resubmit the patches neither update the branch, but
>>> I need your ACK on merging this patch into yours:
>>>
>>> diff --git a/data/os/dragonflybsd.org/dragonflybsd-4.6.0.xml.in
>>> b/data/os/dragonflybsd.org/dragonflybsd-4.6.0.xml.in
>>> index 77b9cd1..c7a7d79 100644
>>> --- a/data/os/dragonflybsd.org/dragonflybsd-4.6.0.xml.in
>>> +++ b/data/os/dragonflybsd.org/dragonflybsd-4.6.0.xml.in
>>> @@ -8,7 +8,7 @@
>>>        <family>dragonflybsd</family>
>>>        <distro>dragonflybsd</distro>
>>>        <_vendor>DragonFlyBSD Project</_vendor>
>>> -    <upgrades id="http://dragonflybsd.org/dragonflybsd/4.2.4"/>
>>> +    <upgrades id="http://dragonflybsd.org/dragonflybsd/4.4.3"/>
>>>
>>>        <release-date>2017-08-02</release-date>
>>>      </os>
>>> diff --git a/data/os/dragonflybsd.org/dragonflybsd-4.8.0.xml.in
>>> b/data/os/dragonflybsd.org/dragonflybsd-4.8.0.xml.in
>>> index ad03481..27aad07 100644
>>> --- a/data/os/dragonflybsd.org/dragonflybsd-4.8.0.xml.in
>>> +++ b/data/os/dragonflybsd.org/dragonflybsd-4.8.0.xml.in
>>> @@ -13,7 +13,7 @@
>>>        <release-date>2017-03-27</release-date>
>>>
>>>        <media arch="x86_64">
>>> -    <url>http://mirror-master.dragonflybsd.org/iso-images/dfly-x86_64-4.8.0_REL.iso</url>
>>> +    <url>http://mirror-master.dragonflybsd.org/iso-images/dfly-x86_64-4.8.0_REL.iso.bz2</url>
>>>          <iso>
>>>            <system-id>DragonFly</system-id>
>>>            <volume-id>DragonFly v4.8.0</volume-id>
>>> diff --git a/data/os/dragonflybsd.org/dragonflybsd-4.8.1.xml.in
>>> b/data/os/dragonflybsd.org/dragonflybsd-4.8.1.xml.in
>>> index ea4809e..5e1cadc 100644
>>> --- a/data/os/dragonflybsd.org/dragonflybsd-4.8.1.xml.in
>>> +++ b/data/os/dragonflybsd.org/dragonflybsd-4.8.1.xml.in
>>> @@ -13,7 +13,7 @@
>>>        <release-date>2017-08-02</release-date>
>>>
>>>        <media arch="x86_64">
>>> -      <url>http://mirror-master.dragonflybsd.org/iso-images/dfly-x86_64-4.8.1_REL.iso</url>
>>> +      <url>http://mirror-master.dragonflybsd.org/iso-images/dfly-x86_64-4.8.1_REL.iso.bz2</url>
>>>          <iso>
>>>            <system-id>DragonFly</system-id>
>>>            <volume-id>DragonFly v4.8.1</volume-id>
>>> diff --git a/data/os/dragonflybsd.org/dragonflybsd-5.0.0.xml.in
>>> b/data/os/dragonflybsd.org/dragonflybsd-5.0.0.xml.in
>>> index 338f4aa..9630f1c 100644
>>> --- a/data/os/dragonflybsd.org/dragonflybsd-5.0.0.xml.in
>>> +++ b/data/os/dragonflybsd.org/dragonflybsd-5.0.0.xml.in
>>> @@ -13,7 +13,7 @@
>>>        <release-date>2017-10-16</release-date>
>>>
>>>        <media arch="x86_64">
>>> -      <url>http://mirror-master.dragonflybsd.org/iso-images/dfly-x86_64-5.0.0_REL.iso</url>
>>> +      <url>http://mirror-master.dragonflybsd.org/iso-images/dfly-x86_64-5.0.0_REL.iso.bz2</url>
>>>          <iso>
>>>            <system-id>DragonFly</system-id>
>>>            <volume-id>DragonFly v5.0.0</volume-id>
>>> diff --git a/data/os/dragonflybsd.org/dragonflybsd-5.0.1.xml.in
>>> b/data/os/dragonflybsd.org/dragonflybsd-5.0.1.xml.in
>>> index 508308a..c90d22f 100644
>>> --- a/data/os/dragonflybsd.org/dragonflybsd-5.0.1.xml.in
>>> +++ b/data/os/dragonflybsd.org/dragonflybsd-5.0.1.xml.in
>>> @@ -13,7 +13,7 @@
>>>        <release-date>2017-11-06</release-date>
>>>
>>>        <media arch="x86_64">
>>> -      <url>http://mirror-master.dragonflybsd.org/iso-images/dfly-x86_64-5.0.1_REL.iso</url>
>>> +      <url>http://mirror-master.dragonflybsd.org/iso-images/dfly-x86_64-5.0.1_REL.iso.bz2</url>
>>>          <iso>
>>>            <system-id>DragonFly</system-id>
>>>            <volume-id>DragonFly v5.0.1</volume-id>
>>> diff --git a/data/os/dragonflybsd.org/dragonflybsd-5.0.2.xml.in
>>> b/data/os/dragonflybsd.org/dragonflybsd-5.0.2.xml.in
>>> index abc7ed9..23aceaf 100644
>>> --- a/data/os/dragonflybsd.org/dragonflybsd-5.0.2.xml.in
>>> +++ b/data/os/dragonflybsd.org/dragonflybsd-5.0.2.xml.in
>>> @@ -13,7 +13,7 @@
>>>        <release-date>2017-12-04</release-date>
>>>
>>>        <media arch="x86_64">
>>> -      <url>http://mirror-master.dragonflybsd.org/iso-images/dfly-x86_64-5.0.2_REL.iso</url>
>>> +      <url>http://mirror-master.dragonflybsd.org/iso-images/dfly-x86_64-5.0.2_REL.iso.bz2</url>
>>>          <iso>
>>>            <system-id>DragonFly</system-id>
>>>            <volume-id>DragonFly v5.0.2</volume-id>
>>> diff --git a/data/os/dragonflybsd.org/dragonflybsd-5.2.0.xml.in
>>> b/data/os/dragonflybsd.org/dragonflybsd-5.2.0.xml.in
>>> index cf14883..bd759e2 100644
>>> --- a/data/os/dragonflybsd.org/dragonflybsd-5.2.0.xml.in
>>> +++ b/data/os/dragonflybsd.org/dragonflybsd-5.2.0.xml.in
>>> @@ -13,7 +13,7 @@
>>>        <release-date>2018-04-10</release-date>
>>>
>>>        <media arch="x86_64">
>>> -      <url>http://mirror-master.dragonflybsd.org/iso-images/dfly-x86_64-5.2.0_REL.iso</url>
>>> +      <url>http://mirror-master.dragonflybsd.org/iso-images/dfly-x86_64-5.2.0_REL.iso.bz2</url>
>>>          <iso>
>>>            <system-id>DragonFly</system-id>
>>>            <volume-id>DragonFly v5.2.0</volume-id>
>>> diff --git a/data/os/dragonflybsd.org/dragonflybsd-5.2.1.xml.in
>>> b/data/os/dragonflybsd.org/dragonflybsd-5.2.1.xml.in
>>> index 89dd5ef..cbfc4f4 100644
>>> --- a/data/os/dragonflybsd.org/dragonflybsd-5.2.1.xml.in
>>> +++ b/data/os/dragonflybsd.org/dragonflybsd-5.2.1.xml.in
>>> @@ -13,7 +13,7 @@
>>>        <release-date>2018-05-21</release-date>
>>>
>>>        <media arch="x86_64">
>>> -      <url>http://mirror-master.dragonflybsd.org/iso-images/dfly-x86_64-5.2.1_REL.iso</url>
>>> +      <url>http://mirror-master.dragonflybsd.org/iso-images/dfly-x86_64-5.2.1_REL.iso.bz2</url>
>>>          <iso>
>>>            <system-id>DragonFly</system-id>
>>>            <volume-id>DragonFly v5.2.1</volume-id>
>>>
>>>
>>> The reasons for this patch are:
>>> - A copy and paste error in the upgrades of one version;
>>> - Old releases doesn't seem to be available as a pure ISO. :-/
>>>
>>
>> That looks good to me Fabiano ! Thanks for the review :)
> 
> Patches have been pushed Today and are part of v20180720 release.
> I've commented out the .iso.bz2 images after a brief discussion with
> danpb on #virt, as we (at least) imply that the links are for actual
> ISOs.
> We may change it as soon as we add some metadata to record the
> compression method (one more thing to add to my TODO list).
> 
> Thanks for the contribution!
> 

Thanks for the details, sounds like a good solution.

Regards,

>>
>>>>
>>>>>
>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>>> Sorry for the long time taken for reviewing those.
>>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>> No problem.
>>>>>>>>>>>
>>>>>>>>>>> I'll seend a new one when it's ready.
>>>>>>>>>>
>>>>>>>>>> Thanks a lot!
>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>>>> [...]
>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>
>>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>
>>>>>>
>>>>>>
>>>>>
>>>>>
>>>>
>>>>
>>>>
>>>> --
>>>> Fabiano Fidêncio
>>>
>>>
>>>
>>
>>
> 
> 
> 



_______________________________________________
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