Skip to content

[libffi] Add pkgconfig file#12126

Merged
ras0219-msft merged 2 commits intomicrosoft:masterfrom
c72578:2020-06-26_libffi_Add_pkgconfig_file
Jul 6, 2020
Merged

[libffi] Add pkgconfig file#12126
ras0219-msft merged 2 commits intomicrosoft:masterfrom
c72578:2020-06-26_libffi_Add_pkgconfig_file

Conversation

@c72578
Copy link
Contributor

@c72578 c72578 commented Jun 26, 2020

Describe the pull request

  • What does your PR fix? Fixes #

    • Configure and add the libffi.pc file.
    • Furthermore, enable the check for same 32/64bit-ness in
      libffiConfigVersion.cmake again, which had caused issues in the past
      with newer versions of glib using meson build, e.g. when building for
      x86-windows target on an x64-windows host.
      Now, libffi is found by current glib (e.g. 2.64.3) during
      meson build using the libffi pkgconfig file.
  • Which triplets are supported/not supported? Have you updated the CI baseline?
    All triplets are supported. CI baseline: no update required.

  • Does your PR follow the maintainer guide?
    Yes

- Configure and add the libffi.pc file.
- Furthermore, enable the check for same 32/64bit-ness in
  libffiConfigVersion.cmake again, which had caused issues in the past
  with newer versions of glib using meson build, e.g. when building for
  x86-windows target on an x64-windows host.
  Now, libffi is found by current glib (e.g. 2.64.3) during
  meson build using the libffi pkgconfig file.
@c72578 c72578 marked this pull request as ready for review June 26, 2020 15:30
@vicroms vicroms added the info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this. label Jun 26, 2020
@c72578
Copy link
Contributor Author

c72578 commented Jun 27, 2020

Info concerning REGRESSION: paraview:x64-linux, which is unrelated to this PR here:
From install-x64-linux-dbg-err.log:

ninja: error: '/mnt/vcpkg-ci/installed/x64-linux/lib/libmath.a', needed by 'bin/pvserver', missing and no known rule to make it

Edit: PR #12144 is about fixing paraview:x64-linux build

@LilyWangL LilyWangL added the category:port-feature The issue is with a library, which is requesting new capabilities that didn’t exist label Jun 29, 2020
@ras0219-msft
Copy link
Contributor

LGTM, thanks for the improvement!

@ras0219-msft ras0219-msft merged commit a05cec9 into microsoft:master Jul 6, 2020
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:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants