some fixes and first version with oprocess (out-ifdefed)

18 files changed, 139 insertions, 49 deletions

diff --git a/noncore/unsupported/oipkg/TODO b/noncore/unsupported/oipkg/TODO
index e7b017c..6e0e6ca 100644
--- a/noncore/unsupported/oipkg/TODO
+++ b/noncore/unsupported/oipkg/TODO
@@ -1,6 +1,9 @@
-* check for networt before installing from network
-* Settings Class
-* parse "to install" and "to remove" from status
+* check for network before installing from network
+* Settings Class / new setting dialog
+* parse "to install" and "to remove" from status, and change status
+	i.e. improve status handling / modify status file
+	dest keyword in status? -> cworth
 * install local file dialog
+* parse lists files after opening feed / doclnk
 * qcop
 * error handling
@@ -12,2 +15,6 @@
 	i.e. name, desc, files etc
 * display new list after update
+* make sure oipkg installs to the correct dest when getting packet to install from list
+	dest keyword?
+* cancel button while installing
+* duplicate button for settings-servers
diff --git a/noncore/unsupported/oipkg/installdialog.cpp b/noncore/unsupported/oipkg/installdialog.cpp
index a2c9702..bb50103 100644
--- a/noncore/unsupported/oipkg/installdialog.cpp
+++ b/noncore/unsupported/oipkg/installdialog.cpp
@@ -72,6 +72,6 @@ InstallDialog::InstallDialog( PackageManagerSettings* s, QWidget* parent,  const
 
     InstallDialogLayout->addWidget( GroupBoxOptions, 1, 0 );
-		toRemoveItem = new QCheckListItem( ListViewPackages, QObject::tr("To remove") );
-		toInstallItem = new QCheckListItem( ListViewPackages, QObject::tr("To install") );
+		toRemoveItem = new QCheckListItem( ListViewPackages, tr("To remove") );
+		toInstallItem = new QCheckListItem( ListViewPackages, tr("To install") );
 
 }
