tags 519548 +patch
thanks

Here is a patch to fix mondo so that it stops seeing if cdrecord is
available on systems that have wodim installed.

I just tested it now and it seems to work fine, but I really think that
mondo should have a function to check alternative names for some
utilities (the case of wodim, cdrecord, dvdrecord, whatever) is
potentially just one of these cases.

I may be submitting other patches to the project, if this is desired.


Regards,

-- 
Rogério Brito : rbr...@{mackenzie,ime.usp}.br : GPG key 1024D/7C2CAEB8
http://www.ime.usp.br/~rbrito : http://meusite.mackenzie.com.br/rbrito
Projects: algorithms.berlios.de : lame.sf.net : vrms.alioth.debian.org
--- mondo-2.2.7/src/common/libmondo-tools.c	2008-10-02 20:50:44.000000000 -0300
+++ mondo-2.2.7/src/common/libmondo-tools.c	2009-04-08 07:37:45.000000000 -0300
@@ -598,8 +598,10 @@
 			strcpy(cdr_exe, "cdrecord");
 		} else if (find_home_of_exe("dvdrecord")) {
 			strcpy(cdr_exe, "dvdrecord");
+		} else if (find_home_of_exe("wodim")) {
+			strcpy(cdr_exe, "wodim");
 		} else {
-			fatal_error("Please install either cdrecord or dvdrecord.");
+			fatal_error("Please install cdrecord, dvdrecord or wodim.");
 		}
 		if (bkpinfo->nonbootable_backup) {
 			strcpy(mondo_mkisofs_sz, MONDO_MKISOFS_NONBOOT);
@@ -1007,8 +1009,9 @@
 	}
 #endif
 	retval += whine_if_not_found(MKE2FS_OR_NEWFS);
-	retval += whine_if_not_found("mkisofs");
-	if (system("which dvdrecord > /dev/null 2> /dev/null")) {
+	retval += whine_if_not_found("genisoimage");
+	if (system("which dvdrecord > /dev/null 2> /dev/null") &&
+	    system("which wodim > /dev/null 2> /dev/null")) {
 		retval += whine_if_not_found("cdrecord");
 	}
 	retval += whine_if_not_found("bzip2");

Reply via email to