fix python conventions

1 files changed, 4 insertions, 4 deletions

diff --git a/tools/shared.py b/tools/shared.py
index c0c48b68..9f51f008 100644
--- a/tools/shared.py
+++ b/tools/shared.py
@@ -550,14 +550,14 @@ set(CMAKE_FIND_ROOT_PATH_MODE_PACKAGE ONLY)''' % { 'winfix': '' if not WINDOWS e
     if configure: # Useful in debugging sometimes to comment this out (and the lines below up to and including the |link| call)
       Building.configure(configure + configure_args, stdout=open(os.path.join(project_dir, 'configure_'), 'w'),
                                                      stderr=open(os.path.join(project_dir, 'configure_err'), 'w'), env=env)
-    def openMakeOut(i, mode='r'):
+    def open_make_out(i, mode='r'):
       return open(os.path.join(project_dir, 'make_' + str(i)), mode)
     
-    def openMakeErr(i, mode='r'):
+    def open_make_err(i, mode='r'):
       return open(os.path.join(project_dir, 'make_err' + str(i)), mode)
     
     for i in range(2): # FIXME: Sad workaround for some build systems that need to be run twice to succeed (e.g. poppler)
-      with openMakeOut(i, 'w') as make_out, openMakeErr(i, 'w') as make_err:
+      with open_make_out(i, 'w') as make_out, open_make_err(i, 'w') as make_err:
         Building.make(make + make_args, stdout=make_out,
                                         stderr=make_err, env=env)
       try:
@@ -570,7 +570,7 @@ set(CMAKE_FIND_ROOT_PATH_MODE_PACKAGE ONLY)''' % { 'winfix': '' if not WINDOWS e
       except:
         if i > 0:
           # Due to the ugly hack above our best guess is to output the first run
-          with openMakeErr(0) as ferr:
+          with open_make_err(0) as ferr:
             for line in ferr:
               sys.stderr.write(line)
           raise Exception('could not build library ' + name)