> On Feb 23, 2017, at 6:23 AM, James Bognar <james.bog...@salesforce.com> wrote:
> 
> Thanks Stian,
> 
> Bugs opened for the NOTICE file, port conflict issue, and failing
> testcase.  I thought we shook out all the timezone-related testcase
> failures on the last release, but I'll revisit.
> 
> For the copyright, isn't the normal convention "Copyright [first year],
> [last year]" with a comma?

Yes. NOTICE is correct for this year. Next year, 2016, 2018. Will not get 
verbose.

Craig
> 
> 
> 
> On Thu, Feb 23, 2017 at 6:35 AM, Stian Soiland-Reyes <st...@apache.org>
> wrote:
> 
>> On Wed, 22 Feb 2017 12:05:26 -0500, James Bognar <jamesbog...@apache.org>
>> wrote:
>>> The Apache Juneau Incubator PPMC has voted *+3* to release Apache Juneau
>>> 6.1.0-incubating RC2.
>>> The binaries are available at:
>>> https://dist.apache.org/repos/dist/dev/incubator/juneau/
>> binaries/juneau-6.1.0-incubating-RC2/
>>> 
>>> The release candidate to be voted over is available at:
>>> https://dist.apache.org/repos/dist/dev/incubator/juneau/
>> source/juneau-6.1.0-incubating-RC2/
>> 
>> Hi, apologies for not having a chance to look at this earlier.
>> 
>> My vote: +1 (binding)
>> 
>> Checked:
>> + pgp signatures
>> + sha1 checksums
>> + mvn apache-rat:check
>> + LICENSE, NOTICE, DISCLAIMER
>> - mvn clean install FAILS - conflict with Dropbox
>>  + works with -DskipTests
>> + No unexpected binaries in src (beyond PNG)
>> + maven repo matches binary distro (same sha1 on jars and zips)
>> 
>> NOTICE uses format "Copyright 2016, 2017" -- this will over time get
>> verbose
>> and I would generally think a time span is sufficient (e.g. next year it
>> can be
>> 2016-2018), but this is just a stylistic choice for the podling. :)
>> 
>> NOTICE says "Apache Juneau" instead of "Apache Juneau (incubating)"
>> (However there is also a DISCLAIMER)
>> 
>> 
>> Building on Windows fails with:
>> 
>> java.net.BindException: Address already in use: bind
>> 
>> Failed tests:
>>  DefaultContentTypesTest.testRestMethodParsersSerializers:119 Response
>> status code was not correct.  Expected: '415'.  Actual: '0'
>>  ErrorConditionsTest.test404and405:184 Response status code was not
>> correct.  Expected: '404'.  Actual: '0'
>>  ErrorConditionsTest.test412:215 Response status code was not correct.
>> Expected: '412'.  Actual: '0'
>>  ErrorConditionsTest.testNonExistentBeanProperties:51 Response status
>> code was not correct.  Expected: '400'.  Actual: '0'
>> (..)
>> Tests in error:
>>  AcceptCharsetTest.testCharsetOnResponse:72 » RestCall localhost:10001
>> failed t...
>>  AcceptCharsetTest.testQValues:36->check1:57 » RestCall localhost:10001
>> failed ...
>>  BeanContextPropertiesTest.testClassTransforms:32 » RestCall
>> localhost:10001 fa...
>> (..)
>> [INFO] Apache Juneau REST Tests ........................... FAILURE [04:35
>> min]
>> 
>> It turns out port 10001 is used by Dropbox:
>> 
>> λ netstat -b
>> 
>> Active Connections
>> 
>>  Proto  Local Address          Foreign Address        State
>>  TCP    127.0.0.1:10001        biggie:56458           TIME_WAIT
>>  TCP    127.0.0.1:10001        biggie:56459           TIME_WAIT
>>  TCP    127.0.0.1:50228        biggie:50229           ESTABLISHED
>> [Dropbox.exe]
>> 
>> If I stop Dropbox, then tcsd_win32.exe (related to Trusted Platform Module)
>> binds the port 10001 insted. This seems like a popular internal port!
>> 
>> Ideally the Juneau tests should try to bind the port dynamically rather
>> than be
>> hardcoded to 10001.  As this is a technical issue it does not affect my
>> IPMC
>> vote.
>> 
>> 
>> Stopping both daemons, the rest tests now fail with a single error:
>> 
>> BeanContextPropertiesTest.testClassTransforms:33
>> expected:<d1=2001-07-04T1[5:30:45Z,d2=2001-07-05T15:30:
>> 45Z,d3=2001-07-06T15:30:45Z]>
>> but was:<d1=2001-07-04T1[6:30:45+01:00,d2=2001-07-05T16:30:45+
>> 01:00,d3=2001-07-06T16:30:45+01:00]>
>> 
>> Timezone issue?  (btw, I'm in time zone London, which is Z/GMT during
>> winter
>> time, but 4th July would be during daylight saving +01)
>> 
>> 
>> 
>> 
>> Tested with:
>> 
>> D:\1\juneau-6.1.0-incubating
>> λ mvn -version
>> Apache Maven 3.3.9 (bb52d8502b132ec0a5a3f4c09453c07478323dc5;
>> 2015-11-10T16:41:47+00:00)
>> Maven home: C:\Program Files\apache-maven-3.3.9\bin\..
>> Java version: 1.8.0_121, vendor: Oracle Corporation
>> Java home: C:\Program Files\Java\jdk1.8.0_121\jre
>> Default locale: en_GB, platform encoding: Cp1252
>> OS name: "windows 10", version: "10.0", arch: "amd64", family: "dos"
>> 
>> --
>> Stian Soiland-Reyes
>> http://orcid.org/0000-0001-9842-9718
>> 
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
>> For additional commands, e-mail: general-h...@incubator.apache.org
>> 
>> 
> 
> 
> -- 
> James Bognar

Craig L Russell
c...@apache.org



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

Reply via email to