GNU bug report logs - #72626
[PATCH] Add biblesync

Previous Next

Package: guix-patches;

Reported by: peepofroggings <at> tutanota.de

Date: Wed, 14 Aug 2024 20:04:02 UTC

Severity: normal

Tags: patch

Done: Nicolas Goaziou <mail <at> nicolasgoaziou.fr>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: peepofroggings <at> tutanota.de
Subject: bug#72626: closed (Re: [bug#72626] [PATCH] Add biblesync)
Date: Wed, 21 Aug 2024 08:56:03 +0000
[Message part 1 (text/plain, inline)]
Your bug report

#72626: [PATCH] Add biblesync

which was filed against the guix-patches package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to 72626 <at> debbugs.gnu.org.

-- 
72626: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=72626
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems
[Message part 2 (message/rfc822, inline)]
From: Nicolas Goaziou <mail <at> nicolasgoaziou.fr>
To: peepofroggings--- via Guix-patches via <guix-patches <at> gnu.org>
Cc: 72626-done <at> debbugs.gnu.org, peepofroggings <at> tutanota.de
Subject: Re: [bug#72626] [PATCH] Add biblesync
Date: Wed, 21 Aug 2024 10:54:34 +0200
Hello,

peepofroggings--- via Guix-patches via <guix-patches <at> gnu.org> writes:

> From 5065444ffc09c5bfc47fe2ae6b4e806744bba9a1 Mon Sep 17 00:00:00 2001
> From: <peepofroggings <at> tutanota.de>
> Date: Wed, 14 Aug 2024 02:45:00 +0200
> Subject: [PATCH] Add biblesync

Thank you.

I normalized the commit message, fixed some issues with your patch,and
applied it.
 
>        (license license:zlib))))

There are whitespace issues in your email.
>  
> +(define-public biblesync
> +  (package
> +    (name "biblesync")
> +    (version "2.1.0")
> +    (source
> +      (origin
> +              (method git-fetch)
> +              (uri
> +               (git-reference
> +                (url "https://github.com/karlkleinpaste/biblesync")
> +                (commit version))))

The parenthesis are off and as such the package cannot build properly.

> +              (file-name (git-file-name name version))
> +      (sha256
> +        (base32 "1s6hx0av2fb2rddhr36v406r8bf054hi2m1j7a62npyqqjngr0wh"))))

The hash was wrong, so I updated it.

> +    (build-system cmake-build-system)
> +    (arguments '(#:tests? #f))

I added a comment explaining why (in a way…) tests couldn't be run.

I also made the process build shared library instead of static one.

> +    (synopsis "C++ library implementing the BibleSync protocol")
> +    (description
> +      "BibleSync is a multicast protocol to support Bible software shared
> +      co-navigation. It uses LAN multicast in either a personal/small team
> +      mutual navigation motif or in a classroom environment where there are
> +      Speakers plus the Audience. The library implementing the protocol is a
> +      single C++ class providing a complete yet minimal public interface
> +      to support mode setting, setup for packet reception, transmit on local
> +      navigation, and handling of incoming packets. This library is not specific
> +      to any particular Bible software framework, completely agnostic as to
> +      structure of layers above BibleSync, and is not a product of The Sword
> +      Project. But its first implementation has been grafted into
> Xiphos.")

I fixed indentation and tweaked description a bit.

> +    (inputs (list `(,util-linux "lib")))
> +    (home-page "https://github.com/karlkleinpaste/biblesync")
> +    (license public-domain)))

The above should have been (license license:public-domain)

From the Guix repository, you can try building your package with
"./pre-inst-env guix build biblesync". You will be able to catch most of
the mistakes above.

Regards,
-- 
Nicolas Goaziou



[Message part 3 (message/rfc822, inline)]
From: peepofroggings <at> tutanota.de
To: Guix Patches <guix-patches <at> gnu.org>
Subject: [PATCH] Add biblesync
Date: Wed, 14 Aug 2024 21:48:30 +0200 (CEST)
From 5065444ffc09c5bfc47fe2ae6b4e806744bba9a1 Mon Sep 17 00:00:00 2001
From: <peepofroggings <at> tutanota.de>
Date: Wed, 14 Aug 2024 02:45:00 +0200
Subject: [PATCH] Add biblesync

---
gnu/packages/cpp.scm | 33 +++++++++++++++++++++++++++++++++
1 file changed, 33 insertions(+)

diff --git a/gnu/packages/cpp.scm b/gnu/packages/cpp.scm
index c1c2a40..ac45a00 100644
--- a/gnu/packages/cpp.scm
+++ b/gnu/packages/cpp.scm
@@ -40,6 +40,7 @@
;;; Copyright © 2023 Foundation Devices, Inc. <hello <at> foundationdevices.com>
;;; Copyright © 2023 Paul A. Patience <paul <at> apatience.com>
;;; Copyright © 2024 dan <i <at> dan.games>
+;;; Copyright © 2024 Peepo Froggings <peepofroggings <at> tutanota.de>
;;;
;;; This file is part of GNU Guix.
;;;
@@ -188,6 +189,38 @@ (define-public asmjit
development effort.")
       (license license:zlib))))
 
+(define-public biblesync
+  (package
+    (name "biblesync")
+    (version "2.1.0")
+    (source
+      (origin
+              (method git-fetch)
+              (uri
+               (git-reference
+                (url "https://github.com/karlkleinpaste/biblesync")
+                (commit version))))
+              (file-name (git-file-name name version))
+      (sha256
+        (base32 "1s6hx0av2fb2rddhr36v406r8bf054hi2m1j7a62npyqqjngr0wh"))))
+    (build-system cmake-build-system)
+    (arguments '(#:tests? #f))
+    (synopsis "C++ library implementing the BibleSync protocol")
+    (description
+      "BibleSync is a multicast protocol to support Bible software shared
+      co-navigation. It uses LAN multicast in either a personal/small team
+      mutual navigation motif or in a classroom environment where there are
+      Speakers plus the Audience. The library implementing the protocol is a
+      single C++ class providing a complete yet minimal public interface
+      to support mode setting, setup for packet reception, transmit on local
+      navigation, and handling of incoming packets. This library is not specific
+      to any particular Bible software framework, completely agnostic as to
+      structure of layers above BibleSync, and is not a product of The Sword
+      Project. But its first implementation has been grafted into Xiphos.")
+    (inputs (list `(,util-linux "lib")))
+    (home-page "https://github.com/karlkleinpaste/biblesync")
+    (license public-domain)))
+
(define-public castxml
   (package
     (name "castxml")
-- 
2.45.2





This bug report was last modified 334 days ago.

Previous Next


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