-
Notifications
You must be signed in to change notification settings - Fork 261
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
Egeria 'platform' definition from gradle isn't usable #7421
Labels
gradle
Gradle build (new initiative)
Comments
It looks like some refactoring is needed to make use of the 'java-platform' plugin, and very explicitly create the BOM, then consuming it in subprojects, RATHER than just declaring constraints as is done currently. |
fyi @mandy-chessell |
planetf1
added a commit
to planetf1/egeria
that referenced
this issue
Mar 14, 2023
Signed-off-by: Nigel Jones <[email protected]>
planetf1
added a commit
to planetf1/egeria
that referenced
this issue
Mar 14, 2023
Signed-off-by: Nigel Jones <[email protected]>
planetf1
added a commit
to planetf1/egeria
that referenced
this issue
Mar 14, 2023
Signed-off-by: Nigel Jones <[email protected]>
planetf1
added a commit
to planetf1/egeria
that referenced
this issue
Mar 14, 2023
Signed-off-by: Nigel Jones <[email protected]>
planetf1
added a commit
to planetf1/egeria
that referenced
this issue
Mar 14, 2023
Signed-off-by: Nigel Jones <[email protected]>
planetf1
added a commit
to planetf1/egeria
that referenced
this issue
Mar 14, 2023
Signed-off-by: Nigel Jones <[email protected]>
planetf1
added a commit
to planetf1/egeria
that referenced
this issue
Mar 14, 2023
…with 3.x/maven Signed-off-by: Nigel Jones <[email protected]>
Tested with PR (above)
|
Re-opening as testing is needed now the initial code change is in. |
LeeVi3w
pushed a commit
to LeeVi3w/egeria
that referenced
this issue
Mar 16, 2023
Signed-off-by: Nigel Jones <[email protected]> Signed-off-by: Liviu Enache <[email protected]>
LeeVi3w
pushed a commit
to LeeVi3w/egeria
that referenced
this issue
Mar 16, 2023
Signed-off-by: Nigel Jones <[email protected]> Signed-off-by: Liviu Enache <[email protected]>
LeeVi3w
pushed a commit
to LeeVi3w/egeria
that referenced
this issue
Mar 16, 2023
Signed-off-by: Nigel Jones <[email protected]> Signed-off-by: Liviu Enache <[email protected]>
LeeVi3w
pushed a commit
to LeeVi3w/egeria
that referenced
this issue
Mar 16, 2023
Signed-off-by: Nigel Jones <[email protected]> Signed-off-by: Liviu Enache <[email protected]>
LeeVi3w
pushed a commit
to LeeVi3w/egeria
that referenced
this issue
Mar 16, 2023
Signed-off-by: Nigel Jones <[email protected]> Signed-off-by: Liviu Enache <[email protected]>
LeeVi3w
pushed a commit
to LeeVi3w/egeria
that referenced
this issue
Mar 16, 2023
Signed-off-by: Nigel Jones <[email protected]> Signed-off-by: Liviu Enache <[email protected]>
LeeVi3w
pushed a commit
to LeeVi3w/egeria
that referenced
this issue
Mar 16, 2023
…with 3.x/maven Signed-off-by: Nigel Jones <[email protected]> Signed-off-by: Liviu Enache <[email protected]>
The egeria BOM should now be correct. any problems will be raised as new issues |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
When building connectors, one useful technique is to set dependency constraints using the versions we have for core egeria.
This helps keep dependencies aligned, and reduces the risk of classpath clashes whilst avoiding having to manual keep versions in place.
This can be done in the consumer of egeria using either
or
More info at https://docs.gradle.org/current/userguide/platforms.html#sub:platforms-vs-catalog
This is similar to an 'import' scope dependency in maven
With the migration to gradle, an attempt to update the xtdb 4.0 connector to use the platform definition failed with:
This needs to be addressed so that we have a good platform definitions that consumers can use, and is a critical need for version 4 release
The text was updated successfully, but these errors were encountered: