Merge pull request #74 from jdlangs/fix-73

Only overwrite `sys.argv` if we have ARGS

Author: jdlangs

.travis.yml

@@ -1,8 +1,7 @@
 language: julia
 julia:
-  - 0.7
   - 1.0
-  - 1.1
+  - 1.2
   - nightly
 matrix:
   allow_failures:

Project.toml

@@ -7,7 +7,7 @@ PyCall = "438e738f-606a-5dbb-bf0a-cddfbfd45ab0"
 
 [compat]
 PyCall = "≥ 1.90.0"
-julia = "≥ 0.7.0"
+julia = "≥ 1"
 
 [extras]
 Test = "8dfed614-e22c-5e08-85e1-65c5234f0b40"

src/RobotOS.jl

@@ -18,7 +18,9 @@ include("callbacks.jl")
 function __init__()
     #Put julia's ARGS into python's so remappings will work
     copy!(_py_sys, pyimport("sys"))
-    _py_sys."argv" = ARGS
+    if length(ARGS) > 0
+        _py_sys.argv = ARGS
+    end
 
     #Fill in empty PyObjects
     if ! (dirname(@__FILE__) in _py_sys."path")

test/rospy.jl

@@ -1,3 +1,5 @@
+import PyCall
+
 #Test basic rospy interactions
 init_node("jltest", anonymous=true)
 
@@ -32,3 +34,9 @@ RobotOS.get_node_uri()
 RobotOS.get_caller_id()
 RobotOS.get_published_topics()
 RobotOS.get_ros_root()
+
+#Issue 73 - Corruption of Python sys.argv
+PyCall.py"""
+import argparse
+argparse.ArgumentParser()
+"""