[MMBASE CVS] applications/resources/src/org/mmbase/util/images ImageMagickImageConverter.java

[Michiel Meeuwissen]

Update of /var/cvs/applications/resources/src/org/mmbase/util/images
In directory 
james.mmbase.org:/tmp/cvs-serv4240/applications/resources/src/org/mmbase/util/images

Modified Files:
        ImageMagickImageConverter.java 
Log Message:
logging only


See also: 
http://cvs.mmbase.org/viewcvs/applications/resources/src/org/mmbase/util/images


Index: ImageMagickImageConverter.java
===================================================================
RCS file: 
/var/cvs/applications/resources/src/org/mmbase/util/images/ImageMagickImageConverter.java,v
retrieving revision 1.16
retrieving revision 1.17
diff -u -b -r1.16 -r1.17
--- ImageMagickImageConverter.java      22 Apr 2009 08:03:58 -0000      1.16
+++ ImageMagickImageConverter.java      30 Apr 2009 09:29:52 -0000      1.17
@@ -27,7 +27,7 @@
  * @author Michiel Meeuwissen
  * @author Nico Klasens
  * @author Jaco de Groot
- * @version $Id: ImageMagickImageConverter.java,v 1.16 2009/04/22 08:03:58 
michiel Exp $
+ * @version $Id: ImageMagickImageConverter.java,v 1.17 2009/04/30 09:29:52 
michiel Exp $
  */
 public class ImageMagickImageConverter extends AbstractImageConverter 
implements ImageConverter {
     private static final Logger log = 
Logging.getLoggerInstance(ImageMagickImageConverter.class);
@@ -67,18 +67,18 @@
         ByteArrayOutputStream outputStream = new ByteArrayOutputStream();
         switch (method) {
         case METHOD_LAUNCHER: {
-            try {
-
-                CommandLauncher launcher = new CommandLauncher("ConvertImage");
                 log.debug("Starting convert");
                 List<String> cmd = new ArrayList<String>();
                 for (String arg : args) {
                     cmd.add(arg);
                 }
+            try {
+
+                CommandLauncher launcher = new CommandLauncher("ConvertImage");
                 launcher.execute(converterPath, cmd.toArray(EMPTY));
                 launcher.waitAndRead(outputStream, errorStream);
             } catch (ProcessException e) {
-                log.error("Convert test failed. " + converterPath + " (" + 
e.getMessage() + ")");
+                log.error("Convert test failed. " + converterPath + cmd + " (" 
+ e.getMessage() + ")");
             }
             break;
         }
@@ -190,7 +190,7 @@
             // now check if the specified ImageConvert.converterRoot does 
exist and is a directory
             File checkConvDir = new File(converterRoot).getAbsoluteFile();
             if (!checkConvDir.exists()) {
-                log.error( "ImageConvert.ConverterRoot " + converterRoot + " 
in " + configFile + " does not exist");
+                log.error( "ImageConvert.ConverterRoot " + converterRoot + " 
in " + configFile + " does not exist (using any way)");
             } else if (!checkConvDir.isDirectory()) {
                 log.error( "ImageConvert.ConverterRoot " + converterRoot + " 
in " + configFile + " is not a directory");
             } else {
_______________________________________________
Cvs mailing list
Cvs@lists.mmbase.org
http://lists.mmbase.org/mailman/listinfo/cvs