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

Issue 1038753002: Disable LLVM's use of strerror_s on Windows (Closed)

Created:
5 years, 9 months ago by Derek Schuff
Modified:
5 years, 9 months ago
Reviewers:
JF
CC:
native-client-reviews_googlegroups.com
Base URL:
https://chromium.googlesource.com/native_client/src/native_client.git@master
Target Ref:
refs/heads/master
Project:
nacl
Visibility:
Public.

Description

Disable LLVM's use of strerror_s on Windows If available, LLVM uses Windows' strerror_s function for printing errors. Because mingw builds link dynamically against msvcrt.dll, clang will not run on systems (XP) which lack this function in their version of the dll. Since we still have buildbots which run SCons on XP, we want clang to run on XP, and strerror_s seems to be the only missing dependence. NOTRY=true BUG= https://code.google.com/p/nativeclient/issues/detail?id=3936 R=jfb@chromium.org Committed: https://chromium.googlesource.com/native_client/src/native_client/+/bd9bca36ad7a42e512245cf298c0f27397073cb8

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -1 line) Patch
M toolchain_build/toolchain_build_pnacl.py View 1 chunk +2 lines, -1 line 0 comments Download

Messages

Total messages: 7 (0 generated)
Derek Schuff
tested that a binary built this way runs on XP, so will use this workaround.
5 years, 9 months ago (2015-03-26 00:07:06 UTC) #1
JF
You should instead define WINVER to _WIN32_WINNT_WINXP (0x0501).
5 years, 9 months ago (2015-03-26 00:32:44 UTC) #2
JF
On 2015/03/26 00:32:44, JF wrote: > You should instead define WINVER to _WIN32_WINNT_WINXP (0x0501). Oh ...
5 years, 9 months ago (2015-03-26 00:33:55 UTC) #3
Derek Schuff
On 2015/03/26 00:33:55, JF wrote: > On 2015/03/26 00:32:44, JF wrote: > > You should ...
5 years, 9 months ago (2015-03-26 16:02:56 UTC) #4
Derek Schuff
On 2015/03/26 16:02:56, Derek Schuff wrote: > On 2015/03/26 00:33:55, JF wrote: > > On ...
5 years, 9 months ago (2015-03-26 16:37:27 UTC) #5
JF
lgtm, but I do think we want to define WINVER too. Separate change?
5 years, 9 months ago (2015-03-26 16:47:47 UTC) #6
Derek Schuff
5 years, 9 months ago (2015-03-26 16:52:42 UTC) #7
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
bd9bca36ad7a42e512245cf298c0f27397073cb8 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698