You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cxf.apache.org by Jason Pell <ja...@pellcorp.com> on 2012/11/03 09:55:13 UTC

Re: Remove DoubleCheckedLocking from build-utils/trunk/buildtools/src/main/resources/cxf-checkstyle.xml

Hi,

Updating the checkstyle plugin to latest in eclipse causes issues
because checkstyle 5.6 no longer supports DoubleCheckedLocking?  Was
wondering if we might remove this from the
cxf-checkstyle.xml file and up the build tools version for cxf 2.7.1?

I opened a jira some time back related to this and was hoping to fix
this to remove the eclipse import error.

https://issues.apache.org/jira/browse/CXF-4530

Can I get some feedback from you guys on the merits or otherwise of this?

Thanks
Jason

Re: Remove DoubleCheckedLocking from build-utils/trunk/buildtools/src/main/resources/cxf-checkstyle.xml

Posted by Jason Pell <ja...@pellcorp.com>.
If we do not updated it, we at least need to update the documentation
to make it clear
that installing the latest checkstyle version will not work with cxf
at the moment.

http://cxf.apache.org/setting-up-eclipse.html
http://cxf.apache.org/cxf-m2eclipse.html



On Sat, Nov 3, 2012 at 7:55 PM, Jason Pell <ja...@pellcorp.com> wrote:
> Hi,
>
> Updating the checkstyle plugin to latest in eclipse causes issues
> because checkstyle 5.6 no longer supports DoubleCheckedLocking?  Was
> wondering if we might remove this from the
> cxf-checkstyle.xml file and up the build tools version for cxf 2.7.1?
>
> I opened a jira some time back related to this and was hoping to fix
> this to remove the eclipse import error.
>
> https://issues.apache.org/jira/browse/CXF-4530
>
> Can I get some feedback from you guys on the merits or otherwise of this?
>
> Thanks
> Jason

Re: Remove DoubleCheckedLocking from build-utils/trunk/buildtools/src/main/resources/cxf-checkstyle.xml

Posted by Jason Pell <ja...@pellcorp.com>.
I see you beat me to it :-)

On Tue, Nov 6, 2012 at 2:39 AM, Daniel Kulp <dk...@apache.org> wrote:
>
> On Nov 3, 2012, at 4:55 AM, Jason Pell <ja...@pellcorp.com> wrote:
>
>> Hi,
>>
>> Updating the checkstyle plugin to latest in eclipse causes issues
>> because checkstyle 5.6 no longer supports DoubleCheckedLocking?  Was
>> wondering if we might remove this from the
>> cxf-checkstyle.xml file and up the build tools version for cxf 2.7.1?
>
> Sure.   Go for it.   :-)
>
> Dan
>
>
>
>> I opened a jira some time back related to this and was hoping to fix
>> this to remove the eclipse import error.
>>
>> https://issues.apache.org/jira/browse/CXF-4530
>>
>> Can I get some feedback from you guys on the merits or otherwise of this?
>>
>> Thanks
>> Jason
>
> --
> Daniel Kulp
> dkulp@apache.org - http://dankulp.com/blog
> Talend Community Coder - http://coders.talend.com
>

Re: Remove DoubleCheckedLocking from build-utils/trunk/buildtools/src/main/resources/cxf-checkstyle.xml

Posted by Daniel Kulp <dk...@apache.org>.
On Nov 3, 2012, at 4:55 AM, Jason Pell <ja...@pellcorp.com> wrote:

> Hi,
> 
> Updating the checkstyle plugin to latest in eclipse causes issues
> because checkstyle 5.6 no longer supports DoubleCheckedLocking?  Was
> wondering if we might remove this from the
> cxf-checkstyle.xml file and up the build tools version for cxf 2.7.1?

Sure.   Go for it.   :-)

Dan 



> I opened a jira some time back related to this and was hoping to fix
> this to remove the eclipse import error.
> 
> https://issues.apache.org/jira/browse/CXF-4530
> 
> Can I get some feedback from you guys on the merits or otherwise of this?
> 
> Thanks
> Jason

-- 
Daniel Kulp
dkulp@apache.org - http://dankulp.com/blog
Talend Community Coder - http://coders.talend.com