Join GitHub today
GitHub is home to over 20 million developers working together to host and review code, manage projects, and build software together.
update ubuntu build system #24
Conversation
mzanetti
added some commits
Jan 22, 2015
RobertMe
reviewed
Jan 22, 2015
| @@ -0,0 +1,43 @@ | ||
| +<RCC> |
RobertMe
Jan 22, 2015
Contributor
What are the reasons/advantages of using resources? Is this something which should also be done for Sailfish OS?
mzanetti
Jan 22, 2015
Owner
The thing is, in ubuntu I have two kits in QtCreator: one for desktop, and one for the phone using click packages.
For the click packages, I need to have install steps that install things to / (need to talk the sdk guy to find out if that's supposed to stay like that). However, the desktop build is ran directly from the build directory. In order to have the main.cpp find the qml files, I would need to copy them to the build dir. I can't do that with the install step to $$OUT_PWD as I did before, because that would install the things into the wrong place when building a click package.
So one way to solve this is the resource file, as they can be found with qrc://, regardless of the install paths.
I hear rumors that using a qrc file is supposed to be faster at loading too because it doesn't need to open many files, reads contents from ram when needed etc. I've yet to see numbers that prove this though.
That said, I've figured this causes the issue that files from the image cache can't be found any more with this branch. Not sure yet why. I need to figure that and then I'll decide if I want resources for ubuntu or not. You are free to choose if you want that or not.
RobertMe
Jan 22, 2015
Contributor
Ok, so mostly build related. I'll stick to normal files on Sailfish then.
I've also read about performance I think, but well, it isn't like the app or loading the QML is slow, so no need to update those for it.
mzanetti commentedJan 22, 2015
Now that the Ubuntu SDK also supports qmake cross compiling, update
the build system to do that. Also dropping the old
run_on_ubuntu_touch script that has been used to build packages
and run on the phone so far. As Ubuntu now supports click packages
on all systems (not just phones) it doesn't look like the
debian packaging is going to be used any more, so dropping that too.