GNU bug report logs -
#51148
[PATCH] gnu: Add python-tinydb.
Previous Next
Reported by: jgart <jgart <at> dismail.de>
Date: Tue, 12 Oct 2021 05:46:02 UTC
Severity: normal
Tags: patch
Done: Ludovic Courtès <ludo <at> gnu.org>
Bug is archived. No further changes may be made.
To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 51148 in the body.
You can then email your comments to 51148 AT debbugs.gnu.org in the normal way.
Toggle the display of automated, internal messages from the tracker.
Report forwarded
to
guix-patches <at> gnu.org
:
bug#51148
; Package
guix-patches
.
(Tue, 12 Oct 2021 05:46:02 GMT)
Full text and
rfc822 format available.
Acknowledgement sent
to
jgart <jgart <at> dismail.de>
:
New bug report received and forwarded. Copy sent to
guix-patches <at> gnu.org
.
(Tue, 12 Oct 2021 05:46:02 GMT)
Full text and
rfc822 format available.
Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):
* gnu/packages/databases.scm (python-tinydb): New variable.
---
gnu/packages/databases.scm | 40 ++++++++++++++++++++++++++++++++++++++
1 file changed, 40 insertions(+)
diff --git a/gnu/packages/databases.scm b/gnu/packages/databases.scm
index b1dd44da1c..60970b9c8e 100644
--- a/gnu/packages/databases.scm
+++ b/gnu/packages/databases.scm
@@ -579,6 +579,46 @@ the API, and provides features such as:
@end itemize")
(license license:bsd-3)))
+(define-public python-tinydb
+ (package
+ (name "python-tinydb")
+ (version "4.5.2")
+ (source (origin
+ (method url-fetch)
+ (uri (pypi-uri "tinydb" version))
+ (sha256
+ (base32
+ "1x9c4s42930wwal3ds0plwb57kg5c3gj7kbpy64c29vq478b463x"))))
+ (build-system python-build-system)
+ (arguments
+ ;; PyPi tarball does not contain tests and github repository does not
+ ;; have a setup.py file (only pyproject).
+ `(#:tests? #f
+ #:phases
+ (modify-phases %standard-phases
+ (replace 'check
+ (lambda* (#:key inputs outputs tests? #:allow-other-keys)
+ (when tests?
+ (add-installed-pythonpath inputs outputs)
+ (setenv "PYTHONPATH" (string-append ".:" (getenv "PYTHONPATH")))
+ (invoke "pytest")))))))
+ (native-inputs
+ `(("python-pytest" ,python-pytest)
+ ("python-pytest-cov" ,python-pytest-cov)
+ ("python-pycodestyle" ,python-pycodestyle)
+ ("python-pyyaml" ,python-pyyaml)))
+ (propagated-inputs
+ `(("python-typing-extensions" ,python-typing-extensions)))
+ (home-page "https://github.com/msiemens/tinydb")
+ (synopsis
+ "TinyDB is a lightweight document oriented database")
+ (description
+"TinyDB is a lightweight document oriented database optimized for
+your happiness. It's written in pure Python and has no external
+dependencies. The targets are small apps that would be blown away by a
+SQL-DB or an external database server.")
+ (license license:expat)))
+
(define-public python-pylibmc
(package
(name "python-pylibmc")
--
2.33.0
Information forwarded
to
guix-patches <at> gnu.org
:
bug#51148
; Package
guix-patches
.
(Thu, 11 Nov 2021 05:26:01 GMT)
Full text and
rfc822 format available.
Message #8 received at submit <at> debbugs.gnu.org (full text, mbox):
Hi jgart,
Em ter, 2021-10-12 às 01:44 -0400, jgart escreveu:
> * gnu/packages/databases.scm (python-tinydb): New variable.
> ---
> gnu/packages/databases.scm | 40
Don't miss your copyright in this file.
> ++++++++++++++++++++++++++++++++++++++
> 1 file changed, 40 insertions(+)
>
> diff --git a/gnu/packages/databases.scm
> b/gnu/packages/databases.scm
> index b1dd44da1c..60970b9c8e 100644
> --- a/gnu/packages/databases.scm
> +++ b/gnu/packages/databases.scm
> @@ -579,6 +579,46 @@ the API, and provides features such as:
> @end itemize")
> (license license:bsd-3)))
>
> +(define-public python-tinydb
> + (package
> + (name "python-tinydb")
> + (version "4.5.2")
> + (source (origin
> + (method url-fetch)
> + (uri (pypi-uri "tinydb" version))
> + (sha256
> + (base32
> +
> "1x9c4s42930wwal3ds0plwb57kg5c3gj7kbpy64c29vq478b463x"))))
> + (build-system python-build-system)
The indentation is off below.
> + (arguments
> + ;; PyPi tarball does not contain tests and github repository
> does not
> + ;; have a setup.py file (only pyproject).
I recently noticed this pattern of shipping only Poetry in the
repository while at the same time not shipping tests in PyPI. I don't
know why this is happening or what we can do about it in guix, the best
for now is to source from PyPI and skip tests like in this package.
> + `(#:tests? #f
> + #:phases
> + (modify-phases %standard-phases
> + (replace 'check
> + (lambda* (#:key inputs outputs tests? #:allow-other-
> keys)
> + (when tests?
> + (add-installed-pythonpath inputs outputs)
> + (setenv "PYTHONPATH" (string-append ".:" (getenv
> "PYTHONPATH")))
The two lines above are redundant. The second line is the manual way of
doing what the first one does. But if we don't have a way to run the
tests, I'd not override the check phase.
> + (invoke "pytest")))))))
> + (native-inputs
> + `(("python-pytest" ,python-pytest)
> + ("python-pytest-cov" ,python-pytest-cov)
> + ("python-pycodestyle" ,python-pycodestyle)
> + ("python-pyyaml" ,python-pyyaml)))
Test-related native inputs are also useless if tests can't run.
> + (propagated-inputs
> + `(("python-typing-extensions" ,python-typing-extensions)))
> + (home-page "https://github.com/msiemens/tinydb")
> + (synopsis
> + "TinyDB is a lightweight document oriented database")
Same line for the synopsis.
> + (description
> +"TinyDB is a lightweight document oriented database optimized for
> +your happiness. It's written in pure Python and has no external
> +dependencies. The targets are small apps that would be blown away
> by a
> +SQL-DB or an external database server.")
I'd suggest "TinyDB is a small document oriented database written in
pure Python with no external dependencies. [...]"
> + (license license:expat)))
> +
> (define-public python-pylibmc
> (package
> (name "python-pylibmc")
Information forwarded
to
guix-patches <at> gnu.org
:
bug#51148
; Package
guix-patches
.
(Thu, 11 Nov 2021 05:28:02 GMT)
Full text and
rfc822 format available.
Message #11 received at 51148 <at> debbugs.gnu.org (full text, mbox):
Hi jgart,
Em ter, 2021-10-12 às 01:44 -0400, jgart escreveu:
> * gnu/packages/databases.scm (python-tinydb): New variable.
> ---
> gnu/packages/databases.scm | 40
Don't miss your copyright in this file.
> ++++++++++++++++++++++++++++++++++++++
> 1 file changed, 40 insertions(+)
>
> diff --git a/gnu/packages/databases.scm
> b/gnu/packages/databases.scm
> index b1dd44da1c..60970b9c8e 100644
> --- a/gnu/packages/databases.scm
> +++ b/gnu/packages/databases.scm
> @@ -579,6 +579,46 @@ the API, and provides features such as:
> @end itemize")
> (license license:bsd-3)))
>
> +(define-public python-tinydb
> + (package
> + (name "python-tinydb")
> + (version "4.5.2")
> + (source (origin
> + (method url-fetch)
> + (uri (pypi-uri "tinydb" version))
> + (sha256
> + (base32
> +
> "1x9c4s42930wwal3ds0plwb57kg5c3gj7kbpy64c29vq478b463x"))))
> + (build-system python-build-system)
The indentation is off below.
> + (arguments
> + ;; PyPi tarball does not contain tests and github repository
> does not
> + ;; have a setup.py file (only pyproject).
I recently noticed this pattern of shipping only Poetry in the
repository while at the same time not shipping tests in PyPI. I don't
know why this is happening or what we can do about it in guix, the best
for now is to source from PyPI and skip tests like in this package.
> + `(#:tests? #f
> + #:phases
> + (modify-phases %standard-phases
> + (replace 'check
> + (lambda* (#:key inputs outputs tests? #:allow-other-
> keys)
> + (when tests?
> + (add-installed-pythonpath inputs outputs)
> + (setenv "PYTHONPATH" (string-append ".:" (getenv
> "PYTHONPATH")))
The two lines above are redundant. The second line is the manual way of
doing what the first one does. But if we don't have a way to run the
tests, I'd not override the check phase.
> + (invoke "pytest")))))))
> + (native-inputs
> + `(("python-pytest" ,python-pytest)
> + ("python-pytest-cov" ,python-pytest-cov)
> + ("python-pycodestyle" ,python-pycodestyle)
> + ("python-pyyaml" ,python-pyyaml)))
Test-related native inputs are also useless if tests can't run.
> + (propagated-inputs
> + `(("python-typing-extensions" ,python-typing-extensions)))
> + (home-page "https://github.com/msiemens/tinydb")
> + (synopsis
> + "TinyDB is a lightweight document oriented database")
Same line for the synopsis.
> + (description
> +"TinyDB is a lightweight document oriented database optimized for
> +your happiness. It's written in pure Python and has no external
> +dependencies. The targets are small apps that would be blown away
> by a
> +SQL-DB or an external database server.")
I'd suggest "TinyDB is a small document oriented database written in
pure Python with no external dependencies. [...]".
> + (license license:expat)))
> +
> (define-public python-pylibmc
> (package
> (name "python-pylibmc")
Information forwarded
to
guix-patches <at> gnu.org
:
bug#51148
; Package
guix-patches
.
(Thu, 11 Nov 2021 16:34:01 GMT)
Full text and
rfc822 format available.
Message #14 received at 51148 <at> debbugs.gnu.org (full text, mbox):
Hi Vinicius,
Attached are the changes you requested.
Thanks for the code review.
It's much appreciated!
* gnu/packages/databases.scm (python-tinydb): New variable.
---
gnu/packages/databases.scm | 23 +++++++++++++++++++++++
1 file changed, 23 insertions(+)
diff --git a/gnu/packages/databases.scm b/gnu/packages/databases.scm
index 92a7aa2761..2495ad8a28 100644
--- a/gnu/packages/databases.scm
+++ b/gnu/packages/databases.scm
@@ -4072,6 +4072,29 @@ (define-public freetds
connecting to MS SQL and Sybase servers over TCP/IP.")
(license license:lgpl2.0+)))
+(define-public python-tinydb
+ (package
+ (name "python-tinydb")
+ (version "4.5.2")
+ (source (origin
+ (method url-fetch)
+ (uri (pypi-uri "tinydb" version))
+ (sha256
+ (base32 "1x9c4s42930wwal3ds0plwb57kg5c3gj7kbpy64c29vq478b463x"))))
+ (build-system python-build-system)
+ ;; PyPi tarball does not contain tests and github repository does not
+ ;; have a setup.py file (only pyproject).
+ (arguments `(#:tests? #f))
+ (propagated-inputs
+ `(("python-typing-extensions" ,python-typing-extensions)))
+ (home-page "https://github.com/msiemens/tinydb")
+ (synopsis "TinyDB is a lightweight document oriented database")
+ (description
+"TinyDB is a small document oriented database written in pure Python
+with no external dependencies. The targets are small apps that would
+be blown away by a SQL-DB or an external database server.")
+ (license license:expat)))
+
(define-public sequeler
(package
(name "sequeler")
--
2.33.1
Information forwarded
to
guix-patches <at> gnu.org
:
bug#51148
; Package
guix-patches
.
(Fri, 12 Nov 2021 03:49:01 GMT)
Full text and
rfc822 format available.
Message #17 received at 51148 <at> debbugs.gnu.org (full text, mbox):
I forgot to add the copyright at the top.
Attached is version 3 of the patchset.
* gnu/packages/databases.scm (python-tinydb): New variable.
---
gnu/packages/databases.scm | 24 ++++++++++++++++++++++++
1 file changed, 24 insertions(+)
diff --git a/gnu/packages/databases.scm b/gnu/packages/databases.scm
index 92a7aa2761..87fa1da895 100644
--- a/gnu/packages/databases.scm
+++ b/gnu/packages/databases.scm
@@ -53,6 +53,7 @@
;;; Copyright © 2021 Simon Streit <simon <at> netpanic.org>
;;; Copyright © 2021 Alexandre Hannud Abdo <abdo <at> member.fsf.org>
;;; Copyright © 2021 Simon Tournier <zimon.toutoune <at> gmail.com>
+;;; Copyright © 2021 Jorge Gomez <jgart <at> dismail.de>
;;;
;;; This file is part of GNU Guix.
;;;
@@ -4072,6 +4073,29 @@ (define-public freetds
connecting to MS SQL and Sybase servers over TCP/IP.")
(license license:lgpl2.0+)))
+(define-public python-tinydb
+ (package
+ (name "python-tinydb")
+ (version "4.5.2")
+ (source (origin
+ (method url-fetch)
+ (uri (pypi-uri "tinydb" version))
+ (sha256
+ (base32 "1x9c4s42930wwal3ds0plwb57kg5c3gj7kbpy64c29vq478b463x"))))
+ (build-system python-build-system)
+ ;; PyPi tarball does not contain tests and github repository does not
+ ;; have a setup.py file (only pyproject).
+ (arguments `(#:tests? #f))
+ (propagated-inputs
+ `(("python-typing-extensions" ,python-typing-extensions)))
+ (home-page "https://github.com/msiemens/tinydb")
+ (synopsis "TinyDB is a lightweight document oriented database")
+ (description
+"TinyDB is a small document oriented database written in pure Python
+with no external dependencies. The targets are small apps that would
+be blown away by a SQL-DB or an external database server.")
+ (license license:expat)))
+
(define-public sequeler
(package
(name "sequeler")
--
2.33.1
Reply sent
to
Ludovic Courtès <ludo <at> gnu.org>
:
You have taken responsibility.
(Tue, 23 Nov 2021 08:23:02 GMT)
Full text and
rfc822 format available.
Notification sent
to
jgart <jgart <at> dismail.de>
:
bug acknowledged by developer.
(Tue, 23 Nov 2021 08:23:02 GMT)
Full text and
rfc822 format available.
Message #22 received at 51148-done <at> debbugs.gnu.org (full text, mbox):
Hi,
jgart <jgart <at> dismail.de> skribis:
> I forgot to add the copyright at the top.
In the meantime another line had been added for you. :-)
> Attached is version 3 of the patchset.
>
> * gnu/packages/databases.scm (python-tinydb): New variable.
I applied and tweaked indentation.
Thanks, and thanks Vinicius for reviewing,
Ludo’.
Information forwarded
to
guix-patches <at> gnu.org
:
bug#51148
; Package
guix-patches
.
(Tue, 23 Nov 2021 08:38:02 GMT)
Full text and
rfc822 format available.
Message #25 received at 51148-done <at> debbugs.gnu.org (full text, mbox):
On Tue, 23 Nov 2021 09:22:39 +0100 Ludovic Courtès <ludo <at> gnu.org> wrote:
Thanks Ludo and Vinicius!
all best,
jgart
bug archived.
Request was from
Debbugs Internal Request <help-debbugs <at> gnu.org>
to
internal_control <at> debbugs.gnu.org
.
(Tue, 21 Dec 2021 12:24:08 GMT)
Full text and
rfc822 format available.
This bug report was last modified 2 years and 89 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.