GNU bug report logs - #54581
Add emacs-sqlite3-api package

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 54581 in the body.
You can then email your comments to 54581 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Roman Scherer <roman.scherer <at> burningswell.com>
To: guix-patches <at> gnu.org
Subject: Add emacs-sqlite3-api package
Date: Sat, 26 Mar 2022 15:09:46 +0100

[Message part 1 (text/plain, inline)]

Hello Guix,

this patch adds the emacs-sqlite3-api package to Guix. It is a dynamic
module for GNU Emacs 25+ that provides direct access to the core SQLite3
C API from Emacs Lisp.

Could you please review it?

Thanks, Roman.

[0001-Add-emacs-sqlite3-api-package.patch (text/x-patch, attachment)]

[signature.asc (application/pgp-signature, inline)]

Message #8 received at 54581 <at> debbugs.gnu.org (full text, mbox):

From: Maxime Devos <maximedevos <at> telenet.be>
To: Roman Scherer <roman.scherer <at> burningswell.com>, 54581 <at> debbugs.gnu.org
Subject: Re: [bug#54581] Add emacs-sqlite3-api package
Date: Sat, 26 Mar 2022 22:17:57 +0100

[Message part 1 (text/plain, inline)]

Roman Scherer schreef op za 26-03-2022 om 15:09 [+0100]:
> +               ;; Compile the shared object file.
> +               (apply invoke "make" "CC=gcc" make-flags)

This is broken when cross-compiling.  Use (string-append "CC=" ,(cc-
for-target)) instead of "CC=gcc".

Greetings,
Maxime.

[signature.asc (application/pgp-signature, inline)]

Message #11 received at 54581 <at> debbugs.gnu.org (full text, mbox):

From: Maxime Devos <maximedevos <at> telenet.be>
To: Roman Scherer <roman.scherer <at> burningswell.com>, 54581 <at> debbugs.gnu.org
Subject: Re: [bug#54581] Add emacs-sqlite3-api package
Date: Sat, 26 Mar 2022 22:20:44 +0100

[Message part 1 (text/plain, inline)]

Roman Scherer schreef op za 26-03-2022 om 15:09 [+0100]:
> +      (description "SQLite3 is a dynamic module for GNU Emacs 25+
> that provides
> +direct access to the core SQLite3 C API from Emacs Lisp.")

While this package integrates SQLite3 into Emacs, SQLite itself is not
part of Emacs, so I would reformulate this a bit.  The description is
also a bit short, ‘(guix)Synopses and Descriptions’ recommends longer
descriptions.  Additionally, the only Emacs in Guix, emacs <at> 27.2, is
25+, so no need to write 25+ here.

Greetings,
Maxime.

[signature.asc (application/pgp-signature, inline)]

Message #14 received at 54581 <at> debbugs.gnu.org (full text, mbox):

From: Maxime Devos <maximedevos <at> telenet.be>
To: Roman Scherer <roman.scherer <at> burningswell.com>, 54581 <at> debbugs.gnu.org
Subject: Re: [bug#54581] Add emacs-sqlite3-api package
Date: Sat, 26 Mar 2022 22:21:32 +0100

[Message part 1 (text/plain, inline)]

Roman Scherer schreef op za 26-03-2022 om 15:09 [+0100]:
> +         #:tests? #f))

I see some tests in the 'tests' directory.

[signature.asc (application/pgp-signature, inline)]

Message #17 received at 54581 <at> debbugs.gnu.org (full text, mbox):

From: Maxime Devos <maximedevos <at> telenet.be>
To: Roman Scherer <roman.scherer <at> burningswell.com>, 54581 <at> debbugs.gnu.org
Subject: Re: [bug#54581] Add emacs-sqlite3-api package
Date: Sat, 26 Mar 2022 22:23:40 +0100

[Message part 1 (text/plain, inline)]

Roman Scherer schreef op za 26-03-2022 om 15:09 [+0100]:
> +             (lambda* (#:key (make-flags '()) outputs #:allow-other-
> keys)

'make-flags' is never set, so I would remove it and simplify the invoke
to

  (invoke "make" ... (string-append "CC=" ,(cc-for-target)))

Greetings,
Maxime.

[signature.asc (application/pgp-signature, inline)]

Message #20 received at 54581 <at> debbugs.gnu.org (full text, mbox):

From: Maxime Devos <maximedevos <at> telenet.be>
To: Roman Scherer <roman.scherer <at> burningswell.com>, 54581 <at> debbugs.gnu.org
Subject: Re: [bug#54581] Add emacs-sqlite3-api package
Date: Sat, 26 Mar 2022 22:25:01 +0100

[Message part 1 (text/plain, inline)]

Roman Scherer schreef op za 26-03-2022 om 15:09 [+0100]:
> +         #:phases
> +         (modify-phases %standard-phases

Could consts.c be built from source?

[signature.asc (application/pgp-signature, inline)]

Message #23 received at 54581 <at> debbugs.gnu.org (full text, mbox):

From: Roman Scherer <roman.scherer <at> burningswell.com>
To: Maxime Devos <maximedevos <at> telenet.be>
Cc: 54581 <at> debbugs.gnu.org
Subject: Re: [bug#54581] Add emacs-sqlite3-api package
Date: Sun, 27 Mar 2022 19:55:16 +0200

[Message part 1 (text/plain, inline)]

Hi Maxime,

thanks for the review. I attached 2 patches.

0001-Add-emacs-sqlite3-api-package.patch

In this one I addressed your following comments:

- I changed the synopsis and the description
- Removed the make flags
- I'm using cc-for-target now
- Enabled tests

The tests are passing, and I tested it manually by evaluating the Emacs
Lisp code snippets in [1]. Everything seems to work fine.

0001-Add-emacs-sqlite3-api-package-generate-constants.patch

In this patch I did the same as above, and I generated the consts.c file
by running make in the "tools" directory. The tests are passing fine,
but when I manually test it by evaluating the Elisp snippets in [1] I
get an error that the sqlite-open-readwrite and sqlite-open-create are
not defined in Emacs. The .so file is loaded by Emacs but somehow it
does not work.

I looked at the diff between the consts.c committed in the repository
and the generated one, but I could not see any changes related to those
functions. I'm still a bit lost about this and could need some help.

But I also wonder if generating the consts.c file is actually better
than just using the checked in consts.c file. The command to generate
the file fetches a web page from the Sqlite website and builds the
consts.c file with this information. And I think this is kind of a
brittle approach. How would this approach would even work with Software
heritage?

I think my preference would be to go without generating the consts.c
file, the patch works, it uses the same file the maintainer of the
packages is using, and I believe it is less brittle. :)

What do you think?

[1] https://github.com/pekingduck/emacs-sqlite3-api#introduction

[0001-Add-emacs-sqlite3-api-package.patch (text/x-patch, attachment)]

[0001-Add-emacs-sqlite3-api-package-generate-constants.patch (text/x-patch, inline)]

From f2726c3c87d890dfb547e7e9de880c5c57954ccf Mon Sep 17 00:00:00 2001
From: r0man <roman <at> burningswell.com>
Date: Sat, 26 Mar 2022 14:59:01 +0100
Subject: [PATCH] Add emacs-sqlite3-api package

This patch adds the emacs-sqlite3-api package to Guix.

The package provides a dynamic module for Emacs that allows direct access to
the SQLite C interface.  It only exposes a subset of the full SQLite C
interface, but should satisfy most user's needs.
---
 gnu/packages/emacs-xyz.scm | 51 ++++++++++++++++++++++++++++++++++++++
 1 file changed, 51 insertions(+)

diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index 42fc13f4c2..87595db74c 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -5731,6 +5731,57 @@ (define-public emacs-sqlite
 It is not intended as a user interface.")
       (license license:gpl3+))))
 
+(define-public emacs-sqlite3-api
+  (let ((version "0.15")
+        (revision "0")
+        (commit "7cb4b660fe30deb8a4229f3abb18bd99ca9c971c"))
+    (package
+      (name "emacs-sqlite3-api")
+      (version (git-version version revision commit))
+      (source (origin
+                (method git-fetch)
+                (uri (git-reference
+                      (url "https://github.com/pekingduck/emacs-sqlite3-api")
+                      (commit commit)))
+                (file-name (git-file-name name version))
+                (sha256
+                 (base32
+                  "1b7if1dp6i5kqwhq25gna89xbca66i4mmgx1a5yn12kncfdgs6d7"))))
+      (build-system emacs-build-system)
+      (arguments
+       `(#:modules ((guix build emacs-build-system)
+                    (guix build emacs-utils)
+                    (guix build utils))
+         #:phases
+         (modify-phases %standard-phases
+           (add-after 'unpack 'patch-module-load
+             (lambda* (#:key outputs #:allow-other-keys)
+               (chmod "sqlite3.el" #o644)
+               (emacs-substitute-sexps "sqlite3.el"
+                 ("(require 'sqlite3-api nil t)"
+                  `(module-load ,(string-append (assoc-ref outputs "out")
+                                                "/lib/sqlite3-api.so"))))))
+           (add-before 'check 'build-emacs-module
+             (lambda* (#:key (make-flags '()) outputs #:allow-other-keys)
+               ;; Generate the consts.c file.
+               (invoke "make" "--directory=tools")
+               ;; Compile the shared object file.
+               (invoke "make" (string-append "CC=" ,(cc-for-target)))
+               ;; Move the shared object file into /lib.
+               (install-file "sqlite3-api.so"
+                             (string-append (assoc-ref outputs "out")
+                                            "/lib")))))
+         #:tests? #t
+         #:test-command '("make" "test" "EMACS=emacs")))
+      (inputs (list sqlite))
+      (native-inputs (list curl pandoc python))
+      (home-page "https://github.com/pekingduck/emacs-sqlite3-api")
+      (synopsis "SQLite dynamic module for Emacs Lisp")
+      (description "This package provides a dynamic module for Emacs that allows
+direct access to the SQLite C interface.  It only exposes a subset of the full
+SQLite C interface, but should satisfy most user's needs.")
+      (license license:gpl3+))))
+
 (define-public emacs-sr-speedbar
   (let ((commit "77a83fb50f763a465c021eca7343243f465b4a47")
         (revision "0"))
-- 
2.34.0

[Message part 4 (text/plain, inline)]

Maxime Devos <maximedevos <at> telenet.be> writes:

> [[PGP Signed Part:Undecided]]
> Roman Scherer schreef op za 26-03-2022 om 15:09 [+0100]:
>> +         #:phases
>> +         (modify-phases %standard-phases
>
> Could consts.c be built from source?
>
> [[End of PGP Signed Part]]

[signature.asc (application/pgp-signature, inline)]

Message #26 received at 54581 <at> debbugs.gnu.org (full text, mbox):

From: Maxime Devos <maximedevos <at> telenet.be>
To: Roman Scherer <roman.scherer <at> burningswell.com>
Cc: 54581 <at> debbugs.gnu.org
Subject: Re: [bug#54581] Add emacs-sqlite3-api package
Date: Sun, 27 Mar 2022 21:19:52 +0200

[Message part 1 (text/plain, inline)]

Roman Scherer schreef op zo 27-03-2022 om 19:55 [+0200]:
> +           (add-before 'check 'build-emacs-module
> +             (lambda* (#:key (make-flags '()) outputs #:allow-other-
> keys)

'make-flags' can be removed here.

[signature.asc (application/pgp-signature, inline)]

Message #29 received at 54581 <at> debbugs.gnu.org (full text, mbox):

From: Maxime Devos <maximedevos <at> telenet.be>
To: Roman Scherer <roman.scherer <at> burningswell.com>
Cc: 54581 <at> debbugs.gnu.org
Subject: Re: [bug#54581] Add emacs-sqlite3-api package
Date: Sun, 27 Mar 2022 21:20:46 +0200

[Message part 1 (text/plain, inline)]

Roman Scherer schreef op zo 27-03-2022 om 19:55 [+0200]:
> +         #:tests? #t

To allow (*) cross-compilation, this needs to be #:tests? ,(%current-
target-system).

(*) cross-compilation won't work yet because Emacs is not yet cross-
compilable.

Greetings,
Maxime.

[signature.asc (application/pgp-signature, inline)]

Message #32 received at 54581 <at> debbugs.gnu.org (full text, mbox):

From: Maxime Devos <maximedevos <at> telenet.be>
To: Roman Scherer <roman.scherer <at> burningswell.com>
Cc: 54581 <at> debbugs.gnu.org
Subject: Re: [bug#54581] Add emacs-sqlite3-api package
Date: Sun, 27 Mar 2022 21:34:23 +0200

[Message part 1 (text/plain, inline)]

Roman Scherer schreef op zo 27-03-2022 om 19:55 [+0200]:
> But I also wonder if generating the consts.c file is actually better
> than just using the checked in consts.c file. The command to generate
> the file fetches a web page from the Sqlite website and builds the
> consts.c file with this information. And I think this is kind of a
> brittle approach. How would this approach would even work with
> Software
> heritage?

As-is, it won't even build, because the build container does not have
network access.  Something you could try, is adding an origin to the
'inputs':

(package
  (name "emacs-sqlite...")
  (inputs `(("web-page" ,(origin [...]
https://sqlite.org/c3ref/constlist.html [...])))))

and do some appropriate substitutions.  Problem is that this page is
not stable ...

To resolve this, you could modify gen-consts.py to not use the web page
to filter things out, basically, delete:

useful_codes = []
with open(sys.argv[1]) as f:
  for l in f.readlines():
    useful_codes.append(l.rstrip())

and

  #print("{0}".format(fields[1]))
  if not fields[1] in useful_codes:
    #print("{0} excluded".format(fields[1]))
    continue

and strip out the curl stuff from gen-consts.sh.

If you do this, don't forget to remove the non-deterministic NOW=$(date
...).

Greetings,
Maxime.

[signature.asc (application/pgp-signature, inline)]

Message #35 received at 54581 <at> debbugs.gnu.org (full text, mbox):

From: Maxime Devos <maximedevos <at> telenet.be>
To: Roman Scherer <roman.scherer <at> burningswell.com>
Cc: 54581 <at> debbugs.gnu.org
Subject: Re: [bug#54581] Add emacs-sqlite3-api package
Date: Sun, 27 Mar 2022 21:45:26 +0200

[Message part 1 (text/plain, inline)]

Roman Scherer schreef op zo 27-03-2022 om 19:55 [+0200]:
> I think my preference would be to go without generating the consts.c
> file, the patch works, it uses the same file the maintainer of the
> packages is using, and I believe it is less brittle. :)

I don't expect any practical problems but technically it isn't source
code, and rather a binary (or, as GPL words it, ‘object code’), at
least according to the GPL's definition:

  The "source code" for a work means the preferred form of the work
  for making modifications to it.  "Object code" means any non-source
  form of a work.

, and Guix builds things from source. I don't think constants.c is in
preferred form, rather it seems to be sqlite.h

Maybe it's not much of a problem here though, I donk't know.

I don't see the relevancy of what files the maintainer is using.  What
does maintainerhood have to do with anything?

Greetings,
Maxime.

[signature.asc (application/pgp-signature, inline)]

Message #38 received at 54581 <at> debbugs.gnu.org (full text, mbox):

From: Roman Scherer <roman.scherer <at> burningswell.com>
To: Maxime Devos <maximedevos <at> telenet.be>
Cc: 54581 <at> debbugs.gnu.org
Subject: Re: [bug#54581] Add emacs-sqlite3-api package
Date: Fri, 01 Apr 2022 11:07:57 +0200

[Message part 1 (text/plain, inline)]

Hi Maxime,

here's a new patch as per your suggestion. I'm removing the code that
downloads from the internet, and the call to date for the NOW variable.

I also remove the SQLITE_STATIC and SQLITE_TRANSIENT sections in the
generated constants.c file, they cause a warning and would have been
remove by the original script.

I changed the #:tests? keyword to (%current-target-system), but I
noticed the tests of the package are now not run anymore when I build
the package. Is this expected?

I tried to specify a target when building the package, but this gives me
the following error:

./pre-inst-env guix build emacs-sqlite3-api --target=x86_64
guix build: error: /home/roman/workspace/guix/gnu/packages/emacs-xyz.scm:5738:4: emacs-sqlite3-api <at> 0.15-0.7cb4b66: build system `emacs' does not support cross builds

Do you know how I can trigger the tests when building this package from
the command line, or even better from within Emacs?

Maxime Devos <maximedevos <at> telenet.be> writes:

> [[PGP Signed Part:Undecided]]
> Roman Scherer schreef op zo 27-03-2022 om 19:55 [+0200]:
>> I think my preference would be to go without generating the consts.c
>> file, the patch works, it uses the same file the maintainer of the
>> packages is using, and I believe it is less brittle. :)
>
> I don't expect any practical problems but technically it isn't source
> code, and rather a binary (or, as GPL words it, ‘object code’), at
> least according to the GPL's definition:
>
>   The "source code" for a work means the preferred form of the work
>   for making modifications to it.  "Object code" means any non-source
>   form of a work.
>
> , and Guix builds things from source. I don't think constants.c is in
> preferred form, rather it seems to be sqlite.h
>
> Maybe it's not much of a problem here though, I donk't know.
>
> I don't see the relevancy of what files the maintainer is using.  What
> does maintainerhood have to do with anything?

Well, I was looking at the last time the consts.c has been modified by
the maintainer and it was 2 years ago.

I was thinking it would be more stable to use the code that ships with
the repository, rather than building something from the internet which
hasn't been updated for 2 years.

But if this file is considered object code, ok.

> Greetings,
> Maxime.
>
> [[End of PGP Signed Part]]

[0001-Add-emacs-sqlite3-api-package.patch (text/x-patch, attachment)]

[signature.asc (application/pgp-signature, inline)]

Message #41 received at 54581 <at> debbugs.gnu.org (full text, mbox):

From: Maxime Devos <maximedevos <at> telenet.be>
To: Roman Scherer <roman.scherer <at> burningswell.com>
Cc: 54581 <at> debbugs.gnu.org
Subject: Re: [bug#54581] Add emacs-sqlite3-api package
Date: Fri, 01 Apr 2022 11:57:40 +0200

[Message part 1 (text/plain, inline)]

Roman Scherer schreef op vr 01-04-2022 om 11:07 [+0200]:
> I changed the #:tests? keyword to (%current-target-system), but I
> noticed the tests of the package are now not run anymore when I build
> the package. Is this expected?

You need the inverse condition:

  #:tests? ,(not (%current-target-system))

otherwise, tests are only run when cross-compiling, and not when
building natively.

Greetings,
Maxime.

[signature.asc (application/pgp-signature, inline)]

Message #44 received at 54581 <at> debbugs.gnu.org (full text, mbox):

From: Maxime Devos <maximedevos <at> telenet.be>
To: Roman Scherer <roman.scherer <at> burningswell.com>
Cc: 54581 <at> debbugs.gnu.org
Subject: Re: [bug#54581] Add emacs-sqlite3-api package
Date: Fri, 01 Apr 2022 11:59:04 +0200

[Message part 1 (text/plain, inline)]

Roman Scherer schreef op vr 01-04-2022 om 11:07 [+0200]:
> ./pre-inst-env guix build emacs-sqlite3-api --target=x86_64
> guix build: error: /home/roman/workspace/guix/gnu/packages/emacs-
> xyz.scm:5738:4: emacs-sqlite3-api <at> 0.15-0.7cb4b66: build system
> `emacs' does not support cross builds

FWIW x86_64 is not a valid triple, you need something like

   --target=aarch64-linux-gnu

or
   --target=x86_64-linux-gnu

or
   --target=i686-pc-gnu # or was it i586?

though that won't work either because emacs-build-system doesn't
support cross-compilation yet.

Geetings,
Maxime.

[signature.asc (application/pgp-signature, inline)]

Message #47 received at 54581 <at> debbugs.gnu.org (full text, mbox):

From: Roman Scherer <roman.scherer <at> burningswell.com>
To: Maxime Devos <maximedevos <at> telenet.be>
Cc: 54581 <at> debbugs.gnu.org
Subject: Re: [bug#54581] Add emacs-sqlite3-api package
Date: Fri, 01 Apr 2022 12:11:39 +0200

[Message part 1 (text/plain, inline)]

Ok, thank you. The attached patch uses the inverse condition now.

[0001-Add-emacs-sqlite3-api-package.patch (text/x-patch, attachment)]

[Message part 3 (text/plain, inline)]

Maxime Devos <maximedevos <at> telenet.be> writes:

> [[PGP Signed Part:Undecided]]
> Roman Scherer schreef op vr 01-04-2022 om 11:07 [+0200]:
>> I changed the #:tests? keyword to (%current-target-system), but I
>> noticed the tests of the package are now not run anymore when I build
>> the package. Is this expected?
>
> You need the inverse condition:
>
>   #:tests? ,(not (%current-target-system))
>
> otherwise, tests are only run when cross-compiling, and not when
> building natively.
>
> Greetings,
> Maxime.
>
> [[End of PGP Signed Part]]

[signature.asc (application/pgp-signature, inline)]

Message #50 received at 54581 <at> debbugs.gnu.org (full text, mbox):

From: Roman Scherer <roman.scherer <at> burningswell.com>
To: Maxime Devos <maximedevos <at> telenet.be>
Cc: 54581 <at> debbugs.gnu.org
Subject: Re: [bug#54581] Add emacs-sqlite3-api package
Date: Fri, 01 Apr 2022 12:13:05 +0200

[Message part 1 (text/plain, inline)]

Ok, understood. Thanks for the explanation.

Maxime Devos <maximedevos <at> telenet.be> writes:

> [[PGP Signed Part:Undecided]]
> Roman Scherer schreef op vr 01-04-2022 om 11:07 [+0200]:
>> ./pre-inst-env guix build emacs-sqlite3-api --target=x86_64
>> guix build: error: /home/roman/workspace/guix/gnu/packages/emacs-
>> xyz.scm:5738:4: emacs-sqlite3-api <at> 0.15-0.7cb4b66: build system
>> `emacs' does not support cross builds
>
> FWIW x86_64 is not a valid triple, you need something like
>
>    --target=aarch64-linux-gnu
>
> or
>    --target=x86_64-linux-gnu
>
> or
>    --target=i686-pc-gnu # or was it i586?
>
> though that won't work either because emacs-build-system doesn't
> support cross-compilation yet.
>
> Geetings,
> Maxime.
>
> [[End of PGP Signed Part]]

[signature.asc (application/pgp-signature, inline)]

Message #53 received at 54581 <at> debbugs.gnu.org (full text, mbox):

From: Roman Scherer <roman.scherer <at> burningswell.com>
To: Maxime Devos <maximedevos <at> telenet.be>
Cc: 54581 <at> debbugs.gnu.org
Subject: Re: [bug#54581] Add emacs-sqlite3-api package
Date: Thu, 05 May 2022 09:46:01 +0200

[Message part 1 (text/plain, inline)]

Hi Maxime,

is there anything I can do to move this forward?

Roman

Roman Scherer <roman.scherer <at> burningswell.com> writes:

> [[PGP Signed Part:Undecided]]
>
> Ok, thank you. The attached patch uses the inverse condition now.
>
> [2. text/x-patch; 0001-Add-emacs-sqlite3-api-package.patch]...
>
>
> Maxime Devos <maximedevos <at> telenet.be> writes:
>
>> [[PGP Signed Part:Undecided]]
>> Roman Scherer schreef op vr 01-04-2022 om 11:07 [+0200]:
>>> I changed the #:tests? keyword to (%current-target-system), but I
>>> noticed the tests of the package are now not run anymore when I build
>>> the package. Is this expected?
>>
>> You need the inverse condition:
>>
>>   #:tests? ,(not (%current-target-system))
>>
>> otherwise, tests are only run when cross-compiling, and not when
>> building natively.
>>
>> Greetings,
>> Maxime.
>>
>> [[End of PGP Signed Part]]
>
> [[End of PGP Signed Part]]

[signature.asc (application/pgp-signature, inline)]

Message #56 received at 54581 <at> debbugs.gnu.org (full text, mbox):

From: Maxime Devos <maximedevos <at> telenet.be>
To: Roman Scherer <roman.scherer <at> burningswell.com>
Cc: 54581 <at> debbugs.gnu.org
Subject: Re: [bug#54581] Add emacs-sqlite3-api package
Date: Thu, 05 May 2022 12:51:09 +0200

[Message part 1 (text/plain, inline)]

Roman Scherer schreef op do 05-05-2022 om 09:46 [+0200]:
> 
> Hi Maxime,
> 
> is there anything I can do to move this forward?
> 
> Roman

I am currently not reviewing packages much, so for now, you will have
to wait for another reviewer (or committer if the package definition is
good).

Greetings,
Maxime.

[signature.asc (application/pgp-signature, inline)]

Message #61 received at 54581-done <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Goaziou <mail <at> nicolasgoaziou.fr>
To: Roman Scherer <roman.scherer <at> burningswell.com>
Cc: 54581-done <at> debbugs.gnu.org, Maxime Devos <maximedevos <at> telenet.be>
Subject: Re: [bug#54581] Add emacs-sqlite3-api package
Date: Sun, 18 Sep 2022 14:44:00 +0200

Hello,

Roman Scherer <roman.scherer <at> burningswell.com> writes:

> Ok, thank you. The attached patch uses the inverse condition now.

Thank you. I updated the package to latest 0.16, rewrote the arguments
with G-expressions, and applied it all.

Regards,
-- 
Nicolas Goaziou

Message #64 received at 54581-done <at> debbugs.gnu.org (full text, mbox):

From: Roman Scherer <roman <at> burningswell.com>
To: Nicolas Goaziou <mail <at> nicolasgoaziou.fr>
Cc: 54581-done <at> debbugs.gnu.org, Maxime Devos <maximedevos <at> telenet.be>
Subject: Re: [bug#54581] Add emacs-sqlite3-api package
Date: Mon, 19 Sep 2022 08:46:15 +0200

[Message part 1 (text/plain, inline)]

Thank you Nicolas

On Sun, Sep 18, 2022, 14:44 Nicolas Goaziou <mail <at> nicolasgoaziou.fr> wrote:

> Hello,
>
> Roman Scherer <roman.scherer <at> burningswell.com> writes:
>
> > Ok, thank you. The attached patch uses the inverse condition now.
>
> Thank you. I updated the package to latest 0.16, rewrote the arguments
> with G-expressions, and applied it all.
>
> Regards,
> --
> Nicolas Goaziou
>

[Message part 2 (text/html, inline)]

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.