GNU bug report logs -
#77991
[PATCH 0/4] Upgrade Scotch to 7.0.7
Previous Next
To reply to this bug, email your comments to 77991 AT debbugs.gnu.org.
Toggle the display of automated, internal messages from the tracker.
Report forwarded
to
andreas <at> enge.fr, bavier <at> posteo.net, sharlatanus <at> gmail.com, guix-patches <at> gnu.org
:
bug#77991
; Package
guix-patches
.
(Tue, 22 Apr 2025 15:31:01 GMT)
Full text and
rfc822 format available.
Acknowledgement sent
to
Ludovic Courtès <ludo <at> gnu.org>
:
New bug report received and forwarded. Copy sent to
andreas <at> enge.fr, bavier <at> posteo.net, sharlatanus <at> gmail.com, guix-patches <at> gnu.org
.
(Tue, 22 Apr 2025 15:31:01 GMT)
Full text and
rfc822 format available.
Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):
Hello,
This fixes a bug with Scotch’s parmetis compatibility layer
CMake files, upgrades it, and modernizes the definitions.
Tested with:
guix build -P1 scotch -P1 pt-scotch -P1 scotch32 -P1 pt-scotch32
… ignoring fenics* and superlu-dist, which are already broken
on ‘master’.
Thanks,
Ludo’.
Ludovic Courtès (4):
gnu: scotch: Remove “-shared” deprecated aliases.
gnu: scotch: Keep *.cmake in the main output.
gnu: scotch, scotch32, pt-scotch, pt-scotch32: Use gexps.
gnu: scotch: Update to 7.0.7.
gnu/packages/maths.scm | 78 +++++++++++++++++++-----------------------
1 file changed, 36 insertions(+), 42 deletions(-)
base-commit: 7a4193ec4a09fecc68dc726c1e0bbb5ad03d404a
--
2.49.0
Information forwarded
to
andreas <at> enge.fr, bavier <at> posteo.net, sharlatanus <at> gmail.com, guix-patches <at> gnu.org
:
bug#77991
; Package
guix-patches
.
(Tue, 22 Apr 2025 15:33:02 GMT)
Full text and
rfc822 format available.
Message #8 received at 77991 <at> debbugs.gnu.org (full text, mbox):
From: Ludovic Courtès <ludovic.courtes <at> inria.fr>
These aliases were added in commit
d4709d78c3b460d7be02f15f6cc6c85f398b1a46, Feb. 2022.
* gnu/packages/maths.scm (scotch-shared, pt-scotch-shared): Remove.
Change-Id: I971bbf999a9fce5dae05c865316defe55eaeb212
---
gnu/packages/maths.scm | 11 -----------
1 file changed, 11 deletions(-)
diff --git a/gnu/packages/maths.scm b/gnu/packages/maths.scm
index b0c5b8685c..2c53f6d8af 100644
--- a/gnu/packages/maths.scm
+++ b/gnu/packages/maths.scm
@@ -5290,17 +5290,6 @@ (define-public pt-scotch32
(synopsis
"Programs and libraries for graph algorithms (with MPI and 32-bit integers)")))
-(define-public scotch-shared
- ;; There used to be separate shared library variants while the default would
- ;; provide .a files including PIC objects. With the switch to CMake, .a
- ;; files contain non-PIC objects, which breaks some users, and switching to
- ;; shared libraries by default seems to make more sense, as discussed here:
- ;; <https://issues.guix.gnu.org/47619#2>.
- (deprecated-package "scotch-shared" scotch))
-
-(define-public pt-scotch-shared
- (deprecated-package "pt-scotch-shared" pt-scotch))
-
(define-public gklib
(let ((commit "8bd6bad750b2b0d90800c632cf18e8ee93ad72d7")
(revision "1"))
--
2.49.0
Information forwarded
to
andreas <at> enge.fr, bavier <at> posteo.net, sharlatanus <at> gmail.com, guix-patches <at> gnu.org
:
bug#77991
; Package
guix-patches
.
(Tue, 22 Apr 2025 15:33:03 GMT)
Full text and
rfc822 format available.
Message #11 received at 77991 <at> debbugs.gnu.org (full text, mbox):
From: Ludovic Courtès <ludovic.courtes <at> inria.fr>
This fixes a bug whereby ‘SCOTCHConfig.cmake’ would fail to find
‘scotchmetisTargets.cmake’ because it expects it to be in the same
directory as itself, ${CMAKE_CURRENT_LIST_DIR}.
* gnu/packages/maths.scm (scotch)[arguments]: Adjust ‘install-metis’
phase to keep *.cmake in “out”.
Change-Id: If8d8f890852c35cede6e81c33d18b9101988a635
---
gnu/packages/maths.scm | 17 ++++++++++++-----
1 file changed, 12 insertions(+), 5 deletions(-)
diff --git a/gnu/packages/maths.scm b/gnu/packages/maths.scm
index 2c53f6d8af..6a11103e3b 100644
--- a/gnu/packages/maths.scm
+++ b/gnu/packages/maths.scm
@@ -5220,15 +5220,22 @@ (define-public scotch
(modify-phases %standard-phases
(add-after 'install 'install-metis
(lambda* (#:key outputs #:allow-other-keys)
+ ;; Move the METIS compatibility library to a separate output to
+ ;; avoid a name clash on <metis.h>.
(let* ((out (assoc-ref outputs "out"))
(metis (assoc-ref outputs "metis"))
(prefix (string-length out)))
(for-each (lambda (file)
- (let ((target (string-append
- metis
- (string-drop file prefix))))
- (mkdir-p (dirname target))
- (rename-file file target)))
+ ;; 'SCOTCHConfig.cmake' includes
+ ;; *metisTargets.cmake from
+ ;; ${CMAKE_CURRENT_LIST_DIR} so keep all *.cmake
+ ;; files in "out".
+ (unless (string-suffix? ".cmake" file)
+ (let ((target (string-append
+ metis
+ (string-drop file prefix))))
+ (mkdir-p (dirname target))
+ (rename-file file target))))
(find-files out "metis"))))))))
(home-page "https://www.labri.fr/perso/pelegrin/scotch/")
(properties
--
2.49.0
Information forwarded
to
andreas <at> enge.fr, bavier <at> posteo.net, sharlatanus <at> gmail.com, guix-patches <at> gnu.org
:
bug#77991
; Package
guix-patches
.
(Tue, 22 Apr 2025 15:33:03 GMT)
Full text and
rfc822 format available.
Message #14 received at 77991 <at> debbugs.gnu.org (full text, mbox):
* gnu/packages/maths.scm (scotch, scotch32)
(pt-scotch, pt-scotch32): Use gexps.
Change-Id: I93febc994c5345e133eddbd016b1974d08df5bdf
---
gnu/packages/maths.scm | 70 ++++++++++++++++++++----------------------
1 file changed, 34 insertions(+), 36 deletions(-)
diff --git a/gnu/packages/maths.scm b/gnu/packages/maths.scm
index 6a11103e3b..abd05b9d63 100644
--- a/gnu/packages/maths.scm
+++ b/gnu/packages/maths.scm
@@ -5214,29 +5214,30 @@ (define-public scotch
(list flex bison gfortran))
(outputs '("out" "metis"))
(arguments
- `(#:configure-flags '("-DBUILD_SHARED_LIBS=YES" "-DINTSIZE=64"
- "-DBUILD_PTSCOTCH=OFF")
- #:phases
- (modify-phases %standard-phases
- (add-after 'install 'install-metis
- (lambda* (#:key outputs #:allow-other-keys)
- ;; Move the METIS compatibility library to a separate output to
- ;; avoid a name clash on <metis.h>.
- (let* ((out (assoc-ref outputs "out"))
- (metis (assoc-ref outputs "metis"))
- (prefix (string-length out)))
- (for-each (lambda (file)
- ;; 'SCOTCHConfig.cmake' includes
- ;; *metisTargets.cmake from
- ;; ${CMAKE_CURRENT_LIST_DIR} so keep all *.cmake
- ;; files in "out".
- (unless (string-suffix? ".cmake" file)
- (let ((target (string-append
- metis
- (string-drop file prefix))))
- (mkdir-p (dirname target))
- (rename-file file target))))
- (find-files out "metis"))))))))
+ (list #:configure-flags #~'("-DBUILD_SHARED_LIBS=YES" "-DINTSIZE=64"
+ "-DBUILD_PTSCOTCH=OFF")
+ #:phases
+ #~(modify-phases %standard-phases
+ (add-after 'install 'install-metis
+ (lambda* (#:key outputs #:allow-other-keys)
+ ;; Move the METIS compatibility library to a separate output to
+ ;; avoid a name clash on <metis.h>.
+ (let* ((out (assoc-ref outputs "out"))
+ (metis (assoc-ref outputs "metis"))
+ (prefix (string-length out)))
+ (for-each (lambda (file)
+ ;; 'SCOTCHConfig.cmake' includes
+ ;; *metisTargets.cmake from
+ ;; ${CMAKE_CURRENT_LIST_DIR} so keep all
+ ;; *.cmake files in "out".
+ (unless (string-suffix? ".cmake" file)
+ (let ((target
+ (string-append metis
+ (string-drop file
+ prefix))))
+ (mkdir-p (dirname target))
+ (rename-file file target))))
+ (find-files out "metis"))))))))
(home-page "https://www.labri.fr/perso/pelegrin/scotch/")
(properties
`((release-monitoring-url
@@ -5260,9 +5261,8 @@ (define-public scotch32
(name "scotch32")
(arguments
(substitute-keyword-arguments (package-arguments scotch)
- ((#:configure-flags flags ''())
- ''("-DBUILD_SHARED_LIBS=YES" "-DBUILD_PTSCOTCH=OFF"
- "-DINTSIZE=32"))))
+ ((#:configure-flags flags #~'())
+ #~'("-DBUILD_SHARED_LIBS=YES" "-DBUILD_PTSCOTCH=OFF" "-DINTSIZE=32"))))
(synopsis
"Programs and libraries for graph algorithms (32-bit integers)")))
@@ -5274,13 +5274,12 @@ (define-public pt-scotch
(list openmpi)) ;headers include MPI headers
(arguments
(substitute-keyword-arguments (package-arguments scotch)
- ((#:configure-flags flags ''())
- ''("-DBUILD_SHARED_LIBS=YES" "-DBUILD_PTSCOTCH=ON"
- "-DINTSIZE=64"))
- ((#:phases phases '%standard-phases)
- `(modify-phases ,phases
- (add-before 'check 'mpi-setup
- ,%openmpi-setup)))))
+ ((#:configure-flags flags #~'())
+ #~'("-DBUILD_SHARED_LIBS=YES" "-DBUILD_PTSCOTCH=ON" "-DINTSIZE=64"))
+ ((#:phases phases #~%standard-phases)
+ #~(modify-phases #$phases
+ (add-before 'check 'mpi-setup
+ #$%openmpi-setup)))))
(synopsis "Programs and libraries for graph algorithms (with MPI)")))
(define-public pt-scotch32
@@ -5291,9 +5290,8 @@ (define-public pt-scotch32
(list openmpi)) ;headers include MPI headers
(arguments
(substitute-keyword-arguments (package-arguments pt-scotch)
- ((#:configure-flags flags ''())
- ''("-DBUILD_SHARED_LIBS=YES" "-DBUILD_PTSCOTCH=ON"
- "-DINTSIZE=32"))))
+ ((#:configure-flags flags #~'())
+ #~'("-DBUILD_SHARED_LIBS=YES" "-DBUILD_PTSCOTCH=ON" "-DINTSIZE=32"))))
(synopsis
"Programs and libraries for graph algorithms (with MPI and 32-bit integers)")))
--
2.49.0
Information forwarded
to
andreas <at> enge.fr, bavier <at> posteo.net, sharlatanus <at> gmail.com, guix-patches <at> gnu.org
:
bug#77991
; Package
guix-patches
.
(Tue, 22 Apr 2025 15:33:04 GMT)
Full text and
rfc822 format available.
Message #17 received at 77991 <at> debbugs.gnu.org (full text, mbox):
From: Ludovic Courtès <ludovic.courtes <at> inria.fr>
* gnu/packages/maths.scm (scotch): Update to 7.0.7.
Change-Id: Idef8e0956d9da1aef13fb423c53db60b6a61b04b
---
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 abd05b9d63..5683c24f84 100644
--- a/gnu/packages/maths.scm
+++ b/gnu/packages/maths.scm
@@ -5190,7 +5190,7 @@ (define-public superlu-dist
(define-public scotch
(package
(name "scotch")
- (version "7.0.4")
+ (version "7.0.7")
(source
(origin
(method git-fetch)
@@ -5199,7 +5199,7 @@ (define-public scotch
(commit (string-append "v" version))))
(file-name (git-file-name name version))
(sha256
- (base32 "0rbc51albpd2923dkirpkj8rfkic6rsvwqqnv1mmsk391zhk3amr"))
+ (base32 "0r46bmnz9xjlgcb3vvlx3sg2qh4gfgga89vs4vlbzz3s4lj48g46"))
(modules '((guix build utils)))
(snippet
#~(substitute* "src/libscotchmetis/library_parmetis.h"
--
2.49.0
This bug report was last modified 1 day ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.