You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@royale.apache.org by GitBox <gi...@apache.org> on 2020/06/12 14:30:49 UTC

[GitHub] [royale-asjs] carlosrovira opened a new issue #866: Crux regression in release mode

carlosrovira opened a new issue #866:
URL: https://github.com/apache/royale-asjs/issues/866


   Hi @greg-dove,
   
   I'm getting the following error in some Crux applications. Only release, debug is working ok. I guess is something related to latest changes.
   
   For example, "CruxQuickStart" is working ok, but "todomvc-jewel-crux" is showing the problem.
   
   This is the RTE from example "todomvc-jewel-crux" in release mode.
   
   > getDefinitionByName.js:59 Uncaught ReferenceError: Error #1065: Variable org.apache.royale.jewel.View is not defined.
   >        at $q (getDefinitionByName.js:59)
   >        at br.L.Sk (TypeDefinition.as:389)
   >        at Iz.fromTypeDefinition (TypeDescriptor.as:167)
   >        at Uu (TypeCache.as:60)
   >        at nw (BeanFactory.as:616)
   >        at qw (CruxManager.as:107)
   >        at st.L.setUpEventHandler (BeanFactory.as:517)
   >        at oB.Mq.fireListeners (EventDispatcher.as:131)
   >        at qp (eventtarget.js:372)
   >        at Mx.L.Kp (JSStageEvents.as:209)
   > ...(hidden in the browser)...
   > $q @ ContainerDataBinding.as:177
   > L.Sk @ TypeDefinition.js:513
   > Iz.fromTypeDefinition @ TypeDescriptor.as:139
   > Uu @ ArrayListView.js:1088
   > nw @ BeanFactory.as:577
   > qw @ CruxManager.as:69
   > L.setUpEventHandler @ BeanFactory.as:493
   > Mq.fireListeners @ EventDispatcher.as:110
   > qp @ eventtarget.js:349
   > L.Kp @ JSStageEvents.as:155
   > childList (async)
   > ez.addElement @ TextPrompt.js:177
   > L.initialize @ Application.as:706
   > ez.start @ Application.as:668
   > (anonymous) @ index.html:37


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [royale-asjs] carlosrovira commented on issue #866: Crux regression in release mode

Posted by GitBox <gi...@apache.org>.
carlosrovira commented on issue #866:
URL: https://github.com/apache/royale-asjs/issues/866#issuecomment-643583572


   Hi Chris,
   
   it's very strange, since it's clear that Jewel is detected in the dependency graph, from some parent, since in fact js-debug works perfect (and is finding all the jewel classes). The problem is only in js-release, where it finds too, since all compiles fine, but we just see the problem at Runtime when Crux throws the RTE about one class not found in the Reflection sussystem.
   In fact I see the interface of todo mvc rendered showing the title, and the jewel list.
   
   Maybe is a problem related to compiler in relation to some maven structure issue.
   It's clear that is not critic, since we have a workaround, but maybe something to take into account in the one branch refactor.
   
   Thanks


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [royale-asjs] carlosrovira commented on issue #866: Crux regression in release mode

Posted by GitBox <gi...@apache.org>.
carlosrovira commented on issue #866:
URL: https://github.com/apache/royale-asjs/issues/866#issuecomment-643323116


   Hi Greg, I found a work around to fix the problem.
   If I add in my pom.xml the jewel dependency it works on release. What seems to me very strange since it's getting it from the parent:
   
   ```xml
   <dependency>
         <groupId>org.apache.royale.framework</groupId>
         <artifactId>Jewel</artifactId>
         <version>0.9.8-SNAPSHOT</version>
         <type>swc</type>
         <classifier>js</classifier>
       </dependency>
   ```
   
   Maybe something to check with @chrisdutz 
   
   Thanks


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [royale-asjs] chrisdutz commented on issue #866: Crux regression in release mode

Posted by GitBox <gi...@apache.org>.
chrisdutz commented on issue #866:
URL: https://github.com/apache/royale-asjs/issues/866#issuecomment-643328929


   Well as far as I can see it the todomvc-jewel-crux module didn't have the Jewel dependency ... so if you add it and it works, I guess it was simply missing. And I can't see that it's included by a parent module.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org