autopkg / hjuutilainen-recipes

Recipes for AutoPkg
MIT License
49 stars 74 forks source link

Update 1Password8.munki.recipe #282

Closed swy closed 1 month ago

swy commented 1 month ago

Current archive creation method omits the .app in the zip filename, postinstall does not rm as desired. This updates to match.

hjuutilainen commented 1 month ago

Thanks for the PR! Could you change this so that the original .app.zip check stays as it is and instead add a whole new check with an if statement and path /Applications/1Password 7.zip?

swy commented 1 month ago

Sure, makes sense to expand the scope or removals vs changing it.

swy commented 1 month ago

Sorry, prematurely closed. I didn't realize my latest changes track here, thought they needed to be done separately.

hjuutilainen commented 1 month ago

Thank you!