Skip to content

[DO NOT MERGE] aggregate all changes necessary for Swift to build #37

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 25 commits into from

Conversation

QuietMisdreavus
Copy link

@QuietMisdreavus QuietMisdreavus commented Mar 9, 2022

This is a dummy PR branch that aggregates the work necessary to build swift-cmark/gfm with the Swift compiler:

More changes may be added on as more issues are found.

- all the compilers we support have __has_include(), so use that instead
  of checking it in cmake
- nothing in the library uses HAS__ATTRIBUTE__, so remove it
the win32 API uses some architecture-specific defines, but they're not
defined by the MSVC compiler - they're defined directly in windows.h
based on symbols defined by the MSVC compiler. rather than setting the
architecture symbols ourselves, just import windows.h as a whole.

this also needed to disable the short-name symbols in cmark-gfm.h,
because there are some clashing symbols in win32 headers.
@QuietMisdreavus
Copy link
Author

This PR now includes #40, which aggregates some CMake fixes for Windows and Linux.

@QuietMisdreavus
Copy link
Author

This PR now includes #41, which fixes a couple functions in api_test to build in the toolchain on Windows.

@QuietMisdreavus QuietMisdreavus force-pushed the QuietMisdreavus/swift-build branch from e40b59a to 59e0b3c Compare March 15, 2022 22:44
@QuietMisdreavus QuietMisdreavus force-pushed the QuietMisdreavus/swift-build branch from 59e0b3c to fe486ef Compare March 16, 2022 19:12
on Windows SDK version 10.0.17763.0, the SDK headers spuriously trip
this warning. since swift-cmark itself doesn't emit this warning on its
own, diabling this warning will allow it to be built with this SDK.
@QuietMisdreavus
Copy link
Author

This PR now includes #42, which fixes a problem when building with the Windows SDK used on the Swift CI servers.

@QuietMisdreavus
Copy link
Author

All the listed PRs that were originally combined into this branch have been merged into gfm, so i'm closing this PR.

@QuietMisdreavus QuietMisdreavus deleted the QuietMisdreavus/swift-build branch March 18, 2022 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant