Re: [I] gradle's incremental compilation can get confused and lead to odd build errors [lucene]
dweiss commented on issue #12742: URL: https://github.com/apache/lucene/issues/12742#issuecomment-1788129507 I think the reason for an up-to-date status is because we handle module path ourselves ``` // Add modular dependencies and their transitive dependencies to module path. task.options.compilerArgumentProviders.add(modularPaths.compilationArguments) ``` which means gradle can't inspect the _contents_ of those dependencies (as it would with regular classpath entries). I have to think how to handle this better. It's already hairy enough. -- 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
Re: [I] gradle's incremental compilation can get confused and lead to odd build errors [lucene]
dweiss commented on issue #12742: URL: https://github.com/apache/lucene/issues/12742#issuecomment-1788119452 It's not incremental compilation - it's something that is related to task inputs and (likely) modular dependencies. Seems like we're missing something that makes the test compilation task to be marked as up-to-date, even though its dependencies (core) have been recompiled. I couldn't find the cause yet, but I'll keep digging. -- 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
[I] gradle's incremental compilation can get confused and lead to odd build errors [lucene]
dweiss opened a new issue, #12742: URL: https://github.com/apache/lucene/issues/12742 ### Description Recent builds failed with: ``` org.apache.lucene.search.uhighlight.TestLengthGoalBreakIterator > testTargetLen FAILED java.lang.NoSuchMethodError: 'org.apache.lucene.util.automaton.Automaton org.apache.lucene.util.automaton.Automata.makeStringUnion(java.util.Collection)' at __randomizedtesting.SeedInfo.seed([18019D3C4539B4C7:DBF0A37A952BEBF3]:0) at org.apache.lucene.search.uhighlight.CharArrayMatcher.fromTerms(CharArrayMatcher.java:42) at org.apache.lucene.search.uhighlight.MemoryIndexOffsetStrategy.buildCombinedAutomaton(MemoryIndexOffsetStrategy.java:71) at org.apache.lucene.search.uhighlight.MemoryIndexOffsetStrategy.(MemoryIndexOffsetStrategy.java:53) ``` As Hoss noticed, there is a repro to get the above error: ``` It's actually pretty trivial to reproduce this problem locally... git co 11436a848cbcc8302b31ca01e63e57dd35a93b1e ./gradlew clean classes testClasses git co 361f0a4524d93fc1a793aa72bfcc91c00fa3599e ./gradlew test --tests TestFlattenGraphFilter ``` I'll try to figure out why this can happen, it shouldn't. ### Version and environment details _No response_ -- 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.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