Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 809008 - sci-libs/med-4.1.0 - CMake Error at config/.../medMacros.cmake:451 (MESSAGE):
Summary: sci-libs/med-4.1.0 - CMake Error at config/.../medMacros.cmake:451 (MESSAGE):
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Matthias Maier
URL:
Whiteboard:
Keywords: PullRequest
Depends on:
Blocks: hdf5-1.12
  Show dependency tree
 
Reported: 2021-08-19 07:13 UTC by Toralf Förster
Modified: 2021-08-26 18:26 UTC (History)
7 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
emerge-info.txt (emerge-info.txt,16.70 KB, text/plain)
2021-08-19 07:13 UTC, Toralf Förster
Details
CMakeOutput.log (CMakeOutput.log,89.46 KB, text/plain)
2021-08-19 07:13 UTC, Toralf Förster
Details
emerge-history.txt (emerge-history.txt,277.87 KB, text/plain)
2021-08-19 07:13 UTC, Toralf Förster
Details
environment (environment,129.02 KB, text/plain)
2021-08-19 07:13 UTC, Toralf Förster
Details
etc.portage.tar.bz2 (etc.portage.tar.bz2,15.77 KB, application/x-bzip)
2021-08-19 07:13 UTC, Toralf Förster
Details
logs.tar.bz2 (logs.tar.bz2,6.10 KB, application/x-bzip)
2021-08-19 07:13 UTC, Toralf Förster
Details
sci-libs:med-4.1.0:20210818-233310.log (sci-libs:med-4.1.0:20210818-233310.log,4.58 KB, text/plain)
2021-08-19 07:13 UTC, Toralf Förster
Details
temp.tar.bz2 (temp.tar.bz2,30.11 KB, application/x-bzip)
2021-08-19 07:13 UTC, Toralf Förster
Details
Patch to compile med-4.1.0 with hdf5-1.12 (hdf5-1.12.patch,6.34 KB, patch)
2021-08-21 14:35 UTC, Alexandre Ferreira
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Toralf Förster gentoo-dev 2021-08-19 07:13:31 UTC
-- _versionComponents=1;12;1
-- HDF5_VERSION=1.12.1
CMake Error at config/cmake_files/medMacros.cmake:451 (MESSAGE):
  HDF5 version is 1.12.1.  Only versions >= 1.10.2 are supported.
Call Stack (most recent call first):
  CMakeLists.txt:113 (MED_FIND_HDF5)


  -------------------------------------------------------------------

  This is an unstable amd64 chroot image at a tinderbox (==build bot)
  name: 17.1_desktop_gnome-j3-20210816-092520

  -------------------------------------------------------------------

gcc-config -l:
 [1] x86_64-pc-linux-gnu-11.2.0 *
clang version 12.0.1
Target: x86_64-pc-linux-gnu
Thread model: posix
InstalledDir: /usr/lib/llvm/12/bin
/usr/lib/llvm/12
12.0.1
Python 3.9.6
Available Ruby profiles:
  [1]   ruby26 (with Rubygems)
  [2]   ruby30 (with Rubygems) *
Available Rust versions:
  [1]   rust-1.54.0 *
The following VMs are available for generation-2:
Available Java Virtual Machines:
  (none found)

The Glorious Glasgow Haskell Compilation System, version 8.10.4

  HEAD of ::gentoo
commit c26b69a42312c3e5967877acbbb1e4e65c210e67
Author: Repository mirror & CI <repomirrorci@gentoo.org>
Date:   Wed Aug 18 22:21:35 2021 +0000

    2021-08-18 22:21:34 UTC

