[ 
https://issues.apache.org/jira/browse/TINKERPOP-1443?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15795971#comment-15795971
 ] 

ASF GitHub Bot commented on TINKERPOP-1443:
-------------------------------------------

Github user dkuppitz commented on the issue:

    https://github.com/apache/tinkerpop/pull/494
  
    This is required to make it work:
    
    ```
    daniel@cube /projects/apache/tinkerpop (pr-494) $ git diff
    diff --git a/docker/build/Dockerfile.template 
b/docker/build/Dockerfile.template
    index 8364884..d32aa46 100644
    --- a/docker/build/Dockerfile.template
    +++ b/docker/build/Dockerfile.template
    @@ -21,3 +21,7 @@ RUN mkdir -p /usr/src/tinkerpop
     RUN curl -s https://bootstrap.pypa.io/ez_setup.py | python
     WORKDIR /usr/src/tinkerpop
     COPY . /usr/src/tinkerpop
    +RUN chmod 744 ./.travis.install-maven.sh
    +ENV M2_HOME /root/mvn-home
    +ENV PATH ${M2_HOME}/bin:${PATH}
    +RUN ./.travis.install-maven.sh 3.3.9 ${M2_HOME}
    ```
    
    However, this should only go into `tp32`. In `master/` we should make the 
maven installation part of the base image.


> Use an API checker during build
> -------------------------------
>
>                 Key: TINKERPOP-1443
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-1443
>             Project: TinkerPop
>          Issue Type: Improvement
>          Components: build-release
>    Affects Versions: 3.2.2
>            Reporter: Lukas Krejci
>
> Tinkerpop 3.2.2 changed the signature of the method 
> {{GraphTraversal.hasLabel}} from {{(String...)}} to {{(String, String...)}}. 
> While this is certainly an improvement, it is both source and binary 
> incompatible change.
> I.e. even if every usage of {{hasLabel}} had at least one parameter in the 
> user code, none of those calls will work until all the user code is 
> recompiled using Tinkerpop 3.2.2.
> I don't know the versioning policy of Tinkerpop but changes like the above in 
> a micro/patch release are generally unexpected.
> Please consider API checkers like http://revapi.org to warn about such 
> incompatible API changes...



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to