Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace Makefile with autoconf for builds/detection/dist release #10

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

fcartegnie
Copy link

The vc projects have to be checked as they might be broken.

@Conan-Kudo
Copy link

Why autoconf instead of CMake? It would make sense to use CMake to preserve a more portable build environment.

@tdaede
Copy link
Contributor

tdaede commented Aug 12, 2015

Autoconf is also portable, and works well with cross-compiling. There are already hand-maintained VS projects. Was there another target you were looking for with CMake?

@fcartegnie
Copy link
Author

If VS projects are maintained by hand, the main benefits of CMake goes away.
Autotools makes things easier for packagers, does cleanup... but is more complex.
Just a matter of choice. Not a strong request then.

@Conan-Kudo
Copy link

I use CLion and CodeBlocks for C/C++ development. The former actually uses CMake as its project data, and the latter is an export target for CMake. CMake also allows you to generate VS projects, Makefiles and Eclipse CDT project data.

CMake is dead easy for packagers to use (disclosure: I've made both RPM and Debian packages, and do make RPM packages for the Fedora Project). It's just as easy as autotools is, with the benefit of actually working properly in a native Windows environment for those who want that too.

@fcartegnie fcartegnie closed this Aug 13, 2015
@tdaede
Copy link
Contributor

tdaede commented Aug 13, 2015

Any reason you closed this? Are you working on an updated version?

@fcartegnie
Copy link
Author

As everone is pointing out that's no need, no need to merge then.
The 1st commit might have been useful though

@tdaede
Copy link
Contributor

tdaede commented Aug 13, 2015

Uh, I didn't read that as there being no need. The existing Makefile is insufficient.

@elfring
Copy link

elfring commented Aug 14, 2015

How do you think about a few extensions for this approach?

@fcartegnie fcartegnie reopened this Aug 16, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants