Skip to content

Fix split package between swt.spies and swt.tools#1943

Merged
akurtakov merged 1 commit intoeclipse-pde:masterfrom
akurtakov:master
Aug 22, 2025
Merged

Fix split package between swt.spies and swt.tools#1943
akurtakov merged 1 commit intoeclipse-pde:masterfrom
akurtakov:master

Conversation

@akurtakov
Copy link
Copy Markdown
Member

Move Sleak class to o.e.swt.internal.tools.views. It's more correct (or at least less wrong as per
https://github.com/eclipse-platform/eclipse.platform/blob/master/docs/Naming_Conventions.md ).
Removes the nasty problem of split packages signed with different certificates.

Copy link
Copy Markdown
Contributor

@merks merks left a comment

Choose a reason for hiding this comment

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

+1

This change is related to

eclipse-platform/eclipse.platform.releng.aggregator#3190

because it address this problem in a permanent by avoiding the split package altogether.

image

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Aug 22, 2025

Test Results

   765 files  +  147     765 suites  +147   55m 48s ⏱️ +38s
 3 611 tests ±    0   3 557 ✅ +    1   54 💤 ± 0  0 ❌  - 1 
10 833 runs  +2 650  10 670 ✅ +2 627  163 💤 +24  0 ❌  - 1 

Results for commit d68f5ce. ± Comparison against base commit d9ad053.

♻️ This comment has been updated with latest results.

Comment thread ui/org.eclipse.swt.tools.spies/META-INF/MANIFEST.MF Outdated
Move Sleak class to o.e.swt.internal.tools.views. It's more correct (or
at least less wrong as per
https://github.com/eclipse-platform/eclipse.platform/blob/master/docs/Naming_Conventions.md
).
Removes the nasty problem of split packages signed with different
certificates.
@akurtakov
Copy link
Copy Markdown
Member Author

There is PMC member +1 thus merging in order to get the bundle signed proper for RC2.

@akurtakov akurtakov merged commit ebb6634 into eclipse-pde:master Aug 22, 2025
18 of 19 checks passed
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.

3 participants