doomemacs / doomemacs

An Emacs framework for the stubborn martian hacker
MIT License
19.51k stars 3.06k forks source link

`+org-roam-try-init-db-a` seems to break org-roam #8167

Open abgruszecki opened 3 hours ago

abgruszecki commented 3 hours ago

I confirm that...

Expected behavior

org-roam-node-find works.

Current behavior

When I call org-roam-node-find, I instead get an error message that emacsql-sqlite-ensure-binary is not a function. That happens because of this advice: https://github.com/doomemacs/doomemacs/blob/7bc39f2c1402794e76ea10b781dfe586fed7253b/modules/lang/org/contrib/roam2.el#L53-L71.

I recently upgraded to the latest version of Doom, this issue did not appear before for whatever strange reason. Disabling the advice makes org-roam work normally. The advice is strange since it uses/mentions deprecated functions, like emacsql-sqlite-ensure-binary (it seems deprecated?) and org-roam-teardown and org-roam-db-sync and org-roam-setup.

Steps to reproduce

Call org-roam-node-find .

System Information

https://gist.github.com/abgruszecki/36dd4496f9cc5c9d577600eda50caef0

ajensen1234 commented 2 hours ago

I'm getting exactly the same error.