Skip to content

[readosm] Revise portfile, install pc file for windows#21599

Closed
dg0yt wants to merge 5 commits intomicrosoft:masterfrom
dg0yt:readosm
Closed

[readosm] Revise portfile, install pc file for windows#21599
dg0yt wants to merge 5 commits intomicrosoft:masterfrom
dg0yt:readosm

Conversation

@dg0yt
Copy link
Contributor

@dg0yt dg0yt commented Nov 22, 2021

  • What does your PR fix?

    Revises the portfile. Uses pkgconfig for nmake and autotools.
    Installs pc file also for windows. (Needed for [spatialite-tools] Revise and update to 5.0.1 #21594.)

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

    all, yes

  • 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

quantlib:x64-windows-static-md=fail
readosm:x64-windows-static-md=fail
sentencepiece:x64-windows-static-md=fail
spatialite-tools:x64-windows-static-md=fail
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To be revisited and probably fixed with #21594. (Used to be cascaded from the readosm fail entry.)

@dg0yt dg0yt marked this pull request as ready for review November 22, 2021 09:11
@dg0yt dg0yt marked this pull request as draft November 22, 2021 09:14
@dg0yt
Copy link
Contributor Author

dg0yt commented Nov 22, 2021

Needs more integration testing via #21594.

@dg0yt dg0yt marked this pull request as ready for review November 22, 2021 10:07
@JonLiu1993 JonLiu1993 assigned JonLiu1993 and JackBoosY and unassigned JonLiu1993 Nov 22, 2021
@JackBoosY JackBoosY added the category:port-feature The issue is with a library, which is requesting new capabilities that didn’t exist label Nov 22, 2021
@dg0yt
Copy link
Contributor Author

dg0yt commented Nov 23, 2021

Full integration succeeded in #21594. (readosm, libspatialite, spatialite-tools).

@JackBoosY
Copy link
Contributor

I think we should pick PR #21594 instead of this.

@dg0yt
Copy link
Contributor Author

dg0yt commented Nov 25, 2021

I think we should pick PR #21594 instead of this.

Fine for me.

@dg0yt dg0yt closed this Nov 25, 2021
@dg0yt dg0yt deleted the readosm branch December 11, 2021 16:09
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants