Skip to content
This repository has been archived by the owner on Apr 21, 2023. It is now read-only.

made o.e.xtext.ide dep. to org.eclipse.core.runtime optional #446 #447

Merged
merged 1 commit into from Aug 29, 2017

Conversation

cdietrich
Copy link
Member

made o.e.xtext.ide dep. to org.eclipse.core.runtime optional #446
Signed-off-by: Christian Dietrich christian.dietrich@itemis.de

Signed-off-by: Christian Dietrich <christian.dietrich@itemis.de>
@cdietrich
Copy link
Member Author

please review

@@ -8,7 +8,8 @@ Bundle-ActivationPolicy: lazy
Bundle-Vendor: Eclipse Xtext
Require-Bundle: org.eclipse.xtext;visibility:=reexport,
org.eclipse.xtend.lib,
org.eclipse.core.runtime;bundle-version="3.6.0",
org.eclipse.core.runtime;bundle-version="3.6.0";resolution:=optional;x-installation:=greedy,
org.eclipse.equinox.common;bundle-version="3.5.0",
Copy link

Choose a reason for hiding this comment

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

why 3.5 here while core.runtime would be 3.6?

Copy link
Member Author

@cdietrich cdietrich Aug 29, 2017

Choose a reason for hiding this comment

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

copy&paste from orr.eclipse.xtext ?!?

Copy link

Choose a reason for hiding this comment

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

OK, if this valid for o.e.x then it should be fine.

@cdietrich cdietrich merged commit eeba774 into master Aug 29, 2017
@cdietrich cdietrich deleted the cd_issue446 branch August 29, 2017 12:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants