Mailing-List: contact dev-help@tomcat.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Tomcat Developers List" <dev@tomcat.apache.org>
Received-SPF: pass (nike.apache.org: domain of knst.kolinko@gmail.com
 designates 209.85.220.173 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <4F148C52.6070308@apache.org>
References: <4F136148.6050205@apache.org>
	<4F13F192.5020702@kippdata.de>
	<9586CA8A-5930-452C-B99D-793204B7E845@m4x.org>
	<4F148C52.6070308@apache.org>
Date: Tue, 17 Jan 2012 05:15:38 +0400
Message-ID: 
 <CABzHfV=CcZwVuvfUmKJ-ACWZ2cyVpM5S1ex9Q4TVb7qHfE19aw@mail.gmail.com>
Subject: Re: [VOTE] Release Apache Tomcat 7.0.24
From: Konstantin Kolinko <knst.kolinko@gmail.com>
To: Tomcat Developers List <dev@tomcat.apache.org>
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable

2012/1/17 Mark Thomas <markt@apache.org>:
> 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
>>> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
>>>
>>> I noticed that test/webapp-3.0-virtual-library/target is missing from t=
he src dist. So when rebuilding from src dist and running unit tests, I get=
 a failure in TestVirtualContext, because some resource which is usually re=
trieved form the target dir is missing:
>>>
>>> Testcase: testVirtualClassLoader took 7.434 sec
>>> =A0 =A0 =A0 =A0FAILED
>>> expected:<200> but was:<404>
>>> junit.framework.AssertionFailedError: expected:<200> but was:<404>
>>> =A0 =A0 =A0 =A0at org.apache.catalina.loader.TestVirtualContext.assertP=
ageContains(TestVirtualContext.java:302)
>>> =A0 =A0 =A0 =A0at org.apache.catalina.loader.TestVirtualContext.assertP=
ageContains(TestVirtualContext.java:294)
>>> =A0 =A0 =A0 =A0at org.apache.catalina.loader.TestVirtualContext.testVir=
tualClassLoader(TestVirtualContext.java:100)
>>>
>>> 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 r=
eally don't know enough of the build system of tomcat).
>> Otherwise I can rename those directories so that it is not excluded. Jus=
t 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] http://svn.apache.org/viewvc?view=3Drevision&revision=3D1187740

This fault is a combination of mine
[2] http://svn.apache.org/viewvc?view=3Drevision&revision=3D1204938

that applied patterns from gitignore to src distribution and of Sylvain's
[3] http://svn.apache.org/viewvc?view=3Drevision&revision=3D1209731

when Sylvain removed that line from ".gitignore"
and used that name in his commit.


I am generally in favor of using more targeted patterns in "dist-source" ta=
rget,
(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 module=
s.

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: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org