Re: [PATCH 0/8] CMake build system for git
- From
Johannes Schindelin <johannes.schindelin@gmx.de>
- Date
- May 7, 2020, 20:54 UTC
- Message-ID
- <nycvar.QRO.7.76.6.2005072253000.56@tvgsbejvaqbjf.bet>
- In-Reply-To
- <848941588629532@mail.yandex.ru>
Hi Konst,
On Tue, 5 May 2020, Konstantin Tokarev wrote:
Show 44 quoted lines
> 05.05.2020, 00:32, "Johannes Schindelin" <johannes.schindelin@gmx.de>: > > Hi Konst, > > > > On Sun, 3 May 2020, Konstantin Tokarev wrote: > > > >> 03.05.2020, 20:21, "Junio C Hamano" <gitster@pobox.com>: > >> > Sibi Siddharthan <sibisiddharthan.github@gmail.com> writes: > >> > > >> >>> As you say, an extra instruction in INSTALL file to tell users to > >> >>> copy from contrib/cmake may workable, though it is unsatisfactory. > >> >>> But the other one will not simply work. If we need to have a new > >> >>> file with string "CMake" in its name at the top-level *anyway*, we > >> >>> should have the real thing to reduce one step from those who want to > >> >>> use it. Those who do not want to see "CMake" at the toplevel are > >> >>> already harmed either way, if is a dummy or if it is the real thing. > >> >> > >> >> In your opinion, what would be the best way to communicate with users, there is > >> >> an optional CMake build system for git? > >> > > >> > You do not want to hear my opinion, as my priorities would be > >> > different from yours ;-) > >> > > >> > Given that we all agreed that the only reason we contemplate use of > >> > CMake in our project is strictly to help Windows build, i.e. due to > >> > the same reason why we have contrib/buildsystems/, it is not one of > >> > my goals to communicate with general users about optional CMake > >> > support in the first place. It has lower priority than keeping the > >> > project tree and the project history less cluttered. > >> > > >> > So my first preference would be an instruction somewhere in install > >> > or readme that tells those who want to build for windows to copy > >> > from (or perhaps update cmake to offer the "-f" option and tell it > >> > to read from) contrib/cmake/CMakeLists.txt to the toplevel before > >> > doing anything [*1*]. > >> > >> FWIW, CMakeLists.txt doesn't have to be in the root of source tree in > >> order to work. It can perfectly work from contrib/cmake after necessary > >> changes in relative paths. > > > > Would you have an example handy, or a link to an article describing this? > > It's so trivial that I'm not sure what such an article would have to describe. > > https://github.com/annulen/cmake-example
It is trivial all right, but I had hoped for a more elegant solution along the lines of `set_top_level_directory(../..)` or some such. I mistook your mail for having an elegant version of this, and I was curious.
Ciao, Johannes
Show 5 quoted lines
> -- > Regards, > Konstantin > >