Print specific error message for an invalid package

src/rosdep2/main.py

@@ -73,6 +73,7 @@
 from .catkin_packages import find_catkin_packages_in
 from .catkin_packages import set_workspace_packages
 from .catkin_packages import get_workspace_packages
+from catkin_pkg.package import InvalidPackage
 
 
 class UsageError(Exception):
@@ -175,6 +176,9 @@ def rosdep_main(args=None):
     except UnsupportedOs as e:
         print('Unsupported OS: %s\nSupported OSes are [%s]' % (e.args[0], ', '.join(e.args[1])), file=sys.stderr)
         sys.exit(1)
+    except InvalidPackage as e:
+        print(str(e))
+        sys.exit(1)
     except Exception as e:
         print("""
 ERROR: Rosdep experienced an error: %s

test/main/invalid_package_version/package.xml

@@ -0,0 +1,8 @@
+<?xml version="1.0"?>
+<package>
+  <name>foo</name>
+  <version>:{version}</version>
+  <description>Package Foo</description>
+  <license>BSD</license>
+  <maintainer email="[email protected]">Foo Bar</maintainer>
+</package>

test/test_rosdep_main.py

@@ -275,3 +275,15 @@ def test_proxy_detection(self, proxy, bah, build, install):
         with patch.dict('os.environ', {'https_proxy': 'somethings'}, clear=True):
             setup_proxy_opener()
             proxy.assert_called_with({'https': 'somethings'})
+
+    def test_invalid_package_message(self):
+        with fakeout() as b:
+            test_package_dir = os.path.abspath(os.path.join(get_test_dir(), 'main', 'invalid_package_version'))
+            with patch('rosdep2.main.sys.exit') as exit_mock:
+                rosdep_main(['install', '--from-path', test_package_dir])
+                exit_mock.assert_called_with(1)
+            stdout, stderr = b
+            output = stdout.getvalue().splitlines()
+            assert len(output) == 2
+            assert test_package_dir in output[0]
+            assert 'Package version ":{version}" does not follow version conventions' in output[1]