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

.bin folder missing when using --modules-folder #4297

Closed
kilianc opened this issue Sep 2, 2017 · 6 comments
Closed

.bin folder missing when using --modules-folder #4297

kilianc opened this issue Sep 2, 2017 · 6 comments

Comments

@kilianc
Copy link

kilianc commented Sep 2, 2017

Do you want to request a feature or report a bug?
BUG

What is the current behavior?

ENV NODE_ENV production
ENV NODE_PATH /opt/app-node-modules
ENV PATH $PATH:/opt/app-node-modules/.bin

RUN yarn --no-emoji --modules-folder $NODE_PATH --pure-lockfile

.bin is nowhere to be found.

What is the expected behavior?

Behavior (as per yarn@0.24.4 and node@7.10.0) should be .bin folder at /opt/app-node-modules/.bin

Please mention your node.js, yarn and operating system version.

  • node@8.4.0
  • yarn@0.27.5

P.S. I see this behavior bumping our docker images to 8.4.0 from 7.10.0

@BYK
Copy link
Member

BYK commented Sep 5, 2017

This looks like a duplicate of #3724. Please confirm :)

@schmunk42
Copy link

I have the same issue, looks like duplicate of #3724

@kilianc
Copy link
Author

kilianc commented Nov 29, 2017

Any update on this, I have been stuck on yarn@0.24.6 for months :(

@crgt5252
Copy link

Any update on this? We're also blocked by this.

@EECOLOR
Copy link

EECOLOR commented Apr 10, 2018

@BYK yes, it's a duplicate of #3724

voondo added a commit to voondo/yarn that referenced this issue Apr 26, 2018
…e `--modules-folder` flag (yarnpkg#3724, yarnpkg#4297)

**Summary**

This PR fixes yarnpkg#3724 and yarnpkg#4297 where `install` was not installing binary symlinks to the correct location
when the `--modules-folder` flag was provided.

**Test plan**

Manual CLI tests:

Before:
```
> yarn --modules-folder /var/foo
> (ls /var/foo/.bin 2>1 > /dev/null && echo "found") || echo "not found"
not found
```

After:
```
> yarn --modules-folder /var/foo
> (ls /var/foo/.bin 2>1 > /dev/null && echo "found") || echo "not found"
found
```
voondo added a commit to voondo/yarn that referenced this issue Apr 27, 2018
…e `--modules-folder` flag (yarnpkg#3724, yarnpkg#4297)

**Summary**

This PR fixes yarnpkg#3724 and yarnpkg#4297 where `install` was not installing binary symlinks to the correct location
when the `--modules-folder` flag was provided.

**Test plan**

Manual CLI tests:

Before:
```
> yarn --modules-folder /var/foo
> (ls /var/foo/.bin 2>1 > /dev/null && echo "found") || echo "not found"
not found
```

After:
```
> yarn --modules-folder /var/foo
> (ls /var/foo/.bin 2>1 > /dev/null && echo "found") || echo "not found"
found
```
voondo added a commit to voondo/yarn that referenced this issue Apr 27, 2018
…e `--modules-folder` flag (yarnpkg#3724, yarnpkg#4297)

**Summary**

This PR tries to fixes yarnpkg#3724 and yarnpkg#4297 where `install` was not installing binary symlinks to the correct location
when the `--modules-folder` flag was provided.

WIP - CURRENTLY DOESN'T WORK, DON'T MERGE IT

**Test plan**

Manual CLI tests:

Before:
```
> yarn --modules-folder /var/foo
> (ls /var/foo/.bin 2>1 > /dev/null && echo "found") || echo "not found"
not found
```

After:
```
> yarn --modules-folder /var/foo
> (ls /var/foo/.bin 2>1 > /dev/null && echo "found") || echo "not found"
found
```

Automatic test:

__tests__/commands/install/bin-links.js -t "install should respect --modules-folder flag"

**Remaining issues**

* Problem with `yarn check --verify-tree`:
```
error "standard#eslint" not installed
error "standard#eslint-config-standard" not installed
error "standard#eslint-config-standard-jsx" not installed
error "standard#eslint-plugin-promise" not installed
error "standard#eslint-plugin-react" not installed
error "standard#eslint-plugin-standard" not installed
error "standard#standard-engine" not installed
```
* Automatic test fails (while manual CLI test works)
@BYK
Copy link
Member

BYK commented May 3, 2018

Closing in favor of #3724.

@BYK BYK closed this as completed May 3, 2018
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

No branches or pull requests

5 participants