[asterisk-commits] mkrelease: Use the release name instead of the version tag w... (repotools[master])

[SVN commits to the Asterisk project]

Joshua Colp has submitted this change and it was merged.

Change subject: mkrelease: Use the release name instead of the version tag when naming files
......................................................................


mkrelease: Use the release name instead of the version tag when naming files

When creating the README and ChangeLog, the release name (which is a
path safe version of the tag) should be used instead of the version tag.
This prevents tags with '/' in their name from being used as a suffix to
the ChangeLog and README files.

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

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



diff --git a/mkrelease.py b/mkrelease.py
index ab1706c..642c383 100755
--- a/mkrelease.py
+++ b/mkrelease.py
@@ -501,9 +501,9 @@
         path - The location of the extracted Asterisk release
         """
         shutil.copyfile(os.path.join(path, 'README'),
-                        'README-{0}'.format(version))
+                        'README-{0}'.format(release_name))
         shutil.copyfile(os.path.join(path, 'ChangeLog'),
-                        'ChangeLog-{0}'.format(version))
+                        'ChangeLog-{0}'.format(release_name))
         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/605
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I24109f245c69f4cc80ecdcdca2047df81416cc6e
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: Mark Michelson <mmichelson at digium.com>