> On Jan. 12, 2019, 10:12 p.m., Zsombor Gegesy wrote: > > security-admin/pom.xml > > Line 608 (original), 611 (patched) > > <https://reviews.apache.org/r/69731/diff/1/?file=2119346#file2119346line611> > > > > ${basedir}/target is the same as ${project.build.directory}, isn't it? > > Can you use it everywhere?
correct, fixed it in the latest patch. - Csaba ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69731/#review211937 ----------------------------------------------------------- On Jan. 13, 2019, 7:18 p.m., Csaba Koncz wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/69731/ > ----------------------------------------------------------- > > (Updated Jan. 13, 2019, 7:18 p.m.) > > > Review request for ranger. > > > Bugs: RANGER-2321 > https://issues.apache.org/jira/browse/RANGER-2321 > > > Repository: ranger > > > Description > ------- > > - Docker image building installs the packages needed by installing and > running PhantomJS > - modified Maven build so that PhantomJS downloading and execution can be > skipped by adding -DskipJSTests to the command line > > > Diffs > ----- > > README.txt 8aa177927 > build_ranger_using_docker.sh fa77d910f > security-admin/pom.xml 14cafb4b2 > security-admin/src/main/webapp/package-lock.json 9deaf0071 > security-admin/src/main/webapp/package.json 880f6b34f > security-admin/src/test/javascript/karma-common.conf.js 4dc9bb60f > security-admin/src/test/javascript/package-lock.json PRE-CREATION > security-admin/src/test/javascript/package.json PRE-CREATION > > > Diff: https://reviews.apache.org/r/69731/diff/2/ > > > Testing > ------- > > The following succeeds and runs JavaScript tests: > > ``` > ./build_ranger_using_docker.sh -build_image mvn clean verify -am -pl > security-admin > ``` > > The following succeeds and does not run JavaScript tests and does not > download PhantomJS: > ``` > mvn clean verify -am -pl security-admin -DskipJSTests > ``` > > > Thanks, > > Csaba Koncz > >
