search

ss7m / paleofetch

neofetch, but written in C
MIT License
167 stars 49 forks source link

Add CMake #5

Closed otreblan closed 4 years ago

otreblan commented 4 years ago

The Makefile was removed

otreblan commented 4 years ago

Also added to the aur, but until this is merged it will use my fork because CMake.

ss7m commented 4 years ago

I'm not an experienced C programmer- what advantage does cmake have over regular make, especially for a project as small as this?

otreblan commented 4 years ago

CMake is a build system generator. It can be used for more than making Makefiles. https://stackoverflow.com/questions/25789644/difference-between-using-makefile-and-cmake-to-compile-the-code

ss7m commented 4 years ago

Would you mind also updating the README to reflect the changed installation process?

otreblan commented 4 years ago

ok

ss7m commented 4 years ago

I appreciate your taking interest in my project, but I'm still not convinced there's much to gain from using CMake, versus what I perceive to be the added complexity. Considering that the source code consists of 1 file, and that this change will make it harder for me to maintain this in the future, I don't think I'm going to merge this at this time.

Titaniumtown commented 4 years ago

can this pull request be re-opened?