Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
[wtp-dev] Understanding Runtimes and Multiple facets

Hi,

I am having a bit of a rough time understand how multiple facets and runtimes work.

I wanted to have a project with three facets:

jst.java
jst.jbi.bindingcomponent <- New JBI support
jst.maven <- Support for Apache Maven via facet

I have runtimes configured to support jst.jbi.bindingcomponent .. However,  they never get enabled,  it seems the filters on the runtimes exclude a runtime because any of the facets aren't supported? Does this mean that a runtime needs to support all the facets of a project to be used with it or was the intention that if your runtime supports one of the facets then you use it?


For an example of what I mean I patched the following to allow a runtime to be added in the properties?


### Eclipse Workspace Patch 1.0
#P org.eclipse.wst.common.project.facet.ui
Index: src/org/eclipse/wst/common/project/facet/ui/internal/RuntimesPropertyPage.java
===================================================================
RCS file: /cvsroot/webtools/wst/components/common/plugins/org.eclipse.wst.common.project.facet.ui/src/org/eclipse/wst/common/project/facet/ui/internal/RuntimesPropertyPage.java,v
retrieving revision 1.2
diff -u -r1.2 RuntimesPropertyPage.java
--- src/org/eclipse/wst/common/project/facet/ui/internal/RuntimesPropertyPage.java    16 May 2006 18:33:29 -0000    1.2
+++ src/org/eclipse/wst/common/project/facet/ui/internal/RuntimesPropertyPage.java    26 Jun 2006 18:25:55 -0000
@@ -98,13 +98,13 @@
                             final IProjectFacetVersion fv
                                 = (IProjectFacetVersion) itr.next();
                            
-                            if( ! runtime.supports( fv ) )
+                            if( runtime.supports( fv ) )
                             {
-                                return false;
+                                return true;
                             }
                         }
                        
-                        return true;
+                        return false;
                     }
                 }
             );


Cheers

P

Back to the top