Changeset 121762 in webkit
- Timestamp:
- Jul 3, 2012 6:43:13 AM (12 years ago)
- Location:
- trunk
- Files:
-
- 11 edited
Legend:
- Unmodified
- Added
- Removed
-
trunk/Source/JavaScriptCore/API/JSBase.h
r81135 r121762 72 72 #define JS_EXPORT __attribute__((visibility("default"))) 73 73 #elif defined(WIN32) || defined(_WIN32) || defined(_WIN32_WCE) || defined(__CC_ARM) || defined(__ARMCC__) 74 #if defined(BUILDING_JavaScriptCore) || defined( BUILDING_WTF)74 #if defined(BUILDING_JavaScriptCore) || defined(STATICALLY_LINKED_WITH_JavaScriptCore) 75 75 #define JS_EXPORT __declspec(dllexport) 76 76 #else -
trunk/Source/JavaScriptCore/ChangeLog
r121717 r121762 1 2012-07-03 Jocelyn Turcotte <jocelyn.turcotte@nokia.com> Joel Dillon <joel.dillon@codethink.co.uk> 2 3 [Qt][Win] Fix broken QtWebKit5.lib linking 4 https://bugs.webkit.org/show_bug.cgi?id=88321 5 6 Reviewed by Kenneth Rohde Christiansen. 7 8 The goal is to have different ports build systems define STATICALLY_LINKED_WITH_WTF 9 when building JavaScriptCore, if both are packaged in the same DLL, instead 10 of relying on the code to handle this. 11 The effects of BUILDING_* and STATICALLY_LINKED_WITH_* are currently the same 12 except for a check in Source/JavaScriptCore/config.h. 13 14 Keeping the old way for the WX port as requested by the port's contributors. 15 For non-Windows ports there is no difference between IMPORT and EXPORT, no 16 change is needed. 17 18 * API/JSBase.h: 19 JS symbols shouldn't be included by WTF objects anymore. Remove the export when BUILDING_WTF. 20 * JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCoreCommon.vsprops: 21 Make sure that JavaScriptCore uses import symbols of WTF for the Win port. 22 * runtime/JSExportMacros.h: 23 1 24 2012-07-02 Filip Pizlo <fpizlo@apple.com> 2 25 -
trunk/Source/JavaScriptCore/JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCoreCommon.vsprops
r120745 r121762 8 8 Name="VCCLCompilerTool" 9 9 AdditionalIncludeDirectories=""$(ConfigurationBuildDir)\obj\JavaScriptCore\DerivedSources\";../../;../../API/;../../parser/;../../bytecompiler/;../../dfg/;../../disassembler;../../jit/;../../llint/;../../runtime/;../../tools/;../../bytecode/;../../interpreter/;../../wtf/;../../profiler;../../assembler/;../../debugger/;../../heap/;"$(WebKitLibrariesDir)\include";"$(WebKitLibrariesDir)\include\private";"$(ConfigurationBuildDir)\include";"$(ConfigurationBuildDir)\include\JavaScriptCore";"$(ConfigurationBuildDir)\include\private";"$(ConfigurationBuildDir)\include\private\JavaScriptCore";"$(WebKitLibrariesDir)\include\pthreads"" 10 PreprocessorDefinitions=" __STD_C"10 PreprocessorDefinitions="STATICALLY_LINKED_WITH_WTF;__STD_C" 11 11 ForcedIncludeFiles="ICUVersion.h" 12 12 /> -
trunk/Source/JavaScriptCore/runtime/JSExportMacros.h
r103688 r121762 37 37 #if USE(EXPORT_MACROS) 38 38 39 #if defined(BUILDING_JavaScriptCore) 39 #if defined(BUILDING_JavaScriptCore) || defined(STATICALLY_LINKED_WITH_JavaScriptCore) 40 40 #define JS_EXPORT_PRIVATE WTF_EXPORT 41 41 #else … … 51 51 #if !PLATFORM(CHROMIUM) && OS(WINDOWS) && !defined(BUILDING_WX__) && !COMPILER(GCC) 52 52 53 #if defined(BUILDING_JavaScriptCore) 53 #if defined(BUILDING_JavaScriptCore) || defined(STATICALLY_LINKED_WITH_JavaScriptCore) 54 54 #define JS_EXPORTDATA __declspec(dllexport) 55 55 #else -
trunk/Source/WTF/ChangeLog
r121578 r121762 1 2012-07-03 Jocelyn Turcotte <jocelyn.turcotte@nokia.com> Joel Dillon <joel.dillon@codethink.co.uk> 2 3 [Qt][Win] Fix broken QtWebKit5.lib linking 4 https://bugs.webkit.org/show_bug.cgi?id=88321 5 6 Reviewed by Kenneth Rohde Christiansen. 7 8 Instead of letting a module's headers know which other modules depend on them, 9 have depending modules define explicitely that they want its symbols exported too. 10 11 JavaScriptCore should then be compiled with both BUILDING_JavaScriptCore and 12 STATICALLY_LINKED_WITH_WTF. 13 14 * wtf/ExportMacros.h: 15 1 16 2012-06-29 Tony Chang <tony@chromium.org> 2 17 -
trunk/Source/WTF/wtf/ExportMacros.h
r119857 r121762 66 66 #endif 67 67 68 // Currently WTF is embedded statically in JSCore, which exports69 // WTF symbols in the JSCore shared library.70 // Because of this, we need to make sure that we use WTF_EXPORT71 // when building JavaScriptCore as well as WTF.72 73 68 // FIXME: When all ports are using the export macros, we should replace 74 69 // WTF_EXPORTDATA with WTF_EXPORT_PRIVATE macros. 75 #if defined(BUILDING_WTF) || defined(BUILDING_JavaScriptCore)70 #if defined(BUILDING_WTF) || defined(STATICALLY_LINKED_WITH_WTF) || (PLATFORM(WX) && defined(BUILDING_JavaScriptCore)) 76 71 #define WTF_EXPORTDATA WTF_EXPORT 77 72 #else … … 82 77 83 78 #if !PLATFORM(CHROMIUM) && OS(WINDOWS) && !COMPILER(GCC) 84 #if defined(BUILDING_WTF) || defined( BUILDING_JavaScriptCore)79 #if defined(BUILDING_WTF) || defined(STATICALLY_LINKED_WITH_WTF) 85 80 #define WTF_EXPORTDATA __declspec(dllexport) 86 81 #else … … 99 94 #endif // USE(EXPORT_MACROS) 100 95 101 #if defined(BUILDING_WTF) || defined(BUILDING_JavaScriptCore)96 #if defined(BUILDING_WTF) || defined(STATICALLY_LINKED_WITH_WTF) || (PLATFORM(WX) && defined(BUILDING_JavaScriptCore)) 102 97 #define WTF_EXPORT_PRIVATE WTF_EXPORT 103 98 #else -
trunk/Source/WebCore/ChangeLog
r121756 r121762 1 2012-07-03 Jocelyn Turcotte <jocelyn.turcotte@nokia.com> Joel Dillon <joel.dillon@codethink.co.uk> 2 3 [Qt][Win] Fix broken QtWebKit5.lib linking 4 https://bugs.webkit.org/show_bug.cgi?id=88321 5 6 Reviewed by Kenneth Rohde Christiansen. 7 8 * platform/PlatformExportMacros.h: 9 1 10 2012-07-03 Philip Rogers <pdr@google.com> 2 11 -
trunk/Source/WebCore/platform/PlatformExportMacros.h
r102869 r121762 36 36 #if USE(EXPORT_MACROS) 37 37 38 #if defined(BUILDING_WebCore) || defined(BUILDING_WebKit) 38 #if defined(BUILDING_WebCore) || defined(BUILDING_WebKit) || \ 39 defined(STATICALLY_LINKED_WITH_WebCore) || defined(STATICALLY_LINKED_WITH_WebKit) 39 40 #define WEBKIT_EXPORTDATA WTF_EXPORT 40 41 #else … … 46 47 #if !PLATFORM(CHROMIUM) && OS(WINDOWS) && !defined(BUILDING_WX__) && !COMPILER(GCC) 47 48 48 #if defined(BUILDING_WebCore) || defined(BUILDING_WebKit) 49 #if defined(BUILDING_WebCore) || defined(BUILDING_WebKit) || \ 50 defined(STATICALLY_LINKED_WITH_WebCore) || defined(STATICALLY_LINKED_WITH_WebKit) 49 51 #define WEBKIT_EXPORTDATA __declspec(dllexport) 50 52 #else -
trunk/Tools/ChangeLog
r121760 r121762 1 2012-07-03 Jocelyn Turcotte <jocelyn.turcotte@nokia.com> Joel Dillon <joel.dillon@codethink.co.uk> 2 3 [Qt][Win] Fix broken QtWebKit5.lib linking 4 https://bugs.webkit.org/show_bug.cgi?id=88321 5 6 Reviewed by Kenneth Rohde Christiansen. 7 8 On windows the import/export definition is part of the symbol's signature. 9 Define STATICALLY_LINKED_WITH_$$library for each dependend module 10 that is being linked statically to make sure that they can be linked together 11 later on. 12 13 Also do not compile Assertions.cpp in DumpRenderTree anymore since all the 14 used symbols are exported and it would cause a duplicate symbols error. 15 16 * DumpRenderTree/qt/DumpRenderTree.pro: 17 * qmake/mkspecs/features/default_post.prf: 18 1 19 2012-07-03 Szilard Ledan <szledan@inf.u-szeged.hu> 2 20 -
trunk/Tools/DumpRenderTree/qt/DumpRenderTree.pro
r118155 r121762 38 38 39 39 SOURCES += \ 40 $${ROOT_WEBKIT_DIR}/Source/WTF/wtf/Assertions.cpp \41 40 $$PWD/../WorkQueue.cpp \ 42 41 DumpRenderTreeQt.cpp \ -
trunk/Tools/qmake/mkspecs/features/default_post.prf
r121545 r121762 213 213 } 214 214 215 # Anything not linking dynamically to QtWebKit should make sure to have its export 216 # macros syncrhonized with the code that it will be linked with statically. 217 !contains(QT, webkit) { 218 DEFINES += STATICALLY_LINKED_WITH_$$library 219 } 220 215 221 LIBS = $$existing_libs $$LIBS 216 222 }
Note: See TracChangeset
for help on using the changeset viewer.