[jbosstools-dev] JBossTools (trunk): added src folder to fix source gathering related errors

[Rob Cernich]

I'm not sure but I believe Tycho may filter the output directory. And, given the way Eclipse build.properties work, you probably need to add bin/forge and bin/forge.bat to the list of includes, which means they will be in the bin dir in the resulting archive. 

Theoretically, you should not need to include .classpath or .project or .settings as m2eclipse w/tycho should take care of configuring the project correctly. If you want to get fancy, you could also configure a dependency:copy execution to pull in the relevant jars (e.g. jboss-modules.jar) and place them in the correct locations. This would save having jars checked into svn (drawback is that you have to run process-(re)sources, to copy down the jars, before the project will configure correctly). It would also allow you to more easily update the jars used (at the cost of potentially pulling down an unstable snapshot, if that's what you're using). If you'd like an example, you can look at the org.switchyard.tools.core plugin in the switchyard tools git repo (https://github.com/jboss-switchyard/tools/tree/master/eclipse/plugins/org.switchyard.tools.core). 

Best, 
Rob 

----- Original Message -----

> Digging further I noticed another potential problem. bin folder has
> two files that never gets into bundle. Do we need them?

> On 10/20/2011 04:53 PM, Denis Golovin wrote:
> > It looks like it is fine do not have src folder in tycho 0.13.0. No
> > failures, but no .source.jars as well.
> 

> > The right fix is
> 

> > add '.' into libraries list like this
> 

> > which means only build.properties should be changed like shown
> > below
> 

> > This fix forks for both tycho versions 0.12.0 and 0.13.0
> 

> > Denis
> 

> > On 10/20/2011 01:40 PM, Max Andersen wrote:
> 
> > > Is there not a way to conditionally enable the source packaging
> > > instead of these "fixes" ?
> > 
> 

> > > Denis Golovin committed 35852 on branch trunk in JBossTools
> > 
> 
> > > added src folder to fix source gathering related errors
> > 
> 
> > > 
> > 
> 
> > > · /trunk/forge/plugins/org.jboss.tools.forge.runtime/.classpath
> > > 	(-0,
> > > +1) 	| History 	| Source 	| Diff
> > 
> 
> > > · /trunk/forge/plugins/org.jboss.tools.forge.runtime/src 	(-0,
> > > +0)
> > > 	|
> > > History 	| Source
> > 
> 

> > > https://source.jboss.org/changelog/JBossTools?cs=35852
> > 
> 

> > > Sent with MobileRSS HD
> > 
> 

> > > Sent from a mobile device
> > 
> 

> > > _______________________________________________
> > 
> 
> > > jbosstools-dev mailing list jbosstools-dev at lists.jboss.org
> > > https://lists.jboss.org/mailman/listinfo/jbosstools-dev
> > 
> 
> > _______________________________________________
> 
> > jbosstools-dev mailing list jbosstools-dev at lists.jboss.org
> > https://lists.jboss.org/mailman/listinfo/jbosstools-dev
> 
> _______________________________________________
> jbosstools-dev mailing list
> jbosstools-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/jbosstools-dev
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/jbosstools-dev/attachments/20111020/9f9a560f/attachment-0001.html 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: screenshot132.png
Type: image/png
Size: 33072 bytes
Desc: not available
Url : http://lists.jboss.org/pipermail/jbosstools-dev/attachments/20111020/9f9a560f/attachment-0003.png 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: image/png
Size: 65851 bytes
Desc: not available
Url : http://lists.jboss.org/pipermail/jbosstools-dev/attachments/20111020/9f9a560f/attachment-0004.png 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: image/png
Size: 25638 bytes
Desc: not available
Url : http://lists.jboss.org/pipermail/jbosstools-dev/attachments/20111020/9f9a560f/attachment-0005.png