Skip to content

Add QtGui dependency if QCORO_WITH_QTQUICK=ON#209

Merged
danvratil merged 2 commits intoqcoro:mainfrom
a17r:missingdep
Jan 18, 2024
Merged

Add QtGui dependency if QCORO_WITH_QTQUICK=ON#209
danvratil merged 2 commits intoqcoro:mainfrom
a17r:missingdep

Conversation

@a17r
Copy link
Contributor

@a17r a17r commented Jan 18, 2024

QImage is used in qcoro/quick/qcoroimageprovider.{cpp,h}

QImage is used in qcoro/quick/qcoroimageprovider.{cpp,h}

Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org>
@danvratil
Copy link
Collaborator

Thanks. Doesn't it get pulled in transitively through depending on QtQuick? In any case, no harm in specifying the dep explicitly.

@danvratil danvratil enabled auto-merge January 18, 2024 16:22
@a17r
Copy link
Contributor Author

a17r commented Jan 18, 2024

Thanks. Doesn't it get pulled in transitively through depending on QtQuick?

It does right now, but we shouldn't rely on that. :)

@danvratil danvratil merged commit 69cdf18 into qcoro:main Jan 18, 2024
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