-
Notifications
You must be signed in to change notification settings - Fork 6.7k
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
[casclib] Add new port #6744
[casclib] Add new port #6744
Conversation
I also see a failure on Mac, is that expected? |
No, it should work on Mac OS. Could you send me the failure logs? I can only access the failure logs for Windows and Linux it seems. |
We are in the process of bringing up a new osx test pipeline that is accessible by the community. You can find the logs in that new pipeline here: https://dev.azure.com/vcpkg/public/_build/results?buildId=1566&view=results. If you run into issues getting these logs let me know so I can fix the pipeline. Link directly to the logs: https://dev.azure.com/vcpkg/c1ee48cb-0df2-4ab3-8384-b1df5a79fe53/_apis/build/builds/1566/artifacts?artifactName=port%20build%20failure%20logs&api-version=5.1-preview.5&%24format=zip |
/azp run |
It looks like the build is not finding
I don't have access to a Mac right now, but my guess is that is indirectly included, and the version of the tools used on our Mac test machine is a different version than what you have so the file is not getting pulled in. I bet if you add a patch that puts |
It looks like <ctype.h> is included in CascPort.h for Windows/Linux but not for Mac. I have it open so I will just add the patch to see if that makes the CI system happy. |
OSX is happy now :) |
Thanks for solving the issues with Mac compilation! I was very busy lately so didn't get to it. |
Add support for the CascLib library. This library does not target UWP as a platform.