rmannibucau commented on pull request #73: URL: https://github.com/apache/johnzon/pull/73#issuecomment-766165691
Well my reasoning is: 1. Jpms is not adopted and will not probably so maybe not a big deal 2. If not opened it is likely intended in jpms land so probably better to respect it in the spec (was not discussed) So overall your impl could be a toggle but *my* feeling is that it shouldnt be the default if we want to support jpms correctly - but once again jpms can be dead today so maybe not a big deal. Side note: no logging is likely desired so failling or swallowing probably? Anyone else has an opinion on it? ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
