hboutemy commented on a change in pull request #150:
URL: https://github.com/apache/royale-compiler/pull/150#discussion_r429611022



##########
File path: pom.xml
##########
@@ -340,24 +341,6 @@
         </executions>
       </plugin>
 
-      <!-- This plugin cleans up the jars for anything that might cause 
problems for reproducible builds -->
-      <!--
-        Even if the core maven plugins currently would support doing this 
without, we would be required
-        to set the line-separator and the Antlr3 plugin crashes if this is set 
to something non-default
-      -->
-      <plugin>
-        <groupId>io.github.zlika</groupId>
-        <artifactId>reproducible-build-maven-plugin</artifactId>
-        <version>0.11</version>
-        <executions>
-          <execution>
-            <goals>
-              <goal>strip-jar</goal>
-            </goals>
-          </execution>
-        </executions>
-      </plugin>
-
       <!-- Plugin to detect problems with JDK incompatibilities -->

Review comment:
       > The zlika plugin has made it possible to have Windows and Mac binaries 
match.
   
   AFAIK, it does not work: did you really check the results?
   I tried to rebuild royale-compiler on my Linux box, comparing every output 
jar file with reference on Maven central with buildinfo:buildinfo automation, 
and I get many artifacts that are different (reference release has visibly been 
done on Windows)




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to