You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2010/01/14 20:48:42 UTC

svn commit: r899380 - /tomcat/trunk/dist.xml

Author: markt
Date: Thu Jan 14 19:48:41 2010
New Revision: 899380

URL: http://svn.apache.org/viewvc?rev=899380&view=rev
Log:
Skipping installer shouldn't break package-winzip target

Modified:
    tomcat/trunk/dist.xml

Modified: tomcat/trunk/dist.xml
URL: http://svn.apache.org/viewvc/tomcat/trunk/dist.xml?rev=899380&r1=899379&r2=899380&view=diff
==============================================================================
--- tomcat/trunk/dist.xml (original)
+++ tomcat/trunk/dist.xml Thu Jan 14 19:48:41 2010
@@ -288,15 +288,7 @@
   </target>
 
   <!-- ================= DIST: Create Windows Installer =================== -->
-  <target name="installer"
-   description="Create Windows installer" unless="skip.installer">
-    <echo message="Builds a Windows installer based on Nullsoft Installer"/>
-    <copy todir="${tomcat.dist}">
-      <fileset dir="res" />
-    </copy>
-    <copy file="${nsis.installoptions.dll}" todir="${tomcat.dist}" />
-    <copy file="${nsis.nsexec.dll}" todir="${tomcat.dist}" />
-    <copy file="${nsis.nsisdl.dll}" todir="${tomcat.dist}" />
+  <target name="win-prepare" >
     <!-- 32 bit -->
     <copy file="res/procrun/tomcat${version.major}.exe"
         tofile="${tomcat.dist}/bin/tomcat${version.major}.exe" />
@@ -324,6 +316,18 @@
     <echo append="false" file="${tomcat.dist}/bin/x64/README">
 Apache Tomcat ${version} native binaries for Win64 AMD64/EMT64 platform.
     </echo>
+  </target>
+
+  <!-- ================= DIST: Create Windows Installer =================== -->
+  <target name="installer" description="Create Windows installer"
+  	  unless="skip.installer" depends="win-prepare" >
+    <echo message="Builds a Windows installer based on Nullsoft Installer"/>
+    <copy todir="${tomcat.dist}">
+      <fileset dir="res" />
+    </copy>
+    <copy file="${nsis.installoptions.dll}" todir="${tomcat.dist}" />
+    <copy file="${nsis.nsexec.dll}" todir="${tomcat.dist}" />
+    <copy file="${nsis.nsisdl.dll}" todir="${tomcat.dist}" />
 
     <!-- Make sure the RELEASE-NOTES has Windows line endings as they may be
          displayed post-install -->
@@ -418,7 +422,7 @@
   </target>
 
   <!-- Packages the core windows zip distros -->
-  <target name="package-winzip">
+  <target name="package-winzip" depends="win-prepare">
     <fixcrlf srcdir="${tomcat.dist}" includes="*.txt,LICENSE,NOTICE" eol="crlf" encoding="ISO-8859-1" fixlast="false" />
     <fixcrlf srcdir="${tomcat.dist}/conf" eol="crlf" encoding="ISO-8859-1" fixlast="false" />
 



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org


Re: svn commit: r899380 - /tomcat/trunk/dist.xml

Posted by Mark Thomas <ma...@apache.org>.
On 16/01/2010 20:00, Konstantin Kolinko wrote:
> 2010/1/14  <ma...@apache.org>:
>> Author: markt
>> Date: Thu Jan 14 19:48:41 2010
>> New Revision: 899380
>>
>> URL: http://svn.apache.org/viewvc?rev=899380&view=rev
>> Log:
>> Skipping installer shouldn't break package-winzip target
>>
>> Modified:
>>    tomcat/trunk/dist.xml
>>
> 
> I think that all content of the new "win-prepare" target (added in
> this rev) belongs to the existing "dist-static" target.
> 
> I wonder, why none of the "package-" targets have explicit
> dependencies on "dist-" targets.

Stuff to sort out post 6.0.24, along with any remaining line-ending
issues like the windows installer and the deployer packages.

Mark



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org


Re: svn commit: r899380 - /tomcat/trunk/dist.xml

Posted by Konstantin Kolinko <kn...@gmail.com>.
2010/1/14  <ma...@apache.org>:
> Author: markt
> Date: Thu Jan 14 19:48:41 2010
> New Revision: 899380
>
> URL: http://svn.apache.org/viewvc?rev=899380&view=rev
> Log:
> Skipping installer shouldn't break package-winzip target
>
> Modified:
>    tomcat/trunk/dist.xml
>

I think that all content of the new "win-prepare" target (added in
this rev) belongs to the existing "dist-static" target.

I wonder, why none of the "package-" targets have explicit
dependencies on "dist-" targets.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org