2015-01-03 19:32 GMT+01:00 sebb <seb...@gmail.com>: > On 3 January 2015 at 12:34, Benedikt Ritter <brit...@apache.org> wrote: > > Hello Sebb, > > > > 2015-01-03 4:35 GMT+01:00 sebb <seb...@gmail.com>: > > > >> I've made quite a bit of progress. > >> > >> However the IANA text file merges all the TLDs into one file - there > >> is no indication of whether the TLD is a country code or not. > >> This does not matter for ASCII codes, because they are easy to > recognise. > >> Unfortunately it looks as though the only way to distinguish them is > >> to scan the HTML version of the file and check the Type column. > >> Not sure it is possible to automate this without a lot of work, so it > >> may take a bit longer that I had hoped. > >> > > > > No problem. Is there anything we can do to help you? > > Turns out that the html format is not all that hard to parse, as it > appears to be auto generated. > I have now got it working so it creates separate sorted lists of > missing general and cc entries with comments. > > I will commit the missing entries shortly. > > There are still a couple of issues I think it would be good to resolve: > 1) root, um and yu are not in the txt list. I think they should be > dropped (VALIDATOR-350) >
Agreed > 2) lists are now already sorted, and there are unit tests to check > this, so I think we could drop the static{} block (VALIDATOR-349) > If they are sorted, yes we can drop the static block. > > WDYT? > > > Benedikt > > > > > >> > >> > >> On 2 January 2015 at 14:47, Benedikt Ritter <brit...@apache.org> wrote: > >> > Hi, > >> > > >> > this vote in canceled to fix the problems discovered by sebb. > >> > I will waint until the XN-- entries are added before I roll out a new > RC. > >> > > >> > Thanks! > >> > Benedikt > >> > > >> > 2015-01-01 19:02 GMT+01:00 Benedikt Ritter <brit...@apache.org>: > >> > > >> >> Hi all, > >> >> > >> >> we have fixed the issues which where found in RC1 so I'd like to > call a > >> >> new vote to release Apache Commons Validator based on RC2. > >> >> > >> >> Changes from RC1: > >> >> - [VALIDATOR-307] - isValid checks if the given address is only IPV4 > >> >> address and not IPV6 > >> >> - [VALIDATOR-347] - toLowerCase() method is Locale-sensitive and > should > >> >> not be used > >> >> - [VALIDATOR-348] - Update TLD list to latest version (Version > >> 2014123000) > >> >> - [VALIDATOR-336] - CUSIPCheckDigit thinks invalid CUSIP is valid. > >> >> - [VALIDATOR-345] - ISINCheckDigit fails to reject invalid > (non-numeric) > >> >> check digits > >> >> - [VALIDATOR-346] - SedolCheckDigit fails to reject invalid > >> (non-numeric) > >> >> check digits > >> >> - Removed STATUS.html > >> >> - Added README.md to binary and source distribution > >> >> - Fixed encoding of source files in build by setting commons.encoding > >> >> property > >> >> - Fixed JIRA report to contain the issues of the project > >> >> - Reverted dependency to commons-beanutils to 1.8.3 since this is the > >> >> latest JDK 1.4 compatible release > >> >> - Added JDK requirements to release notes. > >> >> > >> >> Validator 1.4.1-RC2 is available for review here: > >> >> https://dist.apache.org/repos/dist/dev/commons/validator/ (svn > >> revision > >> >> 7629) > >> >> > >> >> The tag is here: > >> >> > >> >> > >> > http://svn.apache.org/repos/asf/commons/proper/validator/tags/VALIDATOR_1_4_1_RC2/ > >> >> (svn revision 1648888) > >> >> > >> >> Maven artifacts are here: > >> >> > >> >> > >> > https://repository.apache.org/content/repositories/orgapachecommons-1073/commons-validator/commons-validator/1.4.1/ > >> >> > >> >> Details of changes since 1.4 are in the release notes: > >> >> > >> >> > >> > https://dist.apache.org/repos/dist/dev/commons/validator/RELEASE-NOTES.txt > >> >> > >> >> I have tested this with JDK 1.6, 1.7 and 1.8 using maven 3.2.5. > >> >> > >> >> Site (some links my be broken but will be fixed when the site is > >> deployed): > >> >> http://people.apache.org/~britter/validator-1.4.1-RC2/ > >> >> > >> >> Clirr Report: > >> >> > >> http://people.apache.org/~britter/validator-1.4.1-RC2/clirr-report.html > >> >> > >> >> RAT Report: > >> >> > http://people.apache.org/~britter/validator-1.4.1-RC2/rat-report.html > >> >> > >> >> Keys: > >> >> https://www.apache.org/dist/commons/KEYS > >> >> > >> >> Please review this release candidate and vote. > >> >> This vote will close no sooner than 72 hours from now, i.e. after > >> >> 2015/01/04 19:00CET > >> >> > >> >> [ ] +1 Release these artifacts > >> >> [ ] +0 OK, but... > >> >> [ ] -0 OK, but really should fix... > >> >> [ ] -1 I oppose this release because... > >> >> > >> >> Thanks! > >> >> Benedikt > >> >> > >> >> > >> >> -- > >> >> http://people.apache.org/~britter/ > >> >> http://www.systemoutprintln.de/ > >> >> http://twitter.com/BenediktRitter > >> >> http://github.com/britter > >> >> > >> > > >> > > >> > > >> > -- > >> > 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 > > -- http://people.apache.org/~britter/ http://www.systemoutprintln.de/ http://twitter.com/BenediktRitter http://github.com/britter