[maven][resolve] Provide a better logging experience for maven users when resolving#7255
Merged
Merged
Conversation
Contributor
|
@timothyjward Thanks. Have you noticed the error in the run? Also I would like to ask @kwin for a review. |
timothyjward
force-pushed
the
fix/7217-resolve-log
branch
2 times, most recently
from
5768fec to
c702b39
Compare
…when resolving This commit updates the bndrun resolve process to be better able to log externally. The main driver for this is from maven plugins which currently get little to no logging from the resolve process. These now use an SLF4J logger to log resolve messages, while still logging to a temporary file to maintain the existing error reporting behaviour. Fixes bndtools#7217 Signed-off-by: Tim Ward <timothyjward@apache.org>
timothyjward
force-pushed
the
fix/7217-resolve-log
branch
from
c702b39 to
11b8fb9
Compare
chrisrueger
approved these changes
Jun 4, 2026
Contributor
|
Thanks, LGTM. |
kwin
approved these changes
Jun 5, 2026
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
3 participants
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.
This commit updates the bndrun resolve process to be better able to log externally. The main driver for this is from maven plugins which currently get little to no logging from the resolve process. These now use an SLF4J logger to log resolve messages, while still logging to a temporary file to maintain the existing error reporting behaviour.
Fixes #7217