Merge branch 'fix-build-skip' into 'master'

Fix use of should_skip_build()

Closes #20

See merge request !34

bob/devtools/scripts/build.py

@@ -139,17 +139,15 @@ def build(recipe_dir, python, condarc, config, no_test, append_file,
 
     # pre-renders the recipe - figures out the destination
     metadata = get_rendered_metadata(d, conda_config)
-    rendered_recipe = get_parsed_recipe(metadata)
-    path = get_output_path(metadata, conda_config)
 
     # checks if we should actually build this recipe
     if should_skip_build(metadata):
-      logger.info('Skipping UNSUPPORTED build of %s-%s-py%s for %s',
-          rendered_recipe['package']['name'],
-          rendered_recipe['package']['version'], python.replace('.',''),
-          arch)
+      logger.info('Skipping UNSUPPORTED build of %s for %s', recipe_dir, arch)
       continue
 
+    rendered_recipe = get_parsed_recipe(metadata)
+    path = get_output_path(metadata, conda_config)
+
     # gets the next build number
     build_number, _ = next_build_number(channels[0], os.path.basename(path))
 

bob/devtools/scripts/rebuild.py

@@ -135,18 +135,14 @@ def rebuild(recipe_dir, python, condarc, config, append_file,
     # pre-renders the recipe - figures out the destination
     metadata = get_rendered_metadata(d, conda_config)
 
-    rendered_recipe = get_parsed_recipe(metadata)
-
-    path = get_output_path(metadata, conda_config)
-
     # checks if we should actually build this recipe
     if should_skip_build(metadata):
-      logger.info('Skipping UNSUPPORTED build of %s-%s-py%s for %s',
-          rendered_recipe['package']['name'],
-          rendered_recipe['package']['version'], python.replace('.',''),
-          arch)
+      logger.info('Skipping UNSUPPORTED build of %s for %s', recipe_dir, arch)
       continue
 
+    rendered_recipe = get_parsed_recipe(metadata)
+    path = get_output_path(metadata, conda_config)
+
     # Get the latest build number
     build_number, existing = next_build_number(channels[0],
         os.path.basename(path))