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

ocb-stubblr #27

Closed
wants to merge 1 commit into from
Closed

ocb-stubblr #27

wants to merge 1 commit into from

Conversation

pqwy
Copy link
Contributor

@pqwy pqwy commented Oct 24, 2016

Showcases how to get rid of ocamlbuild boilerplate.

Can't be merged until https://github.com/pqwy/ocb-stubblr is released.

-Wl,-no-as-needed is left hanging in the air. So far, I cannot recreate the failure.

@dbuenzli
Copy link
Owner

@pqwy There's an explanation about -Wl,-no-as-needed on this page shouldn't you add this support directly into ocb-stubblr ?

@pqwy
Copy link
Contributor Author

pqwy commented Nov 3, 2016

@dbuenzli I was thinking about that. I'm not sure.

  • It covers a fairly specific use-case, where you want to link against a library, but only ever dlsym anything from it.
  • It can be done with relative ease: definition, use.
  • I still can't get my hands on a Linux distribution where this is needed. It isn't on any Linux I have access to.

Speaking of specific use-case, this particular PR showcases ocb-stubblr, but why not just move linking completely to runtime? With that, tsdl doesn't need its stubs, and doesn't need the stubblr either.

@dbuenzli
Copy link
Owner

dbuenzli commented Dec 3, 2016

So your patch is in as 46ffd02. Thanks.

@dbuenzli dbuenzli closed this Dec 3, 2016
@dbuenzli dbuenzli mentioned this pull request Jul 8, 2017
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