GNU bug report logs - #45531
[PATCH] gnu: hdf5: Update to 1.12.0.

Previous Next

Package: guix-patches;

Reported by: Vincent Legoll <vincent.legoll <at> gmail.com>

Date: Tue, 29 Dec 2020 12:13:02 UTC

Severity: normal

Tags: patch

Done: Vincent Legoll <vincent.legoll <at> gmail.com>

Bug is archived. No further changes may be made.

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 45531 in the body.
You can then email your comments to 45531 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to guix-patches <at> gnu.org:
bug#45531; Package guix-patches. (Tue, 29 Dec 2020 12:13:02 GMT) Full text and rfc822 format available.

Acknowledgement sent to Vincent Legoll <vincent.legoll <at> gmail.com>:
New bug report received and forwarded. Copy sent to guix-patches <at> gnu.org. (Tue, 29 Dec 2020 12:13:02 GMT) Full text and rfc822 format available.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Vincent Legoll <vincent.legoll <at> gmail.com>
To: guix-patches <at> gnu.org
Cc: Vincent Legoll <vincent.legoll <at> gmail.com>
Subject: [PATCH] gnu: hdf5: Update to 1.12.0.
Date: Tue, 29 Dec 2020 13:12:03 +0100
* gnu/packages/maths.scm (hdf5): Update to 1.12.0.
---
 gnu/packages/maths.scm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/maths.scm b/gnu/packages/maths.scm
