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

Support macOS asset naming scheme for versions prior to v0.102.0 #10

Merged
merged 1 commit into from
Mar 17, 2023
Merged

Support macOS asset naming scheme for versions prior to v0.102.0 #10

merged 1 commit into from
Mar 17, 2023

Conversation

alexgeorgousis
Copy link

@alexgeorgousis alexgeorgousis commented Mar 17, 2023

Closes #9

Overview

As described in #9, there was a change in the hugo naming scheme for macOS releases, which this plugin hasn't taken into account. The issue in question describes a problem with installing version 0.99.1, but the problem exists for all versions up-to-and-including 0.101.0.

To demonstrate the problem, version v0.102.0 includes the following macOS assets:

Whereas the version preceding it, v0.101.0, has the following macOS assets:

So, they seem to have replaced universal with architecture specific assets, but only for versions prior to v0.102.0.

Change Summary

This PR updates the util function download_release() to set the arch part of the hugo asset URL to universal for macOS, only if the version is greater than or equal to v0.102.0.

@alexgeorgousis alexgeorgousis changed the title Support macOS asset naming scheme for versions before v0.102.0 Support macOS asset naming scheme for versions prior to v0.102.0 Mar 17, 2023
@nklmilojevic
Copy link
Owner

Thanks for the PR @alexgeorgousis!

@nklmilojevic nklmilojevic merged commit bc457b1 into nklmilojevic:main Mar 17, 2023
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.

Not being able to install 0.99.1 on Mac OS Monterrey
2 participants