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

Add isegHAL headers n libs #2

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

bhill-slac
Copy link
Contributor

Adds the headers and libs from isegHAL so developers don't need to build it.

@ffeldbauer
Copy link
Owner

I would not like to copy those files into this package. This way you always have to modify this package, when isegHal is updated. Even if there is no API change, which means you do not need to modify anything else in this package.

The way I solve this problem is, by defining ISEGHAL in the configure/RELEASE file. The Makefile uses it to create USR_INCLUDE and USR_LDFLAGS for compiling. Just copy the lib and headers from isegHal where ever you want and set the path in configure/RELEASE.
You can simply copy the latest version from the CC24 controller directly to your build server, so you do not need to build them yourself.

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 this pull request may close these issues.

2 participants