[gentoo-commits] gentoo commit in xml/htdocs/proj/en/qa: automagic.xml

[Diego Petteno (flameeyes)]

flameeyes    11/06/01 23:23:01

  Modified:             automagic.xml
  Log:
  Update guide improving the autotools code in it.
  
  Not only the code was using if/fi conditionals (which can conflict
  with pkg-config macros depending on the project), but it was using the
  == comparison bashism.
  
  Fix both so that the code is more compliant with proper autotools.
  
  Also rewrite the note about using cache values to work around the
  issues.

Revision  Changes    Path
1.7                  xml/htdocs/proj/en/qa/automagic.xml

file : http://sources.gentoo.org/viewvc.cgi/gentoo/xml/htdocs/proj/en/qa/automagic.xml?rev=1.7&view=markup
plain: http://sources.gentoo.org/viewvc.cgi/gentoo/xml/htdocs/proj/en/qa/automagic.xml?rev=1.7&content-type=text/plain
diff : http://sources.gentoo.org/viewvc.cgi/gentoo/xml/htdocs/proj/en/qa/automagic.xml?r1=1.6&r2=1.7

Index: automagic.xml
===================================================================
RCS file: /var/cvsroot/gentoo/xml/htdocs/proj/en/qa/automagic.xml,v
retrieving revision 1.6
retrieving revision 1.7
diff -u -r1.6 -r1.7
--- automagic.xml	8 Nov 2008 08:41:18 -0000	1.6
+++ automagic.xml	1 Jun 2011 23:23:01 -0000	1.7
@@ -1,12 +1,12 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <!DOCTYPE guide SYSTEM "http://www.gentoo.org/dtd/guide.dtd">
-<!-- $Header: /var/cvsroot/gentoo/xml/htdocs/proj/en/qa/automagic.xml,v 1.6 2008/11/08 08:41:18 serkan Exp $ -->
+<!-- $Header: /var/cvsroot/gentoo/xml/htdocs/proj/en/qa/automagic.xml,v 1.7 2011/06/01 23:23:01 flameeyes Exp $ -->
 
 <guide link="/proj/en/qa/automagic.xml" lang="en">
 <title>Automagic dependencies, what they are and how to fix them</title>
 
 <author title="Author">
-  <mail link="flameeyes@g.o">Diego Pettenò</mail>
+  <mail link="flameeyes@g.o">Diego Elio Pettenò</mail>
 </author>
 <author title="Author">
   <mail link="serkan@g.o">Serkan Kaba</mail>
@@ -21,8 +21,8 @@
 <!-- See http://creativecommons.org/licenses/by-sa/2.5 -->
 <license/>
 
-<version>0.2</version>
-<date>2008-11-07</date>
+<version>0.3</version>
+<date>2011-06-01</date>
 
 <chapter> <!-- Introduction -->
 <title>Introduction</title>
@@ -169,17 +169,17 @@
 <pre caption="Adding an enabled-by-default check for an optional dependency">
 <i>AC_ARG_WITH([foo], AS_HELP_STRING([--without-foo], [Build without foo library (default: test)]))</i>
 
-<i>if test "x$with_foo" != "xno"; then</i>
+<i>AS_IF([test "x$with_foo" != "xno"], [</i>
   PKG_CHECK_MODULES([FOO], [foo >= 0.1])
-<i>fi</i>
+<i>])</i>
 </pre>
 
 <pre caption="Adding a disabled-by-default check for an optional dependency">
 <i>AC_ARG_WITH([foo], AS_HELP_STRING([--with-foo], [Build with foo library (default: disabled)]))</i>
 
-<i>if test "x$with_foo" == "xyes"; then</i>
+<i>AS_IF([test "x$with_foo" = "xyes"], [</i>
   PKG_CHECK_MODULES([FOO], [foo >= 0.1])
-<i>fi</i>
+<i>])</i>
 </pre>
 
 <p>
@@ -199,13 +199,11 @@
 </warn>
 
 <p>
-There's actually another way to workaround, without fixing (and patching), the
-automagic dependencies generated by <b>AC_CHECK_LIB</b>, and it's by playing
-around with the cache values used by <c>autoconf</c>. This method is actually
-deprecated because it does not fix the issue to the roots and might create
-problems if upstream changes the tests a bit using a different cache variable
-name. Also, in this way fixes can't be sent upstream for integration in later
-versions.
+While it is possible to work around the issue of automagic dependencies without
+patching <c>configure.ac</c>, by messing with the autoconf cache values, this
+method is not recommended. This does not fix the original issue, and cannot be
+sent upstream for integration in new versions, and can actually conflict when
+tests are slightly different between environments.
 </p>
 
 </body>