osmith submitted this change.

View Change


Approvals: pespin: Looks good to me, but someone else must approve neels: Looks good to me, approved Jenkins Builder: Verified laforge: Looks good to me, approved
publish-manuals-for-tags: enable IU/PFCP VTY cmds

Enable configure options to build manuals with all VTY commands in the
script that builds manuals for tagged releases.

Related: OS#6013
Change-Id: I9685857348e3b38f13acc1429c7a49fb9e5d92f3
---
M scripts/manuals/publish-manuals-for-tags.sh
1 file changed, 28 insertions(+), 1 deletion(-)

diff --git a/scripts/manuals/publish-manuals-for-tags.sh b/scripts/manuals/publish-manuals-for-tags.sh
index b40b602..99e117c 100755
--- a/scripts/manuals/publish-manuals-for-tags.sh
+++ b/scripts/manuals/publish-manuals-for-tags.sh
@@ -213,6 +213,19 @@
fi
}

+# Additional configure options to use, so manuals include all VTY commands
+# $1: repo name
+get_configure_opts_from_repo_name() {
+ case "$1" in
+ osmo-hnbgw)
+ echo "--enable-pfcp"
+ ;;
+ osmo-msc|osmo-sgsn)
+ echo "--enable-iu"
+ ;;
+ esac
+}
+
# $1: docs dir
get_repo_name_from_docs_dir() {
case "$1" in
@@ -323,6 +336,7 @@
build_publish_manuals() {
local repo="$1"
local tag="$2"
+ local configure_opts="--enable-manuals $(get_configure_opts_from_repo_name "$repo")"
echo "$LOG_PREFIX Building manuals"

if ! docker run \
@@ -361,7 +375,7 @@
;;
*)
su build -c \"autoreconf -fi\"
- su build -c \"./configure --enable-manuals\"
+ su build -c \"./configure $configure_opts\"
su build -c \"make -j$(nproc)\"
;;
esac

To view, visit change 32434. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: I9685857348e3b38f13acc1429c7a49fb9e5d92f3
Gerrit-Change-Number: 32434
Gerrit-PatchSet: 4
Gerrit-Owner: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: neels <nhofmeyr@sysmocom.de>
Gerrit-Reviewer: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged