We had an internal discussion about this and the consensus from AMD is that bumping the dependence up to 2.7 would be fine. There might be some short-term internal impacts but we can work around them, and being able to migrate to argparse is worth it.
We feel we should keep to the base 2.7 for now though, since it's not that difficult to work around the post-2.7 features that are in use. Steve On Thu, Apr 14, 2016 at 10:41 AM Gross, Joe <[email protected]> wrote: > Hi Andreas, > > Speaking for myself only, I can say that python2.7 works fine. The > restriction is that the version of hg that we have was linked to python2.7 > and has some code that is incompatible with >python2.7. As far as I have > seen so far, the only problem was the autojunk parameter. > > Joe > > -----Original Message----- > From: Andreas Sandberg [mailto:[email protected]] > Sent: Thursday, April 14, 2016 7:48 AM > To: Gross, Joe <[email protected]>; gem5 Developer List < > [email protected]>; [email protected] > Cc: Reinhardt, Steve <[email protected]> > Subject: Re: [gem5-dev] changeset in gem5: misc: Don't build region.py as > a PySource > > Hi Joe, > > Sorry about that. I¹ll look into it and push a patch that fixes this. > > In practice, we seem to depend on Python 2.6 for things like declaring > abstract methods in interfaces, the with statement, the multiprocessing > package, advanced string formatting, and probably a lot of other things as > well. I have updated the wiki to reflect this. > > We should probably consider changing the requirement to Python 2.7 since > that¹s what most users are likely to use. None of the currently supported > Ubuntu versions ship with anything older than 2.7.3. Let¹s save that for > another thread though. > > //Andreas > > On 13/04/2016, 21:45, "Gross, Joe" <[email protected]> wrote: > > >Hi Andreas, > > > >This checkin recently made it downstream to our internal repo and I > >notice that you've added a SequenceMatcher with the autojunk parameter. > >Currently gem5 claims to require python 2.5-2.7 > >(http://www.m5sim.org/Dependencies). However, adding this parameter now > >requires > >python2.7.1-2.7.11(https://docs.python.org/2/library/difflib.html) and > >thus the style checker breaks whenever it is run on our system(python2.7). > > > >Could you update the style checker script to use calls that are supported > >by python2.5-2.7.11? We would appreciate it and can then re-enable the > >style checker. Thanks! > > > >Joe > > > >-----Original Message----- > >From: gem5-dev [mailto:[email protected]] On Behalf Of Andreas > >Sandberg > >Sent: Wednesday, March 30, 2016 10:23 AM > >To: [email protected] > >Subject: [gem5-dev] changeset in gem5: misc: Don't build region.py as a > >PySource > > > >changeset 1f92ce16e171 in /z/repo/gem5 > >details: http://repo.gem5.org/gem5?cmd=changeset;node=1f92ce16e171 > >description: > >misc: Don't build region.py as a PySource > > > >The style refactor change (style: Refactor the style checker as a > >Python package) moved region.py from src/python/m5/util/ to > >util/style/. The SConscript update accidentally got lost in that > >commit. This commit removes region.py from src/python/SConscript. > > > >Signed-off-by: Andreas Sandberg <[email protected]> > > > >diffstat: > > > > src/python/SConscript | 1 - > > 1 files changed, 0 insertions(+), 1 deletions(-) > > > >diffs (11 lines): > > > >diff -r 2201a8c86538 -r 1f92ce16e171 src/python/SConscript > >--- a/src/python/SConscriptWed Mar 30 15:56:02 2016 +0100 > >+++ b/src/python/SConscriptWed Mar 30 16:21:27 2016 +0100 > >@@ -59,7 +59,6 @@ > > PySource('m5.util', 'm5/util/jobfile.py') > > PySource('m5.util', 'm5/util/multidict.py') > > PySource('m5.util', 'm5/util/orderdict.py') > >-PySource('m5.util', 'm5/util/region.py') > > PySource('m5.util', 'm5/util/smartdict.py') > > PySource('m5.util', 'm5/util/sorteddict.py') > > PySource('m5.util', 'm5/util/terminal.py') > >_______________________________________________ > >gem5-dev mailing list > >[email protected] > >http://m5sim.org/mailman/listinfo/gem5-dev > > IMPORTANT NOTICE: The contents of this email and any attachments are > confidential and may also be privileged. If you are not the intended > recipient, please notify the sender immediately and do not disclose the > contents to any other person, use it for any purpose, or store or copy the > information in any medium. Thank you. > > _______________________________________________ > gem5-dev mailing list > [email protected] > http://m5sim.org/mailman/listinfo/gem5-dev > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
