[Asterisk-code-review] res pjsip dlg options: Fix MODULEINFO section. (asterisk[certified/13.1])

Joshua Colp asteriskteam at digium.com
Sat Apr 30 11:50:08 CDT 2016


Joshua Colp has uploaded a new change for review.

  https://gerrit.asterisk.org/2737

Change subject: res_pjsip_dlg_options: Fix MODULEINFO section.
......................................................................

res_pjsip_dlg_options: Fix MODULEINFO section.

Removed the extra space before "MODULEINFO" in res_pjsip_dlg_options.
This extra space prevented any of the dependencies from being seen by
menuselect, so building with default options would fail if PJSIP was
not installed.

This also makes the tool that extracts information for menuselect
tolerant of multiple spaces in the future.

ASTERISK-25033 #close
Reported by: Peter Whisker

Change-Id: Iccd54846f70c4a7a50cb5bf70b7bb5cb4bab3698
---
M build_tools/get_moduleinfo
M res/res_pjsip_dlg_options.c
2 files changed, 2 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/37/2737/1

diff --git a/build_tools/get_moduleinfo b/build_tools/get_moduleinfo
index 92bc7e9..2d846ad 100644
--- a/build_tools/get_moduleinfo
+++ b/build_tools/get_moduleinfo
@@ -1,3 +1,4 @@
 /\/\*\*\* MODULEINFO/ {printit=1; next}
 /\*\*\*\// {if (printit) exit}
 /.*/ {if (printit) print}
+
diff --git a/res/res_pjsip_dlg_options.c b/res/res_pjsip_dlg_options.c
index 54c9f86..564c51f 100644
--- a/res/res_pjsip_dlg_options.c
+++ b/res/res_pjsip_dlg_options.c
@@ -16,7 +16,7 @@
  * at the top of the source tree.
  */
 
-/***  MODULEINFO
+/*** MODULEINFO
 	<depend>pjproject</depend>
 	<depend>res_pjsip</depend>
 	<depend>res_pjsip_session</depend>

-- 
To view, visit https://gerrit.asterisk.org/2737
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Iccd54846f70c4a7a50cb5bf70b7bb5cb4bab3698
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: certified/13.1
Gerrit-Owner: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Corey Farrell <git at cfware.com>



More information about the asterisk-code-review mailing list