Skip to content

Use PdeProjectAnalyzer to compute required packages#1862

Merged
laeubi merged 1 commit intoeclipse-pde:masterfrom
laeubi:use_bnd_to_calculate_used
Jul 10, 2025
Merged

Use PdeProjectAnalyzer to compute required packages#1862
laeubi merged 1 commit intoeclipse-pde:masterfrom
laeubi:use_bnd_to_calculate_used

Conversation

@laeubi
Copy link
Copy Markdown
Contributor

@laeubi laeubi commented Jul 10, 2025

Currently the GatherUnusedDependenciesOperation uses JDT reference search to determine if a package is used. This is not only very slow but also unreliable as it does not take into account indirect references e.g. from superclasses or method signatures.

This now uses the PdeProjectAnalyzer to compute all required packages once and then check if an import is providing such package.

@laeubi laeubi added the noteworthy Noteworthy feature label Jul 10, 2025
Currently the GatherUnusedDependenciesOperation uses JDT reference
search to determine if a package is used. This is not only very slow but
also unreliable as it does not take into account indirect references
e.g. from superclasses or method signatures.

This now uses the PdeProjectAnalyzer to compute all required packages
once and then check if an import is providing such package.
@laeubi laeubi force-pushed the use_bnd_to_calculate_used branch from c3f027e to 2d76237 Compare July 10, 2025 07:49
@github-actions
Copy link
Copy Markdown

Test Results

   765 files  ±0     765 suites  ±0   49m 37s ⏱️ - 1m 31s
 3 611 tests ±0   3 557 ✅ +1   54 💤 ±0  0 ❌  - 1 
10 833 runs  ±0  10 670 ✅ +1  163 💤 ±0  0 ❌  - 1 

Results for commit 2d76237. ± Comparison against base commit 7ac2ba1.

@laeubi laeubi merged commit da73cec into eclipse-pde:master Jul 10, 2025
18 of 19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

noteworthy Noteworthy feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant