Skip to content

(wd-security) Update installer file location#284

Open
brogers5 wants to merge 1 commit intomkevenaar:masterfrom
brogers5:wd-security/fix-filelocation
Open

(wd-security) Update installer file location#284
brogers5 wants to merge 1 commit intomkevenaar:masterfrom
brogers5:wd-security/fix-filelocation

Conversation

@brogers5
Copy link
Contributor

@brogers5 brogers5 commented Feb 6, 2025

Description

This changeset updates wd-security's fileLocation variable in the install script to reflect a recent change in the upstream ZIP archive's directory structure.

Motivation and Context

wd-security was recently auto-updated to v2.1.3.304, and the resulting package that was pushed to the Community Repository has failed Package Verification testing, due to an outdated value.

How Has this Been Tested?

  1. choco pack'd the resulting package while within the wd-security directory.
  2. choco upgrade wd-security --source="'.'"
  3. Confirmed the install script completed successfully.

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

N/A

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Migrated package (a package has been migrated from another repository)

Checklist:

  • My code follows the code style of this repository.
  • My change requires a change to documentation (this usually means the notes in the description of a package).
  • I have updated the documentation accordingly (this usually means the notes in the description of a package).
  • I have updated the package description and it is less than 4000 characters.
  • All files are up to date with the latest Contributing Guidelines
  • The added/modified package passed install/uninstall in the chocolatey test environment.
  • The changes only affect a single package (not including meta package).

@AppVeyorBot
Copy link

❌ Package verification failed, please review the Appveyor Logs and the provided Artifacts before requesting a human reviewer to take a look.

@brogers5
Copy link
Contributor Author

brogers5 commented Feb 6, 2025

Verification failed due to #277 and is unrelated to my changes.

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