Module doesn't work on Puppet 4 when sudo is not installed about puppet-sudo

[Falkor]

Indeed, the module has not yet been tested against puppet 4. Thanks for reporting this error.

There is a custom fact (in lib/facter/sudo.rb) that provides the ::sudoversion fact.
Could it be that you do not load custom facts prior to your instantiation of the sudo module ?

[crispygoth]

I have custom facts in other modules that are being run OK, and if I run the agent with --debug it does mention the sudo.rb fact file:

Debug: Loading facts from /var/lib/puppet/lib/facter/sudo.rb

It looks like this is a breaking change to versioncmp made in Puppet 4.3.0, which now enforces that the arguments are strings, and will therefore not accept undef as one of the arguments: PUP-4780

I guess the module either needs to check whether sudoversion is defined before trying to versioncmp it, or the custom fact should return an empty string instead of undef when sudo is not installed.