search

RC-Paves3-build / plovr

Automatically exported from code.google.com/p/plovr
0 stars 0 forks source link

Enable ability to turn on generateExports #15

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
What steps will reproduce the problem?
1. Use Case* : want to turn on the generateExports flag in the Compiler

What is the expected output? What do you see instead?
Would expect to be able to specify generate_exports in the JSON file and have 
the flag turned on in the Compiler.

What version of the product are you using? On what operating system?
72db5dfcc1

Please provide any additional information below.
It's possible to set a generate_exports flag on the Compiler if you change the 
source (which I've done), but it seems quite complicated in plovr to simply 
turn on a flag. 

If I could get a little more information on how specific compiler flags are 
turned on, I could implement this and write a patch.

Original issue reported on code.google.com by stvhl00 on 26 Oct 2010 at 10:49

GoogleCodeExporter commented 8 years ago

Original comment by bolinf...@gmail.com on 26 Oct 2010 at 7:39

GoogleCodeExporter commented 8 years ago

Original comment by bolinf...@gmail.com on 27 Oct 2010 at 1:06

GoogleCodeExporter commented 8 years ago 
Flag to turn this on in the compiler should be inbound soon:

http://code.google.com/p/closure-compiler/issues/detail?id=269

Original comment by stvhl00 on 25 Feb 2011 at 10:10

GoogleCodeExporter commented 8 years ago 
As of a recent change 
(http://code.google.com/p/plovr/source/detail?r=68e1855f6282939e130fa45d62e8fe4d
f81c7c82), you should be able to do this with the following addition to your 
plovr config:

"experimental-compiler-options": {
  "generateExports": true
}

As I have discussed before, I don't like the way that @export is currently 
supported (some users of your library may want to export the function, others 
may not), so ultimately, I hope to provide a better exports API in plovr.

But that's no reason to deny plovr users access to the existing generateExports 
option, so now you have a reasonable workaround!

Original comment by bolinf...@gmail.com on 24 Apr 2011 at 11:44