[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#41637] [PATCH] gnu: Add hashcash.
From: |
Mathieu Othacehe |
Subject: |
[bug#41637] [PATCH] gnu: Add hashcash. |
Date: |
Wed, 03 Jun 2020 08:55:39 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.3 (gnu/linux) |
Hey Jakub,
Thanks for fixing the cross-compilation, a full review this time :)
> + (let* ((outdir (assoc-ref outputs "out"))
> + (bindir (string-append outdir "/bin"))
> + (mandir (string-append outdir "/share/man/man1"))
> + (docdir (string-append outdir "/share/doc/hashcash-"
> ,version)))
> + ;; make install assumes /usr and doesn't provide a way to
> override it
> + (install-file "hashcash" bindir)
> + (install-file "hashcash.1" mandir)
> + (install-file "README" docdir)
> + (install-file "LICENSE" docdir)
> + (install-file "CHANGELOG" docdir)
I think you can set the variables INSTALL_PATH, MAN_INSTALL_PATH and
DOC_INSTALL_PATH instead.
> + #t))))))
> + (home-page "https://www.hashcash.org/")
> + (synopsis "Denial-of-service countermeasure")
> + (description "Hashcash is a proof-of-work algorithm, which has been used
> +as a denial-of-service countermeasure technique in a number of systems.
You can remove "in a number of systems".
> +A hashcash stamp constitutes a proof-of-work which takes a parametrizable
> +amount of work to compute for the sender. The recipient can verify received
> +hashcash stamps efficiently.
> +
> +This package contains a command-line tool for computing and verifying
> hashcash
> +stamps.")
> + (license license:public-domain)))
I'm also concerned by this line in the Makefile:
--8<---------------cut here---------------start------------->8---
# request static link of -lcrypto only
LIBCRYPTO=/usr/lib/libcrypto.a
--8<---------------cut here---------------end--------------->8---
We should maybe add "openssl" to the inputs and fix this variable
(that's what Nix does).
Thanks,
Mathieu