[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#30702] [PATCH] services: nginx: Support extra content in the http b
From: |
Christopher Baines |
Subject: |
[bug#30702] [PATCH] services: nginx: Support extra content in the http block. |
Date: |
Sat, 17 Mar 2018 23:46:49 +0000 |
User-agent: |
mu4e 1.0; emacs 25.3.1 |
Ludovic Courtès <address@hidden> writes:
> Hello,
>
> Christopher Baines <address@hidden> skribis:
>
>> This helpful when adding content to the nginx configuration file, which isn't
>> supported by the record type used for the configuration. For example, like
>> adding proxy_cache_path configuration.
>>
>> * gnu/packages/web.scm (<nginx-configuration>): Add new extra-content field.
>> (nginx-configuration-extra-content): New field accessor.
>> (default-nginx-config): Add support for the extra-content field.
>> * doc/guix.texi (NGINX): Document the new extra-content field.
>> ---
>> doc/guix.texi | 4 ++++
>> gnu/services/web.scm | 9 +++++++--
>> 2 files changed, 11 insertions(+), 2 deletions(-)
>>
>> diff --git a/doc/guix.texi b/doc/guix.texi
>> index 057272df4..151bc7ddd 100644
>> --- a/doc/guix.texi
>> +++ b/doc/guix.texi
>> @@ -15431,6 +15431,10 @@ use the size of the processors cache line.
>> @item @code{server-names-hash-bucket-max-size} (default: @code{#f})
>> Maximum bucket size for the server names hash tables.
>>
>> address@hidden @code{extra-content} (default: @code{'()})
>> +Extra content for the @code{http} block. Should be a list of strings or
>> +G-expressions.
>
> I find it surprising that it’s a list rather than a string or
> string-valued gexp. Thoughts?
Lists are used as the type for other fields in related records. I chose
a list as it's easy to add things to, but thinking about it, that's
probably true for strings and gexps as well.
Saying it can be either a string or a gexp might be a little tricky, as
string operations wouldn't work on the gexp. My familiarity with
string-valued gexps is a little limited though, I've only just started
using file-append more, as I think that's how it works. How would you go
about adding say a string to a gexp?
signature.asc
Description: PGP signature