The branch, master, has been updated.

- Log -----------------------------------------------------------------

commit 04a8db4af8376a6938f5b75a50c70d92c5e5b394
Author: Jean-Marc Lasgouttes <lasgout...@lyx.org>
Date:   Tue Mar 19 14:21:57 2013 +0100

    Get rid of LYX_ERROR in our configure file.
    
    We use the standard AC_MSG_ERROR instead, which just stops the configure 
process where the problem happens.

diff --git a/config/lyxinclude.m4 b/config/lyxinclude.m4
index a21ce0a..c9b48d0 100644
--- a/config/lyxinclude.m4
+++ b/config/lyxinclude.m4
@@ -19,7 +19,7 @@ AC_ARG_ENABLE(build-type,
     prof*)  build_type=profiling;;
     gprof*) build_type=gprof;;
     rel*) build_type=release;;
-    *) AC_ERROR([Bad build type specification \"$enableval\". Please use one 
of rel(ease), pre(release), dev(elopment), prof(iling), or gprof]);;
+    *) AC_MSG_ERROR([bad build type specification \"$enableval\". Please use 
one of rel(ease), pre(release), dev(elopment), prof(iling), or gprof]);;
    esac],
   [case AC_PACKAGE_VERSION in
     *svn*|*dev*) build_type=development;;
@@ -34,7 +34,7 @@ case $build_type in
                 LYX_DATE="not released yet" ;;
     prerelease) lyx_prerelease=yes ;;
 esac
-    
+
 AC_SUBST(lyx_devel_version)
 ])
 
@@ -61,15 +61,6 @@ AC_MSG_RESULT([$withval])
 ])
 
 
