Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

mzR fails to build on bioc devel windows (hdf5 related) #143

Closed
sgibb opened this issue Dec 23, 2017 · 3 comments
Closed

mzR fails to build on bioc devel windows (hdf5 related) #143

sgibb opened this issue Dec 23, 2017 · 3 comments

Comments

@sgibb
Copy link
Contributor

sgibb commented Dec 23, 2017

Since some time mzR fails to build from source on windows and bioc devel because of a missing hdf5 cpp header file (build report):

...
In file included from pwiz/data/msdata/mz5/Configuration_mz5.hpp:27:0,
                 from pwiz/data/msdata/mz5/Configuration_mz5.cpp:24:
pwiz/data/msdata/mz5/Datastructures_mz5.hpp:30:19: fatal error: H5Cpp.h: No such file or directory
 #include "H5Cpp.h"
                   ^
compilation terminated.
make: *** [pwiz/data/msdata/mz5/Configuration_mz5.o] Error 1
ERROR: compilation failed for package 'mzR'

The same error was present on Travis and my local machine which could be solved by adding libhdf5-dev as new dependency:

- libhdf5-dev

I guess this problem is related to the (re)introduction of mz5 (d4910ec; I don't want to start the discussion of #120 again; IMHO this code is merged so the PR could be closed anyway; the "automatic detection of HDF5" would be a different problem).

Is there a way to fix the build on bioc devel's windows machines? E.g. adding it to SystemRequirements in DESCRIPTION and/or inform the bioc build service maintainers that we need HDF5 cpp for windows?

@sneumann
Copy link
Owner

sneumann commented Jan 3, 2018

Hi,
following the suggestion by Martin Morgan on https://stat.ethz.ch/pipermail/bioc-devel/2018-January/012576.html I have started the branch https://github.com/sneumann/mzR/tree/feature/LinkToRhdf5lib
Note: since pwiz is including #include "H5Cpp.h"rather than #include "c++/H5Cpp.h" we need to find a fix for grimbough/Rhdf5lib#5
Locally I have worked around that by mv -i c++/H5* . inside /usr/local/lib/R/site-library/Rhdf5lib/include.
Yours, Steffen

@sneumann
Copy link
Owner

sneumann commented Jan 4, 2018

With some more workarounds on the travis side, I now have a successful build at https://travis-ci.org/sneumann/mzR/builds/324953759

@sneumann
Copy link
Owner

Fixed now in 2.13.3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants