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

[pre-test]OSGi: OSGi and bnd version updates and - Cleanups #9422

Closed
wants to merge 1 commit into from

Conversation

stbischof
Copy link
Contributor

@stbischof stbischof commented Nov 17, 2020

Separate PR to update bnd and osgi as discussed here #9361 and #9321

@stbischof stbischof force-pushed the osgi_cleanup branch 2 times, most recently from 16b6bbe to 8a46671 Compare November 17, 2020 10:25
@stbischof stbischof force-pushed the osgi_cleanup branch 3 times, most recently from dcabb45 to 392d91d Compare November 18, 2020 20:59
Signed-off-by: Stefan Bischof <stbischof@bipolis.org>
@denis-anisimov
Copy link
Contributor

@stbischof , osgi repo is not public: https://github.com/vaadin/osgi
All OSGi related code is moved now there and Flow repo doesn't contain OSGi specific code anymore except config for manifest files.

the master branch in osgi repo is protected and validation is enabled.
Please :

  • rebase the changes against the master branch.
  • check that validation in osgi repo passes

Since this PR is for Flow and doesn't touch OSGi related code you should run the validation by yourself in the https://github.com/vaadin/osgi/tree/master/test-containers/felix-jetty.
You don't need to run all the tests. Just check that there are no errors and check at least one view.

Please make all OSGi related code PRs now in osgi repo. Thanks.

@pleku pleku closed this Feb 3, 2021
@pleku
Copy link
Contributor

pleku commented Feb 3, 2021

Closed as for the OSGi support is in its own public repository since version 6.0.

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

Successfully merging this pull request may close these issues.

4 participants