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

1-click installer script #69

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

1-click installer script #69

wants to merge 1 commit into from

Conversation

arslancharyev31
Copy link

Hello. First of all, I want to thank you for the AHI, it works well. However, I thought that the installation process could've been more streamlined. For this reason, I have made a script that automatically downloads, extracts, installs, and copies the latest versions of all the necessary packages. It cleans up after itself as well but leaves the AHI folder since it contains useful examples and tools (like Monitor.ahk). It also prompts to restart the computer at the end. It does not have any error checking routines, but it enforces the admin & version requirements to eliminate the most common issues. I'm posting it here in case someone might find it useful or convenient, under the CC0 license.

Hello. First of all, I want to thank you for the AHI, it works well. However, I thought that the installation process could've been more streamlined. For this reason, I have made a script that automatically downloads, extracts, installs, and copies the latest versions of all the necessary packages. It cleans up after itself as well but leaves the AHI folder since it contains useful examples and tools (like `Monitor.ahk`). It also prompts to restart the computer at the end. It does not have any error checking routines, but it enforces the admin & version requirements to eliminate the most common issues. I'm posting it here in case someone might find it useful or convenient, under the CC0 license.
@evilC
Copy link
Owner

evilC commented Oct 17, 2020

Hi, many thanks for contributing, I have some thoughts though

  • It assumes AHK is not installed? Or at the very least will force an upgrade.
    Should we not assume AHK is already installed? Or ideally check if it is installed and prompt to upgrade if it is?

  • It deletes the Interception installer. This should probably be left so that the user can uninstall from it

@arslancharyev31
Copy link
Author

  1. It does indeed install or upgrade the AHK installation. I can add a prompt for that.
  2. Maybe instead it would be better to write a companion uninstall script, which fetches the installer and uninstalls the Interception. This way users don't have to bother about where to save the installer so that in the future they might delete Interception. In addition, the uninstaller would delete AHI libs from My Documents, as there is no point in keeping them since they don't work without Interception.

Just to be clear, the primary goal of this script is to offer a very simple way to install the AHI. If users want more granular control, then manual installation should be recommended.

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.

2 participants