Michael Brown has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/8654


Change subject: [DOCS] include version when building
......................................................................

[DOCS] include version when building

This patch includes the canonical Impala version from
bin/save-version.sh in the doc builds' PDF and HTML output.

- Move impala_keydefs.ditamap to impala_keydefs.ditamap.in: this is now
  the checked in file.
- Include a marker in impala_keydefs.ditamap.in to replace the
  $IMPALA_VERSION when running "make".
- Use bin/save-version.sh to generate an updated version.info for the
  docs build. This is kept separate from bin/version.info since a full
  Impala build should not be required when just building the docs.
- Make the generated impala_keydefs.ditamap depend on both its source
  template and version.info; generate it using bash. Set SHELL so that
  we can use set -u and set -o pipefail (/bin/sh does not provide
  these).
- Update README on how to set IMPALA_HOME: this is needed so the
  absolute location of bin/save-version.sh can be known.

Testing:
"make" generated both PDF and HTML, and the version string was evident
and correct in both.

Change-Id: Id2036993636c8f73c07790bd4bdf4da0b721549a
---
M docs/.gitignore
M docs/Makefile
M docs/README.md
M docs/impala.ditamap
R docs/impala_keydefs.ditamap.in
5 files changed, 36 insertions(+), 3 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/54/8654/1
--
To view, visit http://gerrit.cloudera.org:8080/8654
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Id2036993636c8f73c07790bd4bdf4da0b721549a
Gerrit-Change-Number: 8654
Gerrit-PatchSet: 1
Gerrit-Owner: Michael Brown <mi...@cloudera.com>
Gerrit-Reviewer: Jim Apple <jbapple-imp...@apache.org>
Gerrit-Reviewer: John Russell <jruss...@cloudera.com>
Gerrit-Reviewer: Laurel Hale <lau...@cloudera.com>
Gerrit-Reviewer: Michael Brown <mi...@cloudera.com>

Reply via email to