mkevenaar / chocolatey-packages

My chocolatey packages
https://mkevenaar.github.io/chocolatey-packages/
Apache License 2.0
42 stars 70 forks source link

(samsung-magician) Fix blocking issues with AHK v1 script #172

Closed brogers5 closed 1 year ago

brogers5 commented 1 year ago

Description

This changeset tweaks the package's AutoHotKey install script to reflect changes made in more recent releases to the installer.

Motivation and Context

While regression testing a fix for #167, I noticed there were a couple issues with the current v1-compatible script that needed to be fixed. I felt this deserved a PR in its own right, as I didn't want to lump it into the v2 conversion effort.

Samsung has made a couple breaking changes that impacted the AutoHotKey script. Specifically:

Both of these issues normally would've caused Package Verifier to fail, but these slipped through since the package is currently exempted.

How Has this Been Tested?

Environments

Dev

Steps

  1. Created a test package that consumes the modified script (i.e. choco pack).
  2. Install/downgrade to AutoHotKey v1.1 to satisfy current package dependency while avoiding current v2-related failures (i.e. choco install autohotkey.portable --version=1.1.36.02 --allow-downgrade).
  3. Install test package version with backup source to accommodate dotnet4.6.1 dependency (i.e. choco install samsung-magician --source="'.;https://community.chocolatey.org/api/v2/'").
  4. Confirm the AutoHotKey script executes successfully, and guides the installer to completion without requiring manual user input.

Screenshot (if appropriate, usually isn't needed):

N/A

Types of changes

Checklist:

AppVeyorBot commented 1 year ago

:white_check_mark: Package verification completed without issues. PR is now pending human review

mkevenaar commented 1 year ago

@brogers5 would you be willing to make the conversion to V2 as well?

brogers5 commented 1 year ago

@mkevenaar Yes, I'll take that on, although there's a couple more out-of-scope enhancements I'd like to make to the script first. You can expect PRs for those in the near future.

mkevenaar commented 1 year ago

@mkevenaar Yes, I'll take that on, although there's a couple more out-of-scope enhancements I'd like to make to the script first. You can expect PRs for those in the near future.

Ok, then I'll merge this one first and wait for other PRs. Thank you for your contribution!