You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@maven.apache.org by "Mark H. Wilkinson" <mh...@kremvax.net> on 2003/08/22 15:36:23 UTC

Re: maven eclipse plug-in should support eclipse checkstyle plug-in

On Fri, 2003-08-22 at 11:57, jira@codehaus.org wrote:
> The following comment has been added to this issue:
> 
>      Author: Emmanuel Venisse
>     Created: Fri, 22 Aug 2003 5:56 AM
>        Body:
> I'm -1 for this modification.
> 
> a) it exists more than one checkstyle plugin for eclipse
>  - http://www.mvmsoft.de/content/plugins/checkclipse/checkclipse.htm
>  - http://eclipse-cs.sourceforge.net/
> b) we can't integrate all eclipse plugin fetures in eclipse project file.

What would you suggest as an alternative solution to this problem? The
current situation doesn't work because running 'maven eclipse' to add
new dependencies to the project classpath overwrites any other additions
to the .project and .classpath files made through the Eclipse user
interface.

-Mark.


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org


Re: maven eclipse plug-in should support eclipse checkstyle plug-in

Posted by Emmanuel Venisse <ev...@ifrance.com>.
I don't have time to investigate on your plugin modification, but your
addBuilder/addNature taglibs seems to be interesting.

----- Original Message ----- 
From: "Gilles Dodinet" <rh...@wanadoo.fr>
To: "Maven Developers List" <de...@maven.apache.org>
Sent: Saturday, August 23, 2003 2:52 PM
Subject: Re: maven eclipse plug-in should support eclipse checkstyle plug-in


> I think that the files i provided for MAVEN-525 do exactly what you
> want. it defines a taglib which declares among others a tag that adds
> project nature and project builder to .project using jelly:xml. perhpas
> it may help.
>
> Ben Walding wrote:
>
> > A better way might be for plugins to register themselves into the
> > eclipse plugin and be passed a DOM of the .classpath / .project. They
> > then manipulate this as required.
> >
> > Similar to how the report plugin works.
> >
> > Mark H. Wilkinson wrote:
> >
> >> On Fri, 2003-08-22 at 11:57, jira@codehaus.org wrote:
> >>
> >>
> >>> The following comment has been added to this issue:
> >>>
> >>>     Author: Emmanuel Venisse
> >>>    Created: Fri, 22 Aug 2003 5:56 AM
> >>>       Body:
> >>> I'm -1 for this modification.
> >>>
> >>> a) it exists more than one checkstyle plugin for eclipse
> >>> - http://www.mvmsoft.de/content/plugins/checkclipse/checkclipse.htm
> >>> - http://eclipse-cs.sourceforge.net/
> >>> b) we can't integrate all eclipse plugin fetures in eclipse project
> >>> file.
> >>>
> >>
> >>
> >> What would you suggest as an alternative solution to this problem? The
> >> current situation doesn't work because running 'maven eclipse' to add
> >> new dependencies to the project classpath overwrites any other
additions
> >> to the .project and .classpath files made through the Eclipse user
> >> interface.
> >>
> >> -Mark.
> >>
> >>
> >> ---------------------------------------------------------------------
> >> To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
> >> For additional commands, e-mail: dev-help@maven.apache.org
> >>
> >>
> >>
> >
> >
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
> > For additional commands, e-mail: dev-help@maven.apache.org
> >
> >
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
> For additional commands, e-mail: dev-help@maven.apache.org
>
> _____________________________________________________________________
> Envie de discuter en "live" avec vos amis ? Télécharger MSN Messenger
> http://www.ifrance.com/_reloc/m la 1ère messagerie instantanée de France

_____________________________________________________________________
Envie de discuter en "live" avec vos amis ? Télécharger MSN Messenger
http://www.ifrance.com/_reloc/m la 1ère messagerie instantanée de France


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org


Re: maven eclipse plug-in should support eclipse checkstyle plug-in

Posted by Gilles Dodinet <rh...@wanadoo.fr>.
I think that the files i provided for MAVEN-525 do exactly what you 
want. it defines a taglib which declares among others a tag that adds 
project nature and project builder to .project using jelly:xml. perhpas 
it may help.

Ben Walding wrote:

> A better way might be for plugins to register themselves into the 
> eclipse plugin and be passed a DOM of the .classpath / .project. They 
> then manipulate this as required.
>
> Similar to how the report plugin works.
>
> Mark H. Wilkinson wrote:
>
>> On Fri, 2003-08-22 at 11:57, jira@codehaus.org wrote:
>>  
>>
>>> The following comment has been added to this issue:
>>>
>>>     Author: Emmanuel Venisse
>>>    Created: Fri, 22 Aug 2003 5:56 AM
>>>       Body:
>>> I'm -1 for this modification.
>>>
>>> a) it exists more than one checkstyle plugin for eclipse
>>> - http://www.mvmsoft.de/content/plugins/checkclipse/checkclipse.htm
>>> - http://eclipse-cs.sourceforge.net/
>>> b) we can't integrate all eclipse plugin fetures in eclipse project 
>>> file.
>>>   
>>
>>
>> What would you suggest as an alternative solution to this problem? The
>> current situation doesn't work because running 'maven eclipse' to add
>> new dependencies to the project classpath overwrites any other additions
>> to the .project and .classpath files made through the Eclipse user
>> interface.
>>
>> -Mark.
>>
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
>> For additional commands, e-mail: dev-help@maven.apache.org
>>
>>  
>>
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
> For additional commands, e-mail: dev-help@maven.apache.org
>
>



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org


Re: maven eclipse plug-in should support eclipse checkstyle plug-in

Posted by Emmanuel Venisse <ev...@ifrance.com>.
Yes, it's a better way.

