Hi Everyone,

We caught an issue under Cygwin 2.4.1 with GCC 5.3.0. The issue was a 
missing include, and it was easy enough to clear. Also see 
https://github.com/weidai11/cryptopp/issues/137 .

While examining the issue, this caught my eye (from config.h, around line 
505:

504
505    #if defined(_WIN32) || defined(__CYGWIN__)
506    #define CRYPTOPP_WIN32_AVAILABLE
507    #endif
...


I believe that define should be __MINGW32__, and not __CYGWIN__:

$ git diff
diff --git a/config.h b/config.h
index 41103fe..177babd 100644
--- a/config.h
+++ b/config.h
@@ -502,7 +502,7 @@ NAMESPACE_END
 
 #ifndef NO_OS_DEPENDENCE
 
-#if defined(_WIN32) || defined(__CYGWIN__)
+#if defined(_WIN32) || defined(__MINGW__)
 #define CRYPTOPP_WIN32_AVAILABLE
 #endif

Would folks who use Cygwin and MinGW make the change and report back?

-- 
-- 
You received this message because you are subscribed to the "Crypto++ Users" 
Google Group.
To unsubscribe, send an email to [email protected].
More information about Crypto++ and this group is available at 
http://www.cryptopp.com.
--- 
You received this message because you are subscribed to the Google Groups 
"Crypto++ Users" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to