recipes icon indicating copy to clipboard operation
recipes copied to clipboard

Use of "munki_repo_changed" to detect the need of makecatalog

Open choules opened this issue 2 years ago • 1 comments

The MakeCatalogsProcessor checks the AutoPKG results, if a MunkiImporter has imported an item: https://github.com/autopkg/recipes/blob/b575ed8e1621b8bf88a3a847d9792c2c1a0b85a7/Munki/MakeCatalogsProcessor.py#L75

Since "import" is not the only action which may modify the repository, it would be great to check for the processor output munki_repo_changed, which seems to be more generic.

Background: We are currently working on a processor, which may modify some pkginfo files, even though no new items have been imported. It would be great to have the option to rebuild the catalogs using the MakeCatalogs.munki recipe afterwards.

I would be happy to provide a pull request, if this has a chance to be integrated.

choules avatar Mar 09 '22 16:03 choules

That's a neat idea and I'd love to see your implementation in a PR.

On Wed, Mar 9, 2022 at 8:38 AM Matthias Choules @.***> wrote:

The MakeCatalogsProcessor checks the AutoPKG results, if a MunkiImporter has imported an item:

https://github.com/autopkg/recipes/blob/b575ed8e1621b8bf88a3a847d9792c2c1a0b85a7/Munki/MakeCatalogsProcessor.py#L75

Since "import" is not the only action which may modify the repository, it would be great to check for the processor output "munki_repo_changed", which seems to be more generic.

Background: We are currently working on a processor, which may modify some pkginfo files, even though no new items have been imported. It would be great to have the option to rebuild the catalogs using the MakeCatalogs.munki recipe afterwards.

I would be happy to provide a pull request, if this has a chance to be integrated.

— Reply to this email directly, view it on GitHub https://github.com/autopkg/recipes/issues/418, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAJFTX7PWERGWYGHVXPC3ATU7DHYLANCNFSM5QKCMAGA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you are subscribed to this thread.Message ID: @.***>

--

Nick McSpadden @.***

nmcspadden avatar Mar 09 '22 18:03 nmcspadden