> On Aug. 18, 2014, 10:03 a.m., Nilay Vaish wrote: > > 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. > > Andreas Hansson wrote: > I agree that it would be nice, but I think the time it would take to > develop something > automatic would far exceed the time needed to bump it when needed. > > Nilay Vaish wrote: > I suggest we remove the dependency stuff altogether from this file. > We expect the users to read the documentation on the website anyway. > > Andreas Hansson wrote: > I'm happy with that. Anyone else got any thoughts on this one?
I agree. Better to just replace the dependency text in the README with the URL for the appropriate wiki page, and then it will never be out of date again. Maybe list the tools we currently depend on (without mentioning versions) so people know that they need more than just a C++ compiler, to encourage them to actually check the link, but specifically say that this list is just an example. - Steve ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2351/#review5268 ----------------------------------------------------------- On Aug. 17, 2014, 3: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, 3: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
