Switch from 1) calling /etc/init.d/pcmcia to calling cardctl eject, cardctl insert..

and 2) OProcess instead of system()

1 files changed, 14 insertions, 13 deletions

diff --git a/noncore/settings/networksettings/wlan/wlanimp.cpp b/noncore/settings/networksettings/wlan/wlanimp.cpp
index 1782c22..db03266 100644
--- a/noncore/settings/networksettings/wlan/wlanimp.cpp
+++ b/noncore/settings/networksettings/wlan/wlanimp.cpp
@@ -11,13 +11,14 @@
 #include <qradiobutton.h>
 #include <qcheckbox.h>
 #include <qtabwidget.h>
 #include <qcombobox.h>
 
 /* system() */
-#include <stdlib.h>
+//#include <stdlib.h>
+#include <opie/oprocess.h>
 
 #define WIRELESS_OPTS "/etc/pcmcia/wireless.opts"
 
 /**
  * Constructor, read in the wireless.opts file for parsing later.
  */ 
@@ -222,25 +223,25 @@ void WLANImp::accept(){
   changeAndSaveSettingFile();
   
   // Try to save the interfaces settings.
   if(!interfaceSetup->saveChanges())
     return;
 
-  // Restart the device now that the settings have changed
-  QString initpath;
-  if( QDir("/etc/rc.d/init.d").exists() )
-    initpath = "/etc/rc.d/init.d";
-  else if( QDir("/etc/init.d").exists() )
-    initpath = "/etc/init.d";
+  OProcess insert;
+//  OProcess eject;
 
-  // It would be kinda cool if we didn't have to do this and could just to ifup/down
-  
-  if( initpath )
-    system(QString("%1/pcmcia stop").arg(initpath));
-  if( initpath )
-    system(QString("%1/pcmcia start").arg(initpath));
+  insert << "sh -c \"cardctl insert && cardctl eject\"";
+//  eject << "cardctl eject";
+
+//  if (!eject.start(OProcess::Block, OProcess::NoCommunication) ) {
+//    qWarning("could not start cardctl eject");
+//  }
+
+  if (!insert.start(OProcess::DontCare, OProcess::NoCommunication) ) {
+    qWarning("could not start cardctl insert");
+  }
 
   // Close out the dialog
   QDialog::accept();
 }
 
 // wlanimp.cpp