[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#60201] [PATCH] gnu: Add libswell
From: |
Ricardo Wurmus |
Subject: |
[bug#60201] [PATCH] gnu: Add libswell |
Date: |
Mon, 02 Jan 2023 12:32:19 +0100 |
User-agent: |
mu4e 1.8.13; emacs 28.2 |
Hi,
thank you for the patch!
Could you add a comment why tests are disabled?
I recommend using a gexp for the phases:
(arguments
(list
;; There are no tests.
#:tests? #false
#:phases
#~(modify-phases … )))
Then you can use #$output instead of binding the ‘outputs’ keyword
argument and looking up the output with label ‘out’.
Is it really enough to just install the .so? Shouldn’t the headers also
be installed?
Why are origin and commit separated as a global definition? That seems
unnecessary.
--
Ricardo
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- [bug#60201] [PATCH] gnu: Add libswell,
Ricardo Wurmus <=