[ 
https://issues.apache.org/jira/browse/ZOOKEEPER-2756?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16013019#comment-16013019
 ] 

Michael Han commented on ZOOKEEPER-2756:
----------------------------------------

that is why I was asking: we either delete the old sln files if the change 
breaks them, or find a way to be compatible. I am +1 on the idea of removing 
old files completely. Removing the sln files will break apache jenkins windows 
build for ZK which requires those files, but I suspect nobody cares about it 
(because the build has been broken for a long time), but would be good to file 
a jira about it.

Can't find commit 065056b0b240b7bdff2eebe41db86a9a3ea6ecfc in apache/zookeeper 
repo.



> Add CMake build system for better cross-platform support
> --------------------------------------------------------
>
>                 Key: ZOOKEEPER-2756
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-2756
>             Project: ZooKeeper
>          Issue Type: Improvement
>          Components: build, c client
>    Affects Versions: 3.5.2
>         Environment: Windows and Linux
>            Reporter: Andrew Schwartzmeyer
>            Assignee: Andrew Schwartzmeyer
>              Labels: build, windows
>         Attachments: ZOOKEEPER-2756.patch
>
>
> The C bindings primary build system is Autotools. This obviously does not 
> work for Windows, and so the original port to Windows simply added a Visual 
> Studio solution to the project, splitting the build system. As new versions 
> of Visual Studio have come along, new (probably auto-converted) solutions 
> have come along (see zookeeper.sln vs zookeeper-vs2013.sln). When Mesos 
> started being ported to Windows, a Visual Studio 2015 solution was needed, 
> and the previous developer created yet another solution, and setup Mesos' 
> build to patch ZooKeeper and add the 2015 solution. Now Visual Studio 2017 
> was released, and in the process of moving Mesos ahead, I realized that I 
> would either have to make *yet another* converted solution for ZooKeeper. So 
> instead I tackled the root problem, and ported the Autotools build to CMake, 
> which is a meta-build system which generates files for the in-use platform 
> (whether it be Linux or Solaris or MacOS or Windows).
> NOTE: I already have this patch, and will submit it. It has a couple TODOs, 
> and some other things in it that were necessary for Mesos that may need to be 
> pulled into separate patches.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to