<div>Dear Richard,</div><div> </div><div>28.12.2020, 17:39, "Richard B. Kreckel" <kreckel@in.terlu.de>:</div><div>> I recall that I tried your CMake patches for CLN twice and ran into some</div><div>> issues, see [1].</div><div> </div><div>> Why introduce src/cl_config.h in addition to include/cln/config.h?</div><div> </div><div>Because include/cln/config.h is a public header, and we don't want to pollute it</div><div>with PACKAGE, VERSION, HAVE_FOO and other autotools defined macros (otherwise</div><div>any code using CLN and autotools would fail to build).</div><div> </div><div>Autotools build does the same thing, the only difference is that the cl_config.h</div><div>resides in the `autoconf' directory.</div><div> </div><div>> There should be only one location for this stuff.</div><div> </div><div>It would be nice, but for now it's difficult. For one include/cln/types.h needs</div><div>some macros computed by configure (or CMake) to define sintD, sintE, sintC,</div><div>sintP, etc.</div><div> </div><div>> Unfortunately, I'm getting build errors when building CLN with autotools</div><div>> and with CMake in an alternating way due to multiple definitions of</div><div>> GMP_DEMANDS_UINTD_LONG etc.</div><div> </div><div>I've checked both type builds like this:</div><div> </div><div>autoreconf -i</div><div>mkdir _build_autotools && cd _build_autotools && ../configure --disable-static && cd ..</div><div>mkdir _build_cmake && cd _build_cmake && cmake -GNinja -DCMAKE_BUILD_TYPE=RelWithDebInfo .. && cd ..</div><div>make -j4 -C _build_autotools &</div><div>cmake --build _build_cmake &</div><div> </div><div>Both builds run concurrently and have completed just fine. Hence the question:</div><div>could you please explain what exactly are you doing?</div><div> </div><div>Best regards,</div><div>      Alexey</div><div> </div>