[GitHub] [lucene] dweiss commented on pull request #923: LUCENE-10200: Replace classpath with modulepath in the demo tutorial

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

dweiss commented on PR #923:
URL: https://github.com/apache/lucene/pull/923#issuecomment-1137165723

   Modular are not new, they're just not widespread... I agree with @mocobeta that if you provide an explicit command line then there is little harm in not explaining all the options. The problem with classpath is that you need to include all JARs individually  this is handled much better by modules since you include the directory path (not each individual module JAR).
   
   This said, I don't have a strong opinion about going either way. 


-- 
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@lucene.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org