[GitHub] [maven-file-management] cstamas commented on pull request #11: [MSHARED-1090] Update module

2022-06-28 Thread GitBox
cstamas commented on PR #11: URL: https://github.com/apache/maven-file-management/pull/11#issuecomment-1169218949 General: this is a utility class, is NOT a component. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [maven-file-management] cstamas commented on pull request #11: [MSHARED-1090] Update module

2022-06-28 Thread GitBox
cstamas commented on PR #11: URL: https://github.com/apache/maven-file-management/pull/11#issuecomment-1169217540 verbose is orthogonal to log level. That's all IMHO -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [maven-file-management] cstamas commented on pull request #11: [MSHARED-1090] Update module

2022-06-28 Thread GitBox
cstamas commented on PR #11: URL: https://github.com/apache/maven-file-management/pull/11#issuecomment-1169217153 re verbose: disagree. This is "verbose", will tell more, that's all. Log level is INFO -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [maven-file-management] cstamas commented on pull request #11: [MSHARED-1090] Update module

2022-06-28 Thread GitBox
cstamas commented on PR #11: URL: https://github.com/apache/maven-file-management/pull/11#issuecomment-1169190109 Tidied up completely -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [maven-file-management] cstamas commented on pull request #11: [MSHARED-1090] Update module

2022-06-28 Thread GitBox
cstamas commented on PR #11: URL: https://github.com/apache/maven-file-management/pull/11#issuecomment-1168588059 > Why do you I have the feeling that I have seen this with Plexus Utils too?! This is "already seen" to me as well, no idea where but really looks familiar... -- This

[GitHub] [maven-file-management] cstamas commented on pull request #11: [MSHARED-1090] Update module

2022-06-27 Thread GitBox
cstamas commented on PR #11: URL: https://github.com/apache/maven-file-management/pull/11#issuecomment-1167323621 https://github.com/search?l=Java=org.apache.maven.shared.model.fileset.util.FileSetManager=Code -- This is an automated message from the Apache Git Service. To respond to

[GitHub] [maven-file-management] cstamas commented on pull request #11: [MSHARED-1090] Update module

2022-06-27 Thread GitBox
cstamas commented on PR #11: URL: https://github.com/apache/maven-file-management/pull/11#issuecomment-1167322093 maven-jar-plugin for sure, unsure for others, but gh search reveals a LOT of uses across many (non our) plugins -- This is an automated message from the Apache Git

[GitHub] [maven-file-management] cstamas commented on pull request #11: [MSHARED-1090] Update module

2022-06-27 Thread GitBox
cstamas commented on PR #11: URL: https://github.com/apache/maven-file-management/pull/11#issuecomment-1167304592 Hm, maybe we need NoopMessageSink to retain existing behaviour with defautl ctor... as now it will use sl4j logger and this is behaviour change from before when def ctor used.

[GitHub] [maven-file-management] cstamas commented on pull request #11: [MSHARED-1090] Update module

2022-06-27 Thread GitBox
cstamas commented on PR #11: URL: https://github.com/apache/maven-file-management/pull/11#issuecomment-1167272364 Of course, but did not want to mix the two (there are a LOT of other changes already requiring it) -- This is an automated message from the Apache Git Service. To respond to