Skip to content
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

Add support for using a hosted eclipse platform as target #1624

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

laeubi
Copy link
Contributor

@laeubi laeubi commented Feb 20, 2025

Currently if one uses the "Running Platform" in an Eclipse launched from within an SDK it can happen that a project from the host workspace is chosen as a dependency. In such case there are some issues that only the root folder is added and then compilation fails.

This now enhances the PDEClasspathContainer to detect the situation and adding the output folders instead of the root.

@laeubi laeubi force-pushed the add_support_for_hosted_projects_in_target branch from 336e8fc to 612f45a Compare February 20, 2025 13:56
Copy link

github-actions bot commented Feb 20, 2025

Test Results

   285 files  ±0     285 suites  ±0   46m 53s ⏱️ - 4m 19s
 3 608 tests ±0   3 532 ✅ ±0   76 💤 ±0  0 ❌ ±0 
11 016 runs  ±0  10 785 ✅ ±0  231 💤 ±0  0 ❌ ±0 

Results for commit 3aefcb7. ± Comparison against base commit f941e23.

♻️ This comment has been updated with latest results.

@HannesWell
Copy link
Member

Currently if one uses the "Running Platform" in an Eclipse launched from within an SDK it can happen that a project from the host workspace is chosen as a dependency. In such case there are some issues that only the root folder is added and then compilation fails.

Isn't this already handled by the target-platform weaving done by PDE based on the dev-classpath of Equinox?

@laeubi
Copy link
Contributor Author

laeubi commented Mar 4, 2025

Isn't this already handled by the target-platform weaving done by PDE based on the dev-classpath of Equinox?

If then it seem to not work in all situations.

@laeubi laeubi force-pushed the add_support_for_hosted_projects_in_target branch from 612f45a to dc16218 Compare March 4, 2025 09:34
Currently if one uses the "Running Platform" in an Eclipse launched from
within an SDK it can happen that a project from the host workspace is
chosen as a dependency. In such case there are some issues that only the
root folder is added and then compilation fails.

This now enhances the PDEClasspathContainer to detect the situation and
adding the output folders instead of the root.
@laeubi laeubi force-pushed the add_support_for_hosted_projects_in_target branch from dc16218 to 3aefcb7 Compare March 5, 2025 09:44
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.

2 participants