Skip to content

[vcpkg baseline][qt5-base] Fix glib link issues on linux#16954

Merged
vicroms merged 27 commits intomicrosoft:masterfrom
Neumann-A:fix_qt5_base
Apr 6, 2021
Merged

[vcpkg baseline][qt5-base] Fix glib link issues on linux#16954
vicroms merged 27 commits intomicrosoft:masterfrom
Neumann-A:fix_qt5_base

Conversation

@Neumann-A
Copy link
Contributor

@Neumann-A Neumann-A commented Mar 29, 2021 •
edited by PhoebeHui
Loading

Fix CI issues in:
#16948
#16897

closes #16942

Edit: Failure in above PRs:
c++: error: /mnt/vcpkg-ci/installed/x64-linux/debug/lib//libglib-2.0.a: No such file or directory
c++: error: /mnt/vcpkg-ci/installed/x64-linux/debug/lib//libglib-2.0.a: No such file or directory
c++: error: /mnt/vcpkg-ci/installed/x64-linux/debug/lib//libglib-2.0.a: No such file or directory

@NancyLi1013 NancyLi1013 self-assigned this Mar 30, 2021
@NancyLi1013 NancyLi1013 added the category:port-bug The issue is with a library, which is something the port should already support label Mar 30, 2021
@NancyLi1013 NancyLi1013 changed the title [qt5-base] Fix glib link issues on linux [vcpkg baseline][qt5-base] Fix glib link issues on linux Mar 30, 2021
@NancyLi1013 NancyLi1013 mentioned this pull request Mar 30, 2021
Closed
@JackBoosY
Copy link
Contributor

JackBoosY commented Mar 30, 2021

Ping @ras0219 @ras0219-msft for review this PR.
Can we have other better ways to achieve this?

@Neumann-A
Copy link
Contributor Author

Neumann-A commented Mar 30, 2021

Just a note: Passing --triplet=${TARGET_TRIPLET} to the list command does nothing. (I would at least assume it will filter the list.)

Neumann-A
Neumann-A commented Mar 30, 2021
View reviewed changes
Neumann-A
Neumann-A commented Mar 30, 2021
View reviewed changes
vicroms
vicroms approved these changes Mar 30, 2021
View reviewed changes
Copy link
Member

@vicroms vicroms left a comment

Choose a reason for hiding this comment

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

LGTM

@vicroms
Copy link
Member

vicroms commented Mar 30, 2021

An alternative is to run vcpkg x-package-info {port}:{triplet} --x-installed --x-json, which will produce a features array as part of the JSON output.

ras0219-msft
ras0219-msft requested changes Mar 30, 2021
View reviewed changes
Copy link
Contributor

@ras0219-msft ras0219-msft left a comment

Choose a reason for hiding this comment

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

I'd really prefer to avoid this approach (recursive vcpkg to inspect installed files).

If nothing else, harfbuzz should install a file that we can test for existence (such as share/harfbuzz/has_glib_feature.

Neumann-A
Neumann-A commented Mar 31, 2021
View reviewed changes
Neumann-A
Neumann-A commented Mar 31, 2021
View reviewed changes
Neumann-A
Neumann-A commented Mar 31, 2021
View reviewed changes
Neumann-A
Neumann-A commented Mar 31, 2021
View reviewed changes
Neumann-A
Neumann-A commented Apr 1, 2021
View reviewed changes
@PhoebeHui
Copy link
Contributor

PhoebeHui commented Apr 1, 2021

The forest issue would be fixed by PR #16836

@PhoebeHui PhoebeHui mentioned this pull request Apr 2, 2021
Merged
This was referenced Apr 2, 2021
Merged
Merged
@JackBoosY JackBoosY requested a review from ras0219-msft April 5, 2021 04:25
@JackBoosY JackBoosY added category:vcpkg-feature The issue is a new capability of the tool that doesn’t already exist and we haven’t committed requires:discussion labels Apr 5, 2021
@JackBoosY
Copy link
Contributor

JackBoosY commented Apr 5, 2021

Ping @ras0219 @ras0219-msft for review this PR again.

strega-nil
strega-nil suggested changes Apr 5, 2021
View reviewed changes
Copy link
Contributor

@strega-nil strega-nil left a comment

Choose a reason for hiding this comment

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

minor cmake changes

@Neumann-A @strega-nil
Apply suggestions from code review
2c21038 
Co-authored-by: nicole mazzuca <mazzucan@outlook.com>
@vicroms
Copy link
Member

vicroms commented Apr 5, 2021

Waiting for clean CI

@JackBoosY JackBoosY added info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this. and removed requires:discussion labels Apr 6, 2021
@vicroms vicroms merged commit 1bb0fe6 into microsoft:master Apr 6, 2021
@NancyLi1013 NancyLi1013 mentioned this pull request Apr 7, 2021
Merged
@Neumann-A Neumann-A deleted the fix_qt5_base branch April 7, 2021 14:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vicroms vicroms vicroms approved these changes

@ras0219-msft ras0219-msft Awaiting requested review from ras0219-msft

+2 more reviewers

@strega-nil strega-nil strega-nil approved these changes

@JackBoosY JackBoosY JackBoosY approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@NancyLi1013 NancyLi1013

@JackBoosY JackBoosY

Labels

category:port-bug The issue is with a library, which is something the port should already support category:vcpkg-feature The issue is a new capability of the tool that doesn’t already exist and we haven’t committed info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[cmake|linux] build failure

7 participants

@Neumann-A @JackBoosY @vicroms @PhoebeHui @strega-nil @ras0219-msft @NancyLi1013