Always include documention in the build

By default run AsciiDoc and package the documentation as part of
the WAR. This ensures the default `mvn package` will include user
documentation, as folks have come to expect it to exist in builds.

If AsciiDoc is not available documentation can be skipped by
passing -Dgerrit.documentation.skip=true on the command line.
This naming convention matches with -Dmaven.test.skip=true as
a way to avoid running unit tests.

Change-Id: Ic433c5bb291c9abb48117d61c2eabc031a5c5ca0
diff --git a/gerrit-war/pom.xml b/gerrit-war/pom.xml
index dc1a3f7..71ed71e 100644
--- a/gerrit-war/pom.xml
+++ b/gerrit-war/pom.xml
@@ -242,7 +242,7 @@
             <id>include-documentation</id>
             <phase>process-classes</phase>
             <configuration>
-              <target if="gerrit.include-documentation">
+              <target unless="gerrit.documentation.skip">
                 <property name="src" location="${basedir}/../Documentation" />
                 <property name="out" location="${project.build.directory}/${project.build.finalName}" />
                 <property name="dst" location="${out}/Documentation" />
@@ -269,7 +269,7 @@
             <id>include-release-notes</id>
             <phase>process-classes</phase>
             <configuration>
-              <target if="gerrit.include-documentation">
+              <target unless="gerrit.documentation.skip">
                 <property name="src" location="${basedir}/../ReleaseNotes" />
                 <property name="out" location="${project.build.directory}/${project.build.finalName}" />
                 <property name="dst" location="${out}/ReleaseNotes" />