You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by GitBox <gi...@apache.org> on 2020/04/04 08:44:29 UTC
[GitHub] [camel-website] AemieJ opened a new pull request #314: Make
modification in README regarding yarn build
AemieJ opened a new pull request #314: Make modification in README regarding yarn build
URL: https://github.com/apache/camel-website/pull/314
* I observed that in a few cases when I run `yarn build` within the project directory, it throws the error **Javascript heap out of memory**. Hence, I found a solution and I think it will be better to put it in the README documentation for a better understanding for the users.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [camel-website] zregvart commented on a change in pull request
#314: Make modification in README regarding yarn build
Posted by GitBox <gi...@apache.org>.
zregvart commented on a change in pull request #314: Make modification in README regarding yarn build
URL: https://github.com/apache/camel-website/pull/314#discussion_r403449295
##########
File path: README.md
##########
@@ -137,6 +137,12 @@ To build the website go to the project root directory and run:
$ yarn install # needed only once, or if dependencies change
$ yarn build # to perform the build
+In case `yarn build` throws the error: **JavaScript heap out of memory**, make the following change within `package.json`
+and the issue will be resolved:
+```
+"build": "run-s --max_old_space_size=4096 build:*"
+```
+
Review comment:
If we need to configure old space for all cases, i.e. the build is broken without this then I would change the `package.json` otherwise if it's an specific issue some contributors might experience but it's not widely and issue for everyone, I'd suggest to set the `NODE_OPTIONS` environment variable:
```suggestion
In case `yarn build` throws the error: **JavaScript heap out of memory**, the issue can be resolved by increasing the memory used by node.js by setting `NODE_OPTIONS` environment variable to include `--max_old_space_size`, for example to increase the old space to 4GB do:
```shell
$ export NODE_OPTIONS="--max_old_space_size=4096"
```
```
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [camel-website] zregvart merged pull request #314: Make
modification in README regarding yarn build
Posted by GitBox <gi...@apache.org>.
zregvart merged pull request #314: Make modification in README regarding yarn build
URL: https://github.com/apache/camel-website/pull/314
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services