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