[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] improve nginx-service
From: |
Hartmut Goebel |
Subject: |
Re: [PATCH] improve nginx-service |
Date: |
Fri, 4 Nov 2016 23:58:01 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.4.0 |
Am 04.11.2016 um 23:12 schrieb Julien Lepiller:
> I'd like to get these patches accepted first, but I'm already working
> on adding more configuration lines.
BTW: I just started typing down a simple "location" record type, and
started to think if these record types are really the way to go for the
configuration.
In nginx AFAIK you can define many options on the server level, on the
vhost level and per location. Additionally locations can be nested. And
some configuration options are quite complex, e.g. "listen" [1] and
maybe worth their own type. So we'll create a lot of code for both the
record types and for converting them to config-file text.
In the long run, we will get maintenance problems, I'm afraid, if we are
only allow options known to us and written into the service. On each
update we'd need to check if any option was added.
So IMHO the recard types are not the way to go. We need a more flexible
way, maybe like used for XML (see default-build.xml in
guix/build/ant-build-system.scm), but with support for default values.
Unfortunatly I'm no scheme hacker and can not propose any solution for
this :-(
[1] https://nginx.org/en/docs/http/ngx_http_core_module.html#listen
--
Regards
Hartmut Goebel
| Hartmut Goebel | address@hidden |
| www.crazy-compilers.com | compilers which you thought are impossible |
- Re: [PATCH] improve nginx-service, Hartmut Goebel, 2016/11/02
- Re: [PATCH] improve nginx-service, Ludovic Courtès, 2016/11/03
- Re: [PATCH] improve nginx-service, Hartmut Goebel, 2016/11/03
- Re: [PATCH] improve nginx-service, Ludovic Courtès, 2016/11/04
- Re: [PATCH] improve nginx-service, Julien Lepiller, 2016/11/04
- Re: [PATCH] improve nginx-service, Hartmut Goebel, 2016/11/04
- Re: [PATCH] improve nginx-service, Julien Lepiller, 2016/11/04
- Re: [PATCH] improve nginx-service, Hartmut Goebel, 2016/11/04
- Re: [PATCH] improve nginx-service, Julien Lepiller, 2016/11/06
- Re: [PATCH] improve nginx-service,
Hartmut Goebel <=
- Re: [PATCH] improve nginx-service, Tobias Geerinckx-Rice, 2016/11/06
- Re: [PATCH] improve nginx-service, Ludovic Courtès, 2016/11/06
- Re: [PATCH] improve nginx-service, Julien Lepiller, 2016/11/20
- Re: [PATCH] improve nginx-service, Hartmut Goebel, 2016/11/22
- Re: [PATCH] improve nginx-service, julien lepiller, 2016/11/23
- Re: [PATCH] improve nginx-service, Clément Lassieur, 2016/11/25
- is using eval good style in guile?(was: [PATCH] improve nginx-service), Hartmut Goebel, 2016/11/25
- Re: is using eval good style in guile?, Andy Wingo, 2016/11/25
- Re: is using eval good style in guile?, Clément Lassieur, 2016/11/26
- Re: [PATCH] improve nginx-service, Ludovic Courtès, 2016/11/27