Skip to content

[geos] Install pc file also for Windows#20502

Merged
BillyONeal merged 4 commits intomicrosoft:masterfrom
dg0yt:geos-pc
Oct 6, 2021
Merged

[geos] Install pc file also for Windows#20502
BillyONeal merged 4 commits intomicrosoft:masterfrom
dg0yt:geos-pc

Conversation

@dg0yt
Copy link
Contributor

@dg0yt dg0yt commented Oct 4, 2021

  • What does your PR fix?

    This PR adds pkg-config file installation for Windows.
    With port vcpkg-pkgconfig-get-modules and pkg-config parameter --msvc-syntax, the geos.pc file can be used to easily lookup the target link libraries needed for geos in particular in nmake builds. Cf. [libspatialite] Use pkgconfig for nmake and autotools #20480 (libspatite), [gdal:x64-windows-static] missing geos dependencies #15526 (gdal).
    This may facilitate removing the custom debug postfix in a future revision.
    (I'm going to make similar changes to other geographic packages.)

    This PR also fixes a build-time warning on stdout.

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

    unchanged, no

  • Does your PR follow the maintainer guide?

    yes

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

    yes

@BillyONeal BillyONeal merged commit c8fe3a5 into microsoft:master Oct 6, 2021
@BillyONeal
Copy link
Member

Thanks for the bug fix and sorry about the macos PR hiccups!

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.

2 participants