-
Notifications
You must be signed in to change notification settings - Fork 40
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
updated Horizon API calls to match Omnissa changes #104
Conversation
Tested and working. Note the version variable value also changed to be just the number.
Hi, fails for me at CodeSignatureVerifier, input_path has the old pkg name. btw. this needs some more changes in other repos/recipes |
Not sure how I missed that. Updated commit. |
thanks, looks good to me |
not sure the correct way to go about this, but I submitted a pull request to your repo @smaddock to update the munki recipe that will depend on this pull request. I also added some additional Omnissa rebranding changes including renaming the folder and recipe file names |
Changing the filenames and recipe identifiers will break existing dependencies with no indication of why. It is of course up to the maintainers of this repo, but I would suggest instead adding new recipes with the new name, and adding a deprecation notice to this one advising people to switch, which will keep recipes working while providing a path forward. |
My vote would be new recipes for Omnissa Horizon Client, and deprecate the VMware ones. I also don't think @scriptingosx is particularly active with AutoPkg these days, so may welcome another interested party taking over the maintenance? |
I added some based off this PR and the Munki recipe: https://github.com/autopkg/dataJAR-recipes/tree/master/Omnissa%20Horizon%20Client |
Added a deprecation notice to my PR, and then noticed @paul-cossey made a better PR. :) Going to close this one. |
Tested and working. Note the version variable value also changed to be just the number.