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

Support automatic updates from gitlab package registry #2491

Merged

Conversation

MichaelRitzert
Copy link
Contributor

With this code, automatic updates can be retrieved from gitlab CI builds.

The update mechanism are implemented as different plugins detected via the ServiceLoader. All detected plugins are checked to find one that has all configuration in the ini file. This plugin will be used.

The current code that updates from plain URLs has been refactored but is functionally unchanged.

Several update providers can be configured through the java ServiceLoader
mechanism. Only one of them can be enabled at runtime by providing the
required configuration.
@kasemir
Copy link
Collaborator

kasemir commented Feb 10, 2023

I had put this into the 'review' section of the github web page, which doesn't always show up for everybody, so here it is again as a comment:

When you add dependencies, then please also add them to the target platform, see https://github.com/ControlSystemStudio/phoebus/blob/master/dependencies/README.md and https://github.com/ControlSystemStudio/phoebus/blob/master/dependencies/phoebus-target/pom.xml. With the current settings, we already have javax.json-api-1.0.jar, while you request 1.1.4. javax.jms-api would be a new one. In the end, please assert that all depedencies that you need are listed in the https://github.com/ControlSystemStudio/phoebus/blob/master/dependencies/phoebus-target/pom.xml and that we end up with one version for javax.json-api

In other words, this PR adds dependencies to javax.jms-api 2.0.1 and javax.json-api 1.1.4, which at the time conflicted with the versions in the target platform.

Meanwhile, #2528 happened. When building the target platform right now, it includes javax.jms-api-2.0.1.jar and javax.json-api-1.1.4.jar, same versions as in this PR, so I think we're all set!

@kasemir kasemir merged commit e7a6ec4 into ControlSystemStudio:master Feb 10, 2023
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

Successfully merging this pull request may close these issues.

2 participants