<p>Joshua Colp <strong>submitted</strong> this change.</p><p><a href="https://gerrit.asterisk.org/c/asterisk/+/13301">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  Joshua Colp: Looks good to me, approved; Approved for Submit

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">Add directories and README files for staging changes.<br><br>While doing a release, problems were encountered running the<br>process-staging-changes script. These directories and the README.md<br>files were missing from the certified branches. Adding them in to be in<br>line with the other branches.<br><br>Also fixing the .gitreview file to point to the correct branch.<br><br>Change-Id: Ia9e2ecce31ef35838da5f3166aac321b577d16c2<br>---<br>M .gitreview<br>A doc/CHANGES-staging/README.md<br>A doc/UPGRADE-staging/README.md<br>3 files changed, 74 insertions(+), 1 deletion(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/.gitreview b/.gitreview</span><br><span>index c55b750..4cd7762 100644</span><br><span>--- a/.gitreview</span><br><span>+++ b/.gitreview</span><br><span>@@ -1,5 +1,5 @@</span><br><span> [gerrit]</span><br><span style="color: hsl(0, 100%, 40%);">-defaultbranch=certified/13</span><br><span style="color: hsl(120, 100%, 40%);">+defaultbranch=certified/13.21</span><br><span> #</span><br><span> # Intentional padding to ensure it is possible to point a commit</span><br><span> # to an alternative gerrit server/repository without breaking</span><br><span>diff --git a/doc/CHANGES-staging/README.md b/doc/CHANGES-staging/README.md</span><br><span>new file mode 100644</span><br><span>index 0000000..8a2407f</span><br><span>--- /dev/null</span><br><span>+++ b/doc/CHANGES-staging/README.md</span><br><span>@@ -0,0 +1,37 @@</span><br><span style="color: hsl(120, 100%, 40%);">+## **DO NOT REMOVE THIS FILE!**</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+The only files that should be added to this directory are ones that will be</span><br><span style="color: hsl(120, 100%, 40%);">+used by the release script to update the CHANGES file automatically. The only</span><br><span style="color: hsl(120, 100%, 40%);">+time that it is necessary to add something to the CHANGES-staging directory is</span><br><span style="color: hsl(120, 100%, 40%);">+if you are either adding a new feature to Asterisk or adding new functionality</span><br><span style="color: hsl(120, 100%, 40%);">+to an existing feature. The file does not need to have a meaningful name, but</span><br><span style="color: hsl(120, 100%, 40%);">+it probably should. If there are multiple items that need documenting, you can</span><br><span style="color: hsl(120, 100%, 40%);">+add multiple files, each with their own description. If the message is going to</span><br><span style="color: hsl(120, 100%, 40%);">+be the same for each subject, then you can add multiple subject headers to one</span><br><span style="color: hsl(120, 100%, 40%);">+file. The "Subject: xxx" line is case sensitive! For example, if you are making</span><br><span style="color: hsl(120, 100%, 40%);">+a change to PJSIP, then you might add the file "res_pjsip_my_cool_feature" to</span><br><span style="color: hsl(120, 100%, 40%);">+this directory, with a short description of what it does. If you are adding</span><br><span style="color: hsl(120, 100%, 40%);">+multiple entries, they should be done in the same commit to avoid merge</span><br><span style="color: hsl(120, 100%, 40%);">+conflicts. Here's an example:</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+> Subject: res_pjsip</span><br><span style="color: hsl(120, 100%, 40%);">+> Subject: Core</span><br><span style="color: hsl(120, 100%, 40%);">+></span><br><span style="color: hsl(120, 100%, 40%);">+> Here's a pretty good description of my new feature that explains exactly what</span><br><span style="color: hsl(120, 100%, 40%);">+> it does and how to use it.</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+Here's a master-only example:</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+> Subject: res_ari</span><br><span style="color: hsl(120, 100%, 40%);">+> Master-Only: True</span><br><span style="color: hsl(120, 100%, 40%);">+></span><br><span style="color: hsl(120, 100%, 40%);">+> This change will only go into the master branch. The "Master-Only" header</span><br><span style="color: hsl(120, 100%, 40%);">+> will never be in a change not in master.</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+Note that the second subject has another header: "Master-Only". Changes that go</span><br><span style="color: hsl(120, 100%, 40%);">+into the master branch and ONLY the master branch are the only ones that should</span><br><span style="color: hsl(120, 100%, 40%);">+have this header. Also, the value can only be "true" or "True". The</span><br><span style="color: hsl(120, 100%, 40%);">+"Master-Only" part of the header IS case-sensitive, however!</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+For more information, check out the wiki page:</span><br><span style="color: hsl(120, 100%, 40%);">+https://wiki.asterisk.org/wiki/display/AST/CHANGES+and+UPGRADE.txt</span><br><span>diff --git a/doc/UPGRADE-staging/README.md b/doc/UPGRADE-staging/README.md</span><br><span>new file mode 100644</span><br><span>index 0000000..8147161</span><br><span>--- /dev/null</span><br><span>+++ b/doc/UPGRADE-staging/README.md</span><br><span>@@ -0,0 +1,36 @@</span><br><span style="color: hsl(120, 100%, 40%);">+## **DO NOT REMOVE THIS FILE!**</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+The only files that should be added to this directory are ones that will be</span><br><span style="color: hsl(120, 100%, 40%);">+used by the release script to update the UPGRADE.txt file automatically. The</span><br><span style="color: hsl(120, 100%, 40%);">+only time that it is necessary to add something to the UPGRADE-staging directory</span><br><span style="color: hsl(120, 100%, 40%);">+is if you are making a breaking change to an existing feature in Asterisk. The</span><br><span style="color: hsl(120, 100%, 40%);">+file does not need to have a meaningful name, but it probably should. If there</span><br><span style="color: hsl(120, 100%, 40%);">+are multiple items that need documenting, you can add multiple files, each with</span><br><span style="color: hsl(120, 100%, 40%);">+their own description. If the message is going to be the same for each subject,</span><br><span style="color: hsl(120, 100%, 40%);">+then you can add multiple subject headers to one file. The "Subject: xxx" line</span><br><span style="color: hsl(120, 100%, 40%);">+is case sensitive! For example, if you are making a change to PJSIP, then you</span><br><span style="color: hsl(120, 100%, 40%);">+might add the file "res_pjsip_my_cool_feature" to this directory, with a short</span><br><span style="color: hsl(120, 100%, 40%);">+description of what it does. If you are adding multiple entries, they should be</span><br><span style="color: hsl(120, 100%, 40%);">+done in the same commit to avoid merge conflicts. Here's an example:</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+> Subject: res_pjsip</span><br><span style="color: hsl(120, 100%, 40%);">+> Subject: Core</span><br><span style="color: hsl(120, 100%, 40%);">+></span><br><span style="color: hsl(120, 100%, 40%);">+> Here's a pretty good description of my new feature that explains exactly what</span><br><span style="color: hsl(120, 100%, 40%);">+> it does and how to use it.</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+Here's a master-only example:</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+> Subject: res_ari</span><br><span style="color: hsl(120, 100%, 40%);">+> Master-Only: True</span><br><span style="color: hsl(120, 100%, 40%);">+></span><br><span style="color: hsl(120, 100%, 40%);">+> This change will only go into the master branch. The "Master-Only" header</span><br><span style="color: hsl(120, 100%, 40%);">+> will never be in a change not in master.</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+Note that the second subject has another header: "Master-Only". Changes that go</span><br><span style="color: hsl(120, 100%, 40%);">+into the master branch and ONLY the master branch are the only ones that should</span><br><span style="color: hsl(120, 100%, 40%);">+have this header. Also, the value can only be "true" or "True". The</span><br><span style="color: hsl(120, 100%, 40%);">+"Master-Only" part of the header IS case-sensitive, however!</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+For more information, check out the wiki page:</span><br><span style="color: hsl(120, 100%, 40%);">+https://wiki.asterisk.org/wiki/display/AST/CHANGES+and+UPGRADE.txt</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/13301">change 13301</a>. To unsubscribe, or for help writing mail filters, 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/c/asterisk/+/13301"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: certified/13.21 </div>
<div style="display:none"> Gerrit-Change-Id: Ia9e2ecce31ef35838da5f3166aac321b577d16c2 </div>
<div style="display:none"> Gerrit-Change-Number: 13301 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Benjamin Keith Ford <bford@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@digium.com> </div>
<div style="display:none"> Gerrit-MessageType: merged </div>