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

Fix udunits on mac / travis #977

Closed
alliepiper opened this issue Jan 21, 2015 · 4 comments · Fixed by #978
Closed

Fix udunits on mac / travis #977

alliepiper opened this issue Jan 21, 2015 · 4 comments · Fixed by #978
Assignees

Comments

@alliepiper
Copy link
Contributor

PR #847 updated the udunits version and removed the patch that fixed compilation on Travis's mac boxes (the failing flex issue, #769).

Either update the udunits compilation to use the new CMake support, or reapply the flex patch.

@alliepiper alliepiper self-assigned this Jan 21, 2015
@alliepiper
Copy link
Contributor Author

@aashish24 Their CMake system won't work -- it doesn't install the udunits.h header, which is needed by UVCDAT/Packages/unidata to build the python udunits interface.

I'll remake the patch for their configure script.

@doutriaux1
Copy link
Contributor

@dlonie I thought @jbeezley just updated their cmake to use 3.1 I guess the brew version is broken then?

If so it might be worth further editing @jbeezley patch to simply build CMake like we do for Linux?

@jbeezley
Copy link
Contributor

I think he is referring to the CMakeLists.txt file provided by udunits.

@alliepiper
Copy link
Contributor Author

It's a problem with the way the udunits package has written their CMake build system -- it doesn't install the same files as their autoconf script does.

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

Successfully merging a pull request may close this issue.

3 participants