Skip to content

Commit

Permalink
Fix for PR 845
Browse files Browse the repository at this point in the history
`use_optional_includes` should only be used for `additional_includes`,
otherwise things don't work.
  • Loading branch information
Morgan Haskel committed Feb 11, 2015
1 parent 8e5d051 commit 1581972
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -1005,7 +1005,7 @@ Setting `add_listen` to 'false' stops the vhost from creating a Listen statement

#####`use_optional_includes`

Specifies if for apache > 2.4 it should use IncludeOptional instead of Include.
Specifies if for apache > 2.4 it should use IncludeOptional instead of Include for `additional_includes`. Defaults to 'false'.

#####`additional_includes`

Expand Down
2 changes: 1 addition & 1 deletion templates/httpd.conf.erb
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,7 @@ LogFormat "<%= format -%>" <%= nickname %>
<%- end -%>
<% end -%>

<%- if scope.function_versioncmp([@apache_version, '2.4']) >= 0 && @use_optional_includes -%>
<%- if scope.function_versioncmp([@apache_version, '2.4']) >= 0 -%>
IncludeOptional "<%= @confd_dir %>/*.conf"
<%- else -%>
Include "<%= @confd_dir %>/*.conf"
Expand Down

0 comments on commit 1581972

Please sign in to comment.