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

fix: improve setup instructions and enforce consistent GOBIN usage #152

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

mkungla
Copy link

@mkungla mkungla commented Jan 31, 2025

This PR includes two changes:

  • Improves README instructions to better guide new users.
  • Ensures that GOBIN is always set and used consistently, preventing binaries from being installed in unexpected locations.

closes #151
closes #150

@mkungla mkungla requested a review from a team as a code owner January 31, 2025 22:26
Previously, when GOBIN was empty, default packages listed in
$ASDF_GOLANG_DEFAULT_PACKAGES_FILE were installed to
$ASDF_INSTALL_PATH/packages/bin. This could cause confusion,
as user-installed binaries might end up in different locations.

This commit ensures that GOBIN is always set, so all binaries
are consistently installed into $ASDF_INSTALL_PATH/bin.
Additionally, the use of $ASDF_INSTALL_PATH/packages/bin is
deprecated to prevent inconsistencies.

Closes asdf-community#151

Signed-off-by: Marko Kungla <[email protected]>
@mkungla mkungla changed the title docs: enhance instructions for updating shell startup fix: improve setup instructions and enforce consistent GOBIN usage Feb 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
1 participant