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

Build Error: not found the fbjs\lib\warning #9350

Closed
teemosauce opened this issue Apr 6, 2017 · 6 comments
Closed

Build Error: not found the fbjs\lib\warning #9350

teemosauce opened this issue Apr 6, 2017 · 6 comments

Comments

@teemosauce
Copy link

teemosauce commented Apr 6, 2017

$ npm run build

[email protected] build D:\github\react
npm run version-check && node scripts/rollup/build.js

[email protected] version-check D:\github\react
node ./scripts/tasks/version-check.js

STARTING react.development.js (umd_dev)
Error: Could not load D:\github\react\node_modules\fbjs\lib\warning (imported by D:\github\react\src\isomorphic\modern\class\ReactNoopUpdateQueue.js): ENOENT: no such file or directory, open 'D:\github\react\node_modules\fbjs\lib\warning'
at D:\github\react\node_modules\rollup\dist\rollup.js:9461:10

But I can found the package in the node_modules.(win10 64)

@gaearon
Copy link
Collaborator

gaearon commented Apr 6, 2017

It is quite possible that the build process is broken on Windows as we just rewrote it. Would you like to help fix it?

@ccsang
Copy link

ccsang commented Apr 7, 2017

It seems like that warning.js is resolved to 'D:\github\react\node_modules\fbjs\lib\warning', it's lack of a extension like '.js'

@ccsang
Copy link

ccsang commented Apr 7, 2017

It't due to "rollup-plugin-alias", isFilePath is not work on windows.
See this issue

@JeffWong16
Copy link

Has this problem been solved? I still have the same problem

This was referenced Sep 30, 2017
@aweary
Copy link
Contributor

aweary commented Oct 4, 2017

@JeffWong16 would you like to test this out again and see if it's still broken?

@gaearon
Copy link
Collaborator

gaearon commented Oct 4, 2017

Let's track this in #9540.

@gaearon gaearon closed this as completed Oct 4, 2017
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

Successfully merging a pull request may close this issue.

5 participants