diff --git a/CMakeLists.txt b/CMakeLists.txt
index 27dccc96..c81584d2 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -18,9 +18,8 @@
cmake_minimum_required (VERSION 3.4)
project (barrier C CXX)
-option (BARRIER_BUILD_LEGACY_GUI "Build the legacy GUI" ON)
-option (BARRIER_BUILD_LEGACY_SERVICE "Build the legacy service (barrierd)" ON)
-option (BARRIER_BUILD_LEGACY_INSTALLER "Build the legacy installer" ON)
+option (BARRIER_BUILD_GUI "Build the GUI" ON)
+option (BARRIER_BUILD_INSTALLER "Build the installer" ON)
set (CMAKE_CXX_STANDARD 14)
set (CMAKE_CXX_EXTENSIONS OFF)
@@ -355,7 +354,7 @@ macro (configure_files srcDir destDir)
endforeach (templateFile)
endmacro (configure_files)
-if (${BARRIER_BUILD_LEGACY_INSTALLER})
+if (${BARRIER_BUILD_INSTALLER})
#
# macOS app Bundle
#
@@ -393,6 +392,6 @@ if (${CMAKE_SYSTEM_NAME} MATCHES "Linux")
endif()
else()
- message (STATUS "NOT configuring the v1 installer")
+ message (STATUS "NOT configuring the installer")
endif()
add_subdirectory (src)
diff --git a/dist/wix/Product.wxs b/dist/wix/Product.wxs
index e2af4b08..5cb924ec 100644
--- a/dist/wix/Product.wxs
+++ b/dist/wix/Product.wxs
@@ -31,12 +31,6 @@
-
-
-
-
-
- NOT LEGACY_UNINSTALL_EXISTS
diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt
index 911fef3c..3a8e83eb 100644
--- a/src/CMakeLists.txt
+++ b/src/CMakeLists.txt
@@ -19,6 +19,7 @@ include_directories (${CMAKE_CURRENT_BINARY_DIR}/lib)
add_subdirectory(lib)
add_subdirectory(cmd)
-if (BARRIER_BUILD_LEGACY_GUI)
-add_subdirectory(gui)
-endif (BARRIER_BUILD_LEGACY_GUI)
+
+if (BARRIER_BUILD_GUI)
+ add_subdirectory(gui)
+endif()
diff --git a/src/cmd/CMakeLists.txt b/src/cmd/CMakeLists.txt
index 0c6d1a92..0fdfe339 100644
--- a/src/cmd/CMakeLists.txt
+++ b/src/cmd/CMakeLists.txt
@@ -18,7 +18,7 @@ add_subdirectory(barrierc)
add_subdirectory(barriers)
add_subdirectory(syntool)
-if (BARRIER_BUILD_LEGACY_SERVICE)
+if (WIN32)
add_subdirectory(barrierd)
-endif (BARRIER_BUILD_LEGACY_SERVICE)
+endif()
diff --git a/src/cmd/barrierd/barrierd.cpp b/src/cmd/barrierd/barrierd.cpp
index 30423c7a..dd351f9a 100644
--- a/src/cmd/barrierd/barrierd.cpp
+++ b/src/cmd/barrierd/barrierd.cpp
@@ -16,7 +16,7 @@
* along with this program. If not, see .
*/
-#include "barrier/DaemonApp.h"
+#include "barrier/win32/DaemonApp.h"
#include
diff --git a/src/gui/CMakeLists.txt b/src/gui/CMakeLists.txt
index a5dc9874..5d248912 100644
--- a/src/gui/CMakeLists.txt
+++ b/src/gui/CMakeLists.txt
@@ -6,17 +6,17 @@ set (CMAKE_AUTORCC ON)
set (CMAKE_AUTOUIC ON)
set (CMAKE_INCLUDE_CURRENT_DIR ON)
-file (GLOB LEGACY_GUI_SOURCE_FILES src/*.cpp src/*.h)
-file (GLOB LEGACY_GUI_UI_FILES src/*.ui)
+file (GLOB GUI_SOURCE_FILES src/*.cpp src/*.h)
+file (GLOB GUI_UI_FILES src/*.ui)
if (WIN32)
- set (LEGACY_GUI_RC_FILES res/win/Barrier.rc)
+ set (GUI_RC_FILES res/win/Barrier.rc)
endif()
add_executable (barrier WIN32
- ${LEGACY_GUI_SOURCE_FILES}
- ${LEGACY_GUI_UI_FILES}
- ${LEGACY_GUI_RC_FILES}
+ ${GUI_SOURCE_FILES}
+ ${GUI_UI_FILES}
+ ${GUI_RC_FILES}
res/Barrier.qrc
)
diff --git a/src/gui/src/LogWindowBase.ui b/src/gui/src/LogWindowBase.ui
index c7272a70..f3359474 100644
--- a/src/gui/src/LogWindowBase.ui
+++ b/src/gui/src/LogWindowBase.ui
@@ -6,7 +6,7 @@
0
0
- 600
+ 800
371
diff --git a/src/gui/src/MainWindow.cpp b/src/gui/src/MainWindow.cpp
index ff9725c0..b92e5f7c 100644
--- a/src/gui/src/MainWindow.cpp
+++ b/src/gui/src/MainWindow.cpp
@@ -980,7 +980,7 @@ bool MainWindow::on_m_pButtonBrowseConfigFile_clicked()
bool MainWindow::on_m_pActionSave_triggered()
{
- QString fileName = QFileDialog::getSaveFileName(this, tr("Save configuration as..."));
+ QString fileName = QFileDialog::getSaveFileName(this, tr("Save configuration as..."), QString(), tr("Barrier Configuration (*.sgc)"));
if (!fileName.isEmpty() && !serverConfig().save(fileName))
{
diff --git a/src/lib/arch/win32/ArchDaemonWindows.cpp b/src/lib/arch/win32/ArchDaemonWindows.cpp
index fdd50c43..efcf235d 100644
--- a/src/lib/arch/win32/ArchDaemonWindows.cpp
+++ b/src/lib/arch/win32/ArchDaemonWindows.cpp
@@ -697,15 +697,7 @@ ArchDaemonWindows::installDaemon()
void
ArchDaemonWindows::uninstallDaemon()
{
- // remove legacy services if installed.
- if (isDaemonInstalled(LEGACY_SERVER_DAEMON_NAME)) {
- uninstallDaemon(LEGACY_SERVER_DAEMON_NAME);
- }
- if (isDaemonInstalled(LEGACY_CLIENT_DAEMON_NAME)) {
- uninstallDaemon(LEGACY_CLIENT_DAEMON_NAME);
- }
-
- // remove new service if installed.
+ // remove service if installed.
if (isDaemonInstalled(DEFAULT_DAEMON_NAME)) {
uninstallDaemon(DEFAULT_DAEMON_NAME);
}
diff --git a/src/lib/arch/win32/ArchDaemonWindows.h b/src/lib/arch/win32/ArchDaemonWindows.h
index ad69779e..2db97923 100644
--- a/src/lib/arch/win32/ArchDaemonWindows.h
+++ b/src/lib/arch/win32/ArchDaemonWindows.h
@@ -142,9 +142,6 @@ private:
#define DEFAULT_DAEMON_NAME _T("Barrier")
#define DEFAULT_DAEMON_INFO _T("Manages the Barrier foreground processes.")
-#define LEGACY_SERVER_DAEMON_NAME _T("Barrier Server")
-#define LEGACY_CLIENT_DAEMON_NAME _T("Barrier Client")
-
static const TCHAR* const g_daemonKeyPath[] = {
_T("SOFTWARE"),
_T("The Barrier Project"),
diff --git a/src/lib/barrier/DaemonApp.cpp b/src/lib/barrier/win32/DaemonApp.cpp
similarity index 93%
rename from src/lib/barrier/DaemonApp.cpp
rename to src/lib/barrier/win32/DaemonApp.cpp
index 8a431f95..62fecf8f 100644
--- a/src/lib/barrier/DaemonApp.cpp
+++ b/src/lib/barrier/win32/DaemonApp.cpp
@@ -16,10 +16,7 @@
* along with this program. If not, see .
*/
-// TODO: split this class into windows and unix to get rid
-// of all the #ifdefs!
-
-#include "barrier/DaemonApp.h"
+#include "barrier/win32/DaemonApp.h"
#include "barrier/App.h"
#include "barrier/ArgParser.h"
@@ -37,8 +34,6 @@
#include "base/log_outputters.h"
#include "base/Log.h"
-#if SYSAPI_WIN32
-
#include "arch/win32/ArchMiscWindows.h"
#include "arch/win32/XArchWindows.h"
#include "barrier/Screen.h"
@@ -50,8 +45,6 @@
#define WIN32_LEAN_AND_MEAN
#include
-#endif
-
#include
#include
#include
@@ -68,25 +61,15 @@ mainLoopStatic()
}
int
-unixMainLoopStatic(int, const char**)
-{
- return mainLoopStatic();
-}
-
-#if SYSAPI_WIN32
-int
-winMainLoopStatic(int, const char**)
+mainLoopStatic(int, const char**)
{
return ArchMiscWindows::runDaemon(mainLoopStatic);
}
-#endif
DaemonApp::DaemonApp() :
m_ipcServer(nullptr),
m_ipcLogOutputter(nullptr),
- #if SYSAPI_WIN32
m_watchdog(nullptr),
- #endif
m_events(nullptr),
m_fileLogOutputter(nullptr)
{
@@ -100,10 +83,8 @@ DaemonApp::~DaemonApp()
int
DaemonApp::run(int argc, char** argv)
{
-#if SYSAPI_WIN32
// win32 instance needed for threading, etc.
ArchMiscWindows::setInstanceWin32(GetModuleHandle(NULL));
-#endif
Arch arch;
arch.init();
@@ -115,10 +96,8 @@ DaemonApp::run(int argc, char** argv)
bool uninstall = false;
try
{
-#if SYSAPI_WIN32
// sends debug messages to visual studio console window.
log.insert(new MSWindowsDebugOutputter());
-#endif
// default log level to system setting.
string logLevel = arch.setting("LogLevel");
@@ -133,7 +112,6 @@ DaemonApp::run(int argc, char** argv)
if (arg == "/f" || arg == "-f") {
foreground = true;
}
-#if SYSAPI_WIN32
else if (arg == "/install") {
uninstall = true;
arch.installDaemon();
@@ -143,7 +121,6 @@ DaemonApp::run(int argc, char** argv)
arch.uninstallDaemon();
return kExitSuccess;
}
-#endif
else {
stringstream ss;
ss << "Unrecognized argument: " << arg;
@@ -158,11 +135,7 @@ DaemonApp::run(int argc, char** argv)
mainLoop(false);
}
else {
-#if SYSAPI_WIN32
- arch.daemonize("Barrier", winMainLoopStatic);
-#elif SYSAPI_UNIX
- arch.daemonize("Barrier", unixMainLoopStatic);
-#endif
+ arch.daemonize("Barrier", mainLoopStatic);
}
return kExitSuccess;
@@ -214,10 +187,8 @@ DaemonApp::mainLoop(bool daemonized)
m_ipcLogOutputter = new IpcLogOutputter(*m_ipcServer, kIpcClientGui, true);
CLOG->insert(m_ipcLogOutputter);
-#if SYSAPI_WIN32
m_watchdog = new MSWindowsWatchdog(daemonized, false, *m_ipcServer, *m_ipcLogOutputter);
m_watchdog->setFileLogOutputter(m_fileLogOutputter);
-#endif
m_events->adoptHandler(
m_events->forIpcServer().messageReceived(), m_ipcServer,
@@ -225,8 +196,6 @@ DaemonApp::mainLoop(bool daemonized)
m_ipcServer->listen();
-#if SYSAPI_WIN32
-
// install the platform event queue to handle service stop events.
m_events->adoptBuffer(new MSWindowsEventQueueBuffer(m_events));
@@ -238,13 +207,11 @@ DaemonApp::mainLoop(bool daemonized)
}
m_watchdog->startAsync();
-#endif
+
m_events->loop();
-#if SYSAPI_WIN32
m_watchdog->stop();
delete m_watchdog;
-#endif
m_events->removeHandler(
m_events->forIpcServer().messageReceived(), m_ipcServer);
@@ -266,11 +233,7 @@ DaemonApp::mainLoop(bool daemonized)
void
DaemonApp::foregroundError(const char* message)
{
-#if SYSAPI_WIN32
MessageBox(NULL, message, "Barrier Service", MB_OK | MB_ICONERROR);
-#elif SYSAPI_UNIX
- cerr << message << endl;
-#endif
}
std::string
@@ -338,7 +301,6 @@ DaemonApp::handleIpcMessage(const Event& e, void*)
}
}
-#if SYSAPI_WIN32
// eg. no log-to-file while running in foreground
if (m_fileLogOutputter != nullptr) {
String logFilename;
@@ -354,7 +316,6 @@ DaemonApp::handleIpcMessage(const Event& e, void*)
}
m_fileLogOutputter->setLogFilename(logFilename.c_str());
}
-#endif
}
else {
LOG((CLOG_DEBUG "empty command, elevate=%d", cm->elevate()));
@@ -372,12 +333,11 @@ DaemonApp::handleIpcMessage(const Event& e, void*)
LOG((CLOG_ERR "failed to save settings, %s", e.what()));
}
-#if SYSAPI_WIN32
// tell the relauncher about the new command. this causes the
// relauncher to stop the existing command and start the new
// command.
m_watchdog->setCommand(command, cm->elevate());
-#endif
+
break;
}
@@ -392,10 +352,8 @@ DaemonApp::handleIpcMessage(const Event& e, void*)
LOG((CLOG_DEBUG "ipc hello, type=%s", type.c_str()));
-#if SYSAPI_WIN32
const char * serverstatus = m_watchdog->isProcessActive() ? "active" : "not active";
LOG((CLOG_INFO "server status: %s", serverstatus));
-#endif
m_ipcLogOutputter->notifyBuffer();
break;
diff --git a/src/lib/barrier/DaemonApp.h b/src/lib/barrier/win32/DaemonApp.h
similarity index 96%
rename from src/lib/barrier/DaemonApp.h
rename to src/lib/barrier/win32/DaemonApp.h
index e2e7bdf1..2a8484b3 100644
--- a/src/lib/barrier/DaemonApp.h
+++ b/src/lib/barrier/win32/DaemonApp.h
@@ -27,9 +27,7 @@ class Event;
class IpcLogOutputter;
class FileLogOutputter;
-#if SYSAPI_WIN32
class MSWindowsWatchdog;
-#endif
class DaemonApp {
@@ -48,9 +46,7 @@ private:
public:
static DaemonApp* s_instance;
-#if SYSAPI_WIN32
MSWindowsWatchdog* m_watchdog;
-#endif
private:
IpcServer* m_ipcServer;