<p>Corey Farrell <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/6025">View Change</a></p><p>Patch set 1:</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Should this include a note in UPGRADE.txt?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I'm not sure. It seems like a minor change. Say.c always had<br>targets for these sounds in places they didn't exist by default, so<br>it never worked. The modification just targets the files where they<br>now exist.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I didn't move the files that already existed in the core package,<br>which is the core supported package. Changes to the extra package<br>should be inconsequential as nothing relies on those sounds.</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Parts of say.c will no longer work with core sounds 1.5 + extra<br>sounds 1.5.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Which parts will no longer work? Maybe I overlooked something?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Sorry I was confused by part of the commit message:<br>The 1.6 sounds release moves the second and minute files into core from Extra</p><p style="white-space: pre-wrap; word-wrap: break-word;">I misread this to mean that second/minute existed in the extra archive within the digits/ folder.</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">>Can we also get an announcement of the new sounds versions to the<br>same mailing lists that announce Asterisk upgrades?</p><p style="white-space: pre-wrap; word-wrap: break-word;">I generally make an announcement on the lists when we release a new<br>sounds version. I'll be sure to hit asterisk-announce as well as<br>asterisk-users. I was also going to post on the blog.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I just posted ASTERISK-27142, please see this before making any announcement of the new asterisk-sounds-core.</p><ul style="list-style: none; padding-left: 20px;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/6025">change 6025</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/6025"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 14 </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I05d9d4bee6a7237030530a46e7eb3df15f13f702 </div>
<div style="display:none"> Gerrit-Change-Number: 6025 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Rusty Newton <rnewton@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Corey Farrell <git@cfware.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: Rusty Newton <rnewton@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 18 Jul 2017 15:55:03 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>