[GitHub] [maven-resolver] olamy commented on pull request #269: [MNG-7705] Socket based SyncContext for multi-process/single-host locking

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

olamy commented on PR #269:
URL: https://github.com/apache/maven-resolver/pull/269#issuecomment-1475449506

   > > > I already planned to move **build time java requirement** to 11 for HTTP/2 transports, and leaving almost all modules at release=8 except for two HTTP/2 transport that would be release=11. Am fine to move build time req even to 17 and keep it on latest LTS... @michael-o ?
   > > 
   > > 
   > > It's purely build time because it's generating a multi-release jar, so runtime min requirement isn't changed.
   > 
   > Why not use https://github.com/kohlschutter/junixsocket for everything below 17? I wouldn't use 16 at all, it is dead and according to nio-dev there were still bugs in the UDS implementation.
   
   this one is very light https://github.com/jnr/jnr-unixsocket and works fine (used by Jetty project https://github.com/eclipse/jetty.project/tree/jetty-10.0.x/jetty-unixsocket).
   Looking really quickly at the code, you could simplify really the code by using Jetty `UnixSocketConnector` easy example here: https://github.com/eclipse/jetty.project/blob/jetty-10.0.x/jetty-unixsocket/jetty-unixsocket-server/src/test/java/org/eclipse/jetty/unixsocket/UnixSocketServer.java


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