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

[libconfig] build on macos #19374

Merged
merged 1 commit into from
Aug 19, 2021

Conversation

jamesathey
Copy link
Contributor

Describe the pull request

  • What does your PR fix?

Fixes compilation errors on macOS by making the compilation flags better match what the upstream libconfig CMakeLists.txt is doing. In particular, the line find_path(STDINT_H stdint.h) was picking up the kernel header version of stdint.h on macOS, altering the header search paths and preventing libconfig from compiling against the C standard library.

Also updates the library to the latest patch release (1.7.2 -> 1.7.3).

  • Which triplets are supported/not supported? Have you updated the CI baseline?

This PR addresses the build failure on macOS, so this PR also removes the "fail" line from ci.baseline.txt.

Yes.

  • If you have added/updated a port: Have you run ./vcpkg x-add-version --all and committed the result?

Yes.

@ghost
Copy link

ghost commented Aug 5, 2021

CLA assistant check
All CLA requirements met.

@jamesathey jamesathey force-pushed the libconfig-build-on-macos branch from 6e9f1d5 to 318ff58 Compare August 5, 2021 03:52
@JackBoosY JackBoosY added the category:port-feature The issue is with a library, which is requesting new capabilities that didn’t exist label Aug 5, 2021
ports/libconfig/CONTROL Outdated Show resolved Hide resolved
ports/libconfig/CMakeLists.txt Show resolved Hide resolved
ports/libconfig/CMakeLists.txt Show resolved Hide resolved
* remove "find_path(STDINT_H stdint.h)". On clang/macos, this picks up the kernel headers version of stdint.h, instead of the C standard library, breaking compilation
* only define YY_NO_UNISTD_H and YY_USE_CONST on Windows, like the upstream CMakeLists.txt
* set the C standard to C99, like the upstream CMakeLists.txt
* update to 1.7.3
* remove the "fail" line from ci.baseline.txt
* run ./vcpkg x-add-version --all to update metadata files
* reset port-version
@jamesathey jamesathey force-pushed the libconfig-build-on-macos branch from 318ff58 to c31ada0 Compare August 5, 2021 15:09
@JackBoosY
Copy link
Contributor

I think it's good. Please click ready for review.

@jamesathey jamesathey marked this pull request as ready for review August 6, 2021 13:43
@jamesathey
Copy link
Contributor Author

If #5518 were still open and not closed wont-fix, this PR would fix it.

@JackBoosY JackBoosY added the info:reviewed Pull Request changes follow basic guidelines label Aug 9, 2021
@jamesathey jamesathey changed the title Libconfig build on macos [libconfig] build on macos Aug 9, 2021
@dan-shaw dan-shaw merged commit bd7fc2c into microsoft:master Aug 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:port-feature The issue is with a library, which is requesting new capabilities that didn’t exist info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants