Refactor release scripts, automatically update Wandbox on release

scripts/generateSingleHeader.py

@@ -9,9 +9,9 @@ import datetime
 import string
 
 from scriptCommon import catchPath
-from releaseCommon import Version
 
 
+def generate(v):
     includesParser = re.compile( r'\s*#\s*include\s*"(.*)"' )
     guardParser = re.compile( r'\s*#.*TWOBLUECUBES_CATCH_.*_INCLUDED')
     defineParser = re.compile( r'\s*#define')
@@ -25,38 +25,38 @@ seenHeaders = set([])
     rootPath = os.path.join( catchPath, 'include/' )
     outputPath = os.path.join( catchPath, 'single_include/catch.hpp' )
 
-includeImpl = True
+
+    globals = {
+        'ifdefs'     :  0,
+        'implIfDefs' : -1,
+        'includeImpl': True
+    }
 
     for arg in sys.argv[1:]:
         arg = string.lower(arg)
         if arg == "noimpl":
-        includeImpl = False
+            globals['includeImpl'] = False
             print( "Not including impl code" )
         else:
             print( "\n** Unrecognised argument: " + arg + " **\n" )
             exit(1)
 
     out = open( outputPath, 'w' )
-ifdefs = 0
-implIfDefs = -1
 
     def write( line ):
-    if includeImpl or implIfDefs == -1:
+        if globals['includeImpl'] or globals['implIfDefs'] == -1:
             out.write( line )
 
     def parseFile( path, filename ):
-    global ifdefs
-    global implIfDefs
-
         f = open( path + filename, 'r' )
         blanks = 0
         for line in f:
             if ifParser.match( line ):
-            ifdefs = ifdefs + 1
+                globals['ifdefs'] += 1
             elif endIfParser.match( line ):
-            ifdefs = ifdefs - 1
-            if ifdefs == implIfDefs:
-                implIfDefs = -1
+                globals['ifdefs'] -= 1
+                if globals['ifdefs'] == globals['implIfDefs']:
+                    globals['implIfDefs'] = -1
             m = includesParser.match( line )
             if m:
                 header = m.group(1)
@@ -74,7 +74,7 @@ def parseFile( path, filename ):
                         parseFile( rootPath + headerPath + sep, headerFile )
             else:
                 if ifImplParser.match(line):
-                implIfDefs = ifdefs
+                    globals['implIfDefs'] = globals['ifdefs']
                 if (not guardParser.match( line ) or defineParser.match( line ) ) and not commentParser1.match( line )and not commentParser2.match( line ):
                     if blankParser.match( line ):
                         blanks = blanks + 1
@@ -84,7 +84,6 @@ def parseFile( path, filename ):
                         write( line.rstrip() + "\n" )
 
 
-v = Version()
     out.write( "/*\n" )
     out.write( " *  Catch v{0}\n".format( v.getVersionString() ) )
     out.write( " *  Generated: {0}\n".format( datetime.datetime.now() ) )
@@ -101,5 +100,9 @@ out.write( "#define TWOBLUECUBES_SINGLE_INCLUDE_CATCH_HPP_INCLUDED\n" )
     parseFile( rootPath, 'catch.hpp' )
 
     out.write( "#endif // TWOBLUECUBES_SINGLE_INCLUDE_CATCH_HPP_INCLUDED\n\n" )
-
+    out.close()
     print ("Generated single include for Catch v{0}\n".format( v.getVersionString() ) )
+
+if __name__ == '__main__':
+    from releaseCommon import Version
+    generate(Version())

scripts/majorRelease.py

@@ -1,13 +1,10 @@
 #!/usr/bin/env python
 
 from  __future__ import  print_function
-from releaseCommon import Version
+import releaseCommon
 
-v = Version()
+v = releaseCommon.Version()
 v.incrementMajorVersion()
-v.updateVersionFile()
-v.updateReadmeFile()
-v.updateConanFile()
-v.updateConanTestFile()
+releaseCommon.performUpdates(v)
 
 print( "Updated Version.hpp, README and Conan to v{0}".format( v.getVersionString() ) )

scripts/minorRelease.py

@@ -1,13 +1,10 @@
 #!/usr/bin/env python
 
 from  __future__ import  print_function
-from releaseCommon import Version
+import releaseCommon
 
-v = Version()
+v = releaseCommon.Version()
 v.incrementMinorVersion()
-v.updateVersionFile()
-v.updateReadmeFile()
-v.updateConanFile()
-v.updateConanTestFile()
+releaseCommon.performUpdates(v)
 
 print( "Updated Version.hpp, README and Conan to v{0}".format( v.getVersionString() ) )

scripts/patchRelease.py

@@ -1,13 +1,10 @@
 #!/usr/bin/env python
 
 from  __future__ import  print_function
-from releaseCommon import Version
+import releaseCommon
 
-v = Version()
+v = releaseCommon.Version()
 v.incrementPatchNumber()
-v.updateVersionFile()
-v.updateReadmeFile()
-v.updateConanFile()
-v.updateConanTestFile()
+releaseCommon.performUpdates(v)
 
 print( "Updated Version.hpp, README and Conan to v{0}".format( v.getVersionString() ) )

scripts/releaseCommon.py

@@ -6,6 +6,8 @@ import re
 import string
 
 from scriptCommon import catchPath
+import generateSingleHeader
+import updateWandbox
 
 versionParser = re.compile( r'(\s*static\sVersion\sversion)\s*\(\s*(.*)\s*,\s*(.*)\s*,\s*(.*)\s*,\s*\"(.*)\"\s*,\s*(.*)\s*\).*' )
 rootPath = os.path.join( catchPath, 'include/' )
@@ -76,8 +78,12 @@ class Version:
         for line in lines:
             f.write( line + "\n" )
 
-    def updateReadmeFile(self):
+def updateReadmeFile(version):
     downloadParser = re.compile( r'<a href=\"https://github.com/philsquared/Catch/releases/download/v\d+\.\d+\.\d+/catch.hpp\">' )
+    success, wandboxLink = updateWandbox.uploadFiles()
+    if not success:
+        print('Error when uploading to wandbox: {}'.format(wandboxLink))
+        exit(1)
     f = open( readmePath, 'r' )
     lines = []
     for line in f:
@@ -85,17 +91,19 @@ class Version:
     f.close()
     f = open( readmePath, 'w' )
     for line in lines:
-            line = downloadParser.sub( r'<a href="https://github.com/philsquared/Catch/releases/download/v{0}/catch.hpp">'.format(self.getVersionString()) , line)
+        line = downloadParser.sub( r'<a href="https://github.com/philsquared/Catch/releases/download/v{0}/catch.hpp">'.format(version.getVersionString()) , line)
+        if '[![Try online](https://img.shields.io/badge/try-online-blue.svg)]' in line:
+            line = '[![Try online](https://img.shields.io/badge/try-online-blue.svg)]({0})'.format(wandboxLink)
         f.write( line + "\n" )
 
-    def updateConanFile(self):
+def updateConanFile(version):
     conanParser = re.compile( r'    version = "\d+\.\d+\.\d+.*"')
     f = open( conanPath, 'r' )
     lines = []
     for line in f:
         m = conanParser.match( line )
         if m:
-                lines.append( '    version = "{0}"'.format(format(self.getVersionString())) )
+            lines.append( '    version = "{0}"'.format(format(version.getVersionString())) )
         else:
             lines.append( line.rstrip() )
     f.close()
@@ -103,17 +111,28 @@ class Version:
     for line in lines:
         f.write( line + "\n" )
 
-    def updateConanTestFile(self):
+def updateConanTestFile(version):
     conanParser = re.compile( r'    requires = \"Catch\/\d+\.\d+\.\d+.*@%s\/%s\" % \(username, channel\)')
     f = open( conanTestPath, 'r' )
     lines = []
     for line in f:
         m = conanParser.match( line )
         if m:
-                lines.append( '    requires = "Catch/{0}@%s/%s" % (username, channel)'.format(format(self.getVersionString())) )
+            lines.append( '    requires = "Catch/{0}@%s/%s" % (username, channel)'.format(format(version.getVersionString())) )
         else:
             lines.append( line.rstrip() )
     f.close()
     f = open( conanTestPath, 'w' )
     for line in lines:
         f.write( line + "\n" )
+
+def performUpdates(version):
+    # First update version file, so we can regenerate single header and
+    # have it ready for upload to wandbox, when updating readme
+    version.updateVersionFile()
+    # ToDo: Regenerate single header
+    generateSingleHeader.generate(version)
+    updateReadmeFile(version)
+
+    updateConanFile(version)
+    updateConanTestFile(version)

scripts/updateWandbox.py

@@ -0,0 +1,47 @@
+#!/usr/bin/env python
+
+import json
+import os
+import urllib2
+
+from scriptCommon import catchPath
+
+def upload(options):
+    request = urllib2.Request('http://melpon.org/wandbox/api/compile.json')
+    request.add_header('Content-Type', 'application/json')
+    response = urllib2.urlopen(request, json.dumps(options))
+    return json.loads(response.read())
+
+main_file = '''
+#define CATCH_CONFIG_MAIN  // This tells Catch to provide a main() - only do this in one cpp file
+#include "catch.hpp"
+
+unsigned int Factorial( unsigned int number ) {
+    return number <= 1 ? number : Factorial(number-1)*number;
+}
+
+TEST_CASE( "Factorials are computed", "[factorial]" ) {
+    REQUIRE( Factorial(1) == 1 );
+    REQUIRE( Factorial(2) == 2 );
+    REQUIRE( Factorial(3) == 6 );
+    REQUIRE( Factorial(10) == 3628800 );
+}
+'''
+
+def uploadFiles():
+    response = upload({
+        'compiler': 'gcc-head',
+        'code': main_file,
+        'codes': [{
+            'file': 'catch.hpp',
+            'code': open(os.path.join(catchPath, 'single_include', 'catch.hpp')).read()
+        }],
+        'options': 'c++11,cpp-no-pedantic,boost-nothing',
+        'compiler-option-raw': '-DCATCH_CONFIG_FAST_COMPILE',
+        'save': True
+    })
+
+    if 'status' in response and not 'compiler_error' in response:
+        return True, response['url']
+    else:
+        return False, response