unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* [bug#28217] [PATCH] gnu: hdf4: Avoid unneeded store references to compilers.
@ 2017-08-24 13:18 Thomas Danckaert
  2017-08-31 13:00 ` Ludovic Courtès
  0 siblings, 1 reply; 3+ messages in thread
From: Thomas Danckaert @ 2017-08-24 13:18 UTC (permalink / raw)
  To: 28217

[-- Attachment #1: Type: Text/Plain, Size: 110 bytes --]

Hi,

this patch removes unnecessary references to the C and Fortran 
compilers from the HDF4 package.

Thomas

[-- Attachment #2: 0001-gnu-hdf4-Avoid-unneeded-store-references-to-compiler.patch --]
[-- Type: Text/X-Patch, Size: 1556 bytes --]

From 6c6c0df09c48c6f2f9b8e77f5e0f88d61f812f3f Mon Sep 17 00:00:00 2001
From: Thomas Danckaert <post@thomasdanckaert.be>
Date: Thu, 24 Aug 2017 14:27:29 +0200
Subject: [PATCH] gnu: hdf4: Avoid unneeded store references to compilers.

* gnu/packages/maths.scm (hdf4) [arguments]: Add phase 'patch-settings to
  truncate hashes of compiler store items.
---
 gnu/packages/maths.scm | 13 ++++++++++++-
 1 file changed, 12 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/maths.scm b/gnu/packages/maths.scm
index 07784defb..16c61dda9 100644
--- a/gnu/packages/maths.scm
+++ b/gnu/packages/maths.scm
@@ -605,7 +605,18 @@ computations.")
                (("@HDF_BUILD_SHARED_TRUE@AM_LDFLAGS = \
 -R\\$\\(abs_top_builddir\\)/mfhdf/libsrc/\\.libs \
 -R\\$\\(abs_top_builddir\\)/hdf/src/\\.libs \\$\\(XDR_ADD\\)") ""))
-             #t)))))
+             #t))
+         (add-after 'configure 'patch-settings
+           (lambda _
+             ;; libhdf4.settings contains the full path of the
+             ;; compilers used, and its contents are included in
+             ;; .so-files.  We truncate the hashes to avoid
+             ;; unnecessary store references to those compilers:
+             (substitute* "libhdf4.settings"
+               (("(/gnu/store/)([a-Z0-9]*)" all prefix hash)
+                (string-append prefix (string-take hash 10) "...")))
+             #t))
+         )))
     (home-page "https://www.hdfgroup.org/products/hdf4/")
     (synopsis
      "Library and multi-object file format for storing and managing data")
-- 
2.14.1


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [bug#28217] [PATCH] gnu: hdf4: Avoid unneeded store references to compilers.
  2017-08-24 13:18 [bug#28217] [PATCH] gnu: hdf4: Avoid unneeded store references to compilers Thomas Danckaert
@ 2017-08-31 13:00 ` Ludovic Courtès
  2017-09-01  7:32   ` bug#28217: " Thomas Danckaert
  0 siblings, 1 reply; 3+ messages in thread
From: Ludovic Courtès @ 2017-08-31 13:00 UTC (permalink / raw)
  To: Thomas Danckaert; +Cc: 28217

Thomas Danckaert <post@thomasdanckaert.be> skribis:

> From 6c6c0df09c48c6f2f9b8e77f5e0f88d61f812f3f Mon Sep 17 00:00:00 2001
> From: Thomas Danckaert <post@thomasdanckaert.be>
> Date: Thu, 24 Aug 2017 14:27:29 +0200
> Subject: [PATCH] gnu: hdf4: Avoid unneeded store references to compilers.
>
> * gnu/packages/maths.scm (hdf4) [arguments]: Add phase 'patch-settings to
>   truncate hashes of compiler store items.

LGTM, it’s a good idea.

I assume hdf4 doesn’t rely on these file names at all for its
functionality, right?

Thanks!

Ludo’.

^ permalink raw reply	[flat|nested] 3+ messages in thread

* bug#28217: [PATCH] gnu: hdf4: Avoid unneeded store references to compilers.
  2017-08-31 13:00 ` Ludovic Courtès
@ 2017-09-01  7:32   ` Thomas Danckaert
  0 siblings, 0 replies; 3+ messages in thread
From: Thomas Danckaert @ 2017-09-01  7:32 UTC (permalink / raw)
  To: ludo; +Cc: 28214-done, 28217-done

From: ludo@gnu.org (Ludovic Courtès)
Subject: Re: [bug#28217] [PATCH] gnu: hdf4: Avoid unneeded store 
references to compilers.
Date: Thu, 31 Aug 2017 15:00:34 +0200

> I assume hdf4 doesn’t rely on these file names at all for its
> functionality, right?

No, it's just some extra information.

I pushed this, as well as the hdf5 patches.  Thanks for having a look!

Thomas

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2017-09-01  7:33 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-08-24 13:18 [bug#28217] [PATCH] gnu: hdf4: Avoid unneeded store references to compilers Thomas Danckaert
2017-08-31 13:00 ` Ludovic Courtès
2017-09-01  7:32   ` bug#28217: " Thomas Danckaert

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).