> On Sept. 24, 2018, 1:03 p.m., Nitin Galave wrote: > > security-admin/pom.xml > > Lines 562 (patched) > > <https://reviews.apache.org/r/68770/diff/2/?file=2090907#file2090907line562> > > > > maven build stuck after appyinng this patch > > Csaba Koncz wrote: > The first maven build takes longer because nodejs needs to be downloaded > by the frontend plugin. > > Could you post the console output? > Can you locate the line belonging to the nodejs download? > > In my case, this is the command I issue: > > mvn package -DskipTests -pl security-admin -am > > And this is how the frontend plugin executions start: > ... > [INFO] > [INFO] --- frontend-maven-plugin:1.6:install-node-and-npm (install node > and npm) @ security-admin-web --- > [INFO] Node v8.12.0 is already installed. > [INFO] NPM 6.4.1 is already installed. > [INFO] > ... > > Nitin Galave wrote: > For compiling and packaging we use > mvn clean compile package assembly:assembly install > and it excute the rat-plugin > > > maven build success if we ignore rat-plugin in pom.xml > I think we should exclude npm and node_modeuls directory for rat-plugin > (root dir : pom.xml).
Oh, thank you. I overlooked the excludeSubProjects setting in the parent pom (https://github.com/apache/ranger/blob/master/pom.xml#L569). This is a bit silly because now the files are checked twice for licenses during the verify phase: both in the submodules and at the end by the parent module. And the exclusions must contain wildcards so that they work even when evaluated in different locations. I updated the patch, the nodejs exclusions are in the parent pom now. In the long run it might be worth setting excludeSubProjects=true and move the submodule-specific rat exclusions to the sobmodule poms. - Csaba ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68770/#review208940 ----------------------------------------------------------- On Sept. 26, 2018, 8:58 a.m., Csaba Koncz wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/68770/ > ----------------------------------------------------------- > > (Updated Sept. 26, 2018, 8:58 a.m.) > > > Review request for ranger. > > > Bugs: RANGER-2220 > https://issues.apache.org/jira/browse/RANGER-2220 > > > Repository: ranger > > > Description > ------- > > Maven build modified so that the admin UI JavaScript resources are > concatenated/compressed using the RequireJS Optimizer. > During the packaging phase index.html is rewritten so that it loads the > compressed JS file instead of the JS source tree. > > > Diffs > ----- > > pom.xml d8e3a5da9 > security-admin/.gitignore 5a3a67338 > security-admin/package-lock.json PRE-CREATION > security-admin/package.json PRE-CREATION > security-admin/pom.xml 84bbd6650 > security-admin/src/main/webapp/index.html 60ef5a75b > security-admin/src/main/webapp/libs/bower/backbone-forms/js/bootstrap.js > 276175505 > > security-admin/src/main/webapp/libs/bower/backbone.bootstrap-modal/js/backbone.bootstrap-modal.js > 6cd05b03d > security-admin/src/main/webapp/minify.build.js PRE-CREATION > security-admin/src/main/webapp/scripts/Init.js 4493f62e0 > security-admin/src/main/webapp/scripts/Main.js d518afbf2 > > > Diff: https://reviews.apache.org/r/68770/diff/3/ > > > Testing > ------- > > Performed: > > mvn package -pl security-admin -am > cd security-admin/ > cd security-admin/target/security-admin-web-2.0.0-SNAPSHOT > http-server > # opened http://localhost:8080 and inspected browser network tab and console: > # network tab lists only 26 requests > # console contains no packaging related errors (it contains 4 errors due to > missing backend service) > > > Thanks, > > Csaba Koncz > >