Skip to content

Adjusted Makefile to prevent SQLite calls hitting the system libs which prevents Unity editor crashes in Linux. #70

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 2 commits into from
May 13, 2025

Conversation

alansley
Copy link
Contributor

Tested and working with the Makefile adjustments on latest Arch Linux circa 2025/05/13 using Unity 6000.1.2f1.

@gilzoide gilzoide linked an issue May 13, 2025 that may be closed by this pull request
@gilzoide
Copy link
Owner

Hey @alansley, thank you very much for taking the time to look into the crash and contributing a fix!

I'm not really sure we need to add an RPATH to this library. Its only dependencies are libm.so.6 and libc.so.6 (you can find that out with objdump -p libgilzoide-sqlite-net.so), so my understanding is that it would make absolutely no difference unless there's a libm.so.6 and/or libc.so.6 in the same folder as it in exported projects, which I'm quite sure there isn't 🤔
Have you tried a build without it?

@alansley
Copy link
Contributor Author

Hi @gilzoide - you're welcome!

And you're completely right about not needing the RPATH - tested and working without it.

As such,, I've updated my PR so that it only adds the -Wl,-Bsymbolic linker flag & I also re-linked libgilzoide-sqlite-net.so from the existing sqlite3.o~. All working - good as gold.

Thanks very much for your library! =D

@gilzoide
Copy link
Owner

Nice, thank you very much! I'll just let GitHub Actions build it again, merge afterwards and release a new bugfix version.

@gilzoide gilzoide merged commit f37dadb into gilzoide:main May 13, 2025
4 checks passed
@Spamming-Spammer
Copy link

Excellent. Thank you for looking into this!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unity 6 Editor crash on play in Linux
3 participants