emacsql-sqlite-builtin

emacsql-sqlite-builtin

[Gregor Kling]

[-- Attachment #1: Type: text/plain, Size: 880 bytes --]

Hello,

I am using emacs-29 (compiled today) which works fine with the exception 
of org-roam.

Using the recommended (org-roam-database-connector 'sqlite-builtin), I 
get the following error:

Debugger entered--Lisp error: (invalid-slot-type 
emacsql-sqlite-builtin-connection process process #<sqlite 
db=0x561fe8f964b8 name=orgroam.db>)
   signal(invalid-slot-type (emacsql-sqlite-builtin-connection process 
process

Now my knowledge of the inner workings are lets say in the beginnings 
respective to emacs lisp.

But it seems this might not be a problem with org-roam per se, but with 
the new sqlite embed.

Can anyone approve?

greetings
gregor
-- 
Gregor Kling

Abteilung ITS, Sachgebiet ITS-N
Anwendungsentwicklung (IPAM, Tooling, Automation)
DNS, DHCP
Technische Hochschule Mittelhessen
University of Applied Sciences
Tel: 0641/309-1292
E-Mail: gregor.kling@its.thm.de

[-- Attachment #2: S/MIME Cryptographic Signature --]
[-- Type: application/pkcs7-signature, Size: 4735 bytes --]

Re: emacsql-sqlite-builtin

[Eli Zaretskii]

> Date: Tue, 31 Jan 2023 12:10:35 +0100
> From: Gregor Kling <gregor.kling@its.thm.de>
> 
> I am using emacs-29 (compiled today) which works fine with the exception 
> of org-roam.
> 
> Using the recommended (org-roam-database-connector 'sqlite-builtin), I 
> get the following error:
> 
> Debugger entered--Lisp error: (invalid-slot-type 
> emacsql-sqlite-builtin-connection process process #<sqlite 
> db=0x561fe8f964b8 name=orgroam.db>)
>    signal(invalid-slot-type (emacsql-sqlite-builtin-connection process 
> process
> 
> Now my knowledge of the inner workings are lets say in the beginnings 
> respective to emacs lisp.
> 
> But it seems this might not be a problem with org-roam per se, but with 
> the new sqlite embed.
> 
> Can anyone approve?

I suggest to post this first to the Org forum.

Re: emacsql-sqlite-builtin

[Tim Cross]

Eli Zaretskii <eliz@gnu.org> writes:

>> Date: Tue, 31 Jan 2023 12:10:35 +0100
>> From: Gregor Kling <gregor.kling@its.thm.de>
>> 
>> I am using emacs-29 (compiled today) which works fine with the exception 
>> of org-roam.
>> 
>> Using the recommended (org-roam-database-connector 'sqlite-builtin), I 
>> get the following error:
>> 
>> Debugger entered--Lisp error: (invalid-slot-type 
>> emacsql-sqlite-builtin-connection process process #<sqlite 
>> db=0x561fe8f964b8 name=orgroam.db>)
>>    signal(invalid-slot-type (emacsql-sqlite-builtin-connection process 
>> process
>> 
>> Now my knowledge of the inner workings are lets say in the beginnings 
>> respective to emacs lisp.
>> 
>> But it seems this might not be a problem with org-roam per se, but with 
>> the new sqlite embed.
>> 
>> Can anyone approve?
>
> I suggest to post this first to the Org forum.


Note that org-roam is not part of org. You probably need an org-roam
specific forum as org-roam development is not (typically) discussed o
org-devel.

Re: emacsql-sqlite-builtin

[Richard Kim]

Tim Cross <theophilusx@gmail.com> writes:

> Eli Zaretskii <eliz@gnu.org> writes:
>
>>> Date: Tue, 31 Jan 2023 12:10:35 +0100
>>> From: Gregor Kling <gregor.kling@its.thm.de>
>>>
>>> I am using emacs-29 (compiled today) which works fine with the exception
>>> of org-roam.
>>>
>>> Using the recommended (org-roam-database-connector 'sqlite-builtin), I
>>> get the following error:
>>>
>>> Debugger entered--Lisp error: (invalid-slot-type
>>> emacsql-sqlite-builtin-connection process process #<sqlite
>>> db=0x561fe8f964b8 name=orgroam.db>)
>>>    signal(invalid-slot-type (emacsql-sqlite-builtin-connection process
>>> process
>>>
>>> Now my knowledge of the inner workings are lets say in the beginnings
>>> respective to emacs lisp.
>>>
>>> But it seems this might not be a problem with org-roam per se, but with
>>> the new sqlite embed.
>>>
>>> Can anyone approve?
>>
>> I suggest to post this first to the Org forum.
>
>
> Note that org-roam is not part of org. You probably need an org-roam
> specific forum as org-roam development is not (typically) discussed o
> org-devel.

org-roam works fine for me on emacs 29 built 3 days ago on Ubuntu 22.04.
I've been using 'sqlite for years and I tried 'sqlite-builtin today for
the first time.  It worked without any hitch once I installed
emacsql-sqlite-builtin package.

Re: emacsql-sqlite-builtin

[c.buhtz]

FYI I opened a related question at OrgRoam discourse

https://org-roam.discourse.group/t/sqlite-suport-is-not-available-with-emacs-29-on-windows/2991


On 2023-01-31 22:00 Richard Kim <emacs18@gmail.com> wrote:
> Tim Cross <theophilusx@gmail.com> writes:
> 
> > Eli Zaretskii <eliz@gnu.org> writes:
> >
> >>> Date: Tue, 31 Jan 2023 12:10:35 +0100
> >>> From: Gregor Kling <gregor.kling@its.thm.de>
> >>>
> >>> I am using emacs-29 (compiled today) which works fine with the
> >>> exception of org-roam.
> >>>
> >>> Using the recommended (org-roam-database-connector
> >>> 'sqlite-builtin), I get the following error:
> >>>
> >>> Debugger entered--Lisp error: (invalid-slot-type
> >>> emacsql-sqlite-builtin-connection process process #<sqlite
> >>> db=0x561fe8f964b8 name=orgroam.db>)
> >>>    signal(invalid-slot-type (emacsql-sqlite-builtin-connection
> >>> process process
> >>>
> >>> Now my knowledge of the inner workings are lets say in the
> >>> beginnings respective to emacs lisp.
> >>>
> >>> But it seems this might not be a problem with org-roam per se,
> >>> but with the new sqlite embed.
> >>>
> >>> Can anyone approve?
> >>
> >> I suggest to post this first to the Org forum.
> >
> >
> > Note that org-roam is not part of org. You probably need an org-roam
> > specific forum as org-roam development is not (typically) discussed
> > o org-devel.
> 
> org-roam works fine for me on emacs 29 built 3 days ago on Ubuntu
> 22.04. I've been using 'sqlite for years and I tried 'sqlite-builtin
> today for the first time.  It worked without any hitch once I
> installed emacsql-sqlite-builtin package.
> 
> 

Re: emacsql-sqlite-builtin

[Gregor Kling]

[-- Attachment #1: Type: text/plain, Size: 1332 bytes --]

On 31.01.23 22:04, Tim Cross wrote:
> 
> Eli Zaretskii <eliz@gnu.org> writes:
> 
>>> Date: Tue, 31 Jan 2023 12:10:35 +0100
>>> From: Gregor Kling <gregor.kling@its.thm.de>

>>> Using the recommended (org-roam-database-connector 'sqlite-builtin), I
>>> get the following error:
>>>
>>> Debugger entered--Lisp error: (invalid-slot-type
>>> emacsql-sqlite-builtin-connection process process #<sqlite
>>> db=0x561fe8f964b8 name=orgroam.db>)
>>>     signal(invalid-slot-type (emacsql-sqlite-builtin-connection process
>>> process

>> I suggest to post this first to the Org forum.
> 
> 
> Note that org-roam is not part of org. You probably need an org-roam
> specific forum as org-roam development is not (typically) discussed o
> org-devel.

The use of org-roam reveals only the symptom.

I took it out from the equation and tried the following:

(emacsql-sqlite (file-truename "~/a.db"))

which *Messages* me a "Selecting deleted buffer"

whereas using (emacsql-sqlite-builtin (file-truename "~/a.db"))

it spits out:

eieio--validate-slot-value: Invalid slot type: 
emacsql-sqlite-builtin-connection...


-- 
Gregor Kling

Abteilung ITS, Sachgebiet ITS-N
Anwendungsentwicklung (IPAM, Tooling, Automation)
DNS, DHCP
Technische Hochschule Mittelhessen
University of Applied Sciences
Tel: 0641/309-1292
E-Mail: gregor.kling@its.thm.de


[-- Attachment #2: S/MIME Cryptographic Signature --]
[-- Type: application/pkcs7-signature, Size: 4735 bytes --]

Re: emacsql-sqlite-builtin

[Basil L. Contovounesios]

Gregor Kling [2023-02-01 09:00 +0100] wrote:

> The use of org-roam reveals only the symptom.
>
> I took it out from the equation and tried the following:
>
> (emacsql-sqlite (file-truename "~/a.db"))
>
> which *Messages* me a "Selecting deleted buffer"

This sounds like a recently fixed issue[1].

[1] https://github.com/magit/emacsql/issues/106

> whereas using (emacsql-sqlite-builtin (file-truename "~/a.db"))
>
> it spits out:
>
> eieio--validate-slot-value: Invalid slot type:
> emacsql-sqlite-builtin-connection...

Have you tried updating/reinstalling your EmacSQL packages?
What versions do you have installed?

Your examples evaluate for me without error on Emacs 29 and 30, with
emacsql, emacsql-sqlite, and emacs-sqlite-builtin version
3.1.1.50snapshot0.20230120.213337 installed from NonGNU-devel ELPA.

If they still don't work for you after updating/reinstalling, I suggest
reporting the issue on the EmacSQL issue tracker[2].

[2] https://github.com/magit/emacsql/issues

---

In case it's relevant: the EmacSQL packages are distributed by the Emacs
project only on NonGNU-devel ELPA[3][4][5], not NonGNU ELPA[6] which is
what the user option package-archives points to by default.

[3] https://elpa.nongnu.org/nongnu-devel/emacsql.html
[4] https://elpa.nongnu.org/nongnu-devel/emacsql-sqlite.html
[5] https://elpa.nongnu.org/nongnu-devel/emacsql-sqlite-builtin.html
[6] https://elpa.nongnu.org/nongnu/

The EmacSQL packages are also distributed by MELPA[7][8][9], and have
been for longer.

[7] https://melpa.org/#/emacsql
[8] https://melpa.org/#/emacsql-sqlite
[9] https://melpa.org/#/emacsql-sqlite-builtin

However, emacsql-sqlite-builtin doesn't seem to be available on MELPA
Stable[10].

[10] https://stable.melpa.org

It seems like there were some issues with updating the packages from
MELPA a few months ago[11], but I presume those have been fixed now.

[11] https://github.com/magit/emacsql/issues/103

HTH,

-- 
Basil

Re: emacsql-sqlite-builtin

[Gregor Kling]

[-- Attachment #1: Type: text/plain, Size: 2753 bytes --]

On 01.02.23 12:10, Basil L. Contovounesios wrote:
> Gregor Kling [2023-02-01 09:00 +0100] wrote:
> 
>> The use of org-roam reveals only the symptom.
>>

> Have you tried updating/reinstalling your EmacSQL packages?

Yes.

> What versions do you have installed?

  Status: Installed in ‘emacsql-3.1.1/’ (unsigned).
     Version: 3.1.1


> Your examples evaluate for me without error on Emacs 29 and 30, with
> emacsql, emacsql-sqlite, and emacs-sqlite-builtin version
> 3.1.1.50snapshot0.20230120.213337 installed from NonGNU-devel ELPA.
Package emacsql-sqlite-builtin is installed.

      Status: Installed in ‘emacsql-sqlite-builtin-20221127.2146/’ 
(unsigned).
     Version: 20221127.2146

There is no functional difference between 
3.1.1.50snapshot0.20230120.213337 and emacsql-sqlite-builtin-20221127.2146:

diff emacsql-sqlite-builtin.el 
~/.emacs.d/elpa/emacsql-sqlite-builtin-20221127.2146/emacsql-sqlite-builtin.el
8c8,9
< ;; Package-Version: 3.1.1.50-git
---
> ;; Package-Version: 20221127.2146
> ;; Package-X-Original-Version: 3.1.1.50-git
9a11
> ;; Package-Commit: 6b2e65bdf785364cf7c34c31fea5812e1e58c657


> If they still don't work for you after updating/reinstalling, I suggest
> reporting the issue on the EmacSQL issue tracker[2].
> 
> [2] https://github.com/magit/emacsql/issues
> 

The fact that it is working for you, means there must
be some issue with my setup.

Before going down the road of further reporting, it seems to me i have 
to dig in front of my own door..

Thanks for the pointers.

> In case it's relevant: the EmacSQL packages are distributed by the Emacs
> project only on NonGNU-devel ELPA[3][4][5], not NonGNU ELPA[6] which is
> what the user option package-archives points to by default.
> 
> [3] https://elpa.nongnu.org/nongnu-devel/emacsql.html
> [4] https://elpa.nongnu.org/nongnu-devel/emacsql-sqlite.html
> [5] https://elpa.nongnu.org/nongnu-devel/emacsql-sqlite-builtin.html
> [6] https://elpa.nongnu.org/nongnu/
> 
> The EmacSQL packages are also distributed by MELPA[7][8][9], and have
> been for longer.
> 
> [7] https://melpa.org/#/emacsql
> [8] https://melpa.org/#/emacsql-sqlite
> [9] https://melpa.org/#/emacsql-sqlite-builtin
> 
> However, emacsql-sqlite-builtin doesn't seem to be available on MELPA
> Stable[10].
> 
> [10] https://stable.melpa.org
> 
> It seems like there were some issues with updating the packages from
> MELPA a few months ago[11], but I presume those have been fixed now.
> 
> [11] https://github.com/magit/emacsql/issues/103
> 
> HTH,
> 

-- 
Gregor Kling

Abteilung ITS, Sachgebiet ITS-N
Anwendungsentwicklung (IPAM, Tooling, Automation)
DNS, DHCP
Technische Hochschule Mittelhessen
University of Applied Sciences
Tel: 0641/309-1292
E-Mail: gregor.kling@its.thm.de


[-- Attachment #2: S/MIME Cryptographic Signature --]
[-- Type: application/pkcs7-signature, Size: 4735 bytes --]

Re: emacsql-sqlite-builtin

[Basil L. Contovounesios]

Gregor Kling [2023-02-01 13:21 +0100] wrote:

> On 01.02.23 12:10, Basil L. Contovounesios wrote:
>
>> What versions do you have installed?
>
>  Status: Installed in ‘emacsql-3.1.1/’ (unsigned).

This looks like the version from MELPA Stable.

>  Status: Installed in ‘emacsql-sqlite-builtin-20221127.2146/’ (unsigned).

And this like the version from MELPA.

This seems like a likely culprit, because the emacsql-connection class
changed between these versions.

What is your value of package-archives?

I suggest making sure all your EmacSQL packages are installed from the
same archive:

  either https://elpa.nongnu.org/nongnu-devel/
      or https://melpa.org/packages/

This should ensure that their versions are mutually consistent.

If different versions of the same package exist in different archives,
you can tell Emacs precisely where to get different packages from by
customising the user options package-archive-priorities and
package-pinned-packages.

HTH,

-- 
Basil

Re: emacsql-sqlite-builtin

[Richard Kim]

"Basil L. Contovounesios" <contovob@tcd.ie> writes:

> Gregor Kling [2023-02-01 13:21 +0100] wrote:
>
>> On 01.02.23 12:10, Basil L. Contovounesios wrote:
>>
>>> What versions do you have installed?
>>
>>  Status: Installed in ‘emacsql-3.1.1/’ (unsigned).
>
> This looks like the version from MELPA Stable.
>
>>  Status: Installed in ‘emacsql-sqlite-builtin-20221127.2146/’ (unsigned).
>
> And this like the version from MELPA.
>
> This seems like a likely culprit, because the emacsql-connection class
> changed between these versions.
>
> What is your value of package-archives?
>
> I suggest making sure all your EmacSQL packages are installed from the
> same archive:
>
>   either https://elpa.nongnu.org/nongnu-devel/
>       or https://melpa.org/packages/
>
> This should ensure that their versions are mutually consistent.
>
> If different versions of the same package exist in different archives,
> you can tell Emacs precisely where to get different packages from by
> customising the user options package-archive-priorities and
> package-pinned-packages.
>
> HTH,

You need to pick up Jan 18 fix in emacsql, i.e.,
https://github.com/magit/emacsql/commit/2e6056df110e76da4eabb413f8781c4f1edddf20
which fixed the "deleted buffer" problem initially reported against
emacs as https://debbugs.gnu.org/cgi/bugreport.cgi?bug=60872

Alternately you can build emacs using a version prior to cfd2b3504ab
which was checked in on Jan 12.  If you do, then you can stay with older
emacsql.

Re: emacsql-sqlite-builtin

[Gregor Kling]

[-- Attachment #1: Type: text/plain, Size: 1915 bytes --]

On 01.02.23 16:35, Basil L. Contovounesios wrote:
> Gregor Kling [2023-02-01 13:21 +0100] wrote:
> 
>> On 01.02.23 12:10, Basil L. Contovounesios wrote:
>>
>>> What versions do you have installed?
>>
>>   Status: Installed in ‘emacsql-3.1.1/’ (unsigned).
> 
> This looks like the version from MELPA Stable.
> 
>>   Status: Installed in ‘emacsql-sqlite-builtin-20221127.2146/’ (unsigned).
> 
> And this like the version from MELPA.
> 
> This seems like a likely culprit, because the emacsql-connection class
> changed between these versions.
> 
> What is your value of package-archives?
> 
> I suggest making sure all your EmacSQL packages are installed from the
> same archive:
> 
>    either https://elpa.nongnu.org/nongnu-devel/
>        or https://melpa.org/packages/
> 
> This should ensure that their versions are mutually consistent.
>  > If different versions of the same package exist in different archives,
> you can tell Emacs precisely where to get different packages from by
> customising the user options package-archive-priorities and
> package-pinned-packages.

In the aftermath this would have been better placed in
help-gnu-emacs because of lack of knowledge.

Nevertheless, thanks Basil, for breaking the tie.

(setq package-archives
       '(("GNU ELPA"     . "https://elpa.gnu.org/packages/")
         ("nongnu"       . "https://elpa.nongnu.org/nongnu/")
         ("nongnu-devel" . "https://elpa.nongnu.org/nongnu-devel/"))
       package-archive-priorities
       '(("GNU ELPA"     . 10)
         ("nongnu"       . 8)
         ("nongnu-devel" . 6))
(setq package-pinned-packages
       '(("emacsql-sqlite-builtin" . "nongnu-devel")))

has solved the issue.

-- 
Gregor Kling

Abteilung ITS, Sachgebiet ITS-N
Anwendungsentwicklung (IPAM, Tooling, Automation)
DNS, DHCP
Technische Hochschule Mittelhessen
University of Applied Sciences
Tel: 0641/309-1292
E-Mail: gregor.kling@its.thm.de


[-- Attachment #2: S/MIME Cryptographic Signature --]
[-- Type: application/pkcs7-signature, Size: 4735 bytes --]

Re: emacsql-sqlite-builtin

[Eli Zaretskii]

> Date: Wed, 1 Feb 2023 09:00:55 +0100
> Cc: emacs-devel@gnu.org
> From: Gregor Kling <gregor.kling@its.thm.de>
> 
> The use of org-roam reveals only the symptom.
> 
> I took it out from the equation and tried the following:
> 
> (emacsql-sqlite (file-truename "~/a.db"))
> 
> which *Messages* me a "Selecting deleted buffer"
> 
> whereas using (emacsql-sqlite-builtin (file-truename "~/a.db"))
> 
> it spits out:
> 
> eieio--validate-slot-value: Invalid slot type: 
> emacsql-sqlite-builtin-connection...

So the problem is in emacsql?  That package is not in Emacs and not on
ELPA, is it?  If so, I suggest to report this to the developers of
emacsql.