Pretty sure something will need to change in GoogDepsWriter. Don't know what
just yet. Might be best if you can come up with a simple test case.. I'm
working on a circular dependency issue right now that might also affect
GoogDepsWriter. If you build some of our examples, do any show missing
goog.addDependencies?
-Alex
On 12/18/18, 11:12 PM, "Harbs" <harbs.li...@gmail.com> wrote:
So where do you suggest we fix it? In GoogDepsWriter.removeRequires?
I’m not completely following what’s going on there.
Harbs
> On Dec 19, 2018, at 12:52 AM, Alex Harui <aha...@adobe.com.INVALID> wrote:
>
> Yeah, that makes more sense. So somehow the information we get back from
processing the Royale Dependency List isn't recovering all of the dependency
data. I had noticed that we didn't list @extends and @implements in the
Dependency Lists and wondered how that was working. Maybe it isn't working and
we've been getting lucky where we've seen it work.
>
> -Alex
>
> On 12/18/18, 1:59 PM, "Harbs" <harbs.li...@gmail.com> wrote:
>
> The working one has (in this order):
> goog.addDependency('../../../org/apache/royale/core/Application.js',
['org.apache.royale.core.Application'],
['org.apache.royale.core.ApplicationBase', 'org.apache.royale.core.IStrand',
'org.apache.royale.core.IParent', 'org.apache.royale.events.IEventDispatcher',
'org.apache.royale.core.IInitialViewApplication',
'org.apache.royale.core.IPopUpHost', 'org.apache.royale.core.IPopUpHostParent',
'org.apache.royale.core.IRenderedObject']);
>
goog.addDependency('../../../org/apache/royale/core/IInitialViewApplication.js',
['org.apache.royale.core.IInitialViewApplication'],
['org.apache.royale.events.IEventDispatcher']);
> goog.addDependency('../../../org/apache/royale/core/IPopUpHost.js',
['org.apache.royale.core.IPopUpHost'], []);
>
goog.addDependency('../../../org/apache/royale/core/IPopUpHostParent.js',
['org.apache.royale.core.IPopUpHostParent'],
['org.apache.royale.core.IParent']);
>
goog.addDependency('../../../org/apache/royale/core/ApplicationBase.js',
['org.apache.royale.core.ApplicationBase'],
['org.apache.royale.core.HTMLElementWrapper',
'org.apache.royale.core.IFlexInfo']);
> goog.addDependency('../../../org/apache/royale/core/IFlexInfo.js',
['org.apache.royale.core.IFlexInfo'], []);
>
> The broken one has:
>
> goog.addDependency('../../../org/apache/royale/core/Application.js',
['org.apache.royale.core.Application'],
['org.apache.royale.core.ApplicationBase', 'org.apache.royale.core.IStrand',
'org.apache.royale.core.IParent', 'org.apache.royale.events.IEventDispatcher',
'org.apache.royale.core.IInitialViewApplication',
'org.apache.royale.core.IPopUpHost', 'org.apache.royale.core.IPopUpHostParent',
'org.apache.royale.core.IRenderedObject']);
>
goog.addDependency('../../../org/apache/royale/core/BrowserResizeListener.js',
['org.apache.royale.core.BrowserResizeListener'],
['org.apache.royale.core.IBead']);
> goog.addDependency('../../../org/apache/royale/core/CallLaterBead.js',
['org.apache.royale.core.CallLaterBead'], ['org.apache.royale.core.IBead']);
>
> (No addDependency for ApplicationBase)
>
> The broken one also has no addDependency for HTMLElementWrapper,
IFlexInfo and I’m sure many others.
>
>> On Dec 18, 2018, at 11:43 PM, Alex Harui <aha...@adobe.com.INVALID>
wrote:
>>
>> Is there a goog.addDependency for ApplicationBase.js in the working and
non-working index.html, and are they different?
>
>
>
