incubator-esme-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Vassil Dichev <vdic...@apache.org>
Subject Re: [VOTE] Approve the release of apache-esme-incubating-1.0 - (Yes Again :->)
Date Tue, 23 Feb 2010 08:06:23 GMT
We should by all means do String.trim on input values- from properties
or otherwise.


On Tue, Feb 23, 2010 at 9:36 AM, Sig Rinde <sig@rinde.com> wrote:
> Ethan / Dick
>
> Noticed that the default.props file that was included in today's RC
> indeed had much trailing characters (spaces) so even if I used vi to
> add the role.api_test line to it I had to clean it up first.
>
> Unless you're aware of the issue that I stumbled into last you'll walk
> into the trap. Space characters are not easy to see unless you poke
> around looking for them...
>
> Suggest you clean up that file so it's free of that easy-to-fall-into-trap.
>
> Sig
>
> On 22 February 2010 14:37, Ethan Jewett <esjewett@gmail.com> wrote:
>> Whew, that's a relief :-)
>>
>> On Mon, Feb 22, 2010 at 1:10 AM, Sig Rinde <sig@rinde.com> wrote:
>>> Ethan,
>>>
>>> you were right! So much easier to find errors when one knows where to look :)
>>>
>>> A single tailing space character after the rol.api_test line, sneaky
>>> little invisible character...
>>>
>>> My fault, used copy past using textmate. Next time vi only...
>

Mime
View raw message