Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1)

Unified Diff: third_party/libxslt/README.chromium

Issue 1770093002: Avoid illegal definition of snprintf in VS 2015 (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixed typo Created 4 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « third_party/libxml/win32/config.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/libxslt/README.chromium
diff --git a/third_party/libxslt/README.chromium b/third_party/libxslt/README.chromium
index 138e4eed58242b15d1c8dd21a52e3b8ae56e62f2..702c8e3e1334b9fd0fde8756ebd11c3033392daf 100644
--- a/third_party/libxslt/README.chromium
+++ b/third_party/libxslt/README.chromium
@@ -34,6 +34,8 @@ On Windows:
In win32\ run
cscript //E:jscript configure.js compiler=msvc iconv=no xslt_debug=no mem_debug=no debugger=no modules=no
and copy config.h to Linux in win32/
+Patch win32/config.h to add "#if _MSC_VER < 1900 // Cannot define this in VS 2015 and above!"
scottmg 2016/03/07 20:32:36 This should be in the libxml readme, not the libxs
brucedawson 2016/03/07 21:17:07 D'oh! Done.
+and "#endif around the #define of snprintf.
Remove:
doc/
« no previous file with comments | « third_party/libxml/win32/config.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698