Merge remote-tracking branch 'upstream/master'

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_GUI "Build the GUI" ON)
-option (BARRIER_BUILD_LEGACY_SERVICE "Build the legacy service (barrierd)" ON)
+option (BARRIER_BUILD_INSTALLER "Build the installer" ON)
-option (BARRIER_BUILD_LEGACY_INSTALLER "Build the legacy 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)

dist/wix/Product.wxs

@@ -31,12 +31,6 @@
 		<Property Id="ARPPRODUCTICON" Value="barrier.ico"/>
 		<Property Id="WIXUI_INSTALLDIR" Value="INSTALLFOLDER"/>
 		<Property Id="MSIRESTARTMANAGERCONTROL" Value="Disable"/>
-		<Property Id="LEGACY_UNINSTALL_EXISTS">
-			<RegistrySearch Id="LegacyRegistrySearch" Key="SOFTWARE\Wow6432Node\Microsoft\Windows\CurrentVersion\Uninstall\Barrier" Name="UninstallString" Root="HKLM" Type="file" Win64="no">
-				<FileSearch Id="LegacyFileSearch" Name="uninstall.exe"/>
-			</RegistrySearch>
-		</Property>
-		<Condition Message="An existing installation of $(var.Name) was detected, please uninstall it before continuing.">NOT LEGACY_UNINSTALL_EXISTS
     </Condition>
 		<CustomAction ExeCommand="" FileKey="GuiProgram" Id="StartGui" Return="asyncNoWait"/>
 		<UI>

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)
+if (BARRIER_BUILD_GUI)
-endif (BARRIER_BUILD_LEGACY_GUI)
+    add_subdirectory(gui)
+endif()

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()
 

src/cmd/barrierd/barrierd.cpp

@@ -16,7 +16,7 @@
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-#include "barrier/DaemonApp.h"
+#include "barrier/win32/DaemonApp.h"
 
 #include <iostream>
 

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 GUI_SOURCE_FILES src/*.cpp src/*.h)
-file (GLOB LEGACY_GUI_UI_FILES src/*.ui)
+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}
+    ${GUI_SOURCE_FILES}
-    ${LEGACY_GUI_UI_FILES}
+    ${GUI_UI_FILES}
-    ${LEGACY_GUI_RC_FILES}
+    ${GUI_RC_FILES}
     res/Barrier.qrc
 )
 

src/gui/src/LogWindowBase.ui

@@ -6,7 +6,7 @@
    <rect>
     <x>0</x>
     <y>0</y>
-    <width>600</width>
+    <width>800</width>
     <height>371</height>
    </rect>
   </property>

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))
     {

src/lib/arch/win32/ArchDaemonWindows.cpp

@@ -697,15 +697,7 @@ ArchDaemonWindows::installDaemon()
 void
 ArchDaemonWindows::uninstallDaemon()
 {
-    // remove legacy services if installed.
+    // remove service 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.
     if (isDaemonInstalled(DEFAULT_DAEMON_NAME)) {
         uninstallDaemon(DEFAULT_DAEMON_NAME);
     }

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"),

src/lib/barrier/win32/DaemonApp.cpp

@@ -16,10 +16,7 @@
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-// TODO: split this class into windows and unix to get rid
+#include "barrier/win32/DaemonApp.h"
-// of all the #ifdefs!
-
-#include "barrier/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 <Windows.h>
 
-#endif
-
 #include <string>
 #include <iostream>
 #include <sstream>
@@ -68,25 +61,15 @@ mainLoopStatic()
 }
 
 int
-unixMainLoopStatic(int, const char**)
+mainLoopStatic(int, const char**)
-{
-    return mainLoopStatic();
-}
-
-#if SYSAPI_WIN32
-int
-winMainLoopStatic(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", mainLoopStatic);
-            arch.daemonize("Barrier", winMainLoopStatic);
-#elif SYSAPI_UNIX
-            arch.daemonize("Barrier", unixMainLoopStatic);
-#endif
         }
 
         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;

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;