ChristopherSchultz commented on PR #733:
URL: https://github.com/apache/tomcat/pull/733#issuecomment-2161606328

   > When running `ant deploy` to reproduce the build, `pre-release` target 
will not have been executed
   
   If you are trying to reproduce the build, they you will have 
`build.properties.release` as generated by the release manager. That file 
contains (e.g.):
   
       ant.tstamp.now.iso=2024-06-11T20:57:36Z
   
   When running "deploy", I see this:
   
   ```
   Setting project property: ant.tstamp.now.iso -> 2024-06-11T20:57:36Z
   Setting project property: ant.tstamp.now.iso -> 2024-06-11T20:57:36Z
   Override ignored for property "ant.tstamp.now.iso"
   Setting project property: ant.tstamp.now.iso -> 2024-06-11T20:57:36Z
   Override ignored for property "ant.tstamp.now.iso"
   ... [many times repeated]
   ```
   
   However, when *not* using a source-release package:
   ```
   $ ant -d deploy | grep ant.tstamp.now.iso
   Property "ant.tstamp.now.iso" has not been set
   Setting ro project property: ant.tstamp.now.iso -> ${ant.tstamp.now.iso}
      [tstamp] magic property ant.tstamp.now.iso ignored as 
'${ant.tstamp.now.iso}' is not in valid ISO pattern
      [tstamp] magic property ant.tstamp.now.iso ignored as 
'${ant.tstamp.now.iso}' is not in valid ISO pattern
   ```
   
   > May not be an actual issue, but it disrupts the flawlessness of the 
process.
   
   +1
   
   The irony of the situation is that `ant.tstamp.now.iso` can be used to "set" 
the build timestamp for ant but ant does not set that property itself if it's 
not specified e.g. on the command-line. So it's basically never a good idea to 
use `${ant.tstamp.now.iso}` in your own build file.
   


-- 
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.

To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


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

Reply via email to