You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zookeeper.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2017/10/03 20:52:00 UTC

[jira] [Commented] (ZOOKEEPER-2887) define dependency versions in build.xml to be easily overridden in build.properties

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

ASF GitHub Bot commented on ZOOKEEPER-2887:
-------------------------------------------

GitHub user tamaashu opened a pull request:

    https://github.com/apache/zookeeper/pull/389

    ZOOKEEPER-2887: define dependency versions in build.xml to be easily …

    …overridden in build.properties
    
    Backported to branch 3.4.
    If the dependency versions are defined in build.xml they can be easily
    overridden by re-defining them in build.properties
    This process can be useful to avoid classpath clashes among different
    Hadoop components
    
    Author: Tamas Penzes <ta...@cloudera.com>
    Author: Tamás Pénzes <pe...@gmail.com>
    
    Reviewers: Patrick Hunt <ph...@apache.org>, Michael Han <ha...@apache.org>
    
    Closes #357 from tamaashu/ZOOKEEPER-2887 and squashes the following commits:
    
    5e2f43fb [Tamás Pénzes] Merge branch 'master' into ZOOKEEPER-2887
    65558c09 [Tamas Penzes] ZOOKEEPER-2887: define dependency versions in build.xml to be easily overridden in build.properties
    6cf315fd [Tamas Penzes] ZOOKEEPER-2887: define dependency versions in build.xml to be easily overridden in build.properties
    
    Change-Id: Ic19d67cc68ed4595ae0636d8200287400e98e2a1
    
    # Conflicts:
    #	ivy.xml

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/tamaashu/zookeeper ZOOKEEPER-2887-3.4

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/zookeeper/pull/389.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #389
    
----

----


> define dependency versions in build.xml to be easily overridden in build.properties
> -----------------------------------------------------------------------------------
>
>                 Key: ZOOKEEPER-2887
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-2887
>             Project: ZooKeeper
>          Issue Type: Improvement
>          Components: build
>            Reporter: Tamas Penzes
>            Assignee: Tamas Penzes
>             Fix For: 3.6.0
>
>
> Dependency versions are defined in ivy.xml, which is suboptimal since it is hard to override them from a script.
> If we defined the versions in the main build.xml (just as we do with audience-annotations.version) and use variables in ivy.xml then we could easily override the versions with creating a build.properties file, which mechanism is already built in.
> This way the dependency versions could be replaced by sed or any simple command line tool.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)