Re: [m5-dev] Review Request: Util: Get rid of the make_release.py script.

2011-02-15 Thread Gabe Black
Did you mean you just wanted a second opinion on the RELEASE_NOTES
change before this went in, or that you wanted a second opinion on this
too? The former has happened but not the later. Are you ok with this
going in now?

Gabe

On 02/14/11 06:50, Nathan Binkert wrote:
 This is an automatically generated e-mail. To reply, visit:
 http://reviews.m5sim.org/r/488/


 Fine with me, though I'd like a second opinion.  (I would like a second 
 opinion on the RELEASE_NOTES change).

 - Nathan


 On February 14th, 2011, 1:22 a.m., Gabe Black wrote:

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

 /Updated 2011-02-14 01:22:11/


   Description

 Util: Get rid of the make_release.py script.

 Since we're not doing releases any more we don't really need this script. If
 we need it in the future, we can resurrect it from the history.


   Diffs

 * util/make_release.py (e8f4bb35dca9)

 View Diff http://reviews.m5sim.org/r/488/diff/


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

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


Re: [m5-dev] Review Request: Util: Get rid of the make_release.py script.

2011-02-15 Thread nathan binkert
Both.  I like both changes, but I figure that we should allow others to
weigh in.

On Wed, Feb 16, 2011 at 12:09 AM, Gabe Black gbl...@eecs.umich.edu wrote:

  Did you mean you just wanted a second opinion on the RELEASE_NOTES change
 before this went in, or that you wanted a second opinion on this too? The
 former has happened but not the later. Are you ok with this going in now?

 Gabe


 On 02/14/11 06:50, Nathan Binkert wrote:

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

 Fine with me, though I'd like a second opinion.  (I would like a second 
 opinion on the RELEASE_NOTES change).


 - Nathan

 On February 14th, 2011, 1:22 a.m., Gabe Black wrote:
   Review request for Default, Ali Saidi, Gabe Black, Steve Reinhardt, and
 Nathan Binkert.
 By Gabe Black.

 *Updated 2011-02-14 01:22:11*
 Description

 Util: Get rid of the make_release.py script.

 Since we're not doing releases any more we don't really need this script. If
 we need it in the future, we can resurrect it from the history.

   Diffs

- util/make_release.py (e8f4bb35dca9)

 View Diff http://reviews.m5sim.org/r/488/diff/


 ___
 m5-dev mailing listm5-dev@m5sim.orghttp://m5sim.org/mailman/listinfo/m5-dev



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


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


Re: [m5-dev] Review Request: Util: Get rid of the make_release.py script.

2011-02-15 Thread Ali Saidi

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

Ship it!


- Ali


On 2011-02-14 01:22:11, Gabe Black wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 http://reviews.m5sim.org/r/488/
 ---
 
 (Updated 2011-02-14 01:22:11)
 
 
 Review request for Default, Ali Saidi, Gabe Black, Steve Reinhardt, and 
 Nathan Binkert.
 
 
 Summary
 ---
 
 Util: Get rid of the make_release.py script.
 
 Since we're not doing releases any more we don't really need this script. If
 we need it in the future, we can resurrect it from the history.
 
 
 Diffs
 -
 
   util/make_release.py e8f4bb35dca9 
 
 Diff: http://reviews.m5sim.org/r/488/diff
 
 
 Testing
 ---
 
 
 Thanks,
 
 Gabe
 


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


Re: [m5-dev] Review Request: Util: Get rid of the make_release.py script.

2011-02-14 Thread Nathan Binkert

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


Fine with me, though I'd like a second opinion.  (I would like a second opinion 
on the RELEASE_NOTES change).

- Nathan


On 2011-02-14 01:22:11, Gabe Black wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 http://reviews.m5sim.org/r/488/
 ---
 
 (Updated 2011-02-14 01:22:11)
 
 
 Review request for Default, Ali Saidi, Gabe Black, Steve Reinhardt, and 
 Nathan Binkert.
 
 
 Summary
 ---
 
 Util: Get rid of the make_release.py script.
 
 Since we're not doing releases any more we don't really need this script. If
 we need it in the future, we can resurrect it from the history.
 
 
 Diffs
 -
 
   util/make_release.py e8f4bb35dca9 
 
 Diff: http://reviews.m5sim.org/r/488/diff
 
 
 Testing
 ---
 
 
 Thanks,
 
 Gabe
 


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