search

netmanagers / puppet-nginx

[UNMAINTAINED] Puppet Nginx module
http://www.example42.com
Other
37 stars 60 forks source link

De-duplicating disable_default_site vs. config_file_default_purge #49

Closed erik-smit closed 10 years ago

erik-smit commented 10 years ago

De-duplicating disable_default_site vs. config_file_default_purge.

And also remove default from sites-available on debian-alikes, otherwise this param is non-functional due to https://github.com/netmanagers/puppet-nginx/blob/master/manifests/init.pp#L311

javierbertoli commented 10 years ago

Erik, thanks for this patch. It's a little backward incompatible as it removes a parameter, but I think it's worth merging as it cleans up nicely the API. Thanks again!