ansible/ansible — pull request #1641 by eest (2012-11-17 22:09:34+00:00)
Title: Refactor module to use subclasses like user module.

* Basically the moving parts from the original service module arranged in
subclasses.
* General structure and helper methods comes from the user module.
* Less forgiving to unsupported platforms: it requires a subclass per platform.
(This makes it easier to work on one platform without having to think about.
what other platform might be affected in unexpected ways).
* Now has basic OpenBSD support.
* Solaris support needs to be added.

Thanks to @dhozac for general advice and Linux testing.
Thanks to @bcoca for clearing up some FreeBSD questions.

-------------------------

Please review this, as far as I know it has been tested on Ubuntu 12.04, OpenBSD 5.2, FreeBSD 8.2 Fedora 15 and CentOS.

There are still some problems with the FreeBSD enable stuff, but it seems to be the same problems in the old version of the module and I'm looking into it.

Also, this version breaks Solaris support. I wanted to take a look at adding a proper subclass making it work, but the form that was needed to be filled in to get hold of the installation media stopped me from doing so :(. Maby @romeotheriault could take a look if the rework is considered good enough?