[Asterisk-code-review] mkrelease.py: Don't append the project name to ChangeLogs an... (repotools[master])

Matt Jordan asteriskteam at digium.com
Tue Jul 28 08:23:30 CDT 2015


Matt Jordan has submitted this change and it was merged.

Change subject: mkrelease.py: Don't append the project name to ChangeLogs and README files
......................................................................


mkrelease.py: Don't append the project name to ChangeLogs and README files

The ChangeLog and README files are always of the form
'ChangeLog-{version}' and 'README-{version}'. Currently, we are also
appending the project name, which is incorrect. This patch adds the
modified version name (subtituting out path delimiters) so that the
files have the expected name.

Change-Id: I9b6851725b3e2fadf26c86a8600505fa5b942622
---
M mkrelease.py
1 file changed, 2 insertions(+), 2 deletions(-)

Approvals:
  Matt Jordan: Looks good to me, approved; Verified
  Joshua Colp: Looks good to me, but someone else must approve



diff --git a/mkrelease.py b/mkrelease.py
index 6fec6b1..b88440b 100755
--- a/mkrelease.py
+++ b/mkrelease.py
@@ -534,9 +534,9 @@
         path - The location of the extracted Asterisk release
         """
         shutil.copyfile(os.path.join(path, 'README'),
-                        'README-{0}'.format(release_name))
+                        'README-{0}'.format(version.replace('/', '-')))
         shutil.copyfile(os.path.join(path, 'ChangeLog'),
-                        'ChangeLog-{0}'.format(release_name))
+                        'ChangeLog-{0}'.format(version.replace('/', '-')))
         summary_prefix = '{0}-summary'.format(release_name)
         shutil.copyfile(os.path.join(path, '{0}.html'.format(summary_prefix)),
                         '{0}.html'.format(summary_prefix))

-- 
To view, visit https://gerrit.asterisk.org/975
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I9b6851725b3e2fadf26c86a8600505fa5b942622
Gerrit-PatchSet: 1
Gerrit-Project: repotools
Gerrit-Branch: master
Gerrit-Owner: Matt Jordan <mjordan at digium.com>
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Matt Jordan <mjordan at digium.com>



More information about the asterisk-code-review mailing list