tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Mark Thomas <>
Subject Re: [VOTE] Release Apache Tomcat 7.0.24
Date Tue, 17 Jan 2012 08:46:43 GMT
On 17/01/2012 01:15, Konstantin Kolinko wrote:
> 2012/1/17 Mark Thomas <>:
>> On 16/01/2012 19:04, Sylvain Laurent wrote:
>>> On 16 janv. 2012, at 10:44, Rainer Jung wrote:
>>>> 1) Unit test failure due to missing target directory in src dist
>>>> ================================================================
>>>> I noticed that test/webapp-3.0-virtual-library/target is missing from the
src dist. So when rebuilding from src dist and running unit tests, I get a failure in TestVirtualContext,
because some resource which is usually retrieved form the target dir is missing:
>>>> Testcase: testVirtualClassLoader took 7.434 sec
>>>>        FAILED
>>>> expected:<200> but was:<404>
>>>> junit.framework.AssertionFailedError: expected:<200> but was:<404>
>>>>        at org.apache.catalina.loader.TestVirtualContext.assertPageContains(
>>>>        at org.apache.catalina.loader.TestVirtualContext.assertPageContains(
>>>>        at org.apache.catalina.loader.TestVirtualContext.testVirtualClassLoader(
>>>> AFAIK the root cause is, that we exclude **/target/** in the ant target dist-source
from being copied.
>>>> Note that there is also test/webapp-3.0-virtual-library/target.
>>>> I don't know, whether we can simply drop this exclude, or should rename the
directory. I CC'd Sylvain explicitely, maybe he can comment on it.
>>> If there are no side effect to dropping this exclude, you may do it (I really
don't know enough of the build system of tomcat).
>>> Otherwise I can rename those directories so that it is not excluded. Just let
me know.
>> If I recall correctly, those directories are excluded because some IDEs
>> use that name by default. If you could rename them that would be great.
> Not IDEs, but Maven. It writes its output into directories named
> "target", in each of its modules. (I guess that is not configurable,
> grr...)
> That is why "target" was present in .gitignore when it was first added,
> [1]
> This fault is a combination of mine
> [2]
> that applied patterns from gitignore to src distribution and of Sylvain's
> [3]
> when Sylvain removed that line from ".gitignore"
> and used that name in his commit.

Thanks for the full explanation. I didn't have time to look into that

In that case, I agree with your plan below and will do that shortly.


> I am generally in favor of using more targeted patterns in "dist-source" target,
> (That is, comment out "**/target/**" for now, with a comment that we
> use that name,
> and add the same comment to .gitignore
> and later exclude more specific patterns when a need comes)
> but consider if one of developers used Maven to work on something in modules.
> Currently modules/jdbc-pool and modules/tomcat-lite have pom.xml,
> though svn:ignore property on those directories does not list "target"
> as excluded.
> Best regards,
> Konstantin Kolinko
> ---------------------------------------------------------------------
> To unsubscribe, e-mail:
> For additional commands, e-mail:

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message