[Libreoffice-commits] core.git: configure.ac

Michael Stahl mstahl at redhat.com
Thu Mar 23 14:48:28 UTC 2017


 configure.ac |    2 --
 1 file changed, 2 deletions(-)

New commits:
commit 5113441728bdf0a6e2c11922478d5515ef886b8e
Author: Michael Stahl <mstahl at redhat.com>
Date:   Thu Mar 23 13:33:40 2017 +0100

    configure: remove hack to add quotes to $DEVENV again
    
    This was added because CPPUNITTRACE='"$(DEVENV)" /debugexe' is less
    convenient to write than CPPUNITTRACE='$(DEVENV) /debugexe'.
    
    However DavidO points out that CPPUNITTRACE=TRUE is even more
    convenient and sort of optimal already, and it turns out it's broken by
    the extra quotes, so remove the hack.
    
    Change-Id: Ic5d94f9a0774cd41821b68728404c205552d674e
    Reviewed-on: https://gerrit.libreoffice.org/35589
    Reviewed-by: David Ostrovsky <david at ostrovsky.org>
    Tested-by: Jenkins <ci at libreoffice.org>

diff --git a/configure.ac b/configure.ac
index fd44c66ba942..60ea0ead62b4 100644
--- a/configure.ac
+++ b/configure.ac
@@ -3456,8 +3456,6 @@ if test "$_os" = "WINNT"; then
     if test ! -e "$DEVENV"; then
         AC_MSG_ERROR([No devenv.exe found, Visual Studio installation broken?])
     fi
-    # add quotes around it, this is an ugly hack for convenience
-    DEVENV=\"${DEVENV}\"
 
     dnl ===========================================================
     dnl  Check for the corresponding mspdb*.dll


More information about the Libreoffice-commits mailing list