Some Debian support for Apache

[jmnavarrol]

A bit of Debian support added:

• Supporting files get directly under /var/www/webalizer
• The web basedir is /var/www/webalizer instead of /var/www/usage
• Web alias becomes accordingly /webalizer instead of /usage
• Apache config template gets modified along changes above.

[traylenator]

Am less keen on direct reference to $webalizer::params::apache_conffile in the manifest. Pass in apache_conffile like the other vars so that they it can be set optionally via hiera.

[jmnavarrol]

Fair enough: I already changed it so the apache config file is a param for this class (with default from webalizer::params).

[traylenator]

@jmnavarrol - I should have merged this before doing a new release. Please rebase and I'll look quickly this time.

[jmnavarrol]

@traylenator

Please rebase and I'll look quickly this time.

Done. Please have a look at it.

[traylenator]

I set the default be redhat since it's easier to test and it's good style to specify a default
rather than just a compilation fail mid flight

Can you look at the tests failing. It's just looks to be some white space.

THanks.

[jmnavarrol]

@traylenator:

Can you look at the tests failing. It's just looks to be some white space.

These seem to be the errors (just a sample):

manifests/config.pp - ERROR: tab character found on line 11
manifests/config.pp - ERROR: two-space soft tabs not used on line 11

Yes, I indent using tabs instead of puppetlabs-suggested two spaces. It is an on purpose decision that I don't think I'm going to change in the near future, sorry.

[jmnavarrol]

...in the meantime, I updated my master branch so the module can manage multiple webalizer conf files.

You might want to have a look at it.