Skip to content

for static build export kissfft as a private lib#137

Closed
oe1rsa wants to merge 1 commit into
myriadrf:developfrom
oe1rsa:issue/131
Closed

for static build export kissfft as a private lib#137
oe1rsa wants to merge 1 commit into
myriadrf:developfrom
oe1rsa:issue/131

Conversation

@oe1rsa

@oe1rsa oe1rsa commented Jun 14, 2025

Copy link
Copy Markdown
Contributor

When statically linking to liblimesuiteng one
also needs to link against the kissft lib.
Since the user may or may not have installed
kissfft as a static lib, we provide a private
version which is linked as liblimesuiteng-kissfft-float.a when linking against liblimesuiteng.
When the user also has installed the upstream
kissfft she may link against this version, but
then the results become dependent on where the
both libraries appear on the linker line.
Usually the first wins. So take care!

This pull request will resolve issue #131

When statically linking to liblimesuiteng one
also needs to link against the kissft lib.
Since the user may or may not have installed
kissfft as a static lib, we provide a private
version which is linked as liblimesuiteng-kissfft-float.a
when linking against liblimesuiteng.
When the user also has installed the upstream
kissfft she may link against this version, but
then the results become dependent on where the
both libraries appear on the linker line.
Usually the first wins. So take care!
@rjonaitis

Copy link
Copy Markdown
Member

dependency on kissfft.a has been removed by integrating it's source into limesuiteng.a b742fc6

@rjonaitis rjonaitis closed this Jun 14, 2025
@oe1rsa oe1rsa deleted the issue/131 branch June 15, 2025 09:05
@oe1rsa

oe1rsa commented Jun 15, 2025

Copy link
Copy Markdown
Contributor Author

Thank you! Works for me.
Btw.: I like your solution, you took the harder route, however.

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