Bug#1144053: elpa-org-roam: org-roam requires emacsql sqlite backend

Package: elpa-org-roam
Version: 2.3.1-1
Severity: normal
X-Debbugs-Cc: [email protected]

Dear Maintainer,

The packaged version of org-roam has a requirement on emacsql-sqlite
(see /usr/share/emacs/site-lisp/elpa-src/org-roam-2.3.1/org-roam.el
line 86) or in salsa:

    https://salsa.debian.org/emacsen-team/org-roam/-/blob/v2.3.1/org-roam.el?ref_type=tags#L86

However, that is not currently listed as package requirement.  There
is a requirement on elpa-emacsql which provides only the high level
abstractions but not on the sqlite backend.  Without elpa-emacsql-sqlite
installed, starting org-roam in emacs leads to the following error:

    File is missing: Cannot open load file, No such file or directory, emacsql-sqlite

    To ensure normal operation, you should investigate and remove the
    cause of the error in your initialization file.  Start Emacs with
    the ‘--debug-init’ option to view a complete error backtrace.

and the full backtrace from `--debug-init` is:

    Debugger entered--Lisp error: (file-missing "Cannot open load file" "No such file or directory" "emacsql-sqlite")
      require(emacsql-sqlite)
      load-with-code-conversion("/usr/share/emacs/site-lisp/elpa/org-roam-2.3.1/org-roam.el" "/usr/share/emacs/site-lisp/elpa/org-roam-2.3.1/org-roam.el" nil t)
      require(org-roam)

I suggest that elpa-emacsql-sqlite be added as a requirement for
elpa-org-roam (however, I think that things have changed for the
latest version of org-roam so this may apply only to the version
currently on Debian stable).

Best wishes
David


-- System Information:
Debian Release: 13.6
  APT prefers stable-updates
  APT policy: (500, 'stable-updates'), (500, 'stable-security'), (500, 'stable')
Architecture: amd64 (x86_64)

Kernel: Linux 6.12.101+deb13-amd64 (SMP w/8 CPU threads; PREEMPT)
Locale: LANG=en_GB.UTF-8, LC_CTYPE=en_GB.UTF-8 (charmap=UTF-8), LANGUAGE=en_GB:en
Shell: /bin/sh linked to /usr/bin/dash
Init: systemd (via /run/systemd/system)
LSM: AppArmor: enabled

Versions of packages elpa-org-roam depends on:
ii  dh-elpa-helper           2.1.9
ii  elpa-dash                2.20.0+dfsg-1
ii  elpa-emacsql             4.3.0-1
ii  elpa-magit-section       4.3.5-1+deb13u1
ii  emacs-common [elpa-org]  1:30.1+1-6
ii  emacsen-common           3.0.8

Versions of packages elpa-org-roam recommends:
ii  emacs              1:30.1+1-6
ii  emacs-gtk [emacs]  1:30.1+1-6

Versions of packages elpa-org-roam suggests:
pn  org-roam-doc  <none>

-- no debconf information