Changes by: Jason Hickey (jyh at cs.caltech.edu)
Date: 2006-06-18 11:15:52 -0700 (Sun, 18 Jun 2006)
Revision: 9343
Log message:

      Pass rule options as a Map, not an array of length-2 arrays
      (which is not a sensible value anyway).
      
      Also, we weren't expanding :value: dependencies in
      Omake_rule.eval_rule.  Suppose we had a rule like the
      following.
      
         a: b :value: $X :exists: foo
            ...
      
      The dependency is parsed as <sequence $`X " ">.  This is
      a primitive value, so the $`X was never evaluated (and
      so changing the value of X had no effect).
      
      To do it right, flatten the values with values_of_value,
      then take the primitive values.  You will see that the
      value counts are now much larger.
      

Changes  Path
+41 -37 omake-branches/0.9.8.x/src/build/omake_rule.ml
+54 -1 omake-branches/0.9.8.x/src/builtin/omake_builtin_object.ml
+6 -4 omake-branches/0.9.8.x/src/builtin/omake_builtin_target.ml
+9 -5 omake-branches/0.9.8.x/src/env/omake_ir_ast.ml
+9 -0 omake-branches/0.9.8.x/src/eval/omake_value.ml
+1 -0 omake-branches/0.9.8.x/src/eval/omake_value.mli
+1 -0 omake-branches/0.9.8.x/src/ir/omake_ir.ml
+3 -0 omake-branches/0.9.8.x/src/ir/omake_symbol.ml