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 missing -L for libyaml #3
Conversation
ups/eupspkg.cfg.sh
Outdated
@@ -3,5 +3,5 @@ TAP_PACKAGE=1 | |||
|
|||
build() | |||
{ | |||
CFLAGS=-I${LIBYAML_DIR}/include python setup.py --with-libyaml build | |||
CFLAGS="-I${LIBYAML_DIR}/include -L${LIBYAML_DIR}/lib" python setup.py --with-libyaml build |
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.
Why is it added to CFLAGS rather than LDFLAGS?
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.
Because I didn't know about LDFLAGS? :-) Lemme give that a try...
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.
LDFLAGS seems to work fine for the container build, and is definitely better. Updated PR and re-running Jenkins...
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.
CFLAGS is for compilation and LDFLAGS is for linking.
Similar to the previous change for CFLAGS, a -L option is needed on LDFLAGS to find the libyaml library at link on some systems
Sorry, I was on vacation. |
The recent pyyaml change to build on top of libyaml busted the Qserv base container builds.
It looks like a necessary -L option for libyaml is missing off the LDFLAGS additions in the build() override in the eupspkg.cfg.sh; in miniconda installs this goes unnoticed since the lib is picked up from miniconda. Since the Qserv base containers are non-miniconda builds, they broke.