Subject: [bug#66688] [PATCH 2/2] gnu: Add python-djitellopy.

From: Maxim Cournoyer <[email-masked]>
Date: Wed, 03 Jan 2024 21:29:28 -0500

Hi!

Sharlatan Hellseher <[email-masked]> writes:

> * gnu/packages/python-xyz.scm (python-djitellopy): New variable.
> ---
>  gnu/packages/python-xyz.scm | 40 +++++++++++++++++++++++++++++++++++++
>  1 file changed, 40 insertions(+)
>
> diff --git a/gnu/packages/python-xyz.scm b/gnu/packages/python-xyz.scm
> index 1558c74492..ef3ec0ee97 100644
> --- a/gnu/packages/python-xyz.scm
> +++ b/gnu/packages/python-xyz.scm
> @@ -29932,6 +29932,46 @@ (define-public python-parallel
>  applications with variable CPU loads).")
>      (license license:bsd-3)))
>  
> +(define-public python-djitellopy
> +  (package
> +    (name "python-djitellopy")
> +    (version "2.5.0")
> +    (source
> +     (origin
> +       (method url-fetch)
> +       (uri (pypi-uri "djitellopy" version))
> +       (sha256
> +        (base32 "1kc0syb4hpn7fay0rxpazmczag6jw3pncrrc6v762jj0afiwkrps"))))
> +    (build-system pyproject-build-system)
> +    (arguments
> +     (list
> +      #:tests? #f ; No tests provided.
> +      #:phases
> +      #~(modify-phases %standard-phases
> +          ;; XXX: Can't detect opencv-python version. The input opencv might not
> +          ;; set the version correctly.
> +          (delete 'sanity-check))))
> +    ;; opencv provides OpenCV-Python which is Listed as install requirement.
> +    (propagated-inputs
> +     (list opencv
> +           python-av
> +           python-numpy
> +           python-pillow))
> +    (home-page "https://github.com/damiafuentes/DJITelloPy")
> +    (synopsis "DJI Tello drone lib with video streaming, swarms and state packets")
> +    (description
> +     "DJI Tello drone python interface using the official Tello SDK and Tello
> +EDU SDK.
> +
> +This library has the following features:
> +@itemize
> +@item Implementation of all tello commands
> +@item Retrieve a video stream easely
> +@item Receive and parse state packets
> +@item Control a swarm of drones
> +@end itemize")
> +    (license license:expat)))
> +
>  (define-public python-djvulibre
>    (package
>      (name "python-djvulibre")

I've taken the liberty to do the following edits:

--8<---------------cut here---------------start------------->8---
modified   gnu/packages/python-xyz.scm
@@ -30288,30 +30288,28 @@ (define-public python-djitellopy
     (build-system pyproject-build-system)
     (arguments
      (list
-      #:tests? #f ; No tests provided.
+      #:tests? #f                       ;no test suite
       #:phases
       #~(modify-phases %standard-phases
-          ;; XXX: Can't detect opencv-python version. The input opencv might not
-          ;; set the version correctly.
+          ;; OpenCV does not ship metadata files for its Python library, which
+          ;; makes it invisible to the sanity_check script (see:
+          ;; https://github.com/opencv/opencv/issues/24810).
           (delete 'sanity-check))))
-    ;; opencv provides OpenCV-Python which is Listed as install requirement.
     (propagated-inputs
-     (list opencv
+     (list opencv                       ;for opencv-python
            python-av
            python-numpy
            python-pillow))
     (home-page "https://github.com/damiafuentes/DJITelloPy")
-    (synopsis "DJI Tello drone lib with video streaming, swarms and state packets")
+    (synopsis "DJI Tello drone library with video streaming, swarms and state packets")
     (description
      "DJI Tello drone python interface using the official Tello SDK and Tello
-EDU SDK.
-
-This library has the following features:
+EDU SDK. This library has the following features:
 @itemize
 @item Implementation of all tello commands
-@item Retrieve a video stream easely
+@item Retrieve a video stream easily
 @item Receive and parse state packets
-@item Control a swarm of drones
+@item Control a swarm of drones.
 @end itemize")
     (license license:expat)))
 --8<---------------cut here---------------end--------------->8---

I'll push it shortly.

-- 
Thanks,
Maxim