Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[port] Port to QtQuick2/Qt5 #6

Merged
merged 3 commits into from
Aug 7, 2013
Merged

[port] Port to QtQuick2/Qt5 #6

merged 3 commits into from
Aug 7, 2013

Conversation

sledges
Copy link
Contributor

@sledges sledges commented Aug 2, 2013

No description provided.

Name=Calendar
Exec=/usr/bin/single-instance /opt/qmlcalendar/bin/qmlcalendar
Categories=Applications;
Exec=invoker --type=qtcomponents-qt5 -s /opt/qmlcalendar/bin/qmlcalendar
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is wrong. If you'd like to use the qtcomponents-qt5 booster, you must add PKGCONFIG += qdeclarative5-boostable and the MDeclarativeCache usage in main.cpp. Other projects like qmlcalc should have examples of what to do.

Or, that can be a separate change later and this line can be left unchanged.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok I shall add boostable soon after (even though now the app launches without problems). Would you like me to add boostable commit to this PR?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If it's ready quickly, sure. Otherwise, a separate PR would be fine.

It will still launch without problems, but it's much slower: first the qtcomponents-qt5 booster is failing to launch, then it falls back to a direct unboosted launch.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, boosted in the same PR. All tested, works

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

Requires:
- libdeclarative-organizer
- qt5-qtdeclarative-pim-organizer
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

libdeclarative-organizer is a Qt4 package. It's replaced by qt5-qtdeclarative-pim-organizer, so it should be removed.

@special
Copy link
Contributor

special commented Aug 6, 2013

Looks good to me once those dependencies are fixed. Thanks!

@sledges
Copy link
Contributor Author

sledges commented Aug 7, 2013

Good to go, cleaned up packaging, still works :)

special added a commit that referenced this pull request Aug 7, 2013
[port] Port to QtQuick2/Qt5
@special special merged commit 2a63193 into nemomobile:master Aug 7, 2013
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