OTHER: Change a Parameter from QDir to QString.

I think this will be needed when I add zipped skin support later

README

@@ -1,4 +1,4 @@
-Required dependencys:
+Required dependencies:
 	* QT4.2.x           http://www.trolltech.no/
 	* XMMS2             http://xmms2.xmms.org/
 	* XMMS2 C++ bindings (make sure you have boost_signals when compiling xmms2)
@@ -16,6 +16,8 @@ Compiling:
 	5) ????
 	6) profit
 
+	To change the install directory, use 'qmake PREFIX=<path>' at step 1
+
 Compilation problems:
 	- Errormessage "WARNING: Found potential symbol conflict of <somefile>"
 	  -> make sure you use uic from Qt4 (uic -version)

config.pri

@@ -2,6 +2,3 @@ DEFINES += PROMOE_VERSION="\"\\\"0.1-dev\\\"\""
 
 DEPENDPATH += $PWD/src $PWD/lib
 INCLUDEPATH += $$PWD/src $$PWD/lib
-
-PREFIX = /usr/local
-BINDIR = $$PREFIX/bin

promoe.pro

@@ -4,6 +4,11 @@ SUBDIRS = lib src
 
 include (config.pri)
 
-target.path = $$BINDIR
-target.files = promoe
-INSTALLS += target
+isEmpty(PREFIX): PREFIX = /usr/local
+isEmpty(BINDIR): BINDIR = $$PREFIX/bin
+
+message ( "Install path set to:" $$PREFIX )
+
+binarys.path = $$BINDIR
+binarys.files = promoe
+INSTALLS += binarys

src/Skin.cpp

@@ -71,10 +71,11 @@ Skin::setSkin (const QString& name)
 }
 
 const QPixmap
-Skin::getPixmap (const QString& file, QDir dir)
+Skin::getPixmap (const QString& file, const QString &path)
 {
 	/* check for files in zip and check if file exists */
 
+	QDir dir (path);
 	dir.setFilter (QDir::Files);
 
 	QFileInfoList list = dir.entryInfoList();
@@ -93,10 +94,10 @@ Skin::getPixmap (const QString& file, QDir dir)
 const QPixmap
 Skin::getPixmap (const QString& file)
 {
-	QDir dir;
+/*	QDir dir;
 
 	dir.setPath (m_path);
-/*	dir.setFilter (QDir::Files);
+	dir.setFilter (QDir::Files);
 
 	QFileInfoList list = dir.entryInfoList();
 	for (int i = 0; i < list.size(); ++i) {
@@ -109,7 +110,7 @@ Skin::getPixmap (const QString& file)
 
 	return QPixmap ();
 */
-	return getPixmap (file, dir);
+	return getPixmap (file, m_path);
 }
 
 bool

src/Skin.h

@@ -37,7 +37,7 @@ class Skin : public QObject
 		static Skin *getInstance (void);
 
 		bool setSkin (const QString& name);
-		static const QPixmap getPixmap (const QString&, QDir);
+		static const QPixmap getPixmap (const QString&, const QString&);
 
 		const QSize getSize (uint item) const { return m_sizes[item]; };
 		const QPoint getPos (uint item) const { return m_positions[item]; };

src/SkinChooser.cpp

@@ -63,20 +63,19 @@ SkinList::SkinList (QWidget *parent) : QListWidget (parent)
 		settings.setValue ("skin/searchpath", searchpath);
 	}
 
-	QFileInfoList list;
 	QDir d;
-	d.setFilter (QDir::Dirs);
+	d.setFilter (QDir::Dirs|QDir::NoDotAndDotDot);
+	QFileInfoList list;
 	foreach (QString path, searchpath) {
 		d.setPath (path);
 		list += d.entryInfoList();
 	}
 
 	foreach (QFileInfo fileInfo, list) {
-		QDir dir (fileInfo.filePath());
-		QPixmap p = Skin::getPixmap ("main", dir);
+		QPixmap p = Skin::getPixmap ("main", fileInfo.filePath());
 		if (!p.isNull()) {
-			new SkinChooserItem (QIcon (p), dir.dirName(), dir.absolutePath(),
-			                     this);
+			new SkinChooserItem (QIcon (p), fileInfo.baseName (),
+			                     fileInfo.absoluteFilePath (), this);
 		}
 	}