[ZOOKEEPER-3786] Simplify generation of VersionInfoMain and Info

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 3.7.0
• Component/s: build
• Labels: pull-request-available
• Assignee: Christopher Tubbs
• Reporter: Christopher Tubbs
• Created: 07/Apr/20 22:44
• Updated: 17/May/23 12:34
• Resolved: 09/Apr/20 11:05

Description

The VerGen command is currently compiled and executed in a separate step prior to compilation of the rest of zookeeper-server. This is wasteful, complicated, and unnecessary.

Maven has built-in resource filtering for classes, so you can inject the system properties into the build and generate sources to be compiled that have the necessary information. (For example, see Accumulo's FilteredConstants).

I have a pull request ready to address this, which I will push up shortly.

Issue Links

• fixes: ZOOKEEPER-4273 Forward port ZOOKEEPER-3931: "zkServer.sh version" returns a trailing dash
• links to: GitHub Pull Request #1310