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 validate_openexr_libs.sh to validate .so symlinks #1058

Merged
merged 2 commits into from
Jun 24, 2021

Conversation

cary-ilm
Copy link
Member

Invoked by the ci, but also helpful as a standalone in validating new releases. Validates the pkg-config and library symlinks.

Invoked by the ci, but also helpful as a standalone in validating new
releases. Validates the pkg-config and library symlinks.

Signed-off-by: Cary Phillips <cary@ilm.com>
Signed-off-by: Cary Phillips <cary@ilm.com>
Copy link
Contributor

@kdt3rd kdt3rd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The build seems to work, so my comment around use of g++ is probably not relevant, otherwise I don't see any issues - obviously, you can configure openexr that will break this, but this being a CI only script, not too worried about that.


echo -e '#include <ImfHeader.h>\n#include <OpenEXRConfig.h>\n#include <stdio.h>\nint main() { puts(OPENEXR_PACKAGE_STRING); Imf::Header h; return 0; }' > $VALIDATE_CPP

g++ $CXX_FLAGS $VALIDATE_CPP -o $VALIDATE_BIN $LD_FLAGS
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

will this work on the build matrix variants where we are using clang? (not sure if docker will put a gcc in there)

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It probably does make sense to use the same compiler as the build matrix, I'll look into a way to pass that in, although it works for now.

We recently had problems with installing broken symlinks and pkg-configs, so this is just a check in the ci to ensure that bug doesn't creep back in, not intended as a general utility or part of the test suite.

@cary-ilm cary-ilm merged commit 8e1ae91 into AcademySoftwareFoundation:master Jun 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants