Re: [PATCH V3] Add support for HFS+ partitions

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

 



On Tue, 2012-02-14 at 15:22 -0500, Matthew Garrett wrote:
> We'll want to actually create HFS+ partitions, so add the missing bits
> for that.
> ---
> 
> _mountType was missing - hfs+ is called hfsplus in-kernel, so it's needed.

Ack.

> 
>  anaconda.spec.in                 |    1 +
>  pyanaconda/storage/formats/fs.py |    8 +++++++-
>  2 files changed, 8 insertions(+), 1 deletions(-)
> 
> diff --git a/anaconda.spec.in b/anaconda.spec.in
> index 07557e9..7966b93 100644
> --- a/anaconda.spec.in
> +++ b/anaconda.spec.in
> @@ -135,6 +135,7 @@ Requires: zenity
>  %endif
>  Requires: createrepo >= %{createrepover}
>  Requires: squashfs-tools
> +Requires: hfsplus-tools
>  Requires: genisoimage >= %{genisoimagever}
>  Requires: GConf2 >= %{gconfversion}
>  %ifarch %{ix86} x86_64
> diff --git a/pyanaconda/storage/formats/fs.py b/pyanaconda/storage/formats/fs.py
> index 0942402..51d5ee7 100644
> --- a/pyanaconda/storage/formats/fs.py
> +++ b/pyanaconda/storage/formats/fs.py
> @@ -1357,11 +1357,17 @@ class AppleBootstrapFS(HFS):
>  register_device_format(AppleBootstrapFS)
>  
> 
> -# this doesn't need to be here
>  class HFSPlus(FS):
>      _type = "hfs+"
>      _modules = ["hfsplus"]
>      _udevTypes = ["hfsplus"]
> +    _mkfs = "mkfs.hfsplus"
> +    _fsck = "fsck.hfsplus"
> +    _packages = ["hfsplus-tools"]
> +    _formattable = True
> +    _mountType = "hfsplus"
> +    _minSize = 1
> +    _maxSize = 2 * 1024 * 1024
>      partedSystem = fileSystemType["hfs+"]
>  
>  register_device_format(HFSPlus)


_______________________________________________
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