Skip to content

Don't fail framework start/stop on errors with plurl register/unregister - #1365

Merged
tjwatson merged 1 commit into
eclipse-equinox:masterfrom
tjwatson:plurlFail
Oct 1, 2026
Merged

tjwatson merged 1 commit into
eclipse-equinox:masterfrom
tjwatson:plurlFail

Conversation

@tjwatson

@tjwatson tjwatson commented Oct 1, 2026

Copy link
Copy Markdown
Contributor

In the OSGi TCK there was a test bug in their test class loader that loads the framework implementation. This caused Error to be thrown out of plurl. If this happens the framework should not fall down completely and fail to start or stop.

This change makes sure we can recover if this happens and move on with the framework start/stop when an Error is thrown.

In the OSGi TCK there was a test bug in their test class loader that
loads the framework implementation.  This caused Error to be thrown
out of plurl. If this happens the framework should not fall down
completely and fail to start or stop.

This change makes sure we can recover if this happens and
move on with the framework start/stop when an Error is thrown.
@github-actions

github-actions Bot commented Oct 1, 2026

Copy link
Copy Markdown

Test Results

  231 files  ±0    231 suites  ±0   47m 14s ⏱️ -18s
2 219 tests ±0  2 170 ✅ ±0   49 💤 ±0  0 ❌ ±0 
6 420 runs  ±0  6 307 ✅ ±0  113 💤 ±0  0 ❌ ±0 

Results for commit a908fd5. ± Comparison against base commit 6bd83d7.

@tjwatson
tjwatson merged commit 66584d1 into eclipse-equinox:master Oct 1, 2026
28 of 29 checks passed
@tjwatson
tjwatson deleted the plurlFail branch October 5, 2026 14:19
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.

1 participant