Skip to content

Delegate getValue/setValue in DefaultBuildContext to legacy build API#108

Merged
laeubi merged 1 commit intocodehaus-plexus:masterfrom
GEBIT:pr/delegate-values-to-legacy
Feb 5, 2026
Merged

Delegate getValue/setValue in DefaultBuildContext to legacy build API#108
laeubi merged 1 commit intocodehaus-plexus:masterfrom
GEBIT:pr/delegate-values-to-legacy

Conversation

@Volodjam
Copy link
Contributor

@Volodjam Volodjam commented Feb 3, 2026

This prevents objects from being stored in the singleton DefaultBuildContext instead of the thread-local build context (ThreadBuildContext) in Eclipse, avoiding unintended cross-project state sharing.

Fix #107

@laeubi
Copy link
Collaborator

laeubi commented Feb 4, 2026

I think formally one should only use the get/set values if isIncremental() return true and the API even states that it will return always null in this case...

@laeubi
Copy link
Collaborator

laeubi commented Feb 4, 2026

@Volodjam you need to run 'mvn spotless:apply' to fix formating violations.

This prevents objects from being stored in the singleton
DefaultBuildContext instead of the thread-local build context
(ThreadBuildContext) in Eclipse, avoiding unintended cross-project state
sharing.
@Volodjam Volodjam force-pushed the pr/delegate-values-to-legacy branch from 5a9a8b6 to 1f8b782 Compare February 4, 2026 14:44
@Volodjam
Copy link
Contributor Author

Volodjam commented Feb 5, 2026

Done. Formatting issues are fixed and the branch has been force-pushed.

@laeubi laeubi merged commit 87e1417 into codehaus-plexus:master Feb 5, 2026
11 checks passed
@laeubi
Copy link
Collaborator

laeubi commented Feb 5, 2026

Thanks for spotting this!

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.

Let the DefaultBuildContext also delegate getValue and setValue to the legacy build-api

2 participants