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 | h.goe...@crazy-compilers.com | | www.crazy-compilers.com | compilers which you thought are impossible |
