[lorax] Remove hfsplus-tools dependency (#818913)

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

 



Mac boot images are optional. Don't require hfsplus-tools
by default, but warn the user that he needs to install them
if he wants to create mac boot images.
---
 lorax.spec              |    1 -
 src/pylorax/__init__.py |    7 +++++++
 2 files changed, 7 insertions(+), 1 deletion(-)

diff --git a/lorax.spec b/lorax.spec
index e136430..fd39bec 100644
--- a/lorax.spec
+++ b/lorax.spec
@@ -23,7 +23,6 @@ Requires:       isomd5sum
 Requires:       glibc
 Requires:       util-linux
 Requires:       dosfstools
-Requires:       hfsplus-tools
 Requires:       genisoimage
 Requires:       parted
 Requires:       gzip
diff --git a/src/pylorax/__init__.py b/src/pylorax/__init__.py
index 44598d5..147fdbd 100644
--- a/src/pylorax/__init__.py
+++ b/src/pylorax/__init__.py
@@ -135,6 +135,13 @@ class Lorax(BaseLoraxClass):
 
         assert self._configured
 
+        if domacboot:
+            try:
+                subprocess.check_call("rpm -q hfsplus-tools")
+            except subprocess.CalledProcessError:
+                logger.critical("you need to install hfsplus-tools to create mac images")
+                sys.exit(1)
+
         # set up work directory
         self.workdir = workdir or tempfile.mkdtemp(prefix="pylorax.work.")
         if not os.path.isdir(self.workdir):
-- 
1.7.9.5

_______________________________________________
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