Hi, a question that may be is stupid, but why when I install the CDIFacet forge does a createNewFile ?? This, in case the facet was yet there,a launch a RuntimeException. Is done to prevent overwriting of custom beans.xml ?? Thank you. -- **************************************** http://www.lucamasini.net http://twitter.com/lmasini http://www.linkedin.com/pub/luca-masini/7/10/2b9 **************************************** _______________________________________________ forge-dev mailing list forge-dev(a)lists.jboss.org https://lists.jboss.org/mailman/listinfo/forge-dev

Hey Luca, could you also give us the steps to reproduce? Thanks! On Sun, Nov 18, 2012 at 8:25 AM, George Gastaldi <ggastald(a)redhat.com&gt;wrote: > Sounds like a bug to me. Can you reproduce it with a test case ? We'll > check it asap. > > Thanks ! > > George Gastaldi > > Em 18/11/2012, às 08:30, Luca Masini <luca.masini(a)gmail.com&gt; escreveu: > > Hi, a question that may be is stupid, but why when I install the CDIFacet > forge does a createNewFile ?? > > This, in case the facet was yet there,a launch a RuntimeException. > > Is done to prevent overwriting of custom beans.xml ?? > > Thank you. > > -- > **************************************** > http://www.lucamasini.net > http://twitter.com/lmasini > http://www.linkedin.com/pub/luca-masini/7/10/2b9 > **************************************** > > _______________________________________________ > forge-dev mailing list > forge-dev(a)lists.jboss.org > https://lists.jboss.org/mailman/listinfo/forge-dev > > > _______________________________________________ > forge-dev mailing list > forge-dev(a)lists.jboss.org > https://lists.jboss.org/mailman/listinfo/forge-dev > > -- Lincoln Baxter, III http://ocpsoft.org "Simpler is better." _______________________________________________ forge-dev mailing list forge-dev(a)lists.jboss.org https://lists.jboss.org/mailman/listinfo/forge-dev

Yes, that looks right. What is the underlying exception from the file creation error? That's a bit strange. On Sun, Nov 18, 2012 at 3:49 PM, Luca Masini < luca.masini.mailing.list(a)gmail.com&gt; wrote: > Well, I'm trying to help developing the forge-errai plugin, and doing the > scaffolding I added this: > > @Alias("errai-ui") > @Help("Errai scaffolding") > @RequiresFacet({ WebResourceFacet.class, > DependencyFacet.class, > PersistenceFacet.class, > EJBFacet.class, > CDIFacet.class }) > > And when doing the install the CDIFacet is installed I got this: > > java.lang.RuntimeException: Failed to create required > [/Users/lucamasini/Documents/workspaceJBossTools/forge-2/src/main/webapp/WEB-INF/beans.xml] > at > org.jboss.forge.spec.javaee.cdi.CDIFacetImpl.install(CDIFacetImpl.java:63) > at > org.jboss.forge.project.BaseProject.performInstallation(BaseProject.java:240) > at org.jboss.forge.project.BaseProject.installFacet(BaseProject.java:166) > at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) > at > sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39) > at > sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25) > at java.lang.reflect.Method.invoke(Method.java:597) > at > org.jboss.weld.bean.proxy.AbstractBeanInstance.invoke(AbstractBeanInstance.java:48) > at > org.jboss.weld.bean.proxy.ProxyMethodHandler.invoke(ProxyMethodHandler.java:125) > at > org.jboss.weld.proxies.Project$-1946286523$Proxy$_$$_WeldClientProxy.installFacet(Project$-1946286523$Proxy$_$$_WeldClientProxy.java) > at > org.jboss.forge.shell.project.FacetInstallationHandler.performInstallation(FacetInstallationHandler.java:105) > at > org.jboss.forge.shell.project.FacetInstallationHandler.installDependencies(FacetInstallationHandler.java:159) > at > org.jboss.forge.shell.project.FacetInstallationHandler.performInstallation(FacetInstallationHandler.java:100) > at > org.jboss.forge.shell.project.FacetInstallationHandler.install(FacetInstallationHandler.java:81) > at > org.jboss.forge.shell.project.FacetInstallationHandler.installRequest(FacetInstallationHandler.java:59) > at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) > at > sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39) > at > sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25) > ... > ... > ... > > Am I using it in the right way ? > > > 2012/11/18 Lincoln Baxter, III <lincolnbaxter(a)gmail.com&gt; > > Hey Luca, could you also give us the steps to reproduce? Thanks! >> >> >> On Sun, Nov 18, 2012 at 8:25 AM, George Gastaldi <ggastald(a)redhat.com&gt;wrote: >> >>> Sounds like a bug to me. Can you reproduce it with a test case ? We'll >>> check it asap. >>> >>> Thanks ! >>> >>> George Gastaldi >>> >>> Em 18/11/2012, às 08:30, Luca Masini <luca.masini(a)gmail.com&gt; escreveu: >>> >>> Hi, a question that may be is stupid, but why when I install the >>> CDIFacet forge does a createNewFile ?? >>> >>> This, in case the facet was yet there,a launch a RuntimeException. >>> >>> Is done to prevent overwriting of custom beans.xml ?? >>> >>> Thank you. >>> >>> -- >>> **************************************** >>> http://www.lucamasini.net >>> http://twitter.com/lmasini >>> http://www.linkedin.com/pub/luca-masini/7/10/2b9 >>> **************************************** >>> >>> _______________________________________________ >>> forge-dev mailing list >>> forge-dev(a)lists.jboss.org >>> https://lists.jboss.org/mailman/listinfo/forge-dev >>> >>> >>> _______________________________________________ >>> forge-dev mailing list >>> forge-dev(a)lists.jboss.org >>> https://lists.jboss.org/mailman/listinfo/forge-dev >>> >>> >> >> >> -- >> Lincoln Baxter, III >> http://ocpsoft.org >> "Simpler is better." >> >> _______________________________________________ >> forge-dev mailing list >> forge-dev(a)lists.jboss.org >> https://lists.jboss.org/mailman/listinfo/forge-dev >> >> > > > -- > **************************************** > http://www.lucamasini.net > http://twitter.com/lmasini > http://www.linkedin.com/pub/luca-masini/7/10/2b9 > **************************************** > > _______________________________________________ > forge-dev mailing list > forge-dev(a)lists.jboss.org > https://lists.jboss.org/mailman/listinfo/forge-dev > > -- Lincoln Baxter, III http://ocpsoft.org "Simpler is better." _______________________________________________ forge-dev mailing list forge-dev(a)lists.jboss.org https://lists.jboss.org/mailman/listinfo/forge-dev

Hum, I guess it makes sense. The CDIFacetImpl isInstalled() returns false only if the config file exists and the required dependencies are not found. Need a check on that. Fixing now. On 11/19/2012 02:28 PM, Luca Masini wrote: > It complaints that the file exists. > > It's true, but I should be able to get the facet anyway in my plugin, > don't you think ? > > > 2012/11/19 Lincoln Baxter, III <lincolnbaxter(a)gmail.com > <mailto:lincolnbaxter@gmail.com>> > > Yes, that looks right. What is the underlying exception from the > file creation error? That's a bit strange. > > > > On Sun, Nov 18, 2012 at 3:49 PM, Luca Masini > <luca.masini.mailing.list(a)gmail.com > <mailto:luca.masini.mailing.list@gmail.com>> wrote: > > Well, I'm trying to help developing the forge-errai plugin, > and doing the scaffolding I added this: > > @Alias("errai-ui") > @Help("Errai scaffolding") > @RequiresFacet({ WebResourceFacet.class, > DependencyFacet.class, > PersistenceFacet.class, > EJBFacet.class, > CDIFacet.class }) > > And when doing the install the CDIFacet is installed I got this: > > java.lang.RuntimeException: Failed to create required > [/Users/lucamasini/Documents/workspaceJBossTools/forge-2/src/main/webapp/WEB-INF/beans.xml] > at > org.jboss.forge.spec.javaee.cdi.CDIFacetImpl.install(CDIFacetImpl.java:63) > at > org.jboss.forge.project.BaseProject.performInstallation(BaseProject.java:240) > at > org.jboss.forge.project.BaseProject.installFacet(BaseProject.java:166) > at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) > at > sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39) > at > sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25) > at java.lang.reflect.Method.invoke(Method.java:597) > at > org.jboss.weld.bean.proxy.AbstractBeanInstance.invoke(AbstractBeanInstance.java:48) > at > org.jboss.weld.bean.proxy.ProxyMethodHandler.invoke(ProxyMethodHandler.java:125) > at > org.jboss.weld.proxies.Project$-1946286523$Proxy$_$$_WeldClientProxy.installFacet(Project$-1946286523$Proxy$_$$_WeldClientProxy.java) > at > org.jboss.forge.shell.project.FacetInstallationHandler.performInstallation(FacetInstallationHandler.java:105) > at > org.jboss.forge.shell.project.FacetInstallationHandler.installDependencies(FacetInstallationHandler.java:159) > at > org.jboss.forge.shell.project.FacetInstallationHandler.performInstallation(FacetInstallationHandler.java:100) > at > org.jboss.forge.shell.project.FacetInstallationHandler.install(FacetInstallationHandler.java:81) > at > org.jboss.forge.shell.project.FacetInstallationHandler.installRequest(FacetInstallationHandler.java:59) > at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) > at > sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39) > at > sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25) > ... > ... > ... > > Am I using it in the right way ? > > > 2012/11/18 Lincoln Baxter, III <lincolnbaxter(a)gmail.com > <mailto:lincolnbaxter@gmail.com>> > > Hey Luca, could you also give us the steps to reproduce? > Thanks! > > > On Sun, Nov 18, 2012 at 8:25 AM, George Gastaldi > <ggastald(a)redhat.com <mailto:ggastald@redhat.com>> wrote: > > Sounds like a bug to me. Can you reproduce it with a > test case ? We'll check it asap. > > Thanks ! > > George Gastaldi > > Em 18/11/2012, às 08:30, Luca Masini > <luca.masini(a)gmail.com > <mailto:luca.masini@gmail.com>> escreveu: > >> Hi, a question that may be is stupid, but why when I >> install the CDIFacet forge does a createNewFile ?? >> >> This, in case the facet was yet there,a launch a >> RuntimeException. >> >> Is done to prevent overwriting of custom beans.xml ?? >> >> Thank you. >> >> -- >> **************************************** >> http://www.lucamasini.net >> http://twitter.com/lmasini >> http://www.linkedin.com/pub/luca-masini/7/10/2b9 >> **************************************** >> _______________________________________________ >> forge-dev mailing list >> forge-dev(a)lists.jboss.org >> <mailto:forge-dev@lists.jboss.org> >> https://lists.jboss.org/mailman/listinfo/forge-dev > > _______________________________________________ > forge-dev mailing list > forge-dev(a)lists.jboss.org > <mailto:forge-dev@lists.jboss.org> > https://lists.jboss.org/mailman/listinfo/forge-dev > > > > > -- > Lincoln Baxter, III > http://ocpsoft.org > "Simpler is better." > > _______________________________________________ > forge-dev mailing list > forge-dev(a)lists.jboss.org <mailto:forge-dev@lists.jboss.org> > https://lists.jboss.org/mailman/listinfo/forge-dev > > > > > -- > **************************************** > http://www.lucamasini.net > http://twitter.com/lmasini > http://www.linkedin.com/pub/luca-masini/7/10/2b9 > **************************************** > > _______________________________________________ > forge-dev mailing list > forge-dev(a)lists.jboss.org <mailto:forge-dev@lists.jboss.org> > https://lists.jboss.org/mailman/listinfo/forge-dev > > > > > -- > Lincoln Baxter, III > http://ocpsoft.org > "Simpler is better." > > _______________________________________________ > forge-dev mailing list > forge-dev(a)lists.jboss.org <mailto:forge-dev@lists.jboss.org> > https://lists.jboss.org/mailman/listinfo/forge-dev > > > > > -- > **************************************** > http://www.lucamasini.net > http://twitter.com/lmasini > http://www.linkedin.com/pub/luca-masini/7/10/2b9 > **************************************** > > > _______________________________________________ > forge-dev mailing list > forge-dev(a)lists.jboss.org > https://lists.jboss.org/mailman/listinfo/forge-dev -- *George Gastaldi* | /Senior Software Engineer/ JBoss Forge Team Red Hat _______________________________________________ forge-dev mailing list forge-dev(a)lists.jboss.org https://lists.jboss.org/mailman/listinfo/forge-dev

Sorry for the delay, the problem went out using the SNAPSHOT. Thank you for your support. 2012/11/19 <ggastald(a)redhat.com <mailto:ggastald@redhat.com>> Luca, Just pushed the changes to the master branch. Can you try it again with the latest version ? Thanks ! On 11/19/2012 02:34 PM, ggastald(a)redhat.com <mailto:ggastald@redhat.com> wrote: > Hum, I guess it makes sense. > The CDIFacetImpl isInstalled() returns false only if the config > file exists and the required dependencies are not found. Need a > check on that. Fixing now. > > On 11/19/2012 02:28 PM, Luca Masini wrote: >> It complaints that the file exists. >> >> It's true, but I should be able to get the facet anyway in my >> plugin, don't you think ? >> >> >> 2012/11/19 Lincoln Baxter, III <lincolnbaxter(a)gmail.com >> <mailto:lincolnbaxter@gmail.com>> >> >> Yes, that looks right. What is the underlying exception from >> the file creation error? That's a bit strange. >> >> >> >> On Sun, Nov 18, 2012 at 3:49 PM, Luca Masini >> <luca.masini.mailing.list(a)gmail.com >> <mailto:luca.masini.mailing.list@gmail.com>> wrote: >> >> Well, I'm trying to help developing the forge-errai >> plugin, and doing the scaffolding I added this: >> >> @Alias("errai-ui") >> @Help("Errai scaffolding") >> @RequiresFacet({ WebResourceFacet.class, >> DependencyFacet.class, >> PersistenceFacet.class, >> EJBFacet.class, >> CDIFacet.class }) >> >> And when doing the install the CDIFacet is installed I >> got this: >> >> java.lang.RuntimeException: Failed to create required >> [/Users/lucamasini/Documents/workspaceJBossTools/forge-2/src/main/webapp/WEB-INF/beans.xml] >> at >> org.jboss.forge.spec.javaee.cdi.CDIFacetImpl.install(CDIFacetImpl.java:63) >> at >> org.jboss.forge.project.BaseProject.performInstallation(BaseProject.java:240) >> at >> org.jboss.forge.project.BaseProject.installFacet(BaseProject.java:166) >> at sun.reflect.NativeMethodAccessorImpl.invoke0(Native >> Method) >> at >> sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39) >> at >> sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25) >> at java.lang.reflect.Method.invoke(Method.java:597) >> at >> org.jboss.weld.bean.proxy.AbstractBeanInstance.invoke(AbstractBeanInstance.java:48) >> at >> org.jboss.weld.bean.proxy.ProxyMethodHandler.invoke(ProxyMethodHandler.java:125) >> at >> org.jboss.weld.proxies.Project$-1946286523$Proxy$_$$_WeldClientProxy.installFacet(Project$-1946286523$Proxy$_$$_WeldClientProxy.java) >> at >> org.jboss.forge.shell.project.FacetInstallationHandler.performInstallation(FacetInstallationHandler.java:105) >> at >> org.jboss.forge.shell.project.FacetInstallationHandler.installDependencies(FacetInstallationHandler.java:159) >> at >> org.jboss.forge.shell.project.FacetInstallationHandler.performInstallation(FacetInstallationHandler.java:100) >> at >> org.jboss.forge.shell.project.FacetInstallationHandler.install(FacetInstallationHandler.java:81) >> at >> org.jboss.forge.shell.project.FacetInstallationHandler.installRequest(FacetInstallationHandler.java:59) >> at sun.reflect.NativeMethodAccessorImpl.invoke0(Native >> Method) >> at >> sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39) >> at >> sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25) >> ... >> ... >> ... >> >> Am I using it in the right way ? >> >> >> 2012/11/18 Lincoln Baxter, III <lincolnbaxter(a)gmail.com >> <mailto:lincolnbaxter@gmail.com>> >> >> Hey Luca, could you also give us the steps to >> reproduce? Thanks! >> >> >> On Sun, Nov 18, 2012 at 8:25 AM, George Gastaldi >> <ggastald(a)redhat.com <mailto:ggastald@redhat.com>> >> wrote: >> >> Sounds like a bug to me. Can you reproduce it >> with a test case ? We'll check it asap. >> >> Thanks ! >> >> George Gastaldi >> >> Em 18/11/2012, às 08:30, Luca Masini >> <luca.masini(a)gmail.com >> <mailto:luca.masini@gmail.com>> escreveu: >> >>> Hi, a question that may be is stupid, but why >>> when I install the CDIFacet forge does a >>> createNewFile ?? >>> >>> This, in case the facet was yet there,a launch >>> a RuntimeException. >>> >>> Is done to prevent overwriting of custom >>> beans.xml ?? >>> >>> Thank you. >>> >>> -- >>> **************************************** >>> http://www.lucamasini.net >>> http://twitter.com/lmasini >>> http://www.linkedin.com/pub/luca-masini/7/10/2b9 >>> **************************************** >>> _______________________________________________ >>> forge-dev mailing list >>> forge-dev(a)lists.jboss.org >>> <mailto:forge-dev@lists.jboss.org> >>> https://lists.jboss.org/mailman/listinfo/forge-dev >> >> _______________________________________________ >> forge-dev mailing list >> forge-dev(a)lists.jboss.org >> <mailto:forge-dev@lists.jboss.org> >> https://lists.jboss.org/mailman/listinfo/forge-dev >> >> >> >> >> -- >> Lincoln Baxter, III >> http://ocpsoft.org >> "Simpler is better." >> >> _______________________________________________ >> forge-dev mailing list >> forge-dev(a)lists.jboss.org >> <mailto:forge-dev@lists.jboss.org> >> https://lists.jboss.org/mailman/listinfo/forge-dev >> >> >> >> >> -- >> **************************************** >> http://www.lucamasini.net >> http://twitter.com/lmasini >> http://www.linkedin.com/pub/luca-masini/7/10/2b9 >> **************************************** >> >> _______________________________________________ >> forge-dev mailing list >> forge-dev(a)lists.jboss.org <mailto:forge-dev@lists.jboss.org> >> https://lists.jboss.org/mailman/listinfo/forge-dev >> >> >> >> >> -- >> Lincoln Baxter, III >> http://ocpsoft.org >> "Simpler is better." >> >> _______________________________________________ >> forge-dev mailing list >> forge-dev(a)lists.jboss.org <mailto:forge-dev@lists.jboss.org> >> https://lists.jboss.org/mailman/listinfo/forge-dev >> >> >> >> >> -- >> **************************************** >> http://www.lucamasini.net >> http://twitter.com/lmasini >> http://www.linkedin.com/pub/luca-masini/7/10/2b9 >> **************************************** >> >> >> _______________________________________________ >> forge-dev mailing list >> forge-dev(a)lists.jboss.org <mailto:forge-dev@lists.jboss.org> >> https://lists.jboss.org/mailman/listinfo/forge-dev > > -- > *George Gastaldi* | /Senior Software Engineer/ > JBoss Forge Team > Red Hat > > > _______________________________________________ > forge-dev mailing list > forge-dev(a)lists.jboss.org <mailto:forge-dev@lists.jboss.org> > https://lists.jboss.org/mailman/listinfo/forge-dev -- *George Gastaldi* | /Senior Software Engineer/ JBoss Forge Team Red Hat _______________________________________________ forge-dev mailing list forge-dev(a)lists.jboss.org <mailto:forge-dev@lists.jboss.org> https://lists.jboss.org/mailman/listinfo/forge-dev -- **************************************** http://www.lucamasini.net http://twitter.com/lmasini http://www.linkedin.com/pub/luca-masini/7/10/2b9 **************************************** _______________________________________________ forge-dev mailing list forge-dev(a)lists.jboss.org https://lists.jboss.org/mailman/listinfo/forge-dev