Script to generate all systems

Script to generate some or all systems

I find it essential to generate all the systems in one command.
I have several CLs in flight where I am changing dart:dom and dart:html
together.

My habit is to run:

  ./go.sh | tee Q

I can inspect file Q if needed.

If I know the IDL has not changed, it is 5x faster to run

  ./go.sh --cached

Systems can be picked:

  ./go.sh native,frog

The missing piece here is that files are generated under two directories
lib/dom/generated/
lib/html/generated/

I propose that we move all the generated files under one directory, so
that the following process gives a *complete* picture of the changes:

  git checkout master
  ./go.sh
  mv ../generated ../generated0
  git checkout work
  ./go.sh
  meld ../generated0 ../generated

Committed: https://code.google.com/p/dart/source/detail?r=7773

[sra1, 2012-05-18 21:08:23]

I would like to recover the ability to generate everything in one command.

For new code like constructors I work on dom and html together so I find it
essential to be able to run everything.

I think it should be our habit to scan all the generated files for problems

Pavel - can you do a followup change to
https://code.google.com/p/dart/source/detail?r=7739 so that this script works? 
If you can't do a followup change soon, can you revert and re-apply later?

[vsm, 2012-05-18 21:19:22]

Why bash instead of python?  I don't think we have bash in the repo right now.

[sra1, 2012-05-18 21:50:51]

On 2012/05/18 21:19:22, vsm wrote:
> Why bash instead of python?  I don't think we have bash in the repo right now

/bin/bash is installed on our linux and Mac machines.

This is just for our convenience.  It should stay short.
The python code for this would be a lot longer and subject to mission creep.
I just want to make it easy to run all the steps and generate all the systems.

[vsm, 2012-05-18 22:04:38]

LGTM, but please add a comment that this is a convenience script and shouldn't
be invoked from our other build / test scripts unless ported to python or dart.

[podivilov, 2012-05-21 10:56:10]

Stephen,

This script currently works for me. Could you please clarify what exactly is
missing?

And I'm all for putting generated files for both dom and html into the same
directory.

https://chromiumcodereview.appspot.com/10383252/diff/3002/lib/dom/scripts/go.sh
File lib/dom/scripts/go.sh (right):

https://chromiumcodereview.appspot.com/10383252/diff/3002/lib/dom/scripts/go....
lib/dom/scripts/go.sh:39:
ALLSYSTEMS="frog,dummy,native,htmlfrog,htmldartium,native"
native appears twice in this list.

[sra1, 2012-05-21 16:31:17]

My apologies - I misunderstood something because of the if-then-else and seeing
the IDL being loaded only once.

We (collectively, not you) will need to make a few changes
  - Vijay's secure wrappers requires another system generated - frog dart:dom /
dart:html will have three systems.
  - I'd like the generated directory to be removed at the beginning to avoid
accidental references to file that are not generated that run.

https://chromiumcodereview.appspot.com/10383252/diff/3002/lib/dom/scripts/go.sh
File lib/dom/scripts/go.sh (right):

https://chromiumcodereview.appspot.com/10383252/diff/3002/lib/dom/scripts/go....
lib/dom/scripts/go.sh:39:
ALLSYSTEMS="frog,dummy,native,htmlfrog,htmldartium,native"
On 2012/05/21 10:56:11, podivilov wrote:
> native appears twice in this list.

Thanks!

[podivilov, 2012-05-21 18:39:10]

Maybe we can have just a single system "systemhtml" which generate dart:html
interfaces and implementations in one pass. It could generate a dart interface
right away, and use a frog/dartium subgenerator to generate corresponding
implementation. And the same for "systemdom". That way there will be less code
duplication. WDYT?

I think deleting generated directory is the right thing to do, as I remember it
used to work that way some time ago.