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

[ROS-O] drop obsolete use of boost::bind and _1 #167

Open
wants to merge 3 commits into
base: ros1
Choose a base branch
from

Conversation

v4hn
Copy link

@v4hn v4hn commented Aug 20, 2024

Lambdas are preferred since c++11.
The transitively included use of _1 starts failing with a recently proposed change to adapt the boost-bind include API in ros_comm. ros-o/ros_comm#3

v4hn added 3 commits August 21, 2024 00:51
Lambdas are preferred since c++11.
The transitively included use of _1 starts failing with a recently
proposed change to adapt the boost-bind include API in ros_comm.
ros-o/ros_comm#3
default almost everywhere and log4cxx breaks without 17 support for
years.
... where there is no package.
Only a script was installed from the setup.py
but catkin provides a separate command for that
http://docs.ros.org/en/jade/api/catkin/html/howto/format2/installing_python.html

I retained the installation to the global path. It is unusual to do
that, but it has been like that for a long time.

This started failing with recent setuptools with the following error
message:

---
error: Multiple top-level packages discovered in a flat-layout: ['debian', 'config', 'launch'].

To avoid accidental inclusion of unwanted files or directories,
setuptools will not proceed with this build.

If you are trying to create a single distribution with multiple packages
on purpose, you should not rely on automatic discovery.
Instead, consider the following options:

1. set up custom discovery (`find` directive with `include` or `exclude`)
2. use a `src-layout`
3. explicitly set `py_modules` or `packages` with a list of names

To find more information, look for "package discovery" on setuptools docs.
---
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.

1 participant