[Asterisk-code-review] release summary: Strip off illegal path character in tags (repotools[master])

[Matt Jordan]

Matt Jordan has uploaded a new change for review.

  https://gerrit.asterisk.org/382

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(-)


  git pull ssh://gerrit.asterisk.org:29418/repotools refs/changes/82/382/1

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: newchange
Gerrit-Change-Id: If86246cfe6c92d26b06e82be9a35119591aea103
Gerrit-PatchSet: 1
Gerrit-Project: repotools
Gerrit-Branch: master
Gerrit-Owner: Matt Jordan <mjordan at digium.com>