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

[bug] MacOS build replaces version with literal https in download URL #548

Closed
yozachar opened this issue Apr 18, 2023 · 4 comments
Closed

Comments

@yozachar
Copy link

Repository | Failed run

linux

  Switched to branch '7a122abe5c15b92ec5980aa3f9c28f59344367cb'
  redirect url: https://github.com/pypa/pipx/releases/tag/1.2.0
  * Downloading pipx release 1.2.0...
  pipx 1.2.0 installation was successful!
  1.2.0

macos

 Switched to branch '7a122abe5c15b92ec5980aa3f9c28f59344367cb'
  redirect url: https://github.com/pypa/pipx/releases/tag/1.2.0
  * Downloading pipx release https...
  curl: (22) The requested URL returned error: 404
  asdf-pipx: Could not download https://github.com/pypa/pipx/releases/download/https/pipx.pyz
  FAILED: install exited with an error

Why?

@yozachar yozachar changed the title MacOS build replace version with literal https [bug] MacOS build replace version with literal https string in download URL Apr 18, 2023
@yozachar yozachar changed the title [bug] MacOS build replace version with literal https string in download URL [bug] MacOS build replaces version with literal https string in download URL Apr 18, 2023
@yozachar yozachar changed the title [bug] MacOS build replaces version with literal https string in download URL [bug] MacOS build replaces version with literal https in download URL Apr 18, 2023
@jthegedus
Copy link
Contributor

I've seen this Issue, had a brief look and couldn't identify anything specific. I looked at other plugins recently added to the repository to see if they had similar issues, and they do not.

This plugin template is meant to be a starting point, your plugin may require specific modification.

@jthegedus
Copy link
Contributor

Perhaps your plugin was having issues because of the latest-stable script. We just updated/fixed support for macOS in asdf-vm/asdf-plugin-template#65

Please try updating your plugin with that change.

@jthegedus
Copy link
Contributor

I don't believe the fix for your issue is in the Action itself, because the action is just executing your plugin which is the point of failure.

@yozachar
Copy link
Author

Well this indeed was the problem:

https://github.com/asdf-vm/asdf-plugin-template/pull/65/files

-    version="$(printf "%s\n" "$redirect_url" | sed 's|.*/tag/v\?||')"
+    version="$(printf "%s\n" "$redirect_url" | sed 's|.*/tag/v\{0,1\}||')"

Last run

Fixed by: asdf-vm/asdf-plugin-template#65

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 a pull request may close this issue.

2 participants