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

Add support for platform URLs in automatic manifest generation plugins #3370

Merged
merged 1 commit into from
Jan 16, 2024

Conversation

laeubi
Copy link
Member

@laeubi laeubi commented Jan 16, 2024

PDE allows the definition of extraClassPathJars what is very similar to bnd -classpath, one usually uses a platform://. Currently there are two issues:

  1. the tycho-compile does not understand these
  2. bnd tries to resolve them with httpclient

This adds the code to correctly resolve them in a similar way as Tycho does for extraClassPathJars.

@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 Jan 16, 2024
@laeubi laeubi force-pushed the platform_url_in_bnd branch 2 times, most recently from 24952ef to acb71db Compare January 16, 2024 09:56
PDE allows the definition of extraClassPathJars what is very similar to
bnd -classpath, one usually uses a platform:/<reference to
project>/<somejar>. Currently there are two issues:

1) the tycho-compile does not understand these
2) bnd tries to resolve them with httpclient

This adds the code to correctly resolve them in a similar way as Tycho
does for extraClassPathJars.
Copy link

Test Results

  579 files  ±0    579 suites  ±0   3h 55m 54s ⏱️ + 11m 57s
  383 tests ±0    375 ✅ ±0   7 💤 ±0  1 ❌ ±0 
1 149 runs  ±0  1 126 ✅ ±0  22 💤 ±0  1 ❌ ±0 

For more details on these failures, see this check.

Results for commit 7c57bbc. ± Comparison against base commit 1f99d2d.

@laeubi laeubi merged commit efe4ce1 into eclipse-tycho:main Jan 16, 2024
7 of 10 checks passed
Copy link

💚 All backports created successfully

Status Branch Result
tycho-4.0.x

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.

1 participant