[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#30404] [PATCH] Add cat-avatar-generator-service.
From: |
Ludovic Courtès |
Subject: |
[bug#30404] [PATCH] Add cat-avatar-generator-service. |
Date: |
Fri, 09 Feb 2018 23:19:17 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.3 (gnu/linux) |
Julien Lepiller <address@hidden> skribis:
> By the way, if you don't know the author of this package, he's awesome:
> https://www.peppercarrot.com/
Looks very nice indeed!
> From 5375bd9633cb37b3f26d16742fdc451f3296f723 Mon Sep 17 00:00:00 2001
> From: Julien Lepiller <address@hidden>
> Date: Fri, 12 Jan 2018 23:08:50 +0100
> Subject: [PATCH 1/2] gnu: Add cat-avatar-generator.
>
> * gnu/packages/web.scm (cat-avatar-generator): New variable.
LGTM.
> From b5e362ac44bf3bd284ba14e886960311b67a4019 Mon Sep 17 00:00:00 2001
> From: Julien Lepiller <address@hidden>
> Date: Fri, 12 Jan 2018 23:14:14 +0100
> Subject: [PATCH 2/2] gnu: Add cat-avatar-generator-service.
>
> * gnu/services/web.scm (cat-avatar-generator-service): New variable.
[...]
> +(define* (cat-avatar-generator-service
> + #:key
> + (cache-dir "/var/cache/cat-avatar-generator")
> + (configuration (nginx-server-configuration)))
> + (simple-service 'cat-http-server nginx-service-type
> + (list (nginx-server-configuration
Normally the 3rd argument would be aligned with the 1st here.
You’ll hate me for asking that, but could you add a couple of lines to
the manual about this service?
Otherwise LGTM!
Thanks,
Ludo’.