OSX: Guess actual executable from the app bundle directory

FossilOrigin-Name: 7988e53166771ea71d916a1ab714eb671d07dea8

fuel.pro

@@ -122,6 +122,7 @@ unix:!macx {
 
 macx {
 	SOURCES += ext/qtkeychain/keychain_mac.cpp
+	LIBS += -framework CoreFoundation -framework Security
 }
 
 win32 {

manifest

@@ -1,5 +1,5 @@
-C Custom\sactions\snow\sexpect\smacros\sfor\sfile\sand\sfolder\sexpansion\nCustom\sactions\snow\strigger\seven\swithout\sa\sselection\nUse\snative\spath\sseparators\sin\scustom\sactions\n
-D 2015-08-21T10:54:20.418
+C OSX:\sGuess\sactual\sexecutable\sfrom\sthe\sapp\sbundle\sdirectory
+D 2015-08-22T17:32:24.250
 F .travis.yml 77966888a81c4ceee1fcc79bce842c9667ad8a35
 F debian/changelog eb4304dfcb6bb66850ec740838090eb50ce1249b
 F debian/compat b6abd567fa79cbe0196d093a067271361dc6ca8b
@@ -37,7 +37,7 @@ F ext/qtkeychain/qkeychain_export.h d756528188ef9bf3c4461ecc80048f06c362b54b
 F ext/qtkeychain/testclient.cpp cb1290a9584b627306a7bfdf1c02a8bbae503f5f
 F ext/qtkeychain/translations/qtkeychain_de.ts 0a70c8205c066c30ed8172f0670942de1fc5eede
 F ext/qtkeychain/translations/qtkeychain_ro.ts f16939382fd1a047b0692426bc82847347f14b32
-F fuel.pro 7802dc9513da36539818ee1e927ec1e3494ecb54
+F fuel.pro 2de14a4a6bd3e5af9d9e7c78f870453fedb38577
 F intl/convert.bat 357d461ee8c6a7be6d2f60ac77c3232678ffb513 x
 F intl/convert.sh ac6edc8d99b575601cf9c6b85b18c9e25875b6e0 x
 F intl/de_DE.ts e2faceab920ac60c97bbc6fba038e261d51fc741
@@ -231,7 +231,7 @@ F src/FslSettingsDialog.cpp e00907d493fba469e48a008aecda88426350b5ac
 F src/FslSettingsDialog.h dfe2a61884a55a74cbb9206b6f6b482b979725e7
 F src/LoggedProcess.cpp 2a1e5c94bc1e57c8984563e66c210e43a14dc60c
 F src/LoggedProcess.h 85df7c635c807a5a0e8c4763f17a0752aaff7261
-F src/MainWindow.cpp dfa81f94f04e418bdd22985b898855733842d496
+F src/MainWindow.cpp 48ed62679d02f918479cd1610309443cd15d57ae
 F src/MainWindow.h f4cffbe4d360d30aa2eeaa25fc6d50d0a39c617f
 F src/RemoteDialog.cpp 8540cc5e2e41c4127ed8a028d84691604fa6ecac
 F src/RemoteDialog.h 5e0438c2bd7c79b1bb44bfbd58c2181b544a9e5d
@@ -243,9 +243,9 @@ F src/Settings.cpp 258d3f466f6a125ce2b8519d6d57a312cbc44a3f
 F src/Settings.h 0a10b0b83fe804bdc7dac58eed06b5b6ee422055
 F src/SettingsDialog.cpp fa0c70eaf0fa7edb15de302d041cdb552fe523d5
 F src/SettingsDialog.h 5eb3ae2cbb00ab5544e1889860f5376f69fe47cd
-F src/Utils.cpp a375d6b641658b49d84462773d1e052de0c716dd
+F src/Utils.cpp cfadf3ee981f5ead8635c395610421f8e04a9a42
 F src/Utils.h c6341ee49a8fc35f215facb196d70bf9b1f2fc0f
-F src/Workspace.cpp feab8b238a99cf1a60731aedf07af96010d9795d
+F src/Workspace.cpp c46aeb616712e42dcee72774a572af3a9803dbf1
 F src/Workspace.h 54eef32658b13a34fe78ae26887420e8ff358eaa
 F src/main.cpp d8c65ea5e54102e4989fef9fd8cfd4f13ef8a8f0
 F tools/git-push.sh 62cc58434cae5b7bcd6bd9d4cce8b08739f31cd7 x
@@ -256,11 +256,11 @@ F ui/CloneDialog.ui 4886e7d4f258ea8b852b5eefc860396e35145712
 F ui/CommitDialog.ui 1e5dafa742e9ae07ec937bcda8cda3297ddc6199
 F ui/FileActionDialog.ui 89bb4dc2d0b8adcd41adcb11ec65f2028a09a12d
 F ui/FslSettingsDialog.ui eb3d4cb764cab90b01e82922237d8c42d6ce1749
-F ui/MainWindow.ui f9774e6dddb9462d8072bffd6c511bee7f470b9d
+F ui/MainWindow.ui e2a18caa7482b3ee0dff477592cdc9574b35fe4f
 F ui/RemoteDialog.ui 95a4750d972ed8c49bb10b95db91ff16cfe2dd0b
 F ui/RevisionDialog.ui 27c3b98c665fec014a50cbf3352c0627f75e68cd
 F ui/SettingsDialog.ui 2e1b6ce7a49100088c5649292c1319e62e0302e1
-P 9b67709b92eae80e70f8ae2b775014092f820fdf
-R 53802be5132d1fc255734da4b05ec8a0
-U Kostas
-Z 11e2710ccd6ed208d70aec7542a52542
+P cb27c4e2a0777b5c3fa64a07d19cd6b89e05c76f
+R 19ead9c938bef0606914a30fe37378b3
+U kostas
+Z 6b9bc678ab29fb68e3f8be6bb65cb34f

manifest.uuid

@@ -1 +1 @@
-e017a9126cf63ed060dd88accfcabbd9726e9aa7
+7988e53166771ea71d916a1ab714eb671d07dea8

src/MainWindow.cpp

@@ -263,6 +263,19 @@ MainWindow::MainWindow(Settings &_settings, QWidget *parent, QString *workspaceP
 		a->setIconVisibleInMenu(false);
 	foreach(QAction *a, ui->menuFile->actions())
 		a->setIconVisibleInMenu(false);
+
+	// For some unknown reason on OSX the treeview gets a focus rect. So disable it
+	ui->workspaceTreeView->setAttribute(Qt::WA_MacShowFocusRect, false);
+
+	// Tighen-up the sizing of the main widgets to look slightly more consistent with the OSX style
+	ui->centralWidget->layout()->setContentsMargins(0, 0, 0, 0);
+	ui->workspaceTreeView->setFrameShape(QFrame::NoFrame);
+	ui->fileTableView->setFrameShape(QFrame::NoFrame);
+	ui->splitterVertical->setHandleWidth(1);
+	ui->splitterHorizontal->setHandleWidth(1);
+
+	// Wrong color scheme on Yosemite but better than the standard TabWidget
+	ui->tabWidget->setDocumentMode(true);
 #endif
 
 	abortShortcut = new QShortcut(QKeySequence("Escape"), this);
@@ -284,8 +297,9 @@ MainWindow::MainWindow(Settings &_settings, QWidget *parent, QString *workspaceP
 
 	// Create SearchBox
 	searchBox = new SearchBox(this);
-	searchBox->setPlaceholderText(tr("Filter (%0)").arg(searchShortcut->key().toString()));
-	searchBox->setMaximumWidth(450);
+	searchBox->setPlaceholderText(tr("Filter (%0)").arg(searchShortcut->key().toString(QKeySequence::NativeText)));
+	searchBox->setSizePolicy(QSizePolicy::Preferred, QSizePolicy::Fixed);
+	searchBox->setMinimumWidth(230);
 	ui->mainToolBar->addWidget(searchBox);
 
 	connect( searchBox,
@@ -293,6 +307,12 @@ MainWindow::MainWindow(Settings &_settings, QWidget *parent, QString *workspaceP
 		SLOT( onSearchBoxTextChanged(const QString&)),
 		Qt::DirectConnection );
 
+	// Add another spacer to the right
+	spacer = new QWidget();
+	spacer->setSizePolicy(QSizePolicy::Fixed, QSizePolicy::Expanding);
+	spacer->setMinimumWidth(3);
+	ui->mainToolBar->addWidget(spacer);
+
 	viewMode = VIEWMODE_TREE;
 
 	uiCallback.init(this);

src/Utils.cpp

@@ -54,6 +54,18 @@ QString SelectExe(QWidget *parent, const QString &description)
 	if(!QFile::exists(path))
 		return QString();
 
+#ifdef Q_OS_MACX
+	// Guess actual executable from bundle dir
+	QFileInfo fi(path);
+	if(fi.isDir() && path.indexOf(".app")!=-1)
+	{
+		path += "/Contents/MacOS/" + fi.baseName();
+
+		if(!QFile::exists(path))
+			return QString();
+	}
+#endif
+
 	return path;
 }
 //-----------------------------------------------------------------------------
@@ -517,7 +529,8 @@ QString UrlToStringNoCredentials(const QUrl& url)
 void SplitCommandLine(const QString &commandLine, QString &command, QString &extraParams)
 {
 	// Process command string
-	command = commandLine;
+	command = commandLine.trimmed();
+
 	Q_ASSERT(!command.isEmpty());
 
 	// Split command from embedded params
@@ -525,20 +538,22 @@ void SplitCommandLine(const QString &commandLine, QString &command, QString &ext
 
 	// Command ends after first space
 	QChar cmd_char_end = ' ';
-	int start = 0;
+	int cmd_start = 0;
+	int backtrack = 0;
 
 	// ...unless it is a quoted command
 	if(command[0]=='"')
 	{
 		cmd_char_end = '"';
-		start = 1;
+		cmd_start = 1;
+		backtrack = 1;
 	}
 
-	int cmd_end = command.indexOf(cmd_char_end, start);
+	int cmd_end = command.indexOf(cmd_char_end, cmd_start);
 	if(cmd_end > 0)
 	{
 		extraParams = command.mid(cmd_end+1);
-		command = command.mid(start, cmd_end-1);
+		command = command.mid(cmd_start, cmd_end-backtrack);
 	}
 
 	command = command.trimmed();

src/Workspace.cpp

@@ -99,6 +99,7 @@ bool Workspace::switchWorkspace(const QString& workspace, QSettings &store)
 	store.endArray();
 	store.endGroup();
 
+#if 0 // FIXME: Disabled this because if fossil's remote does not match exactly what we have stored (url and username), it will be automatically added every-time
 	// Add the default url from fossil
 	QUrl default_remote;
 	if(fossil().getRemoteUrl(default_remote) && default_remote.isValid() && !default_remote.isEmpty())
@@ -113,6 +114,7 @@ bool Workspace::switchWorkspace(const QString& workspace, QSettings &store)
 				setRemoteDefault(default_remote);
 		}
 	}
+#endif
 
 	return true;
 }

ui/MainWindow.ui

@@ -38,7 +38,7 @@
      <number>4</number>
     </property>
     <item>
-     <widget class="QSplitter" name="splitter_2">
+     <widget class="QSplitter" name="splitterVertical">
       <property name="sizePolicy">
        <sizepolicy hsizetype="Preferred" vsizetype="Expanding">
         <horstretch>0</horstretch>
@@ -48,7 +48,7 @@
       <property name="orientation">
        <enum>Qt::Vertical</enum>
       </property>
-      <widget class="QSplitter" name="splitter">
+      <widget class="QSplitter" name="splitterHorizontal">
        <property name="sizePolicy">
         <sizepolicy hsizetype="Expanding" vsizetype="Expanding">
          <horstretch>0</horstretch>