Hi Brett,
On Fri, 2004-09-24 at 06:05, Brett Porter wrote:
> I think it was David Jencks that attached a patch to MPIDEA-10 today for
> generated sources. I haven't looked at it yet, but if you want to check
> it out and see if it does the job for you, let me know. I'll probably
> apply it in the next few days once I've had a chance to look anyway.
Yeah looks like he's got a nice generic way for enabling generated
source directories to be included like so:
+ <j:forEach var="genSource"
items="${context.getVariable('maven.idea.generated.source')}">
+ <sourceFolder url="file://$$MODULE_DIR$$/target/${genSource}"
isTestSource="false"/>
+ </j:forEach>
I think it will work for me when specifing this list of generated idea
source directories. Also this patch fixes another annoying issue
altogether having to do with how the idea plugin presumes all maven
projects in the groupId are automatically incuded as modules rather than
as jar deps.
Overall this is an awesome patch. Thanks David!
Brett I'd love to see this applied whenever you get the chance.
<snip/>
Thanks,
Alex
> Alex Karasulu wrote:
>
> >Hiya,
> >
> >I've been playing with the idea plugin to see if I can get it to include
> >the antlr generated path. Here's the change I made by the way does not
> >work:
> >
> >(cvs diff inside maven-plugins/idea in trunk)
> >
> >Index: src/plugin-resources/templates/v4/module.jelly
> >===================================================================
> >RCS file:
> >/home/cvspublic/maven-plugins/idea/src/plugin-resources/templates/v4/module.jelly,v
> >retrieving revision 1.7
> >diff -r1.7 module.jelly
> >123a124,127
> >
> >
> >> <j:if test="${antlrSourcesPresent}">
> >> <maven:makeRelativePath var="value" basedir="${basedir}"
> >>
> >>
> >path="${maven.antlr.target.dir}" separator="/" />
> >
> >
> >> <sourceFolder url="file://$$MODULE_DIR$$/${value}"
> >>
> >>
> >isTestSource="true"/>
> >
> >
> >> </j:if>
> >>
> >>
> >
> >Now this does not work because I did not know if there was an equivalent
> >to sourcesPresent for antlr. I made up 'antlrSourcesPresent' above as a
> >place holder. Was wondering what I should use here instead or does no
> >equivalent property exist for antlr sources?
> >
> >Thanks,
> >Alex
> >
> >
> >
> >---------------------------------------------------------------------
> >To unsubscribe, e-mail: [EMAIL PROTECTED]
> >For additional commands, e-mail: [EMAIL PROTECTED]
> >
> >
> >
> >
> >
> >
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
>
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
