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

Fix lookup of workspace located annotation providers #1063

Merged
merged 1 commit into from
Jan 21, 2024

Conversation

laeubi
Copy link
Contributor

@laeubi laeubi commented Jan 21, 2024

Currently, if a annotation project is part of the workspace (most notable (org.eclipse.pde.api.tools.annotations) it currently does not resolve the binary folders correctly.

This fixes the conversion of plugin models to classpath entries by check if a build model is available and if found uses the binary folders as classpath entries instead of the base location.

Copy link

github-actions bot commented Jan 21, 2024

Test Results

   277 files  ±0     277 suites  ±0   1h 0m 59s ⏱️ - 2m 52s
 3 496 tests ±0   3 460 ✅ ±0   36 💤 ±0  0 ❌ ±0 
10 619 runs  ±0  10 517 ✅ ±0  102 💤 ±0  0 ❌ ±0 

Results for commit cf42481. ± Comparison against base commit 76e5d1f.

♻️ This comment has been updated with latest results.

@laeubi laeubi force-pushed the fix_cp_compute_with_imported branch 2 times, most recently from 5664fa6 to db1a17d Compare January 21, 2024 08:05
Currently, if a annotation project is part of the workspace (most
notable (org.eclipse.pde.api.tools.annotations) it currently does not
resolve the binary folders correctly.

This fixes the conversion of plugin models to classpath entries by check
if a build model is available and if found uses the binary folders as
classpath entries instead of the base location.
@laeubi laeubi force-pushed the fix_cp_compute_with_imported branch from db1a17d to cf42481 Compare January 21, 2024 12:14
@laeubi laeubi merged commit c4c5104 into eclipse-pde:master Jan 21, 2024
14 of 15 checks passed
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.

1 participant