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.

Sign up for GitHub

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

Jump to bottom

build: static link hipo4 #74

Merged
merged 18 commits into from
Feb 19, 2024
Merged

build: static link hipo4 #74

merged 18 commits into from
Feb 19, 2024

Conversation

c-dilks
Copy link
Member

@c-dilks c-dilks commented Jan 7, 2024 •
edited
Loading

Base automatically changed from ci-macos to main January 16, 2024 15:59
@c-dilks c-dilks linked an issue Jan 17, 2024 that may be closed by this pull request
hipo4 libs are in test executables' rpath; consider instead linking against hipo4 static libs #23
Closed
@c-dilks c-dilks marked this pull request as ready for review February 19, 2024 20:53
c-dilks added 18 commits February 19, 2024 17:28
@c-dilks
build: static hipo4
164cd3a
@c-dilks
test: Maurik's fork
1e1cb68
@c-dilks
Revert "test: Maurik's fork"
e9153d1 
This reverts commit 7effb73.
@c-dilks
test: Muarik's fix + patch
dc7db37
@c-dilks
test: Maurik's latest change
2d5e1ab
@c-dilks
fix: hipo fork
8849da1
@c-dilks
build: optional static link
3111103
@c-dilks
fix: consumer build doesn't need hipo4 if static linked
5b21daa
@c-dilks
doc: consumer builds don't need hipo if static linked
37a8add
@c-dilks
fix(ci): lock hipo version
a04c243
@c-dilks
fix: use PkgConfig module to resolve hipo4 in cmake consumer build test
dc50433
@c-dilks
test: remove hipo4 from consumer tests, but use dynamic link in iguan…
50b3eb8 
…a build; this should fail
@c-dilks
Revert "test: remove hipo4 from consumer tests, but use dynamic link …
f0f7c08 
…in iguana build; this should fail"

This reverts commit 4292b40.
@c-dilks
doc: remove note about static build...
0f7161a
@c-dilks
build: force static linking of hipo
afaec3a
@c-dilks
test: drop hipo from rpath, since static linked
b3fd389
@c-dilks
fix: static link hipo in meson consumer example
b18a115
@c-dilks
fix: drop hipo from project_exe_rpath
125e09e
@c-dilks c-dilks changed the title build: static hipo4 build: link against static hipo4 Feb 19, 2024
@c-dilks c-dilks changed the title build: link against static hipo4 build: static link hipo4 Feb 19, 2024
@c-dilks c-dilks merged commit df90e9d into main Feb 19, 2024
55 checks passed
@c-dilks c-dilks deleted the hipo4-static branch February 19, 2024 22:45
@c-dilks c-dilks mentioned this pull request Mar 13, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

hipo4 libs are in test executables' rpath; consider instead linking against hipo4 static libs
1 participant
@c-dilks