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

[qtkeychain] Update to 0.13.2+2 to fix the exported CMake target #24013

Merged
merged 1 commit into from
Apr 13, 2022

Conversation

daschuer
Copy link
Contributor

@daschuer daschuer commented Apr 7, 2022

This allows to link the target application using CMake < 3.18
and fixing a regression since v0.13.0

Since upstream is only two commits ahead, I have references the head commit, instead patching the source manual.
frankosterfeld/qtkeychain@e5eeb17

  • What does your PR fix?

It fixes the exported CMake target

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

No change

Yes

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

Yes

@JonLiu1993 JonLiu1993 self-assigned this Apr 7, 2022
@JonLiu1993 JonLiu1993 added the category:port-update The issue is with a library, which is requesting update new revision label Apr 7, 2022
@JonLiu1993
Copy link
Member

@daschuer ,Have you tested the usage locally?

@daschuer
Copy link
Contributor Author

daschuer commented Apr 7, 2022

Yes, with Linux

@JonLiu1993
Copy link
Member

I have tested the usage successfully on my machine.

@JonLiu1993 JonLiu1993 added the info:reviewed Pull Request changes follow basic guidelines label Apr 12, 2022
@strega-nil-ms strega-nil-ms added requires:author-response and removed info:reviewed Pull Request changes follow basic guidelines labels Apr 12, 2022
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

This is a new experimental fast check for PR issues. Please let us know if this bot is helpful!

PRs must add only one version and must not modify any published versions

When making any changes to a library, the version or port-version in vcpkg.json or CONTROL must be modified.

Error: Local changes detected for qtkeychain but no changes to version or port version.
-- Version: 0.13.2#2
-- Old SHA: 529c94ad5ef10fbe264ada5df17a2ecd55b4a4a5
-- New SHA: 6311d9dd13b97c8a01a980d7b42ea6cfecc4f765
-- Did you remember to update the version or port version?
-- Pass `--overwrite-version` to bypass this check.
***No files were updated.***

This allows to link the target application using CMake < 3.18
and fixing a regression since v0.13.0
@JonLiu1993 JonLiu1993 added the info:reviewed Pull Request changes follow basic guidelines label Apr 13, 2022
@strega-nil-ms strega-nil-ms merged commit 6960e3f into microsoft:master Apr 13, 2022
@strega-nil-ms
Copy link
Contributor

Thanks @daschuer !

@daschuer daschuer deleted the qtkeychain_export_target branch January 5, 2023 07:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:port-update The issue is with a library, which is requesting update new revision info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants