-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviews.m5sim.org/r/86/#review136
-----------------------------------------------------------


Again, I think you should make it clear that this change is for the 
configuration stuff.  I think you should probably fix this one and the previous 
one by doing "config:" instead of "ruby:" at the beginning of the commit 
message.

- Nathan


On 2010-08-05 21:42:44, Brad Beckmann wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviews.m5sim.org/r/86/
> -----------------------------------------------------------
> 
> (Updated 2010-08-05 21:42:44)
> 
> 
> Review request for Default.
> 
> 
> Summary
> -------
> 
> ruby: reorganized how ruby specifies command-line options
> 
> 
> Diffs
> -----
> 
>   configs/common/Options.py a75564db03c3 
>   configs/example/ruby_fs.py a75564db03c3 
>   configs/example/ruby_se.py a75564db03c3 
>   configs/example/rubytest.py a75564db03c3 
>   configs/ruby/MESI_CMP_directory.py PRE-CREATION 
>   configs/ruby/MI_example.py PRE-CREATION 
>   configs/ruby/MOESI_CMP_directory.py PRE-CREATION 
>   configs/ruby/MOESI_CMP_token.py PRE-CREATION 
>   configs/ruby/MOESI_hammer.py PRE-CREATION 
>   configs/ruby/Ruby.py a75564db03c3 
> 
> Diff: http://reviews.m5sim.org/r/86/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Brad
> 
>

_______________________________________________
m5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/m5-dev

Reply via email to