Github user andschwa commented on the issue:
https://github.com/apache/zookeeper/pull/306
> Can you please describe what kinds of test / integration test you did on
windows?
Unfortunately, we don't have the unit tests building on Windows (under any
tools). So I ensured I could reliably build the project with CMake and get the
expected outputs. Moreover, I have a [zk-test
branch](https://github.com/andschwa/mesos/tree/zk-test) in Mesos where I
updated our system to apply these patches to the 3.5.2 release, and build with
them (verifying that the second patch resolved
[MESOS-7541](https://issues.apache.org/jira/browse/MESOS-7541)). I verified I
could link to the libraries as expected, and ran some tests (though I'd like to
give a full `mesos-test` pass of our entire suite).
Furthermore, we have been building and using the first patch (the addition
of the CMake build, but not the re-port changes) since [April
25](https://github.com/apache/mesos/commit/6e64ffaca365ed1e28256d7cb87bf9e1af626a75)
with no problems; so I'd say the CMake system is thoroughly tested for Windows
(at least in its previous form, but only some cosmetics, e.g.
TODOs/comments/default option for building unit tests, were changed when I
updated this PR).
> Can you please update the pull request description by adding a brief
description on what this patch is for and how it did it. The pull request
description will be part of commit message, and it's good to have a informative
commit message.
Wouldn't you rather use the two commit messages,
a58a5a95aef9cafc267a1b4a2bdb37f9e9e26363 and
c550a3e3babcaa3b3891280ac2d61b89fd294d06 as-is (those should be two links to
the commits with their respective messages, but the ASF bot might not copy them
since they're automatic from GitHub)? These are the patches as I'd commit them
(as in, I wouldn't squash them into one, since they're resolving two bugs).
> This pull request is targeting master, which is not going to be released
soon. branch-3.5 and branch-3.4 are branches for next releases, are you going
to send separate pull requests to those branches, or you are fine just with
this merged into master?
I would be happy to provide backports of these two patches, especially for
branch-3.5, so that I can remove the manual patch step in Mesos's build process
as soon as possible :D. But first, let's figure out the patch messages. I could
copy them into the PR description, but then I'm afraid it's going to be
squashed as one patch instead of the two logical patches. I could replace this
PR with two separate PRs, one for each patch, and then repeat that for the two
branches; but I'm not sure you want that either.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
