uschindler edited a comment on pull request #567:
URL: https://github.com/apache/lucene/pull/567#issuecomment-1000280448


   > How can we have failing tests for these problems? We need a test-driven 
approach to fixing this stuff. `gradlew check` is totally happy for me.
   
   This is preparatory. We have a branch where it does not work because of this.
   
   See the issue for explanation. The problem is that getResourceAsStream() and 
getResource() are caller-sensitive and check who calls it. If its a classpath 
app, it works as the checks are gnored. With module system the method returns 
null if you try to load a resource in another module. I will add a test to the 
distribution tests to show this.


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

Reply via email to