You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zookeeper.apache.org by "Patrick Hunt (JIRA)" <ji...@apache.org> on 2010/01/08 19:29:54 UTC
[jira] Commented: (ZOOKEEPER-640) make build.xml more configurable
to ease packaging for linux distros
[ https://issues.apache.org/jira/browse/ZOOKEEPER-640?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12798121#action_12798121 ]
Patrick Hunt commented on ZOOKEEPER-640:
----------------------------------------
Great to see this being packaged for Debian! Shouldn't be a problem. A few questions though:
1) contrib include/exclude seems fine to me, good idea
2) for the "additional.lib.dir" :
a) you want this changed in build.xml at the toplevel and also for the contribs, right? Same variable name
used in both ok?
b) do you really want a dir or a path string instead, ie additional.lib.dir or additional.libs.path (where libs.path is a list of directories/jars)
libs.path would give you more control (you would specify/order the jars explicitly vs just *.jar") This might be important
for example if you have multiple jars with same capability but diff version... it would require you to specify the
list of jars though.
> make build.xml more configurable to ease packaging for linux distros
> --------------------------------------------------------------------
>
> Key: ZOOKEEPER-640
> URL: https://issues.apache.org/jira/browse/ZOOKEEPER-640
> Project: Zookeeper
> Issue Type: Improvement
> Components: build
> Affects Versions: 3.2.1, 3.2.2
> Reporter: Thomas Koch
> Fix For: 3.3.0
>
> Original Estimate: 0.25h
> Remaining Estimate: 0.25h
>
> Hi,
> I started packaging Zookeeper for Debian[1][2]. Thereby I had a problem excluding contrib/rest from the build without patching the upstream tarball. Could you please add some properties to your build.xml that allow me to (de)select contribs? In the example below I can easily override the properties:
> <project name="zookeepercontrib">
>
> <property name="contribfilesetincludes" value="*/build.xml" />
> <property name="contribfilesetexcludes" value="" />
> <fileset id="contribfileset"
> dir="."
> includes="${contribfilesetincludes}"
> excludes="${contribfilesetexcludes}"
> />
> <target name="compile">
> <subant target="jar">
> <fileset refid="contribfileset" />
> </subant>
> </target>
> Could you please also add a line to project.classpath:
> <path id="project.classpath">
> <fileset dir="${additional.lib.dir}" includes="*.jar"/>
> For Debian I may not compile based on the jar files in lib but must use the jars already in Debian.
> [1] http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=561947
> [2] http://git.debian.org/?p=pkg-java/zookeeper.git
> Thank you!
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.