Skip to content
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

[CI] Migrate to Azure Pipelines #65

Merged
merged 3 commits into from
Apr 26, 2020
Merged

[CI] Migrate to Azure Pipelines #65

merged 3 commits into from
Apr 26, 2020

Conversation

flubshi
Copy link
Owner

@flubshi flubshi commented Apr 24, 2020

This Pull Request migrates CI from AppVeyor to Azure Pipelines. This follows kodi-pvr which also moved. Benefit: CI is much faster! YAML file (azure-pipelines.yml) from kodi-pvr was used for this - thanks!

@flubshi flubshi force-pushed the matrix_azure_ci branch 2 times, most recently from 16b80c9 to 5aa1ef4 Compare April 24, 2020 18:21
@AlwinEsch
Copy link
Collaborator

To fix you can add e.g. this:

--- a/include/rapidjson/rapidjson.h
+++ b/include/rapidjson/rapidjson.h
@@ -236,7 +236,7 @@
 #    define RAPIDJSON_ENDIAN RAPIDJSON_BIGENDIAN
 #  elif defined(__i386__) || defined(__alpha__) || defined(__ia64) || defined(__ia64__) || defined(_M_IX86) || defined(_M_IA64) || defined(_M_ALPHA) || defined(__amd64) || defined(__amd64__
 #    define RAPIDJSON_ENDIAN RAPIDJSON_LITTLEENDIAN
-#  elif defined(_MSC_VER) && defined(_M_ARM)
+#  elif defined(_MSC_VER) && (defined(_M_ARM) || defined(_M_ARM64))
 #    define RAPIDJSON_ENDIAN RAPIDJSON_LITTLEENDIAN
 #  elif defined(RAPIDJSON_DOXYGEN_RUNNING)
 #    define RAPIDJSON_ENDIAN

as 0002-fix-win-uwp-arm64.patch on pvr.waipu/depends/common/rapidjson/.

The https://github.com/Tencent/rapidjson have it fixed like them, only still no new release there 😒 to take for the addon.

@flubshi
Copy link
Owner Author

flubshi commented Apr 25, 2020

Wow, works perfectly. Thanks a lot!

@flubshi flubshi merged commit 0650e57 into Matrix Apr 26, 2020
@flubshi flubshi deleted the matrix_azure_ci branch April 26, 2020 11:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants