CMake Support

[liuchang0812]

Hi, @cmumford

Does it make sense if we add cmake support to leveldb? if the answer is YES, I will try to do it.

There are some useful LLVM tools likes clang-tidy/woboq that need cmake support. We will get code format automatically, static check and online code browser if here is CMakeLists.txt.

any comments are appreciated. thx

[Netzeband]

FYI: #475

[jasjuang]

CMake supported is much needed in order to integrate it into vcpkg

[cmumford]

Work on CMake support is underway.

[navidR]

@cmumford is it going to be supported on *nix platforms as well as Windows too?

Please support CMake on *nix as well as Windows.

[pwnall]

Windows support requires a separate effort, because we'll need replacements for the POSIX port and env. That being said, one of the benefits of CMake is the ability to support Windows down the line.

Sorry we have no plans to announce right now, but we are aware of the demand!

[andyleejordan]

@pwnall @cmumford I was watching the Windows Support thread, and just want to make myself available to this effort. I'm leading the port of Mesos to Windows, where we had to switch from Autotools to CMake as well, and in fact, leveldb is one of remaining requirements to bring the Mesos master component to Windows.

I've also previously patched ZooKeeper's C client library to support Windows with a new CMake build system, and for Mesos, wrote documentation covering how to maintain a good CMake build system (after I rewrote it).

Not knowing how familiar leveldb developers are with CMake, I very highly recommend reading this list of common anti-patterns and this modern guide to CMake.

Please feel free to bug me with any questions or work items you have, I'd be happy to help. I would especially like to help test the CMake build as its developed, as I'll be integrating it as a third-party dependency into my build system back at Mesos (using External_ProjectAdd).

[chfast]

Can you share the current status of this task? I can help or even do it from scratch.

[pwnall]

@chfast Thank you very much for the offer! Unfortunately, unless you're a Googler, it's highly unlikely that you can help.

I have a patch for the open-source repository here: https://github.com/pwnall/leveldb/tree/cmake

The work is blocked on integrating this changes into Google's internal source repository. Sorry, that's all I have to share right now.

[chfast]

Any estimations how long it can take?

[jasjuang]

Someone wrote a CMakeLists for leveldb on vcpkg https://github.com/Microsoft/vcpkg/blob/master/ports/leveldb/CMakeLists.txt, and it is fully working on Windows. Simply install leveldb with vcpkg install leveldb

[isaachier]

@pwnall thanks for the CMake support. I think that is good enough for me. On a related note, I wonder why you chose to redefine all of the if defined macros to if macros. CMake can do either (cmakedefine vs. cmakedefine01) and it would have less impact on the existing build system to keep as if defined.

[chfast]

The #if is better than #if defined for compile time options, because it handles a case like -DOPTION=0.