On 11/1/16, 4:09 AM, "Christofer Dutz" <christofer.d...@codecentric.de> wrote:
>Hi, > > > >so I cleaned up the MXMLFlexJSPublisher class greatly. Currently I’m >trying to figgure out why the release output of 3 modules doesn’t seem to >work. The thing is that I never tested the release versions of the >original build, so I’ll probably be comparing the new to the old output >first. But I did encounter some things I have questions about: > >1. I think it would be a good idea to omit this „marmotinini“ >conditional code. What needs to be done differently for a marmotini run >should be passed in by configuration Take it out and I'll see if checkintests still works. We might have already bypassed the marmotinni code paths as far as checkintests is concerned. > >2. The default for outputting stuff if no ouput-directory is >specified is the directory containing the main class ... I would prefer >it to be the current working directory It is? I thought it was a level higher. SWF output goes in the same directory as the main class, but JS output seems to go in the parent. If I transpire foo/bar/MainClass.mxml, I think the output ends up in foo/bin/js-debug. Anyway, what if we had more intelligent defaults. If the main class is in a "src" folder, then use the parent (IMO, that's more desirable for the way Flash Builder sets up projects), if the main class is in a src/main/flex folder, then use ../../../target. > >3. All (image) resources need to be located relative to the main >application class. I think they should be relative to the/any source >directories (Would like to have a separate source and a resource >directory) I'm not sure I want to spend time searching in lots of places, but having one more place to look would be fine with me. > >4. The extern.js files seem to be dumped to the intermediate >„debug-js“ directory. I think they should be dumped somewhere outsite >(they should not be needed in the output) Such as? Again, you could pick different places if you find the Main Class is in a src/main/flex folder. > > > >There’s one part I didn’t quite understand: > > > >if (!googConfiguration.getSkipTranspile()) { > appendEncodedCSS(projectIntermediateMainFile, projectName); >} > > > >What does this do? Also I don’t understand the conditional of only >requiring this if transpilation is not skipped (isn’t transpilation all >the JS output is about ... so what’s the use of skiping transpilation?) >Wouldn’t it be better to simply do a The goal of -skip-transpile is to let someone generate the js-debug version, muck with the results, then have FalconJX send the results to the GCC without overwriting modifications with new generated code. So, first you would run FalconJX with -debug=true to get the js-debug version, then after you muck with the results, you would run FalconJX with -debug=false, -skip-transpile=true and no JS or CSS files would be generated, just the code where GCC is handed the js-debug folder and told to generate the js-release folder should be run. It is basically a short cut for calling GCC yourself from the command line and lets you workaround any bugs in our code-generation. -Alex