Do we need to conditionally load semantic_puppet anymore?

[binford2k]

Do we need this reimplementation? 923f567

[bastelfreak]

I don't see a reason to keep it. Do you want to revert the old commit?

[ekohl]

I don't see why. The commit you refer to comes from a time when Puppet had vendored SemanticPuppet and you could only load one or the other. The vendored implementation has been removed and it's a proper gem now. I thought it was Puppet 4.6, but I see puppetlabs/puppet@429b8bd was only in 6.0.0. Still, that's new enough.