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

[BUG] qownnotes-23.9.4 build on gentoo fails due to lack of QT concurrent module #2847

Closed
erfanara opened this issue Sep 15, 2023 · 9 comments

Comments

@erfanara
Copy link

Hi,

installing app-office/qownnotes-23.9.4 fails:

emerge --ask app-office/qownnotes leads to:

 * Running qmake ...
Info: creating stash file /var/tmp/portage/app-office/qownnotes-23.9.4/work/qownnotes-23.9.4/.qmake.stash
Project MESSAGE: Currently in RELEASE mode.
Project ERROR: Unknown module(s) in QT: concurrent                                                                                                                    [ !! ]

problem can be fixed by installing dev-qt/qtconcurrent:5

I would be happy to help you in fixing and testing ebuilds, should i open a PR?

@pbek
Copy link
Owner

pbek commented Sep 16, 2023

Is this maybe related to #2821?
@catthou, @werdnab, can you also reproduce this issue?
If yes, PRs to https://github.com/pbek/QOwnNotes/blob/main/build-systems/gentoo/qownnotes.ebuild are welcome...

@catthou
Copy link

catthou commented Sep 16, 2023

I've got concurrency installed for something else and 23.9.4 installed and runs good. I don't dare to uninstall it because QT is very fickle on gentoo!

@FireUnderTheMountain
Copy link
Contributor

FireUnderTheMountain commented Sep 16, 2023

The suggested package, dev-qt/qtconcurrent:5, is probably required... @erfanara, were you able to install any prior version of QOwnNotes? Just curious, because I don't see a recent commit that would add this as a dependency.

@pbek - Is the ebuild is supposed to be maintained for all release versions, or out-of-the-box should it only "support" the most recent stable release? I don't really want to add a new global dependency if it's only needed for specific versions. On the other hand, if the ebuild is meant to only install the most recent release, then it could be added as a dependency without concern. Usually gentoo ebuilds have their version number in the ebuild name so you can isolate differences between versions, example: qownnotes-23.9.4.ebuild. Oh, I see, you have a system to generate the qownnotes-*.ebuild files. :)

Note for Qt6, there is no dev-qt/qtconcurrent package, it looks like a module is built using the concurrent useflag in dev-qt/qtbase - though I'm not entirely familiar with Qt to be exactly sure.

@erfanara
Copy link
Author

Well actually i tried to install QOwnNotes on a minimal fresh gentoo and i got that error,
i just tried out some older versions like 23.9.1 and the issue persists.

src/QOwnNotes.pro shows QT dependency modules needed to compile the project i guess:

QT       += core gui widgets sql svg network xml printsupport qml websockets concurrent

Looking at the git blame shows that this line is 3 years old 😅,
So i think the concurrent module can be installed as a dependency even on old ebuilds.

@FireUnderTheMountain
Copy link
Contributor

Most old ebuilds actually throw errors for using an unsupported EAPI at this point, that's why I asked for input on whether or not they are all rebuilt, or only the latest, on each new release. The oldest of them should be cleaned up if they won't be getting EAPI updates or bug fixes.

@pbek
Copy link
Owner

pbek commented Sep 17, 2023

You can create a PR to add the package if you want. And if you want to weed out https://github.com/qownnotes/gentoo-overlay, you could create a PR as well...

FireUnderTheMountain added a commit to FireUnderTheMountain/QOwnNotes that referenced this issue Sep 22, 2023
pbek added a commit that referenced this issue Sep 23, 2023
@werdnab
Copy link

werdnab commented Sep 23, 2023

I also already have qtconcurrent installed as it's a dependency to other packages including nextcloud-client

Andrew

@FireUnderTheMountain
Copy link
Contributor

@erfanara All ebuilds are updated, feel free to report back and close the ticket if solved :)

@erfanara
Copy link
Author

lgtm🔥 , thank you :)

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

No branches or pull requests

5 participants