You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "cstamas (via GitHub)" <gi...@apache.org> on 2023/02/20 12:20:45 UTC

[GitHub] [maven] cstamas opened a new pull request, #1003: [EXPERIMENT] Halve logging

cstamas opened a new pull request, #1003:
URL: https://github.com/apache/maven/pull/1003

   Of the resolver (downloading vs downloaded), just report after
   the fact it was downloaded (or failed to download), no need
   for double lines for same resource/dependency.
   
   This experiment includes https://github.com/apache/maven/pull/989
   but does not affects it (just to create "closer" Maven to
   upcoming release).
   
   As one could expect, IT suite does fail with this change
   as some ITs do assert against the build logs (but the build
   itself did pass OK). Failing ITs:
   
   [ERROR] Failures:
   [ERROR]   MavenITmng6240PluginExtensionAetherProvider.testPluginExtensionDependingOnMavenAetherProvider:83->AbstractMavenIntegrationTestCase.assertEquals:668->AbstractMavenIntegrationTestCase.assertEquals:673 expected: <2> but was: <0>
   [INFO]
   [ERROR] Tests run: 885, Failures: 1, Errors: 0, Skipped: 80
   
   As to be expected, the IT asserts there is log line without
   progress (so DownloadING one, the one that is removed).


-- 
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 specific comment.

To unsubscribe, e-mail: issues-unsubscribe@maven.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [maven] cstamas closed pull request #1003: [EXPERIMENT] Halve logging

Posted by "cstamas (via GitHub)" <gi...@apache.org>.
cstamas closed pull request #1003: [EXPERIMENT] Halve logging
URL: https://github.com/apache/maven/pull/1003


-- 
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 specific comment.

To unsubscribe, e-mail: issues-unsubscribe@maven.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [maven] elharo commented on pull request #1003: [EXPERIMENT] Halve logging

Posted by "elharo (via GitHub)" <gi...@apache.org>.
elharo commented on PR #1003:
URL: https://github.com/apache/maven/pull/1003#issuecomment-1437044998

   I like it. :-)


-- 
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 specific comment.

To unsubscribe, e-mail: issues-unsubscribe@maven.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [maven] cstamas commented on pull request #1003: [EXPERIMENT] Halve logging

Posted by "cstamas (via GitHub)" <gi...@apache.org>.
cstamas commented on PR #1003:
URL: https://github.com/apache/maven/pull/1003#issuecomment-1436995573

   I completely agree, this wrapping I did in this PoC is ugy, and serves the purpose only to see "how would it look"... This PR is def not to be reused, especially not merged.


-- 
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 specific comment.

To unsubscribe, e-mail: issues-unsubscribe@maven.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org