Bug#693610: pointless strict hdf version check
Julian Taylor
jtaylor.debian at googlemail.com
Sun Nov 18 17:11:30 UTC 2012
reassign 693610 hdf5 1.8.9-1~exp1
severity 693610 serious
tags 693610 + experimental
thanks
hdf5 1.8.9 is not compatible with 1.8.8 as long as this check is in place.
Either the check should be relaxed, removed or the soversion bumped.
This kind of thing should be handled by soversion and not runtime <->
compile time checks, so removing the check completely might be the best
option.
attached a debdiff to relax the check to major.minor
-------------- next part --------------
diff -Nru hdf5-1.8.9/debian/changelog hdf5-1.8.9/debian/changelog
--- hdf5-1.8.9/debian/changelog 2012-09-30 17:40:53.000000000 +0200
+++ hdf5-1.8.9/debian/changelog 2012-11-18 17:36:33.000000000 +0100
@@ -1,3 +1,11 @@
+hdf5 (1.8.9-1~exp1ubuntu1) UNRELEASED; urgency=low
+
+ * relax-version-check.patch:
+ don't check the compile time patch version at runtime, this saves us
+ full transitions when it is still compatible.
+
+ -- Julian Taylor <jtaylor at ubuntu.com> Sun, 18 Nov 2012 17:35:31 +0100
+
hdf5 (1.8.9-1~exp1) experimental; urgency=low
* New upstream release (Closes: #673788)
diff -Nru hdf5-1.8.9/debian/control hdf5-1.8.9/debian/control
--- hdf5-1.8.9/debian/control 2012-10-01 18:18:57.000000000 +0200
+++ hdf5-1.8.9/debian/control 2012-11-18 17:36:44.000000000 +0100
@@ -1,7 +1,8 @@
Source: hdf5
Section: science
Priority: optional
-Maintainer: Debian GIS Project <pkg-grass-devel at lists.alioth.debian.org>
+Maintainer: Ubuntu Developers <ubuntu-devel-discuss at lists.ubuntu.com>
+XSBC-Original-Maintainer: Debian GIS Project <pkg-grass-devel at lists.alioth.debian.org>
Uploaders: Francesco Paolo Lovergine <frankie at debian.org>,
Sylvestre Ledru <sylvestre at debian.org>
Build-Depends: dpkg-dev (>= 1.16.1~), libmpich2-dev, zlib1g-dev,
diff -Nru hdf5-1.8.9/debian/control.in hdf5-1.8.9/debian/control.in
--- hdf5-1.8.9/debian/control.in 2012-10-01 18:16:36.000000000 +0200
+++ hdf5-1.8.9/debian/control.in 2012-11-18 17:36:42.000000000 +0100
@@ -1,7 +1,8 @@
Source: hdf5
Section: science
Priority: optional
-Maintainer: Debian GIS Project <pkg-grass-devel at lists.alioth.debian.org>
+Maintainer: Ubuntu Developers <ubuntu-devel-discuss at lists.ubuntu.com>
+XSBC-Original-Maintainer: Debian GIS Project <pkg-grass-devel at lists.alioth.debian.org>
Uploaders: Francesco Paolo Lovergine <frankie at debian.org>,
Sylvestre Ledru <sylvestre at debian.org>
Build-Depends: dpkg-dev (>= 1.16.1~), libmpich2-dev, zlib1g-dev,
diff -Nru hdf5-1.8.9/debian/patches/relax-version-check.patch hdf5-1.8.9/debian/patches/relax-version-check.patch
--- hdf5-1.8.9/debian/patches/relax-version-check.patch 1970-01-01 01:00:00.000000000 +0100
+++ hdf5-1.8.9/debian/patches/relax-version-check.patch 2012-11-18 17:33:57.000000000 +0100
@@ -0,0 +1,20 @@
+Description: remove check on patch version
+ hopefully changes in that number always are compatible.
+ Without this every bugfix update would require a full transition as it will
+ abort when the version does not match exactly.
+ The version check will be executed with compile time values on every file
+ access see e.g. H5F_ACC_RDONLY in src/H5Fpublic.h
+Author: Julian Taylor <jtaylor at ubuntu.com>
+
+--- a/src/H5.c
++++ b/src/H5.c
+@@ -665,8 +665,7 @@
+ disable_version_check = (unsigned int)HDstrtol (s, NULL, 0);
+ }
+
+- if (H5_VERS_MAJOR!=majnum || H5_VERS_MINOR!=minnum ||
+- H5_VERS_RELEASE!=relnum) {
++ if (H5_VERS_MAJOR!=majnum || H5_VERS_MINOR!=minnum) {
+ switch (disable_version_check) {
+ case 0:
+ HDfprintf(stderr, "%s%s", version_mismatch_warning,
diff -Nru hdf5-1.8.9/debian/patches/series hdf5-1.8.9/debian/patches/series
--- hdf5-1.8.9/debian/patches/series 2012-06-14 17:42:44.000000000 +0200
+++ hdf5-1.8.9/debian/patches/series 2012-11-18 17:30:34.000000000 +0100
@@ -1,3 +1,4 @@
mpi.diff
path_max.diff
ullong_force.diff
+relax-version-check.patch
More information about the debian-science-maintainers
mailing list