unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Lukas Gradl <lgradl@openmailbox.org>
To: Efraim Flashner <efraim@flashner.co.il>
Cc: guix-devel@gnu.org
Subject: Re: [PATCH] Add msgpack
Date: Thu, 02 Jun 2016 19:50:58 -0500	[thread overview]
Message-ID: <87y46nyu19.fsf@openmailbox.org> (raw)
In-Reply-To: <20160601044947.GB20978@debian-netbook> (Efraim Flashner's message of "Wed, 1 Jun 2016 07:49:47 +0300")

[-- Attachment #1: Type: text/plain, Size: 2823 bytes --]


Hi,

Thank you for your review!

Efraim Flashner <efraim@flashner.co.il> writes:

> On Mon, May 30, 2016 at 03:50:41PM -0500, Lukas Gradl wrote:
>> 
>> Hello Guix,
>> 
>> Attached is a patch for the c/c++ version of msgpack.  This is a
>> dependency of the Ring.
>> 
>> Thank you!
>> 
>
>> From 25eef52146bc84e83d90e429a0a2a5ca607280bc Mon Sep 17 00:00:00 2001
>> From: Lukas Gradl <lgradl@openmailbox.org>
>> Date: Mon, 30 May 2016 15:46:29 -0500
>> Subject: [PATCH] gnu: serialization: Add msgpack.
>> 
>> * gnu/packages/serialization.scm (msgpack): New variable.
>> ---
>>  gnu/packages/serialization.scm | 31 +++++++++++++++++++++++++++++++
>>  1 file changed, 31 insertions(+)
>> 
>> diff --git a/gnu/packages/serialization.scm b/gnu/packages/serialization.scm
>> index 8dfd21d..524754b 100644
>> --- a/gnu/packages/serialization.scm
>> +++ b/gnu/packages/serialization.scm
>> @@ -1,5 +1,6 @@
>>  ;;; GNU Guix --- Functional package management for GNU
>>  ;;; Copyright © 2015 Ricardo Wurmus <rekado@elephly.net>
>> +;;; Copyright © 2016 Lukas Gradl <lgradl@openmailbox.org>
>>  ;;;
>>  ;;; This file is part of GNU Guix.
>>  ;;;
>> @@ -21,7 +22,11 @@
>>    #:use-module (guix packages)
>>    #:use-module (guix download)
>>    #:use-module (guix build-system cmake)
>> +  #:use-module (guix build-system gnu)
>>    #:use-module (gnu packages)
>> +  #:use-module (gnu packages autotools)
>> +  #:use-module (gnu packages check)
>> +  #:use-module (gnu packages compression)
>>    #:use-module (gnu packages documentation))
>>  
>>  (define-public cereal
>> @@ -72,3 +77,29 @@
>>  arbitrary data types and reversibly turns them into different representations,
>>  such as compact binary encodings, XML, or JSON.")
>>      (license license:bsd-3)))
>> +
>> +
>> +(define-public msgpack
>> +  (package
>> +    (name "msgpack")
>> +    (version "1.4.1")
>> +    (source
>> +     (origin
>> +       (method url-fetch)
>> +       (uri
>> +        (string-append
>> +         "https://github.com/msgpack/msgpack-c/releases/download/"
>> +         "cpp-1.4.1/msgpack-" version ".tar.gz"))
>                 ^^^
>             "cpp-" version "/msgpack-"

Oops! The attached patch fixes that.

>
>> +       (sha256
>> +        (base32
>> +         "0bpjfh9vz0n2k93mph3x15clmigkgs223xfn8h12ymrh5gsi5ica"))))
>> +    (build-system gnu-build-system)
>> +    (native-inputs
>> +     `(("googletest" ,googletest)))
>> +    (inputs
>> +     `(("zlib" ,zlib)))
>> +    (home-page "http://www.msgpack.org")
>> +    (synopsis "Binary serialization library")
>> +    (description "Msgpack is a library for C/C++ that implements binary
>> +serialization.")
>> +    (license license:boost1.0)))
>> -- 
>> 2.7.4
>> 

Thank you!

Best,
Lukas



[-- Attachment #2: 0001-gnu-Add-msgpack.patch --]
[-- Type: text/x-patch, Size: 2166 bytes --]

From e079729d0d2318f218115dfeb046c09a970ba355 Mon Sep 17 00:00:00 2001
From: Lukas Gradl <lgradl@openmailbox.org>
Date: Thu, 2 Jun 2016 19:46:34 -0500
Subject: [PATCH] gnu: Add msgpack.

* gnu/packages/serialization.scm (msgpack): New variable.
---
 gnu/packages/serialization.scm | 31 +++++++++++++++++++++++++++++++
 1 file changed, 31 insertions(+)

diff --git a/gnu/packages/serialization.scm b/gnu/packages/serialization.scm
index 8dfd21d..0f65a00 100644
--- a/gnu/packages/serialization.scm
+++ b/gnu/packages/serialization.scm
@@ -1,5 +1,6 @@
 ;;; GNU Guix --- Functional package management for GNU
 ;;; Copyright © 2015 Ricardo Wurmus <rekado@elephly.net>
+;;; Copyright © 2016 Lukas Gradl <lgradl@openmailbox.org>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -21,7 +22,11 @@
   #:use-module (guix packages)
   #:use-module (guix download)
   #:use-module (guix build-system cmake)
+  #:use-module (guix build-system gnu)
   #:use-module (gnu packages)
+  #:use-module (gnu packages autotools)
+  #:use-module (gnu packages check)
+  #:use-module (gnu packages compression)
   #:use-module (gnu packages documentation))
 
 (define-public cereal
@@ -72,3 +77,29 @@
 arbitrary data types and reversibly turns them into different representations,
 such as compact binary encodings, XML, or JSON.")
     (license license:bsd-3)))
+
+
+(define-public msgpack
+  (package
+    (name "msgpack")
+    (version "1.4.1")
+    (source
+     (origin
+       (method url-fetch)
+       (uri
+        (string-append
+         "https://github.com/msgpack/msgpack-c/releases/download/"
+         "cpp-" version "/msgpack-" version ".tar.gz"))
+       (sha256
+        (base32
+         "0bpjfh9vz0n2k93mph3x15clmigkgs223xfn8h12ymrh5gsi5ica"))))
+    (build-system gnu-build-system)
+    (native-inputs
+     `(("googletest" ,googletest)))
+    (inputs
+     `(("zlib" ,zlib)))
+    (home-page "http://www.msgpack.org")
+    (synopsis "Binary serialization library")
+    (description "Msgpack is a library for C/C++ that implements binary
+serialization.")
+    (license license:boost1.0)))
-- 
2.7.4


  reply	other threads:[~2016-06-03  0:51 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-05-30 20:50 [PATCH] Add msgpack Lukas Gradl
2016-05-31 21:45 ` Leo Famulari
2016-06-01  4:49 ` Efraim Flashner
2016-06-03  0:50   ` Lukas Gradl [this message]
2016-06-06 14:07 ` Leo Famulari
2016-06-11  3:06   ` Leo Famulari
2016-06-11 23:56     ` Lukas Gradl
2016-06-12  0:17       ` Leo Famulari
2016-06-12  4:24         ` Lukas Gradl
2016-06-13 16:58           ` Leo Famulari
2016-06-13 17:59             ` Leo Famulari
2016-06-17 15:13               ` Lukas Gradl
2016-06-19  3:44                 ` Lukas Gradl
2016-06-20 17:09                   ` Leo Famulari
2016-06-21 13:55                     ` Lukas Gradl
2016-06-21 16:06                       ` Leo Famulari
2016-06-21 15:59                     ` Lukas Gradl
2016-06-21 16:49                       ` Efraim Flashner
2016-06-22  6:05                         ` Lukas Gradl
2016-06-22  5:56                       ` Lukas Gradl
2016-06-21  8:38                   ` Ludovic Courtès
2016-06-21 13:31                     ` Lukas Gradl
2016-06-21 14:11                       ` Ludovic Courtès
2016-06-25 15:31                 ` Leo Famulari
2016-06-25 16:40                   ` Lukas Gradl

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://guix.gnu.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87y46nyu19.fsf@openmailbox.org \
    --to=lgradl@openmailbox.org \
    --cc=efraim@flashner.co.il \
    --cc=guix-devel@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).