On Wed, 2016-03-16 at 17:01 -0700, Andreas Schaefer Sr. wrote: > There is another issue I encountered. The RAT plugin is complaining > about IntelliJ Module files (*.iml) files. > > I did include the exclude for **/*.iml but that is only local. I will > include the change to the Parent POM inside the latest project but > there are other POM files that do not have the exclude. > > I discussed this with Robert a while ago but the current Sling will > not build for anyone using IntelliJ to work with Sling.
Where exactly does this fail? Thanks, Robert > > Cheers - Andy > > > > > On Mar 16, 2016, at 2:46 PM, Andreas Schaefer Sr. <schaef...@me.com > > > wrote: > > > > Ok, that did the trick. > > > > If I get a setup working I will debug it as I had a similar issue > > (JCR Calendar issues based on timezones) in the past. I also will > > see if I can reproduce it consistently. > > > > But at least I am now good to test my merge for a pull request. > > > > Thanks - Andy > > > > > > > > On Mar 16, 2016, at 12:43 AM, Bertrand Delacretaz <bdelacretaz@ap > > > ache.org> wrote: > > > > > > On Tue, Mar 15, 2016 at 9:33 PM, Andreas Schaefer Sr. <schaefera@ > > > me.com> wrote: > > > > > > > > ...This line is causing the failure: > > > > Assert.assertEquals(expected.getTimeZone().getRawOffset(), > > > > actual.getTimeZone().getRawOffset()); ... > > > Makes sense, good catch! > > > > > > I have changed the Calendar comparison method in > > > https://issues.apache.org/jira/browse/SLING-5612, does that fix > > > your > > > issue? > > > > > > Also, if you find a way to reproduce it on any system (with > > > -Duser.timezone maybe?) please add that info there, I haven't > > > been > > > able to reproduce it myself. > > > > > > -Bertrand