Skip to content

[qtapplicationmanager] Features & platforms#44555

Merged
vicroms merged 3 commits intomicrosoft:masterfrom
dg0yt:qtapplicationmanager
Apr 5, 2025
Merged

[qtapplicationmanager] Features & platforms#44555
vicroms merged 3 commits intomicrosoft:masterfrom
dg0yt:qtapplicationmanager

Conversation

@dg0yt
Copy link
Contributor

@dg0yt dg0yt commented Mar 22, 2025

@dg0yt
Copy link
Contributor Author

dg0yt commented Mar 22, 2025

x64-linux:

ERROR: Feature "am_multi_process": Forcing to "ON" breaks its condition:
    TARGET Qt::DBus AND TARGET Qt::WaylandCompositor AND LINUX AND QT_FEATURE_shared
Condition values dump:
    TARGET Qt::DBus found
    TARGET Qt::WaylandCompositor found
    LINUX = "1"
    QT_FEATURE_shared = "OFF"

So am_multi_process was never available in x64-linux (static library linkage). Dependencies unused.

@dg0yt dg0yt mentioned this pull request Mar 22, 2025
Closed
@dg0yt dg0yt force-pushed the qtapplicationmanager branch 4 times, most recently from c78983d to 89859dd Compare March 23, 2025 10:36
@dg0yt dg0yt force-pushed the qtapplicationmanager branch from 89859dd to 1fbabad Compare March 23, 2025 10:44
@dg0yt dg0yt marked this pull request as ready for review March 23, 2025 11:01
@dg0yt dg0yt mentioned this pull request Mar 23, 2025
Merged
3 tasks
@jimwang118 jimwang118 added the category:port-feature The issue is with a library, which is requesting new capabilities that didn’t exist label Mar 24, 2025
@dg0yt
Copy link
Contributor Author

dg0yt commented Mar 28, 2025

Ping. 6.8.3 is rolling in, i.e. delay is already causing extra work to one of the contributors.

@PhoebeHui PhoebeHui assigned LilyWangLL and unassigned jimwang118 Apr 2, 2025
@dg0yt
Copy link
Contributor Author

dg0yt commented Apr 4, 2025

dg0yt marked this pull request as ready for review 2 weeks ago.

@vicroms vicroms merged commit 28f1eea into microsoft:master Apr 5, 2025
17 checks passed
@dg0yt dg0yt deleted the qtapplicationmanager branch April 5, 2025 07:13
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

Assignees

@LilyWangLL LilyWangLL

Labels

category:port-feature The issue is with a library, which is requesting new capabilities that didn’t exist

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@dg0yt @vicroms @LilyWangLL @jimwang118