rsyslog has an unfortunate regex parser, apparently
Merge pull request #4 from sschneid/fix_regex

rsyslog has an unfortunate regex parser, apparently
Cleaner support for Fedora, incl. 16 & 17
Merge pull request #5 from sschneid/master

Cleaner support for Fedora, incl. 16 & 17
Pass $::lsbdistmajrelease to RedHat tests

The addition of EL5 support in ae6427a added a nested case statement for the
`lsbdistmajrelease` fact when `$::osfamily == 'RedHat'`. Not passing this
fact in during spec tests caused no variables to be set and tests for RH
were failing.
Change the way that unsupported OSs are caught

The `$::osfamily` case statement in `rsyslog::params` was not catching
fall-throughs from the nested case statements that were introduced for
`$::osfamily == 'RedHat'`. This results in the confusing error:

    Duplicate declaration: File[undef] is already declared

Try to catch these afterwards by testing for unassigned variables, rather
than litter all of the case statements with duplicate default conditions.
Includes the appropraite specs.

NB: This params class still won't work for users of Facter < 1.6.2 which
didn't have an `osfamily` fact.
Restart service when configs are removed

Previously the configs were removed from `/etc/rsyslog.d` but the `rsyslogd`
service wasn't restarted, causing any config options to remain loaded.
Move class relationships into parent class

De-duplicate `require` and `notify` parameters by moving them into the
parent class where each sub-class is instantiated. A side-effect of which is
that new resources within those classes won't forget to notify/require the
relevant sub-class.
Add $remote_conf to other params cases

Added to `debian` in fd37da9. Other OSs should support this too, and it will
make merge conflicts from master easier to resolve.
Merge remote-tracking branch 'upstream/master' into split_remote

Bringing up-to-date with puppetlabs-operations/rsyslog so that we can apply
some fixes that we have submitted upstream.

Conflicts:
	- manifests/params.pp
    fd37da9 and fd37da9 applied to refactored OS stanzas.
	- templates/350-server.conf.erb
    Regex change discarded for 4928525.
Merge branch 'fix_tests_for_redhat' into split_remote

Submitted upstream as:
  - https://github.com/puppetlabs-operations/puppet-rsyslog/pull/6
Merge branch 'class_dependencies' into split_remote

Submitted upstream as:
  - https://github.com/puppetlabs-operations/puppet-rsyslog/pull/7

Conflicts:
	- manifests/service.pp
    Omitted `enable` param per c89fd61.
Merge pull request #2 from alphagov/split_remote_upstream_updates

Fix tests, restart on removal, update from upstream