-
Notifications
You must be signed in to change notification settings - Fork 164
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
Missing class usage StringMatcher in eclipse internal code #514
Comments
@gayanper , thanks for reporting it, could you like to raise a PR? |
@missedone to fix this we need to bump the target version to something like 2020-06, So i need your input on how do you want to handle this, if we change the target version then we problem need to have a maintenance branch and a maintenance p2 site to support old releases. |
@gayanper , checked around the code, we don't heavily rely on StringMatcher, so I think we have two options:
what do you think? |
I thought about making a copy. But not sure about licensing. I couldn’t find license headers or file. Using something else is a good suggestion as well. I can look into that or writing something simple in plugin it self. |
@gayanper , i think the license should be fine, you can see the license header on top of the code https://github.com/eclipse/rt.equinox.bundles/blob/master/bundles/org.eclipse.equinox.common/src/org/eclipse/core/text/StringMatcher.java#L5-L7 and EPL is a weak copyleft license, which grant rights to copy, adapt and distribute the program in source or object code form |
@missedone sure i will look into it. |
The Usage of StringMatcher
org.eclipse.ui.internal.misc.StringMatcher
org.eclipse.ui.internal.ide.StringMatcher
needs to migrate to
org.eclipse.core.text.StringMatcher
according to https://bugs.eclipse.org/bugs/show_bug.cgi?id=564946. This can cause problems if this plugin is used in later versions where this Eclipse version and execute the code which require the missing class.
The text was updated successfully, but these errors were encountered: