commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From sebb <>
Subject Re: [MATH] location for localization resource files
Date Sun, 04 Sep 2011 23:52:21 GMT
On 4 September 2011 18:56, Phil Steitz <> wrote:
> On 9/4/11 10:45 AM, Phil Steitz wrote:
>> On 9/4/11 5:15 AM, sebb wrote:
>>> The location for the localization resources only appears to be needed
>>> in the method:
>>> LocalizedFormats.getLocalizedString()
>>> so it is very easy to change where they are, now that the POM uses the
>>> standard Maven resource location (previously it was using a subdir
>>> only which prevented such a move)
>>> As discussed else-thread we can now move the resources out of META-INF.
>> I am not sure I understand.  These resources are needed at runtime
>> by applications using [math].  I guess you are suggesting that we
>> can put them somewhere else in the source tree and have the build
>> package them there.  Is that what you mean?  In that case, we need
>> to make sure to adjust the ant build as well.  What is wrong with
>> the current setup, exactly?
> Sorry, I read the second message first.  I don't care much, other
> than to make sure the Ant build does not get broken by the change.

I will check that.

> Personally, I prefer to err on the side of more stuff in the
> component's own pom, since it makes these kinds of problems easier
> to diagnose and requires less background knowledge to understand the
> build.

The localization entry in the Math pom was only necessary because the
patch to commons parent (v3) to add N&L to jars failed to take the
default resources directory into account. That was a mistake (*). Had
we used a different method to handle the N&L files the pom probably
would not have needed to specify the resources.

Using the standard Maven directory structure seems to me to be better
than customising the poms for projects that use resources.

(*) When commons parent 21 was updated to add N&L files for test jars,
the same technique was copied; the issue with the MATH test resources
has just revealed that we made a mistake in commons parent 3 and
propagated it in 21.

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

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

View raw message