(MODULES-8272) - Revert "Autorequire dirmngr in apt_key types"

[eimlav]

A bug has been introduced through this commit which creates a dependency cycle in Debian 9 when applying a simple manifest. This reverts commit 53ea6e7.

[ekohl]

I'm very disappointed that as the original author of the patch I wasn't CCed.

AFAIK the current code (after reverting) doesn't properly ensure dirmngr is actually installed when it's needed. There isn't a test case that verifies the regression isn't re-introduced when the dependency is actually added so I'm wondering if this isn't actually a bug in how it's called.

[eimlav]

@ekohl Apologies for not notifying you of the change. Unfortunately, this change was causing dependency cycles in Debian 9 and notable failures in PuppetDB. We decided that the best course of action was to remove the change and do a patch release as a large number of users depend on PuppetDB. However in saying that, we are still aware that this is a present issue with the module and it will be investigated in the foreseeable future (associated ticket).