-
Notifications
You must be signed in to change notification settings - Fork 96
Conversation
@mitchellh Can we merge this pull request, and release a new version. |
@LuisUrr Are you building it from the source to run this version? |
The output in the above description comes from using the binary following a local |
@LuisUrr It seems that notarytool does not take in the password as altool did allow to pass in the password using I believe we might need to update the code in Right now if I use That said, I go see tthe output you shared above, and it somehow says that it is able to accept |
Notarytool takes this |
They are correct, as it works when I pass them directly. Try the following
⤴ This will likely fail. Then try
⤴ This suceeds if the password is correct. |
Is there a place you get this from? I don't see it in notarytool documentation.
|
For altool it clearly states:
|
I see what you mean above, I'd need to test it again and make changes if I can manage to replicate the issue, could look into this next week when I get some time to do so. Thanks for looking into this btw. |
@mitchellh Do you have any plans on merging and making a new release for this one? 🙏 |
I have forked and built a new Gon from this PR. Feel free to give it a try and let me if you are facing issues. The fork is using itself to build so that's a good sign that it works I'd say :D |
Thanks! Sorry for taking so long, I don't really use this tool anymore so its falling by the wayside 😄 I appreciate the help. |
@mitchellh could you please make a release? So that users can benefit from it |
Not really. I don't have any of the required things installed my computer (Go, certs, etc.) and I unfortunately never setup release automation for this repo. :( You'll have to build from source. |
This PR covers migration from altool to notarytool
Output following changes looks like below: