(#19112) deploy manpages in -common package

This mirrors Debian's packaging and allows upgrades from their packages to
ours.
(PE-158) speed license check

Prior to this commit, the CertificateAuthority did not expose any
methods that allow Puppet Enterprise license checking to not require
reading the certificate files twice (once for `list` and again for
`verify`).  This commit allows the CertificateAuthority to list
certificates as objects and then, in turn, be able to issue a judgement
of whether or not these certificate objects are valid/live.
Merge pull request #1647 from jeffweiss/pe158_license_checking_too_slow

(PE-158) speed license check
Merge pull request #1688 from DavidS/fix-19112

(#19112) deploy manpages in -common package
(#14652) Add check for self-dependency cycles

With some changes that were made to how the graph is processed, puppet
2.7.0 lost the ability to detect dependency cycles where a resource
requires itself. Instead of erroring, the resource ended up being
silently ignored. This restores the behavior that existed in 2.6 where
this was considered a cyclic dependency.

The erroneous code had considered any single vertex, strongly connected
component to not be a cycle. This was done because by definition any
single vertex is a strongly connected component, no matter what edges
are involved. By always excluding those vertices, the case of a
self-referencing vertex was not handled. This adds a check for a single
vertex SCC that is adjacent to itself in order to find dependency cycles
of a single resource.

Paired-with: Josh Cooper <josh@puppetlabs.com>
(Maint) Remove unneeded checks for no error raised

There is no need to check for no error being raised because the test
would fail if any error is raised.