On 5 January 2015 at 14:33, Benedikt Ritter <brit...@apache.org> wrote:
> 2015-01-05 15:09 GMT+01:00 sebb <seb...@gmail.com>:
>
>> On 5 January 2015 at 07:03, Benedikt Ritter <brit...@apache.org> wrote:
>> > Hi all,
>> >
>> > there have been some fixes in the last few days (mostly thanks to sebb)
>> and
>> > my feeling is, that we're ready to cut 1.4.1 RC3.
>>
>> Yes, I think it is in a stable state.
>> I have been checking the source with Java 1.4 on a Windows VM and it
>> passes.
>>
>> > If there is something left you would like to add, then please let me
>> know.
>>
>> I'm still wondering about VALIDATOR-235 - allowing IDN names.
>>
>> I've just updated the issue with two possible approaches.
>>
>> Assuming we want to do it at all, I favour the reflection approach, as
>> it is likely the one we would take if the code needed 1.6.
>> When the minimum of Java 1.6 is reached, the reflection code can
>> simply be removed.
>> If the code checks for non-ASCII content first, the reflection code
>> won't be used unnecessarily.
>>
>
> Why not update to Java 1.6 after 1.4.1? Do you rather want to target Java
> 1.5?

No, 1.6 would be fine. We need the IDN class.

> Really, I wouldn't put the effort in something that will be removed after
> this release (meaning the reflection thingy)...

I've already done most of the code as part of DomainValidatorTest.
It just needs to be repackaged slightly.
Also need a unit test, but that will be needed later anyway.

Implementing this would allow the current release to work under 1.6+
for Unicode domains right now - no need for users to wait for the next
release.
So IMO I think it's worth a bit of extra work now.

>
>>
>> I'll look into this shortly, but wait for confirmation before committing.
>>
>> > Otherwise I'll cut RC3 tonight or tomorrow.
>>
>> Thanks!
>>
>> > Regards,
>> > Benedikt
>> >
>> > --
>> > http://people.apache.org/~britter/
>> > http://www.systemoutprintln.de/
>> > http://twitter.com/BenediktRitter
>> > http://github.com/britter
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
>> For additional commands, e-mail: dev-h...@commons.apache.org
>>
>>
>
>
> --
> http://people.apache.org/~britter/
> http://www.systemoutprintln.de/
> http://twitter.com/BenediktRitter
> http://github.com/britter

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

Reply via email to