Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

rosunit: uses python2 print statement, which fails with python3 #11

Closed
jack-oquin opened this issue Mar 31, 2013 · 1 comment

Comments

Projects
None yet
2 participants
@jack-oquin
Copy link
Member

commented Mar 31, 2013

There are numerous uses of the python2 print statement in rosunit.

It should use the new print() function with from __future__ import print_function for python2 compatibility.

This needs to be fixed in hydro-devel.

@jack-oquin

This comment has been minimized.

Copy link
Member Author

commented Mar 31, 2013

I have a patch for this, pull request to follow.

dirk-thomas added a commit that referenced this issue Apr 9, 2013

Merge pull request #13 from jack-oquin/hydro-devel
make rosunit use print_function for python 2 and 3 compatibility (#11)

@dirk-thomas dirk-thomas closed this Apr 9, 2013

johnsonshih added a commit to johnsonshih/ros that referenced this issue Sep 26, 2018

dirk-thomas added a commit that referenced this issue Mar 5, 2019

add rosrun.bat (#208)
* enable rosrun on Windows

* rosrun_fix

* fix indent

* revert unnecessary changes

* Fix libexec path search issue.

* another round of rosrun fix.

* Fix File Not Found messages.

* fix parameter processing

* Rosrun.bat fix (#10)

* Check if file exists and return correct error code

* Generate toplevel forward bat script for rosrun on Windows

* Fix issue that rosrun cannot handle wild char parameter (#11)

* Fix typo (#12)

* Fix rosrun.

* fix rosrun parameter parsing issue (#14)

* fix rosrun parameter parsing issue

* correct file extension extraction

* Fix rosrun bash script (#18)

Fix rosrun bash script

* update rosrun.bat to find python scripts (files with extensions) (#20)

This change makes it so it does a deep search first in the rospack file, then the whole path.

* fix rosrun search path pecking order. (#24)

* fix.
* fix.

* update catkin version requirement for script.bat.in
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.