What kind of Linux and Mac testing did you do on this branch? Do we have a Jenkins job to test this? I don't think we do, but Ruth would know more.

I'm still looking at this.

At present, the intend of this branch and the changes is not to run on Linux or Mac. As first step it's targetted to run on Windows platform. I will not merging it master at present but the changes may live on this branch or on origin/windows which already exists but is out-dated - last updated almost more than 2 years back.

When it goes to master (later?), it will operational on Linux.

Ok. The pull request says you want to merge to master, which seems to be a mistake unless you have tested on Linux.

Most of the 321 file changes are in cppunit and other library functions. Is that true? I didn't review these. I only looked at the changes which seemed to be in VoltDB code. Those seemed to be right to me.

Thanks Bill for reviewing the changes. The pull request has been set-up against master so that it's easier to read the changes happening to master logic. The current branch as is can't go in master at present as this has not been tested to run on Linux.
Correct, lof of newly added files are cppunit ones (will be good to delete the sources of these at some point and just use the library and headers) and are third party sources which can be skipped. They are located in ./include/cppunit and zip files in ./third_party.

review done.

Thanks for the reviewing the changes. I have pushed changes on the review feedback along with clean-up of sources for libevent as it does seem to be needed for Windows C++ client - purpose based on the how project is structured, it looks like libraries are only needed. I have archieved the sources as it contains Windows project too in it so that in future, if needed, the new libraries can be generated.