Whilst comparing these as strings appears to work fine in most cases, I think it'd be safer to use a library. Particularly because we'd have no visibility if the behaviour changed - the tests inside wouldn't run or fail.

How about the following instead? The uses the same logic as Puppet's [versioncmp()](http://docs.puppetlabs.com/references/latest/function.html#versioncmp) and doesn't require an additional `require` (like `Gem::Version` might on Ruby 1.8):
```ruby
if Puppet::Util::Package.versioncmp(Puppet.version, '3.2') >= 0
```