FS#60567 - [hdf5] Workaround patchlevel release requiring useless rebuilds

Attached to Project: Community Packages
Opened by Bruno Pagani (ArchangeGabriel) - Tuesday, 23 October 2018, 18:12 GMT
Last edited by Buggy McBugFace (bugbot) - Saturday, 25 November 2023, 20:10 GMT
Task Type General Gripe
Category Upstream Bugs
Status Closed
Assigned To Bruno Pagani (ArchangeGabriel)
Architecture All
Severity Medium
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

On every patchlevel release, every depending package needs to be rebuild in order to work.

This is crazy, and Debian for instance patch it this way https://salsa.debian.org/debian-gis-team/hdf5/blob/master/debian/patches/relax-version-check.patch

We should think about the best way to handle this, and eventually try to upstream it.

This requires to figure out when that might actually be an issue and why upstream does this.
This task depends upon

Closed by  Buggy McBugFace (bugbot)
Saturday, 25 November 2023, 20:10 GMT
Reason for closing:  Moved
Additional comments about closing:  https://gitlab.archlinux.org/archlinux/p ackaging/packages/hdf5/issues/1
Comment by Eli Schwartz (eschwartz) - Tuesday, 23 October 2018, 19:58 GMT
The current check: https://github.com/HDFGroup/hdf5/blob/39b3b52a941dd5e012e5a472ef940d624941833b/src/H5.c#L760

This function is called as part of the standard macros for doing literally anything in the public header. It calls an abortion for major.minor.patchlevel mismatches, but the existing soname is already ${minor}${patchlevel} except it doesn't get bumped for every patchlevel -- only the ones that need it, presumably.

These checks don't seem to make any sense except to enforce sonames, something the dynamic loader already does quite well.

The likely intent is to forcibly stop people from "doing the stupid" by symlinking incompatible sonames for hysterical raisins. A better check would, obviously, be to actually compare sonames, and this is the route we should investigate.

Debian's patch simply drops the patchlevel comparison, which misses the point and might as well be `if(false)`. It is therefore not upstreamable and hence should not be used.
Comment by Buggy McBugFace (bugbot) - Tuesday, 08 August 2023, 19:11 GMT
This is an automated comment as this bug is open for more then 2 years. Please reply if you still experience this bug otherwise this issue will be closed after 1 month.

Loading...