[DebianGIS-dev] Bug#473871: hdf5: Doesn't actually build openmpi libs

[Francesco P. Lovergine]

On Wed, Apr 02, 2008 at 11:39:12AM +0200, Rafael Laboissiere wrote:
> * Francesco P. Lovergine <frankie at debian.org> [2008-04-02 10:21]:
> 
> > On Tue, Apr 01, 2008 at 11:13:16PM -0400, Adam C Powell IV wrote:
> > > Package: hdf5
> > > Version: 1.6.6-3
> > > 
> > > Greetings,
> > > 
> > > I'm afraid your rules file doesn't build openmpi libraries on the
> > > architectures where it's supposed to, so the openmpi packages are empty.
> > [snip]
> > 
> > I don't like that so much because OpenMPI detection should be done
> > without using paths which could be subject to changes due to
> > other maintainers' choices :-/ 
> > I'll try to find out why it does not work properly and how manage
> > better the issue.
> 
> It was my fault, sorry.  I used hyphens instead of underscores in the
> variable names in debian/rules.  The patch attached below should fix it.
> 
> Thanks,
>  

> Index: debian/rules
> ===================================================================
> --- debian/rules	(revision 1493)
> +++ debian/rules	(working copy)
> @@ -80,7 +80,7 @@
>  		  --enable-shared --enable-production=$(USE_PROD)
>  
>  
> -configure: configure-stamp-debian configure-stamp-serial configure-stamp-lam $(configure-stamp-openmpi) configure-stamp-mpich
> +configure: configure-stamp-debian configure-stamp-serial configure-stamp-lam $(configure_stamp_openmpi) configure-stamp-mpich
>  
>  configure-stamp-debian: debian/control.in
>  	cd debian && for i in *_devlib; do j=`basename $$i _devlib`; \
> @@ -138,7 +138,7 @@
>  		--enable-parallel=yes
>  	touch configure-stamp-mpich
>  
> -build: build-stamp-serial build-stamp-lam $(build-stamp-openmpi) build-stamp-mpich
> +build: build-stamp-serial build-stamp-lam $(build_stamp_openmpi) build-stamp-mpich
>  
>  build-stamp-serial: configure-stamp-serial
>  	dh_testdir
> @@ -171,7 +171,7 @@
>  	grep -v ^\% debian/control.in | sed "s/@VERSION@/$(version)/g" > debian/control
>  	dh_clean configure-stamp* build-stamp* install-stamp* debian/*.shlibs
>  
> -install: install-serial install-lam install-openmpi install-mpich install-doc
> +install: install-serial install-lam $(install_openmpi) install-mpich install-doc
>  
>  install-serial: build-stamp-serial
>  	dh_testdir
> @@ -222,7 +222,7 @@
>  	dh_md5sums -i
>  	dh_builddeb -i
>  
> -binary-arch: build install-serial install-lam $(install-openmpi) install-mpich
> +binary-arch: build install-serial install-lam $(install_openmpi) install-mpich
>  	dh_testdir
>  	dh_testroot 
>  	dh_installdocs $(ARCH_FLAG)

Ah right I missed that! Thanks 

-- 
Francesco P. Lovergine