index b54cd6dff1..09e0837872 100644
--- a/gnu/packages/maths.scm
+++ b/gnu/packages/maths.scm
@@ -1200,7 +1200,7 @@ extremely large and complex data collections.")
 
 (define-public hdf5-1.10
   (package/inherit hdf5-1.8
-    (version "1.10.7")
+    (version "1.12.0")
     (source
      (origin
        (method url-fetch)
@@ -1214,7 +1214,7 @@ extremely large and complex data collections.")
                                         (take (string-split version #\.) 2))
                                  "/src/hdf5-" version ".tar.bz2")))
        (sha256
-        (base32 "0pm5xxry55i0h7wmvc7svzdaa90rnk7h78rrjmnlkz2ygsn8y082"))
+        (base32 "0qazfslkqbmzg495jafpvqp0khws3jkxa0z7rph9qvhacil6544p"))
        (patches (search-patches "hdf5-config-date.patch"))))))
 
 (define-public hdf5
-- 
2.29.2





Information forwarded to guix-patches <at> gnu.org:
bug#45531; Package guix-patches. (Wed, 30 Dec 2020 02:20:01 GMT) Full text and rfc822 format available.

Message #8 received at 45531 <at> debbugs.gnu.org (full text, mbox):

From: Leo Famulari <leo <at> famulari.name>
To: Vincent Legoll <vincent.legoll <at> gmail.com>
Cc: 45531 <at> debbugs.gnu.org
Subject: Re: [bug#45531] [PATCH] gnu: hdf5: Update to 1.12.0.
Date: Tue, 29 Dec 2020 21:19:35 -0500
On Tue, Dec 29, 2020 at 01:12:03PM +0100, Vincent Legoll wrote:
> * gnu/packages/maths.scm (hdf5): Update to 1.12.0.

Thanks!

>  (define-public hdf5-1.10
>    (package/inherit hdf5-1.8
> -    (version "1.10.7")
> +    (version "1.12.0")

The variable name should be updated to reflect the new version, and any
packages using it would also need to be adjusted.

Can you send a revised patch?




Information forwarded to guix-patches <at> gnu.org:
bug#45531; Package guix-patches. (Wed, 30 Dec 2020 16:03:01 GMT) Full text and rfc822 format available.

Message #11 received at 45531 <at> debbugs.gnu.org (full text, mbox):

From: Vincent Legoll <vincent.legoll <at> gmail.com>
To: Leo Famulari <leo <at> famulari.name>
Cc: 45531 <at> debbugs.gnu.org
Subject: Re: [bug#45531] [PATCH] gnu: hdf5: Update to 1.12.0.
Date: Wed, 30 Dec 2020 17:01:58 +0100
Hello,

On Wed, Dec 30, 2020 at 3:19 AM Leo Famulari <leo <at> famulari.name> wrote:
> >  (define-public hdf5-1.10
> >    (package/inherit hdf5-1.8
> > -    (version "1.10.7")
> > +    (version "1.12.0")
>
> The variable name should be updated to reflect the new version, and any
> packages using it would also need to be adjusted.

oops,completely missed that, sorry.

> Can you send a revised patch?

Yep, I'll do.

Thanks for the review

-- 
Vincent Legoll




Information forwarded to guix-patches <at> gnu.org:
bug#45531; Package guix-patches. (Wed, 30 Dec 2020 18:36:02 GMT) Full text and rfc822 format available.

Message #14 received at 45531 <at> debbugs.gnu.org (full text, mbox):

From: zimoun <zimon.toutoune <at> gmail.com>
To: Vincent Legoll <vincent.legoll <at> gmail.com>, Leo Famulari <leo <at> famulari.name>
Cc: 45531 <at> debbugs.gnu.org
Subject: Re: [bug#45531] [PATCH] gnu: hdf5: Update to 1.12.0.
Date: Wed, 30 Dec 2020 19:34:12 +0100
Hi,

On Wed, 30 Dec 2020 at 17:01, Vincent Legoll <vincent.legoll <at> gmail.com> wrote:
> On Wed, Dec 30, 2020 at 3:19 AM Leo Famulari <leo <at> famulari.name> wrote:

>> >  (define-public hdf5-1.10
>> >    (package/inherit hdf5-1.8
>> > -    (version "1.10.7")
>> > +    (version "1.12.0")
>>
>> The variable name should be updated to reflect the new version, and any
>> packages using it would also need to be adjusted.
>
> oops,completely missed that, sorry.

Please check that all the packages depending on hdf5-1.10 correctly
build with 1.12.0, for example: freecad, libmedfile, etc.

And also hdf5-parallel-openmpi which inherits from, so packages
depending on that too.

Therefore, instead it could be simpler to add a new variable hdf5-1.12
and use it as default with the variable hdf5.


All the best,
simon




Information forwarded to guix-patches <at> gnu.org:
bug#45531; Package guix-patches. (Tue, 05 Jan 2021 09:35:02 GMT) Full text and rfc822 format available.

Message #17 received at 45531 <at> debbugs.gnu.org (full text, mbox):

From: Vincent Legoll <vincent.legoll <at> gmail.com>
To: zimoun <zimon.toutoune <at> gmail.com>
Cc: 45531 <at> debbugs.gnu.org, Leo Famulari <leo <at> famulari.name>
Subject: Re: [bug#45531] [PATCH] gnu: hdf5: Update to 1.12.0.
Date: Tue, 5 Jan 2021 10:34:36 +0100
Hello,

On Wed, Dec 30, 2020 at 7:35 PM zimoun <zimon.toutoune <at> gmail.com> wrote:
> Please check that all the packages depending on hdf5-1.10 correctly
> build with 1.12.0, for example: freecad, libmedfile, etc.
>
> And also hdf5-parallel-openmpi which inherits from, so packages
> depending on that too.

Is this specific to that package or should I always rebuild all the
dependencies (& their dependencies) for each package update ?

-- 
Vincent Legoll




Information forwarded to guix-patches <at> gnu.org:
bug#45531; Package guix-patches. (Tue, 05 Jan 2021 12:34:02 GMT) Full text and rfc822 format available.

Message #20 received at 45531 <at> debbugs.gnu.org (full text, mbox):

From: Efraim Flashner <efraim <at> flashner.co.il>
To: Vincent Legoll <vincent.legoll <at> gmail.com>
Cc: 45531 <at> debbugs.gnu.org, zimoun <zimon.toutoune <at> gmail.com>
Subject: Re: [bug#45531] [PATCH] gnu: hdf5: Update to 1.12.0.
Date: Tue, 5 Jan 2021 14:32:59 +0200
[Message part 1 (text/plain, inline)]
On Tue, Jan 05, 2021 at 10:34:36AM +0100, Vincent Legoll wrote:
> Hello,
> 
> On Wed, Dec 30, 2020 at 7:35 PM zimoun <zimon.toutoune <at> gmail.com> wrote:
> > Please check that all the packages depending on hdf5-1.10 correctly
> > build with 1.12.0, for example: freecad, libmedfile, etc.
> >
> > And also hdf5-parallel-openmpi which inherits from, so packages
> > depending on that too.
> 
> Is this specific to that package or should I always rebuild all the
> dependencies (& their dependencies) for each package update ?
> 

The packages which share a source should definitely be tested out. As
far as ones that depend on them test them out to your best ability. It
normally depends on computing power and whether they built previously or
not.

Some package groups have special needs. rust packages only really need
to pass their build phase, and anything that depends on them should just
keep working. I've never gotten a good result from java packages so I'll
build those out to the very end. Due to the way python works it can be
hit-or-miss.

-- 
Efraim Flashner   <efraim <at> flashner.co.il>   אפרים פלשנר
GPG key = A28B F40C 3E55 1372 662D  14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted
[signature.asc (application/pgp-signature, inline)]

Information forwarded to guix-patches <at> gnu.org:
bug#45531; Package guix-patches. (Wed, 06 Jan 2021 18:45:02 GMT) Full text and rfc822 format available.

Message #23 received at 45531 <at> debbugs.gnu.org (full text, mbox):

From: zimoun <zimon.toutoune <at> gmail.com>
To: Vincent Legoll <vincent.legoll <at> gmail.com>
Cc: 45531 <at> debbugs.gnu.org, Leo Famulari <leo <at> famulari.name>
Subject: Re: [bug#45531] [PATCH] gnu: hdf5: Update to 1.12.0.
Date: Wed, 06 Jan 2021 19:38:09 +0100
Hi,

On Tue, 05 Jan 2021 at 10:34, Vincent Legoll <vincent.legoll <at> gmail.com> wrote:
> On Wed, Dec 30, 2020 at 7:35 PM zimoun <zimon.toutoune <at> gmail.com> wrote:
>> Please check that all the packages depending on hdf5-1.10 correctly
>> build with 1.12.0, for example: freecad, libmedfile, etc.
>>
>> And also hdf5-parallel-openmpi which inherits from, so packages
>> depending on that too.
>
> Is this specific to that package or should I always rebuild all the
> dependencies (& their dependencies) for each package update ?

I do not know if there is a rule.  But hdf5 is used by many scientific
packages and so changing the default version needs some care, IMHO.
Upgrading the version could lead to annoying breakages and it is better
to fix them at this upgrading time than to discover them later.

Well, I am not sure about your initial intention with your patch,

 a) add the new variable hdf5-1.12
 b) change the variable hdf5-1.10 by hdf5-1.12 which means replace all
 the occurrences of hdf5-1.10 and check everything builds; note the
 default hdf5 is changed too so it is a lot of rebuilds

Some packages specifically depends on hdf5-1.8 so maybe some packages
also specifically require hdf5-1.10; I have not checked.

These 9 packages specifically use hdf5-1.10:

        hdf5-parallel-openmpi, freecad, libmedfile, hdf5-blosc,
        python-h5py, python-tables, sbcl-hdf5-cffi, r-rhdf5lib,
        sra-tools

so they need special care, at least. :-)

My point is “guix refresh -l hdf5” says the upgrade could have an impact
on a couple of packages and I just wanted be sure that you were aware of
that. :-)


All the best,
simon





Information forwarded to guix-patches <at> gnu.org:
bug#45531; Package guix-patches. (Thu, 07 Jan 2021 08:41:02 GMT) Full text and rfc822 format available.

Message #26 received at 45531 <at> debbugs.gnu.org (full text, mbox):

From: Vincent Legoll <vincent.legoll <at> gmail.com>
To: zimoun <zimon.toutoune <at> gmail.com>
Cc: 45531 <at> debbugs.gnu.org, Leo Famulari <leo <at> famulari.name>
Subject: Re: [bug#45531] [PATCH] gnu: hdf5: Update to 1.12.0.
Date: Thu, 7 Jan 2021 09:40:22 +0100
Hello,

Thanks for the explanation, I'll see if I can build
those & report here.

-- 
Vincent Legoll




Information forwarded to guix-patches <at> gnu.org:
bug#45531; Package guix-patches. (Mon, 11 Jan 2021 17:58:02 GMT) Full text and rfc822 format available.

Message #29 received at 45531 <at> debbugs.gnu.org (full text, mbox):

From: Vincent Legoll <vincent.legoll <at> gmail.com>
To: zimoun <zimon.toutoune <at> gmail.com>
Cc: 45531 <at> debbugs.gnu.org, Leo Famulari <leo <at> famulari.name>
Subject: Re: [bug#45531] [PATCH] gnu: hdf5: Update to 1.12.0.
Date: Mon, 11 Jan 2021 18:57:16 +0100
Hello,

I tried to make it the default, but at least ecl-hdf5-cffi
did not rebuild properly, so I think the best way forward
would be to add 1.12.0 as a separate non-default version,
and then let the users of the dependent packages update
as they want / have tested. I'm not user of any of those...

Or completely drop my (random-)update.

-- 
Vincent Legoll




Information forwarded to guix-patches <at> gnu.org:
bug#45531; Package guix-patches. (Thu, 14 Jan 2021 12:39:01 GMT) Full text and rfc822 format available.

Message #32 received at 45531 <at> debbugs.gnu.org (full text, mbox):

From: zimoun <zimon.toutoune <at> gmail.com>
To: Vincent Legoll <vincent.legoll <at> gmail.com>
Cc: 45531 <at> debbugs.gnu.org, Leo Famulari <leo <at> famulari.name>
Subject: Re: [bug#45531] [PATCH] gnu: hdf5: Update to 1.12.0.
Date: Thu, 14 Jan 2021 13:38:20 +0100
Hi Vincent,

Thank you for testing.

On Mon, 11 Jan 2021 at 18:57, Vincent Legoll <vincent.legoll <at> gmail.com> wrote:

> I tried to make it the default, but at least ecl-hdf5-cffi
> did not rebuild properly, so I think the best way forward
> would be to add 1.12.0 as a separate non-default version,
> and then let the users of the dependent packages update
> as they want / have tested. I'm not user of any of those...

This was my initial suggestion. :-)
On the other hand, if only one package is failing with 1.12.0, maybe
upgrade the default and keep the previous for this specific package
only.

All the best,
simon




Reply sent to Vincent Legoll <vincent.legoll <at> gmail.com>:
You have taken responsibility. (Sun, 21 Feb 2021 21:31:02 GMT) Full text and rfc822 format available.

Notification sent to Vincent Legoll <vincent.legoll <at> gmail.com>:
bug acknowledged by developer. (Sun, 21 Feb 2021 21:31:02 GMT) Full text and rfc822 format available.

Message #37 received at 45531-done <at> debbugs.gnu.org (full text, mbox):

From: Vincent Legoll <vincent.legoll <at> gmail.com>
To: 45531-DONE <at> debbugs.gnu.org
Subject: Closing
Date: Sun, 21 Feb 2021 22:29:41 +0100
Closing as duplicate of :
f601cdc6981888f55882c942dab413cb783caecc
-- 
Vincent Legoll




bug archived. Request was from Debbugs Internal Request <help-debbugs <at> gnu.org> to internal_control <at> debbugs.gnu.org. (Mon, 22 Mar 2021 11:24:04 GMT) Full text and rfc822 format available.

This bug report was last modified 3 years and 8 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.