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

[casclib] Add new port #6744

Merged
merged 5 commits into from
Jun 18, 2019
Merged

[casclib] Add new port #6744

merged 5 commits into from
Jun 18, 2019

Conversation

stijnherfst
Copy link
Contributor

@stijnherfst stijnherfst commented Jun 3, 2019

Add support for the CascLib library. This library does not target UWP as a platform.

@Rastaban
Copy link
Contributor

Rastaban commented Jun 4, 2019

I also see a failure on Mac, is that expected?

@stijnherfst
Copy link
Contributor Author

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.

@Rastaban
Copy link
Contributor

Rastaban commented Jun 5, 2019

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

@Rastaban
Copy link
Contributor

/azp run

@Rastaban
Copy link
Contributor

It looks like the build is not finding isxdigit from <cctype>.

src/CascFiles.cpp:161:39: error: use of undeclared identifier 'isxdigit'
        if (szDataPtr >= szDataEnd || isxdigit(szDataPtr[0]) == 0)
                                      ^

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 #include <cctype> into src/CascCommon.h then it will fix the build issue.

@Rastaban
Copy link
Contributor

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.

@Rastaban
Copy link
Contributor

OSX is happy now :)

@Rastaban Rastaban merged commit bba0b6b into microsoft:master Jun 18, 2019
@stijnherfst
Copy link
Contributor Author

Thanks for solving the issues with Mac compilation! I was very busy lately so didn't get to it.

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.

None yet

2 participants