By putting all SWCs back in the library-path in royale-config, you have re-introduced a problem I was fixing. You can say you didn't revert the actual git commit, but your changes essentially undid my commit. The build passed with my commit, as far as I can tell. IF we all start reverting stuff we don't like, we'll just end up in a revert war. We are all in a time-crunch, not just you. This is cutting-edge development. Things may get in your way some times, but you shouldn't make everyone else pay a price to save you time. You didn't need to make that change in the repo, you could have worked off a branch. If you run into a bug, you'll have to take the time to investigate or hire someone to do it if none of the other volunteers happen to have time.
My 2 cents, -Alex On 11/25/18, 1:56 AM, "Carlos Rovira" <carlosrov...@apache.org> wrote: Hi Alex, El dom., 25 nov. 2018 a las 8:34, Alex Harui (<aha...@adobe.com.invalid>) escribió: > Does configname make any difference in your IDE? Maybe the IDE doesn't > support it. The compiler output should switch to saying it is loading > flex-config instead of royale-config. > > I think there's tow ways, one using addtiionalCompiler options like I show. Another from the AS3&MXML extension is "config: royale" or "config: flex" (values can be as well "air", and others) > The change you made to royale-config-template re-introduces the problem I > was trying to fix. IMO, you should make a local change in a branch instead > of reverting my commits. > I didn't revert your commit, just do a new one to be reverted, since it seems to be something goes wrong with the changes. For your words I understand there's no bugs or things to take care to solve this? > I recommend that you compare the set of manifests loaded by flex-config vs > royale-config. It looks like some are missing. And you are welcome to > make another config.xml for your combination. The problem with having > "everything" in a -config.xml is that it can add too many options to > code-hinting. > Ok, the problem is that I spend yesterday morning trying to see something similar but only the actual patch worked. I tried to add to royale config the MXRoyale.swc for SWF and JS and namespace, but that didn't make a difference. Do you know what's the problem starting from that point? or if the approach is not that? I think I can create one for Jewel applications, and remove from that sets that are just experiments but not usable like createjs, flat, express More over, could revert my change and create a "jewel-config-template.xml" to test that is possible? I was not able to do it yesterday so maybe there's a problem and is not possible right now I'll be out until today until night, and will catch up if you commit something to try it. I understand that would be using addtionalcompiler options path with +configname Another thing that could be good is the one you already commented about separate MXRoyale in MX Rpc, Formatters, and more, to help in this way to code intelligence in IDEs to have only the right libraries the users is using Thanks Carlos > > HTH, > -Alex > > > On 11/24/18, 7:04 AM, "Carlos Rovira" <carlosrov...@apache.org> wrote: > > Hi, > > after doing full rebuild I'm finding a problem in IDE (not in build) > with > the new change +configname=flex > > - we are using Jewel and MXRoyale, just por MX RPC (and maybe some > other > class in MX). > - we are using several libraries and one application > > today I found that all MX code throws error in VS Code and is not > recognize. I see the change about configname. To adapt our config > project > to this, I added to each asconfig file in all royale libs and app in > our > project the line > > "additionalOptions": "+configname=flex" > > but this doesn't solves the problem > > in fact I'm seeing more problems like > > In initializer for 'j:icon', type org.apache.royale.icons.FontIcon is > not > assignable to target type 'org.apache.royale.core.IIcon'. > Implicit coercion of a value with static type MouseEvent to a possibly > unrelated type MouseEvent. > In initializer for 'j:previousButton', type > org.apache.royale.jewel.Group > is not assignable to target type 'org.apache.royale.core.UIBase'. > Ambiguous reference to MouseEvent > > and many more > > This makes hundreds of errors and makes the IDE unusable. I suppose > that > this change not block people waning to use a mix of libraries, and is > just > a matter to know how to update configuration, right? I mean if someone > wants to use in the same application three different buttons( jewel, > basic, > and mx/spark), that should still be possible right? > > What should I do in my project configuration to solve this problem? > > thanks > > -- > Carlos Rovira > > https://na01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fabout.me%2Fcarlosrovira&data=02%7C01%7Caharui%40adobe.com%7Cd0cc9cd060344c7301ad08d652bc54a6%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C636787366186532617&sdata=cKyL3blSiBW9uq7B2%2FLbVpuGiWoFcNKFTK%2B%2FZVt1hsI%3D&reserved=0 > > > -- Carlos Rovira https://na01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fabout.me%2Fcarlosrovira&data=02%7C01%7Caharui%40adobe.com%7Cd0cc9cd060344c7301ad08d652bc54a6%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C636787366186532617&sdata=cKyL3blSiBW9uq7B2%2FLbVpuGiWoFcNKFTK%2B%2FZVt1hsI%3D&reserved=0