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 <[email protected]> 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" <[email protected]> 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 <[email protected]> wrote: >> >> Is there a goog.addDependency for ApplicationBase.js in the working and >> non-working index.html, and are they different? > > >
