Skip to content

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
@laeubi laeubi marked this pull request as draft July 19, 2025 18:54
@laeubi
Copy link
Contributor Author

laeubi commented Jul 19, 2025

I convert this to draft as I currently did not see any issues...

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