Re: [PATCH] sanityCheckMountPoint is no longer called from anywhere.

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

 



On Tue, 2009-09-22 at 16:39 -0400, Chris Lumens wrote:
> ---
>  partIntfHelpers.py |   31 -------------------------------
>  1 files changed, 0 insertions(+), 31 deletions(-)

I was going to revive sanity checking of mountpoints after F12 (string
freeze and all), but I suppose it is possible that my approach will
involve removing this function completely. Looks fine if you really want
to remove it.

Dave

> 
> diff --git a/partIntfHelpers.py b/partIntfHelpers.py
> index a8cde97..6ab2f03 100644
> --- a/partIntfHelpers.py
> +++ b/partIntfHelpers.py
> @@ -79,37 +79,6 @@ def sanityCheckLogicalVolumeName(logvolname):
>  		     "are letters, digits, '.' or '_'.")
>      return None
>  
> -def sanityCheckMountPoint(mntpt, fstype, preexisting, format):
> -    """Sanity check that the mountpoint is valid.
> -
> -    mntpt is the mountpoint being used.
> -    fstype is the file system being used on the request.
> -    preexisting is whether the request was preexisting (request.preexist)
> -    format is whether the request is being formatted or not
> -    """
> -    if mntpt:
> -        passed = 1
> -        if not mntpt:
> -            passed = 0
> -        else:
> -            if mntpt[0] != '/' or (len(mntpt) > 1 and mntpt[-1:] == '/'):
> -                passed = 0
> -	    elif mntpt.find(' ') > -1:
> -		passed = 0
> -                
> -        if not passed:
> -            return _("The mount point %s is invalid.  Mount points must start "
> -                     "with '/' and cannot end with '/', and must contain "
> -                     "printable characters and no spaces." % mntpt)
> -        else:
> -            return None
> -    else:
> -        if (fstype and fstype.mountable and (not preexisting or format)):
> -            return _("Please specify a mount point for this partition.")
> -        else:
> -            # its an existing partition so don't force a mount point
> -            return None
> -
>  def doDeleteDevice(intf, storage, device, confirm=1, quiet=0):
>      """Delete a partition from the request list.
>  

_______________________________________________
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