----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2351/#review5268 -----------------------------------------------------------
I am fine with the patch as such. I think we should either maintain the dependencies here in this file or on the website. I don't see why we would like maintain them in two different places unless we can link them so that one reflects the other. - Nilay Vaish On Aug. 17, 2014, 10:46 a.m., Andreas Hansson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/2351/ > ----------------------------------------------------------- > > (Updated Aug. 17, 2014, 10:46 a.m.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > Changeset 10284:9d637b2b26c7 > --------------------------- > misc: Update dependencies in README and fit in 80 char > > This patch updates the README to reflect the current tool > dependencies, and also reformats certain sections to remove trailing > whitespace and fit the lines to a 80 char limit. > > > Diffs > ----- > > README 79fde1c67ed8 > > Diff: http://reviews.gem5.org/r/2351/diff/ > > > Testing > ------- > > > Thanks, > > Andreas Hansson > > _______________________________________________ gem5-dev mailing list gem5-dev@gem5.org http://m5sim.org/mailman/listinfo/gem5-dev
