[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] gnu: Add darkhttpd.
From: |
Leo Famulari |
Subject: |
Re: [PATCH] gnu: Add darkhttpd. |
Date: |
Fri, 9 Sep 2016 16:41:58 -0400 |
User-agent: |
Mutt/1.7.0 (2016-08-17) |
On Sat, Sep 10, 2016 at 01:26:23AM +0530, Arun Isaac wrote:
> * gnu/packages/web.scm (darkhttpd): New variable.
Hi, thank you for this patch!
> + `(#:tests? #f
When we disable the tests, we have to leave a comment explaining why. It
can be as simple as "No test suite" if that is the case.
> + (replace 'configure
> + (lambda _ (setenv "CC" "gcc")))
This has the effect of setting the CC variable and skipping any other
'configure' actions. Is this intended? If there is no './configure'
script or similar tool, this environment variable could be set in
#:make-flags or in a 'setenv' phase.
> + (replace 'install
> + (lambda* (#:key outputs #:allow-other-keys)
> + (let ((bin (string-append (assoc-ref outputs "out") "/bin")))
> + (mkdir-p bin)
> + (copy-file "darkhttpd" (string-append bin "/darkhttpd"))))))))
mkdir-p and copy-file and be replaced with install-file.
> + (description "darkhttpd is a simple static web server. It is
> +standalone and does not need inetd or ucspi-tcp. It does not need any
> +config files -- you only have to specify the www root.")
You can make an 'em dash' in Texinfo like this:
"...config files---you only..."
> + (home-page "https://unix4lyfe.org/darkhttpd/")
> + (license l:bsd-2)))
The license on the main source file is the ISC license:
https://unix4lyfe.org/gitweb/darkhttpd/blob/HEAD:/darkhttpd.c
Can you send an updated patch?
- [PATCH] gnu: Add darkhttpd., Arun Isaac, 2016/09/09
- Re: [PATCH] gnu: Add darkhttpd.,
Leo Famulari <=
- Re: [PATCH] gnu: Add darkhttpd., Arun Isaac, 2016/09/10
- [PATCH] gnu: Add darkhttpd., Arun Isaac, 2016/09/10
- Re: [PATCH] gnu: Add darkhttpd., Alex Kost, 2016/09/10
- Re: [PATCH] gnu: Add darkhttpd., Arun Isaac, 2016/09/10
- Re: [PATCH] gnu: Add darkhttpd., Alex Kost, 2016/09/10
- Re: [PATCH] gnu: Add darkhttpd., Arun Isaac, 2016/09/11
- [PATCH] gnu: Add darkhttpd., Arun Isaac, 2016/09/11
- Re: [PATCH] gnu: Add darkhttpd., Leo Famulari, 2016/09/13
- Re: [PATCH] gnu: Add darkhttpd., Arun Isaac, 2016/09/13
- [PATCH] gnu: Add darkhttpd., Arun Isaac, 2016/09/14