Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Add possibility to override fileName and base directory of Keycloak Quarkus distribution ZIP archive#24284

Merged
stianst merged 1 commit intokeycloak:mainfrom
pskopek:dist-dir-chnage
Nov 7, 2023
Merged

Add possibility to override fileName and base directory of Keycloak Quarkus distribution ZIP archive#24284
stianst merged 1 commit intokeycloak:mainfrom
pskopek:dist-dir-chnage

Conversation

@pskopek
Copy link
Contributor

@pskopek pskopek commented Oct 25, 2023

Closes #24283

@pskopek pskopek requested review from a team as code owners October 25, 2023 12:14
@pskopek pskopek requested a review from a team October 25, 2023 12:14
@pskopek pskopek added this to the 22.0.6 milestone Oct 25, 2023
@pskopek pskopek requested a review from ASzc October 25, 2023 12:15
@ghost ghost added the flaky-test label Oct 25, 2023
Copy link
Contributor

@ASzc ASzc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems like it works. I'm questioning if build-helper-maven-plugin is really needed here, since maven is capable of concatenation of property values by itself:

    <properties>
        <dist.archive.base.file.name>keycloak</dist.archive.base.file.name>
        <dist.archive.base.dir.name>${dist.archive.base.file.name}</dist.archive.base.dir.name>
        <dist.archive.dir>${dist.archive.base.dir.name}-${project.version}</dist.archive.dir>
    </properties>

but it seems you want to trim any suffixes on the version number:

keycloak-999.0.0-SNAPSHOT/
vs
keycloak-999.0.0/

Perhaps a more generic thing to do here would be to use build-helper-maven-plugin to generate a project.version.suffixless or something, and then use that to let maven itself concat that with whatever we need, which is dist.archive.base.dir.name in this context. This would align with project.version.npm which was added recently for a different purpose.

@ASzc
Copy link
Contributor

ASzc commented Oct 25, 2023

Although now that I think more about project.version.npm, the approach we arrived at there was to generate the value statically at the time project.version is updated (in set-version.sh), and not to try to do it in maven

…uarkus distribution ZIP archive

Closes keycloak#24283

Signed-off-by: Peter Skopek <[email protected]>
@stianst stianst merged commit e5eded0 into keycloak:main Nov 7, 2023
lejdiprifti pushed a commit to lejdiprifti/keycloak that referenced this pull request Nov 7, 2023
…uarkus distribution ZIP archive (keycloak#24284)

Closes keycloak#24283

Signed-off-by: Peter Skopek <[email protected]>
Signed-off-by: Lejdi Prifti <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add possibility to override fileName and base directory of Keycloak Quarkus distribution ZIP archive.

3 participants