-
Notifications
You must be signed in to change notification settings - Fork 33
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
update CFLAGS and LDFLAGS to support raspbian #46
base: master
Are you sure you want to change the base?
Conversation
@@ -7,7 +7,9 @@ package hdf5 | |||
// #cgo LDFLAGS: -lhdf5 -lhdf5_hl | |||
// #cgo darwin CFLAGS: -I/usr/local/include | |||
// #cgo darwin LDFLAGS: -L/usr/local/lib | |||
// #cgo linux CFLAGS: -I/usr/local/include, -I/usr/lib/x86_64-linux-gnu/hdf5/serial/include | |||
// #cgo linux LDFLAGS: -L/usr/local/lib, -L/usr/lib/x86_64-linux-gnu/hdf5/serial/ | |||
// #cgo linux !arm CFLAGS: -I/usr/local/include, -I/usr/lib/x86_64-linux-gnu/hdf5/serial/include |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
perhaps s/!arm/amd64/
instead?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please see #42 (comment)
I will be able to test this in a couple of days. |
Apologies for the long delay in getting to this. I think we have too many assumptions about word sizes for this to work as it stands. When I run the tests, we have a fair amount of breakage.
|
modify cgoflags.go to support raspbian. cgo pkg-config: hdf5 solution does not work with homebrew and ubuntu trusty, though it works on raspbian stretch and ubuntu 18.10.