You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@royale.apache.org by Alex Harui <ah...@adobe.com.INVALID> on 2019/09/18 14:18:05 UTC

Re: [royale-asjs] branch release/0.9.6 updated: Fix releasessteps script

Changing releaseversion to release.version should not be necessary and might break something else.  Can you reproduce the failure?  What is the console log?  How did it not pick up the releaseversion from build.properties?

-Alex

On 9/17/19, 10:53 PM, "piotrz@apache.org" <pi...@apache.org> wrote:

    This is an automated email from the ASF dual-hosted git repository.
    
    piotrz pushed a commit to branch release/0.9.6
    in repository https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgitbox.apache.org%2Frepos%2Fasf%2Froyale-asjs.git&amp;data=02%7C01%7Caharui%40adobe.com%7C780e787968ef41434f6708d73bfc78e8%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C637043827860542212&amp;sdata=qQC5eeDfhB2Rz%2BdGfyJohLsFJuz%2BdkFCb8WPcrwW5PI%3D&amp;reserved=0
    
    
    The following commit(s) were added to refs/heads/release/0.9.6 by this push:
         new 84c2e85  Fix releasessteps script
    84c2e85 is described below
    
    commit 84c2e855c5a316ad21ff3a8ac6ac0cde826eecf0
    Author: Piotr Zarzycki <pi...@gmail.com>
    AuthorDate: Wed Sep 18 07:47:03 2019 +0200
    
        Fix releasessteps script
        
        - Add RoyaleUnit to swcs list and sign target
        - Fix property release.version
    ---
     releasesteps.xml | 43 +++++++++++++++++++++++--------------------
     1 file changed, 23 insertions(+), 20 deletions(-)
    
    diff --git a/releasesteps.xml b/releasesteps.xml
    index 01d0061..5c3edf1 100644
    --- a/releasesteps.xml
    +++ b/releasesteps.xml
    @@ -219,7 +219,7 @@
             <delete file="${artifactfolder}/swcs.txt" />
             <!-- this is a comma-delimited, no spaces, no-line-breaks list used to decide which swcs
              to skip in the rat-check of the binaries and which swcs to examine before approval -->
    -        <property name="swcs-list" value="Basic,Binding,Charts,Collections,Core,CreateJS,DragDrop,Effects,Express,Flat,FontAwesome,Formatters,GoogleMaps,Graphics,HTML,HTML5,Icons,JQuery,Jewel,Crux,Language,MXRoyale,MaterialDesignLite,Mobile,Network,Reflection,RoyaleSite,SparkRoyale,Storage,TLF,Text,XML,BasicTheme,JewelTheme"/>
    +        <property name="swcs-list" value="Basic,Binding,Charts,Collections,Core,CreateJS,DragDrop,Effects,Express,Flat,FontAwesome,Formatters,GoogleMaps,Graphics,HTML,HTML5,Icons,JQuery,Jewel,Crux,Language,MXRoyale,MaterialDesignLite,Mobile,Network,Reflection,RoyaleUnit,RoyaleSite,SparkRoyale,Storage,TLF,Text,XML,BasicTheme,JewelTheme"/>
             <echo file="${artifactfolder}/swcs.txt" message="${swcs-list}"/>
         </target>
         
    @@ -501,6 +501,9 @@
             <antcall target="sign-swc-artifacts" >
                 <param name="swcname" value="RoyaleSite" />
             </antcall>
    +		<antcall target="sign-swc-artifacts" >
    +            <param name="swcname" value="RoyaleUnit" />
    +        </antcall>
             <antcall target="sign-swc-artifacts" >
                 <param name="swcname" value="SparkRoyale" />
             </antcall>
    @@ -880,7 +883,7 @@
         
         <target name="build-ant-source" depends="check-air-home,check-playerglobal-home,check-runtime-env" >
             <mkdir dir="${artifactfolder}/sources" />
    -        <unzip src="${artifactfolder}/artifacts/apache-royale-${releaseversion}-src.zip" dest="${artifactfolder}/sources" >
    +        <unzip src="${artifactfolder}/artifacts/apache-royale-${release.version}-src.zip" dest="${artifactfolder}/sources" >
             </unzip>
             
             <ant dir="${artifactfolder}/sources/royale-asjs" target="all" />
    @@ -891,21 +894,21 @@
             <exec executable="git" dir="${artifactfolder}/sources" failonerror="true" >
                 <arg value="clone" />
                 <arg value="--branch" />
    -            <arg value="org.apache.royale.compiler-${releaseversion}-rc${rc}" />
    +            <arg value="org.apache.royale.compiler-${release.version}-rc${rc}" />
                 <arg value="https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Froyale-compiler.git&amp;data=02%7C01%7Caharui%40adobe.com%7C780e787968ef41434f6708d73bfc78e8%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C637043827860542212&amp;sdata=Qvy8HglyeoGWo6fV45tu7%2FBURYBHpRgWnx%2F4Mk33ZCo%3D&amp;reserved=0" />
                 <arg value="royale-compiler" />
             </exec>
             <exec executable="git" dir="${artifactfolder}/sources" failonerror="true" >
                 <arg value="clone" />
                 <arg value="--branch" />
    -            <arg value="org.apache.royale.typedefs-${releaseversion}-rc${rc}" />
    +            <arg value="org.apache.royale.typedefs-${release.version}-rc${rc}" />
                 <arg value="https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Froyale-typedefs.git&amp;data=02%7C01%7Caharui%40adobe.com%7C780e787968ef41434f6708d73bfc78e8%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C637043827860542212&amp;sdata=8jWPOp%2FH%2FKQr9u4ynBRMnXklSST2yaqUCTazqrc7vlA%3D&amp;reserved=0" />
                 <arg value="royale-typedefs" />
             </exec>
             <exec executable="git" dir="${artifactfolder}/sources" failonerror="true" >
                 <arg value="clone" />
                 <arg value="--branch" />
    -            <arg value="org.apache.royale.framework-${releaseversion}-rc${rc}" />
    +            <arg value="org.apache.royale.framework-${release.version}-rc${rc}" />
                 <arg value="https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Froyale-asjs.git&amp;data=02%7C01%7Caharui%40adobe.com%7C780e787968ef41434f6708d73bfc78e8%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C637043827860542212&amp;sdata=ZqgQ7OSyVeZ4kP84bS16LwO93mWJbv%2Fh22VOsJImUfA%3D&amp;reserved=0" />
                 <arg value="royale-asjs" />
             </exec>
    @@ -916,7 +919,7 @@
         <target name="validate-ant-source" description="do some tweaks so the clone is the same as maven source file set"  >
             <!-- unpack ant src which was generated from maven source package -->
             <mkdir dir="${artifactfolder}/artifacts/source" />
    -        <unzip src="${artifactfolder}/artifacts/apache-royale-${releaseversion}-src.zip" dest="${artifactfolder}/artifacts/source" />
    +        <unzip src="${artifactfolder}/artifacts/apache-royale-${release.version}-src.zip" dest="${artifactfolder}/artifacts/source" />
             <property name="artifactfolder" value="${artifactfolder}" />
             <!-- get the build date from flex-sdk-description and use it -->
             <xmlproperty file="${artifactfolder}/artifacts/source/royale-asjs/flex-sdk-description.xml" prefix="desc" />
    @@ -948,14 +951,14 @@
         </target>
         
         <target name="compare-src-zips" >
    -        <echo>releaseversion: ${releaseversion}</echo>
    +        <echo>release.version: ${release.version}</echo>
             <echo>copy src-zip file to a separate folder</echo>
             <mkdir dir="${artifactfolder}/artifacts/srczip" />
    -        <copy file="${artifactfolder}/artifacts/apache-royale-${releaseversion}-src.zip"
    -            tofile="${artifactfolder}/artifacts/srczip/apache-royale-${releaseversion}-src.zip" />
    +        <copy file="${artifactfolder}/artifacts/apache-royale-${release.version}-src.zip"
    +            tofile="${artifactfolder}/artifacts/srczip/apache-royale-${release.version}-src.zip" />
             <mkdir dir="${artifactfolder}/sources/srczip" />
    -        <copy file="${artifactfolder}/sources/royale-asjs/out/apache-royale-${releaseversion}-src.zip"
    -            tofile="${artifactfolder}/sources/srczip/apache-royale-${releaseversion}-src.zip" />
    +        <copy file="${artifactfolder}/sources/royale-asjs/out/apache-royale-${release.version}-src.zip"
    +            tofile="${artifactfolder}/sources/srczip/apache-royale-${release.version}-src.zip" />
             <echo>running reproducible build plugin on ${artifactfolder}/artifacts/srczip</echo>
             <exec executable="${mvn}" dir="${artifactfolder}" failonerror="true" >
                 <arg value="io.github.zlika:reproducible-build-maven-plugin:0.10:strip-jar" />
    @@ -967,10 +970,10 @@
                 <arg value="-Dreproducible.outputDirectory=${artifactfolder}/sources/srczip" />
             </exec>
             <condition property="filesmatch" value="true">
    -            <filesmatch file1="${artifactfolder}/artifacts/srczip/apache-royale-${releaseversion}-src.zip"
    -            file2="${artifactfolder}/sources/srczip/apache-royale-${releaseversion}-src.zip" />
    +            <filesmatch file1="${artifactfolder}/artifacts/srczip/apache-royale-${release.version}-src.zip"
    +            file2="${artifactfolder}/sources/srczip/apache-royale-${release.version}-src.zip" />
             </condition>
    -        <fail message="apache-royale-${releaseversion}-src.zip does not match" unless="filesmatch" />
    +        <fail message="apache-royale-${release.version}-src.zip does not match" unless="filesmatch" />
         </target>
         
         <target name="validate-ant-bits">
    @@ -980,16 +983,16 @@
             <mkdir dir="${artifactfolder}/artifacts/zip/js" />
             <mkdir dir="${artifactfolder}/artifacts/zip/swf" />
             <mkdir dir="${artifactfolder}/artifacts/zip/src" />
    -        <unzip src="${artifactfolder}/artifacts/apache-royale-${releaseversion}-src.zip" dest="${artifactfolder}/artifacts/zip/src" />
    -        <untar src="${artifactfolder}/artifacts/apache-royale-${releaseversion}-src.tar.gz" dest="${artifactfolder}/artifacts/tar/src" compression="gzip">
    +        <unzip src="${artifactfolder}/artifacts/apache-royale-${release.version}-src.zip" dest="${artifactfolder}/artifacts/zip/src" />
    +        <untar src="${artifactfolder}/artifacts/apache-royale-${release.version}-src.tar.gz" dest="${artifactfolder}/artifacts/tar/src" compression="gzip">
                 <cutdirsmapper dirs="1" />
             </untar>
    -        <unzip src="${artifactfolder}/artifacts/apache-royale-${releaseversion}-bin-js.zip" dest="${artifactfolder}/artifacts/zip/js" />
    -        <untar src="${artifactfolder}/artifacts/apache-royale-${releaseversion}-bin-js.tar.gz" dest="${artifactfolder}/artifacts/tar/js" compression="gzip">
    +        <unzip src="${artifactfolder}/artifacts/apache-royale-${release.version}-bin-js.zip" dest="${artifactfolder}/artifacts/zip/js" />
    +        <untar src="${artifactfolder}/artifacts/apache-royale-${release.version}-bin-js.tar.gz" dest="${artifactfolder}/artifacts/tar/js" compression="gzip">
                 <cutdirsmapper dirs="1" />
             </untar>
    -        <unzip src="${artifactfolder}/artifacts/apache-royale-${releaseversion}-bin-js-swf.zip" dest="${artifactfolder}/artifacts/zip/swf" />
    -        <untar src="${artifactfolder}/artifacts/apache-royale-${releaseversion}-bin-js-swf.tar.gz" dest="${artifactfolder}/artifacts/tar/swf" compression="gzip">
    +        <unzip src="${artifactfolder}/artifacts/apache-royale-${release.version}-bin-js-swf.zip" dest="${artifactfolder}/artifacts/zip/swf" />
    +        <untar src="${artifactfolder}/artifacts/apache-royale-${release.version}-bin-js-swf.tar.gz" dest="${artifactfolder}/artifacts/tar/swf" compression="gzip">
                 <cutdirsmapper dirs="1" />
             </untar>
             <antcall target="validate-ant-jars" />
    
    


