svn commit: r1799423 - in /pig/branches/branch-0.17: CHANGES.txt build.xml

[szita]

Author: szita
Date: Wed Jun 21 10:09:51 2017
New Revision: 1799423

URL: http://svn.apache.org/viewvc?rev=1799423&view=rev
Log:
PIG-5262: Fix jdiff related issues: fail build upon error, correct xml 
character escaping (szita)

Modified:
    pig/branches/branch-0.17/CHANGES.txt
    pig/branches/branch-0.17/build.xml

Modified: pig/branches/branch-0.17/CHANGES.txt
URL: 
http://svn.apache.org/viewvc/pig/branches/branch-0.17/CHANGES.txt?rev=1799423&r1=1799422&r2=1799423&view=diff
==============================================================================
--- pig/branches/branch-0.17/CHANGES.txt (original)
+++ pig/branches/branch-0.17/CHANGES.txt Wed Jun 21 10:09:51 2017
@@ -111,6 +111,8 @@ OPTIMIZATIONS
  
 BUG FIXES
 
+PIG-5262: Fix jdiff related issues: fail build upon error, correct xml 
character escaping (szita)
+
 PIG-5248: Fix TestCombiner#testGroupByLimit after PigOnSpark merge (rohini)
 
 PIG-5245: TestGrunt.testStopOnFailure is flaky (rohini)

Modified: pig/branches/branch-0.17/build.xml
URL: 
http://svn.apache.org/viewvc/pig/branches/branch-0.17/build.xml?rev=1799423&r1=1799422&r2=1799423&view=diff
==============================================================================
--- pig/branches/branch-0.17/build.xml (original)
+++ pig/branches/branch-0.17/build.xml Wed Jun 21 10:09:51 2017
@@ -1574,6 +1574,8 @@
                 <pathelement path="${output.jarfile.core}" />
             </classpath>
         </javadoc>
+        <!-- For some Scala classes <any> is recorded in the jdiff xml as 
return type. Need to xml-escape this properly -->
+        <replace file="${jdiff.xml.dir}/${name}_${version}.xml" 
token="&quot;&lt;any&gt;&quot;" value="&quot;&amp;lt;any&amp;gt;&quot;" />
     </target>
 
     <target name="write-null">
@@ -1601,6 +1603,10 @@
               <pathelement path="${output.jarfile.core}" />
            </classpath>
         </javadoc>
+        <fail message="Failed to generate jdiff change docs">
+            <!-- JDiff errors don't fail the build, have to take care of it 
manually -->
+           <condition><not><available file="${jdiff.build.dir}/changes" 
type="dir" /></not></condition>
+        </fail>
      </target>
 
      <target name="ivy-init-dirs">