Bug 938236 – www-apps/nextcloud ebuild, add a redis USE flag and associated RDEPEND

• Status: UNCONFIRMED
• Product: Gentoo Linux

[Description xiaojie+bugzillagentoo 2024-08-20 09:47:15 UTC]

The prefered (but optionnal) transactional file locking relies on redis. Adding a redis USE flag and RDEPEND will add this dependency.

Reproducible: Always

[Comment 1 xiaojie+bugzillagentoo 2024-08-20 09:48:06 UTC]

Created attachment 900786 [details, diff]
Patch to nextcloud-29.0.2.ebuild

Patch to add redis USE flag and dependency

[Comment 2 Florian Schmaus 2024-08-22 08:33:24 UTC]

Thanks for your report. 

This seems to be more a case for optfeature than an USE flag.

[Comment 3 Bernard Cafarelli 2024-08-24 08:24:56 UTC]

+1, nextcloud can run fine without redis, it is an optional runtime feature, and nothing changes in the code (and ebuild merge) when enabling it. So optfeature sounds great here
Similar entries could also be added for other memory caching options (where redis can also be used):
https://docs.nextcloud.com/server/latest/admin_manual/configuration_server/caching_configuration.html

[Comment 4 xiaojie+bugzillagentoo 2024-08-27 17:26:27 UTC]

I did not even know optfeature existed and was one of the recommended way of setting up this kind of optionnal features. Never too late nor too old to learn new tricks.

[Comment 5 xiaojie+bugzillagentoo 2024-08-27 17:36:23 UTC]

Created attachment 901357 [details, diff]
Patch to nextcloud-29.0.4 ebuild, add some optfeature

Adding optfeature for redis, apcu and memcached memory cache.