Re: [royale-asjs] branch release/0.9.6 updated: Fix releasessteps script

Posted by Alex Harui <ah...@adobe.com.INVALID>.
The problem is that doesn't really make sense.  The build.properties contains a releaseversion property.  How is it that the scripts ran without loading build.properties?   Maybe you had a configuration problem instead of a flaw in the script that worked without problems for both you and me several times already.

IIRC, there are both release.version and releaseversion properties because release.version followed the pattern used in the past, but Jenkins won't let you specify a property with a "." in it, so I started using releaseversion where it was needed.  IMO, we would probably eventually remove all uses of release.version.  Now we are headed the other way.  Maybe it all worked this time, but as soon as someone changes some other releaseversion to release.version something will eventually break.

I understand this has taken much more time than you expected, but it really doesn't save the team time to be guessing at solutions and committing them just because they seem to work.

-Alex

On 9/19/19, 9:26 AM, "Piotr Zarzycki" <pi...@gmail.com> wrote:

    The only explanation which I have is that script failed with error saying
    that  releaseversion is not present. I don't have anything more to add
    unfortunately.
    
    czw., 19 wrz 2019 o 18:19 Alex Harui <ah...@adobe.com.invalid> napisał(a):
    
    > IMO, any change that can't be supported by a technical explanation
    > probably shouldn't be committed.  Can you explain why this change is
    > necessary?
    >
    > -Alex
    >
    > On 9/19/19, 1:03 AM, "Piotr Zarzycki" <pi...@gmail.com> wrote:
    >
    >     I just went trough whole the process and didn't have any kind of
    > problems.
    >     I'm pushing RC2. If you think I should revert that I can. I don't have
    >     anymore console log with it.
    >
    >     śr., 18 wrz 2019 o 16:18 Alex Harui <ah...@adobe.com.invalid>
    > napisał(a):
    >
    >     > Changing releaseversion to release.version should not be necessary
    > and
    >     > might break something else.  Can you reproduce the failure?  What is
    > the
    >     > console log?  How did it not pick up the releaseversion from
    >     > build.properties?
    >     >
    >     > -Alex
    >     >
    >     > On 9/17/19, 10:53 PM, "piotrz@apache.org" <pi...@apache.org> wrote:
    >     >
    >     >     This is an automated email from the ASF dual-hosted git
    > repository.
    >     >
    >     >     piotrz pushed a commit to branch release/0.9.6
    >     >     in repository
    >     >
    > https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgitbox.apache.org%2Frepos%2Fasf%2Froyale-asjs.git&amp;data=02%7C01%7Caharui%40adobe.com%7C7dc8d1a1a5d44668ee6d08d73d1e25cc%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C637045071998685687&amp;sdata=BYLKHd8%2Bjpf2WH%2FoczUZg0RcgxcFLsaSi4ec7p07ZLk%3D&amp;reserved=0
    >     >
    >     >
    >     >     The following commit(s) were added to refs/heads/release/0.9.6
    > by this
    >     > push:
    >     >          new 84c2e85  Fix releasessteps script
    >     >     84c2e85 is described below
    >     >
    >     >     commit 84c2e855c5a316ad21ff3a8ac6ac0cde826eecf0
    >     >     Author: Piotr Zarzycki <pi...@gmail.com>
    >     >     AuthorDate: Wed Sep 18 07:47:03 2019 +0200
    >     >
    >     >         Fix releasessteps script
    >     >
    >     >         - Add RoyaleUnit to swcs list and sign target
    >     >         - Fix property release.version
    >     >     ---
    >     >      releasesteps.xml | 43
    > +++++++++++++++++++++++--------------------
    >     >      1 file changed, 23 insertions(+), 20 deletions(-)
    >     >
    >     >     diff --git a/releasesteps.xml b/releasesteps.xml
    >     >     index 01d0061..5c3edf1 100644
    >     >     --- a/releasesteps.xml
    >     >     +++ b/releasesteps.xml
    >     >     @@ -219,7 +219,7 @@
    >     >              <delete file="${artifactfolder}/swcs.txt" />
    >     >              <!-- this is a comma-delimited, no spaces,
    > no-line-breaks
    >     > list used to decide which swcs
    >     >               to skip in the rat-check of the binaries and which
    > swcs to
    >     > examine before approval -->
    >     >     -        <property name="swcs-list"
    >     >
    > value="Basic,Binding,Charts,Collections,Core,CreateJS,DragDrop,Effects,Express,Flat,FontAwesome,Formatters,GoogleMaps,Graphics,HTML,HTML5,Icons,JQuery,Jewel,Crux,Language,MXRoyale,MaterialDesignLite,Mobile,Network,Reflection,RoyaleSite,SparkRoyale,Storage,TLF,Text,XML,BasicTheme,JewelTheme"/>
    >     >     +        <property name="swcs-list"
    >     >
    > value="Basic,Binding,Charts,Collections,Core,CreateJS,DragDrop,Effects,Express,Flat,FontAwesome,Formatters,GoogleMaps,Graphics,HTML,HTML5,Icons,JQuery,Jewel,Crux,Language,MXRoyale,MaterialDesignLite,Mobile,Network,Reflection,RoyaleUnit,RoyaleSite,SparkRoyale,Storage,TLF,Text,XML,BasicTheme,JewelTheme"/>
    >     >              <echo file="${artifactfolder}/swcs.txt"
    >     > message="${swcs-list}"/>
    >     >          </target>
    >     >
    >     >     @@ -501,6 +501,9 @@
    >     >              <antcall target="sign-swc-artifacts" >
    >     >                  <param name="swcname" value="RoyaleSite" />
    >     >              </antcall>
    >     >     +           <antcall target="sign-swc-artifacts" >
    >     >     +            <param name="swcname" value="RoyaleUnit" />
    >     >     +        </antcall>
    >     >              <antcall target="sign-swc-artifacts" >
    >     >                  <param name="swcname" value="SparkRoyale" />
    >     >              </antcall>
    >     >     @@ -880,7 +883,7 @@
    >     >
    >     >          <target name="build-ant-source"
    >     > depends="check-air-home,check-playerglobal-home,check-runtime-env" >
    >     >              <mkdir dir="${artifactfolder}/sources" />
    >     >     -        <unzip
    >     >
    > src="${artifactfolder}/artifacts/apache-royale-${releaseversion}-src.zip"
    >     > dest="${artifactfolder}/sources" >
    >     >     +        <unzip
    >     >
    > src="${artifactfolder}/artifacts/apache-royale-${release.version}-src.zip"
    >     > dest="${artifactfolder}/sources" >
    >     >              </unzip>
    >     >
    >     >              <ant dir="${artifactfolder}/sources/royale-asjs"
    > target="all"
    >     > />
    >     >     @@ -891,21 +894,21 @@
    >     >              <exec executable="git" dir="${artifactfolder}/sources"
    >     > failonerror="true" >
    >     >                  <arg value="clone" />
    >     >                  <arg value="--branch" />
    >     >     -            <arg
    >     > value="org.apache.royale.compiler-${releaseversion}-rc${rc}" />
    >     >     +            <arg
    >     > value="org.apache.royale.compiler-${release.version}-rc${rc}" />
    >     >                  <arg value="
    >     >
    > https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Froyale-compiler.git&amp;data=02%7C01%7Caharui%40adobe.com%7C7dc8d1a1a5d44668ee6d08d73d1e25cc%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C637045071998685687&amp;sdata=%2FGZmdSHRfVWvWhKsWl9VbIkegqM05Rmc84IwCmMpOEU%3D&amp;reserved=0
    > "
    >     > />
    >     >                  <arg value="royale-compiler" />
    >     >              </exec>
    >     >              <exec executable="git" dir="${artifactfolder}/sources"
    >     > failonerror="true" >
    >     >                  <arg value="clone" />
    >     >                  <arg value="--branch" />
    >     >     -            <arg
    >     > value="org.apache.royale.typedefs-${releaseversion}-rc${rc}" />
    >     >     +            <arg
    >     > value="org.apache.royale.typedefs-${release.version}-rc${rc}" />
    >     >                  <arg value="
    >     >
    > https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Froyale-typedefs.git&amp;data=02%7C01%7Caharui%40adobe.com%7C7dc8d1a1a5d44668ee6d08d73d1e25cc%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C637045071998685687&amp;sdata=K13VLheOk4NP6j4tlJxGN7u0FcL21wKdduMCeSLiqQ8%3D&amp;reserved=0
    > "
    >     > />
    >     >                  <arg value="royale-typedefs" />
    >     >              </exec>
    >     >              <exec executable="git" dir="${artifactfolder}/sources"
    >     > failonerror="true" >
    >     >                  <arg value="clone" />
    >     >                  <arg value="--branch" />
    >     >     -            <arg
    >     > value="org.apache.royale.framework-${releaseversion}-rc${rc}" />
    >     >     +            <arg
    >     > value="org.apache.royale.framework-${release.version}-rc${rc}" />
    >     >                  <arg value="
    >     >
    > https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Froyale-asjs.git&amp;data=02%7C01%7Caharui%40adobe.com%7C7dc8d1a1a5d44668ee6d08d73d1e25cc%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C637045071998695683&amp;sdata=SPEMCJd06bRGjAGB%2Bjz6kVS%2BVqr7iYmQUvMmGo1nWaE%3D&amp;reserved=0
    > "
    >     > />
    >     >                  <arg value="royale-asjs" />
    >     >              </exec>
    >     >     @@ -916,7 +919,7 @@
    >     >          <target name="validate-ant-source" description="do some
    > tweaks so
    >     > the clone is the same as maven source file set"  >
    >     >              <!-- unpack ant src which was generated from maven
    > source
    >     > package -->
    >     >              <mkdir dir="${artifactfolder}/artifacts/source" />
    >     >     -        <unzip
    >     >
    > src="${artifactfolder}/artifacts/apache-royale-${releaseversion}-src.zip"
    >     > dest="${artifactfolder}/artifacts/source" />
    >     >     +        <unzip
    >     >
    > src="${artifactfolder}/artifacts/apache-royale-${release.version}-src.zip"
    >     > dest="${artifactfolder}/artifacts/source" />
    >     >              <property name="artifactfolder"
    > value="${artifactfolder}" />
    >     >              <!-- get the build date from flex-sdk-description and
    > use it
    >     > -->
    >     >              <xmlproperty
    >     >
    > file="${artifactfolder}/artifacts/source/royale-asjs/flex-sdk-description.xml"
    >     > prefix="desc" />
    >     >     @@ -948,14 +951,14 @@
    >     >          </target>
    >     >
    >     >          <target name="compare-src-zips" >
    >     >     -        <echo>releaseversion: ${releaseversion}</echo>
    >     >     +        <echo>release.version: ${release.version}</echo>
    >     >              <echo>copy src-zip file to a separate folder</echo>
    >     >              <mkdir dir="${artifactfolder}/artifacts/srczip" />
    >     >     -        <copy
    >     >
    > file="${artifactfolder}/artifacts/apache-royale-${releaseversion}-src.zip"
    >     >     -
    >     >
    > tofile="${artifactfolder}/artifacts/srczip/apache-royale-${releaseversion}-src.zip"
    >     > />
    >     >     +        <copy
    >     >
    > file="${artifactfolder}/artifacts/apache-royale-${release.version}-src.zip"
    >     >     +
    >     >
    > tofile="${artifactfolder}/artifacts/srczip/apache-royale-${release.version}-src.zip"
    >     > />
    >     >              <mkdir dir="${artifactfolder}/sources/srczip" />
    >     >     -        <copy
    >     >
    > file="${artifactfolder}/sources/royale-asjs/out/apache-royale-${releaseversion}-src.zip"
    >     >     -
    >     >
    > tofile="${artifactfolder}/sources/srczip/apache-royale-${releaseversion}-src.zip"
    >     > />
    >     >     +        <copy
    >     >
    > file="${artifactfolder}/sources/royale-asjs/out/apache-royale-${release.version}-src.zip"
    >     >     +
    >     >
    > tofile="${artifactfolder}/sources/srczip/apache-royale-${release.version}-src.zip"
    >     > />
    >     >              <echo>running reproducible build plugin on
    >     > ${artifactfolder}/artifacts/srczip</echo>
    >     >              <exec executable="${mvn}" dir="${artifactfolder}"
    >     > failonerror="true" >
    >     >                  <arg
    >     >
    > value="io.github.zlika:reproducible-build-maven-plugin:0.10:strip-jar" />
    >     >     @@ -967,10 +970,10 @@
    >     >                  <arg
    >     >
    > value="-Dreproducible.outputDirectory=${artifactfolder}/sources/srczip" />
    >     >              </exec>
    >     >              <condition property="filesmatch" value="true">
    >     >     -            <filesmatch
    >     >
    > file1="${artifactfolder}/artifacts/srczip/apache-royale-${releaseversion}-src.zip"
    >     >     -
    >     >
    > file2="${artifactfolder}/sources/srczip/apache-royale-${releaseversion}-src.zip"
    >     > />
    >     >     +            <filesmatch
    >     >
    > file1="${artifactfolder}/artifacts/srczip/apache-royale-${release.version}-src.zip"
    >     >     +
    >     >
    > file2="${artifactfolder}/sources/srczip/apache-royale-${release.version}-src.zip"
    >     > />
    >     >              </condition>
    >     >     -        <fail message="apache-royale-${releaseversion}-src.zip
    > does
    >     > not match" unless="filesmatch" />
    >     >     +        <fail message="apache-royale-${release.version}-src.zip
    > does
    >     > not match" unless="filesmatch" />
    >     >          </target>
    >     >
    >     >          <target name="validate-ant-bits">
    >     >     @@ -980,16 +983,16 @@
    >     >              <mkdir dir="${artifactfolder}/artifacts/zip/js" />
    >     >              <mkdir dir="${artifactfolder}/artifacts/zip/swf" />
    >     >              <mkdir dir="${artifactfolder}/artifacts/zip/src" />
    >     >     -        <unzip
    >     >
    > src="${artifactfolder}/artifacts/apache-royale-${releaseversion}-src.zip"
    >     > dest="${artifactfolder}/artifacts/zip/src" />
    >     >     -        <untar
    >     >
    > src="${artifactfolder}/artifacts/apache-royale-${releaseversion}-src.tar.gz"
    >     > dest="${artifactfolder}/artifacts/tar/src" compression="gzip">
    >     >     +        <unzip
    >     >
    > src="${artifactfolder}/artifacts/apache-royale-${release.version}-src.zip"
    >     > dest="${artifactfolder}/artifacts/zip/src" />
    >     >     +        <untar
    >     >
    > src="${artifactfolder}/artifacts/apache-royale-${release.version}-src.tar.gz"
    >     > dest="${artifactfolder}/artifacts/tar/src" compression="gzip">
    >     >                  <cutdirsmapper dirs="1" />
    >     >              </untar>
    >     >     -        <unzip
    >     >
    > src="${artifactfolder}/artifacts/apache-royale-${releaseversion}-bin-js.zip"
    >     > dest="${artifactfolder}/artifacts/zip/js" />
    >     >     -        <untar
    >     >
    > src="${artifactfolder}/artifacts/apache-royale-${releaseversion}-bin-js.tar.gz"
    >     > dest="${artifactfolder}/artifacts/tar/js" compression="gzip">
    >     >     +        <unzip
    >     >
    > src="${artifactfolder}/artifacts/apache-royale-${release.version}-bin-js.zip"
    >     > dest="${artifactfolder}/artifacts/zip/js" />
    >     >     +        <untar
    >     >
    > src="${artifactfolder}/artifacts/apache-royale-${release.version}-bin-js.tar.gz"
    >     > dest="${artifactfolder}/artifacts/tar/js" compression="gzip">
    >     >                  <cutdirsmapper dirs="1" />
    >     >              </untar>
    >     >     -        <unzip
    >     >
    > src="${artifactfolder}/artifacts/apache-royale-${releaseversion}-bin-js-swf.zip"
    >     > dest="${artifactfolder}/artifacts/zip/swf" />
    >     >     -        <untar
    >     >
    > src="${artifactfolder}/artifacts/apache-royale-${releaseversion}-bin-js-swf.tar.gz"
    >     > dest="${artifactfolder}/artifacts/tar/swf" compression="gzip">
    >     >     +        <unzip
    >     >
    > src="${artifactfolder}/artifacts/apache-royale-${release.version}-bin-js-swf.zip"
    >     > dest="${artifactfolder}/artifacts/zip/swf" />
    >     >     +        <untar
    >     >
    > src="${artifactfolder}/artifacts/apache-royale-${release.version}-bin-js-swf.tar.gz"
    >     > dest="${artifactfolder}/artifacts/tar/swf" compression="gzip">
    >     >                  <cutdirsmapper dirs="1" />
    >     >              </untar>
    >     >              <antcall target="validate-ant-jars" />
    >     >
    >     >
    >     >
    >     >
    >
    >     --
    >
    >     Piotr Zarzycki
    >
    >     Patreon: *
    > https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.patreon.com%2Fpiotrzarzycki&amp;data=02%7C01%7Caharui%40adobe.com%7C7dc8d1a1a5d44668ee6d08d73d1e25cc%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C637045071998695683&amp;sdata=xZGXi7uqXYJ%2BD4G6SJXZWxa%2Ft7aDqy1wT7eg9sOU5LY%3D&amp;reserved=0
    >     <
    > https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.patreon.com%2Fpiotrzarzycki&amp;data=02%7C01%7Caharui%40adobe.com%7C7dc8d1a1a5d44668ee6d08d73d1e25cc%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C637045071998695683&amp;sdata=xZGXi7uqXYJ%2BD4G6SJXZWxa%2Ft7aDqy1wT7eg9sOU5LY%3D&amp;reserved=0
    > >*
    >
    >
    >
    
    -- 
    
    Piotr Zarzycki
    
    Patreon: *https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.patreon.com%2Fpiotrzarzycki&amp;data=02%7C01%7Caharui%40adobe.com%7C7dc8d1a1a5d44668ee6d08d73d1e25cc%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C637045071998695683&amp;sdata=xZGXi7uqXYJ%2BD4G6SJXZWxa%2Ft7aDqy1wT7eg9sOU5LY%3D&amp;reserved=0
    <https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.patreon.com%2Fpiotrzarzycki&amp;data=02%7C01%7Caharui%40adobe.com%7C7dc8d1a1a5d44668ee6d08d73d1e25cc%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C637045071998695683&amp;sdata=xZGXi7uqXYJ%2BD4G6SJXZWxa%2Ft7aDqy1wT7eg9sOU5LY%3D&amp;reserved=0>*
    


