From mboxrd@z Thu Jan 1 00:00:00 1970 From: ludo@gnu.org (Ludovic =?UTF-8?Q?Court=C3=A8s?=) Subject: bug#32691: OpenMolcas Date: Fri, 14 Sep 2018 11:27:50 +0200 Message-ID: <87h8isjund.fsf@gnu.org> References: <87d0tkk6gh.fsf@fastmail.com> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="=-=-=" Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:45543) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1g0kPR-0003PZ-95 for bug-guix@gnu.org; Fri, 14 Sep 2018 05:29:06 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1g0kPO-0001BI-2q for bug-guix@gnu.org; Fri, 14 Sep 2018 05:29:05 -0400 Received: from debbugs.gnu.org ([208.118.235.43]:35794) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1g0kPN-0001BA-Tg for bug-guix@gnu.org; Fri, 14 Sep 2018 05:29:02 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1g0kPN-0001Gn-Pp for bug-guix@gnu.org; Fri, 14 Sep 2018 05:29:01 -0400 Sender: "Debbugs-submit" Resent-Message-ID: In-Reply-To: <87d0tkk6gh.fsf@fastmail.com> (Eric Brown's message of "Mon, 10 Sep 2018 23:23:42 -0500") List-Id: Bug reports for GNU Guix List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: bug-guix-bounces+gcggb-bug-guix=m.gmane.org@gnu.org Sender: "bug-Guix" To: Eric Brown Cc: 32691@debbugs.gnu.org --=-=-= Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Hi Eric, Eric Brown skribis: > I am pleased to submit a patch that adds OpenMolcas to Guix. Also > included in this submission is a new variable called `openblas-ilp64' > which OpenMolcas needs to compile and run. Here=E2=80=99s a revision of the patch with several modifications: I added comments in the patch (though they don=E2=80=99t explain the Python-related changes for instance; could you add a line about these?), stripped unnecessary additions from the patch, added it to gnu/local.mk, fixed an issue reported by =E2=80=98guix lint=E2=80=99 in the description. However, could you explain why we have #:tests? #f ? In general disabling tests should be accompanied with a comment explaining the reason (lack of a test suite, or =E2=80=9Cunsolvable=E2=80=9D test failures= .) If there are test failures, we=E2=80=99d rather investigate than disable the tests. Could you send an updated patch? Thanks in advance! Ludo=E2=80=99. --=-=-= Content-Type: text/x-patch; charset=utf-8 Content-Disposition: inline; filename=0001-gnu-Add-openmolcas.patch Content-Transfer-Encoding: quoted-printable Content-Description: the patch >From c001d557e1cb23042c092c23d7612ad6197fac3c Mon Sep 17 00:00:00 2001 From: Eric Brown Date: Mon, 10 Sep 2018 23:12:51 -0500 Subject: [PATCH] gnu: Add openmolcas. MIME-Version: 1.0 Content-Type: text/plain; charset=3DUTF-8 Content-Transfer-Encoding: 8bit * gnu/packages/chemistry.scm (openmolcas): New variable. * gnu/packages/patches/openmolcas-pymolcas-openblas.patch: New file. * gnu/local.mk (dist_patch_DATA): Add it. Signed-off-by: Ludovic Court=C3=A8s --- gnu/local.mk | 1 + gnu/packages/chemistry.scm | 52 +++++++++++ .../openmolcas-pymolcas-openblas.patch | 89 +++++++++++++++++++ 3 files changed, 142 insertions(+) create mode 100644 gnu/packages/patches/openmolcas-pymolcas-openblas.patch diff --git a/gnu/local.mk b/gnu/local.mk index 7b230cb6f..0974ce068 100644 --- a/gnu/local.mk +++ b/gnu/local.mk @@ -1006,6 +1006,7 @@ dist_patch_DATA =3D \ %D%/packages/patches/opencascade-oce-glibc-2.26.patch \ %D%/packages/patches/openfoam-4.1-cleanup.patch \ %D%/packages/patches/openldap-CVE-2017-9287.patch \ + %D%/packages/patches/openmolcas-pymolcas-openblas.patch \ %D%/packages/patches/openocd-nrf52.patch \ %D%/packages/patches/opensmtpd-fix-crash.patch \ %D%/packages/patches/openssl-runpath.patch \ diff --git a/gnu/packages/chemistry.scm b/gnu/packages/chemistry.scm index 03120aaee..58151afce 100644 --- a/gnu/packages/chemistry.scm +++ b/gnu/packages/chemistry.scm @@ -1,6 +1,7 @@ ;;; GNU Guix --- Functional package management for GNU ;;; Copyright =C2=A9 2018 Konrad Hinsen ;;; Copyright =C2=A9 2018 Kei Kebreau +;;; Copyright =C2=A9 2018 Eric Brown ;;; ;;; This file is part of GNU Guix. ;;; @@ -26,12 +27,16 @@ #:use-module (gnu packages boost) #:use-module (gnu packages compression) #:use-module (gnu packages documentation) + #:use-module (gnu packages gcc) + #:use-module (gnu packages ghostscript) #:use-module (gnu packages gl) #:use-module (gnu packages gv) #:use-module (gnu packages maths) + #:use-module (gnu packages perl) #:use-module (gnu packages pkg-config) #:use-module (gnu packages python) #:use-module (gnu packages qt) + #:use-module (gnu packages tex) #:use-module (gnu packages xml) #:use-module (guix build-system cmake) #:use-module (guix build-system gnu) @@ -322,3 +327,50 @@ chemical data. It's a collaborative project allowing = anyone to search, convert, analyze, or store data from molecular modeling, chemistry, solid-state materials, biochemistry, or related areas.") (license license:gpl2))) + +(define-public openmolcas + (package + (name "openmolcas") + (version "18.09") + (source (origin + (method url-fetch) + (uri (string-append "https://gitlab.com/Molcas/OpenMolcas" + "/-/archive/v" version "/OpenMolcas-v" + version ".tar.gz")) + (file-name (string-append name "-" version ".tar.gz")) + (sha256 + (base32 + "1krrkff5b4zp9ii1m9v3zzw8d99k72fcwwv3rif8cp10cpxi1wxl")) + (patches (search-patches "openmolcas-pymolcas-openblas.patch= ")))) + (build-system cmake-build-system) + (arguments + `(#:out-of-source? #t + #:tests? #f + #:build-type "Release" + #:configure-flags + (list "-DLINALG=3DOpenBLAS" + (string-append "-DOPENBLASROOT=3D" + (assoc-ref %build-inputs "blas")) + (string-append "-DLINALG_LIBRARIES=3D" + (assoc-ref %build-inputs "blas") + "/lib/libopenblas_ilp64.so")))) + (native-inputs + `(("fortran" ,gfortran) + ("ghostscript" ,ghostscript) + ("perl" ,perl) + ("pkg-config" ,pkg-config) + ("python" ,python) + ("python-pyparsing" ,python-pyparsing) + ("python-six" ,python-six) + ("texlive" ,texlive))) + (inputs + `(("hdf5" ,hdf5) + ("blas" ,openblas-ilp64))) + (home-page "https://gitlab.com/Molcas/OpenMolcas") + (synopsis "Multi-reference electronic structure theory program") + (description "OpenMolcas is a quantum chemistry software package devel= oped +by scientists and intended to be used by scientists. It includes programs= to +apply many different electronic structure methods to chemical systems, but= its +key feature is the multiconfigurational approach, with methods like CASSCF= and +CASPT2.") + (license license:lgpl2.1))) diff --git a/gnu/packages/patches/openmolcas-pymolcas-openblas.patch b/gnu/= packages/patches/openmolcas-pymolcas-openblas.patch new file mode 100644 index 000000000..b18c62e05 --- /dev/null +++ b/gnu/packages/patches/openmolcas-pymolcas-openblas.patch @@ -0,0 +1,89 @@ +Allow bulds with openblas-ilp64. +Remove assumptions about $HOME. + +diff --git a/CMakeLists.txt b/CMakeLists.txt +index a4071fd..53d8df5 100644 +--- a/CMakeLists.txt ++++ b/CMakeLists.txt +@@ -1338,42 +1338,8 @@ if (LINALG STREQUAL "OpenBLAS") + endif () +=20 + # search for the OpenBLAS library +- find_library (LIBOPENBLAS +- NAMES openblas +- PATHS ${OPENBLASROOT} +- PATH_SUFFIXES lib +- NO_DEFAULT_PATH) +- +- if (NOT LIBOPENBLAS) +- message (FATAL_ERROR +- "OpenBLAS library not found, please check that " +- "the OPENBLASROOT variable is set and points to " +- "a valid OpenBLAS installation directory." +- ) +- endif () +- +- # here we check if LIBOPENBLAS has changed after it was processed +- # succesfully, if not we do not need to rerun anything here. +- if (NOT LIBOPENBLAS STREQUAL LIBOPENBLAS_LAST) +- # check if the OpenBLAS library contains LAPACK functiona= lity +- message ("-- Checking OpenBLAS for LAPACK functionality..= .") +- include(CheckFortranFunctionExists) +- set (CMAKE_REQUIRED_LIBRARIES ${LIBOPENBLAS}) +- unset (OPENBLAS_WITH_LAPACK CACHE) +- check_fortran_function_exists("dsyev" OPENBLAS_WITH_LAPAC= K) +- if (NOT OPENBLAS_WITH_LAPACK) +- unset (LIBOPENBLAS CACHE) +- message (FATAL_ERROR +- "LAPACK functionality missing from OpenBL= AS library, " +- "please build OpenBLAS with NO_LAPACK=3D0= defined." +- ) +- endif () +- # save the last location to check if it changed between c= onfigurations +- set (LIBOPENBLAS_LAST ${LIBOPENBLAS} CACHE INTERNAL "last= value." FORCE) +- endif () +- + add_definitions (-D_OPENBLAS_) +- set (LINALG_LIBRARIES ${LIBOPENBLAS}) ++ set (LINALG_LIBRARIES ${LINALG_LIBRARIES}) + else () + mark_as_advanced(FORCE OPENBLASROOT) + endif () +@@ -3167,11 +3168,10 @@ install (PROGRAMS + if (DEFINED PYMOLCAS_TARGET) + install (PROGRAMS + ${PYMOLCAS_TARGET} +- DESTINATION ${CMAKE_INSTALL_PREFIX}/sbin ++ DESTINATION ${CMAKE_INSTALL_PREFIX}/bin + RENAME pymolcas + ) +=20 +- install (CODE "execute_process(COMMAND ${OPENMOLCAS_DIR}/sbin/ins= tall_pymolcas.sh ${PYMOLCAS_TARGET})") + endif () +=20 + if (DEFINED EXTRA_DIR) +@@ -3188,9 +3189,5 @@ if (DEFINED EXTRA_DIR) + DESTINATION ${CMAKE_INSTALL_PREFIX}/bin + ) +=20 +- install (CODE "execute_process(COMMAND ${OPENMOLCAS_DIR}/sbin/ins= tall_driver.sh ${MOLCAS_DRIVER})") + endif () +=20 +-if (IS_DIRECTORY "$ENV{HOME}/.Molcas") +- install (CODE "execute_process(COMMAND sh \"-c\" \"cd ${CMAKE_INS= TALL_PREFIX}; echo `pwd` > $HOME/.Molcas/molcas 2> /dev/null\")") +-endif () +diff --git a/Tools/pymolcas/CMakeLists.txt b/Tools/pymolcas/CMakeLists.txt +index 0d576fa..f9ba6d7 100644 +--- a/Tools/pymolcas/CMakeLists.txt ++++ b/Tools/pymolcas/CMakeLists.txt +@@ -52,9 +52,5 @@ if (PYTHONINTERP_FOUND) + DEPENDS ${PYMOLCAS_TARGET} + ) +=20 +- add_custom_command (TARGET pymolcas +- POST_BUILD +- COMMAND ${OPENMOLCAS_DIR}/sbin/install_pymolcas.s= h ${PYMOLCAS_TARGET} +- ) +=20 + endif () + --=20 2.18.0 --=-=-=--