Skip to content
This repository has been archived by the owner on Aug 20, 2024. It is now read-only.

Syncing needed submodules #41

Open
wants to merge 5 commits into
base: testing/5.8.0
Choose a base branch
from

Conversation

kenfred
Copy link

@kenfred kenfred commented Oct 4, 2017

Adding qtrepotool subrepo. The build instructuctions say it should be there if building directly from a git checkout (http://doc.qt.io/qt-5/windows-building.html). PATH was being appended with qtrepotools/bin, but that directory doesn't exist without updating the submodule.

Adding qtdeclarative subrepo. QtCreator won't allow the resulting Qt package to be in a kit unless qmlscene.exe is present, which is built in qtdeclarative.

… there if building directly from a git checkout (http://doc.qt.io/qt-5/windows-building.html). PATH was being appended with qtrepotools/bin, but that directory doesn't exist without updating the submodule.

Adding qtdeclarative subrepo. QtCreator won't allow the resulting Qt package to be in a kit unless qmlscene.exe is present, which is built in qtdeclarative.
Can't modify dictionary during iteration
Fixing windows call to configure.bat
@ericLemanissier
Copy link

qtdeclarative should definitively be added in the options, but not be mandatory.
Where does the build fail when qtrepotools is not present ?

@kenfred
Copy link
Author

kenfred commented Oct 5, 2017

Ok, I'll make qtdeclarative an option.

Having qtrepotools didn't seem to make a difference in the configurations I tried. However, it is clearly stated as being necessary in the build instructions. The recipe was already referencing the bin directory, just not ensuring the bin directory existed.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants