Skip to content

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

Merged
gilzoide merged 2 commits into
mainfrom
unknown repository
May 13, 2025
Merged

Adjusted Makefile to prevent SQLite calls hitting the system libs which prevents Unity editor crashes in Linux.#70
gilzoide merged 2 commits into
mainfrom
unknown repository

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented May 13, 2025

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

@ghost ghost mentioned this pull request May 13, 2025
@gilzoide gilzoide linked an issue May 13, 2025 that may be closed by this pull request
@gilzoide
Copy link
Copy Markdown
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?

@ghost
Copy link
Copy Markdown
Author

ghost commented May 13, 2025

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
Copy Markdown
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
Copy Markdown

Excellent. Thank you for looking into this!

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.

Unity 6 Editor crash on play in Linux

3 participants