[Omake] SVN Commit: OMake Build System [0.9.8.x] (Rev. 10529)

Jason Hickey jyh at cs.caltech.edu
Thu Apr 26 20:22:31 PDT 2007


We never exported private variables.  Really, we were quite sloppy in code 
like this:

    private.x = 1
    if ...
        x += 2
        export
    println($x)

- The private.x definition doesn't affect future occurrences of x.
- The x += 2 is really
      global.x = $(global.x) 2
  where global.x means "first look for a public definition, then look for private."

  So $(global.x) gets the private value (assuming a public value is not
  already present), and global.x = ... defines both the public value
  and the private value.
- The println($x) is really
      println($(global.x))

This is really terrible.  In any case, revert to the original
behavior.

This will not propagate forward!  0.9.9 is sensible.

----
Changes:
    +1 -1	omake-branches/0.9.8.x/lib/build/OCaml.om
    +25 -17	omake-branches/0.9.8.x/src/env/omake_ir_ast.ml
 
A hyperlinked version of this commit is available at
http://svn.metaprl.org/commitlogs/omake/2007-04.html#07/04/26.20:22:31



More information about the OMake-CVS mailing list