Merge lp:~hingo/drizzle/fix-make-html into lp:~drizzle-trunk/drizzle/development

[Henrik Ingo]

Somewhere along the way we have broken the docs:
 * administration/plugin.rst contains 2 broken links and make html fails.
 * tar ball is missing half of the docs so make html fails spectacularly.

This fixes both. (I'd like to also add make html or make doctest to the jenkins tests, or directly into make test, but will ask for advice on that on IRC.)

[Stewart Smith (stewart)]

On Fri, 10 Feb 2012 21:09:24 -0000, Henrik Ingo <email address hidden> wrote:
> This fixes both. (I'd like to also add make html or make doctest to
> the jenkins tests, or directly into make test, but will ask for advice
> on that on IRC.)

I have a blog post on picking up warnings from sphinx and reporting them
through jenkins.

Unfortunately the external URL check that sphinx can automatically do
doesn't output in a standard format, so it's really hard to parse :(

--
Stewart Smith

[Henrik Ingo (hingo)]

Stewart: We have also sphinx set to fail on warnings, so catching warnings is not a problem. The problem is that make doctest and make html are never run - apparently even I've managed to slip in 2 lines that generate errors.

I think adding html and doctest to the test target is the way to go.

[Stewart Smith (stewart)]

On Sat, 11 Feb 2012 11:39:28 -0000, Henrik Ingo <email address hidden> wrote:
> Stewart: We have also sphinx set to fail on warnings, so catching
> warnings is not a problem. The problem is that make doctest and make
> html are never run - apparently even I've managed to slip in 2 lines
> that generate errors.

sounds good to me.

--
Stewart Smith

[lp:~hingo/drizzle/fix-make-html]

2506. By Henrik Ingo on 2012-02-12

Include generation of docs into make test, so it is actually routinely
tested. Hopefully we'll catch broken docs more often with this.

=== modified file 'docs/administration/plugins.rst'
--- docs/administration/plugins.rst	2011-11-14 22:29:19 +0000
+++ docs/administration/plugins.rst	2012-02-12 21:22:20 +0000
@@ -22,7 +22,7 @@
  * :ref:`collation_dictionary_plugin` - Data Dictionary for schema, table, column, indexes, etc (collation_dictionary)
  * :ref:`compression_plugin` - UDFs for compression functions (compression)
  * :ref:`connection_id_plugin` - Return the current connection_id (connection_id)
- * :ref:`console_plugin` - Console Client (console)
+ * console_plugin - Console Client (console) (TODO: documentation missing)
  * :ref:`crc32_plugin` - CRC32 Function (crc32)
  * :ref:`default_replicator_plugin` - Default Replicator (default_replicator)
  * :ref:`drizzle_protocol_plugin` - Drizzle Protocol (drizzle_protocol)
@@ -32,7 +32,7 @@
  * :ref:`hex_functions_plugin` - Convert a string to HEX() or from UNHEX() (hex_functions)
  * :ref:`information_schema_dictionary_plugin` - Data Dictionary for ANSI information schema, etc (information_schema_dictionary)
  * :ref:`innobase_plugin` - Supports transactions, row-level locking, and foreign keys (innobase)
- * :ref:`ipv6_function_plugin` - IPV6() function (ipv6_function)
+ * ipv6_function_plugin - IPV6() function (ipv6_function) (TODO: documentation missing)
  * :ref:`js_plugin` - Execute JavaScript code with supplied arguments (js)
  * :ref:`length_plugin` - Return the byte length of a string (length)
  * :ref:`logging_stats_plugin` - User Statistics as DATA_DICTIONARY tables (logging_stats)
@@ -40,7 +40,7 @@
  * :ref:`md5_plugin` - UDF for computing md5sum (md5)
  * :ref:`memory_plugin` - Hash based, stored in memory, useful for temporary tables (memory)
  * :ref:`multi_thread_plugin` - One Thread Per Session Scheduler (multi_thread)
- * :ref:`myisam_plugin` - Default engine as of MySQL 3.23 with great performance (myisam)
+ * :ref:`myisam_plugin` - Default engine as of MySQL 3.23, used for temporary tables (myisam)
  * :ref:`mysql_protocol_plugin` - MySQL Protocol Module (mysql_protocol)
  * :ref:`mysql_unix_socket_protocol_plugin` - MySQL Unix Socket Protocol (mysql_unix_socket_protocol)
  * :ref:`protocol_dictionary_plugin` - Provides dictionary for protocol counters. (protocol_dictionary)
 
=== modified file 'docs/include.am'
--- docs/include.am	2011-06-07 21:19:19 +0000
+++ docs/include.am	2012-02-12 21:22:20 +0000
@@ -3,24 +3,21 @@
 # All paths should be given relative to the root
 
 EXTRA_DIST+= \
-	     docs/_static \
-	     docs/_templates \
-	     docs/conf.py \
-	     docs/Doxyfile \
-	     docs/drizzled_header.html \
-	     docs/logo.png \
-	     docs/plugins/list.rst \
-	     docs/pyext/__init__.py \
-	     docs/pyext/options.py \
-	     docs/pyext/confval.py \
-	     docs/pyext/dbtable.py \
-	     docs/_static/drizzle.ico \
+	     ${top_srcdir}/docs/_static \
+	     ${top_srcdir}/docs/_templates \
+	     ${top_srcdir}/docs/conf.py \
+	     ${top_srcdir}/docs/Doxyfile \
+	     ${top_srcdir}/docs/drizzled_header.html \
+	     ${top_srcdir}/docs/*.png \
+	     ${top_srcdir}/docs/pyext/__init__.py \
+	     ${top_srcdir}/docs/pyext/options.py \
+	     ${top_srcdir}/docs/pyext/confval.py \
+	     ${top_srcdir}/docs/pyext/dbtable.py \
+	     ${top_srcdir}/docs/python-inv.txt \
+	     ${top_srcdir}/docs/_static/drizzle.ico \
 	     ${top_srcdir}/docs/*.rst \
-	     ${top_srcdir}/docs/clients/*.rst \
-	     ${top_srcdir}/docs/contributing/*.rst \
-	     ${top_srcdir}/docs/libdrizzle/*.rst \
-	     ${top_srcdir}/docs/installing/*.rst \
-             ${top_srcdir}/docs/testing/*.rst
+	     ${top_srcdir}/docs/*/*.rst \
+	     ${top_srcdir}/docs/*/*/*.rst
 
 # Makefile for Sphinx documentation
 #
 
=== modified file 'tests/include.am'
--- tests/include.am	2011-10-05 20:40:33 +0000
+++ tests/include.am	2012-02-12 21:22:20 +0000
@@ -142,7 +142,7 @@
 # will then calculate the various port numbers it needs from this,
 # making sure each user use different ports.
 
-test: tests/var/drizzle check
+test: html tests/var/drizzle check
 
 # Please keep the list of tests in alphabetical order for ease of
 # maintenance and verification