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

noetic release #136

Merged
merged 6 commits into from
Apr 1, 2020
Merged

Conversation

ahcorde
Copy link

@ahcorde ahcorde commented Mar 26, 2020

  • Python3 changes

  • Use setuptools instead of distutils

Since ros/catkin#1048 catkin prefers to use setuptools instead of distutils. The package.xml doesn't need to include python3-setuptools because catkin exports that dependency for the convenience of all downstream python packages.

  • Bump CMake version to avoid CMP0048

This bumps the minimum CMake version to 3.0.2, which is the minimum supported by ROS Kinetic and new enough to default to the NEW behavior of CMP0048. This avoids a CMake warning when building and testing this package in Debian Buster and Ubuntu Focal.

Signed-off-by: ahcorde <ahcorde@gmail.com>
Signed-off-by: ahcorde <ahcorde@gmail.com>
Signed-off-by: ahcorde <ahcorde@gmail.com>
@sloretz
Copy link

sloretz commented Mar 26, 2020

@ahcorde mind adding a source entry for diagnostics to the Noetic distribution.yaml with test_pull_requests: true? That would allow checking this PR with the ROS 1 build farm

example for a different repo: https://github.com/ros/rosdistro/blob/6dc2f64e9d0242d040a9002f644f352a0a66b19e/noetic/distribution.yaml#L128-L133

@sloretz
Copy link

sloretz commented Mar 27, 2020

@ros-pull-request-builder retest this please

@ahcorde
Copy link
Author

ahcorde commented Mar 31, 2020

I have some issue with the test called discard_stale_not_published_test:

  File "/home/ahcorde/ros_ws/src/diagnostics/diagnostic_aggregator/test/discard_stale_not_published_test.py", line 99, in test_discard_stale
    self.assert_(len(self._agg_expecteds) == 1

@ahcorde
Copy link
Author

ahcorde commented Apr 1, 2020

@ros-pull-request-builder retest this please

diagnostic_aggregator/CHANGELOG.rst Outdated Show resolved Hide resolved
diagnostic_analysis/CHANGELOG.rst Outdated Show resolved Hide resolved
diagnostic_analysis/src/diagnostic_analysis/sparse.py Outdated Show resolved Hide resolved
diagnostic_common_diagnostics/CHANGELOG.rst Outdated Show resolved Hide resolved
diagnostic_updater/CHANGELOG.rst Outdated Show resolved Hide resolved
diagnostics/CHANGELOG.rst Outdated Show resolved Hide resolved
rosdiagnostic/CHANGELOG.rst Outdated Show resolved Hide resolved
self_test/CHANGELOG.rst Outdated Show resolved Hide resolved
test_diagnostic_aggregator/CHANGELOG.rst Outdated Show resolved Hide resolved
Signed-off-by: ahcorde <ahcorde@gmail.com>
Signed-off-by: ahcorde <ahcorde@gmail.com>
Signed-off-by: ahcorde <ahcorde@gmail.com>
@ahcorde
Copy link
Author

ahcorde commented Apr 1, 2020

thank you for the review @sloretz

Copy link

@sloretz sloretz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@trainman419 friendly ping :) (or maybe @Karsten1987) mind taking a look and making a ROS Noetic release? This is one of the big repos to get released; it's currently blocking 148 other repos.

@Karsten1987
Copy link
Contributor

I am trusting @sloretz review :)

@Karsten1987
Copy link
Contributor

I assume the travis CI failures can be addressed in a follow up PR

@Karsten1987 Karsten1987 merged commit dc7a4a6 into ros:noetic-devel Apr 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants