Github user mfenes commented on a diff in the pull request:

    https://github.com/apache/zookeeper/pull/469#discussion_r169695831
  
    --- Diff: build.xml ---
    @@ -1861,4 +1861,18 @@ 
xmlns:cs="antlib:com.puppycrawl.tools.checkstyle.ant">
            <delete dir="${build.dir.eclipse}" />
          </target>
     
    +    <target name="print_compile_classpath">
    --- End diff --
    
    True, my intention with the "_" was to indicate that these targets are 
purely utility targets i.e. it does not make too much sense to run them 
individually, unlike clean, compile, test-core-java etc. targets. If you don't 
like this idea, I can replace the "_"-s with "-"-s if you wish.


---

Reply via email to