Page MenuHomeSolus

hdf5: switch build to autotools
ClosedPublic

Authored by aleksvor on Feb 16 2021, 10:07 AM.
Tags
None
Referenced Files
F10799851: D10550.id25415.diff
Sun, May 28, 4:32 PM
F10796994: D10550.id26219.diff
Sun, May 28, 4:10 AM
F10748782: D10550.diff
Mon, May 15, 10:36 PM
F10739461: D10550.id25849.diff
Sat, May 13, 7:41 PM
F10739018: D10550.diff
Sat, May 13, 3:52 PM
F10738986: D10550.id26219.diff
Sat, May 13, 3:39 PM
F10738925: D10550.id25863.diff
Sat, May 13, 3:04 PM
F10738391: D10550.id26010.diff
Sat, May 13, 11:28 AM
Tokens
"Like" token, awarded by YakoYakoYokuYoku.

Details

Summary

Switch hdf5 build to autotools. CMake build generates broken CMake configuration files which then break builds of all revdeps which use CMake (namely netcdf, cgns, ParaView and VTK >= 9.0).
Autotools don't build libhdf5_tools library, but at this time nothing in the repo uses it.

Test Plan

Built revdeps:

  • netcdf-c
  • cgns
  • paraview
  • gdal
  • octave

Diff Detail

Repository
R1396 hdf5
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Were the other revdeps built for validation, like octave and gdal?

Were the other revdeps built for validation, like octave and gdal?

Yes, I have tested rebuilds of these against hdf5 and can confirm that they are fine. I'll update the test plan.

YakoYakoYokuYoku added inline comments.
package.yml
12–22

I'd suggest to run CMake once so we can have the *.pc files, i.e.:

%cmake_ninja -DHDF5_ENABLE_Z_LIB_SUPPORT=ON -DHDF5_INSTALL_LIB_DIR=%libdir%

The *.pcs reside in solusBuildDir/CMakeFiles/*.pc, then these should be installed and lastly all of them needs to be copied/symlinked to a non versioned *.pc file so that we don't have broken pkg-config dependencies.

Generate pkgconfigs and add non-versioned symlinks for them.

package.yml
31

file reports me broken links:

/usr/lib64/pkgconfig/hdf5.pc: broken symbolic link to /home/build/YPKG/root/hdf5/install//usr/lib64/pkgconfig//hdf5-1.10.4.pc

I'd suggest to remove $PKGCONFIGDIR/ and leave only hdf5-$version.pc, meaning that this will symlink to hdf5-$version.pc if it's in the same dir, this also applies to the rest.

Fix my thinko - correct the targets of symlinks.

Copy versioned pkgconfigs instead of symlinking them.

This revision is now accepted and ready to land.Apr 23 2021, 11:55 AM
This revision was automatically updated to reflect the committed changes.