Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 953378 - www-apps/nextcloud ebuilds broken by the strict-keepdir FEATURE becoming the default in EAPI 8
Summary: www-apps/nextcloud ebuilds broken by the strict-keepdir FEATURE becoming the ...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: Bernard Cafarelli
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2025-04-07 18:37 UTC by John R. Graham
Modified: 2025-04-09 12:36 UTC (History)
3 users (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 John R. Graham gentoo-dev 2025-04-07 18:37:33 UTC
There's a directory that's not present in the nextcloud tarballs that must exist and be writable at runtime for nextcloud to function properly. The src_install() phase function currently creates it like this:

    dodir "${MY_HTDOCSDIR}"/data

and then Portage removes it because the the directory is empty and the strict-keepdir FEATURE now defaults to on. This results results in a nonfunctional install of nextcloud. What the modern ebuild *should* be doing is:

    keepdir "${MY_HTDOCSDIR}"/data

This issue is present in all in-tree nextcloud ebuilds.
Comment 1 Larry the Git Cow gentoo-dev 2025-04-09 12:04:57 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=4257da097741a25c0062532db9af9db552bc6a79

commit 4257da097741a25c0062532db9af9db552bc6a79
Author:     Florian Schmaus <flow@gentoo.org>
AuthorDate: 2025-04-09 12:01:37 +0000
Commit:     Florian Schmaus <flow@gentoo.org>
CommitDate: 2025-04-09 12:03:32 +0000

    www-apps/nextcloud: use keepdir instead of dodir
    
    Empty directories created by dodir will be stripped by portage with
    strict-keepdir enabled (the default). Instead, we should use keepdir.
    
    Thanks to John R. Graham for reporting this.
    
    Closes: https://bugs.gentoo.org/953378
    Signed-off-by: Florian Schmaus <flow@gentoo.org>

 www-apps/nextcloud/nextcloud-29.0.10.ebuild | 2 +-
 www-apps/nextcloud/nextcloud-29.0.11.ebuild | 2 +-
 www-apps/nextcloud/nextcloud-29.0.12.ebuild | 2 +-
 www-apps/nextcloud/nextcloud-29.0.14.ebuild | 2 +-
 www-apps/nextcloud/nextcloud-30.0.4.ebuild  | 2 +-
 www-apps/nextcloud/nextcloud-30.0.5.ebuild  | 2 +-
 www-apps/nextcloud/nextcloud-30.0.6.ebuild  | 2 +-
 www-apps/nextcloud/nextcloud-30.0.8.ebuild  | 2 +-
 www-apps/nextcloud/nextcloud-31.0.2.ebuild  | 2 +-
 9 files changed, 9 insertions(+), 9 deletions(-)
Comment 2 Sam James archtester Gentoo Infrastructure gentoo-dev Security 2025-04-09 12:06:00 UTC
That obviously needs a new revision.
Comment 3 Larry the Git Cow gentoo-dev 2025-04-09 12:36:20 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=15527b09190950b4a0803b81471c3e257390cbf4

commit 15527b09190950b4a0803b81471c3e257390cbf4
Author:     Florian Schmaus <flow@gentoo.org>
AuthorDate: 2025-04-09 12:33:00 +0000
Commit:     Florian Schmaus <flow@gentoo.org>
CommitDate: 2025-04-09 12:35:57 +0000

    www-apps/nextcloud: bump revisions due to previous keepdir change
    
    Closes: https://bugs.gentoo.org/953378
    Signed-off-by: Florian Schmaus <flow@gentoo.org>

 .../nextcloud/{nextcloud-29.0.10.ebuild => nextcloud-29.0.10-r1.ebuild}   | 0
 .../nextcloud/{nextcloud-29.0.11.ebuild => nextcloud-29.0.11-r1.ebuild}   | 0
 .../nextcloud/{nextcloud-29.0.12.ebuild => nextcloud-29.0.12-r1.ebuild}   | 0
 .../nextcloud/{nextcloud-29.0.14.ebuild => nextcloud-29.0.14-r1.ebuild}   | 0
 .../nextcloud/{nextcloud-30.0.4.ebuild => nextcloud-30.0.4-r1.ebuild}     | 0
 .../nextcloud/{nextcloud-30.0.5.ebuild => nextcloud-30.0.5-r1.ebuild}     | 0
 .../nextcloud/{nextcloud-30.0.6.ebuild => nextcloud-30.0.6-r1.ebuild}     | 0
 .../nextcloud/{nextcloud-30.0.8.ebuild => nextcloud-30.0.8-r1.ebuild}     | 0
 .../nextcloud/{nextcloud-31.0.2.ebuild => nextcloud-31.0.2-r1.ebuild}     | 0
 9 files changed, 0 insertions(+), 0 deletions(-)