Azure / azure-event-hubs-c

C client library for Azure Event Hubs https://azure.microsoft.com/services/event-hubs
Other
7 stars 20 forks source link

Visual Studio 14 Win64 compilation issue. #7

Open piotrzaremba opened 7 years ago

piotrzaremba commented 7 years ago

I am having issues compiling the library using /TP.

I use the cmake-gui to create a default Visual Studio 14 Win64 solution. I select properties -> C/C++/Advanced/Compile As within Visual Studio. I receive several compilation issues especially in the shared util code base. Any suggestions?

dcristoloveanu commented 7 years ago

I created PR #17 to fix the issues that the EH SDK repo has when building with /TP only for the production code (not for the tests, that is a different story). The rest of the issues are in C shared and uamqp. I can also fix the ones in uAMQP, the C shared ones will have to have an issue filed there and addressed.

Thanks, /Dan

dcristoloveanu commented 7 years ago

@piotrzaremba Also created PRs on uamqp: https://github.com/Azure/azure-uamqp-c/pull/85 and C shared utility: https://github.com/Azure/azure-c-shared-utility/pull/70

Hope to get those in and then the production code will compile with /TP. For unit tests more work is needed.