Re: [royale-asjs] branch release/0.9.6 updated: Fix releasessteps script

Posted by Piotr Zarzycki <pi...@gmail.com>.
The only explanation which I have is that script failed with error saying
that  releaseversion is not present. I don't have anything more to add
unfortunately.

czw., 19 wrz 2019 o 18:19 Alex Harui <ah...@adobe.com.invalid> napisał(a):

> IMO, any change that can't be supported by a technical explanation
> probably shouldn't be committed.  Can you explain why this change is
> necessary?
>
> -Alex
>
> On 9/19/19, 1:03 AM, "Piotr Zarzycki" <pi...@gmail.com> wrote:
>
>     I just went trough whole the process and didn't have any kind of
> problems.
>     I'm pushing RC2. If you think I should revert that I can. I don't have
>     anymore console log with it.
>
>     śr., 18 wrz 2019 o 16:18 Alex Harui <ah...@adobe.com.invalid>
> napisał(a):
>
>     > Changing releaseversion to release.version should not be necessary
> and
>     > might break something else.  Can you reproduce the failure?  What is
> the
>     > console log?  How did it not pick up the releaseversion from
>     > build.properties?
>     >
>     > -Alex
>     >
>     > On 9/17/19, 10:53 PM, "piotrz@apache.org" <pi...@apache.org> wrote:
>     >
>     >     This is an automated email from the ASF dual-hosted git
> repository.
>     >
>     >     piotrz pushed a commit to branch release/0.9.6
>     >     in repository
>     >
> https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgitbox.apache.org%2Frepos%2Fasf%2Froyale-asjs.git&amp;data=02%7C01%7Caharui%40adobe.com%7Cd1486c9d132a491e274108d73cd7cff9%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C637044769931764780&amp;sdata=ETBx881FDXp%2FwR9qLHW95kJl5uNPTBGDOqsjPvoIBlI%3D&amp;reserved=0
>     >
>     >
>     >     The following commit(s) were added to refs/heads/release/0.9.6
> by this
>     > push:
>     >          new 84c2e85  Fix releasessteps script
>     >     84c2e85 is described below
>     >
>     >     commit 84c2e855c5a316ad21ff3a8ac6ac0cde826eecf0
>     >     Author: Piotr Zarzycki <pi...@gmail.com>
>     >     AuthorDate: Wed Sep 18 07:47:03 2019 +0200
>     >
>     >         Fix releasessteps script
>     >
>     >         - Add RoyaleUnit to swcs list and sign target
>     >         - Fix property release.version
>     >     ---
>     >      releasesteps.xml | 43
> +++++++++++++++++++++++--------------------
>     >      1 file changed, 23 insertions(+), 20 deletions(-)
>     >
>     >     diff --git a/releasesteps.xml b/releasesteps.xml
>     >     index 01d0061..5c3edf1 100644
>     >     --- a/releasesteps.xml
>     >     +++ b/releasesteps.xml
>     >     @@ -219,7 +219,7 @@
>     >              <delete file="${artifactfolder}/swcs.txt" />
>     >              <!-- this is a comma-delimited, no spaces,
> no-line-breaks
>     > list used to decide which swcs
>     >               to skip in the rat-check of the binaries and which
> swcs to
>     > examine before approval -->
>     >     -        <property name="swcs-list"
>     >
> value="Basic,Binding,Charts,Collections,Core,CreateJS,DragDrop,Effects,Express,Flat,FontAwesome,Formatters,GoogleMaps,Graphics,HTML,HTML5,Icons,JQuery,Jewel,Crux,Language,MXRoyale,MaterialDesignLite,Mobile,Network,Reflection,RoyaleSite,SparkRoyale,Storage,TLF,Text,XML,BasicTheme,JewelTheme"/>
>     >     +        <property name="swcs-list"
>     >
> value="Basic,Binding,Charts,Collections,Core,CreateJS,DragDrop,Effects,Express,Flat,FontAwesome,Formatters,GoogleMaps,Graphics,HTML,HTML5,Icons,JQuery,Jewel,Crux,Language,MXRoyale,MaterialDesignLite,Mobile,Network,Reflection,RoyaleUnit,RoyaleSite,SparkRoyale,Storage,TLF,Text,XML,BasicTheme,JewelTheme"/>
>     >              <echo file="${artifactfolder}/swcs.txt"
>     > message="${swcs-list}"/>
>     >          </target>
>     >
>     >     @@ -501,6 +501,9 @@
>     >              <antcall target="sign-swc-artifacts" >
>     >                  <param name="swcname" value="RoyaleSite" />
>     >              </antcall>
>     >     +           <antcall target="sign-swc-artifacts" >
>     >     +            <param name="swcname" value="RoyaleUnit" />
>     >     +        </antcall>
>     >              <antcall target="sign-swc-artifacts" >
>     >                  <param name="swcname" value="SparkRoyale" />
>     >              </antcall>
>     >     @@ -880,7 +883,7 @@
>     >
>     >          <target name="build-ant-source"
>     > depends="check-air-home,check-playerglobal-home,check-runtime-env" >
>     >              <mkdir dir="${artifactfolder}/sources" />
>     >     -        <unzip
>     >
> src="${artifactfolder}/artifacts/apache-royale-${releaseversion}-src.zip"
>     > dest="${artifactfolder}/sources" >
>     >     +        <unzip
>     >
> src="${artifactfolder}/artifacts/apache-royale-${release.version}-src.zip"
>     > dest="${artifactfolder}/sources" >
>     >              </unzip>
>     >
>     >              <ant dir="${artifactfolder}/sources/royale-asjs"
> target="all"
>     > />
>     >     @@ -891,21 +894,21 @@
>     >              <exec executable="git" dir="${artifactfolder}/sources"
>     > failonerror="true" >
>     >                  <arg value="clone" />
>     >                  <arg value="--branch" />
>     >     -            <arg
>     > value="org.apache.royale.compiler-${releaseversion}-rc${rc}" />
>     >     +            <arg
>     > value="org.apache.royale.compiler-${release.version}-rc${rc}" />
>     >                  <arg value="
>     >
> https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Froyale-compiler.git&amp;data=02%7C01%7Caharui%40adobe.com%7Cd1486c9d132a491e274108d73cd7cff9%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C637044769931764780&amp;sdata=0WaXWxReg5ZwWf1MXEnGNpICKAmamq59g885pp3I0cQ%3D&amp;reserved=0
> "
>     > />
>     >                  <arg value="royale-compiler" />
>     >              </exec>
>     >              <exec executable="git" dir="${artifactfolder}/sources"
>     > failonerror="true" >
>     >                  <arg value="clone" />
>     >                  <arg value="--branch" />
>     >     -            <arg
>     > value="org.apache.royale.typedefs-${releaseversion}-rc${rc}" />
>     >     +            <arg
>     > value="org.apache.royale.typedefs-${release.version}-rc${rc}" />
>     >                  <arg value="
>     >
> https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Froyale-typedefs.git&amp;data=02%7C01%7Caharui%40adobe.com%7Cd1486c9d132a491e274108d73cd7cff9%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C637044769931764780&amp;sdata=qurNjh0lV7TpvkKjQAnBh48l2YPDxCcxTBP%2BNQe1pH8%3D&amp;reserved=0
> "
>     > />
>     >                  <arg value="royale-typedefs" />
>     >              </exec>
>     >              <exec executable="git" dir="${artifactfolder}/sources"
>     > failonerror="true" >
>     >                  <arg value="clone" />
>     >                  <arg value="--branch" />
>     >     -            <arg
>     > value="org.apache.royale.framework-${releaseversion}-rc${rc}" />
>     >     +            <arg
>     > value="org.apache.royale.framework-${release.version}-rc${rc}" />
>     >                  <arg value="
>     >
> https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Froyale-asjs.git&amp;data=02%7C01%7Caharui%40adobe.com%7Cd1486c9d132a491e274108d73cd7cff9%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C637044769931774775&amp;sdata=2lflk%2BpR5JaskhugM5eptFRYu04ry%2FjE52cJ4ljl5YU%3D&amp;reserved=0
> "
>     > />
>     >                  <arg value="royale-asjs" />
>     >              </exec>
>     >     @@ -916,7 +919,7 @@
>     >          <target name="validate-ant-source" description="do some
> tweaks so
>     > the clone is the same as maven source file set"  >
>     >              <!-- unpack ant src which was generated from maven
> source
>     > package -->
>     >              <mkdir dir="${artifactfolder}/artifacts/source" />
>     >     -        <unzip
>     >
> src="${artifactfolder}/artifacts/apache-royale-${releaseversion}-src.zip"
>     > dest="${artifactfolder}/artifacts/source" />
>     >     +        <unzip
>     >
> src="${artifactfolder}/artifacts/apache-royale-${release.version}-src.zip"
>     > dest="${artifactfolder}/artifacts/source" />
>     >              <property name="artifactfolder"
> value="${artifactfolder}" />
>     >              <!-- get the build date from flex-sdk-description and
> use it
>     > -->
>     >              <xmlproperty
>     >
> file="${artifactfolder}/artifacts/source/royale-asjs/flex-sdk-description.xml"
>     > prefix="desc" />
>     >     @@ -948,14 +951,14 @@
>     >          </target>
>     >
>     >          <target name="compare-src-zips" >
>     >     -        <echo>releaseversion: ${releaseversion}</echo>
>     >     +        <echo>release.version: ${release.version}</echo>
>     >              <echo>copy src-zip file to a separate folder</echo>
>     >              <mkdir dir="${artifactfolder}/artifacts/srczip" />
>     >     -        <copy
>     >
> file="${artifactfolder}/artifacts/apache-royale-${releaseversion}-src.zip"
>     >     -
>     >
> tofile="${artifactfolder}/artifacts/srczip/apache-royale-${releaseversion}-src.zip"
>     > />
>     >     +        <copy
>     >
> file="${artifactfolder}/artifacts/apache-royale-${release.version}-src.zip"
>     >     +
>     >
> tofile="${artifactfolder}/artifacts/srczip/apache-royale-${release.version}-src.zip"
>     > />
>     >              <mkdir dir="${artifactfolder}/sources/srczip" />
>     >     -        <copy
>     >
> file="${artifactfolder}/sources/royale-asjs/out/apache-royale-${releaseversion}-src.zip"
>     >     -
>     >
> tofile="${artifactfolder}/sources/srczip/apache-royale-${releaseversion}-src.zip"
>     > />
>     >     +        <copy
>     >
> file="${artifactfolder}/sources/royale-asjs/out/apache-royale-${release.version}-src.zip"
>     >     +
>     >
> tofile="${artifactfolder}/sources/srczip/apache-royale-${release.version}-src.zip"
>     > />
>     >              <echo>running reproducible build plugin on
>     > ${artifactfolder}/artifacts/srczip</echo>
>     >              <exec executable="${mvn}" dir="${artifactfolder}"
>     > failonerror="true" >
>     >                  <arg
>     >
> value="io.github.zlika:reproducible-build-maven-plugin:0.10:strip-jar" />
>     >     @@ -967,10 +970,10 @@
>     >                  <arg
>     >
> value="-Dreproducible.outputDirectory=${artifactfolder}/sources/srczip" />
>     >              </exec>
>     >              <condition property="filesmatch" value="true">
>     >     -            <filesmatch
>     >
> file1="${artifactfolder}/artifacts/srczip/apache-royale-${releaseversion}-src.zip"
>     >     -
>     >
> file2="${artifactfolder}/sources/srczip/apache-royale-${releaseversion}-src.zip"
>     > />
>     >     +            <filesmatch
>     >
> file1="${artifactfolder}/artifacts/srczip/apache-royale-${release.version}-src.zip"
>     >     +
>     >
> file2="${artifactfolder}/sources/srczip/apache-royale-${release.version}-src.zip"
>     > />
>     >              </condition>
>     >     -        <fail message="apache-royale-${releaseversion}-src.zip
> does
>     > not match" unless="filesmatch" />
>     >     +        <fail message="apache-royale-${release.version}-src.zip
> does
>     > not match" unless="filesmatch" />
>     >          </target>
>     >
>     >          <target name="validate-ant-bits">
>     >     @@ -980,16 +983,16 @@
>     >              <mkdir dir="${artifactfolder}/artifacts/zip/js" />
>     >              <mkdir dir="${artifactfolder}/artifacts/zip/swf" />
>     >              <mkdir dir="${artifactfolder}/artifacts/zip/src" />
>     >     -        <unzip
>     >
> src="${artifactfolder}/artifacts/apache-royale-${releaseversion}-src.zip"
>     > dest="${artifactfolder}/artifacts/zip/src" />
>     >     -        <untar
>     >
> src="${artifactfolder}/artifacts/apache-royale-${releaseversion}-src.tar.gz"
>     > dest="${artifactfolder}/artifacts/tar/src" compression="gzip">
>     >     +        <unzip
>     >
> src="${artifactfolder}/artifacts/apache-royale-${release.version}-src.zip"
>     > dest="${artifactfolder}/artifacts/zip/src" />
>     >     +        <untar
>     >
> src="${artifactfolder}/artifacts/apache-royale-${release.version}-src.tar.gz"
>     > dest="${artifactfolder}/artifacts/tar/src" compression="gzip">
>     >                  <cutdirsmapper dirs="1" />
>     >              </untar>
>     >     -        <unzip
>     >
> src="${artifactfolder}/artifacts/apache-royale-${releaseversion}-bin-js.zip"
>     > dest="${artifactfolder}/artifacts/zip/js" />
>     >     -        <untar
>     >
> src="${artifactfolder}/artifacts/apache-royale-${releaseversion}-bin-js.tar.gz"
>     > dest="${artifactfolder}/artifacts/tar/js" compression="gzip">
>     >     +        <unzip
>     >
> src="${artifactfolder}/artifacts/apache-royale-${release.version}-bin-js.zip"
>     > dest="${artifactfolder}/artifacts/zip/js" />
>     >     +        <untar
>     >
> src="${artifactfolder}/artifacts/apache-royale-${release.version}-bin-js.tar.gz"
>     > dest="${artifactfolder}/artifacts/tar/js" compression="gzip">
>     >                  <cutdirsmapper dirs="1" />
>     >              </untar>
>     >     -        <unzip
>     >
> src="${artifactfolder}/artifacts/apache-royale-${releaseversion}-bin-js-swf.zip"
>     > dest="${artifactfolder}/artifacts/zip/swf" />
>     >     -        <untar
>     >
> src="${artifactfolder}/artifacts/apache-royale-${releaseversion}-bin-js-swf.tar.gz"
>     > dest="${artifactfolder}/artifacts/tar/swf" compression="gzip">
>     >     +        <unzip
>     >
> src="${artifactfolder}/artifacts/apache-royale-${release.version}-bin-js-swf.zip"
>     > dest="${artifactfolder}/artifacts/zip/swf" />
>     >     +        <untar
>     >
> src="${artifactfolder}/artifacts/apache-royale-${release.version}-bin-js-swf.tar.gz"
>     > dest="${artifactfolder}/artifacts/tar/swf" compression="gzip">
>     >                  <cutdirsmapper dirs="1" />
>     >              </untar>
>     >              <antcall target="validate-ant-jars" />
>     >
>     >
>     >
>     >
>
>     --
>
>     Piotr Zarzycki
>
>     Patreon: *
> https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.patreon.com%2Fpiotrzarzycki&amp;data=02%7C01%7Caharui%40adobe.com%7Cd1486c9d132a491e274108d73cd7cff9%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C637044769931774775&amp;sdata=h%2Bj%2BmAfNdnDwvj0ECMxyFKlc%2BY4jc38MhaXirqESy1U%3D&amp;reserved=0
>     <
> https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.patreon.com%2Fpiotrzarzycki&amp;data=02%7C01%7Caharui%40adobe.com%7Cd1486c9d132a491e274108d73cd7cff9%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C637044769931774775&amp;sdata=h%2Bj%2BmAfNdnDwvj0ECMxyFKlc%2BY4jc38MhaXirqESy1U%3D&amp;reserved=0
> >*
>
>
>

