On Thursday, February 20, 2020, Steve Millidge (Payara) <steve.millidge@xxxxxxxxxxx> wrote:
The top level project is EE4J so I don't think it should be renamed GlassFish. There are other consumers of these implementations other than GlassFish.
True, but many of these implementation components do have glassfish in the maven coordinates and/or packages.
The question is whether we want increased consistency there; renaming package names and coordinates to ee4j, renaming org to GlassFish, or just keeping things as they are.
There's a larger discussion about moving some of the repos out of the GlassFish project. They got dumped in there to begin with just to avoid having too many fine grained projects. See the discussion in glasssfish-dev,
or one of the glassfish issues (I forget where exactly).
arjan tijms wrote on 2/18/20 8:47 AM:
Hi,
I did have some thoughts too about "glassfish" vs "ee4j".
Currently almost all implementations are "glassfish" (Soteria, Mojarra, etc). Should we move those all to "ee4j", or, could we instead just as well rename the ee4j repo to "glassfish" once all the specs/apis have been moved to the jakartaee org?
Just some thought...
Kind regards,
Arjan Tijms
On Tue, Feb 18, 2020 at 4:05 PM Kevin Sutter <sutter@xxxxxxxxxx> wrote:
Thanks, Arjan!
I'm thinking that this plugin should probably move to the
org.eclipse.ee4jgroupId, don't you think? And, then possibly rename this repo to remove the "glassfish-" prefix? Just another step in the direction of separating
the Specs from the Implementation... Not convinced it's an immediate requirement for Jakarta EE 9, but I could be swayed... Thoughts?
---------------------------------------------------
Kevin Sutter
STSM, MicroProfile and Jakarta EE architect @ IBM
e-mail: sutter@xxxxxxxxxx Twitter: @kwsutter
phone: tl-553-3620 (office), 507-253-3620 (office)
LinkedIn: https://www.linkedin.com/in/kevinwsutter
I think all API projects should indeed use the plug-in. The plug-in essentially sets a couple of properties, which the POM then feeds back into the Maven Bundle plug-in. Of course you can set these properties manually as well (and
some projects have done this, like I think Servlet), but for consistency and validation it's probably better to use that spec plug-in.
Hope this helps!
Kind regards, Arjan Tijms
On Tue, Feb 18, 2020 at 3:39 PM Kevin Sutter <sutter@xxxxxxxxxx> wrote: Hi Arjan,
Can you clarify which specific plugin you are referring to? Is it theglassfish-spec-version-maven-plugin?
Or, is there a generic, non-glassfish version that you are modifying? And, I'm curious why only a handful of APIs are having this issue. Is this plugin not used by all of the Spec/API projects? Should it be used? Thanks for the education.
---------------------------------------------------
Kevin Sutter
STSM, MicroProfile and Jakarta EE architect @ IBM
e-mail: sutter@xxxxxxxxxx Twitter: @kwsutter
phone: tl-553-3620 (office), 507-253-3620 (office)
LinkedIn: https://www.linkedin.com/in/kevinwsutter
I think I have encountered the same issue but when trying to create an "-M1" version on Jakarta Transactions. Would a similar change be possible for that type of qualifier?
I've updated the plug-in to threat RC1 essentially the same as a -SNAPSHOT qualifier, meaning it's stripped-off for the sake of the spec-version-maven-plugin. This works with the regular final mode (the default), which is the same as for the -SNAPSHOT qualifier.
The result is a MANIFEST.MF with the relevant version set (in this case) to 2.0.0.