-----------------------------------------------------------
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
> 
>

Reply via email to