I was going to check that next. I’ll let you know what I find.

> On Dec 19, 2018, at 9:25 AM, Alex Harui <aha...@adobe.com.INVALID> wrote:
> 
> 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?
>> 
>> 
>> 
> 
> 
> 

Reply via email to