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

nodenv-sync does not respect $NODENV_ROOT #16428

Closed
3 tasks done
kytta opened this issue Jan 4, 2024 · 1 comment · Fixed by #16453
Closed
3 tasks done

nodenv-sync does not respect $NODENV_ROOT #16428

kytta opened this issue Jan 4, 2024 · 1 comment · Fixed by #16453
Assignees
Labels
bug Reproducible Homebrew/brew bug in progress Maintainers are working on this outdated PR was locked due to age

Comments

@kytta
Copy link

kytta commented Jan 4, 2024

brew doctor output

$ brew doctor
Please note that these warnings are just used to help the Homebrew maintainers
with debugging if you file an issue. If everything you use Homebrew for is
working fine: please don't worry or file an issue; just ignore this. Thanks!

Warning: Some installed formulae are deprecated or disabled.
You should find replacements for the following formulae:
  node@16

Warning: Your Xcode (15.0.1) is outdated.
Please update to Xcode 15.1 (or delete it).
Xcode can be updated from the App Store.

(I can't update Xcode at the moment)

Verification

  • My "brew doctor output" above says Your system is ready to brew. and am still able to reproduce my issue.
  • I ran brew update twice and am still able to reproduce my issue.
  • This issue's title and/or description do not reference a single formula e.g. brew install wget. If they do, open an issue at https://github.com/Homebrew/homebrew-core/issues/new/choose instead.

brew config output

$ brew config
HOMEBREW_VERSION: 4.2.2-16-g21a6f93
ORIGIN: https://github.com/Homebrew/brew
HEAD: 21a6f93cc4839377f0bc2a0782181c91ad9643c5
Last commit: 10 hours ago
Core tap JSON: 04 Jan 08:23 UTC
Core cask tap JSON: 04 Jan 08:23 UTC
HOMEBREW_PREFIX: /opt/homebrew
HOMEBREW_BAT: set
HOMEBREW_BAT_THEME: OneHalfDark
HOMEBREW_BROWSER: open
HOMEBREW_CASK_OPTS: []
HOMEBREW_EDITOR: micro
HOMEBREW_MAKE_JOBS: 10
HOMEBREW_NO_ANALYTICS: set
HOMEBREW_NO_ENV_HINTS: set
HOMEBREW_SORBET_RUNTIME: set
Homebrew Ruby: 3.1.4 => /opt/homebrew/Library/Homebrew/vendor/portable-ruby/3.1.4/bin/ruby
CPU: 10-core 64-bit arm_blizzard_avalanche
Clang: 15.0.0 build 1500
Git: 2.43.0 => /opt/homebrew/bin/git
Curl: 8.1.2 => /usr/bin/curl
macOS: 13.6.3-arm64
CLT: 15.1.0.0.1.1700200546
Xcode: 15.0.1
Rosetta 2: false

What were you trying to do (and why)?

I was trying to symlink the Homebrew-installed Node.js versions to my nodenv directory.

What happened (include all command output)?

The command exited without errors, but it created the symlinks inside ~/.nodenv/versions. I am using a custom nodenv path (specified by $NODENV_ROOT, as usual) so that my home directory is not cluttered.

What did you expect to happen?

I expected the symlinks to be created at $NODENV_ROOT/versions

Step-by-step reproduction instructions (by running brew commands)

$ brew nodenv-sync
$ ls $NODENV_ROOT/versions
$ ls ~/.nodenv/versions
16.0.0
16.0.1
...
@kytta kytta added the bug Reproducible Homebrew/brew bug label Jan 4, 2024
@kytta
Copy link
Author

kytta commented Jan 4, 2024

Line in question:

dot_nodenv = Pathname(Dir.home)/".nodenv"

The $NODENV_ROOT environment variable is documented here

@MikeMcQuaid MikeMcQuaid self-assigned this Jan 9, 2024
@MikeMcQuaid MikeMcQuaid added the in progress Maintainers are working on this label Jan 9, 2024
@github-actions github-actions bot added the outdated PR was locked due to age label Feb 9, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 9, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Reproducible Homebrew/brew bug in progress Maintainers are working on this outdated PR was locked due to age
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants