Re: [PATCH 2/4] Use Ext4FS for all extended filesystems.

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

 



On Fri, 2011-09-02 at 13:32 -0500, David Lehman wrote:
> This should eliminate the issues caused by users specifying
> that their ext[23] filesystems should be mounted as ext4 as
> now they will all share a format class.

I should point out that this will have some other consequences. First,
we will no longer offer to create new ext2 or ext3 filesystems. (In
fact, specifying ext2 or 3 via kickstart will lead to us making you an
ext4 fs.) Second, we will no longer offer to migrate existing ext2
filesystems to ext3. I am personally okay with both, but I have no doubt
that someone will object.

Dave

> 
> Related: rhbz#599119
> Related: rhbz#649171
> ---
>  pyanaconda/storage/formats/fs.py |    5 +++--
>  1 files changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/pyanaconda/storage/formats/fs.py b/pyanaconda/storage/formats/fs.py
> index 90d97e7..abd83b8 100644
> --- a/pyanaconda/storage/formats/fs.py
> +++ b/pyanaconda/storage/formats/fs.py
> @@ -1025,7 +1025,7 @@ class Ext2FS(FS):
>          argv = ["-p", self.device, "%dM" % (self.targetSize,)]
>          return argv
>  
> -register_device_format(Ext2FS)
> +#register_device_format(Ext2FS)
>  
> 
>  class Ext3FS(Ext2FS):
> @@ -1037,12 +1037,13 @@ class Ext3FS(Ext2FS):
>      _defaultMigrateOptions = ["-O", "extents"]
>      partedSystem = fileSystemType["ext3"]
>  
> -register_device_format(Ext3FS)
> +#register_device_format(Ext3FS)
>  
> 
>  class Ext4FS(Ext3FS):
>      """ ext4 filesystem. """
>      _type = "ext4"
> +    _udevTypes = ["ext2", "ext3"]
>      _defaultFormatOptions = ["-t", "ext4"]
>      _migratable = False
>      _modules = ["ext4"]


_______________________________________________
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