Re: [PATCH] gnu: Add darkhttpd.

2016-09-15 Thread Leo Famulari
On Thu, Sep 15, 2016 at 11:47:16AM +0530, Arun Isaac wrote: > > > I found some tests! See attached. Part of the test suite wants some > > static library from Clang / LLVM; I couldn't make that work in 5 minutes > > so I gave up ;) I think it's okay to commit in this state. > > Yeah, the darkhttpd

Re: [PATCH] gnu: Add darkhttpd.

2016-09-14 Thread Arun Isaac
> I found some tests! See attached. Part of the test suite wants some > static library from Clang / LLVM; I couldn't make that work in 5 minutes > so I gave up ;) I think it's okay to commit in this state. Yeah, the darkhttpd git repo has some test cases, but the release tarball does not. Since w

Re: [PATCH] gnu: Add darkhttpd.

2016-09-14 Thread Leo Famulari
ants some static library from Clang / LLVM; I couldn't make that work in 5 minutes so I gave up ;) I think it's okay to commit in this state. What do you think? >From 8268e04b18292047ff1922a151ad7458b443b9a0 Mon Sep 17 00:00:00 2001 From: Arun Isaac Date: Wed, 14 Sep 2016 11:31:59 +

[PATCH] gnu: Add darkhttpd.

2016-09-13 Thread Arun Isaac
* gnu/packages/web.scm (darkhttpd): New variable. --- gnu/packages/web.scm | 38 ++ 1 file changed, 38 insertions(+) diff --git a/gnu/packages/web.scm b/gnu/packages/web.scm index 20c7d12..2b73446 100644 --- a/gnu/packages/web.scm +++ b/gnu/packages/web.scm @@

Re: [PATCH] gnu: Add darkhttpd.

2016-09-13 Thread Arun Isaac
> If I understand correctly, this will not work as expected. When the > master branch gets another commit, our package's hash will become > invalid. Oops! I didn't think of that. I'll fix it. > Did you notice if there is any documentation that could be installed too? There's only a README file.

Re: [PATCH] gnu: Add darkhttpd.

2016-09-13 Thread Leo Famulari
On Mon, Sep 12, 2016 at 12:50:14AM +0530, Arun Isaac wrote: > * gnu/packages/web.scm (darkhttpd): New variable. Thanks for this patch! > +(version "1.12") > +(source > + (origin > + ;; The darkhttpd release tarball URL fails to download with a > + ;; 'TLS warning alert'. D

[PATCH] gnu: Add darkhttpd.

2016-09-11 Thread Arun Isaac
* gnu/packages/web.scm (darkhttpd): New variable. --- gnu/packages/web.scm | 38 ++ 1 file changed, 38 insertions(+) diff --git a/gnu/packages/web.scm b/gnu/packages/web.scm index 20c7d12..2b73446 100644 --- a/gnu/packages/web.scm +++ b/gnu/packages/web.scm @@

Re: [PATCH] gnu: Add darkhttpd.

2016-09-11 Thread Arun Isaac
> No, no, I don't insist; this problem shouldn't block packaging, > git-fetch will be OK, but I think there should be a comment explaining > why the tarball is not used. Sure, I'll send a new patch shortly with all suggested changes incorporated. signature.asc Description: PGP signature

Re: [PATCH] gnu: Add darkhttpd.

2016-09-11 Thread Ludovic Courtès
Arun Isaac skribis: >> Why 'git-fetch'? There is a release tarball: >> >> https://unix4lyfe.org/darkhttpd/darkhttpd-1.12.tar.bz2 > > Yes, there is a release tarball. But, there is some server side issue > with the url. When I try to download it with wget, I get a "The server > name sent was not

Re: [PATCH] gnu: Add darkhttpd.

2016-09-10 Thread Alex Kost
Arun Isaac (2016-09-10 18:00 +0530) wrote: >> Why 'git-fetch'? There is a release tarball: >> >> https://unix4lyfe.org/darkhttpd/darkhttpd-1.12.tar.bz2 > > Yes, there is a release tarball. But, there is some server side issue > with the url. When I try to download it with wget, I get a "The serve

Re: [PATCH] gnu: Add darkhttpd.

2016-09-10 Thread Arun Isaac
> Why 'git-fetch'? There is a release tarball: > > https://unix4lyfe.org/darkhttpd/darkhttpd-1.12.tar.bz2 Yes, there is a release tarball. But, there is some server side issue with the url. When I try to download it with wget, I get a "The server name sent was not recognized" warning. wget ignor

Re: [PATCH] gnu: Add darkhttpd.

2016-09-10 Thread Alex Kost
Arun Isaac (2016-09-10 12:21 +0530) wrote: > * gnu/packages/web.scm (darkhttpd): New variable. > --- > gnu/packages/web.scm | 33 + > 1 file changed, 33 insertions(+) > > diff --git a/gnu/packages/web.scm b/gnu/packages/web.scm > index 20c7d12..287c389 100644 > ---

[PATCH] gnu: Add darkhttpd.

2016-09-09 Thread Arun Isaac
* gnu/packages/web.scm (darkhttpd): New variable. --- gnu/packages/web.scm | 33 + 1 file changed, 33 insertions(+) diff --git a/gnu/packages/web.scm b/gnu/packages/web.scm index 20c7d12..287c389 100644 --- a/gnu/packages/web.scm +++ b/gnu/packages/web.scm @@ -14,6

Re: [PATCH] gnu: Add darkhttpd.

2016-09-09 Thread Arun Isaac
> Can you send an updated patch? Sure, I'll make the changes and send an updated patch soon.

Re: [PATCH] gnu: Add darkhttpd.

2016-09-09 Thread Leo Famulari
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. >

[PATCH] gnu: Add darkhttpd.

2016-09-09 Thread Arun Isaac
* gnu/packages/web.scm (darkhttpd): New variable. --- gnu/packages/web.scm | 34 ++ 1 file changed, 34 insertions(+) diff --git a/gnu/packages/web.scm b/gnu/packages/web.scm index 20c7d12..261eed2 100644 --- a/gnu/packages/web.scm +++ b/gnu/packages/web.scm @@ -14,