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

Issue 8936006: Fix MinGW build. (Closed)

Created:
9 years ago by Yang
Modified:
8 years, 11 months ago
Reviewers:
yext
CC:
v8-dev
Visibility:
Public.

Description

Fix MinGW build. BUG=v8:1724

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -0 lines) Patch
M src/win32-headers.h View 1 chunk +2 lines, -0 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Yang
Please confirm that this fixes the MinGW build (since it apparently has been broken in ...
9 years ago (2011-12-13 14:12:06 UTC) #1
yext
9 years ago (2011-12-13 14:34:02 UTC) #2
On 2011/12/13 14:12:06, Yang wrote:
> Please confirm that this fixes the MinGW build (since it apparently has been
> broken in r9471).

Yes this patch fixed the problem of r9471. Thanks.

However it's still unable to build a shared library under MinGW, it will have
some undefined reference errors. The solution is to add library references
"winmm" and "ws2_32" to MinGW shared library settings. So can you please write a
patch for this issue? Thanks.

Powered by Google App Engine
This is Rietveld 408576698