Could you please replace FetchContent_Declare(ftxui) with find_package(ftxui REQUIRED)?

aaleino / todoman

A terminal program that helps to create detailed to-do lists and keeps track of time

MIT License

24 stars 2 forks source link

Could you please replace FetchContent_Declare(ftxui) with find_package(ftxui REQUIRED)? #2

Open yurivict opened 1 year ago

aaleino commented 1 year ago

There are some quick-and-dirty modifications included in the FTXUI version fetched by the current CMake -files, so I would have to see how to get past them. I intend to remove them and include the original FTXUI library instead, but this will take a while.

yurivict commented 1 year ago

FetchContent_Declare isn't the right way. git-submodules are designed to include custom revisions of github projects. git-submodule is better handled by package managers, while FetchContent_Declare isn't handled at all because downloads aren't allowed during package builds.