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

[backport] Transitive dependency projects are not discovered with --also-make #3720

Merged
merged 1 commit into from
Mar 29, 2024

Conversation

laeubi
Copy link
Member

@laeubi laeubi commented Mar 29, 2024

Currently it can happen that if one selects a project with -pl / -am this project has initially not any maven dependencies but then later on tycho adds one as a dependency and then the one declared in the pom are not found.

This now adds an additional step each time a project is added as a dependency to look if that project also has maven dependencies that map to reactor projects.

This also includes some fixups to the readme and changelog of the 4.x branch.

Currently it can happen that if one selects a project with -pl / -am
this project has initially not any maven dependencies but then later on
tycho adds one as a dependency and then the one declared in the pom are
not found.

This now adds an additional step each time a project is added as a
dependency to look if that project also has maven dependencies that map
to reactor projects.
@laeubi laeubi enabled auto-merge (rebase) March 29, 2024 10:48
@laeubi laeubi changed the title Transitive dependency projects are not discovered with --also-make [backport] Transitive dependency projects are not discovered with --also-make Mar 29, 2024
Copy link

Test Results

  591 files  ±0    591 suites  ±0   3h 45m 37s ⏱️ - 14m 20s
  404 tests ±0    396 ✅  - 1   6 💤 ±0  2 ❌ +1 
1 212 runs  ±0  1 191 ✅  - 1  19 💤 ±0  2 ❌ +1 

For more details on these failures, see this check.

Results for commit 45cf05f. ± Comparison against base commit 42d40a9.

@laeubi laeubi merged commit 52b66fd into eclipse-tycho:tycho-4.0.x Mar 29, 2024
9 of 11 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