unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* [bug#32053] [PATCH 1/2] gnu: Add libqmatrixclient.
@ 2018-07-04 17:42 Arun Isaac
  2018-07-04 17:46 ` [bug#32053] [PATCH 2/2] gnu: Add quaternion Arun Isaac
                   ` (2 more replies)
  0 siblings, 3 replies; 10+ messages in thread
From: Arun Isaac @ 2018-07-04 17:42 UTC (permalink / raw)
  To: 32053

* gnu/packages/messaging.scm (libqmatrixclient): New variable.
---
 gnu/packages/messaging.scm | 26 +++++++++++++++++++++++++-
 1 file changed, 25 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/messaging.scm b/gnu/packages/messaging.scm
index 181cdecc9..a9db199f9 100644
--- a/gnu/packages/messaging.scm
+++ b/gnu/packages/messaging.scm
@@ -9,7 +9,7 @@
 ;;; Copyright © 2016 Andy Patterson <ajpatter@uwaterloo.ca>
 ;;; Copyright © 2016, 2017, 2018 Clément Lassieur <clement@lassieur.org>
 ;;; Copyright © 2017 Mekeor Melire <mekeor.melire@gmail.com>
-;;; Copyright © 2017 Arun Isaac <arunisaac@systemreboot.net>
+;;; Copyright © 2017, 2018 Arun Isaac <arunisaac@systemreboot.net>
 ;;; Copyright © 2017, 2018 Tobias Geerinckx-Rice <me@tobias.gr>
 ;;; Copyright © 2017 Theodoros Foradis <theodoros@foradis.org>
 ;;; Copyright © 2017 Rutger Helling <rhelling@mykolab.com>
@@ -1644,4 +1644,28 @@ c-toxcore and ncurses.  It provides audio calls, sound and desktop
 notifications, and Python scripting support.")
     (license license:gpl3+)))
 
+(define-public libqmatrixclient
+  (package
+    (name "libqmatrixclient")
+    (version "0.3.0.2")
+    (source
+     (origin
+       (method url-fetch)
+       (uri (string-append "https://github.com/QMatrixClient/libqmatrixclient/archive/v"
+                           version ".tar.gz"))
+       (file-name (string-append name "-" version ".tar.gz"))
+       (sha256
+        (base32
+         "0yl7lw0s2z98xkxbklkb3i8xcd9af9qayl1zxl070d8ykl6ayqy3"))))
+    (build-system cmake-build-system)
+    (inputs
+     `(("qt" ,qt)))
+    (arguments
+     `(#:tests? #f)) ; No tests
+    (home-page "https://matrix.org/docs/projects/sdk/libqmatrixclient.html")
+    (synopsis "Qt5 Matrix client library")
+    (description "libqmatrixclient is a Qt5 library to write clients for the
+Matrix protocol.  Quaternion is the reference client implementation.
+Quaternion and libqmatrixclient together form the QMatrixClient project.")
+    (license license:lgpl2.1+)))
 ;;; messaging.scm ends here
-- 
2.15.1

^ permalink raw reply related	[flat|nested] 10+ messages in thread

* [bug#32053] [PATCH 2/2] gnu: Add quaternion.
  2018-07-04 17:42 [bug#32053] [PATCH 1/2] gnu: Add libqmatrixclient Arun Isaac
@ 2018-07-04 17:46 ` Arun Isaac
  2018-07-09 12:37   ` Ludovic Courtès
  2018-07-09 12:34 ` [bug#32053] [PATCH 1/2] gnu: Add libqmatrixclient Ludovic Courtès
  2018-07-12 10:51 ` [bug#32053] [PATCH v2 0/2] Add quaternion Arun Isaac
  2 siblings, 1 reply; 10+ messages in thread
From: Arun Isaac @ 2018-07-04 17:46 UTC (permalink / raw)
  To: 32053

* gnu/packages/messaging.scm (quaternion): New variable.
---
 gnu/packages/messaging.scm | 35 +++++++++++++++++++++++++++++++++++
 1 file changed, 35 insertions(+)

diff --git a/gnu/packages/messaging.scm b/gnu/packages/messaging.scm
index a9db199f9..9e921ba16 100644
--- a/gnu/packages/messaging.scm
+++ b/gnu/packages/messaging.scm
@@ -1668,4 +1668,39 @@ notifications, and Python scripting support.")
 Matrix protocol.  Quaternion is the reference client implementation.
 Quaternion and libqmatrixclient together form the QMatrixClient project.")
     (license license:lgpl2.1+)))
+
+(define-public quaternion
+  (package
+    (name "quaternion")
+    (version "0.0.9.2")
+    (source
+     (origin
+       (method url-fetch)
+       (uri (string-append "https://github.com/QMatrixClient/Quaternion/archive/v"
+                           version ".tar.gz"))
+       (file-name (string-append name "-" version ".tar.gz"))
+       (sha256
+        (base32
+         "0s2mpw11s2n21ds1spf94j1p2lj2px2bv5zxldlcx81ch0rb4ng8"))))
+    (build-system cmake-build-system)
+    (inputs
+     `(("libqmatrixclient-source" ,(package-source libqmatrixclient))
+       ("qt" ,qt)))
+    (arguments
+     `(#:tests? #f ; No tests
+       #:phases
+       (modify-phases %standard-phases
+         (add-after 'unpack 'unpack-libqmatrixclient
+           (lambda* (#:key inputs #:allow-other-keys)
+             (invoke "tar" "-C" "lib" "-xvf"
+                     (assoc-ref inputs "libqmatrixclient-source")
+                     "--strip-components=1"))))))
+    (home-page "https://matrix.org/docs/projects/client/quaternion.html")
+    (synopsis "Qt5 desktop client for the Matrix protocol")
+    (description "Quaternion is a Qt5 desktop client for the Matrix protocol.
+It uses libqmatrixclient and is its reference client implementation.
+Quaternion and libqmatriclient together form the QMatrixClient project.")
+    (license (list license:gpl3+ ; all source code
+                   license:lgpl3+)))) ; icons/breeze
+
 ;;; messaging.scm ends here
-- 
2.15.1

^ permalink raw reply related	[flat|nested] 10+ messages in thread

* [bug#32053] [PATCH 1/2] gnu: Add libqmatrixclient.
  2018-07-04 17:42 [bug#32053] [PATCH 1/2] gnu: Add libqmatrixclient Arun Isaac
  2018-07-04 17:46 ` [bug#32053] [PATCH 2/2] gnu: Add quaternion Arun Isaac
@ 2018-07-09 12:34 ` Ludovic Courtès
  2018-07-12 10:51 ` [bug#32053] [PATCH v2 0/2] Add quaternion Arun Isaac
  2 siblings, 0 replies; 10+ messages in thread
From: Ludovic Courtès @ 2018-07-09 12:34 UTC (permalink / raw)
  To: Arun Isaac; +Cc: 32053

Arun Isaac <arunisaac@systemreboot.net> skribis:

> * gnu/packages/messaging.scm (libqmatrixclient): New variable.

[...]

> +    (home-page "https://matrix.org/docs/projects/sdk/libqmatrixclient.html")
> +    (synopsis "Qt5 Matrix client library")

What about “Qt5 client library for the Matrix messaging protocol”?

> +    (description "libqmatrixclient is a Qt5 library to write clients for the
> +Matrix protocol.  Quaternion is the reference client implementation.
         ^
s/protocol/instant messaging protocol/

Apart from that it LGTM, thank you!

Ludo’.

^ permalink raw reply	[flat|nested] 10+ messages in thread

* [bug#32053] [PATCH 2/2] gnu: Add quaternion.
  2018-07-04 17:46 ` [bug#32053] [PATCH 2/2] gnu: Add quaternion Arun Isaac
@ 2018-07-09 12:37   ` Ludovic Courtès
  2018-07-10  5:23     ` Arun Isaac
  0 siblings, 1 reply; 10+ messages in thread
From: Ludovic Courtès @ 2018-07-09 12:37 UTC (permalink / raw)
  To: Arun Isaac; +Cc: 32053

Hello,

Arun Isaac <arunisaac@systemreboot.net> skribis:

> * gnu/packages/messaging.scm (quaternion): New variable.

[...]

> +    (inputs
> +     `(("libqmatrixclient-source" ,(package-source libqmatrixclient))
> +       ("qt" ,qt)))
> +    (arguments
> +     `(#:tests? #f ; No tests
> +       #:phases
> +       (modify-phases %standard-phases
> +         (add-after 'unpack 'unpack-libqmatrixclient
> +           (lambda* (#:key inputs #:allow-other-keys)
> +             (invoke "tar" "-C" "lib" "-xvf"
> +                     (assoc-ref inputs "libqmatrixclient-source")
> +                     "--strip-components=1"))))))

That doesn’t sound great.  :-)

Could we arrange to have it use an externally-built libqtmatrixclient.so?
Perhaps that’ll involve modifying the CMake/Qt/whatever build files?

> +    (home-page "https://matrix.org/docs/projects/client/quaternion.html")
> +    (synopsis "Qt5 desktop client for the Matrix protocol")

What about: “Graphical client for the Matrix instant messaging protocol”?

> +    (description "Quaternion is a Qt5 desktop client for the Matrix protocol.

s/protocol/instant messaging protocol/

It’s all about giving enough context so that people know what kind of
matrix we’re talking about.  ;-)

Thank you,
Ludo’.

^ permalink raw reply	[flat|nested] 10+ messages in thread

* [bug#32053] [PATCH 2/2] gnu: Add quaternion.
  2018-07-09 12:37   ` Ludovic Courtès
@ 2018-07-10  5:23     ` Arun Isaac
  0 siblings, 0 replies; 10+ messages in thread
From: Arun Isaac @ 2018-07-10  5:23 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: 32053


>> +    (inputs
>> +     `(("libqmatrixclient-source" ,(package-source libqmatrixclient))
>> +       ("qt" ,qt)))
>> +    (arguments
>> +     `(#:tests? #f ; No tests
>> +       #:phases
>> +       (modify-phases %standard-phases
>> +         (add-after 'unpack 'unpack-libqmatrixclient
>> +           (lambda* (#:key inputs #:allow-other-keys)
>> +             (invoke "tar" "-C" "lib" "-xvf"
>> +                     (assoc-ref inputs "libqmatrixclient-source")
>> +                     "--strip-components=1"))))))
>
> That doesn’t sound great.  :-)
>
> Could we arrange to have it use an externally-built libqtmatrixclient.so?
> Perhaps that’ll involve modifying the CMake/Qt/whatever build files?

Yes, it isn't great. But I did that because that's the build process
upstream recommends. :-( I will explore alternatives and get back to
you.

>> +    (synopsis "Qt5 desktop client for the Matrix protocol")
>
> What about: “Graphical client for the Matrix instant messaging protocol”?
>
>> +    (description "Quaternion is a Qt5 desktop client for the Matrix protocol.
>
> s/protocol/instant messaging protocol/
>
> It’s all about giving enough context so that people know what kind of
> matrix we’re talking about.  ;-)

Sure, I'll make these synopsis and description changes (for both
libqmatrixclient and quaternion). In my opinion, upstream calling their
protocol "Matrix" is a somewhat poor choice. The word "Matrix" is easily
confused with many other things.

^ permalink raw reply	[flat|nested] 10+ messages in thread

* [bug#32053] [PATCH v2 0/2] Add quaternion
  2018-07-04 17:42 [bug#32053] [PATCH 1/2] gnu: Add libqmatrixclient Arun Isaac
  2018-07-04 17:46 ` [bug#32053] [PATCH 2/2] gnu: Add quaternion Arun Isaac
  2018-07-09 12:34 ` [bug#32053] [PATCH 1/2] gnu: Add libqmatrixclient Ludovic Courtès
@ 2018-07-12 10:51 ` Arun Isaac
  2018-07-12 10:51   ` [bug#32053] [PATCH v2 1/2] gnu: Add libqmatrixclient Arun Isaac
                     ` (2 more replies)
  2 siblings, 3 replies; 10+ messages in thread
From: Arun Isaac @ 2018-07-12 10:51 UTC (permalink / raw)
  To: 32053

I have made changes to the synopsis and description as you suggested. I also
patched quaternion to dynamically link to libqmatrixclient.

Arun Isaac (2):
  gnu: Add libqmatrixclient.
  gnu: Add quaternion.

 gnu/packages/messaging.scm | 75 +++++++++++++++++++++++++++++++++++++++++++++-
 1 file changed, 74 insertions(+), 1 deletion(-)

-- 
2.15.1

^ permalink raw reply	[flat|nested] 10+ messages in thread

* [bug#32053] [PATCH v2 1/2] gnu: Add libqmatrixclient.
  2018-07-12 10:51 ` [bug#32053] [PATCH v2 0/2] Add quaternion Arun Isaac
@ 2018-07-12 10:51   ` Arun Isaac
  2018-07-12 10:51   ` [bug#32053] [PATCH v2 2/2] gnu: Add quaternion Arun Isaac
  2018-07-12 14:56   ` [bug#32053] [PATCH v2 0/2] " Ludovic Courtès
  2 siblings, 0 replies; 10+ messages in thread
From: Arun Isaac @ 2018-07-12 10:51 UTC (permalink / raw)
  To: 32053

* gnu/packages/messaging.scm (libqmatrixclient): New variable.
---
 gnu/packages/messaging.scm | 28 +++++++++++++++++++++++++++-
 1 file changed, 27 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/messaging.scm b/gnu/packages/messaging.scm
index cdcd1225f..69ed7ee88 100644
--- a/gnu/packages/messaging.scm
+++ b/gnu/packages/messaging.scm
@@ -9,7 +9,7 @@
 ;;; Copyright © 2016 Andy Patterson <ajpatter@uwaterloo.ca>
 ;;; Copyright © 2016, 2017, 2018 Clément Lassieur <clement@lassieur.org>
 ;;; Copyright © 2017 Mekeor Melire <mekeor.melire@gmail.com>
-;;; Copyright © 2017 Arun Isaac <arunisaac@systemreboot.net>
+;;; Copyright © 2017, 2018 Arun Isaac <arunisaac@systemreboot.net>
 ;;; Copyright © 2017, 2018 Tobias Geerinckx-Rice <me@tobias.gr>
 ;;; Copyright © 2017 Theodoros Foradis <theodoros@foradis.org>
 ;;; Copyright © 2017 Rutger Helling <rhelling@mykolab.com>
@@ -1644,4 +1644,30 @@ c-toxcore and ncurses.  It provides audio calls, sound and desktop
 notifications, and Python scripting support.")
     (license license:gpl3+)))
 
+(define-public libqmatrixclient
+  (package
+    (name "libqmatrixclient")
+    (version "0.3.0.2")
+    (source
+     (origin
+       (method url-fetch)
+       (uri (string-append "https://github.com/QMatrixClient/libqmatrixclient/archive/v"
+                           version ".tar.gz"))
+       (file-name (string-append name "-" version ".tar.gz"))
+       (sha256
+        (base32
+         "0yl7lw0s2z98xkxbklkb3i8xcd9af9qayl1zxl070d8ykl6ayqy3"))))
+    (build-system cmake-build-system)
+    (inputs
+     `(("qt" ,qt)))
+    (arguments
+     `(#:configure-flags (list "-DBUILD_SHARED_LIBS=ON")
+       #:tests? #f)) ; No tests
+    (home-page "https://matrix.org/docs/projects/sdk/libqmatrixclient.html")
+    (synopsis "Qt5 client library for the Matrix instant messaging protocol")
+    (description "libqmatrixclient is a Qt5 library to write clients for the
+Matrix instant messaging protocol.  Quaternion is the reference client
+implementation.  Quaternion and libqmatrixclient together form the
+QMatrixClient project.")
+    (license license:lgpl2.1+)))
 ;;; messaging.scm ends here
-- 
2.15.1

^ permalink raw reply related	[flat|nested] 10+ messages in thread

* [bug#32053] [PATCH v2 2/2] gnu: Add quaternion.
  2018-07-12 10:51 ` [bug#32053] [PATCH v2 0/2] Add quaternion Arun Isaac
  2018-07-12 10:51   ` [bug#32053] [PATCH v2 1/2] gnu: Add libqmatrixclient Arun Isaac
@ 2018-07-12 10:51   ` Arun Isaac
  2018-07-12 14:56   ` [bug#32053] [PATCH v2 0/2] " Ludovic Courtès
  2 siblings, 0 replies; 10+ messages in thread
From: Arun Isaac @ 2018-07-12 10:51 UTC (permalink / raw)
  To: 32053

* gnu/packages/messaging.scm (quaternion): New variable.
---
 gnu/packages/messaging.scm | 47 ++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 47 insertions(+)

diff --git a/gnu/packages/messaging.scm b/gnu/packages/messaging.scm
index 69ed7ee88..1212a06ee 100644
--- a/gnu/packages/messaging.scm
+++ b/gnu/packages/messaging.scm
@@ -1670,4 +1670,51 @@ Matrix instant messaging protocol.  Quaternion is the reference client
 implementation.  Quaternion and libqmatrixclient together form the
 QMatrixClient project.")
     (license license:lgpl2.1+)))
+
+(define-public quaternion
+  (package
+    (name "quaternion")
+    (version "0.0.9.2")
+    (source
+     (origin
+       (method url-fetch)
+       (uri (string-append "https://github.com/QMatrixClient/Quaternion/archive/v"
+                           version ".tar.gz"))
+       (file-name (string-append name "-" version ".tar.gz"))
+       (sha256
+        (base32
+         "0s2mpw11s2n21ds1spf94j1p2lj2px2bv5zxldlcx81ch0rb4ng8"))))
+    (build-system cmake-build-system)
+    (inputs
+     `(("libqmatrixclient" ,libqmatrixclient)
+       ("qt" ,qt)))
+    (arguments
+     `(#:tests? #f ; No tests
+       #:phases
+       (modify-phases %standard-phases
+         (add-after 'unpack 'fix-libqmatrixclient-dynamic-linking
+           ;; Upstream recommends statically linking with
+           ;; libqmatrixclient. Patch the source so that we can dynamically
+           ;; link instead. In a future release, when upstream moves to
+           ;; dynamic linking, remove this phase.
+           (lambda _
+             (substitute* "CMakeLists.txt"
+               (("^add_subdirectory\\(lib\\)" all)
+                (string-append "#" all)))
+             (for-each
+              (lambda (file)
+                (substitute* file
+                  (("#include \"lib/([^\"]*)\"" all header)
+                   (string-append "#include <" header ">"))))
+              (find-files "client" "\\.(cpp|h)$"))
+             #t)))))
+    (home-page "https://matrix.org/docs/projects/client/quaternion.html")
+    (synopsis "Graphical client for the Matrix instant messaging protocol")
+    (description "Quaternion is a Qt5 desktop client for the Matrix instant
+messaging protocol.  It uses libqmatrixclient and is its reference client
+implementation.  Quaternion and libqmatriclient together form the
+QMatrixClient project.")
+    (license (list license:gpl3+ ; all source code
+                   license:lgpl3+)))) ; icons/breeze
+
 ;;; messaging.scm ends here
-- 
2.15.1

^ permalink raw reply related	[flat|nested] 10+ messages in thread

* [bug#32053] [PATCH v2 0/2] Add quaternion
  2018-07-12 10:51 ` [bug#32053] [PATCH v2 0/2] Add quaternion Arun Isaac
  2018-07-12 10:51   ` [bug#32053] [PATCH v2 1/2] gnu: Add libqmatrixclient Arun Isaac
  2018-07-12 10:51   ` [bug#32053] [PATCH v2 2/2] gnu: Add quaternion Arun Isaac
@ 2018-07-12 14:56   ` Ludovic Courtès
  2018-07-12 17:56     ` bug#32053: " Arun Isaac
  2 siblings, 1 reply; 10+ messages in thread
From: Ludovic Courtès @ 2018-07-12 14:56 UTC (permalink / raw)
  To: Arun Isaac; +Cc: 32053

Arun Isaac <arunisaac@systemreboot.net> skribis:

> I have made changes to the synopsis and description as you suggested. I also
> patched quaternion to dynamically link to libqmatrixclient.
>
> Arun Isaac (2):
>   gnu: Add libqmatrixclient.
>   gnu: Add quaternion.

Awesome, LGTM.  Thank you!

Ludo’.

^ permalink raw reply	[flat|nested] 10+ messages in thread

* bug#32053: [PATCH v2 0/2] Add quaternion
  2018-07-12 14:56   ` [bug#32053] [PATCH v2 0/2] " Ludovic Courtès
@ 2018-07-12 17:56     ` Arun Isaac
  0 siblings, 0 replies; 10+ messages in thread
From: Arun Isaac @ 2018-07-12 17:56 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: 32053-done


Pushed, thanks for the review!

^ permalink raw reply	[flat|nested] 10+ messages in thread

end of thread, other threads:[~2018-07-12 17:57 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-07-04 17:42 [bug#32053] [PATCH 1/2] gnu: Add libqmatrixclient Arun Isaac
2018-07-04 17:46 ` [bug#32053] [PATCH 2/2] gnu: Add quaternion Arun Isaac
2018-07-09 12:37   ` Ludovic Courtès
2018-07-10  5:23     ` Arun Isaac
2018-07-09 12:34 ` [bug#32053] [PATCH 1/2] gnu: Add libqmatrixclient Ludovic Courtès
2018-07-12 10:51 ` [bug#32053] [PATCH v2 0/2] Add quaternion Arun Isaac
2018-07-12 10:51   ` [bug#32053] [PATCH v2 1/2] gnu: Add libqmatrixclient Arun Isaac
2018-07-12 10:51   ` [bug#32053] [PATCH v2 2/2] gnu: Add quaternion Arun Isaac
2018-07-12 14:56   ` [bug#32053] [PATCH v2 0/2] " Ludovic Courtès
2018-07-12 17:56     ` bug#32053: " Arun Isaac

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).