keeleysam-recipes icon indicating copy to clipboard operation
keeleysam-recipes copied to clipboard

Autopkg 2.7 Google Talk Plugin - Munki Pkg info Receipts Editor Python 3.10 error

Open lifeunexpected opened this issue 2 years ago • 6 comments

The following failures occurred: Recipe: local.munki.Override.MicrosoftOffice365Suite

Message: Unable to import 'com.github.keeleysam.recipes.GoogleTalkPlugin/MunkiPkginfoReceiptsEditor', likely due to syntax or Python error.

This happened after upgrading to the new version of autopkg

AutoPkg now uses Python 3.10.6, and PyObjc 8.5. Python 3.10 brings several new general improvements and may result in some warnings or errors in processors still using some legacy Python2-conversion-to-3-isms. Please file appropriate issues with recipe authors or bring it to our attention in #autopkg in Slack. https://github.com/autopkg/autopkg/releases/tag/v2.7

lifeunexpected avatar Aug 25 '22 07:08 lifeunexpected

same for TeamViewerHostCustom

TomBoehm avatar Aug 26 '22 08:08 TomBoehm

A bunch of recipes we use rely on this processor. I hope @vmiller's fix can be merged soon (thanks!).

flammable avatar Aug 30 '22 16:08 flammable

I also hope this can be merged soon. Affects: com.github.foigus.munki.microsoftremotedesktop com.github.dataJAR-recipes.munki.Microsoft Word 2019

precursorca avatar Sep 06 '22 11:09 precursorca

I'm experiencing this with the apizz-recipe for GoogleDrive.munki, wherein it can't find [read,write]Plist in plistlib module for autopkg version calling Python 3.10 out of ~/Library/Autopkg/Python3/Python.frameworks/Version/3.10/bin. My sol'n was to change [read,write]Plist into the methods used in 3.10 ('load' and 'dump') in MunkiPkginfoReceiptsEditor.py. I then ran autopkg update-trust-info for GoogleDrive.munki and now it's working.

mdt1024 avatar Sep 09 '22 20:09 mdt1024

This processor was merged into core - MunkiOptionalReceiptsEditor. You don't need to use this specific shared processor anymore.

nmcspadden avatar Sep 10 '22 00:09 nmcspadden

Nice. Makes a lot of sense for this to be a core processor. Sounds like we need to file issues/pull requests to some recipes to use the new core processor.

vmiller avatar Sep 10 '22 11:09 vmiller