emerge -qpvO sci-libs/med
[ebuild  N    ] sci-libs/med-4.1.0  USE="fortran -doc -mpi -python -test" PYTHON_SINGLE_TARGET="python3_9 -python3_8"
Comment 1 Toralf Förster gentoo-dev 2021-08-19 07:13:32 UTC
Created attachment 734050 [details]
emerge-info.txt
Comment 2 Toralf Förster gentoo-dev 2021-08-19 07:13:34 UTC
Created attachment 734053 [details]
CMakeOutput.log
Comment 3 Toralf Förster gentoo-dev 2021-08-19 07:13:35 UTC
Created attachment 734056 [details]
emerge-history.txt
Comment 4 Toralf Förster gentoo-dev 2021-08-19 07:13:37 UTC
Created attachment 734059 [details]
environment
Comment 5 Toralf Förster gentoo-dev 2021-08-19 07:13:38 UTC
Created attachment 734062 [details]
etc.portage.tar.bz2
Comment 6 Toralf Förster gentoo-dev 2021-08-19 07:13:39 UTC
Created attachment 734065 [details]
logs.tar.bz2
Comment 7 Toralf Förster gentoo-dev 2021-08-19 07:13:40 UTC
Created attachment 734068 [details]
sci-libs:med-4.1.0:20210818-233310.log
Comment 8 Toralf Förster gentoo-dev 2021-08-19 07:13:41 UTC
Created attachment 734071 [details]
temp.tar.bz2
Comment 9 Alexandre Ferreira 2021-08-20 13:01:04 UTC
Same here
Comment 10 James Beddek 2021-08-21 07:05:01 UTC
Ditto
Comment 11 James Beddek 2021-08-21 07:30:23 UTC
Even ignoring the version make check, sci-libs/med fails to compile if sci-libs/hdf5 is not installed with the 0/1.10.5 subslot.

Installing =sci-libs/hdf5-1.10.5-r1 and sci-libs/med compiles successfully.
Comment 12 Alexandre Ferreira 2021-08-21 14:35:16 UTC
Created attachment 734884 [details, diff]
Patch to compile med-4.1.0 with hdf5-1.12
Comment 13 Alexandre Ferreira 2021-08-21 14:38:32 UTC
Arch linux had a patch but that was not enough. Weirdly, the generated H5versions.h for hdf5 required setting 16 and 18 API versions so all the required functions are defined. I cannot compile with mpi support on my machine so the mpi is untested.
Comment 14 Hendrik v. Raven 2021-08-22 11:31:35 UTC
Thanks for the patch, fixes the problem on my system.

This bug should also block hdf5-1.12 #808733
Comment 15 Bernd 2021-08-24 05:19:34 UTC
In the patch, the lines which check for H5_VERS_MINOR seem to be redundant. 12 is greater than 10, so IMO there's no need to change these lines.

I'm already working on a patch, to change the function calls, which have changed with hdf5 v1.12.
Comment 16 Bernd 2021-08-24 06:48:04 UTC
Didn't pay enough attention to the error lines following the H5_VERS_MINOR, so these are indeed needed. But I'm not sure if this is correct and med actually implements 1.8 API. It should, however, or else they need to change a lot of their functions.

I didn't need the line changing hversionMMR in MedfileCompatibility.c, however and we don't need to change the lines for the autotools files, which we don't use anymore.
Comment 17 Larry the Git Cow gentoo-dev 2021-08-25 06:17:57 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=350ea3e89d0e87e35a3a4b5f2ce92b55aae9d226

commit 350ea3e89d0e87e35a3a4b5f2ce92b55aae9d226
Author:     Bernd Waibel <waebbl-gentoo@posteo.net>
AuthorDate: 2021-08-24 15:45:11 +0000
Commit:     Andrew Ammerlaan <andrewammerlaan@gentoo.org>
CommitDate: 2021-08-25 06:13:24 +0000

    sci-libs/med: fix build against hdf5-1.12
    
    Thanks to Alexandre Ferreira for providing the patch.
    
    Closes: https://bugs.gentoo.org/809008
    Package-Manager: Portage-3.0.20, Repoman-3.0.3
    Signed-off-by: Bernd Waibel <waebbl-gentoo@posteo.net>
    Closes: https://github.com/gentoo/gentoo/pull/22096
    Signed-off-by: Andrew Ammerlaan <andrewammerlaan@gentoo.org>

 .../med-4.1.0-0003-build-against-hdf5-1.12.patch   | 117 +++++++++++++++++++++
 sci-libs/med/med-4.1.0.ebuild                      |   6 +-
 2 files changed, 121 insertions(+), 2 deletions(-)
Comment 18 Bernd 2021-08-26 06:14:49 UTC
Re-opening this. The PR has to be reverted, as it breaks sci-libs/gmsh without the CI reporting it before it was merged.
Comment 19 Bernd 2021-08-26 18:26:26 UTC
The package falls back to using hdf5-1.10 whenever USE=mpi is provided, so there's no need to mask the mpi use flag on med. The ebuild itself builds, only the package.use.mask change happened to introduce the break in ::gentoo.

Closing this bug.