[Asterisk-code-review] build: Add staging directories for future changes. (...asterisk[13])

George Joseph asteriskteam at digium.com
Wed Mar 27 14:25:17 CDT 2019


George Joseph has submitted this change and it was merged. ( https://gerrit.asterisk.org/c/asterisk/+/11195 )

Change subject: build: Add staging directories for future changes.
......................................................................

build: Add staging directories for future changes.

This is the first step in changing the release process so that changes
made to the CHANGES and UPGRADE.txt files do not result in merge
conflicts every time multiple people modify these files. The changes
made will go in these new directories: doc/CHANGES-staging and
doc/UPGRADE-staging. The README.md files explain how things will work,
but here's a little overview. When you make a change that would go in
either CHANGES or UPGRADE.txt, this should instead be documented in a
new file in the doc/CHANGES-staging or doc/UPGRADE-staging directory,
respectively. The format will look like this:

   Subject: res_pjsip

   A description that explains the changes made and why. The release
   script will handle the bulleting and section separators! The
   'Subject:' header is case-sensitive.

   You can still separate with new lines within your description.

   Subject: res_ari
   Master-Only: true

   You can have more than one subject, and they don't have to be the
   same! Also, the 'Master-Only' header should always be true and is
   also case-sensitive (but the value is not - you can have 'true' or
   'True'). This header will only ever be present in the master branch.

For more information, check out the wiki page:
https://wiki.asterisk.org/wiki/display/AST/CHANGES+and+UPGRADE.txt

This is an initial change for ASTERISK_28111. Functionally, this will
make no difference, but it will prep the directories for when the
changes from CHANGES and UPGRADE.txt are extracted.

Change-Id: I8d852f284f66ac456b26dcb899ee46babf7d15b6
---
A doc/CHANGES-staging/README.md
A doc/UPGRADE-staging/README.md
2 files changed, 65 insertions(+), 0 deletions(-)

Approvals:
  Friendly Automation: Verified
  George Joseph: Looks good to me, approved; Approved for Submit



diff --git a/doc/CHANGES-staging/README.md b/doc/CHANGES-staging/README.md
new file mode 100644
index 0000000..da0df2a
--- /dev/null
+++ b/doc/CHANGES-staging/README.md
@@ -0,0 +1,33 @@
+## **DO NOT REMOVE THIS FILE!**
+
+The only files that should be added to this directory are ones that will be
+used by the release script to update the CHANGES file automatically. The only
+time that it is necessary to add something to the CHANGES-staging directory is
+if you are either adding a new feature to Asterisk or adding new functionality
+to an existing feature. The file does not need to have a meaningful name, but
+it probably should. If there are multiple items that need documenting, each can
+be separated with a subject line, which should always start with "Subject:",
+followed by the subject of the change. This is case sensitive! For example, if
+you are making a change to PJSIP, then you might add the file
+"res_pjsip_my_cool_feature" to this directory, with a short description of what
+it does. If you are adding multiple entries, they should be done in the same
+commit to avoid merge conflicts. Here's an example:
+
+> Subject: res_pjsip
+>
+> Here's a pretty good description of my new feature that explains exactly what
+> it does and how to use it.
+>
+> Subject: core
+> Master-Only: true
+>
+> Here's another description of something else I added that is a big enough
+> change to warrant another entry in the CHANGES file.
+
+Note that the second subject has another header: "Master-Only". Changes that go
+into the master branch and ONLY the master branch are the only ones that should
+have this header. Also, the value can only be "true" or "True". The
+"Master-Only" part of the header IS case-sensitive, however!
+
+For more information, check out the wiki page:
+https://wiki.asterisk.org/wiki/display/AST/CHANGES+and+UPGRADE.txt
diff --git a/doc/UPGRADE-staging/README.md b/doc/UPGRADE-staging/README.md
new file mode 100644
index 0000000..1ef9334
--- /dev/null
+++ b/doc/UPGRADE-staging/README.md
@@ -0,0 +1,32 @@
+## **DO NOT REMOVE THIS FILE!**
+
+The only files that should be added to this directory are ones that will be
+used by the release script to update the UPGRADE.txt file automatically. The
+only time that it is necessary to add something to the UPGRADE-staging directory
+is if you are making a breaking change to an existing feature in Asterisk. The
+file does not need to have a meaningful name, but it probably should. If there
+are multiple items that need documenting, each can be separated with a subject
+line, which should always start with "Subject:", followed by the subject of the
+change. This is case sensitive! For example, if you are making a change to PJSIP,
+then you might add the file "res_pjsip_breaking_change" to this directory, with
+a short description of what it does. If you are adding multiple entries, they
+should be done in the same commit to avoid merge conflicts. Here's an example:
+
+> Subject: res_pjsip
+>
+> Here's a pretty good description of what I changed that explains exactly what
+> it does and why it breaks things (and why they needed to be broken).
+>
+> Subject: core
+> Master-Only: true
+>
+> Here's another description of something else I added that is a big enough
+> change to warrant another entry in the UPDATE.txt file.
+
+Note that the second subject has another header: "Master-Only". Changes that go
+into the master branch and ONLY the master branch are the only ones that should
+have this header. Also, the value can only be "true" or "True". The
+"Master-Only" part of the header IS case-sensitive, however!
+
+For more information, check out the wiki page:
+https://wiki.asterisk.org/wiki/display/AST/CHANGES+and+UPGRADE.txt

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

Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Change-Id: I8d852f284f66ac456b26dcb899ee46babf7d15b6
Gerrit-Change-Number: 11195
Gerrit-PatchSet: 2
Gerrit-Owner: Benjamin Keith Ford <bford at digium.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20190327/d2f9d9cb/attachment-0001.html>


More information about the asterisk-code-review mailing list