Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

nonFinal, do we still need it? #23

Open
arjantijms opened this issue Feb 5, 2020 · 4 comments
Open

nonFinal, do we still need it? #23

arjantijms opened this issue Feb 5, 2020 · 4 comments

Comments

@arjantijms
Copy link
Contributor

The plug-in uses a convention for something called "nonFinal", which I think didn't carry over to Jakarta:

e.g.

//  OSGi Bundle-SymbolicName:   ${API_PACKAGE}-api
//  OSGi bundle specversion:    ${SPEC_VERSION}.99.b${SPEC_BUILD}
//  OSGi Bundle-Version:        ${SPEC_VERSION}.99.b${SPEC_BUILD}
//  jar Extension-Name:         ${API_PACKAGE}
//  jar Specification-Version:  ${SPEC_VERSION}.99.${SPEC_BUILD}
//  jar Implementation-Version: ${NEW_SPEC_VERSION}-b${SPEC_BUILD}

Do we need to retain this?

@bshannon @romain-grecourt

@romain-grecourt
Copy link
Member

The versioning rules still mentions non final.

@arjantijms
Copy link
Contributor Author

The versioning rules still mentions non final.

You're right, good find ;) Though is this "officially" adopted? We may also need an update here anyway for the -SNAPSHOT and -RCx cases.

One small thing, the option is now called "nonFinal", which means we often have to say "nonFinal" = false. Double negatives are in general hard for people to get right at a glance.

I know there's some history here, but going forward, can't we just call it final with values true and false?

@bshannon
Copy link

bshannon commented Feb 6, 2020

I think the intent was that final was the default if nothing was said so you would
only ever say nonFinal=true.

@romain-grecourt
Copy link
Member

I do agree that double negations are hard to understand. A rename to final seems like a good future improvement.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants