Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 617416 - net-misc/nextcloud-client add IUSE to disable qtwebkit dependency
Summary: net-misc/nextcloud-client add IUSE to disable qtwebkit dependency
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Bernard Cafarelli
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-05-03 18:42 UTC by Rasmus Thomsen
Modified: 2017-05-23 15:12 UTC (History)
1 user (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Rasmus Thomsen 2017-05-03 18:42:10 UTC
Hello,

right now nextcloud-client depends on qtwebkit, which isn't used for anything but the initial authentication to the nextcloud server. It's pretty trivial to remove the dependency on it [1]. By disabling it users would have to authenticate themselves with an app password (which is supported by all (somewhat recent) nextcloud & owncloud servers). Could you consider adding this as useflag, as qtwebkit is both a pretty massive package and deprecated?

1 https://github.com/owncloud/client/issues/4692#issuecomment-255057854
Comment 1 Bernard Cafarelli gentoo-dev 2017-05-19 17:00:07 UTC
OK, for simple authentication, qtwebkit is not needed indeed (only for Shibboleth SSO).

So I added a shibboleth USE-flag in owncloud-client and nextcloud-client (2.3.1-r1 for this one, 2.3.2 is not available yet), to make the dependency optional.

Thanks for the report!
Comment 2 Tom Briden 2017-05-23 14:11:14 UTC
Hi,

nextcloud-client still requires qtwidgets, which isn't a specific dependency (it is a dependency of qtwebkit) so no longer compiles if built without the new shibboleth USE-flag.
Comment 3 Bernard Cafarelli gentoo-dev 2017-05-23 15:12:32 UTC
Nice catch! As we still use the bundled qtsingleapplication, we do not have a direct dependency on it. Added to both owncloud-client and nextcloud-client