Kaydet (Commit) 75892872 authored tarafından jan iversen's avatar jan iversen Kaydeden (comit) Michael Meeks

check for missing COMSPEC variable in cygwin.

Cygwin unsets the COMSPEC environment variable on some installations.
This is due to an error reported first time in 2005, but not solved.
There are no exact pattern when this happens, in this case it happened
on 2 windows 8.1 VM installations (one private one azure).

added check in configure.ac to alert the user earlier

COMSPEC is used in sal/osl to start processes and as such vital, and in 
some perl scripts, therefore hardcoding to e.g. cmd32.exe (or the power
shell) is not an option.

Limited check to work only for cygwin.

Change-Id: I52bb6667c52560ed1a239ac301811605b8cddcbf
Reviewed-on: https://gerrit.libreoffice.org/20255Tested-by: 's avatarJenkins <ci@libreoffice.org>
Reviewed-by: 's avatarMichael Meeks <michael.meeks@collabora.com>
Tested-by: 's avatarMichael Meeks <michael.meeks@collabora.com>
üst 628d84d9
...@@ -282,6 +282,10 @@ if test "$build_os" = "cygwin"; then ...@@ -282,6 +282,10 @@ if test "$build_os" = "cygwin"; then
PathFormat "$BUILDDIR" PathFormat "$BUILDDIR"
BUILDDIR="$formatted_path" BUILDDIR="$formatted_path"
x_Cygwin= x_Cygwin=
AC_MSG_CHECKING(for explicit COMSPEC)
if test -z "$COMSPEC"; then
AC_MSG_ERROR([COMSPEC not set in Environment, please set it an rerun])
fi
fi fi
AC_SUBST(SRC_ROOT) AC_SUBST(SRC_ROOT)
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment