[systemd-devel] [PATCH] fstab-generator: do not check btrfs and xfs

[Lennart Poettering]

On Sun, 29.06.14 21:51, Kai Krakow (hurikhan77 at gmail.com) wrote:

> > This sounds really unnecessary, no? We already have fsck_exists()  in
> > place that since a very recent commit of mine even detects a per-fstype
> > fsck implementation being linked to /bin/true... I also downgraded all
> > warnings for cases like that to LOG_DEBUG, hence the xfs/btrfs case
> > should already be covered nicely, and fully generic... Why do we need
> > another explicit blacklist on top of that?
> 
> My version of fsck.btrfs is not symlinked to /bin/true, it's an ordinary 
> binary generating the following output:
> 
> # /sbin/fsck.btrfs /dev/sdb3
> If you wish to check the consistency of a BTRFS filesystem or
> repair a damaged filesystem, see btrfs(8) subcommand 'check'.

Is this an upstream thing? Or is that specific to your distro?

It sounds really wrong to me to do something like what they are
doing. Either they provide a real implementation, or they don't supply
an implementation. Either is fine. But if they supply an implementation
that justs prints a warning will simply mean that various tools
(including systemd) will invoke it, for no reason.

Lennart

-- 
Lennart Poettering, Red Hat