Make use of -lppapi explicit to allow for correct library order.

Make use of -lppapi explicit to allow for correct library order.

This CL removes the implicit -lppapi and pthread in the Makefiles.  Since ppapi lib 
does not exist on the host (on NaCl it's actually a linker script), we need to 
create a bogus ppapi.lib for host Pepper development.  In addition, on Windows we
do not have a pthread library so we create a mini pthread library to support the
examples.


This CL also fixes tries which are broken on Windows.

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=148006

[binji, 2012-07-23 18:47:14]

lgtm

http://codereview.chromium.org/10815039/diff/15001/native_client_sdk/src/exam...
File native_client_sdk/src/examples/file_io/example.dsc (right):

http://codereview.chromium.org/10815039/diff/15001/native_client_sdk/src/exam...
native_client_sdk/src/examples/file_io/example.dsc:2: 'TOOLS': ['newlib',
'glibc', 'pnacl'],
Is this broken on win host?

http://codereview.chromium.org/10815039/diff/15001/native_client_sdk/src/libr...
File native_client_sdk/src/libraries/ppapi/ppapi_externs.c (right):

http://codereview.chromium.org/10815039/diff/15001/native_client_sdk/src/libr...
native_client_sdk/src/libraries/ppapi/ppapi_externs.c:2: const char
*g_BogusSymbol;
Maybe a comment here explaining why this is necessary...

http://codereview.chromium.org/10815039/diff/15001/native_client_sdk/src/libr...
File native_client_sdk/src/libraries/pthread/pthread.c (right):

http://codereview.chromium.org/10815039/diff/15001/native_client_sdk/src/libr...
native_client_sdk/src/libraries/pthread/pthread.c:39: }
nit: } else {
or in this case, could just remove the braces altogether.

http://codereview.chromium.org/10815039/diff/15001/native_client_sdk/src/libr...
File native_client_sdk/src/libraries/pthread/pthread.h (right):

http://codereview.chromium.org/10815039/diff/15001/native_client_sdk/src/libr...
native_client_sdk/src/libraries/pthread/pthread.h:10: * Implementation of
pthread.h for building the SDK natively Windows.
natively on Windows

[noelallen1, 2012-07-23 21:44:54]

http://codereview.chromium.org/10815039/diff/15001/native_client_sdk/src/exam...
File native_client_sdk/src/examples/file_io/example.dsc (right):

http://codereview.chromium.org/10815039/diff/15001/native_client_sdk/src/exam...
native_client_sdk/src/examples/file_io/example.dsc:2: 'TOOLS': ['newlib',
'glibc', 'pnacl'],
Yeah, for stuff not checked yet.

On 2012/07/23 18:47:14, binji wrote:
> Is this broken on win host?

http://codereview.chromium.org/10815039/diff/15001/native_client_sdk/src/libr...
File native_client_sdk/src/libraries/pthread/pthread.c (right):

http://codereview.chromium.org/10815039/diff/15001/native_client_sdk/src/libr...
native_client_sdk/src/libraries/pthread/pthread.c:39: }
On 2012/07/23 18:47:14, binji wrote:
> nit: } else {
> or in this case, could just remove the braces altogether.

Done.

http://codereview.chromium.org/10815039/diff/15001/native_client_sdk/src/libr...
File native_client_sdk/src/libraries/pthread/pthread.h (right):

http://codereview.chromium.org/10815039/diff/15001/native_client_sdk/src/libr...
native_client_sdk/src/libraries/pthread/pthread.h:10: * Implementation of
pthread.h for building the SDK natively Windows.
On 2012/07/23 18:47:14, binji wrote:
> natively on Windows

Done.

[commit-bot: I haz the power, 2012-07-23 22:55:11]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/noelallen@chromium.org/10815039/8004

[commit-bot: I haz the power, 2012-07-24 00:38:06]

Change committed as 148006