-
Notifications
You must be signed in to change notification settings - Fork 914
docs: document issue with direnv and nix-shell on macOS #15990
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
Conversation
Including the suggested -- and tested -- workaround and a link to the direnv issue.
All contributors have signed the CLA ✍️ ✅ |
I have read the CLA Document and I hereby sign the CLA |
@@ -46,6 +46,13 @@ direnv: export +AR +AS +CC +CONFIG_SHELL +CXX +HOST_PATH +IN_NIX_SHELL +LD +NIX_ | |||
🎉 | |||
``` | |||
|
|||
> **Note:** On macOS, a [direnv bug](https://github.com/direnv/direnv/issues/1345) can cause `nix-shell` to fail to build or run `coder`. If you see `error: creating directory` when attempting to run, build, or test, adding one line to your `.envrc` should fix the problem: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Honestly, I would create a subsection for Troubleshooting and mention this issue. Otherwise, there is a risk we'll end up with 10 notes referring different issues.
I'll defer the final decision to @EdwardAngert 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a trick (beyond commenting "recheck", which isn't doing it) to have the contrib / cla
check redone?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It might just be busted, @paulroub
I had this issue with some of my own PRs - some I was able to fix when I confirmed my git config user and email (pasting below, just in case), and some just still fail. We're working on figuring out why and fixing it, but it's not a required check for us to merge
git config --local user.name "myusername" && \
git config --local user.email "no-reply-email+myusername@users.noreply.github.com"
edit: scratch that - looks like you've been added in coder/cla@061cd21
I'm changing the base branch so that I can test some linting and add some edits on that branch |
d10d9b5
into
coder:15990-contrib-nix-shell
Including the suggested -- and tested -- workaround and a link to the direnv issue.