-
Notifications
You must be signed in to change notification settings - Fork 12
Description
-
ROS Version: Humble
-
OS Version: Ubuntu22.04
-
Python Version: 3.10.12
-
setuptools Version: 75.2.0
Hello, when I build this project from source, some warnings occurred.
Hers is the output of the command colcon build:
Starting >>> launch_system_modes
/home/qiguanxiao/.local/lib/python3.10/site-packages/setuptools/_distutils/dist.py:261: UserWarning: Unknown distribution option: 'tests_require'
warnings.warn(msg)
--- stderr: launch_system_modes
/home/qiguanxiao/.local/lib/python3.10/site-packages/setuptools/_distutils/dist.py:261: UserWarning: Unknown distribution option: 'tests_require'
warnings.warn(msg)
---
Finished <<< launch_system_modes [1.53s]
These warnings show the tests_require option is expired. But the python is the default package with ubuntu 22.04 and setuptools is installed by pip.
And then I use colcon test to test this project, there were also some warnings.
Here is the output about the command colcon test:
Starting >>> launch_system_modes
/home/qiguanxiao/.local/lib/python3.10/site-packages/setuptools/_distutils/dist.py:261: UserWarning: Unknown distribution option: 'tests_require'
warnings.warn(msg)
--- stderr: launch_system_modes
----------------------------------------------------------------------
Ran 0 tests in 0.000s
OK
---
Finished <<< launch_system_modes [0.72s]
It seems like that there were no python testing programs to run and no result file pytest.xml was generated in build directory.
Then I tried to modify the setup.py file in package launch_system_modes and replaced the option tests_require with extras_require.
Finally it can be built and test correctly.
Here is the modified content in setpu.py file:
tests_require=['pytest'], # remove
extras_require={
'test': ['pytest'],
}, # add
But there are some format errors after running colcon test. Here is the result:
Now the python and setuptools version are updated very quickly, could you tell me if it is necessary to replace the tests_require option to adapt to the current environment.
Thanks!