Various fixes to support building within the toolchain (#4985)

* Add -Wno-int-conversion and -fPIC

* Move block runtime sources into separate ignored folder

* Install private static libs in static swift build

* Don't include stdlib toolchain rpath

* Autolink static libraries in static swift build

* Bump WINVER/_WIN32_WINNT to Windows 10

* Repair package manifest

Author: jmschonfeld

CMakeLists.txt

@@ -46,6 +46,8 @@ set(CMAKE_LIBRARY_OUTPUT_DIRECTORY ${CMAKE_BINARY_DIR}/lib)
 set(CMAKE_RUNTIME_OUTPUT_DIRECTORY ${CMAKE_BINARY_DIR}/bin)
 set(CMAKE_Swift_MODULE_DIRECTORY ${CMAKE_BINARY_DIR}/swift)
 
+set(CMAKE_POSITION_INDEPENDENT_CODE YES)
+
 # Fetchable dependcies
 include(FetchContent)
 if (_SwiftFoundationICU_SourceDIR)
@@ -98,6 +100,7 @@ list(APPEND _Foundation_common_build_flags
     "-Wno-unused-variable"
     "-Wno-unused-function"
     "-Wno-microsoft-enum-forward-reference"
+    "-Wno-int-conversion"
     "-fconstant-cfstrings"
     "-fexceptions" # TODO: not on OpenBSD
     "-fdollars-in-identifiers"

Package.swift

@@ -20,6 +20,7 @@ let coreFoundationBuildSettings: [CSetting] = [
         "-Wno-unused-variable",
         "-Wno-unused-function",
         "-Wno-microsoft-enum-forward-reference",
+        "-Wno-int-conversion",
         "-fconstant-cfstrings",
         "-fexceptions", // TODO: not on OpenBSD
         "-fdollars-in-identifiers",
@@ -29,13 +30,12 @@ let coreFoundationBuildSettings: [CSetting] = [
         "\(Context.packageDirectory)/Sources/CoreFoundation/internalInclude/CoreFoundation_Prefix.h",
         // /EHsc for Windows
     ]),
-    .unsafeFlags(["-I/usr/lib/swift"], .when(platforms: [.linux, .android])) // dispatch
+    .unsafeFlags(["-I/usr/lib/swift/Block"], .when(platforms: [.linux, .android])) // dispatch
 ]
 
 // For _CFURLSessionInterface, _CFXMLInterface
 let interfaceBuildSettings: [CSetting] = [
     .headerSearchPath("../CoreFoundation/internalInclude"),
-    .headerSearchPath("../CoreFoundation/include"),
     .define("DEBUG", .when(configuration: .debug)),
     .define("CF_BUILDING_CF"),
     .define("DEPLOYMENT_ENABLE_LIBDISPATCH"),
@@ -50,14 +50,15 @@ let interfaceBuildSettings: [CSetting] = [
         "-Wno-unused-variable",
         "-Wno-unused-function",
         "-Wno-microsoft-enum-forward-reference",
+        "-Wno-int-conversion",
         "-fconstant-cfstrings",
         "-fexceptions", // TODO: not on OpenBSD
         "-fdollars-in-identifiers",
         "-fno-common",
         "-fcf-runtime-abi=swift"
         // /EHsc for Windows
     ]),
-    .unsafeFlags(["-I/usr/lib/swift"], .when(platforms: [.linux, .android])) // dispatch
+    .unsafeFlags(["-I/usr/lib/swift/Block"], .when(platforms: [.linux, .android])) // dispatch
 ]
 
 let swiftBuildSettings: [SwiftSetting] = [
@@ -78,11 +79,11 @@ let package = Package(
     dependencies: [
         .package(
             url: "https://github.com/apple/swift-foundation-icu",
-            from: "0.0.7"
+            from: "0.0.8"
         ),
         .package(
            url: "https://github.com/apple/swift-foundation",
-           revision: "3297fb33b49ba2d1161ba12757891c7b91c73a3e"
+           revision: "ef8a7787c355edae3c142e4dff8767d05a32c51f"
         ),
     ],
     targets: [
@@ -91,7 +92,7 @@ let package = Package(
             dependencies: [
                 .product(name: "FoundationEssentials", package: "swift-foundation"),
                 .product(name: "FoundationInternationalization", package: "swift-foundation"),
-                "_CoreFoundation"
+                "CoreFoundation"
             ],
             path: "Sources/Foundation",
             swiftSettings: swiftBuildSettings
@@ -100,8 +101,8 @@ let package = Package(
             name: "FoundationXML",
             dependencies: [
                 .product(name: "FoundationEssentials", package: "swift-foundation"),
-                .targetItem(name: "Foundation", condition: nil),
-                "_CoreFoundation",
+                "Foundation",
+                "CoreFoundation",
                 "_CFXMLInterface"
             ],
             path: "Sources/FoundationXML",
@@ -111,25 +112,26 @@ let package = Package(
             name: "FoundationNetworking",
             dependencies: [
                 .product(name: "FoundationEssentials", package: "swift-foundation"),
-                .targetItem(name: "Foundation", condition: nil),
-                "_CoreFoundation",
+                "Foundation",
+                "CoreFoundation",
                 "_CFURLSessionInterface"
             ],
             path: "Sources/FoundationNetworking",
             swiftSettings:swiftBuildSettings
         ),
         .target(
-            name: "_CoreFoundation",
+            name: "CoreFoundation",
             dependencies: [
                 .product(name: "_FoundationICU", package: "swift-foundation-icu"),
             ],
             path: "Sources/CoreFoundation",
+            exclude: ["BlockRuntime"],
             cSettings: coreFoundationBuildSettings
         ),
         .target(
             name: "_CFXMLInterface",
             dependencies: [
-                "_CoreFoundation",
+                "CoreFoundation",
                 "Clibxml2",
             ],
             path: "Sources/_CFXMLInterface",
@@ -138,7 +140,7 @@ let package = Package(
         .target(
             name: "_CFURLSessionInterface",
             dependencies: [
-                "_CoreFoundation",
+                "CoreFoundation",
                 "Clibcurl",
             ],
             path: "Sources/_CFURLSessionInterface",
@@ -163,15 +165,15 @@ let package = Package(
         .executableTarget(
             name: "plutil",
             dependencies: [
-                .targetItem(name: "Foundation", condition: nil)
+                "Foundation"
             ]
         ),
         .executableTarget(
             name: "xdgTestHelper",
             dependencies: [
-                .targetItem(name: "Foundation", condition: nil),
-                .targetItem(name: "FoundationXML", condition: nil),
-                .targetItem(name: "FoundationNetworking", condition: nil)
+                "Foundation",
+                "FoundationXML",
+                "FoundationNetworking"
             ]
         ),
         .target(
@@ -181,16 +183,16 @@ let package = Package(
             // We believe Foundation is the only project that needs to take this rather drastic measure.
             name: "XCTest",
             dependencies: [
-                .targetItem(name: "Foundation", condition: nil)
+                "Foundation"
             ],
             path: "Sources/XCTest"
         ),
         .testTarget(
             name: "TestFoundation",
             dependencies: [
-                .targetItem(name: "Foundation", condition: nil),
-                .targetItem(name: "FoundationXML", condition: nil),
-                .targetItem(name: "FoundationNetworking", condition: nil),
+                "Foundation",
+                "FoundationXML",
+                "FoundationNetworking",
                 .targetItem(name: "XCTest", condition: .when(platforms: [.linux])),
                 "xdgTestHelper"
             ],

Sources/CoreFoundation/data.c renamed to Sources/CoreFoundation/BlockRuntime/data.c

@@ -97,8 +97,6 @@ add_library(CoreFoundation STATIC
     CFUUID.c
     CFWindowsUtilities.c
     CFXMLPreferencesDomain.c
-    data.c
-    runtime.c
     uuid.c)
 
 target_include_directories(CoreFoundation
@@ -123,3 +121,10 @@ install(DIRECTORY
             include/
         DESTINATION
             lib/swift/CoreFoundation)
+
+if(NOT BUILD_SHARED_LIBS)
+    install(TARGETS CoreFoundation
+        ARCHIVE DESTINATION lib/swift_static/${SWIFT_SYSTEM_NAME}
+        LIBRARY DESTINATION lib/swift_static/${SWIFT_SYSTEM_NAME}
+        RUNTIME DESTINATION bin)
+endif()

Sources/CoreFoundation/internalInclude/Block.h renamed to Sources/CoreFoundation/BlockRuntime/include/Block.h

@@ -119,11 +119,11 @@ typedef char * Class;
 #define WIN32_LEAN_AND_MEAN
 
 #ifndef WINVER
-#define WINVER  0x0601
+#define WINVER  0x0A00
 #endif
 
 #ifndef _WIN32_WINNT
-#define _WIN32_WINNT 0x0601
+#define _WIN32_WINNT 0x0A00
 #endif
 
 // The order of these includes is important

Sources/CoreFoundation/internalInclude/Block_private.h renamed to Sources/CoreFoundation/BlockRuntime/include/Block_private.h

@@ -155,6 +155,16 @@ target_link_libraries(Foundation
         FoundationEssentials
         FoundationInternationalization)
 
+if(NOT BUILD_SHARED_LIBS)
+    target_compile_options(Foundation PRIVATE
+        "SHELL:$<$<COMPILE_LANGUAGE:Swift>:-Xfrontend -public-autolink-library -Xfrontend CoreFoundation>")
+    target_compile_options(Foundation PRIVATE
+        "SHELL:$<$<COMPILE_LANGUAGE:Swift>:-Xfrontend -public-autolink-library -Xfrontend _FoundationICU>")
+endif()
+
+target_link_options(Foundation PRIVATE
+    "SHELL:-no-toolchain-stdlib-rpath")
+
 set_target_properties(Foundation PROPERTIES
     INSTALL_RPATH "$ORIGIN"
     BUILD_RPATH "$<TARGET_FILE_DIR:swiftDispatch>")

Sources/CoreFoundation/runtime.c renamed to Sources/CoreFoundation/BlockRuntime/runtime.c

@@ -56,6 +56,18 @@ target_link_libraries(FoundationNetworking
     PUBLIC
         Foundation)
 
+if(NOT BUILD_SHARED_LIBS)
+    target_compile_options(FoundationNetworking PRIVATE
+        "SHELL:$<$<COMPILE_LANGUAGE:Swift>:-Xfrontend -public-autolink-library -Xfrontend _CFURLSessionInterface>")
+    target_compile_options(FoundationNetworking PRIVATE
+        "SHELL:$<$<COMPILE_LANGUAGE:Swift>:-Xfrontend -public-autolink-library -Xfrontend CoreFoundation>")
+    target_compile_options(FoundationNetworking PRIVATE
+        "SHELL:$<$<COMPILE_LANGUAGE:Swift>:-Xfrontend -public-autolink-library -Xfrontend curl>")
+endif()
+
+target_link_options(FoundationNetworking PRIVATE
+    "SHELL:-no-toolchain-stdlib-rpath")
+
 set_target_properties(FoundationNetworking PROPERTIES
     INSTALL_RPATH "$ORIGIN")