Skip to content
This repository was archived by the owner on May 31, 2025. It is now read-only.

fix passing multiple args to add_rostest#791

Merged
dirk-thomas merged 1 commit intokinetic-develfrom
fix_multiple_args_add_rostest
Apr 12, 2016
Merged

fix passing multiple args to add_rostest#791
dirk-thomas merged 1 commit intokinetic-develfrom
fix_multiple_args_add_rostest

Conversation

@dirk-thomas
Copy link
Copy Markdown
Member

Fixes #790.

@efernandez
Copy link
Copy Markdown
Contributor

👍 Exactly the fix I was thinking of.

I you give me a few minutes, I can checkout the code and give it a try. Not sure if the tests already check that.

BTW, any chance this could get backported to indigo?

@dirk-thomas
Copy link
Copy Markdown
Member Author

Absolutely, please give it a try and yes, it will be backported since it is a clear bug fix.

@efernandez
Copy link
Copy Markdown
Contributor

@dirk-thomas How can I test it? I've just checkout the repo on my workspace, but it still uses the system version.

@dirk-thomas
Copy link
Copy Markdown
Member Author

After cloning this branch into your workspace and building it other packages in the workspace will use the rostest package from the workspace hence the custom version.

@efernandez
Copy link
Copy Markdown
Contributor

Ok, let me catkin clean && catkin build then (not just catkin build).

@efernandez
Copy link
Copy Markdown
Contributor

Confirmed. It works.

I tested directly with this branch on indigo, which works fine. I guess you'll cherry-pick the commit from this PR into indigo. I guess it'll work too, but please tag me on the PR against indigo if you want me to test it again.

@dirk-thomas dirk-thomas merged commit bf39f5c into kinetic-devel Apr 12, 2016
@dirk-thomas dirk-thomas deleted the fix_multiple_args_add_rostest branch April 12, 2016 17:46
@dirk-thomas
Copy link
Copy Markdown
Member Author

Thanks for testing it.

Since the patch is straight forward I just cherry-picked it to indigo-devel: 7f251de

@efernandez
Copy link
Copy Markdown
Contributor

Thanks for the extremely fast fix.

@efernandez
Copy link
Copy Markdown
Contributor

@dirk-thomas When would the indigo-devel commit get into the next release? :)

@dirk-thomas
Copy link
Copy Markdown
Member Author

While the previous release is in shadow-fixed there are a couple of regressions. So I will likely create a new release in the next days and after a shorter soak time it will be synced (hopefully 😉).

rsinnet pushed a commit to MisoRobotics/ros_comm that referenced this pull request Jun 19, 2017
fix passing multiple args to add_rostest
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants