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

Inno6 installer #31

Open
wants to merge 35 commits into
base: main
Choose a base branch
from
Open

Inno6 installer #31

wants to merge 35 commits into from

Conversation

willwade
Copy link

@willwade willwade commented Jun 28, 2023

  • Using inno6 installer
  • Moved the location of log files C:/Users/<USER_NAME>/Gameface/log.txt
  • Moved location of config files C:/Users/<USER_NAME>/Gameface/
  • upgraded customtkinter & google theme file likewise (re: customtkinter.CTkSwitch initialization missing fg_color argument #7)
  • FIxes attribute error (Fix attribute error #30 )
  • Created a GitHub action to build the installer. Outputs an installer version exe and a zip of the --onefile pyinstaller build with config directory.
  • Updated docs
  • Provides option to "Run at startup" which makes it run at startup
  • Provides option for desktop shortcut
  • Adds icon to app
  • Fixed versions in requirements and started to add platform specific aspects (for Availability for Linux + DepthAI #2 work)

But NOTE: This is NOT signed. I hope Google can sort that out with a codesigning action in the GHActions.

Due to that - if people want to run the installer before that's fixed, they may need to run this from a terminal prompt if your machine is locked down e.g. & '.\Gameface installer.exe'

(If you want to try the installer here it is - Windows Installer and Portable Build (like currently))

@willwade
Copy link
Author

fixes #33 and would also fix #7 and #29

This was referenced Jun 29, 2023
@willwade
Copy link
Author

This is ready for review :)

@deFrisselle
Copy link

Was just asked by a disability advocate about the possibility of making the installation easier and easier to understand plus autorunning at boot Nice to see there are commits for this in review Hope to see it in the next release so I can give him the good news

@willwade
Copy link
Author

@deFrisselle I’m not convinced anyone’s actually reviewing it. This whole repo seems pretty, frustratingly, “slow”. But I’d love to be corrected on that ;) @bit-dome

@deFrisselle
Copy link

Unfortunate, the installer works quite well

@afgs10
Copy link

afgs10 commented Aug 18, 2023

@deFrisselle I’m not convinced anyone’s actually reviewing it. This whole repo seems pretty, frustratingly, “slow”. But I’d love to be corrected on that ;) @bit-dome

Though i love the concept and the original devs, maybe forking would lead to a much better future for this project. Would you be interested in that willwade? Even if just for the installer changes and auto-run

@willwade
Copy link
Author

@deFrisselle I’m not convinced anyone’s actually reviewing it. This whole repo seems pretty, frustratingly, “slow”. But I’d love to be corrected on that ;) @bit-dome

Though i love the concept and the original devs, maybe forking would lead to a much better future for this project. Would you be interested in that willwade? Even if just for the installer changes and auto-run

Well see #13 - where my colleague hints at this.. thing is maybe we are being harsh here. There are commits fixing some issues just two weeks ago - just they don't seem to be attacking or using the issue queue so its hard to see the progress...

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

3 participants