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

koga enhancement: Allow #'library function to have a path spec. #1642

Closed
wants to merge 1 commit into from

Conversation

dg1sbg
Copy link
Contributor

@dg1sbg dg1sbg commented Oct 19, 2024

Expanded koga'a #'library function (actually, #'configure-library) to support the following way of specifying a libary:

(k:library "nddscpp2d" :path #p"/Applications/rti_connext_dds-7.3.0/lib/arm64Darwin20clang12.0/" :kind :dynamic)

The path will be added as -L_path_ to the linker flags.

@yitzchak
Copy link
Member

yitzchak commented Oct 19, 2024

Clasp currently doesn't need non-pkgconfig support. Some user's have built extensions that needed to use libraries which don't have pkgconfig support. In that case directly adding the appropriate ldflags is probably the easiest route. If I can finish up dynamic extension support then this will be moot since extensions won't need to be built directly into Clasp, i.e. extensions will just link against libclasp.so.

In other words, I'd prefer to just expose the needed ldflags functions if we must in the short term.

@dg1sbg dg1sbg closed this Oct 24, 2024
@dg1sbg
Copy link
Contributor Author

dg1sbg commented Oct 24, 2024

As @yitzchak pointed out, there will be another approach to handle pkconfig-less external libraries.

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.

None yet

2 participants