-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Optional dependencies still have problem #1285
Comments
Same problem here. Node 6.5, win 7
|
Same issue here (windows 7) |
I am ending up with a lot more MB in node_modules installing with yarn. I am certain that one big factor is |
Same issue on win10 and yarn version is 0.16.1, also resemble error |
This should have been fixed by #525, but it looks like it regressed at some point 😢 . |
@Daniel15, I don't think that module works as it specifies the But if I exclude the platform check for $ yarn add @daniel15/win-test-2
info No lockfile found.
[1/4] 🔍 Resolving packages...
[2/4] 🚚 Fetching packages...
warning @daniel15/[email protected]: The platform "darwin" is incompatible with this module.
info "@daniel15/[email protected]" is an optional dependency and failed compatibility check. Excluding it from installation.
[3/4] 🔗 Linking dependencies...
[4/4] 📃 Building fresh packages...
success Saved lockfile.
success Saved 2 new dependencies.
├─ @daniel15/[email protected]
└─ @daniel15/[email protected]
✨ Done in 1.68s. Verified the optionalDependency module was installed too: $ ls node_modules/@daniel15
win-test win-test-2 It is interesting that it reports excluding it from installation, but installs it anyways. |
@wyze - Oops, my mistake! Sorry for the misleading comment. I just realised my test package was configured incorrectly. |
In case it helps, running |
+1
Expected behavior:
Actual behavior:
Repeated installation attempts eventually makes it happy, but clearing out node_modules and package.json and doing a fresh install of Update: @kriscarle I think that might be a fluke. I tried installing |
I tried use yarn and i faced with the issue:
When i use npm, it was work normally. on the one hand, new behaviour is good, |
my dependencies (they partially are not open source)
private islands-tools package.json:
|
Fixed via #2116. |
b4 just (re)opening ... Is the following still evidence of this^ issue, unresolved? Or a new issue?
|
I am still getting this error. I am using the latest version 0.27.5 on Windows.
|
@Igor-Vuk that looks more like a |
@BYK it seems you are right. It happens when I install compression-webpack-plugin. |
yarn install command not work |
@jubayed |
@2075 npm -v problem . |
Seeing the exact same output as @pgnd with yarn |
I also see the same as @pgnd:
|
Same here on my production server, where as on my MacOS no issue with npm or yarn. I'm using latest nodeJs tried with 6 and 8, and i'm using latest Yarn 1.2.1.
or this:
|
same issue with latest yarn 1.3.2, node 8.x
|
Same issue as @gkatsanos |
@MatrixOfDeath - Any lines with @gkatsanos That just looks like a warning, not an error. It's legitimate, as fsevents doesn't work on Linux (only on MacOS). |
Daniel, but, see all other messages above.. |
I was trying to install the
OS: Windows 10 |
Do you want to request a feature or report a bug?
Bug
What is the current behavior?
Optional dependency is excluded but still try to build.
If the current behavior is a bug, please provide the steps to reproduce.
yarn add chokidar
What is the expected behavior?
Incompatible optional dependency should be excluded completly.
Please mention your node.js, yarn and operating system version.
node4
yarn0.16.1
windows7
The text was updated successfully, but these errors were encountered: