Package: ros-rospkg / 1.2.9-1

0001-Set-default-ROS_PACKAGE_PATH-to-usr-share.patch Patch series | download
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
From: Jochen Sprickerhof <git@jochen.sprickerhof.de>
Date: Wed, 8 Apr 2015 18:01:46 +0200
Subject: Set default ROS_PACKAGE_PATH to /usr/share

---
 src/rospkg/environment.py       | 2 +-
 test/test_rospkg_environment.py | 6 +++---
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/rospkg/environment.py b/src/rospkg/environment.py
index e807978..f05b563 100644
--- a/src/rospkg/environment.py
+++ b/src/rospkg/environment.py
@@ -114,7 +114,7 @@ def get_ros_package_path(env=None):
     """
     if env is None:
         env = os.environ
-    return env.get(ROS_PACKAGE_PATH, None)
+    return env.get(ROS_PACKAGE_PATH, '') + ':/usr/share'
 
 
 def get_ros_home(env=None):
diff --git a/test/test_rospkg_environment.py b/test/test_rospkg_environment.py
index 2e44ce0..5512814 100644
--- a/test/test_rospkg_environment.py
+++ b/test/test_rospkg_environment.py
@@ -52,13 +52,13 @@ def test_get_ros_root():
 
 def test_get_ros_package_path():
     from rospkg import get_ros_package_path
-    assert get_ros_package_path(env={}) is None
+    assert ':/usr/share' == get_ros_package_path(env={})
     env = {'ROS_PACKAGE_PATH': ':'}
-    assert ':' == get_ros_package_path(env=env)
+    assert '::/usr/share' == get_ros_package_path(env=env)
 
     # trip-wire tests. Cannot guarantee that ROS_PACKAGE_PATH is set
     # to valid value on test machine, just make sure logic doesn't crash
-    assert os.environ.get('ROS_PACKAGE_PATH', None) == get_ros_package_path()
+    assert os.environ.get('ROS_PACKAGE_PATH', ':/usr/share') == get_ros_package_path()
 
 
 def test_get_log_dir():