Hi, Thanks for all the explanations. I wrote a quick update inlined below. However, I am still investigating and I will probably have better status within a day or two Regards, Avner > -----Original Message----- > From: Sage Weil > Sent: Tuesday, August 23, 2016 18:21 > > On Tue, 23 Aug 2016, Avner Ben Hanoch wrote: > > For removing autotools support, I think that we should first announce > > that autotools will not be supported starting <some-future-day> and > > till then support both of them in parallel and response to questions > > about it. > > I think we're basically proposing some-future-day as now. > > > Also, we first need to make sure that we have comprehensive > > instructions for using cmake. Notice, that many developers are not > > familiar with cmake; hence, there might be need for extra explanations. > > > > For example, I tried now the instructions in > https://github.com/ceph/ceph/blob/master/README.md > > ./do_cmake.sh > > cd build > > make > > and I got the error: > > make: *** No targets specified and no makefile found. Stop. > > > > Probably, because of the explanation "This assumes you make your build > > dir a subdirectory of the ceph.git checkout. If you put it elsewhere, > > just replace .. above with a correct path to the checkout." However, > > if you are not familiar with cmake, then this explanation does not > > tell you how to recover from the error. > > The do_cmake.sh script is doing 'mkdir build', so assuming you actually ran it > from the git checkout (i.e., ./do_cmake.sh) then this should be the case. Is > that actually what caused the error? I did run "cd build; make" from the git checkout (I did ./do_cmake.sh) when I had the error make: *** No targets specified and no makefile found. Stop. I am still investigating, because sometimes it succeeds and sometimes it fails > > I agree that step 0 is to make README.md complete for cmake. Perhaps step > 1 is to remove autotools instructions too. > > > Also, I am not sure it is clear to all developers how to combine above > > instructions with options that you could provide to ./configure script. > > There is no more ./configure :). There are a new set of options you can pass > cmake that are referenced here in README.md: > > https://github.com/ceph/ceph/blob/master/README.md#cmake- > options I have many link errors after using ./do_cmake.sh -DWITH_XIO=ON I am still investigating > > > Hence, I think removing autotools support should not be immediate > > Please help us identify any (other) documentation gaps so we can get past > this. I thought README.md was in good shape... > > sage > -- To unsubscribe from this list: send the line "unsubscribe ceph-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
