ntop / n2n

Peer-to-peer VPN
GNU General Public License v3.0
6.14k stars 928 forks source link

Remove the non-maintained CMake build system. #1042

Closed hamishcoleman closed 1 year ago

hamishcoleman commented 2 years ago

It is difficult to maintain one build system, let alone two - especially if one build system is known to be better maintained and the other build system continues to be used - with bug and feature requests raised against it.

The maintainers are aware that there are people using the CMake system, but believe that the extra maintainability of only having one build system, combined with the knowledge that the current CI system demonstrates that all supported architectures are working with the Makefiles make the use of these Makefiles a more sustainable direction.

codecov-commenter commented 2 years ago

Codecov Report

Base: 20.74% // Head: 20.74% // No change to project coverage :thumbsup:

Coverage data is based on head (4322854) compared to base (06c489f). Patch has no changes to coverable lines.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## dev #1042 +/- ## ======================================= Coverage 20.74% 20.74% ======================================= Files 47 47 Lines 8447 8447 ======================================= Hits 1752 1752 Misses 6695 6695 ``` Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None)

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

fengdaolong commented 2 years ago

Nowadays, more and more programs use the cmake build system, its cross-platform aspect has advantages over make, and removing cmake in n2n is not a good choice.

Logan007 commented 2 years ago

I honestly have no idea how we can maintain the CMake system and how to always keep it in sync with the automake tools at each and every change.

As far as I can see, the main concern might be with Windows platform (for which we have/offer a solution)?

llfj commented 2 years ago

I always use CMAKE to cross compile N2N, which is much more convenient than MAKE. I do not wish to remove CMAKE.

hamishcoleman commented 2 years ago

On Wed, Aug 10, 2022 at 02:30:22AM -0700, llfj wrote:

I always use CMAKE to cross compile N2N, which is much more convenient than MAKE. I do not wish to remove CMAKE.

What do you need to improve the convenience of make. You see, I always use MAKE to cross compile N2N as it is much more convenient.

Logan007 commented 2 years ago

Also, who will maintain CMake here?