[GitHub] [incubator-pekko] mdedetrich commented on pull request #81: Remove sbtx usage

[GitBox <gi...@apache.org>]

mdedetrich commented on PR #81:
URL: https://github.com/apache/incubator-pekko/pull/81#issuecomment-1369594221

   > yes indeed, we could rename, that's the alternative. However, naturally, this means these options would also be picked up when starting sbt locally, which is maybe not what we want 🤔 ?
   
   So the underlying point of the issue and this PR is that `sbtx` is actually not used in any distribution if you install sbt the standard way. A lot of functionality over time has moved from `sbtx` to `sbt` and today there is no reason to use sbt. This is why @SethTisue was annoyed with `setup-scala` aliasing `sbt` to `sbtx` because its misleading.
   
   The tl;dr is that nowadays if you have sbt on Linux/Mac/Windows its just using standard sbt (not `sbtx`) and its ignoring those cli options anyways
   
   > Do we actually know if these options are necessary at all? some of them might no longer be relevant, e.g. UseCompressedOops seems to be a default nowadays
   
   That is a fair point but I would fix this in a separate PR just to make things clear.


-- 
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: notifications-unsubscribe@pekko.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@pekko.apache.org
For additional commands, e-mail: notifications-help@pekko.apache.org