Remove 'cppfront' Subfolder and Use FetchContent for `cppfront` Dependency instead

modern-cmake / cppfront

CMake wrapper around the cppfront repository

BSD 3-Clause "New" or "Revised" License

73 stars 19 forks source link

Remove 'cppfront' Subfolder and Use FetchContent for `cppfront` Dependency instead #142

Closed shemeshg closed 3 months ago

shemeshg commented 3 months ago

I'd like to propose removing the 'cppfront' subfolder from the project. This change would allow users to independently manage the cppfront version that the project refers to, without being tied to the project's updates.

Instead, let's adjust the FetchContent example to include the GIT_REPOSITORY for cppfront and cppfront-cmake that will demonstrate how to link between them

or maybe it is none issue since we can always add_subdirectory(cppfront) and change sub_directory branch

alexreinking commented 3 months ago

I just updated main to the latest cppfront.

I'm quite hesitant to use FetchContent. I don't like requiring an internet connection during the build.