On Fri, Sep 27, 2019 at 12:49 PM Mark Thomas <ma...@apache.org> wrote:
> On 26/09/2019 23:04, build...@apache.org wrote: > > The Buildbot has detected a new failure on builder tomcat-7-trunk while > building tomcat. Full details are available at: > > https://ci.apache.org/builders/tomcat-7-trunk/builds/1465 > > > > Buildbot URL: https://ci.apache.org/ > > > > Buildslave for this Build: asf946_ubuntu > > > > Build Reason: The AnyBranchScheduler scheduler named > 'on-tomcat-7-commit' triggered this build > > Build Source Stamp: [branch 7.0.x] > d0685f927186da2e0be56c28371e5207ee09ed20 > > Blamelist: Mark Thomas <ma...@apache.org> > > > > BUILD FAILED: failed compile_1 > > That was unexpected. This appears to have been triggered by: > > https://github.com/apache/tomcat/commit/247c8c32c44f65ce7bf3035541eedb8ffc97d5d0 > > Why this is the case isn't immediately obvious to me. I'm currently > investigating. > > The test seems to modify the listener array returned (it likely doesn't work now since it's a copy on write array following that commit), you apparently refactored that away in the new branches like 8.5 and the test looks much cleaner there. https://github.com/apache/tomcat/blob/master/test/org/apache/catalina/core/TestStandardContextResources.java#L128 vs https://github.com/apache/tomcat/blob/7.0.x/test/org/apache/catalina/core/TestStandardContextResources.java#L132 Rémy
