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
ISPN-11511 More consistent branding #8082
ISPN-11511 More consistent branding #8082
Conversation
03c5eac
to
5ccc7cb
Compare
Rebased |
@@ -161,7 +168,7 @@ public static void printFullVersionInformation() { | |||
* Returns version information as a string. | |||
*/ | |||
public static String printVersion() { | |||
return INSTANCE.brandname + " '" + INSTANCE.codename + "' " + INSTANCE.version; | |||
return INSTANCE.brandName + " '" + INSTANCE.codename + "' " + INSTANCE.version; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it should be Infinispan
instead of the brand name.
return INSTANCE.brandName + " '" + INSTANCE.codename + "' " + INSTANCE.version; | |
return "Infinispan '" + INSTANCE.codename + "' " + INSTANCE.version; |
This is used in CLI (version
command) and it is logged when the server starts:
ISPN000128: Infinispan version: My Brand 'N/A' 11.0.0-SNAPSHOT
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
or change its format:
<brand> <brand_version> (Infinispan <project_version>)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
May I suggest to move the codename to the end and use the brand version instead:
return INSTANCE.brandName + " '" + INSTANCE.codename + "' " + INSTANCE.version; | |
return INSTANCE.brandName + " " + INSTANCE.brandVersion + " (" + INSTANCE.codename + ")"; |
server/runtime/src/main/java/org/infinispan/server/logging/Log.java
Outdated
Show resolved
Hide resolved
commons/all/src/main/java/org/infinispan/commons/util/Version.java
Outdated
Show resolved
Hide resolved
026893d
to
1e70749
Compare
* Introduce an infinispan.brand.version property * Align all shell tools to print the version in the same way
1e70749
to
262db90
Compare
integrated! thanks @tristantarrant ! |
https://issues.redhat.com/browse/ISPN-11511
To get consistent branding:
-Dinfinispan.brand.name="My Brand" -Dinfinispan.brand.version="8.0.0.GA" -Dinfinispan.codename="Infinispan 11.0.0.Dev03"
will result in the server (and other tools) printing the following banner:
ISPN080001: My Brand 8.0.0.GA (Infinispan 11.0.0.Dev03)