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

2026-08-11 Thread Nicholas D Steeves
Control: severity -1 serious
Justification: Policy §3.5 violation (missing required dependency)

Xiyue Deng  writes:

> Note that currently trixie and sid/forky share the same version of
> elpa-org-roam (2.3.1-1). You cannot upload a higher version in stable
> (yet) because there is no version available in between and hence there
> is no upgrade path (and your upload to stable-proposed-updates will
> likely be rejected with the same reason).

+deb13u1 is the "in between" version...the upgrade path is 2.3.1-1,
2.3.1-1+deb13u1, (>> 2.3.1-1+deb13u1).  Point release manager will
install +deb13u1 to testing and unstable when it is installed to
stable-updates, when that is the most correct action.

> I think you'll need to first upload a no-op 2.3.1-2 and wait for it to
> enter Forky, and then prepare 2.3.1-1+deb13u1 for stable so that an
> upgrade path is available.

No, when dependencies are wrong they also need to be fixed in sid and
testing, so I have fixed it there too.

> P.S. For stable updates the bug it fixes also needs to be important or
> above. I have bumped the severity to important here as well.

This is actually an RC bug that would have been caught if the
Uploader[s] of this package installed and used the package.

Cheers,
Nicholas


signature.asc
Description: PGP signature


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

2026-08-10 Thread Xiyue Deng
Control: severity -1 important
Justification: breaks functionality

Nicholas D Steeves  writes:

> Control: tag -1 trixie
>
> Hi,
>
> David Miguel Susano Pinto  writes:
>
>> 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
>
> [snip]
>
>> I suggest that elpa-emacsql-sqlite be added as a requirement for
>> elpa-org-roam
>
> Agreed
>
>> (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).
>
> Elpa-emacsql-sqlite is an empty dummy package in sid/unstable and this
> bug this only affects trixie.  I think this bug meets the criteria for a
> stable update because the regression risk is zero.
>
> Give me a few minutes to make the upload and file the unblock bug, but
> please ping me if you don't see an unblock bug in a week.
>

Note that currently trixie and sid/forky share the same version of
elpa-org-roam (2.3.1-1). You cannot upload a higher version in stable
(yet) because there is no version available in between and hence there
is no upgrade path (and your upload to stable-proposed-updates will
likely be rejected with the same reason).

I think you'll need to first upload a no-op 2.3.1-2 and wait for it to
enter Forky, and then prepare 2.3.1-1+deb13u1 for stable so that an
upgrade path is available.

P.S. For stable updates the bug it fixes also needs to be important or
above. I have bumped the severity to important here as well.

> Cheers,
> Nicholas
>

-- 
Regards,
Xiyue Deng


signature.asc
Description: PGP signature


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

2026-08-10 Thread Nicholas D Steeves
Control: tag -1 trixie

Hi,

David Miguel Susano Pinto  writes:

> 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

[snip]

> I suggest that elpa-emacsql-sqlite be added as a requirement for
> elpa-org-roam

Agreed

> (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).

Elpa-emacsql-sqlite is an empty dummy package in sid/unstable and this
bug this only affects trixie.  I think this bug meets the criteria for a
stable update because the regression risk is zero.

Give me a few minutes to make the upload and file the unblock bug, but
please ping me if you don't see an unblock bug in a week.

Cheers,
Nicholas


signature.asc
Description: PGP signature


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

2026-08-10 Thread David Miguel Susano Pinto
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-dash2.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  

-- no debconf information