[Asterisk-code-review] pbx_variables: add missing ASTSBINDIR variable (asterisk[18])

Joshua Colp asteriskteam at digium.com
Wed Jan 12 08:35:55 CST 2022


Joshua Colp has submitted this change. ( https://gerrit.asterisk.org/c/asterisk/+/17840 )

Change subject: pbx_variables: add missing ASTSBINDIR variable
......................................................................

pbx_variables: add missing ASTSBINDIR variable

Every config variable in the directories
section of asterisk.conf currently has a
counterpart built-in variable containing
the value of the config option, except
for the last one, astsbindir, which should
have an ASTSBINDIR variable.

However, the actual corresponding ASTSBINDIR
variable is missing in pbx_variables.c.

This adds the missing variable so that all
the config options have their corresponding
variable.

ASTERISK-29847 #close

Change-Id: I36006faf471825b36ebc8aa5e87a3bcb38d446fc
---
M main/pbx_variables.c
1 file changed, 2 insertions(+), 0 deletions(-)

Approvals:
  Joshua Colp: Looks good to me, approved; Approved for Submit



diff --git a/main/pbx_variables.c b/main/pbx_variables.c
index 7a85989..6f7439f 100644
--- a/main/pbx_variables.c
+++ b/main/pbx_variables.c
@@ -350,6 +350,8 @@
 			s = ast_config_AST_RUN_DIR;
 		} else if (!strcmp(var, "ASTLOGDIR")) {
 			s = ast_config_AST_LOG_DIR;
+		} else if (!strcmp(var, "ASTSBINDIR")) {
+			s = ast_config_AST_SBIN_DIR;
 		} else if (!strcmp(var, "ENTITYID")) {
 			ast_eid_to_str(workspace, sizeof(workspace), &ast_eid_default);
 			s = workspace;

-- 
To view, visit https://gerrit.asterisk.org/c/asterisk/+/17840
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 18
Gerrit-Change-Id: I36006faf471825b36ebc8aa5e87a3bcb38d446fc
Gerrit-Change-Number: 17840
Gerrit-PatchSet: 2
Gerrit-Owner: N A <mail at interlinked.x10host.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: Joshua Colp <jcolp at sangoma.com>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20220112/5e148091/attachment-0001.html>


More information about the asterisk-code-review mailing list