Re: [PATCH] gnu: Add hdf4

2016-10-16 Thread Andreas Enge
Hi Thomas, On Fri, Oct 14, 2016 at 11:00:29AM +0200, Thomas Danckaert wrote: > as far as I understand, the header hdfi.h defines fixed width datatypes such > as float64, uint8, int32, ... depending on the detected architecture, and > it's not detecting mips and arm properly. > > I've checked, and

Re: [PATCH] gnu: Add hdf4

2016-10-14 Thread Thomas Danckaert
From: Andreas Enge Subject: Re: [PATCH] gnu: Add hdf4 Date: Fri, 14 Oct 2016 10:11:45 +0200 Hello, the packages fail to compile on arm and mips: http://hydra.gnu.org/eval/109281#tabs-new http://hydra.gnu.org/build/1521682 Hi, as far as I understand, the header hdfi.h defines fixed

Re: [PATCH] gnu: Add hdf4

2016-10-14 Thread Andreas Enge
Hello, the packages fail to compile on arm and mips: http://hydra.gnu.org/eval/109281#tabs-new http://hydra.gnu.org/build/1521682 Could you maybe have a look at them? Thanks! Andreas

Re: [PATCH] gnu: Add hdf4

2016-10-12 Thread Ludovic Courtès
Thomas Danckaert skribis: >> Would “hdf4-minimal” sound appropriate here? That’s a convention we >> use >> in similar cases. Otherwise “hdf4-alt” is fine with me. > > With --disable-netcdf, the library still includes all the same > features, the netCDF API function names are just mangled to avo

Re: [PATCH] gnu: Add hdf4

2016-10-12 Thread Thomas Danckaert
From: l...@gnu.org (Ludovic Courtès) Subject: Re: [PATCH] gnu: Add hdf4 Date: Wed, 12 Oct 2016 14:54:40 +0200 - I've also added a variant “hdf4-alt”: by default, HDF4 includes a netCDF API to access HDF4 files, which clashes with the symbols in the real netCDF library when you t

Re: [PATCH] gnu: Add hdf4

2016-10-12 Thread Ludovic Courtès
Hello! Thomas Danckaert skribis: > this patch continues the work done by Jeremy Robst to package HDF4 > > (https://lists.gnu.org/archive/html/guix-devel/2016-06/msg00069.html) Nice! > - Back then, people reported occasional build failures. I've disabled >parallel tests and don't have any

Re: [PATCH] gnu: Add hdf4

2016-06-03 Thread Ludovic Courtès
Hello, Jeremy Robst skribis: >>> Any ideas? >> >> I get the same failures intermittently. Other times the build passes. > > Yes, I've built it many times over the past few days - sometimes I get > a failure, other times it builds ok, like you. If I do > > ./pre-inst-env guix gc > ./pre-inst-env

Re: [PATCH] gnu: Add hdf4

2016-06-02 Thread Jeremy Robst
Hi, Any ideas? I get the same failures intermittently. Other times the build passes. Yes, I've built it many times over the past few days - sometimes I get a failure, other times it builds ok, like you. If I do ./pre-inst-env guix gc ./pre-inst-env guix build hdf4 it usually (but not al

Re: [PATCH] gnu: Add hdf4

2016-06-02 Thread Eric Bavier
On 2016-05-29 17:07, l...@gnu.org wrote: Hello, Jeremy Robst skribis: Here's the patch as an attachment - sorry for the hassle. I will try and work out what's going wrong so I can send proper patches in future. No problem! I tried the attached patch, which includes mostly cosmetic changes

Re: [PATCH] gnu: Add hdf4

2016-05-29 Thread Ludovic Courtès
Hello, Jeremy Robst skribis: > Here's the patch as an attachment - sorry for the hassle. I will try > and work out what's going wrong so I can send proper patches in > future. No problem! I tried the attached patch, which includes mostly cosmetic changes compared to the one you posted. Howeve

Re: [PATCH] gnu: Add hdf4

2016-05-29 Thread Jeremy Robst
Hi, Jeremy: could you send it as an attachment, or using ‘git send-email’? Here's the patch as an attachment - sorry for the hassle. I will try and work out what's going wrong so I can send proper patches in future. Jeremy -- j...@bas.ac.uk | (work) 01223 221402 (fax) 01223 362616 Unix Syst

Re: [PATCH] gnu: Add hdf4

2016-05-28 Thread Ludovic Courtès
Leo Famulari skribis: > On Wed, May 25, 2016 at 06:42:04PM +0100, Jeremy Robst wrote: >> Hi, >> >> On Tue, 24 May 2016, Eric Bavier wrote: >> >> > Could we apply a variation of the hdf5-config-date.patch? >> >> Ok, I've based a patch on this one, and removed the other unreproduciable >> output

Re: [PATCH] gnu: Add hdf4

2016-05-27 Thread Leo Famulari
On Wed, May 25, 2016 at 06:42:04PM +0100, Jeremy Robst wrote: > Hi, > > On Tue, 24 May 2016, Eric Bavier wrote: > > > Could we apply a variation of the hdf5-config-date.patch? > > Ok, I've based a patch on this one, and removed the other unreproduciable > output, so I've not got a reproduciable

Re: [PATCH] gnu: Add hdf4

2016-05-25 Thread Jeremy Robst
Hi, On Tue, 24 May 2016, Eric Bavier wrote: Could we apply a variation of the hdf5-config-date.patch? Ok, I've based a patch on this one, and removed the other unreproduciable output, so I've not got a reproduciable build. This description is noticeably more verbose than that of the hdf5

Re: [PATCH] gnu: Add hdf4

2016-05-24 Thread Eric Bavier
On 2016-05-24 08:20, Jeremy Robst wrote: Hi, I could apply the previous patch, but I've created a new one (below) that handles the tests (I think). +(license (license:x11-style + "https://www.hdfgroup.org/ftp/HDF/HDF_Current/src/unpacked/COPYING"; To me, this looks more

Re: [PATCH] gnu: Add hdf4

2016-05-24 Thread Jeremy Robst
Hi, I could apply the previous patch, but I've created a new one (below) that handles the tests (I think). +(license (license:x11-style + "https://www.hdfgroup.org/ftp/HDF/HDF_Current/src/unpacked/COPYING"; To me, this looks more like a variation on the BSD license tha

Re: [PATCH] gnu: Add hdf4

2016-05-23 Thread Leo Famulari
On Mon, May 23, 2016 at 06:22:09PM +0100, Jeremy Robst wrote: Welcome, and thanks for the patch! Unfortunately, I can't seem to apply it to my source tree. Is anybody else able to do so? I can't figure *why* it doesn't apply. Jeremy, are you able to apply it with `patch -p1` or `git am`? > +(def