Re: [gem5-dev] Review Request: copyright: Add code for finding all copyright blocks and create a COPYING file

[Steve Reinhardt]

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



src/python/m5/main.py


This is printed out every time, right?  If so, it seems pretty verbose.  
How about just:
'gem5 is copyrighted software; use the --copyright option for details.'

If you want to work the URL in, I'd put it in the initial header, e.g.,
'gem5 Simulator System, http://gem5.org'



- Steve


On 2011-06-02 14:49:29, Nathan Binkert wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> http://reviews.m5sim.org/r/726/
> ---
> 
> (Updated 2011-06-02 14:49:29)
> 
> 
> Review request for Default, Ali Saidi, Gabe Black, Steve Reinhardt, and 
> Nathan Binkert.
> 
> 
> Summary
> ---
> 
> copyright: Add code for finding all copyright blocks and create a COPYING file
> 
> The end of the COPYING file was generated with:
> % python ./util/find_copyrights.py configs src system tests util
> 
> Update -C command line option to spit out COPYING file
> 
> [ Some notes on this change.  1) I could add the find_copyrights.py function 
> to the build to automatically generate the list.  Should we do that?  For 
> now, I've appended the list to the COPYING file.  We should also probably get 
> rid of the AUTHORS file.  What do you all think? (either that or we should 
> update it) ]
> 
> 
> Diffs
> -
> 
>   COPYING PRE-CREATION 
>   LICENSE aa00cee9abb1 
>   src/SConscript aa00cee9abb1 
>   src/python/m5/main.py aa00cee9abb1 
>   util/find_copyrights.py PRE-CREATION 
> 
> Diff: http://reviews.m5sim.org/r/726/diff
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Nathan
> 
>

___
gem5-dev mailing list
gem5-dev@m5sim.org
http://m5sim.org/mailman/listinfo/gem5-dev

[gem5-dev] Review Request: copyright: Add code for finding all copyright blocks and create a COPYING file

[Nathan Binkert]

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

Review request for Default, Ali Saidi, Gabe Black, Steve Reinhardt, and Nathan 
Binkert.


Summary
---

copyright: Add code for finding all copyright blocks and create a COPYING file

The end of the COPYING file was generated with:
% python ./util/find_copyrights.py configs src system tests util

Update -C command line option to spit out COPYING file

[ Some notes on this change.  1) I could add the find_copyrights.py function to 
the build to automatically generate the list.  Should we do that?  For now, 
I've appended the list to the COPYING file.  We should also probably get rid of 
the AUTHORS file.  What do you all think? (either that or we should update it) ]


Diffs
-

  COPYING PRE-CREATION 
  LICENSE aa00cee9abb1 
  src/SConscript aa00cee9abb1 
  src/python/m5/main.py aa00cee9abb1 
  util/find_copyrights.py PRE-CREATION 

Diff: http://reviews.m5sim.org/r/726/diff


Testing
---


Thanks,

Nathan

___
gem5-dev mailing list
gem5-dev@m5sim.org
http://m5sim.org/mailman/listinfo/gem5-dev