[asterisk-commits] release summary: Strip off illegal path character in tags (repotools[master])

[SVN commits to the Asterisk project]

Joshua Colp has submitted this change and it was merged.

Change subject: release_summary: Strip off illegal path character in tags
......................................................................


release_summary: Strip off illegal path character in tags

Some tags may contain a path character, i.e., '/'. This patch strips
those off and replaces them with a hyphen, '-'. This causes tags such as
"certified/13.1" to be treated as "certified-13.1" for the purposes of
the release-summary file name.

REP-15

Change-Id: If86246cfe6c92d26b06e82be9a35119591aea103
---
M release_summary.py
1 file changed, 2 insertions(+), 1 deletion(-)

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



diff --git a/release_summary.py b/release_summary.py
index 2f25a08..6348c3e 100755
--- a/release_summary.py
+++ b/release_summary.py
@@ -450,8 +450,9 @@
         if out_file:
             fn_base = out_file
         else:
+            file_version = self.options.version.replace('/', '-')
             fn_base = "{0}-{1}-summary".format(self.options.project,
-                      self.options.version)
+                      file_version)
 
         report = HTML()
         root = report.html(xmlns="http://www.w3.org/1999/xhtml")

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

Gerrit-MessageType: merged
Gerrit-Change-Id: If86246cfe6c92d26b06e82be9a35119591aea103
Gerrit-PatchSet: 4
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>
Gerrit-Reviewer: Samuel Galarneau <sgalarneau at digium.com>