Mailing-List: contact dev-help@commons.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Commons Developers List" <dev@commons.apache.org>
Received-SPF: pass (athena.apache.org: domain of sebbaz@gmail.com designates
 209.85.220.171 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <47633F36-A558-49F7-A654-2BBEC0B2E052@dslextreme.com>
References: <2698EA53-8175-4BCA-A99C-C57D6526D2B1@dslextreme.com>
	<CAOGo0VZfo6=CuAGFe-BDA7FtCsAh0NSER36tzzuxTrp61dgbxQ@mail.gmail.com>
	<73B0EDE0-08DB-42B7-8121-488895D2F798@dslextreme.com>
	<CAOGo0VZYycUb1_Q8k2qGxJyZmw+yxQ9nq44vW5vE=DtG=F3F=g@mail.gmail.com>
	<47633F36-A558-49F7-A654-2BBEC0B2E052@dslextreme.com>
Date: Sun, 14 Aug 2011 19:20:21 +0100
Message-ID: 
 <CAOGo0VaXV0_nx9bkUh1uafycwerEY=axvtSUFONfu8FVQqrvjw@mail.gmail.com>
Subject: Re: [Vote] Release Commons VFS 2.0
From: sebb <sebbaz@gmail.com>
To: Commons Developers List <dev@commons.apache.org>
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable

On 14 August 2011 18:49, Ralph Goers <ralph.goers@dslextreme.com> wrote:
>
> On Aug 14, 2011, at 10:09 AM, sebb wrote:
>
>> On 14 August 2011 18:03, Ralph Goers <ralph.goers@dslextreme.com> wrote:
>>> Thanks, Sebb. See below.
>>>
>>> On Aug 14, 2011, at 9:50 AM, sebb wrote:
>>>
>>>> On 14 August 2011 16:25, Ralph Goers <ralph.goers@dslextreme.com> wrot=
e:
>>>>> This is a vote to release Apache Commons VFS 2.0.
>>>>>
>>>>> Since the last candidate the package name was changed from vfs to vfs=
2. Many of the Jira issues were reviewed and those that required a possibly=
 incompatible API change were addressed. Most instances of StringBuffer wer=
e replaced with StringBuilder. Some synchronization issues were addressed. =
Many javadoc and some checkstyle issues were addressed. The filesystem docu=
mentation was improved to list file system capabilities.
>>>>>
>>>>> [ ] +1 release it
>>>>> [ ] +0 go ahead I don't care
>>>>> [X] -1 no, do not release it because...
>>>>
>>>> Notice file still shows 2010.
>>>>
>>>> It also refers to the (optional) Javamail library.
>>>> Since that is not actually included in any of the archives, it should
>>>> not be mentioned.
>>>>
>>>> [If it were necessary to include a mention of JavaMail in NOTICE, then
>>>> it would be necessary to include the corresponding license in LICENSE]
>>>>
>>>> =3D=3D
>>>>
>>>> Not a blocker, but a pain for checking the release:
>>>> - lots of source files contain @version $Revision $. This uses local
>>>> time, which means my checkout of SVN does not match the source
>>>> archives.
>>>>
>>>> There are some EOL issues:
>>>>
>>>> svn ps svn:eol-style native README.txt
>>>> svn ps svn:eol-style native RELEASE-NOTES.txt
>>>> svn ps svn:eol-style native osgi/MANIFEST.MF
>>>> svn ps svn:eol-style native src/changes/announcement.vm
>>>>
>>>> I'm not convinced that the code really needs to be
>>>> binary-incompatible, but that is a separate discussion.
>>>>
>>>> However, the release notes ought to mention code is not source
>>>> compatible with VFS 1.x and that the package name has changed.
>>>>
>>>> mvn install produces some warnings:
>>>>
>>>> [WARNING] Ignoring project type pom - supportedProjectTypes =3D [jar, =
bundle]
>>>> [WARNING] DEPRECATED [tasks]: Use target instead
>>>
>>> I can't fix the above two lines. They are caused by changes in commons-=
parent version 21.
>>
>> Huh?
>>
>> The second warning is caused by using <tasks> instead of <target>; you
>> can fix that easily.
>
> You are right. That was the antrun plugin.
>
>>
>> Not sure about the first, but I would be suprised if it is anything to
>> do with Commons Parent (except that later plugins may be more picky
>> about obsolete syntax).
>
> Commons Parent added the maven-bundle plugin.

As far as I can tell, that was added in version 8.
It was certainly earlier than version 21.
Experimentation with CP versions shows that the warning message starts
with version 13, which coincides with updating from Apache pom v4 to
v7.

> The Warning you are seeing there is because it applies to all projects in=
 a multi-project build, some of which may be declared as a type other than =
bundle or jar. =A0It also added the jar plugin to generate the test jar. Th=
is required me to create a bogus MANIFEST.MF in the parent project since th=
e maven-bundle plugin doesn't generate a manifest.

No idea how to solve that; perhaps Dennis L can help?

> Ralph
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
>
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org