----- Original Message ----- 
From: "Ben Walding" <be...@walding.com>
To: "Maven Developers List" <de...@maven.apache.org>
Sent: Saturday, August 23, 2003 11:43 AM
Subject: Re: maven eclipse plug-in should support eclipse checkstyle plug-in


> A better way might be for plugins to register themselves into the
> eclipse plugin and be passed a DOM of the .classpath / .project. They
> then manipulate this as required.
>
> Similar to how the report plugin works.
>
> Mark H. Wilkinson wrote:
>
> >On Fri, 2003-08-22 at 11:57, jira@codehaus.org wrote:
> >
> >
> >>The following comment has been added to this issue:
> >>
> >>     Author: Emmanuel Venisse
> >>    Created: Fri, 22 Aug 2003 5:56 AM
> >>       Body:
> >>I'm -1 for this modification.
> >>
> >>a) it exists more than one checkstyle plugin for eclipse
> >> - http://www.mvmsoft.de/content/plugins/checkclipse/checkclipse.htm
> >> - http://eclipse-cs.sourceforge.net/
> >>b) we can't integrate all eclipse plugin fetures in eclipse project
file.
> >>
> >>
> >
> >What would you suggest as an alternative solution to this problem? The
> >current situation doesn't work because running 'maven eclipse' to add
> >new dependencies to the project classpath overwrites any other additions
> >to the .project and .classpath files made through the Eclipse user
> >interface.
> >
> >-Mark.
> >
> >
> >---------------------------------------------------------------------
> >To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
> >For additional commands, e-mail: dev-help@maven.apache.org
> >
> >
> >
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
> For additional commands, e-mail: dev-help@maven.apache.org
>
> _____________________________________________________________________
> Envie de discuter en "live" avec vos amis ? Télécharger MSN Messenger
> http://www.ifrance.com/_reloc/m la 1ère messagerie instantanée de France

_____________________________________________________________________
Envie de discuter en "live" avec vos amis ? Télécharger MSN Messenger
http://www.ifrance.com/_reloc/m la 1ère messagerie instantanée de France


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org


Re: maven eclipse plug-in should support eclipse checkstyle plug-in

Posted by Martin Skopp <sk...@riege.de>.
On Sat, 2003-08-23 at 11:43, Ben Walding wrote:
> A better way might be for plugins to register themselves into the 
> eclipse plugin and be passed a DOM of the .classpath / .project. They 
> then manipulate this as required.

+0
-- 
Martin Skopp
Riege Software International GmbH
Support: mailto:maint@riege.com, Information: http://www.riege.com
 
This email is intended to be viewed with a nonproportional font.
Public Key on http://www.keyserver.net, Key-ID: 3D4027B5
Fingerprint: 1970 C78D 9A1D 99FA 5CE4  5C0D 29E6 6A95 3D40 27B5



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org


Re: maven eclipse plug-in should support eclipse checkstyle plug-in

Posted by Ben Walding <be...@walding.com>.
A better way might be for plugins to register themselves into the 
eclipse plugin and be passed a DOM of the .classpath / .project. They 
then manipulate this as required.

Similar to how the report plugin works.

Mark H. Wilkinson wrote:

>On Fri, 2003-08-22 at 11:57, jira@codehaus.org wrote:
>  
>
>>The following comment has been added to this issue:
>>
>>     Author: Emmanuel Venisse
>>    Created: Fri, 22 Aug 2003 5:56 AM
>>       Body:
>>I'm -1 for this modification.
>>
>>a) it exists more than one checkstyle plugin for eclipse
>> - http://www.mvmsoft.de/content/plugins/checkclipse/checkclipse.htm
>> - http://eclipse-cs.sourceforge.net/
>>b) we can't integrate all eclipse plugin fetures in eclipse project file.
>>    
>>
>
>What would you suggest as an alternative solution to this problem? The
>current situation doesn't work because running 'maven eclipse' to add
>new dependencies to the project classpath overwrites any other additions
>to the .project and .classpath files made through the Eclipse user
>interface.
>
>-Mark.
>
>
>---------------------------------------------------------------------
>To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
>For additional commands, e-mail: dev-help@maven.apache.org
>
>  
>



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org


RE: maven eclipse plug-in should support eclipse checkstyle plug-in

Posted by Eric Pugh <ep...@upstate.com>.
Is the solution then to change the plugin to respect modifications made by
users?  Maybe have an eclipse target that respects those modifications, and
an eclipse:clean that rebuilds it from scratch?

ERic Pugh

> -----Original Message-----
> From: Mark H. Wilkinson [mailto:mhw-maven-dev@kremvax.net]
> Sent: Friday, August 22, 2003 3:36 PM
> To: Maven Developers List
> Subject: Re: maven eclipse plug-in should support eclipse checkstyle
> plug-in
>
>
> On Fri, 2003-08-22 at 11:57, jira@codehaus.org wrote:
> > The following comment has been added to this issue:
> >
> >      Author: Emmanuel Venisse
> >     Created: Fri, 22 Aug 2003 5:56 AM
> >        Body:
> > I'm -1 for this modification.
> >
> > a) it exists more than one checkstyle plugin for eclipse
> >  - http://www.mvmsoft.de/content/plugins/checkclipse/checkclipse.htm
> >  - http://eclipse-cs.sourceforge.net/
> > b) we can't integrate all eclipse plugin fetures in eclipse
> project file.
>
> What would you suggest as an alternative solution to this problem? The
> current situation doesn't work because running 'maven eclipse' to add
> new dependencies to the project classpath overwrites any
> other additions
> to the .project and .classpath files made through the Eclipse user
> interface.
>
> -Mark.
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
> For additional commands, e-mail: dev-help@maven.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org