#27475 Add kashmir, crypto++, opendht-with-msgpack-c++-11, and some updates.

Message #13 received at 27475@debbugs.gnu.org (full text, mbox, reply):

Received: (at 27475) by debbugs.gnu.org; 11 Oct 2017 17:23:03 +0000
From debbugs-submit-bounces@debbugs.gnu.org Wed Oct 11 13:23:02 2017
Received: from localhost ([127.0.0.1]:34434 helo=debbugs.gnu.org)
	by debbugs.gnu.org with esmtp (Exim 4.84_2)
	(envelope-from <debbugs-submit-bounces@debbugs.gnu.org>)
	id 1e2Kik-0000Tb-F3
	for submit@debbugs.gnu.org; Wed, 11 Oct 2017 13:23:02 -0400
Received: from li622-129.members.linode.com ([212.71.249.129]:53678
 helo=mira.cbaines.net) by debbugs.gnu.org with esmtp (Exim 4.84_2)
 (envelope-from <mail@cbaines.net>) id 1e2Kii-0000T1-BG
 for 27475@debbugs.gnu.org; Wed, 11 Oct 2017 13:23:01 -0400
Received: by mira.cbaines.net (Postfix, from userid 113)
 id 5F103147628; Wed, 11 Oct 2017 18:22:58 +0100 (BST)
X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on mira.cbaines.net
X-Spam-Level: 
X-Spam-Status: No, score=-2.9 required=5.0 tests=ALL_TRUSTED,BAYES_00,
 URIBL_BLOCKED autolearn=unavailable autolearn_force=no version=3.4.0
Received: from localhost (cpc102582-walt20-2-0-cust14.13-2.cable.virginm.net
 [86.27.34.15])
 by mira.cbaines.net (Postfix) with ESMTPSA id DFD56147626;
 Wed, 11 Oct 2017 18:22:57 +0100 (BST)
Date: Wed, 11 Oct 2017 18:22:54 +0100
From: Christopher Baines <mail@cbaines.net>
To: Adonay Felipe Nogueira <adfeno@openmailbox.org>
Subject: Re: [bug#27475] Add kashmir, crypto++, opendht-with-msgpack-c++-11,
 and some updates.
Message-ID: <20171011182254.101c14ce@cbaines.net>
In-Reply-To: <8737apqrlj.fsf@openmailbox.org>
References: <8737apqrlj.fsf@openmailbox.org>
X-Mailer: Claws Mail 3.15.1-dirty (GTK+ 2.24.31; x86_64-unknown-linux-gnu)
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
 boundary="Sig_/L9proWUIEjIxDDKrLilzSt7"; protocol="application/pgp-signature"
X-Spam-Score: -0.0 (/)
X-Debbugs-Envelope-To: 27475
Cc: 27475@debbugs.gnu.org
X-BeenThere: debbugs-submit@debbugs.gnu.org
X-Mailman-Version: 2.1.18
Precedence: list
List-Id: <debbugs-submit.debbugs.gnu.org>
List-Unsubscribe: <https://debbugs.gnu.org/cgi-bin/mailman/options/debbugs-submit>, 
 <mailto:debbugs-submit-request@debbugs.gnu.org?subject=unsubscribe>
List-Archive: <https://debbugs.gnu.org/cgi-bin/mailman/private/debbugs-submit/>
List-Post: <mailto:debbugs-submit@debbugs.gnu.org>
List-Help: <mailto:debbugs-submit-request@debbugs.gnu.org?subject=help>
List-Subscribe: <https://debbugs.gnu.org/cgi-bin/mailman/listinfo/debbugs-submit>, 
 <mailto:debbugs-submit-request@debbugs.gnu.org?subject=subscribe>
Errors-To: debbugs-submit-bounces@debbugs.gnu.org
Sender: "Debbugs-submit" <debbugs-submit-bounces@debbugs.gnu.org>
X-Spam-Score: -0.0 (/)

[Message part 1 (text/plain, inline)]

On Sat, 24 Jun 2017 13:26:48 -0300
Adonay Felipe Nogueira <adfeno@openmailbox.org> wrote:

> Here follows a series of patches, it's part of the attempt to package
> GNU Ring. However, these contributions don't need custom patches from
> Savir Faire Linux. The only doubt is in regards to the need to have a
> variant of msgpack that uses C++ 11 instead, and have a variant of
> opendht that makes use of such msgpack variant instead.

I've just noticed that no one has taken a look at this yet! Not quite
sure how that has happened. Here are my comments.

The first general comment is that these patches don't quite apply now,
so it would be useful to send another set which do apply cleanly.

Starting with the kashmir package.

> +     (source (origin
> +	      (method url-fetch)
> +	      (uri (string-append "https://github.com/Corvusoft/"
> name
> +				  "-dependency/archive/" commit
> ".tar.gz"))
> +	      (file-name (string-append name "-" version ".tar.gz"))
> +	      (sha256
> +	       (base32
> +		"11hbh3d7w74gng9qgz3hdrzvxjgil1pymk2bzxv1ajh9vp60xxs9"))))

Looks like this patch may have got caught up in the recent hash changes
from GitHub. I had to change the sha256 before this would build.

The directory structure in this package looks a little odd. The Debian
package has kashmir-dependency/kashmir as include/kashmir, which might
be a little more usual.

guix lint also reports some issues:

/home/chris/.config/guix/latest/gnu/packages/cpp.scm:36:4: kashmir@0-1.2f3913f: tabulation on line 39, column 0
+ more tab issues
/home/chris/.config/guix/latest/gnu/packages/cpp.scm:36:4: kashmir@0-1.2f3913f: tabulation on line 71, column 0
/home/chris/.config/guix/latest/gnu/packages/cpp.scm:36:4: kashmir@0-1.2f3913f: line 72 is way too long (98 characters)
/home/chris/.config/guix/latest/gnu/packages/cpp.scm:36:4: kashmir@0-1.2f3913f: line 73 is way too long (176 characters)


As for the crypto++ package.

guix lint reports lots of tabs.

> +		       (add-after 'build 'build-shared
> +				  (lambda _
> +				    ;; By default, only the static
> +				    ;; library is built.
> +				    (zero?
> +				     (system* "make" "shared"))))

I think you could get rid of this phase by adding "shared" to the
#:make-flags.

I'm not sure why the binary is has .exe in the name? Even
the inbuilt help calls it cryptest:

  Unrecognized command. Run "cryptest h" to obtain usage information.

I have no idea what this does, but running it in a container fails, as
it seems to be looking for data in the doc output:

  → guix environment --container --ad-hoc crypto++:bin -- cryptest.exe
  CryptoPP::Exception caught: FileStore: error opening file for reading:
 TestData/usage.dat


On to the opendht update.

> +  (let ((release "1.3.3")
> +	(revision 5))

I'm not why this revision is being used?

Also, with the msgpack-c++-11 package, I built both msgpack and
msgpack-c++-11 and diffed the contents with diffoscope, and they are
practically identical as far as I can see. I'm guessing there was
supposed to be a difference?


Last but not least, for the argon2 package, for which I don't have any
questions or suggestions. Ignoring the tabs, I think it looks fine.

I hope this helps Adonay :)

[Message part 2 (application/pgp-signature, inline)]

Send a report that this bug log contains spam.