----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/71587/#review218125 -----------------------------------------------------------
pom.xml Line 183 (original), 183 (patched) <https://reviews.apache.org/r/71587/#comment305708> Can you please rebase it to master branch? Patch fails to apply. ``` $ git apply --check -v < ~/Downloads/0001-RANGER-2606-added-module-to-sign-artifacts-profile-t.patch Checking patch pom.xml... Hunk #1 succeeded at 117 (offset 1 line). error: while searching for: <velocity.version>1.7</velocity.version> <zookeeper.version>3.4.14</zookeeper.version> <codehaus.woodstox.stax2api.version>3.1.4</codehaus.woodstox.stax2api.version> <fasterxml.woodstox.version>5.0.3</fasterxml.woodstox.version> <fasterxml.jackson.version>2.9.9</fasterxml.jackson.version> <kstruct.gethostname4j.version>0.0.2</kstruct.gethostname4j.version> <jna.version>5.2.0</jna.version> <jna-platform.version>5.2.0</jna-platform.version> <!-- presto plugin deps --> <presto.airlift.version>0.178</presto.airlift.version> <presto.bval-jsr.version>2.0.0</presto.bval-jsr.version> error: patch failed: pom.xml:183 error: pom.xml: patch does not apply ``` - Velmurugan Periasamy On Oct. 7, 2019, 9:24 p.m., Selvamohan Neethiraj wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/71587/ > ----------------------------------------------------------- > > (Updated Oct. 7, 2019, 9:24 p.m.) > > > Review request for ranger, Colm O hEigeartaigh, Abhay Kulkarni, Ramesh Mani, > and Velmurugan Periasamy. > > > Repository: ranger > > > Description > ------- > > Added all modules to the sign-artifacts profile to fix this issue ... > > > Diffs > ----- > > pom.xml afc27d2cb > > > Diff: https://reviews.apache.org/r/71587/diff/1/ > > > Testing > ------- > > Ran it from v2.0 release codebase to check if it is able to upload as part of > the deployment. It works! > * Also, fixed some of the TAB characters (replaced with SPACE to look nice) > > > Thanks, > > Selvamohan Neethiraj > >
