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

[Indigo] cherrypick print message improvement - #994

Closed
130s wants to merge 2 commits into
ros:indigo-develfrom
130s:i/cherrypick992_989
Closed

[Indigo] cherrypick print message improvement#994
130s wants to merge 2 commits into
ros:indigo-develfrom
130s:i/cherrypick992_989

Conversation

@130s

@130s 130s commented Feb 18, 2017

Copy link
Copy Markdown
Member

#989, #992 into indigo-devel.

130s and others added 2 commits February 18, 2017 09:58
… confusing.

For example, [this warning msg](https://travis-ci.org/wg-perception/people/jobs/202019288#L3737) (in this [PR](wg-perception/people#49)):

 ```
 * WARN: unrecognized 'group' tag in <node> tag. Node xml is <test name="$(arg testnode_name)" pkg="rostest" test-name="hztest_test" type="hztest">
    <group unless="$(arg expected_success)">
      <node args="-r 0.5 $(find face_detector)/test/face_detector_noface_test_diamondback.bag" name="play" pkg="rosbag" type="play"/>
      <param name="hz" value="0.0"/>
    </group>
```

This is confusing because the tag in question is actually `<test>`, not `<node>`. Although the warning message refers to the exact <test> tag, I didn't try to read it because I was told the issue is with <node>, not <test>.

With this PR the message becomes a bit verbose but hope the readers get better idea.
@dirk-thomas

Copy link
Copy Markdown
Member

Replaced by #1008.

@dirk-thomas dirk-thomas closed this Mar 2, 2017
@130s
130s deleted the i/cherrypick992_989 branch March 2, 2017 19:17
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