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

qflipper: update to 1.1.3. #39325

Merged
merged 1 commit into from
Sep 18, 2022
Merged

Conversation

RunningDroid
Copy link
Contributor

Testing the changes

  • I tested the changes in this PR: YES

wrksrc="qFlipper-${version}"
build_style=qmake
configure_args="-spec linux-g++ CONFIG+=qtquickcompiler DEFINES+=DISABLE_APPLICATION_UPDATES"
Copy link
Member

Choose a reason for hiding this comment

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

Remove -spec

revision=1
_nanopb_version=0.4.5
archs="~i686" #https://github.com/flipperdevices/qFlipper/issues/120
Copy link
Member

Choose a reason for hiding this comment

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

How come it's buildable for arm 32bit?

Copy link
Member

Choose a reason for hiding this comment

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

Well, because you're doing it wrong. Those addition *-devel packages in hostmakedepends is wrong!!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sorry, I added the *-devel packages to hostmakedepends because the cross builds on my machine were complaining about being unable to find the QT modules for those packages and that appeared to resolve it.

Copy link
Member

Choose a reason for hiding this comment

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

Don't add -spec into configure args, it will be set by build-style and some magic needs to be done for cross compilation.
In the end of the day, it will use same spec with linux-g++

Copy link
Member

@sgn sgn left a comment

Choose a reason for hiding this comment

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

How come it's buildable for arm 32bit?

sgn added a commit to RunningDroid/void-packages that referenced this pull request Sep 18, 2022
- disable building on i686 and point to upstream bug
	flipperdevices/qFlipper#120
- add configure args that upstream recommends:
	https://github.com/flipperdevices/qFlipper#standalone-build
- add configure arg to disable update checker

Close: void-linux#39325
@sgn sgn merged commit 6c240d6 into void-linux:master Sep 18, 2022
@RunningDroid RunningDroid deleted the update_qflipper branch September 18, 2022 23:38
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.

None yet

2 participants