<p>George Joseph <strong>merged</strong> this change.</p><p><a href="https://gerrit.asterisk.org/6024">View Change</a></p><div style="white-space:pre-wrap">Approvals:
Joshua Colp: Looks good to me, but someone else must approve
George Joseph: Looks good to me, approved; Approved for Submit
</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">say.c: Fix file locations for second, seconds, minute, minutes files<br><br>The seconds and minutes files have always existed in the base language<br>directory of the Core package. So say.c has always been calling the wrong<br>location (under digits/) for those two files and in the case of second and<br>minute they didn't exist in the Core packages at all.<br><br>The 1.6 sounds release moves the second and minute files into Core from<br>Extra for the languages that already had them. A future release will include<br>the second and minute files for languages that didn't already have them.<br><br>This patch just changes all the target locations for second, seconds,<br>minute, and minutes that were under the digits subdir to be under the root of<br>sounds instead. Which is where the sounds will be for some languages after 1.6<br>sounds and for all languages after a future release.<br><br>ASTERISK-25810 #close<br><br>Change-Id: I05d9d4bee6a7237030530a46e7eb3df15f13f702<br>Reported-by: Nicolas Riendeau<br>---<br>M main/say.c<br>1 file changed, 26 insertions(+), 26 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/main/say.c b/main/say.c<br>index 44f55e2..c97dc9f 100644<br>--- a/main/say.c<br>+++ b/main/say.c<br>@@ -4441,9 +4441,9 @@<br> }<br> if (!res && next_item(&format[offset + 1]) == 'S') { /* minutes only if seconds follow */<br> if (tm.tm_min == 1) {<br>- res = wait_file(chan, ints, "digits/minute", lang);<br>+ res = wait_file(chan, ints, "minute", lang);<br> } else {<br>- res = wait_file(chan, ints, "digits/minutes", lang);<br>+ res = wait_file(chan, ints, "minutes", lang);<br> }<br> }<br> break;<br>@@ -4517,7 +4517,7 @@<br> if (!res) {<br> res = ast_say_number(chan, tm.tm_sec, ints, lang, "f");<br> if (!res) {<br>- res = wait_file(chan, ints, "digits/seconds", lang);<br>+ res = wait_file(chan, ints, "seconds", lang);<br> }<br> }<br> break;<br>@@ -4637,16 +4637,16 @@<br> case 'M':<br> /* Minute */<br> if (next_item(&format[offset + 1]) == 'S') { /* zero 'digits/0' only if seconds follow */<br>- res = ast_say_number(chan, tm.tm_min, ints, lang, "f"); /* female only if we say digits/minutes */<br>+ res = ast_say_number(chan, tm.tm_min, ints, lang, "f"); /* female only if we say minutes */<br> } else if (tm.tm_min > 0) {<br> res = ast_say_number(chan, tm.tm_min, ints, lang, (char *) NULL);<br> }<br> <br> if (!res && next_item(&format[offset + 1]) == 'S') { /* minutes only if seconds follow */<br> if (tm.tm_min == 1) {<br>- res = wait_file(chan, ints, "digits/minute", lang);<br>+ res = wait_file(chan, ints, "minute", lang);<br> } else {<br>- res = wait_file(chan, ints, "digits/minutes", lang);<br>+ res = wait_file(chan, ints, "minutes", lang);<br> }<br> }<br> break;<br>@@ -4720,7 +4720,7 @@<br> if (!res) {<br> res = ast_say_number(chan, tm.tm_sec, ints, lang, "f");<br> if (!res) {<br>- res = wait_file(chan, ints, tm.tm_sec == 1 ? "digits/second" : "digits/seconds", lang);<br>+ res = wait_file(chan, ints, tm.tm_sec == 1 ? "second" : "seconds", lang);<br> }<br> }<br> break;<br>@@ -4853,9 +4853,9 @@<br> if (!res && next_item(&format[offset + 1]) == 'S') { /* minutes only if seconds follow */<br> /* Say minute/minutes depending on whether minutes end in 1 */<br> if ((tm.tm_min % 10 == 1) && (tm.tm_min != 11)) {<br>- res = wait_file(chan, ints, "digits/minute", lang);<br>+ res = wait_file(chan, ints, "minute", lang);<br> } else {<br>- res = wait_file(chan, ints, "digits/minutes", lang);<br>+ res = wait_file(chan, ints, "minutes", lang);<br> }<br> }<br> break;<br>@@ -4930,9 +4930,9 @@<br> res = ast_say_number(chan, tm.tm_sec, ints, lang, "f");<br> /* Say minute/minutes depending on whether seconds end in 1 */<br> if (!res && (tm.tm_sec % 10 == 1) && (tm.tm_sec != 11)) {<br>- res = wait_file(chan, ints, "digits/second", lang);<br>+ res = wait_file(chan, ints, "second", lang);<br> } else {<br>- res = wait_file(chan, ints, "digits/seconds", lang);<br>+ res = wait_file(chan, ints, "seconds", lang);<br> }<br> }<br> break;<br>@@ -5652,7 +5652,7 @@<br> /* Seconds */<br> res = ast_say_number(chan, tm.tm_sec, ints, lang, (char * ) NULL);<br> if (!res) {<br>- res = wait_file(chan, ints, "digits/second", lang);<br>+ res = wait_file(chan, ints, "second", lang);<br> }<br> break;<br> case 'T':<br>@@ -6303,9 +6303,9 @@<br> one = tm.tm_sec % 10;<br> <br> if (one > 1 && one < 5 && ten != 1)<br>- res = wait_file(chan, ints, "digits/seconds", lang);<br>+ res = wait_file(chan, ints, "seconds", lang);<br> else<br>- res = wait_file(chan, ints, "digits/second", lang);<br>+ res = wait_file(chan, ints, "second", lang);<br> }<br> }<br> }<br>@@ -6469,9 +6469,9 @@<br> res = ast_say_number(chan, tm.tm_min, ints, lang, NULL);<br> if (!res) {<br> if (tm.tm_min > 1) {<br>- res = wait_file(chan, ints, "digits/minutes", lang);<br>+ res = wait_file(chan, ints, "minutes", lang);<br> } else {<br>- res = wait_file(chan, ints, "digits/minute", lang);<br>+ res = wait_file(chan, ints, "minute", lang);<br> }<br> }<br> } else {<br>@@ -6567,9 +6567,9 @@<br> res = ast_say_number(chan, tm.tm_sec, ints, lang, NULL);<br> if (!res) {<br> if (tm.tm_sec > 1) {<br>- res = wait_file(chan, ints, "digits/seconds", lang);<br>+ res = wait_file(chan, ints, "seconds", lang);<br> } else {<br>- res = wait_file(chan, ints, "digits/second", lang);<br>+ res = wait_file(chan, ints, "second", lang);<br> }<br> }<br> } else {<br>@@ -6783,7 +6783,7 @@<br> }<br> }<br> if (!res) {<br>- res = wait_file(chan, ints, "digits/minute", lang);<br>+ res = wait_file(chan, ints, "minute", lang);<br> }<br> break;<br> case 'P':<br>@@ -6867,7 +6867,7 @@<br> }<br> }<br> if (!res) {<br>- res = wait_file(chan, ints, "digits/second", lang);<br>+ res = wait_file(chan, ints, "second", lang);<br> }<br> break;<br> case 'T':<br>@@ -7022,7 +7022,7 @@<br> if (tm.tm_min > 0) {<br> res = ast_say_number(chan, tm.tm_min, ints, lang, "f");<br> if (!res)<br>- res = ast_streamfile(chan, "digits/minute", lang);<br>+ res = ast_streamfile(chan, "minute", lang);<br> }<br> return res;<br> }<br>@@ -7117,9 +7117,9 @@<br> res = ast_say_number(chan, tm.tm_min, ints, lang, (char *) NULL);<br> if (!res) {<br> if (tm.tm_min > 1)<br>- res = wait_file(chan, ints, "digits/minutes", lang);<br>+ res = wait_file(chan, ints, "minutes", lang);<br> else<br>- res = wait_file(chan, ints, "digits/minute", lang);<br>+ res = wait_file(chan, ints, "minute", lang);<br> }<br> }<br> return res;<br>@@ -7179,7 +7179,7 @@<br> if (!res)<br> res = ast_say_number(chan, tm.tm_min, ints, lang, (char *) NULL);<br> if (!res)<br>- res = ast_streamfile(chan, "digits/minute", lang);<br>+ res = ast_streamfile(chan, "minute", lang);<br> if (!res)<br> res = ast_waitstream(chan, ints);<br> return res;<br>@@ -7602,7 +7602,7 @@<br> if (!res)<br> res = ast_say_number(chan, tm.tm_min, ints, lang, (char *) NULL);<br> if (!res)<br>- res = ast_streamfile(chan, "digits/minute", lang);<br>+ res = ast_streamfile(chan, "minute", lang);<br> if (!res)<br> res = ast_waitstream(chan, ints);<br> return res;<br>@@ -8480,7 +8480,7 @@<br> if (!res)<br> res = ast_say_number_full_gr(chan, tm.tm_sec, ints, lang, -1, -1);<br> if (!res)<br>- ast_copy_string(nextmsg, "digits/seconds", sizeof(nextmsg));<br>+ ast_copy_string(nextmsg, "seconds", sizeof(nextmsg));<br> res = wait_file(chan, ints, nextmsg, lang);<br> break;<br> case 'T':<br></pre><p>To view, visit <a href="https://gerrit.asterisk.org/6024">change 6024</a>. To unsubscribe, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/6024"/><meta itemprop="name" content="View Change"/></div></div>
<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: merged </div>
<div style="display:none"> Gerrit-Change-Id: I05d9d4bee6a7237030530a46e7eb3df15f13f702 </div>
<div style="display:none"> Gerrit-Change-Number: 6024 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Rusty Newton <rnewton@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@digium.com> </div>