-dnl Usage: LYX_ERROR(message)  Displays the warning "message" and sets the
-dnl flag lyx_error to yes.
-AC_DEFUN([LYX_ERROR],[
-lyx_error_txt="$lyx_error_txt
-** $1
-"
-lyx_error=yes])
-
-
 dnl Usage: LYX_WARNING(message)  Displays the warning "message" and sets the
 dnl flag lyx_warning to yes.
 AC_DEFUN([LYX_WARNING],[
@@ -82,36 +73,20 @@ lyx_warning=yes])
 dnl Usage: LYX_LIB_ERROR(file,library)  Displays an error message indication
 dnl  that 'file' cannot be found because 'lib' may be uncorrectly installed.
 AC_DEFUN([LYX_LIB_ERROR],[
-LYX_ERROR([Cannot find $1. Please check that the $2 library
+AC_MSG_ERROR([cannot find $1. Please check that the $2 library
    is correctly installed on your system.])])
 
 
-dnl Usage: LYX_CHECK_ERRORS  Displays a warning message if a LYX_ERROR
+dnl Usage: LYX_CHECK_WARNINGS  Displays a warning message if a LYX_WARNING
 dnl   has occured previously.
-AC_DEFUN([LYX_CHECK_ERRORS],[
+AC_DEFUN([LYX_CHECK_WARNINGS],[
 if test x$lyx_warning = xyes; then
 cat <<EOF
 === The following minor problems have been detected by configure.
 === Please check the messages below before running 'make'.
 === (see the section 'Problems' in the INSTALL file)
 $lyx_warning_txt
-EOF
-fi
-if test x$lyx_error = xyes; then
-cat <<EOF
-**** The following problems have been detected by configure.
-**** Please check the messages below before running 'make'.
-**** (see the section 'Problems' in the INSTALL file)
-$lyx_error_txt
-$lyx_warning_txt
-EOF
-exit 1
-else
 
-cat <<EOF
-Configuration of LyX was successful.
-Type 'make' to compile the program,
-and then 'make install' to install it.
 EOF
 fi])
 
@@ -354,7 +329,7 @@ AC_DEFUN([LYX_USE_INCLUDED_BOOST],[
                else
                        BOOST_MT=""
                        if test x$lyx_boost_plain != xyes ; then
-                               LYX_ERROR([No suitable boost library found (do 
not use --without-included-boost)])
+                               AC_MSG_ERROR([cannot find suitable boost 
library (do not use --without-included-boost)])
                        fi
                fi
                AC_SUBST(BOOST_MT)
@@ -384,7 +359,7 @@ AC_DEFUN([LYX_USE_INCLUDED_MYTHES],[
                fi
                AC_LANG_POP(C++)
                if test x$lyx_mythes != xyes -o x$ac_cv_header_mythes_h != 
xyes; then
-                       LYX_ERROR([No suitable MyThes library found (do not use 
--without-included-mythes)])
+                       AC_MSG_ERROR([cannot find suitable MyThes library (do 
not use --without-included-mythes)])
                fi
                AC_DEFINE(USE_EXTERNAL_MYTHES, 1, [Define as 1 to use an 
external MyThes library])
                
AC_DEFINE_UNQUOTED(MYTHES_H_LOCATION,$lyx_cv_mythes_h_location,[Location of 
mythes.hxx])
@@ -485,7 +460,7 @@ AC_ARG_WITH(frontend,
   [FRONTENDS="$withval"],[FRONTENDS="qt4"])
 if test "x$FRONTENDS" = x ; then
   AC_MSG_RESULT(none)
-  AC_ERROR("Please select a frontend using --with-frontend")
+  AC_MSG_ERROR("Please select a frontend using --with-frontend")
 fi
 AC_MSG_RESULT($FRONTENDS)
 AC_SUBST(FRONTENDS)
@@ -541,7 +516,7 @@ case $lyx_use_packaging in
                *apple-darwin*) lyx_install_macosx=true ;;
           esac
           lyx_install_posix=true ;;
-    *) LYX_ERROR([Unknown packaging type $lyx_use_packaging]) ;;
+    *) AC_MSG_ERROR([unknown packaging type $lyx_use_packaging.]) ;;
 esac
 AM_CONDITIONAL(INSTALL_MACOSX, $lyx_install_macosx)
 AM_CONDITIONAL(INSTALL_WINDOWS, $lyx_install_windows)
diff --git a/config/qt4.m4 b/config/qt4.m4
index 2fd66b4..d91d9ef 100644
--- a/config/qt4.m4
+++ b/config/qt4.m4
@@ -52,9 +52,7 @@ AC_DEFUN([QT4_CHECK_COMPILE],
 
        if test -z "$qt4_cv_libname"; then
                AC_MSG_RESULT([failed])
-               if test "$FATAL" = 1 ; then
-                       AC_MSG_ERROR([Cannot compile a simple Qt 4 executable. 
Check you have the right \$QT4DIR !])
-               fi
+               AC_MSG_ERROR([cannot compile a simple Qt 4 executable. Check 
you have the right \$QT4DIR.])
        else
                AC_MSG_RESULT([$qt4_cv_libname])
        fi
@@ -92,10 +90,6 @@ AC_DEFUN([QT4_DO_IT_ALL],
        dnl this variable is precious
        AC_ARG_VAR(QT4DIR, [the place where the Qt 4 files are, e.g. 
/usr/lib/qt4])
 
-       dnl Please leave this alone. I use this file in
-       dnl oprofile.
-       FATAL=0
-
        AC_ARG_WITH(qt4-dir, [AC_HELP_STRING([--with-qt4-dir], [where the root 
of Qt 4 is installed])],
                [ qt4_cv_dir=`eval echo "$withval"/` ])
 
@@ -143,7 +137,7 @@ AC_DEFUN([QT4_DO_IT_ALL],
 
        dnl Check qt version
        AS_VERSION_COMPARE($QT4_VERSION, $1, 
-       [LYX_ERROR([LyX requires version $1 of Qt. Only version $QT4_VERSION 
has been found.])
+       [AC_MSG_ERROR([LyX requires at least version $1 of Qt. Only version 
$QT4_VERSION has been found.])
        ])
 
        AC_PATH_PROGS(MOC4, [moc-qt4 moc],[],$qt4_cv_bin:$PATH)
@@ -193,7 +187,7 @@ AC_DEFUN([QT4_DO_MANUAL_CONFIG],
        case $have_x in
            yes) LIBS="$X_PRE_LIBS $LIBS $X_LIBS -lX11 $X_EXTRA_LIBS"
                 CPPFLAGS="$CPPFLAGS $X_CFLAGS";;
-            no) LYX_ERROR([Cannot find X window libraries and/or headers.]);;
+            no) AC_MSG_ERROR([cannot find X window libraries and/or 
headers.]);;
        disable) ;;
        esac
 
diff --git a/configure.ac b/configure.ac
index 0c4d833..81dcdcd 100644
--- a/configure.ac
+++ b/configure.ac
@@ -105,7 +105,7 @@ AC_CHECK_SIZEOF(wchar_t)
 ### We need iconv for unicode support (Qt4 frontend requires it too)
 AM_ICONV
 if test "$am_cv_func_iconv" = no; then
-  LYX_ERROR([Cannot find required library iconv])
+  AC_MSG_ERROR([cannot find required library iconv.])
 else
   LIBS="$LIBS $LIBICONV"
 fi
@@ -118,10 +118,10 @@ AC_CHECK_HEADERS(zlib.h,
 ### check for file magic support (currently optional)
 AC_CHECK_HEADERS(magic.h,
  [AC_CHECK_LIB(magic, magic_open, [LIBS="$LIBS -lmagic"],
-  LYX_WARNING([Cannot find limagic. Please check that the libmagic library
+  LYX_WARNING([cannot find libmagic. Please check that the libmagic library
    is correctly installed on your system.
    Falling back to builtin file format detection.]))],
- [LYX_WARNING([Cannot find magic.h. Please check that the libmagic library
+ [LYX_WARNING([cannot find magic.h. Please check that the libmagic library
    is correctly installed on your system.
    Falling back to builtin file format detection.])])
 
@@ -142,20 +142,20 @@ for frontend in $FRONTENDS ; do
       Qt 4 version:\t\t${QT4_VERSION}\n"
 dnl qt 4 build will fail without moc, uic, or bc
          if test -z "$MOC4"; then
-           LYX_ERROR([moc 4 binary not found !])
+           AC_MSG_ERROR([cannot find moc binary.])
          fi
          if test -z "$UIC4"; then
-           LYX_ERROR([uic 4 binary not found !])
+           AC_MSG_ERROR([cannot find uic binary.!])
          fi
          if test -z "$QT4_LIB"; then
-           LYX_ERROR([qt 4 library not found !])
+           AC_MSG_ERROR([cannot find qt4 library.])
          fi
          if test "$bc_found" = "no"; then
-           LYX_ERROR([bc binary not found !])
+           AC_MSG_ERROR([cannot find bc program.])
          fi
       ;;
     *)
-         LYX_ERROR(Unknown frontend '$frontend');;
+         AC_MSG_ERROR([unknown frontend '$frontend'.]);;
   esac
 done
 
@@ -163,7 +163,7 @@ done
 test "x$prefix" = xNONE && prefix=$default_prefix
 test "x$exec_prefix" = xNONE && exec_prefix='${prefix}'
 if echo $prefix |grep ' ' >/dev/null 2>/dev/null ; then
-  LYX_WARNING([The installation prefix \"${prefix}\" contains a space, which
+  LYX_WARNING([the installation prefix \"${prefix}\" contains a space, which
    causes problems with the Makefiles. The installation will be done in
    directory \"`pwd`/installprefix\" instead. Please move its contents to
    the right place after installation.])
@@ -395,5 +395,11 @@ echo
 printf "$VERSION_INFO"
 echo
 
-# Display a final warning if there has been a LYX_ERROR
-LYX_CHECK_ERRORS
+# Display a final warning if there has been a LYX_WARNING
+LYX_CHECK_WARNINGS
+
+cat <<EOF
+Configuration of LyX was successful.
+Type 'make' to compile the program,
+and then 'make install' to install it.
+EOF

-----------------------------------------------------------------------

Summary of changes:
 config/lyxinclude.m4 |   43 +++++++++----------------------------------
 config/qt4.m4        |   12 +++---------
 configure.ac         |   28 +++++++++++++++++-----------
 3 files changed, 29 insertions(+), 54 deletions(-)


hooks/post-receive
-- 
The LyX Source Repository

Reply via email to