change how modules are enabled

[otakup0pe]

Currently we enable extra modules by specifying the full string that is passed to configure. It would be much cleaner to specify them by a simple bool. As part of this we also may wish to revisit which modules are enabled by default.

Compare the two following YAML snippets.

nginx_source_modules_included:
  http_stub_status_module: "--with-http_stub_status_module"

nginx_source_modules_included:
  http_stub_status_module: yes

[soupdiver]

Good idea! 👍
I also felt the current way isn't optimal but so far didn't thought about it much further.

[AgDude]

For modules which are part of the nginx source, what about including them with a simple list. Lots of the modules listed in the docs http://nginx.org/en/docs/ are not included by default.

nginx_source_modules:
  - http_stub_status_module
  - ngx_http_perl_module

Any of these modules are included by prepending the module name with --with- as a build option.

[soupdiver]

With the idea from @AgDude we could avoid to have an extra mapping from the bool values @otakup0pe suggested.
Like this idea