diff --git a/noncore/unsupported/oipkg/main.cpp b/noncore/unsupported/oipkg/main.cpp
index de25e27..f72540c 100644
--- a/noncore/unsupported/oipkg/main.cpp
+++ b/noncore/unsupported/oipkg/main.cpp
@@ -10,5 +10,5 @@ int main( int argc, char ** argv )
 {
   printf("This is oipkg\n");
-  printf("$ID$\n");
+  printf("$Id$\n");
   debugLevel = 2;
   if (argc > 1)
diff --git a/noncore/unsupported/oipkg/mainwindow.cpp b/noncore/unsupported/oipkg/mainwindow.cpp
index 27c9f4c..11ab406 100644
--- a/noncore/unsupported/oipkg/mainwindow.cpp
+++ b/noncore/unsupported/oipkg/mainwindow.cpp
@@ -34,4 +34,7 @@
 MainWindow::MainWindow( QWidget *parent, const char *name, WFlags f = 0 ) :
   QMainWindow( parent, name, f )
+//  packageListServers( QObject(parent), name ),
+//  packageListSearch( parent, name ),
+//  packageListDocLnk( parent, name )
 {	
   setCaption( tr("Package Manager") );
diff --git a/noncore/unsupported/oipkg/oipkg.pro b/noncore/unsupported/oipkg/oipkg.pro
index 9e2f5f8..7f60ed8 100644
--- a/noncore/unsupported/oipkg/oipkg.pro
+++ b/noncore/unsupported/oipkg/oipkg.pro
@@ -30,5 +30,6 @@ SOURCES		= main.cpp \
 INCLUDEPATH += $(OPIEDIR)/include
 DEPENDPATH	+= $(OPIEDIR)/ioclude
-LIBS            += -lqpe 
+LIBS            += -lqpe
+LIBS            += -lopie
 INTERFACES	= runwindow.ui \
 		pksettingsbase.ui
diff --git a/noncore/unsupported/oipkg/package.cpp b/noncore/unsupported/oipkg/package.cpp
index 4a09b40..97f7813 100644
--- a/noncore/unsupported/oipkg/package.cpp
+++ b/noncore/unsupported/oipkg/package.cpp
@@ -10,10 +10,18 @@
 #include "debug.h"
 
-//Package::~Package()
-//{
-//}
+Package::Package( QObject *parent=0, const char *name=0 )
+	: QObject(parent,name)
+{
+
+}
+
+Package::~Package()
+{
+}
 
-Package::Package( PackageManagerSettings *s )
+Package::Package( PackageManagerSettings *s, QObject *parent=0, const char *name=0 )
+	: QObject(parent,name)
 {
+	Package(parent,name);
 	init(s);
 }
@@ -38,5 +46,6 @@ void Package::init( PackageManagerSettings *s )
 }
 
-Package::Package( QStringList pack, PackageManagerSettings *s  )
+Package::Package( QStringList pack, PackageManagerSettings *s , QObject *parent=0, const char *name=0 )
+	: QObject(parent,name)
 {	
   init(s);
@@ -44,5 +53,5 @@ Package::Package( QStringList pack, PackageManagerSettings *s  )
 }
 
-Package::Package( QString n, PackageManagerSettings *s )
+Package::Package( QString n, PackageManagerSettings *s, QObject *parent=0, const char *name=0 )
 {	
   init(s);
@@ -58,5 +67,5 @@ Package::Package( QString n, PackageManagerSettings *s )
 }
 
-Package::Package( Package *pi )
+Package::Package( Package *pi, QObject *parent=0, const char *name=0 )
 {
   init(pi->settings);
diff --git a/noncore/unsupported/oipkg/package.h b/noncore/unsupported/oipkg/package.h
index 68090dc..f50b9b5 100644
--- a/noncore/unsupported/oipkg/package.h
+++ b/noncore/unsupported/oipkg/package.h
@@ -11,13 +11,14 @@
 #include "pksettings.h"
 
-class Package //: public QObject
+class Package : public QObject
 {
-//  	Q_OBJECT
+  	Q_OBJECT
  public:
-  Package(PackageManagerSettings *);
- // ~Package();
-  Package( QStringList, PackageManagerSettings * );
-  Package( QString, PackageManagerSettings * );
-  Package( Package* );
+  Package(QObject *parent=0, const char *name=0);
+  Package(PackageManagerSettings *s, QObject *parent=0, const char *name=0);
+ ~Package();
+  Package( QStringList, PackageManagerSettings *s, QObject *parent=0, const char *name=0 );
+  Package( QString, PackageManagerSettings *s, QObject *parent=0, const char *name=0 );
+  Package( Package*s, QObject *parent=0, const char *name=0 );
  	
   void    setValue( QString, QString );
diff --git a/noncore/unsupported/oipkg/packagelist.cpp b/noncore/unsupported/oipkg/packagelist.cpp
index 1b572e0..3cc0443 100644
--- a/noncore/unsupported/oipkg/packagelist.cpp
+++ b/noncore/unsupported/oipkg/packagelist.cpp
@@ -11,6 +11,6 @@ static QDict<Package>  *packageListAll;
 static int packageListAllRefCount = 0;
 
-PackageList::PackageList()
-  : packageIter( packageList )
+PackageList::PackageList(QObject *parent=0, const char *name=0)
+  : QObject(parent,name), packageIter( packageList )
 {
   empty=true;
@@ -25,9 +25,9 @@ PackageList::PackageList()
 }
 
-PackageList::PackageList( PackageManagerSettings* s)
-  : packageIter( packageList )
+PackageList::PackageList( PackageManagerSettings* s, QObject *parent, const char *name)
+  : QObject(parent,name), packageIter( packageList )
 {
   settings = s;
-  PackageList();
+  PackageList(parent, name);
 }
 
diff --git a/noncore/unsupported/oipkg/packagelist.h b/noncore/unsupported/oipkg/packagelist.h
index d0f8897..78dcd92 100644
--- a/noncore/unsupported/oipkg/packagelist.h
+++ b/noncore/unsupported/oipkg/packagelist.h
@@ -7,12 +7,12 @@
 #include "debug.h"
 
-class PackageList //:QObject
+class PackageList :QObject
 {
-//  	Q_OBJECT
+  Q_OBJECT
 public:
 //	static QString all = QObject::tr("All");
 
-  PackageList();
-  PackageList( PackageManagerSettings* );
+  PackageList (QObject *parent=0, const char *name=0);
+  PackageList( PackageManagerSettings *s, QObject *parent=0, const char *name=0);
  	virtual ~PackageList();
   void insertPackage( Package* );
diff --git a/noncore/unsupported/oipkg/packagelistdoclnk.cpp b/noncore/unsupported/oipkg/packagelistdoclnk.cpp
index f31f742..b42cfdd 100644
--- a/noncore/unsupported/oipkg/packagelistdoclnk.cpp
+++ b/noncore/unsupported/oipkg/packagelistdoclnk.cpp
@@ -9,12 +9,12 @@
 #include "pksettings.h"
 
-PackageListDocLnk::PackageListDocLnk(PackageManagerSettings* s)
+PackageListDocLnk::PackageListDocLnk(PackageManagerSettings* s, QObject *parent=0, const char *name=0)
 	: PackageList(s)
 {
-	PackageListDocLnk();
+	PackageListDocLnk(parent, name);
 }
 
-PackageListDocLnk::PackageListDocLnk()
-	: PackageList()
+PackageListDocLnk::PackageListDocLnk(QObject *parent=0, const char *name=0)
+	: PackageList(parent, name)
 {
 	Config cfg( "oipkg", Config::User );
diff --git a/noncore/unsupported/oipkg/packagelistdoclnk.h b/noncore/unsupported/oipkg/packagelistdoclnk.h
index fe73658..a63fb59 100644
--- a/noncore/unsupported/oipkg/packagelistdoclnk.h
+++ b/noncore/unsupported/oipkg/packagelistdoclnk.h
@@ -10,6 +10,6 @@ class PackageListDocLnk : public PackageList
 {
 public:
-	PackageListDocLnk();
-	PackageListDocLnk( PackageManagerSettings* s);
+	PackageListDocLnk(QObject *parent=0, const char *name=0);
+	PackageListDocLnk( PackageManagerSettings *s, QObject *parent=0, const char *name=0);
 	virtual ~PackageListDocLnk();
 public slots:
diff --git a/noncore/unsupported/oipkg/packagelistitem.cpp b/noncore/unsupported/oipkg/packagelistitem.cpp
index 55047d5..0c9c676 100644
--- a/noncore/unsupported/oipkg/packagelistitem.cpp
+++ b/noncore/unsupported/oipkg/packagelistitem.cpp
@@ -160,5 +160,5 @@ void PackageListItem::displayDetails()
   setText(0, package->name()+sod );
 	nameItem->setText( 0, QObject::tr("Name: ")+package->name());
-	linkItem->setText( 0, QObject::tr("Link: ")+QString(package->link()?QObject::tr("Yes"):QObject::tr("No")) );
+	linkItem->setText( 0, QObject::tr("Link: ")+package->link()?QObject::tr("Yes"):QObject::tr("No"));
   destItem->setText( 0, QObject::tr("Destination: ")+package->dest() );
   statusItem->setText( 0, QObject::tr("Status: ")+package->status() );
diff --git a/noncore/unsupported/oipkg/packagelistview.cpp b/noncore/unsupported/oipkg/packagelistview.cpp
index 70aa48a..2bb74e9 100644
--- a/noncore/unsupported/oipkg/packagelistview.cpp
+++ b/noncore/unsupported/oipkg/packagelistview.cpp
@@ -60,5 +60,5 @@ void PackageListView::showPopup()
   if ( !activePackage->installed() )
   {
-	  popupMenu->insertItem( QObject::tr("Install to"), destsMenu );
+	  popupMenu->insertItem( tr("Install to"), destsMenu );
   	QStringList dests = settings->getDestinationNames();
    	QString ad = settings->getDestinationName();
@@ -76,9 +76,9 @@ void PackageListView::showPopup()
 		   this, SLOT( changePackageDest( int ) ) );
    }else{
-   	popupAction = new QAction( QObject::tr("Remove"),QString::null,  0, this, 0 );
+   	popupAction = new QAction( tr("Remove"),QString::null,  0, this, 0 );
 		popupAction->addTo( popupMenu );
   	connect( popupAction, SIGNAL( activated() ),
 	   	this , SLOT( toggleProcess() ) );
-   	popupAction = new QAction( QObject::tr("Reinstall"),QString::null,  0, this, 0 );
+   	popupAction = new QAction( tr("Reinstall"),QString::null,  0, this, 0 );
 		popupAction->addTo( popupMenu );
   	popupAction->setEnabled( false );
diff --git a/noncore/unsupported/oipkg/pksettings.cpp b/noncore/unsupported/oipkg/pksettings.cpp
index 09975b0..59d5a3e 100644
--- a/noncore/unsupported/oipkg/pksettings.cpp
+++ b/noncore/unsupported/oipkg/pksettings.cpp
@@ -242,4 +242,5 @@ void PackageManagerSettings::readInstallationSettings()
   Config cfg( "oipkg", Config::User );
   cfg.setGroup( "Settings" );
+
   installationSettingsCount = cfg.readNumEntry( "count", -1 );
   currentSetting  = cfg.readNumEntry( "current", 0 );// o should be -1
diff --git a/noncore/unsupported/oipkg/pksettings.h b/noncore/unsupported/oipkg/pksettings.h
index dcc1e3a..8f9fe62 100644
--- a/noncore/unsupported/oipkg/pksettings.h
+++ b/noncore/unsupported/oipkg/pksettings.h
@@ -10,5 +10,5 @@
 class PackageManagerSettings : public PackageManagerSettingsBase
 {
-//	 Q_OBJECT
+	 Q_OBJECT
 public: 
   PackageManagerSettings( QWidget* ,  const char* , WFlags  );
diff --git a/noncore/unsupported/oipkg/pmipkg.cpp b/noncore/unsupported/oipkg/pmipkg.cpp
index 7b3fa1e..2a534d2 100644
--- a/noncore/unsupported/oipkg/pmipkg.cpp
+++ b/noncore/unsupported/oipkg/pmipkg.cpp
@@ -4,5 +4,7 @@
 #include "packagelistitem.h"
 
-#include <qpe/process.h>
+//#infdef OPROCESS
+#include <opie/oprocess.h>
+//#endif
 #include <qpe/resource.h>
 #include <qpe/config.h>
@@ -33,8 +35,20 @@ PmIpkg::PmIpkg( PackageManagerSettings* s, QWidget* p,  const char * name, WFlag
   settings = s;
  	runwindow = new RunWindow( p, name, true, f );
+#ifdef OPROCESS
+  ipkgProcess = new OProcess();
+  connect ( ipkgProcess, SIGNAL(receivedStdout(OProcess*,char*,int)),
+  					this, SLOT(getIpkgOutput(OProcess*,char*,int)));
+
+  connect ( ipkgProcess, SIGNAL(receivedStderr(OProcess*,char*,int)),
+  					this, SLOT(getIpkgOutput(OProcess*,char*,int)));
+  installDialog = 0;
+#endif
 }
 
 PmIpkg::~PmIpkg()
 {
+#ifdef OPROCESS
+ delete ipkgProcess;
+#endif
 }
 
@@ -44,4 +58,9 @@ bool PmIpkg::runIpkg(const QString& args, const QString& dest )
   QDir::setCurrent("/tmp");
   QString cmd = "/usr/bin/ipkg ";
+#ifdef OPROCESS
+  ipkgProcess->clearArguments();
+  *ipkgProcess << "/usr/bin/ipkg ";
+  QString cmd = "";
+#endif
 	pvDebug( 3,"PmIpkg::runIpkg got dest="+dest);
 	if ( dest == "" )
@@ -66,4 +85,39 @@ bool PmIpkg::runIpkg(const QString& args, const QString& dest )
   out( "running:\n"+cmd+"\n" );
   pvDebug(2,"running:"+cmd);
+#ifdef OPROCESS
+  *ipkgProcess <<  args;
+  out( "running:\n" + cmd);
+  *ipkgProcess << cmd;
+
+//debug
+  ipkgProcess->clearArguments();
+  *ipkgProcess << "/bin/ls ";
+//debug
+  QValueList<QCString> a = ipkgProcess->args();
+  QValueList<QCString>::Iterator it;
+  for( it = a.begin(); it != a.end(); ++it )
+  {
+  	out( *it );
+   	cmd += *it;
+  }
+
+  pvDebug(2,"running:"+cmd);
+  qApp->processEvents();
+//  sleep(1);
+  ret = ipkgProcess->start(OProcess::NotifyOnExit,OProcess::AllOutput);
+  if ( !ret ) {
+     pvDebug(2,"Could not execute '" + cmd);
+     out("\nError while executing "+ cmd+"\n\n");
+     out("\nError while executing\n\n");
+ //    return false;
+  }
+
+  while ( ipkgProcess->isRunning() )
+  {
+   out(".");	
+   pvDebug(7,"wait for oprocess to terminate");
+   qApp->processEvents();
+  };
+#else
   qApp->processEvents();
   FILE *fp;
@@ -92,4 +146,5 @@ bool PmIpkg::runIpkg(const QString& args, const QString& dest )
      pclose(fp);
   }
+#endif
   //out( "Finished!");
   pvDebug(2,QString(ret?"success\n":"failure\n"));
@@ -100,5 +155,5 @@ void PmIpkg::makeLinks(Package *pack)
 {
 	pvDebug( 2, "PmIpkg::makeLinks "+ pack->name());
-  linkPackage( pack->installName(), pack->dest() );
+  linkPackage( pack->name(), pack->dest() );
 }
 
@@ -243,4 +298,5 @@ void PmIpkg::commit()
  	installDialog->close();
   delete installDialog;
+  installDialog = 0;
   out(tr("\nAll done."));
 }
@@ -397,2 +453,14 @@ void PmIpkg::clearLists()
  	to_install.clear();
 }
+
+
+void  PmIpkg::getIpkgOutput(OProcess *proc, char *buffer, int buflen)
+{
+  QString lineStr, lineStrOld;
+  lineStr = buffer;
+  lineStr=lineStr.left(buflen);
+  //Configuring opie-oipkg...Done
+  if (lineStr!=lineStrOld)
+                out(lineStr);
+  lineStrOld = lineStr;
+}
\ No newline at end of file
diff --git a/noncore/unsupported/oipkg/pmipkg.h b/noncore/unsupported/oipkg/pmipkg.h
index d53a74d..2c92348 100644
--- a/noncore/unsupported/oipkg/pmipkg.h
+++ b/noncore/unsupported/oipkg/pmipkg.h
@@ -3,4 +3,5 @@
 
 
+#include <opie/oprocess.h>
 #include <qobject.h>
 #include <qlist.h>
@@ -41,5 +42,9 @@ public slots:
   void removeLinks( const QString &dest );
 
+//private slots:
+	void getIpkgOutput(OProcess *proc, char *buffer, int buflen);
+
 private:
+	OProcess *ipkgProcess;
   PackageManagerSettings* settings;
   RunWindow *runwindow;
@@ -47,9 +52,4 @@ private:
   QList<Package> to_remove;
   QList<Package> to_install;
-  QString fileNameToInstall;
-  QCheckBox *_force_reinstall;
-  QCheckBox *_force_remove;
-  QCheckBox *_force_depends;
-//  void startDialog();
   void makeLinks(Package*);
   void linkPackage( QString, QString );
diff --git a/noncore/unsupported/oipkg/settings.h b/noncore/unsupported/oipkg/settings.h
index f01cb69..9cb7a9b 100644
--- a/noncore/unsupported/oipkg/settings.h
+++ b/noncore/unsupported/oipkg/settings.h
@@ -9,5 +9,5 @@ class PackageManagerSettings : public PackageManagerSettingsBase
 //class PackageManagerSettings : private PackageManagerSettingsBase
 {
-  //	 Q_OBJECT
+  	 Q_OBJECT
 public: 
   PackageManagerSettings( QWidget* ,  const char* , WFlags  );