PR#4: Update to 2.7.1 (RHBZ#2072222) - rpms/highfive

rpms / highfive

#4 Update to 2.7.1 (RHBZ#2072222)

Merged 7 months ago by gui1ty. Opened 7 months ago by gui1ty.

rpms/ gui1ty/highfive rawhide into rawhide

Stop building for i686

Sandro • 7 months ago

6e4ae03

Update to 2.7.1 (RHBZ#2072222)

Sandro • 7 months ago

0c56372

.gitignore

file modified

		`@@ -1,2 +1,3 @@`
		`/highfive-2.2.2.tar.gz`
		`/highfive-2.3.1.tar.gz`
		`+ /highfive-2.7.1.tar.gz`

highfive.spec

file modified

+7 -12

		`@@ -36,7 +36,7 @@`
		`%global debug_package %{nil}`

		`Name: highfive`
		`- Version: 2.3.1`
		`+ Version: 2.7.1`
		`Release: %autorelease`
		`Summary: Header-only C++ HDF5 interface`

		`@@ -44,20 +44,14 @@`
		`URL: https://bluebrain.github.io/HighFive/`
		`Source0: https://github.com/BlueBrain/HighFive/archive/v%{version}/%{name}-%{version}.tar.gz`

		`- # Does not build on 32 bit architectures`
		`- # Issue filed upstream: https://github.com/BlueBrain/HighFive/issues/443`
		`- # https://bugzilla.redhat.com/show_bug.cgi?id=1952348`
		`- #`
		`- # Partially patched, fixing i686; see upstream bug for armv7hl status. We can`
		`- # work around this by disabling OpenCV on armv7hl until a patch is available.`
		`- Patch0: 0001-fix-32bit-arches-use-explicit-casts.patch`
		`- Patch1: 0001-Fix-compiling-invalid-reinterpret_cast-on-32-bit.patch`
		`- Patch2: 27a8f06d58a0bdb5c31a84fd8a653a9433f06082.patch`
		`+ # https://fedoraproject.org/wiki/Changes/EncourageI686LeafRemoval`
		`+ ExcludeArch: %{ix86}`

		`BuildRequires: cmake`
		`BuildRequires: gcc-c++`
		`BuildRequires: git-core`
		`BuildRequires: hdf5-devel`
		`+ BuildRequires: catch-devel`
		`# Technically optional, enabled by default`
		`BuildRequires: boost-devel`
		`# Our choice vs. make`
		`@@ -148,13 +142,14 @@`

		`%check`
		`%if %{with tests}`
		`- %ctest`
		`+ # Run tests sequentially: https://github.com/BlueBrain/HighFive/issues/825`
		`+ %ctest -VV -j 1`
		`%endif`


		`%files devel`
		`%license LICENSE`
		`- %doc README.md VERSION CHANGELOG.md`
		`+ %doc README.md AUTHORS.txt CHANGELOG.md`
		`%{_includedir}/%{name}`
		`%{_libdir}/cmake/%{pretty_name}`

sources

file modified

+1 -1

		`@@ -1,1 +1,1 @@`
		`- SHA512 (highfive-2.3.1.tar.gz) = 984a6da556b1b4dbe183e5d5c6c9b9a15aac29855a67e6c4a5b83bcbf0a0f04749abcde68e8e976d43c4838f797c8d7de7c36f22e11ac3d3e87e4f87e20e2da8`
		`+ SHA512 (highfive-2.7.1.tar.gz) = 4fbbd3898791a67e44329a5d0e20e16454b9393510236563b12fe4346cd4f2785d43d915ea05239ac1568d00651e41d85d93590f01454ffc1b82e7bba28e780a`

gui1ty commented 7 months ago

Remove redundant patches
Add BR: catch-devel (That's catch2 3.x)
Fix %doc files
Run tests sequentially

gui1ty commented 7 months ago

I will check that highfive consumers IQmol, libsonata and morphio still build against the updated package.

gui1ty commented 7 months ago

[citest]

gui1ty commented 7 months ago

I will check that highfive consumers IQmol, libsonata and morphio still build against the updated package.

They all succeeded: https://copr.fedorainfracloud.org/coprs/gui1ty/neuro-sig/builds/

As to the failed scratch build: one of the tests failed on i686. I'll trigger that again and if it keeps failing, disable it. The remaining 287 tests appear to run fine.

gui1ty commented 7 months ago

[citest]

1 new commit added