Skip to content
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

A windows installer for CLI #57

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

HuanzhiMao
Copy link

The installer is the Gorilla-CLI.exe in the dist_exe folder. The scripts that are used to generate the installer are also attached in the dist_exe/scripts folder.

@ShishirPatil
Copy link
Member

Hey @HuanzhiMao I finally got access to a Windows Machine and tested this by just installing the exe. I don't think this works. I get a "Welcome to the App" and a next window message that says the app closed. Can you a) Test it on a windows machine, b) Maybe describe the setup-process if it's not a simple install the exe, c) The app has an alert of "unknown" publisher. What's a good way to fix that? Can we have either the Sky/BAIR labs or Gorilla as a publisher depending on what you think is the quickest route?

@HuanzhiMao
Copy link
Author

I did test it on Windows (cause I'm using windows desktop). You should give it admin access if you install for all user.
image
And after that, you should be able to install by just hit "next"; you can change some settings, but it's not a must.
image
Regarding "unknown" publisher, I can change it in the script. It should be easy to fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants