[GitHub] [maven] mthmulders commented on pull request #995: [MNG-6869] New flag to verify Maven installation status

["mthmulders (via GitHub)" <gi...@apache.org>]

mthmulders commented on PR #995:
URL: https://github.com/apache/maven/pull/995#issuecomment-1424703182

   > Isnt it something a plugin would be a better location?
   
   The current approach does not need any additional download. I think that's a good thing, as we can now use it to also point the user at issues in their configuration that would prevent artifact resolution. If we moved this to a plugin, we'd need artifact resolution to actually work...
   
   > Can need some basic downloads but would enable to do more checks on the project, better configure the dummy artifact - 3.8.6 or maven cant be hardcoded in case of central "mirroring", it can be forbidden intentionally.
   
   I'm open to suggestions for another publicly available artifact. But in order to "prove" that artifact resolution works, we need something that is publicly available - I think something that at the very least is hosted by Central, and maybe by mirrors. 
   
   > It would also enable to maje checks grow, in particular with coming multilocal repo feature for ex.
   
   More thorough checks could reside in a plugin, but as pointed out before, we can't rely on resolution to work until we've checked that it works. So agreed, additional checks could live in a plugin, but I'm convinced that we'll need a few basic checks (among them artifact resolution and local repository setup) to live in Core.


-- 
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