mapi/glapi: Fix dll linkage of GLES1 symbols.
authorJosé Fonseca <jfonseca@vmware.com>
Tue, 25 Nov 2014 23:06:25 +0000 (23:06 +0000)
committerJosé Fonseca <jfonseca@vmware.com>
Wed, 26 Nov 2014 20:31:07 +0000 (20:31 +0000)
This fixes several MSVC warnings like:

  warning C4273: 'glClearColorx' : inconsistent dll linkage

In fact, we should avoid using `declspec(dllexport)` altogether, and use
exclusively the .DEF instead, which gives more precise control of which
symbols must be exported, but all the public GL/GLES headers practically
force us to pick between `declspec(dllexport)` or
`declspec(dllimport)`.

Cc: "10.4" <mesa-stable@lists.freedesktop.org>
Reviewed-by: Brian Paul <brianp@vmware.com>
Reviewed-by: Roland Scheidegger <sroland@vmware.com>
src/mapi/glapi/SConscript

index bc1c43aa865274710b61aba019887baf8d615a22..97ebfe638aeca58afc269ff703e3570242c9e6de 100644 (file)
@@ -16,6 +16,7 @@ if env['platform'] == 'windows':
     env.Append(CPPDEFINES = [
         '_GDI32_', # prevent gl* being declared __declspec(dllimport) in MS headers
         'BUILD_GL32', # declare gl* as __declspec(dllexport) in Mesa headers
+        'KHRONOS_DLL_EXPORTS', # declare gl* as __declspec(dllexport) in Khronos headers
     ])
     if env['gles']:
         env.Append(CPPDEFINES = ['_GLAPI_DLL_EXPORTS'])