-- 

Piotr Zarzycki

Patreon: *https://www.patreon.com/piotrzarzycki
<https://www.patreon.com/piotrzarzycki>*

Re: [royale-asjs] branch release/0.9.6 updated: Fix releasessteps script

Posted by Alex Harui <ah...@adobe.com.INVALID>.
IMO, any change that can't be supported by a technical explanation probably shouldn't be committed.  Can you explain why this change is necessary?

-Alex

On 9/19/19, 1:03 AM, "Piotr Zarzycki" <pi...@gmail.com> wrote:

    I just went trough whole the process and didn't have any kind of problems.
    I'm pushing RC2. If you think I should revert that I can. I don't have
    anymore console log with it.
    
    śr., 18 wrz 2019 o 16:18 Alex Harui <ah...@adobe.com.invalid> napisał(a):
    
    > Changing releaseversion to release.version should not be necessary and
    > might break something else.  Can you reproduce the failure?  What is the
    > console log?  How did it not pick up the releaseversion from
    > build.properties?
    >
    > -Alex
    >
    > On 9/17/19, 10:53 PM, "piotrz@apache.org" <pi...@apache.org> wrote:
    >
    >     This is an automated email from the ASF dual-hosted git repository.
    >
    >     piotrz pushed a commit to branch release/0.9.6
    >     in repository
    > https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgitbox.apache.org%2Frepos%2Fasf%2Froyale-asjs.git&amp;data=02%7C01%7Caharui%40adobe.com%7Cd1486c9d132a491e274108d73cd7cff9%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C637044769931764780&amp;sdata=ETBx881FDXp%2FwR9qLHW95kJl5uNPTBGDOqsjPvoIBlI%3D&amp;reserved=0
    >
    >
    >     The following commit(s) were added to refs/heads/release/0.9.6 by this
    > push:
    >          new 84c2e85  Fix releasessteps script
    >     84c2e85 is described below
    >
    >     commit 84c2e855c5a316ad21ff3a8ac6ac0cde826eecf0
    >     Author: Piotr Zarzycki <pi...@gmail.com>
    >     AuthorDate: Wed Sep 18 07:47:03 2019 +0200
    >
    >         Fix releasessteps script
    >
    >         - Add RoyaleUnit to swcs list and sign target
    >         - Fix property release.version
    >     ---
    >      releasesteps.xml | 43 +++++++++++++++++++++++--------------------
    >      1 file changed, 23 insertions(+), 20 deletions(-)
    >
    >     diff --git a/releasesteps.xml b/releasesteps.xml
    >     index 01d0061..5c3edf1 100644
    >     --- a/releasesteps.xml
    >     +++ b/releasesteps.xml
    >     @@ -219,7 +219,7 @@
    >              <delete file="${artifactfolder}/swcs.txt" />
    >              <!-- this is a comma-delimited, no spaces, no-line-breaks
    > list used to decide which swcs
    >               to skip in the rat-check of the binaries and which swcs to
    > examine before approval -->
    >     -        <property name="swcs-list"
    > value="Basic,Binding,Charts,Collections,Core,CreateJS,DragDrop,Effects,Express,Flat,FontAwesome,Formatters,GoogleMaps,Graphics,HTML,HTML5,Icons,JQuery,Jewel,Crux,Language,MXRoyale,MaterialDesignLite,Mobile,Network,Reflection,RoyaleSite,SparkRoyale,Storage,TLF,Text,XML,BasicTheme,JewelTheme"/>
    >     +        <property name="swcs-list"
    > value="Basic,Binding,Charts,Collections,Core,CreateJS,DragDrop,Effects,Express,Flat,FontAwesome,Formatters,GoogleMaps,Graphics,HTML,HTML5,Icons,JQuery,Jewel,Crux,Language,MXRoyale,MaterialDesignLite,Mobile,Network,Reflection,RoyaleUnit,RoyaleSite,SparkRoyale,Storage,TLF,Text,XML,BasicTheme,JewelTheme"/>
    >              <echo file="${artifactfolder}/swcs.txt"
    > message="${swcs-list}"/>
    >          </target>
    >
    >     @@ -501,6 +501,9 @@
    >              <antcall target="sign-swc-artifacts" >
    >                  <param name="swcname" value="RoyaleSite" />
    >              </antcall>
    >     +           <antcall target="sign-swc-artifacts" >
    >     +            <param name="swcname" value="RoyaleUnit" />
    >     +        </antcall>
    >              <antcall target="sign-swc-artifacts" >
    >                  <param name="swcname" value="SparkRoyale" />
    >              </antcall>
    >     @@ -880,7 +883,7 @@
    >
    >          <target name="build-ant-source"
    > depends="check-air-home,check-playerglobal-home,check-runtime-env" >
    >              <mkdir dir="${artifactfolder}/sources" />
    >     -        <unzip
    > src="${artifactfolder}/artifacts/apache-royale-${releaseversion}-src.zip"
    > dest="${artifactfolder}/sources" >
    >     +        <unzip
    > src="${artifactfolder}/artifacts/apache-royale-${release.version}-src.zip"
    > dest="${artifactfolder}/sources" >
    >              </unzip>
    >
    >              <ant dir="${artifactfolder}/sources/royale-asjs" target="all"
    > />
    >     @@ -891,21 +894,21 @@
    >              <exec executable="git" dir="${artifactfolder}/sources"
    > failonerror="true" >
    >                  <arg value="clone" />
    >                  <arg value="--branch" />
    >     -            <arg
    > value="org.apache.royale.compiler-${releaseversion}-rc${rc}" />
    >     +            <arg
    > value="org.apache.royale.compiler-${release.version}-rc${rc}" />
    >                  <arg value="
    > https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Froyale-compiler.git&amp;data=02%7C01%7Caharui%40adobe.com%7Cd1486c9d132a491e274108d73cd7cff9%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C637044769931764780&amp;sdata=0WaXWxReg5ZwWf1MXEnGNpICKAmamq59g885pp3I0cQ%3D&amp;reserved=0"
    > />
    >                  <arg value="royale-compiler" />
    >              </exec>
    >              <exec executable="git" dir="${artifactfolder}/sources"
    > failonerror="true" >
    >                  <arg value="clone" />
    >                  <arg value="--branch" />
    >     -            <arg
    > value="org.apache.royale.typedefs-${releaseversion}-rc${rc}" />
    >     +            <arg
    > value="org.apache.royale.typedefs-${release.version}-rc${rc}" />
    >                  <arg value="
    > https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Froyale-typedefs.git&amp;data=02%7C01%7Caharui%40adobe.com%7Cd1486c9d132a491e274108d73cd7cff9%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C637044769931764780&amp;sdata=qurNjh0lV7TpvkKjQAnBh48l2YPDxCcxTBP%2BNQe1pH8%3D&amp;reserved=0"
    > />
    >                  <arg value="royale-typedefs" />
    >              </exec>
    >              <exec executable="git" dir="${artifactfolder}/sources"
    > failonerror="true" >
    >                  <arg value="clone" />
    >                  <arg value="--branch" />
    >     -            <arg
    > value="org.apache.royale.framework-${releaseversion}-rc${rc}" />
    >     +            <arg
    > value="org.apache.royale.framework-${release.version}-rc${rc}" />
    >                  <arg value="
    > https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Froyale-asjs.git&amp;data=02%7C01%7Caharui%40adobe.com%7Cd1486c9d132a491e274108d73cd7cff9%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C637044769931774775&amp;sdata=2lflk%2BpR5JaskhugM5eptFRYu04ry%2FjE52cJ4ljl5YU%3D&amp;reserved=0"
    > />
    >                  <arg value="royale-asjs" />
    >              </exec>
    >     @@ -916,7 +919,7 @@
    >          <target name="validate-ant-source" description="do some tweaks so
    > the clone is the same as maven source file set"  >
    >              <!-- unpack ant src which was generated from maven source
    > package -->
    >              <mkdir dir="${artifactfolder}/artifacts/source" />
    >     -        <unzip
    > src="${artifactfolder}/artifacts/apache-royale-${releaseversion}-src.zip"
    > dest="${artifactfolder}/artifacts/source" />
    >     +        <unzip
    > src="${artifactfolder}/artifacts/apache-royale-${release.version}-src.zip"
    > dest="${artifactfolder}/artifacts/source" />
    >              <property name="artifactfolder" value="${artifactfolder}" />
    >              <!-- get the build date from flex-sdk-description and use it
    > -->
    >              <xmlproperty
    > file="${artifactfolder}/artifacts/source/royale-asjs/flex-sdk-description.xml"
    > prefix="desc" />
    >     @@ -948,14 +951,14 @@
    >          </target>
    >
    >          <target name="compare-src-zips" >
    >     -        <echo>releaseversion: ${releaseversion}</echo>
    >     +        <echo>release.version: ${release.version}</echo>
    >              <echo>copy src-zip file to a separate folder</echo>
    >              <mkdir dir="${artifactfolder}/artifacts/srczip" />
    >     -        <copy
    > file="${artifactfolder}/artifacts/apache-royale-${releaseversion}-src.zip"
    >     -
    > tofile="${artifactfolder}/artifacts/srczip/apache-royale-${releaseversion}-src.zip"
    > />
    >     +        <copy
    > file="${artifactfolder}/artifacts/apache-royale-${release.version}-src.zip"
    >     +
    > tofile="${artifactfolder}/artifacts/srczip/apache-royale-${release.version}-src.zip"
    > />
    >              <mkdir dir="${artifactfolder}/sources/srczip" />
    >     -        <copy
    > file="${artifactfolder}/sources/royale-asjs/out/apache-royale-${releaseversion}-src.zip"
    >     -
    > tofile="${artifactfolder}/sources/srczip/apache-royale-${releaseversion}-src.zip"
    > />
    >     +        <copy
    > file="${artifactfolder}/sources/royale-asjs/out/apache-royale-${release.version}-src.zip"
    >     +
    > tofile="${artifactfolder}/sources/srczip/apache-royale-${release.version}-src.zip"
    > />
    >              <echo>running reproducible build plugin on
    > ${artifactfolder}/artifacts/srczip</echo>
    >              <exec executable="${mvn}" dir="${artifactfolder}"
    > failonerror="true" >
    >                  <arg
    > value="io.github.zlika:reproducible-build-maven-plugin:0.10:strip-jar" />
    >     @@ -967,10 +970,10 @@
    >                  <arg
    > value="-Dreproducible.outputDirectory=${artifactfolder}/sources/srczip" />
    >              </exec>
    >              <condition property="filesmatch" value="true">
    >     -            <filesmatch
    > file1="${artifactfolder}/artifacts/srczip/apache-royale-${releaseversion}-src.zip"
    >     -
    > file2="${artifactfolder}/sources/srczip/apache-royale-${releaseversion}-src.zip"
    > />
    >     +            <filesmatch
    > file1="${artifactfolder}/artifacts/srczip/apache-royale-${release.version}-src.zip"
    >     +
    > file2="${artifactfolder}/sources/srczip/apache-royale-${release.version}-src.zip"
    > />
    >              </condition>
    >     -        <fail message="apache-royale-${releaseversion}-src.zip does
    > not match" unless="filesmatch" />
    >     +        <fail message="apache-royale-${release.version}-src.zip does
    > not match" unless="filesmatch" />
    >          </target>
    >
    >          <target name="validate-ant-bits">
    >     @@ -980,16 +983,16 @@
    >              <mkdir dir="${artifactfolder}/artifacts/zip/js" />
    >              <mkdir dir="${artifactfolder}/artifacts/zip/swf" />
    >              <mkdir dir="${artifactfolder}/artifacts/zip/src" />
    >     -        <unzip
    > src="${artifactfolder}/artifacts/apache-royale-${releaseversion}-src.zip"
    > dest="${artifactfolder}/artifacts/zip/src" />
    >     -        <untar
    > src="${artifactfolder}/artifacts/apache-royale-${releaseversion}-src.tar.gz"
    > dest="${artifactfolder}/artifacts/tar/src" compression="gzip">
    >     +        <unzip
    > src="${artifactfolder}/artifacts/apache-royale-${release.version}-src.zip"
    > dest="${artifactfolder}/artifacts/zip/src" />
    >     +        <untar
    > src="${artifactfolder}/artifacts/apache-royale-${release.version}-src.tar.gz"
    > dest="${artifactfolder}/artifacts/tar/src" compression="gzip">
    >                  <cutdirsmapper dirs="1" />
    >              </untar>
    >     -        <unzip
    > src="${artifactfolder}/artifacts/apache-royale-${releaseversion}-bin-js.zip"
    > dest="${artifactfolder}/artifacts/zip/js" />
    >     -        <untar
    > src="${artifactfolder}/artifacts/apache-royale-${releaseversion}-bin-js.tar.gz"
    > dest="${artifactfolder}/artifacts/tar/js" compression="gzip">
    >     +        <unzip
    > src="${artifactfolder}/artifacts/apache-royale-${release.version}-bin-js.zip"
    > dest="${artifactfolder}/artifacts/zip/js" />
    >     +        <untar
    > src="${artifactfolder}/artifacts/apache-royale-${release.version}-bin-js.tar.gz"
    > dest="${artifactfolder}/artifacts/tar/js" compression="gzip">
    >                  <cutdirsmapper dirs="1" />
    >              </untar>
    >     -        <unzip
    > src="${artifactfolder}/artifacts/apache-royale-${releaseversion}-bin-js-swf.zip"
    > dest="${artifactfolder}/artifacts/zip/swf" />
    >     -        <untar
    > src="${artifactfolder}/artifacts/apache-royale-${releaseversion}-bin-js-swf.tar.gz"
    > dest="${artifactfolder}/artifacts/tar/swf" compression="gzip">
    >     +        <unzip
    > src="${artifactfolder}/artifacts/apache-royale-${release.version}-bin-js-swf.zip"
    > dest="${artifactfolder}/artifacts/zip/swf" />
    >     +        <untar
    > src="${artifactfolder}/artifacts/apache-royale-${release.version}-bin-js-swf.tar.gz"
    > dest="${artifactfolder}/artifacts/tar/swf" compression="gzip">
    >                  <cutdirsmapper dirs="1" />
    >              </untar>
    >              <antcall target="validate-ant-jars" />
    >
    >
    >
    >
    
    -- 
    
    Piotr Zarzycki
    
    Patreon: *https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.patreon.com%2Fpiotrzarzycki&amp;data=02%7C01%7Caharui%40adobe.com%7Cd1486c9d132a491e274108d73cd7cff9%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C637044769931774775&amp;sdata=h%2Bj%2BmAfNdnDwvj0ECMxyFKlc%2BY4jc38MhaXirqESy1U%3D&amp;reserved=0
    <https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.patreon.com%2Fpiotrzarzycki&amp;data=02%7C01%7Caharui%40adobe.com%7Cd1486c9d132a491e274108d73cd7cff9%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C637044769931774775&amp;sdata=h%2Bj%2BmAfNdnDwvj0ECMxyFKlc%2BY4jc38MhaXirqESy1U%3D&amp;reserved=0>*
    


