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

Bug#565978: kommander does not recognize --stdin option



tags 565978 + patch                                                         
thanks                                                                               
                                                                                    
Patch created and forwarded upstream
Also requires the use of patch(es) in bug #631671 for kommander to run


-- 

Jim Scadden
Index: kdewebdev-4.12.3/kommander/executor/instance.cpp
===================================================================
--- kdewebdev-4.12.3.orig/kommander/executor/instance.cpp
+++ kdewebdev-4.12.3/kommander/executor/instance.cpp
@@ -102,7 +102,7 @@ bool Instance::build(const KUrl& fname)
   m_instance = 0;
   m_textInstance = 0;
 
-  if (!fname.isValid() || !isFileValid(fname))
+  if (fname.isValid() && !isFileValid(fname))
     return false; // Check if file is correct
 
   // create the main instance, must inherit QDialog
Index: kdewebdev-4.12.3/kommander/executor/main.cpp
===================================================================
--- kdewebdev-4.12.3.orig/kommander/executor/main.cpp
+++ kdewebdev-4.12.3/kommander/executor/main.cpp
@@ -80,7 +80,7 @@ int main(int argc, char *argv[])
   KApplication app;
   
   QObject::connect(&app, SIGNAL(lastWindowClosed()), &app, SLOT(quit()));
-  if (!args->count())
+  if (!args->isSet("stdin") && !args->count())
   {
     KMessageBox::sorry(0, i18n("Error: no dialog given. Use --stdin option to read dialog from standard input.\n"));
     return -1;

Reply to: