-
Notifications
You must be signed in to change notification settings - Fork 203
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
SQLite3 symbols are no longer exported #377
Comments
Sorry, I'm not sure I understand your use case. Can you say more about what you're doing and why? We may not be able to support your use case with the precompiled extensions, but I need to know more to know for sure, and I would like to try to find a way to help you. |
Yes, apologies for not mentioning it in the changelog, I should have. Please read rake-compiler/rake-compiler-dock#87 for the painful details surrounding this decision. There are also other issues linked to from there with more context. The situation with the Ruby 3.2 linker flags makes it hard to support exporting these symbols in the precompiled extension. Maybe there's another way we can support your use case, or make these symbols available when you use system libraries (see your other issue). |
Also see sparklemotion/nokogiri#2746 for a proposal to have Nokogiri behave the same way, for the same reasons. |
And #371 for the relevant change in this project. |
@flavorjones I am doing: LD_PRELOAD=libsqlc.so ruby example.rb where You can see a full example here: https://github.com/libsql/sqld/tree/main/examples/ruby |
@flavorjones Ok, makes sense, so I should work towards figuring out why #376 doesn't work. |
🤯 That's a really interesting use case. Thanks for sharing and for opening this issue. I'll jump back into #376 with you. |
I have been using
LD_PRELOAD
to take over the SQLite API. This works fine in 1.5.3 because the sqlite symbols are exported:However, starting from 1.6.0, they're no longer there:
Is that on purpose?
(Btw, I also tried to explicitly enable system libraries, but can't seem to figure out how to make it work. See #376 for more details.)
The text was updated successfully, but these errors were encountered: