[rtems-source-builder commit] sb: Create hopefully more valid file names

[Sebastian Huber]

Module:    rtems-source-builder
Branch:    master
Commit:    e1257d4fe907d23c946d2cd7d6529821a3891889
Changeset: http://git.rtems.org/rtems-source-builder/commit/?id=e1257d4fe907d23c946d2cd7d6529821a3891889

Author:    Sebastian Huber <sebastian.huber at embedded-brains.de>
Date:      Fri Jun  2 14:38:13 2017 +0200

sb: Create hopefully more valid file names

---

 source-builder/sb/download.py | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/source-builder/sb/download.py b/source-builder/sb/download.py
index 4bba7de..dd7880d 100644
--- a/source-builder/sb/download.py
+++ b/source-builder/sb/download.py
@@ -26,6 +26,7 @@ from __future__ import print_function
 
 import hashlib
 import os
+import re
 import stat
 import sys
 try:
@@ -180,12 +181,9 @@ def _http_parser(source, pathkey, config, opts):
                 raise error.general('gitweb.cgi path missing p or h: %s' % (url))
             source['file'] = '%s-%s.patch' % (p, h)
         #
-        # Check the source file name for any extra request query data and remove if
-        # found. Some hosts do not like file names containing them.
+        # Wipe out everything special in the file name.
         #
-        if '?' in source['file']:
-            qmark = source['file'].find('?')
-            source['file'] = source['file'][:qmark]
+        source['file'] = re.sub(r'[^a-zA-Z0-9.\-]+', '-', source['file'])
     #
     # Check local path
     #