Add classpathDependencies option for target platform resolution#5917
Open
laeubi wants to merge 1 commit into
Open
Add classpathDependencies option for target platform resolution#5917laeubi wants to merge 1 commit into
laeubi wants to merge 1 commit into
Conversation
laeubi
force-pushed
the
tp_cp_config_option
branch
from
a9f0372 to
bf0f729
Compare
Member
Author
|
This naturally fails at the moment. We will verify it first with: then when it is merged and platform adjusted we can restart the check and merge this one as well! |
laeubi
force-pushed
the
tp_cp_config_option
branch
from
3e6d229 to
bf0f729
Compare
Test Results1 041 files 1 041 suites 4h 59m 50s ⏱️ For more details on these errors, see this check. Results for commit e7f094c. ♻️ This comment has been updated with latest results. |
Member
Author
|
One failing test... will investigate tomorrow - Tycho 5 backport is already merged... |
laeubi
force-pushed
the
tp_cp_config_option
branch
from
bf0f729 to
11df6d5
Compare
resolution Adds a new <classpathDependencies> configuration option under <dependency-resolution> in target-platform-configuration with values require (default), optional, and ignore. This controls how jars.extra.classpath entries from build.properties are handled during both preliminary resolution and classpath construction. With 'require', missing extra classpath entries fail the build. With 'optional', a warning is printed (previous behavior). With 'ignore', missing entries are silently skipped.
laeubi
force-pushed
the
tp_cp_config_option
branch
from
11df6d5 to
e7f094c
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds a new
<classpathDependencies>configuration option under<dependency-resolution>in target-platform-configuration with valuesrequire(default),optional, andignore. This controls howjars.extra.classpathentries frombuild.propertiesare handled duringresolution, fixing issues where platform-specific extra classpath
entries cause resolution failures (e.g. SWT SVG fragment).
As a continuation of