Skip to content
New issue

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

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

Already on GitHub? # to your account

unrecognized arguments: --no-deps --no-color #689

Closed
arturmiller opened this issue Oct 5, 2021 · 2 comments
Closed

unrecognized arguments: --no-deps --no-color #689

arturmiller opened this issue Oct 5, 2021 · 2 comments

Comments

@arturmiller
Copy link

System Info

  • Operating System: "Linux xxxxx 5.4.0-88-generic #99-Ubuntu SMP Thu Sep 23 17:29:00 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux"
  • Python Version: Python 3.8.10
  • Version of catkin_tools: 0.7.1
  • ROS Distro: noetic

I installed the most recent master branch with:

pip3 install -U "git+https://github.com/catkin/catkin_tools.git#egg=catkin_tools"

Expected Behavior

When running tests with catkin_tools, it should be able to find the arguments "--no-deps" and "--no-color".

Actual Behavior

When running the tests with catkin_tools, the arguments "--no-deps" and "--no-color" are not found anymore. I get following error message:

catkin: error: unrecognized arguments: --no-deps --no-color

Steps to Reproduce the Issue

Just run following command with any package:

catkin run_tests <any_package> --no-deps --no-color

I saw, that recently there was a bigger change regarding running tests with catkin_tools. I think this might be a regression because of this change. Maybe it is now intended to use these arguments in a different way, but the documentation doesn't say anything about it.

@arturmiller
Copy link
Author

I saw that in the PR #676 in fact the documentation has been updated. It looks like, that it did not propagate to catkin-tools.readthedocs.io, yet. The new documentation does not mention this arguments anymore. Maybe "--no-deps" is now default?
I found a workaround with intended behavior and similar (same?) output:

catkin build <any_package> --no-deps --no-color --interleave-output --catkin-make-args run_tests

@timonegk
Copy link
Member

To replicate the old behavior, you can indeed use the command that you found. However, I would recommend to use the new catkin test verb that can be used like this: catkin test <any_package> --interleave-output. It runs the same tests as run_tests but has a more useful output.
--no-deps became the default for catkin test. I added the --no-deps and --no-color arguments for backwards compatibility in #693, thank you for pointing out the regression.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants