Skip to content

Qt6#52

Merged
jkloetzke merged 8 commits intoBobBuildTool:masterfrom
sixtyfourktec:qt6
Jan 10, 2025
Merged

Qt6#52
jkloetzke merged 8 commits intoBobBuildTool:masterfrom
sixtyfourktec:qt6

Conversation

@sixtyfourktec
Copy link
Contributor

No description provided.

@jkloetzke
Copy link
Member

Excellent work. 👍

There is just one thing that I noticed. If I cross-compile libs::qt6-base then usr/bin/qmake will have absolute paths to the qt6-base-tools that were used. Not sure if this is relevant because I guess this package is effectively unused. Still, we should either fix this (not sure if that is possible easily) or temporarily comment out the package with a note in the recipe why that is.

I did not check the other qt6PackageBin based packages. They have similar problems.

@sixtyfourktec
Copy link
Contributor Author

I did not try to build anything Qt related on the target. Because this is not really a goal right now, I would prefer not to investigate time on this. I assume disabling those tools would mean patching Qt or deleting all those apps, which in turn renders the bin target useless. Do you want to have it removed?

@jkloetzke
Copy link
Member

If it's not worth the effort then I would propose to comment out the bin target(s). I would leave them as reference. Just add a comment why it's not there and whoever has the use case in the future is free to fix the bin-packages. :)

This adds the core foundation for building Qt6 and Qt6 apps. The Qt6 class
can be used to build Qt6 modules, but also Qt6 applications. Simply
inherit from it and call the appropriate build/package functions.
@sixtyfourktec
Copy link
Contributor Author

Done so.

@jkloetzke jkloetzke merged commit 1cb37ad into BobBuildTool:master Jan 10, 2025
1 check passed
@jkloetzke
Copy link
Member

Thanks!

@sixtyfourktec sixtyfourktec deleted the qt6 branch January 10, 2025 08:52
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