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

Add ability to overwrite the architecture used when installing #2

Merged
merged 1 commit into from
Jun 24, 2022

Conversation

yorinasub17
Copy link

This uses a similar environment variable override supported by asdf-hashicorp for overriding the target architecture used to download the binary, which is useful when installing older versions of terragrunt that did not publish darwin/arm64 binaries.

@brianheuston
Copy link

This would be a really useful feature to add. Is there anything blocking this? Or another workaround?

@spider-network
Copy link

+1 @ohmer can you merge that?

@ohmer ohmer self-assigned this Feb 11, 2022
@ohmer ohmer added enhancement New feature or request good first issue Good for newcomers labels Feb 11, 2022
@gebassetti
Copy link

+1

1 similar comment
@uberjew666
Copy link

+1

@brianheuston
Copy link

brianheuston commented Jun 24, 2022

This PR has been around for forever. Unfortunately Ohmer might have a lot going on, so I think it's time we fork and move on.

@ohmer
Copy link
Owner

ohmer commented Jun 24, 2022

This PR has been around for forever. Unfortunately Ohmer might have a lot going on, so I think it's time we fork and move on.

So sorry folks, forgot about this and I don't use terragrunt anymore. Happy to merge without testing. Please take ownership of this if you want to.

@ohmer ohmer merged commit 4a6651a into ohmer:main Jun 24, 2022
@brianheuston
Copy link

This PR has been around for forever. Unfortunately Ohmer might have a lot going on, so I think it's time we fork and move on.

So sorry folks, forgot about this and I don't use terragrunt anymore. Happy to merge without testing. Please take ownership of this if you want to.

No worries! That actually might be something the community might be open to doing since you don't use the tool anymore.

FWIW, I did verify it worked, installing the plugin from the fork. So it should be good to go.

@yorinasub17
Copy link
Author

Please take ownership of this if you want to.

I'll ask around internally (I work at Gruntwork, maintainer of Terragrunt) to see if there is interest in having a gruntwork-io/asdf-terragrunt repo. If not, I'll maintain a fork personally since I use this everyday.

@ohmer
Copy link
Owner

ohmer commented Oct 17, 2023

@yorinasub17 reviving an old thread but please see #3. Anyone at Gruntwork keen?

@yorinasub17
Copy link
Author

Heya! Unfortunately, I've moved on from Gruntwork since I opened this, and I was the only one using asdf there, so there wasn't a lot of interest in maintaining this outside of me. I would take it up if I was still using terragrunt, but I currently am not using it...

Sorry I can't be of help here!

@ohmer
Copy link
Owner

ohmer commented Oct 19, 2023

all good, thanks for the update @yorinasub17!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants