Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 939365 - net-libs/signon-ui[qt6]: Clicking "google" in add "online accounts" (KDE settings) does nothing
Summary: net-libs/signon-ui[qt6]: Clicking "google" in add "online accounts" (KDE sett...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: Gentoo KDE team
URL: https://discuss.kde.org/t/kf6-kcm-kac...
Whiteboard: fixed in net-libs/signon-ui-0.15_p202...
Keywords: PATCH, PullRequest
Depends on:
Blocks: gear-24.08.1-stable
  Show dependency tree
 
Reported: 2024-09-09 08:29 UTC by Luke A. Guest
Modified: 2024-10-04 22:04 UTC (History)
5 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
Working ebuild (signon-ui-0.15_p20231016-r1.ebuild,1.88 KB, application/vnd.gentoo.ebuild)
2024-10-03 06:38 UTC, Flavio Cappelli
Details
Patch to disable tests (slightly different from the original, inspired by Arch PKGBUILD) (signon-ui-0.15_p20171022-disable-tests-r1.patch,252 bytes, patch)
2024-10-03 06:40 UTC, Flavio Cappelli
Details | Diff
Replace the patch signon-ui-0.15_p20171022-fix-username-field-reading.patch (signon-ui-0.15_p20171022-fake-user-agent.patch,411 bytes, patch)
2024-10-03 06:47 UTC, Flavio Cappelli
Details | Diff
This is the same one in Portage, it should still be required (signon-ui-0.15_p20171022-webengine-cachedir-path.patch,925 bytes, patch)
2024-10-03 06:49 UTC, Flavio Cappelli
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Luke A. Guest 2024-09-09 08:29:02 UTC
As the title says.
Comment 1 Optimus 2024-09-16 08:48:20 UTC
Same here

"google-gmail"
Looking for plugin ""
Starting auth session with "oauth2"
Info:
Id: 89
caption: "google-gmail"
owner: ""
userName: ""
qt.dbus.integration: Could not disconnect "com.google.code.AccountsSSO.SingleSignOn.Identity" to "destroyed(QObject*)" : Pointers are not supported: QObject*
"google"
Looking for plugin ""
Starting auth session with "oauth2"
Info:
Id: 90
caption: "google"
owner: ""
userName: ""
qt.dbus.integration: Could not disconnect "com.google.code.AccountsSSO.SingleSignOn.Identity" to "destroyed(QObject*)" : Pointers are not supported: QObject*
Comment 2 Optimus 2024-09-16 08:57:47 UTC
Maybe https://bugs.kde.org/show_bug.cgi?id=480779 unrelated, or partially related, because there is no "browser window" at all.
Comment 3 Flavio Cappelli 2024-10-03 06:36:42 UTC
Hi all,
I managed to get Google Drive working again in PLASMA6:

- emerge kdenetwork-meta to make sure all dependencies are installed
- run systemsetting and remove the old Google account
- emerge my modified signon-ui ebuild (see following attachments)
- reboot
- run systemsettings from konsole (just to see the error messages)
- click on online accounts, click on add new account, select Google:

NOTE: the errors reported in the previous msgs still appear in the terminal
but now the Google browse window opens (with the original ebuild it didn't)

- enlarge the window just enough to have the input field inside
- select your language (I selected Italian)
- enter email + password and follow the verification procedure

That's it, the procedure should succeed and you should have the possibility to open gdrive from dolphin again (for me it worked).

Hoping this helps
Flavio
Comment 4 Flavio Cappelli 2024-10-03 06:38:05 UTC
Created attachment 904711 [details]
Working ebuild
Comment 5 Flavio Cappelli 2024-10-03 06:40:48 UTC
Created attachment 904712 [details, diff]
Patch to disable tests (slightly different from the original, inspired by Arch PKGBUILD)
Comment 6 Flavio Cappelli 2024-10-03 06:47:28 UTC
Created attachment 904717 [details, diff]
Replace the patch signon-ui-0.15_p20171022-fix-username-field-reading.patch

Note: The above patch keeps httpUserAgent from the Gentoo patch signon-ui-0.15_p20171022-fix-username-field-reading.patch (I tried with the httpUserAgent from the Arch patch, but Google told me that "the browser was not secure", so I tried with the Gentoo's httpUserAgent and it worked).
Comment 7 Flavio Cappelli 2024-10-03 06:49:44 UTC
Created attachment 904718 [details, diff]
This is the same one in Portage, it should still be required
Comment 8 Christophe PEREZ 2024-10-03 14:47:26 UTC
(In reply to Flavio Cappelli from comment #3)
> That's it, the procedure should succeed and you should have the possibility
> to open gdrive from dolphin again (for me it worked).

Nice job !
Worked for me too.
No need to reboot, not even to close the session.
Comment 9 Andreas Sturmlechner gentoo-dev 2024-10-04 13:32:17 UTC
(In reply to Flavio Cappelli from comment #4)
> Created attachment 904711 [details]
> Working ebuild
Since you didn't attach a unified diff I need to ask unfortunately - any other changes in there than changing signon-ui-0.15_p20171022-fix-username-field-reading.patch?
Comment 10 Flavio Cappelli 2024-10-04 15:24:56 UTC
Sorry, I'm not sure I fully understand your question, so I'll try to explain what I did.

On KDE Bugs (if I remember well) I found an Arch user who claimed that the signon-ui package currently in Arch worked, so I went to see what patches Arch used and saw that compared to Gentoo, it only used one patch that modified the httpUserAgent. So I stripped the original ebuild down to the bare minimum by applying the same and only patch used by Arch (I even left the tests enabled at the beginning).

The sign-in window appeared (whereas it didn't before) but the authentication procedure failed reporting that the browser was too old. I guessed that the user agent was considered obsolete and saw that Gentoo used a different one, so I tried with that and it worked. It's just a guess but I suppose that the patch "signon-ui-0.15_p20171022-fix-username-field-reading.patch" was created to solve some issues with qml in Qt5/PLASMA5 (because it adds some qml code); I know in the transition from Qt5 to Qt6 (and therefore from PLASMA5 to PLASMA6) some things in qml have changed and in fact it seems that such patch is not more required in PLASMA6 (or at least it breaks something); after all Debian/Ubuntu do not use it, I don't know why OpenSuse introduced such patch.

As for the other patches, they are just small tweaks: I removed "signon-ui-0.15_p20171022-drop-fno-rtti.patch" because it seems to do nothing, while I left "signon-ui-0.15_p20171022-webengine-cachedir-path.patch" because it seems correct to me (QUrl::fromLocalFile() would put "file:" in front of the filename). Finally I adopted the same method as Arch to disable the tests (my patch "signon-ui-0.15_p20171022-disable-tests-r1.patch" acts on "signon-ui.pro" like Arch, where the patch "signon-ui-0.15_p20171022-disable-tests.patch" in portage acts on "tests/tests.pro". I didn't make any other changes other than those described.

My solution does not pretend to be perfect, not even the right one: signon-ui, based on the "qt6" use flag, should be able to work both in PLASMA5 (for those who still use it) and PLASMA6, while my version probably only works on PLASMA6 (since I no longer have PLASMA5 installed I have no way of testing it).

Hoping this helps.
Comment 11 Andreas Sturmlechner gentoo-dev 2024-10-04 18:27:47 UTC
Thanks for the detailed remarks.

I was only asking about the ebuild changes. When people attach whole ebuilds, we cannot tell from the outset what the changes are. That's why maintainers will often ask for a unified diff against the latest ebuild in ::gentoo (or the one that was used as base for your changes), so this can be reviewed in bugzilla already. No longer necessary, I've already fetched your ebuild and assessing the patch differences.
Comment 12 Larry the Git Cow gentoo-dev 2024-10-04 21:04:03 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=549f7778c65881e32c0b497002fd23e7d469aafe

commit 549f7778c65881e32c0b497002fd23e7d469aafe
Author:     Andreas Sturmlechner <asturm@gentoo.org>
AuthorDate: 2024-10-04 18:42:09 +0000
Commit:     Andreas Sturmlechner <asturm@gentoo.org>
CommitDate: 2024-10-04 21:03:15 +0000

    net-libs/signon-ui: Drop IUSE qt6 and broken/obsolete patch
    
    Thanks-to: Flavio Cappelli
    Closes: https://bugs.gentoo.org/939365
    Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org>

 .../signon-ui/signon-ui-0.15_p20231016-r2.ebuild   | 59 ++++++++++++++++++++++
 1 file changed, 59 insertions(+)