search

URDME / urdme

Stochastic reaction-diffusion simulation on unstructured meshes.
www.urdme.org
GNU General Public License v3.0
28 stars 12 forks source link

zlib.h dependency #22

Closed pavolbauer closed 11 years ago

pavolbauer commented 11 years ago

The header file zlib.h is needed in the build of src/read_matfile.c.

This is an external dependency which is not handeled during the installation.

Since URDME_LIBMAT=yes in the default makefile.nsm, a running a URDME-model crashes on a system without zlib.

Can we take this out or do we need dependency managment?

ahellander commented 11 years ago

So what's our options here? Seems to me that we need to require zlib.h. There can't be many platforms that dees not include it in the standard distribution? Alternatively, we can always make the Mathworks libraries the default again, but i.mo. that would be a step in the wrong direction.

Brian: Does the matlib really use compression? If so, can we patch the install script to check for zlib.h (wihtout using e.g. autotools)?

briandrawert commented 11 years ago

Hi Pavol, Can you tell me which platform you are using? I was under the impression that zlib.h was installed by default on all platforms that we support.

pavolbauer commented 11 years ago

Hi Brian, The problem appears on Ubuntu, in the current version. For some reason the matlib works as well if zlib its not included. Is it necessary to have it? Cheers P On 20 Apr 2013 18:35, "Brian Drawert" notifications@github.com wrote:

Hi Pavol, Can you tell me which platform you are using? I was under the impression that zlib.h was installed by default on all platforms that we support.

— Reply to this email directly or view it on GitHubhttps://github.com/URDME/urdme/issues/22#issuecomment-16706892 .