Hi Alex,

thanks for your responses. Well I made it check the closureLib as I wanted to 
allow users to specify that if they wanted to monkey-patch stuff. 
Well for now I removed the VF2JS stuff in my branch. I think I remember us 
discussing that and that consensus was to remove the code.

At least with my last commits I managed the release build to work and I no 
longer dump the closure stuff to release. 

Chris


Am 31.10.16, 15:46 schrieb "Alex Harui" <aha...@adobe.com>:

    
    
    On 10/31/16, 1:14 AM, "Christofer Dutz" <christofer.d...@c-ware.de> wrote:
    
    >My optimization should have an effect if the closureLib is NOT explicitly
    >set and the file „goog/deps.js“ is found on the compilers classpath. This
    >is probably only the case for Maven builds.
    
    I think closureLib is sometimes set for me.  Any reason not to just rely
    on whether you find deps.js on the class path?
    
    >
    >Currently the release build doesn’t kick in at all, even for a Maven
    >build. I am hoping to fix this today after I understood the class a
    >little better.
    
    I assume you know that the release build is contingent on the -debug flag.
     Is there more to it than that?
    
    >
    >Currently it seems as if for a debug build closure is only needed for
    >dumping the dependency js files to the debug directory - is that correct?
    
    The Closure Compiler?  I wasn't aware it was used at all for debug builds,
    but I don't know this code in detail.
    Closure Library is needed to run the goog.provide and goog.require and
    goog.events code.
    
    >The actual closure compiler kicks in with compiling only for the release
    >build – is that correct?
    
    That's my understanding.
    
    >If this is correct, why is the closure code dumped to the debug and the
    >release directory?
    
    Don't know.  I don't see the closure library code in the bin/js-release.
    Does the Closure Compiler compile in-place and remove files?
    
    -Alex
    
    

Reply via email to