Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 713654 - sys-cluster/swift-2.23.1-r2 *XATTR needs to be enabled
Summary: sys-cluster/swift-2.23.1-r2 *XATTR needs to be enabled
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Matthew Thode ( prometheanfire )
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-03-20 15:21 UTC by Alessandro Barbieri
Modified: 2020-03-24 02:05 UTC (History)
2 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 Alessandro Barbieri 2020-03-20 15:21:58 UTC
>>> Running pre-merge checks for sys-cluster/swift-2.23.1-r2
 * Determining the location of the kernel source code
 * Found kernel source directory:
 *     /usr/src/linux
 * Found sources for kernel version:
 *     5.4.13-rt-rt7-x86_64
 * ~EXT3_FS_XATTR needs to be enabled
 * ~SQUASHFS_XATTR needs to be enabled
 * ~CIFS_XATTR needs to be enabled
 * ~JFFS2_FS_XATTR needs to be enabled
 * ~TMPFS_XATTR needs to be enabled
 * ~UBIFS_FS_XATTR needs to be enabled
 * ~EXT2_FS_XATTR needs to be enabled
 * ~REISERFS_FS_XATTR needs to be enabled
 * ~EXT4_FS_XATTR needs to be enabled
 * ~ZFS needs to be enabled

Tell me what to do because I already enabled all the XATTR stuff in my kernel configuration.
Why it also want ZFS? It's not in the default kernel.

cat /usr/src/linux/.config | grep XATTR
CONFIG_EXT2_FS_XATTR=y
CONFIG_REISERFS_FS_XATTR=y
CONFIG_F2FS_FS_XATTR=y
CONFIG_TMPFS_XATTR=y
CONFIG_JFFS2_FS_XATTR=y
CONFIG_UBIFS_FS_XATTR=y
CONFIG_SQUASHFS_XATTR=y
CONFIG_EROFS_FS_XATTR=y
CONFIG_AUFS_XATTR=y
CONFIG_CIFS_XATTR=y
CONFIG_EVM_EXTRA_SMACK_XATTRS=y
CONFIG_EVM_ADD_XATTRS=y
Comment 1 Matthew Thode ( prometheanfire ) archtester Gentoo Infrastructure gentoo-dev Security 2020-03-24 02:04:34 UTC
thanks for asking about this :D

the check is only a warning.  it could probably be done better (and be limited to not include JFFS2 and the like), but what it really should be checking is if ONE of that set is enabled, if so, be happy, otherwise maybe a notice about it.  ZFS is in there because it has xattr support (and the check IS optional).

  1   if linux_config_exists; then
  2     for module in ${CONFIG_CHECK}; do
  3       linux_chkconfig_present ${module} || ewarn "${module} needs to be enabled"
  4     done
  5   fi
Comment 2 Larry the Git Cow gentoo-dev 2020-03-24 02:05:27 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=065e3ee22b780d06d650b400db93dd6fae9cab46

commit 065e3ee22b780d06d650b400db93dd6fae9cab46
Author:     Matthew Thode <prometheanfire@gentoo.org>
AuthorDate: 2020-03-24 02:05:03 +0000
Commit:     Matthew Thode <prometheanfire@gentoo.org>
CommitDate: 2020-03-24 02:05:20 +0000

    sys-cluster/swift: clarify the kernel check
    
    Closes: https://bugs.gentoo.org/713654
    Package-Manager: Portage-2.3.89, Repoman-2.3.20
    Signed-off-by: Matthew Thode <prometheanfire@gentoo.org>

 sys-cluster/swift/swift-2019.2.9999.ebuild | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)