commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From sebb <>
Subject Re: [all] Preparing for commons-parent pom release
Date Wed, 24 Feb 2010 16:40:35 GMT
On 24/02/2010, Niall Pemberton <> wrote:
> On Wed, Feb 24, 2010 at 1:07 AM, sebb <> wrote:
>  > On 24/02/2010, Niall Pemberton <> wrote:
>  >> I'd like to do a release of the commons-parent pom - primarily to
>  >>  upgrade to the latest commons-build-plugin 1.2 release.
>  >>
>  >>  I have also upgraded the plugin versions and changed the "rc" &
>  >>  "release" profiles to now only produce the javadocs and not the whole
>  >>  site (this resolves a problem for multi-module components).
>  >>
>  >>  Are there any other changes or feedback before calling a vote on this?
>  >
>  > I think the default maven.compile.source|target entries should be
>  > removed from the POM.
>  >
>  > Seems to me that projects should have to define these, and not rely on
>  > the default.
> I disagree with this because the whole point of the commons-parent
>  pom.xml is to reduce the amount of dulicate configuration in component
>  projects and I see no benefit in forcing components to define
>  unnecessary properties when the default suits them.

But the compilation source and target are specific to each project,
just like inceptionYear.

Unlike other properties, such as plugin versions, the source and
target versions can never be updated in the parent pom. If it is
updated, that all projects that are currently relying on the default
will have to be updated to override the parent pom. At which point the
parent pom properties become useless anyway.

It's also tedious trying to find the JVM requirements if one has to
look in both the project pom and the parent pom.

>  I also think this is a non-issue. We have a good history over the past
>  few years of making sure components are compatible with the JDK
>  version they target and I don't believe theres a single example of a
>  release that had these properties incorrectly set.
>  Also we did have this discussion before:

I know, and I still disagree.

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

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

View raw message