Re: [royale-asjs] branch release/0.9.6 updated: Fix releasessteps script

Posted by Piotr Zarzycki <pi...@gmail.com>.
I just went trough whole the process and didn't have any kind of problems.
I'm pushing RC2. If you think I should revert that I can. I don't have
anymore console log with it.

śr., 18 wrz 2019 o 16:18 Alex Harui <ah...@adobe.com.invalid> napisał(a):

> Changing releaseversion to release.version should not be necessary and
> might break something else.  Can you reproduce the failure?  What is the
> console log?  How did it not pick up the releaseversion from
> build.properties?
>
> -Alex
>
> On 9/17/19, 10:53 PM, "piotrz@apache.org" <pi...@apache.org> wrote:
>
>     This is an automated email from the ASF dual-hosted git repository.
>
>     piotrz pushed a commit to branch release/0.9.6
>     in repository
> https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgitbox.apache.org%2Frepos%2Fasf%2Froyale-asjs.git&amp;data=02%7C01%7Caharui%40adobe.com%7C780e787968ef41434f6708d73bfc78e8%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C637043827860542212&amp;sdata=qQC5eeDfhB2Rz%2BdGfyJohLsFJuz%2BdkFCb8WPcrwW5PI%3D&amp;reserved=0
>
>
>     The following commit(s) were added to refs/heads/release/0.9.6 by this
> push:
>          new 84c2e85  Fix releasessteps script
>     84c2e85 is described below
>
>     commit 84c2e855c5a316ad21ff3a8ac6ac0cde826eecf0
>     Author: Piotr Zarzycki <pi...@gmail.com>
>     AuthorDate: Wed Sep 18 07:47:03 2019 +0200
>
>         Fix releasessteps script
>
>         - Add RoyaleUnit to swcs list and sign target
>         - Fix property release.version
>     ---
>      releasesteps.xml | 43 +++++++++++++++++++++++--------------------
>      1 file changed, 23 insertions(+), 20 deletions(-)
>
>     diff --git a/releasesteps.xml b/releasesteps.xml
>     index 01d0061..5c3edf1 100644
>     --- a/releasesteps.xml
>     +++ b/releasesteps.xml
>     @@ -219,7 +219,7 @@
>              <delete file="${artifactfolder}/swcs.txt" />
>              <!-- this is a comma-delimited, no spaces, no-line-breaks
> list used to decide which swcs
>               to skip in the rat-check of the binaries and which swcs to
> examine before approval -->
>     -        <property name="swcs-list"
> value="Basic,Binding,Charts,Collections,Core,CreateJS,DragDrop,Effects,Express,Flat,FontAwesome,Formatters,GoogleMaps,Graphics,HTML,HTML5,Icons,JQuery,Jewel,Crux,Language,MXRoyale,MaterialDesignLite,Mobile,Network,Reflection,RoyaleSite,SparkRoyale,Storage,TLF,Text,XML,BasicTheme,JewelTheme"/>
>     +        <property name="swcs-list"
> value="Basic,Binding,Charts,Collections,Core,CreateJS,DragDrop,Effects,Express,Flat,FontAwesome,Formatters,GoogleMaps,Graphics,HTML,HTML5,Icons,JQuery,Jewel,Crux,Language,MXRoyale,MaterialDesignLite,Mobile,Network,Reflection,RoyaleUnit,RoyaleSite,SparkRoyale,Storage,TLF,Text,XML,BasicTheme,JewelTheme"/>
>              <echo file="${artifactfolder}/swcs.txt"
> message="${swcs-list}"/>
>          </target>
>
>     @@ -501,6 +501,9 @@
>              <antcall target="sign-swc-artifacts" >
>                  <param name="swcname" value="RoyaleSite" />
>              </antcall>
>     +           <antcall target="sign-swc-artifacts" >
>     +            <param name="swcname" value="RoyaleUnit" />
>     +        </antcall>
>              <antcall target="sign-swc-artifacts" >
>                  <param name="swcname" value="SparkRoyale" />
>              </antcall>
>     @@ -880,7 +883,7 @@
>
>          <target name="build-ant-source"
> depends="check-air-home,check-playerglobal-home,check-runtime-env" >
>              <mkdir dir="${artifactfolder}/sources" />
>     -        <unzip
> src="${artifactfolder}/artifacts/apache-royale-${releaseversion}-src.zip"
> dest="${artifactfolder}/sources" >
>     +        <unzip
> src="${artifactfolder}/artifacts/apache-royale-${release.version}-src.zip"
> dest="${artifactfolder}/sources" >
>              </unzip>
>
>              <ant dir="${artifactfolder}/sources/royale-asjs" target="all"
> />
>     @@ -891,21 +894,21 @@
>              <exec executable="git" dir="${artifactfolder}/sources"
> failonerror="true" >
>                  <arg value="clone" />
>                  <arg value="--branch" />
>     -            <arg
> value="org.apache.royale.compiler-${releaseversion}-rc${rc}" />
>     +            <arg
> value="org.apache.royale.compiler-${release.version}-rc${rc}" />
>                  <arg value="
> https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Froyale-compiler.git&amp;data=02%7C01%7Caharui%40adobe.com%7C780e787968ef41434f6708d73bfc78e8%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C637043827860542212&amp;sdata=Qvy8HglyeoGWo6fV45tu7%2FBURYBHpRgWnx%2F4Mk33ZCo%3D&amp;reserved=0"
> />
>                  <arg value="royale-compiler" />
>              </exec>
>              <exec executable="git" dir="${artifactfolder}/sources"
> failonerror="true" >
>                  <arg value="clone" />
>                  <arg value="--branch" />
>     -            <arg
> value="org.apache.royale.typedefs-${releaseversion}-rc${rc}" />
>     +            <arg
> value="org.apache.royale.typedefs-${release.version}-rc${rc}" />
>                  <arg value="
> https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Froyale-typedefs.git&amp;data=02%7C01%7Caharui%40adobe.com%7C780e787968ef41434f6708d73bfc78e8%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C637043827860542212&amp;sdata=8jWPOp%2FH%2FKQr9u4ynBRMnXklSST2yaqUCTazqrc7vlA%3D&amp;reserved=0"
> />
>                  <arg value="royale-typedefs" />
>              </exec>
>              <exec executable="git" dir="${artifactfolder}/sources"
> failonerror="true" >
>                  <arg value="clone" />
>                  <arg value="--branch" />
>     -            <arg
> value="org.apache.royale.framework-${releaseversion}-rc${rc}" />
>     +            <arg
> value="org.apache.royale.framework-${release.version}-rc${rc}" />
>                  <arg value="
> https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Froyale-asjs.git&amp;data=02%7C01%7Caharui%40adobe.com%7C780e787968ef41434f6708d73bfc78e8%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C637043827860542212&amp;sdata=ZqgQ7OSyVeZ4kP84bS16LwO93mWJbv%2Fh22VOsJImUfA%3D&amp;reserved=0"
> />
>                  <arg value="royale-asjs" />
>              </exec>
>     @@ -916,7 +919,7 @@
>          <target name="validate-ant-source" description="do some tweaks so
> the clone is the same as maven source file set"  >
>              <!-- unpack ant src which was generated from maven source
> package -->
>              <mkdir dir="${artifactfolder}/artifacts/source" />
>     -        <unzip
> src="${artifactfolder}/artifacts/apache-royale-${releaseversion}-src.zip"
> dest="${artifactfolder}/artifacts/source" />
>     +        <unzip
> src="${artifactfolder}/artifacts/apache-royale-${release.version}-src.zip"
> dest="${artifactfolder}/artifacts/source" />
>              <property name="artifactfolder" value="${artifactfolder}" />
>              <!-- get the build date from flex-sdk-description and use it
> -->
>              <xmlproperty
> file="${artifactfolder}/artifacts/source/royale-asjs/flex-sdk-description.xml"
> prefix="desc" />
>     @@ -948,14 +951,14 @@
>          </target>
>
>          <target name="compare-src-zips" >
>     -        <echo>releaseversion: ${releaseversion}</echo>
>     +        <echo>release.version: ${release.version}</echo>
>              <echo>copy src-zip file to a separate folder</echo>
>              <mkdir dir="${artifactfolder}/artifacts/srczip" />
>     -        <copy
> file="${artifactfolder}/artifacts/apache-royale-${releaseversion}-src.zip"
>     -
> tofile="${artifactfolder}/artifacts/srczip/apache-royale-${releaseversion}-src.zip"
> />
>     +        <copy
> file="${artifactfolder}/artifacts/apache-royale-${release.version}-src.zip"
>     +
> tofile="${artifactfolder}/artifacts/srczip/apache-royale-${release.version}-src.zip"
> />
>              <mkdir dir="${artifactfolder}/sources/srczip" />
>     -        <copy
> file="${artifactfolder}/sources/royale-asjs/out/apache-royale-${releaseversion}-src.zip"
>     -
> tofile="${artifactfolder}/sources/srczip/apache-royale-${releaseversion}-src.zip"
> />
>     +        <copy
> file="${artifactfolder}/sources/royale-asjs/out/apache-royale-${release.version}-src.zip"
>     +
> tofile="${artifactfolder}/sources/srczip/apache-royale-${release.version}-src.zip"
> />
>              <echo>running reproducible build plugin on
> ${artifactfolder}/artifacts/srczip</echo>
>              <exec executable="${mvn}" dir="${artifactfolder}"
> failonerror="true" >
>                  <arg
> value="io.github.zlika:reproducible-build-maven-plugin:0.10:strip-jar" />
>     @@ -967,10 +970,10 @@
>                  <arg
> value="-Dreproducible.outputDirectory=${artifactfolder}/sources/srczip" />
>              </exec>
>              <condition property="filesmatch" value="true">
>     -            <filesmatch
> file1="${artifactfolder}/artifacts/srczip/apache-royale-${releaseversion}-src.zip"
>     -
> file2="${artifactfolder}/sources/srczip/apache-royale-${releaseversion}-src.zip"
> />
>     +            <filesmatch
> file1="${artifactfolder}/artifacts/srczip/apache-royale-${release.version}-src.zip"
>     +
> file2="${artifactfolder}/sources/srczip/apache-royale-${release.version}-src.zip"
> />
>              </condition>
>     -        <fail message="apache-royale-${releaseversion}-src.zip does
> not match" unless="filesmatch" />
>     +        <fail message="apache-royale-${release.version}-src.zip does
> not match" unless="filesmatch" />
>          </target>
>
>          <target name="validate-ant-bits">
>     @@ -980,16 +983,16 @@
>              <mkdir dir="${artifactfolder}/artifacts/zip/js" />
>              <mkdir dir="${artifactfolder}/artifacts/zip/swf" />
>              <mkdir dir="${artifactfolder}/artifacts/zip/src" />
>     -        <unzip
> src="${artifactfolder}/artifacts/apache-royale-${releaseversion}-src.zip"
> dest="${artifactfolder}/artifacts/zip/src" />
>     -        <untar
> src="${artifactfolder}/artifacts/apache-royale-${releaseversion}-src.tar.gz"
> dest="${artifactfolder}/artifacts/tar/src" compression="gzip">
>     +        <unzip
> src="${artifactfolder}/artifacts/apache-royale-${release.version}-src.zip"
> dest="${artifactfolder}/artifacts/zip/src" />
>     +        <untar
> src="${artifactfolder}/artifacts/apache-royale-${release.version}-src.tar.gz"
> dest="${artifactfolder}/artifacts/tar/src" compression="gzip">
>                  <cutdirsmapper dirs="1" />
>              </untar>
>     -        <unzip
> src="${artifactfolder}/artifacts/apache-royale-${releaseversion}-bin-js.zip"
> dest="${artifactfolder}/artifacts/zip/js" />
>     -        <untar
> src="${artifactfolder}/artifacts/apache-royale-${releaseversion}-bin-js.tar.gz"
> dest="${artifactfolder}/artifacts/tar/js" compression="gzip">
>     +        <unzip
> src="${artifactfolder}/artifacts/apache-royale-${release.version}-bin-js.zip"
> dest="${artifactfolder}/artifacts/zip/js" />
>     +        <untar
> src="${artifactfolder}/artifacts/apache-royale-${release.version}-bin-js.tar.gz"
> dest="${artifactfolder}/artifacts/tar/js" compression="gzip">
>                  <cutdirsmapper dirs="1" />
>              </untar>
>     -        <unzip
> src="${artifactfolder}/artifacts/apache-royale-${releaseversion}-bin-js-swf.zip"
> dest="${artifactfolder}/artifacts/zip/swf" />
>     -        <untar
> src="${artifactfolder}/artifacts/apache-royale-${releaseversion}-bin-js-swf.tar.gz"
> dest="${artifactfolder}/artifacts/tar/swf" compression="gzip">
>     +        <unzip
> src="${artifactfolder}/artifacts/apache-royale-${release.version}-bin-js-swf.zip"
> dest="${artifactfolder}/artifacts/zip/swf" />
>     +        <untar
> src="${artifactfolder}/artifacts/apache-royale-${release.version}-bin-js-swf.tar.gz"
> dest="${artifactfolder}/artifacts/tar/swf" compression="gzip">
>                  <cutdirsmapper dirs="1" />
>              </untar>
>              <antcall target="validate-ant-jars" />
>
>
>
>

-- 

Piotr Zarzycki

Patreon: *https://www.patreon.com/piotrzarzycki
<https://www.patreon.com/piotrzarzycki>*