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

Consider the version string when adding bnd requirements #3826

Merged

Conversation

laeubi
Copy link
Member

@laeubi laeubi commented May 6, 2024

Currently Tycho ignores the version string for buildpath entries defined in pde.bnd what leads to faulty dependency requirements.

This now correctly evaluates the version string and maps it to a P2 requirement.

Currently Tycho ignores the version string for buildpath entries defined
in pde.bnd what leads to faulty dependency requirements.

This now correctly evaluates the version string and maps it to a P2
requirement.
@laeubi laeubi added the backport-to-tycho-4.0.x Can be added to a PR to trigger an automatic backport of the change label May 6, 2024
@laeubi laeubi enabled auto-merge (rebase) May 6, 2024 12:37
Copy link

github-actions bot commented May 6, 2024

Test Results

  594 files  ±0    594 suites  ±0   4h 20m 31s ⏱️ + 37m 58s
  420 tests ±0    413 ✅ ±0   7 💤 ±0  0 ❌ ±0 
1 260 runs  ±0  1 238 ✅ ±0  22 💤 ±0  0 ❌ ±0 

Results for commit 867bf59. ± Comparison against base commit 7490426.

@laeubi laeubi merged commit 7e3ba0e into eclipse-tycho:main May 6, 2024
12 checks passed
@eclipse-tycho-bot
Copy link

💚 All backports created successfully

Status Branch Result
tycho-4.0.x

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation and see the Github Action logs for details

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-to-tycho-4.0.x Can be added to a PR to trigger an automatic backport of the change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants