dsmiley commented on PR #12131: URL: https://github.com/apache/lucene/pull/12131#issuecomment-2080324742
IMO the complexity inherent (admittedly isn't a lot) in generating gradle.properties doesn't seem worth the pickiness of the preferences that some of us apparently have which brought it about. Customizing gradlew... having a template somewhere someone must find. The unusualness of it. Code to maintain is not free / without consequence; we have a complex build here. Can someone reiterate why its so important that we do this? The alternative to this is simple -- just commit gradle.properties to source control, and leave it mentioned in `.gitignore` so that anyone can adjust as they wish. We have control over tests.jvms having a computable default (since this is a setting of our invention read by our own scripts, not by gradle itself) -- thus if not specified (and it wouldn't unless someone sets it), we can implement the cap at 12. -- 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-unsubscr...@lucene.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org
