lgtm++

http://codereview.appspot.com/5677069/diff/3002/build.xml
File build.xml (right):

http://codereview.appspot.com/5677069/diff/3002/build.xml#newcode506
build.xml:506: <target name="pluginc"
depends="PluginCompiler,tools,precajole-light"
this messes up some of my manual testing habits, but I'll adapt.

(the bin/cajole script only needs 'ant jars' built, which is fast
because it only builds pluginc not the playground.  so I'll have to
remember to do 'ant precajole jars')

http://codereview.appspot.com/5677069/diff/3002/build.xml#newcode1830
build.xml:1830: <precajole spec="${precajolesrc}/heavy.xml"
dest="${precajolelib}" />
please either add a <precajole> of light.xml here, or include light.xml
in heavy.xml.  it's helpful to have the light js files in the actual
deployed playground.

http://codereview.appspot.com/5677069/

Reply via email to