[jboss-jira] [JBoss JIRA] (WFLY-9806) Clean up use of jboss.as.release.version

Brian Stansberry (JIRA) issues at jboss.org
Mon Feb 12 13:49:00 EST 2018


Brian Stansberry created WFLY-9806:
--------------------------------------

             Summary: Clean up use of jboss.as.release.version
                 Key: WFLY-9806
                 URL: https://issues.jboss.org/browse/WFLY-9806
             Project: WildFly
          Issue Type: Task
          Components: Build System
            Reporter: Brian Stansberry
            Assignee: Brian Stansberry


Our poms have a 'jboss.as.release.version' property that is used for two distinct purposes:

1) In the root pom it drives the value of the JBossAS-Release-Version property in our MANIFEST.mf files.

2) In many places throughout the poms its value is appended to the value of the server.output.dir.prefix property to produce the names of various output dirs used in the build.

This is a bit messy, and is made messier because in the EAP variant of the poms, nearly all[1] of the type 2) uses have been converted to a different property, named 'jboss.eap.release.version'.  This name is not appropriate for upstream. The effect of this is it's unnecessarily difficult to go from a WF tag to an EAP branch.

To clean this up I propose to do the following:

A) Replace all the type 2) uses of jboss.as.release.version in the poms with a new property, "server.output.dir.version".  This better describes the purpose of the property and has a name that is agnostic between WildFly and EAP.

B) Examine some uses of "jboss.eap.release.version" in EAP that are not present in WF and if appropriate port them to WF, but using "server.output.dir.version".

C) Either drop jboss.as.release.version for the type 1) manifest use, and directly use "project.version" (which is the only value ever provided for this), or better comment the declaration of jboss.as.release.version so its sole intended use is clear. (Probably for now I'll do the latter to avoid risk of unintended side effects.)


[1] "Nearly all" because



--
This message was sent by Atlassian JIRA
(v7.5.0#75005)


More information about the jboss-jira mailing list