Skip to content

enable sqlite3 support#1

Open
hanwenguo wants to merge 1 commit intoRadioNoiseE:mainfrom
hanwenguo:sqlite3
Open

enable sqlite3 support#1
hanwenguo wants to merge 1 commit intoRadioNoiseE:mainfrom
hanwenguo:sqlite3

Conversation

@hanwenguo
Copy link

No description provided.

@RadioNoiseE
Copy link
Owner

Hi,

Could you outline some common use cases for SQLite support in Emacs, and give an idea of how much it would increase the size of the Emacs binary?

Thanks for your contribution!

@hanwenguo
Copy link
Author

Hi,

Thanks for replying. Some packages (e.g. forge and org-roam uses sqlite3, and enabling the built-in support can improve the performance and save the user from installing extra packages. As for binary size, the increase is hardly noticable (my build is ~400KB larger than yours), so it's a nice-to-have one.

@RadioNoiseE
Copy link
Owner

Sorry for the late reply — I was busy with my monthly exams.

I really appreciate your detailed reasoning, however I still feel that adding SQLite support isn't essential. This repo is meant to serve as a template demonstrating how to build Emacs with static linking, and users are encouraged to fork it and customize it however they like (e.g., building on macOS 26, using the master branch, and so on).

Thanks again for your contribution!

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