-
-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Parcel 2: Build does not finish. #6473
Comments
It should work better with the latest Parcel nightly and Tailwind 2.2. There was a bunch of recent work on both for this. |
Looks like its working more reliably. I can still recreate this issue by doing the following:
|
I have the same build not finishing problem with a fresh install of Parcel 2.0.0-rc.0.
Assets are correctly bundled in folder, but Node process doesn't terminate. I need to UPDATE:
|
I think I may have hit this problem as well. A few observations that may be helpful with debugging:
|
In debugging, it seems to be related to a problem in This comment from ora issue 135 suggested the problem might be related to setting raw mode:
I found that commenting out these lines makes the hang-on-windows disappear: parcel/packages/core/parcel/src/cli.js Line 253 in cd01373
parcel/packages/core/parcel/src/cli.js Line 263 in cd01373
This probably isn't the right fix, but it is a clue about what is causing it. |
I found that the issue is specific to earlier versions of node (on windows only). Here's what I tested:
Given this, if anyone hits the issue, the workaround of simply upgrading to node 14+ should be enough. @devongovett - is there a project-wide goal of supporting older version of node earlier than 14? If we do want to fix this, the issue is that this line in
I think it's somehow related to the fact that writing to the There are various things that seem fix the problem on node 12+windows, but they all seem kind of hacky, including:
|
Currently, we support Node 12+: parcel/packages/core/parcel/package.json Line 21 in de41583
|
Somehow I had the same problem in CI (GitLab CI running on Linux) using docker image I'm afraid that means the issue affects not only Windows and/or node 12. update: Turn out it was the parcel cache. The build exited correctly once I added |
The cache can currently grow infinitely (as we don't have any garbage collection yet), but it shouldn't cause any slowdowns at build (and many of these files will simply be ignored by Parcel) |
we currently have the same issue as @andyli in our docker builds. For us the build would hang in 50% of cases in the parcel build step in CI. If the build would run through it would finish in seconds, if it hangs it hangs for more than 1 hour. we are running on |
Could be related (although it could be something completely else): parcel-bundler/watcher#159 |
π bug report
When running
npm run p:build
the terminal does not finish running the script however it does say⨠Built in 7.69s
and lists the outputted files with their sizes and build time. It feels like it is watching the files instead of building (Something similar to #748 however im not using typescript or angular)π Configuration (.babelrc, package.json, cli command)
.babelrc does not exist
cli command:
parcel build ./src/index.html --target web
package.json below with important stuff
π€ Expected Behavior
I would expect the build script to run and then exit or throw an error to allow more scripts to run.
π― Current Behavior
Currently I run the build script and it runs as expected but it does not finish running the script even tho the /dist directory has all the bundled files.
π Possible Solution
Not sure
π¦ Context
Trying to get Tailwind and Parcel 2 working together. It all works by adding tailwind to the postCSS file however it does not exit the build script correctly
π» Code Sample
https://github.com/JamesAlexanderHill/tailwind-parcel-val-workshop/tree/integrate-parcel
Branch: integrate-parcel
Run the following:
npm i
npm run p:build
π Your Environment
The text was updated successfully, but these errors were encountered: