From mboxrd@z Thu Jan 1 00:00:00 1970 From: Roger Leigh Subject: Re: /etc/fstab.d yes or not Date: Fri, 20 Jan 2012 15:57:01 +0000 Message-ID: <20120120155701.GL8797@codelibre.net> References: <20120120140444.GC13157@x2.net.home> <20120120.232002.849876084731555262.yamato@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: Masatake YAMATO , kzak@redhat.com, linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org, util-linux@vger.kernel.org To: Kay Sievers Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org List-Id: linux-fsdevel.vger.kernel.org On Fri, Jan 20, 2012 at 03:56:43PM +0100, Kay Sievers wrote: > On Fri, Jan 20, 2012 at 15:20, Masatake YAMATO wr= ote: > >> > >> =A0I'd like to add support for /etc/fstab.d to libmount. The libra= ry is > >> =A0currently used by mount, umount and mount.nfs. The goal is to u= se it > >> =A0on more places. > > > > I'm working on systemd to support /etc/fstab.d. >=20 > It's an old glibc API, and /etc/fstab is ABI, not a service config > file, which now can read more than one file. It's a very different > problem. It an ABI change, not a config extension. I think that's putting the problem a little too strongly. The glibc API is generic: it can (and is) used to parse any file using the fstab(5) file format; it's not restricted to only /etc/mtab and /etc/fstab. setmntent(3) accepts any filename. You are entirely correct that it may break the assumptions made by some software, but it's certainly not "ABI" or even "API" breakage. > Tools rightfully expect that they find all system mounts in that file= , > and not in some new split-up directory. In some cases, fstab is used > to 'check if the device is not a system volume', and that will just > break now, The number of tools which do this is tiny, and could (if we wished) be patched trivially to also look at the files in fstab.d. Alternatively, glibc itself could be updated so that if e.g. setmntent(3) is called with "/etc/fstab" as its filename argument, it will additionally process and include the contents of /etc/fstab.d as well. This would have the advantage of transparency for all users of the glibc interface. Regards, Roger --=20 .''`. Roger Leigh : :' : Debian GNU/Linux http://people.debian.org/~rleigh/ `. `' Printing on GNU/Linux? http://gutenprint.sourceforge.net= / `- GPG Public Key: 0x25BFB848 Please GPG sign your mail.