> On April 1, 2014, 8:50 p.m., Benjamin Hindman wrote: > > src/Makefile.am, lines 161-163 > > <https://reviews.apache.org/r/19854/diff/2/?file=543618#file543618line161> > > > > I suggested pulling out the offending translation units so that people > > could have some context to understand what you mean by this comment. Maybe > > I'm just the ignorant one, but I'd like this comment to be even more > > explicit. Here is another suggestion: > > > > ---- > > As of automake 1.14 the 'subdir-objects' option is turned on by default > > and so we can't have the same file names in sub-directories because the > > resulting object files collide (... how do the relative paths play a role > > here?). For example, we can't have both a master/constants.cpp and a > > slave/constants.cpp. > > ---- > > > > Also, is this fixable? Can we add a TODO and include suggestions to fix > > this so that someone else doesn't need to try and think of those > > suggestions themselves in the future?
I decided to dig deeper into this and uncovered that my original version of 1.14 was a pre-release build, and quite buggy. http://koji.fedoraproject.org/koji/buildinfo?buildID=470561. In that build, my object files were colliding even though they should not. Upon update to automake-1.14.1-1 the issue does not exist, so I've removed the modification. - Timothy ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19854/#review39201 ----------------------------------------------------------- On April 1, 2014, 5:24 p.m., Timothy St. Clair wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/19854/ > ----------------------------------------------------------- > > (Updated April 1, 2014, 5:24 p.m.) > > > Review request for mesos and Benjamin Hindman. > > > Bugs: MESOS-577 > https://issues.apache.org/jira/browse/MESOS-577 > > > Repository: mesos-git > > > Description > ------- > > Enable automake 1.14 > > > Diffs > ----- > > 3rdparty/libprocess/configure.ac 2e0b72c > bootstrap ed0bc36 > configure.ac 5404dc2 > src/Makefile.am 47d03b3 > src/master/constants.cpp 1cb8f22 > src/master/http.cpp 72d8e91 > src/master/master_constants.cpp PRE-CREATION > src/master/master_http.cpp PRE-CREATION > src/slave/constants.cpp 9811bd3 > src/slave/http.cpp 594032d > src/slave/slave_constants.cpp PRE-CREATION > src/slave/slave_http.cpp PRE-CREATION > > Diff: https://reviews.apache.org/r/19854/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Timothy St. Clair > >
