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

Remove signals from find_package(Boost COMPONENTS ...) #1580

Merged

Conversation

de-vri-es
Copy link
Contributor

The packages here use signals2, not signals. Only boost libraries with compiled code should be passed to find_package(Boost COMPONENTS ...), and the signals2 library has always been header only.

Boost 1.69 has removed the deprecated signals library, so the otherwise useless but harmless signals component now breaks the build.

The packages use signals2, not signals. Only boost libraries with
compiled code should be passed to find_package(Boost COMPONENTS ...),
and the signals2 library has always been header only.

Boost 1.69 has removed the deprecated signals library, so the otherwise
useless but harmless `signals` component now breaks the build.
@de-vri-es
Copy link
Contributor Author

I doubt the test failure is related to this PR. Output of the failed test:

max time [60.0s] allotted for test [sub_to_multiple_pubs] of type [test_rospy/test_sub_to_multiple_pubs.py]
  File "/usr/lib/python2.7/unittest/case.py", line 329, in run
    testMethod()
  File "/tmp/ws/src/ros_comm/tools/rostest/src/rostest/runner.py", line 148, in fn
    self.test_parent.run_test(test)
  File "/tmp/ws/src/ros_comm/tools/rostest/src/rostest/rostest_parent.py", line 132, in run_test
    return self.runner.run_test(test)
  File "/tmp/ws/src/ros_comm/tools/roslaunch/src/roslaunch/launch.py", line 689, in run_test
    (test.time_limit, test.test_name, test.package, test.type))

It looks like a slow test that exceeded the allowed running time.

@lsr0
Copy link

lsr0 commented Jan 25, 2019

This issue is now breaking the (unmodified) Arch linux packages, can only assume more places to follow.

@de-vri-es de-vri-es closed this Jan 25, 2019
@de-vri-es de-vri-es reopened this Jan 25, 2019
@de-vri-es
Copy link
Contributor Author

Closed and re-opened PR to re-trigger CI to see if the flaky test will pass on it's own.

@de-vri-es de-vri-es closed this Jan 25, 2019
@de-vri-es de-vri-es reopened this Jan 25, 2019
@de-vri-es
Copy link
Contributor Author

I guess not

@dirk-thomas
Copy link
Member

Thanks for the patch.

The CI failure is due to a test known to be flaky - so ignoring it.

@dirk-thomas dirk-thomas merged commit 41f18ed into ros:melodic-devel Jan 30, 2019
@de-vri-es
Copy link
Contributor Author

Thanks!

@bionade24
Copy link

When does a new release of roscpp with this patch potentially come? I don't want to write a patch for AUR, upstream should be fixed.

@dirk-thomas
Copy link
Member

Whenever I manage to get through the backlog. Hopefully February (no mention of the year though 🤣)...

tahsinkose pushed a commit to tahsinkose/ros_comm that referenced this pull request Apr 15, 2019
The packages use signals2, not signals. Only boost libraries with
compiled code should be passed to find_package(Boost COMPONENTS ...),
and the signals2 library has always been header only.

Boost 1.69 has removed the deprecated signals library, so the otherwise
useless but harmless `signals` component now breaks the build.
@bionade24
Copy link

There is still no release with this patch. When does it come?

myungjoo pushed a commit to myungjoo/tizenport-ros-comm that referenced this pull request May 13, 2020
The packages use signals2, not signals. Only boost libraries with
compiled code should be passed to find_package(Boost COMPONENTS ...),
and the signals2 library has always been header only.

Boost 1.69 has removed the deprecated signals library, so the otherwise
useless but harmless `signals` component now breaks the build.

Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>
myungjoo pushed a commit to nnsuite/tizenport-ros-comm that referenced this pull request May 14, 2020
The packages use signals2, not signals. Only boost libraries with
compiled code should be passed to find_package(Boost COMPONENTS ...),
and the signals2 library has always been header only.

Boost 1.69 has removed the deprecated signals library, so the otherwise
useless but harmless `signals` component now breaks the build.

Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>
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.

4 participants