This one is a dead variable.  I'll kill it and push.

This is a good place to bring up $version and $relversion in params.pp however.  For Redhat we need to pass in both $version and $relversion to get the appropriate package and Debian only needs $version.  I was thinking of potentially doing something like $version becomes an array and so redhat gets version => ['3.1.3', '1'] and we do the appropriate thing based on that.  Mostly just because it feels ugly to have a variable that is redhat specific.

Does that seem better, or worse?