Bug 476404 - Delete Platform API as announced for Eclipse 4.2.
Summary: Delete Platform API as announced for Eclipse 4.2.
Status: RESOLVED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.7.1   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: 4.24 M2   Edit
Assignee: Lars Vogel CLA
QA Contact:
URL:
Whiteboard:
Keywords: api
Depends on: 370248
Blocks: 527665 440136
  Show dependency tree
 
Reported: 2015-09-02 05:09 EDT by Lars Vogel CLA
Modified: 2022-04-20 11:01 EDT (History)
8 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Lars Vogel CLA 2015-09-02 05:09:18 EDT
+++ This bug was initially created as a clone of Bug #370248 +++

From the bug:

The summary of APIs I think we should mark for future deletion:

Entire packages:

org.eclipse.core.boot
org.eclipse.core.runtime.model
org.eclipse.ui.presentations

Methods:

 - Platform#addAuthorizationInfo
 - Platform#getAuthorizationInfo
 - Platform#flushAuthorizationInfo
 - Platform#addProtectionSpace
 - Platform#getProtectionSpace

These are all cases of API that are no longer functional in 4.2.
Comment 1 Eclipse Genie CLA 2015-09-02 07:22:04 EDT
New Gerrit change created: https://git.eclipse.org/r/55080
Comment 3 Lars Vogel CLA 2015-09-11 12:43:45 EDT
.
Comment 4 Eclipse Genie CLA 2015-09-12 02:21:38 EDT
New Gerrit change created: https://git.eclipse.org/r/55794
Comment 6 Dani Megert CLA 2015-09-12 02:23:37 EDT
Reverted since it broke our build.
Comment 7 Eclipse Genie CLA 2017-11-23 04:34:35 EST
New Gerrit change created: https://git.eclipse.org/r/112153
Comment 8 Eclipse Genie CLA 2017-11-23 04:45:23 EST
New Gerrit change created: https://git.eclipse.org/r/112154
Comment 10 Lars Vogel CLA 2017-11-23 06:00:23 EST
(In reply to Dani Megert from comment #6)
> Reverted since it broke our build.

Alex, in the past the removal did cause a build error, not sure why. I added you to the bug in case this happens again.
Comment 12 Dani Megert CLA 2017-11-23 09:59:15 EST
(In reply to Eclipse Genie from comment #9)
> Gerrit change https://git.eclipse.org/r/112153 was merged to [master].
> Commit:
> http://git.eclipse.org/c/platform/eclipse.platform.runtime.git/commit/?id=835d3aff713f4cb4305e521da40222ca9439118b
> 

I reverted this as it breaks CVS support. Affected methods:

org.eclipse.core.runtime.Platform.getAuthorizationInfo(URL, String, String)
org.eclipse.core.runtime.Platform.addAuthorizationInfo(URL, String, String, Map<String, String>)
Comment 13 Eclipse Genie CLA 2017-11-29 02:48:13 EST
New Gerrit change created: https://git.eclipse.org/r/112530
Comment 14 Eclipse Genie CLA 2017-11-29 02:48:16 EST
New Gerrit change created: https://git.eclipse.org/r/112530
Comment 16 Dani Megert CLA 2017-12-05 12:10:21 EST
This bug did not get delivered for the specified target milestone. Please set the target milestone when you plan to deliver the fix.
Comment 17 Lars Vogel CLA 2018-02-14 04:14:33 EST
(In reply to Dani Megert from comment #12)
> I reverted this as it breaks CVS support. Affected methods:
> 
> org.eclipse.core.runtime.Platform.getAuthorizationInfo(URL, String, String)
> org.eclipse.core.runtime.Platform.addAuthorizationInfo(URL, String, String,
> Map<String, String>)

So only these two methods must not be deleted?
Comment 18 Eclipse Genie CLA 2020-02-24 17:55:57 EST
This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet. As such, we're closing this bug.

If you have further information on the current state of the bug, please add it and reopen this bug. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant.

--
The automated Eclipse Genie.
Comment 19 Lars Vogel CLA 2022-04-14 05:21:13 EDT
As CVS was retired, we can now remove the auth methods.
Comment 21 Lars Vogel CLA 2022-04-20 11:01:35 EDT
API removed with https://github.com/eclipse-platform/eclipse.platform.runtime/pull/21