pig-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From sz...@apache.org
Subject svn commit: r1799423 - in /pig/branches/branch-0.17: CHANGES.txt build.xml
Date Wed, 21 Jun 2017 10:09:51 GMT
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">



Mime
View raw message