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

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

 



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 :)

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