Skip to content

Remove ConfiguratorUtils that was deprecated for removal - #2730

Merged
HeikoKlare merged 1 commit into
eclipse-platform:masterfrom
deepika-u:remove-deprecated-apis1
Jun 23, 2026
Merged

Remove ConfiguratorUtils that was deprecated for removal#2730
HeikoKlare merged 1 commit into
eclipse-platform:masterfrom
deepika-u:remove-deprecated-apis1

Conversation

@deepika-u

Copy link
Copy Markdown
Contributor

The ConfiguratorUtils is deprecated for removal for more than 2 years. This pr removes it accordingly.

@github-actions

github-actions Bot commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

Test Results

    54 files  ±0      54 suites  ±0   58m 50s ⏱️ + 5m 22s
 4 677 tests ±0   4 655 ✅ ±0   22 💤 ±0  0 ❌ ±0 
11 925 runs  ±0  11 772 ✅ ±0  153 💤 ±0  0 ❌ ±0 

Results for commit f0dd9e2. ± Comparison against base commit abc8312.

♻️ This comment has been updated with latest results.

@eclipse-platform-bot

Copy link
Copy Markdown
Contributor

This pull request changes some projects for the first time in this development cycle.
Therefore the following files need a version increment:

update/org.eclipse.update.configurator/META-INF/MANIFEST.MF

An additional commit containing all the necessary changes was pushed to the top of this PR's branch. To obtain these changes (for example if you want to push more changes) either fetch from your fork or apply the git patch.

Git patch
From 0043aa4dc6038051fde99dd898c1cb7844410e25 Mon Sep 17 00:00:00 2001
From: Eclipse Platform Bot <platform-bot@eclipse.org>
Date: Fri, 5 Jun 2026 15:30:50 +0000
Subject: [PATCH] Version bump(s) for 4.41 stream


diff --git a/update/org.eclipse.update.configurator/META-INF/MANIFEST.MF b/update/org.eclipse.update.configurator/META-INF/MANIFEST.MF
index cf14f2bbe2..9e92d1fd8c 100644
--- a/update/org.eclipse.update.configurator/META-INF/MANIFEST.MF
+++ b/update/org.eclipse.update.configurator/META-INF/MANIFEST.MF
@@ -2,7 +2,7 @@ Manifest-Version: 1.0
 Bundle-ManifestVersion: 2
 Bundle-Name: %pluginName
 Bundle-SymbolicName: org.eclipse.update.configurator; singleton:=true
-Bundle-Version: 3.5.1000.qualifier
+Bundle-Version: 3.5.1100.qualifier
 Bundle-Activator: org.eclipse.update.internal.configurator.ConfigurationActivator
 Bundle-Vendor: %providerName
 Bundle-Localization: plugin
-- 
2.54.0

Further information are available in Common Build Issues - Missing version increments.

@deepika-u
deepika-u force-pushed the remove-deprecated-apis1 branch 4 times, most recently from c1ee0be to 0557c15 Compare June 11, 2026 11:36
@deepika-u

Copy link
Copy Markdown
Contributor Author

@merks
Could you take a look into this when you get some time please?

@deepika-u
deepika-u force-pushed the remove-deprecated-apis1 branch from 0557c15 to f025d1d Compare June 11, 2026 11:57

@merks merks left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This looks like the intended result of marking things for deletion.

@deepika-u

Copy link
Copy Markdown
Contributor Author

Thanks alot, does that mean do i have any actions to be taken as next step?

@HeikoKlare

Copy link
Copy Markdown
Contributor

API filters for the removed methods need to be added. Without them, the build will keep failing and the PR cannot be merged.

@deepika-u
deepika-u force-pushed the remove-deprecated-apis1 branch 3 times, most recently from cb5df1e to 7050db4 Compare June 17, 2026 10:37
@deepika-u

Copy link
Copy Markdown
Contributor Author

Needed changes are done and build is passing now. Thanks for your time in advance.

@deepika-u
deepika-u force-pushed the remove-deprecated-apis1 branch 2 times, most recently from 88eb487 to 9a38924 Compare June 19, 2026 10:11
@deepika-u
deepika-u force-pushed the remove-deprecated-apis1 branch from 9a38924 to f0dd9e2 Compare June 23, 2026 07:36
@deepika-u

Copy link
Copy Markdown
Contributor Author

@merks @HeikoKlare
Can this be merged now? all checks passed but just rebased again so that it can be merged once tests finish.

@merks

merks commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Looks okay to me. I restarted the Jenkins build to make sure it's okay too.

@HeikoKlare

Copy link
Copy Markdown
Contributor

Looks good to me as well. I have successfully verified with a complete SDK checked out that there are no remaining usages of the removed API inside the SDK.

@HeikoKlare
HeikoKlare merged commit 33cff3e into eclipse-platform:master Jun 23, 2026
18 checks passed
@deepika-u

Copy link
Copy Markdown
Contributor Author

Thanks alot :)

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.

4 participants