munkitools5.munki.recipe munkitools_admin is an update for munkitools_core and munkitools_python?

[igor-sweet]

I think in line 333 it should an requires and not an update_for https://github.com/autopkg/recipes/blob/5db796bcc0da46f2406a125e73e2b045472b0d66/munkitools/munkitools5.munki.recipe#L333

<key>update_for</key> <array> <string>%MUNKITOOLS_PYTHON_NAME%</string> <string>%MUNKITOOLS_CORE_NAME%</string> </array>

[homebysix]

The Munki wiki says this about the difference between requires and update_for:

Items that declare (via update_for) they are updates for other items generally are not added to a manifest; Munki finds and applies them "automatically".

So this allows administrators to specify only one item — munkitools — in their manifest. All other munkitools-related items are found and included automatically (either because they are requirements for munkitools or declared as updates for a required package in munkitools).

The part I'm not super clear on is how administrators can opt out of deploying the admin tools to a specific manifest. The munkitools_admin pkginfo isn't overrideable, so I suppose one method would be for the administrator to create a duplicate of the munkitools5.munki.recipe locally.