You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zeppelin.apache.org by GitBox <gi...@apache.org> on 2020/09/28 15:51:15 UTC
[GitHub] [zeppelin] w1833904464 opened a new pull request #3929: Appears when building zeppelin-web：'npm run build:dist' failed.
w1833904464 opened a new pull request #3929:
URL: https://github.com/apache/zeppelin/pull/3929
### What is this PR for?
When you use root user to build zepeelin on Linux host, because bower does not support root user execution, an exception will be thrown：
`[ERROR] npm ERR! code ELIFECYCLE
[ERROR] npm ERR! errno 3
[ERROR] npm ERR! zeppelin-web@0.0.0 build:dist: `npm-run-all prebuild && grunt pre-webpack-dist && webpack && grunt post-webpack-dist`
[ERROR] npm ERR! Exit status 3
[ERROR] npm ERR!
[ERROR] npm ERR! Failed at the zeppelin-web@0.0.0 build:dist script.
[ERROR] npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
[ERROR]
[ERROR] npm ERR! A complete log of this run can be found in:
[ERROR] npm ERR! /root/.npm/_logs/2020-09-25T18_56_09_331Z-debug.log
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 01:22 min
[INFO] Finished at: 2020-09-26T02:56:09+08:00
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal com.github.eirslett:frontend-maven-plugin:1.6:npm (npm build) on project zeppelin-web: Failed to run task: 'npm run build:dist' failed. org.apache.commons.exec.ExecuteException: Process exited with an error: 3 (Exit value: 3) -> [Help 1]
org.apache.maven.lifecycle.LifecycleExecutionException: Failed to execute goal com.github.eirslett:frontend-maven-plugin:1.6:npm (npm build) on project zeppelin-web: Failed to run task
`
### What type of PR is it?
Improvement | Feature |
### Todos
#### h2. You need to modify the following code in zeppelin-web/package.json
`
"build:dist": "npm-run-all prebuild && grunt pre-webpack-dist && webpack && grunt post-webpack-dist",
"build:ci": "npm-run-all prebuild && grunt pre-webpack-ci && webpack && grunt post-webpack-dist",
`
#### change to：
`
"build:dist": "npm-run-all prebuild && bower install --silent --allow-root && grunt pre-webpack-dist && webpack && grunt post-webpack-dist",
"build:ci": "npm-run-all prebuild && bower install --silent --allow-root && grunt pre-webpack-ci && webpack && grunt post-webpack-dist","lint:watch": "esw --watch src",
`
### What is the Jira issue?
[[ZEPPELIN-5071](https://issues.apache.org/jira/browse/ZEPPELIN-5071)
### How should this be tested?
`
mvn -X clean package -pl 'zeppelin-web' -DskipTests
`
### Screenshots (if appropriate)
### Questions:
* Does the licenses files need update?
* Is there breaking changes for older versions?
* Does this needs documentation?
----------------------------------------------------------------
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
[GitHub] [zeppelin] Reamer commented on pull request #3929: [ZEPPELIN-5071] Appears when building zeppelin-web：'npm run build:dist' failed.
Posted by GitBox <gi...@apache.org>.
Reamer commented on pull request #3929:
URL: https://github.com/apache/zeppelin/pull/3929#issuecomment-700498762
In my opinion we should not activate the root build by default. This should be an additional step for the user, as security is being overridden.
In general, no root rights are required to build Zeppelin.
----------------------------------------------------------------
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
[GitHub] [zeppelin] xinxingi closed pull request #3929: [ZEPPELIN-5071] Appears when building zeppelin-web：'npm run build:dist' failed.
Posted by GitBox <gi...@apache.org>.
xinxingi closed pull request #3929:
URL: https://github.com/apache/zeppelin/pull/3929
----------------------------------------------------------------
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
[GitHub] [zeppelin] xinxingi commented on pull request #3929: [ZEPPELIN-5071] Appears when building zeppelin-web：'npm run build:dist' failed.
Posted by GitBox <gi...@apache.org>.
xinxingi commented on pull request #3929:
URL: https://github.com/apache/zeppelin/pull/3929#issuecomment-700538909
For the security of Zeppelin, root should not be granted by default. However, if the user builds the root user into Zeppelin , he will receive an exception, which is not a good experience for the new user. This question will be provided to future friends for reference, thank you.
----------------------------------------------------------------
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
[GitHub] [zeppelin] Reamer commented on pull request #3929: [ZEPPELIN-5071] Appears when building zeppelin-web：'npm run build:dist' failed.
Posted by GitBox <gi...@apache.org>.
Reamer commented on pull request #3929:
URL: https://github.com/apache/zeppelin/pull/3929#issuecomment-700580222
I think a note on [`how_to_build.html`](https://zeppelin.apache.org/docs/0.9.0-preview2/setup/basics/how_to_build.html) should be sufficient for new 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