The old behaviour was to expect the user to go modify manually a source
file which is not a great idea because that's typically the kind of things
in charge of the configure script.

Signed-off-by: Christophe CURIS <christophe.cu...@free.fr>
---
 README                     |  3 ++-
 configure.ac               | 12 ++++++++++++
 doc/build/Compilation.texi |  4 ++++
 src/wconfig.h.in           |  3 ---
 4 files changed, 18 insertions(+), 4 deletions(-)

diff --git a/README b/README
index e85083b..7195e5f 100644
--- a/README
+++ b/README
@@ -201,7 +201,8 @@ while keeping a nice appearance and good functionality, 
follow the items bellow:
 - edit wconfig.h and disable the NUMLOCK_HACK and the features you don't use
   anyway (keep in mind that some of the #defines might not work, as they are 
not
   fully supported). Make sure to always keep NumLock and ScrollLock turned off.
-- turn on DisableAnimations. You can also #undefine ANIMATIONS in wconfig.h
+- turn on DisableAnimations. You can also specify --disable-animation at 
compile
+  time to the configure script.
 - strip down the default IconPath and PixmapPath entries to contain only 
   the paths that you really have in your system. 
 - do not use large images in the root background
diff --git a/configure.ac b/configure.ac
index 6a02376..491bdc7 100644
--- a/configure.ac
+++ b/configure.ac
@@ -289,6 +289,18 @@ AC_ARG_WITH(incs-from, AS_HELP_STRING([--with-incs-from], 
[pass compiler flags t
 
 dnl Features Configuration
 dnl ======================
+AC_ARG_ENABLE([animations],
+    [AS_HELP_STRING([--disable-animations], [disable permanently animations 
@<:@default=enabled@:>@])],
+    [AS_CASE(["$enableval"],
+        [yes|no], [],
+        [AC_MSG_ERROR([bad value $enableval for --enable-animations])])],
+    [enable_animations="yes"])
+AS_IF([test "x$enable_animations" = "xno"],
+    [unsupported="$unsupported Animations"],
+    [AC_DEFINE([USE_ANIMATIONS], [1], [Defined when user did not request to 
disable animations])
+     supported_core="$supported_core Animations"])
+
+
 AC_ARG_ENABLE([mwm-hints],
     [AS_HELP_STRING([--disable-mwm-hints], [disable support for Motif WM hints 
@<:@default=enabled@:>@])],
     [AS_CASE(["$enableval"],
diff --git a/doc/build/Compilation.texi b/doc/build/Compilation.texi
index 4db3ae0..095718c 100644
--- a/doc/build/Compilation.texi
+++ b/doc/build/Compilation.texi
@@ -566,6 +566,10 @@ screen size.
 @subsection Feature Selection
 
 @table @option
+@item --disable-animations
+Disable animations permanently, by not compiling the corresponding code into 
@sc{Window Maker}.
+When enabled (the default), you still have a run-time configuration option in 
@emph{WPrefs}.
+
 @item --disable-mwm-hints
 Disable support for Motif's MWM Window Manager hints.
 These attributes were introduced by the Motif toolkit to ask for special 
window appearance requests.
diff --git a/src/wconfig.h.in b/src/wconfig.h.in
index be0295c..bac5072 100644
--- a/src/wconfig.h.in
+++ b/src/wconfig.h.in
@@ -33,9 +33,6 @@
  * Also check the features you can enable through configure.
  */
 
-/* If you want animations for iconification, shading, icon arrangement etc. */
-#define USE_ANIMATIONS
-
 /*
  * Undefine BALLOON_TEXT if you don't want balloons for showing extra
  * information, like window titles that are not fully visible.
-- 
2.1.4


-- 
To unsubscribe, send mail to wmaker-dev-unsubscr...@lists.windowmaker.org.

Reply via email to