Re: [anaconda-storage-branch PATCH 6/6] Indicate filesystem is mountable if we have a mount command.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 2009-03-17 at 17:06 -1000, David Cantrell wrote:
> For NTFS, we don't have the name 'ntfs' in /proc/filesystems.  It
> uses fuseblk.  Return True for mountable if the filesystem is
> listed in /proc/filesystems -or- if we have a mount command for
> that filesystem type.
> ---
>  storage/formats/fs.py |    3 ++-
>  1 files changed, 2 insertions(+), 1 deletions(-)
> 
> diff --git a/storage/formats/fs.py b/storage/formats/fs.py
> index a32c65b..71160bb 100644
> --- a/storage/formats/fs.py
> +++ b/storage/formats/fs.py
> @@ -594,7 +594,8 @@ class FS(DeviceFormat):
>  
>      @property
>      def mountable(self):
> -        return self.type in kernel_filesystems
> +        return (self.type in kernel_filesystems) or \
> +               (os.access("/sbin/mount.%s" % (self.type,), os.X_OK))
>  
>      @property
>      def defaultFormatOptions(self):

Looks good, but are we always going to have a kernel module (loaded) for
all the possible mount commands we have, or will that get caught later
if we just can't mount it?

-- 
Jesse Keating
Fedora -- Freedom² is a feature!
identi.ca: http://identi.ca/jkeating

Attachment: signature.asc
Description: This is a digitally signed message part

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/anaconda-devel-list

[Index of Archives]     [Kickstart]     [Fedora Users]     [Fedora Legacy List]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [Yosemite Photos]     [KDE Users]     [Fedora Tools]
  Powered by Linux