This is an automated email from the ASF dual-hosted git repository.

markt pushed a commit to branch 9.0.x
in repository https://gitbox.apache.org/repos/asf/tomcat.git

commit ee97eece61569c80aac084a5aa8decf14a423584
Author: Mark Thomas <ma...@apache.org>
AuthorDate: Tue Mar 4 16:12:39 2025 +0000

    Fix version check
---
 build.xml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/build.xml b/build.xml
index d5843050a3..de5c41fc60 100644
--- a/build.xml
+++ b/build.xml
@@ -2631,7 +2631,7 @@ Apache Tomcat ${version} native binaries for Win64 
AMD64/EMT64 platform.
     <condition property="nsis.executable.unix.version.ok">
       <or>
         <os family="windows"/>
-        <equals arg1="nsis.version" arg2="nsis.executable.unix.version"/>
+        <equals arg1="v${nsis.version}" 
arg2="${nsis.executable.unix.version}"/>
         <not>
           <!-- Zero length string for version.dev indicates a release build -->
           <equals arg1="${version.dev}" arg2=""/>
@@ -2653,7 +2653,7 @@ To skip building the Windows installer, set the 
skip.installer property in build
       depends="-installer-set-properties-2"
       unless="${nsis.executable.unix.version.ok}">
     <fail message="An executable makensis was found at: ${nsis.executable.unix}
-with version [${nsis.executable.unix.version}] but version [${nsis.version}] 
is required
+with version [${nsis.executable.unix.version}] but version [v${nsis.version}] 
is required
 for repeatable builds.
 You may need a custom build of makensis. Instructions for this may be found in 
BUILDING.txt"/>
   </target>


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

Reply via email to