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] do not specify c++ standard #1770

Closed
wants to merge 5 commits into from
Closed

Conversation

v4hn
Copy link
Contributor

@v4hn v4hn commented Oct 4, 2022

it break with current log4cxx which requires c++17. No, going forward it's no option to specify the standard anymore. And it's not necessary either.

@k-okada

it break with current log4cxx which requires c++17.
No, going forward it's no option to specify the standard anymore.
And it's not necessary either.
the non-hpp headers are deprecated since kinetic
it's deprecated to use global _1 from boost for a while now
and the Debian ROS packages do not provide it anymore.
this confused me when I looked through the code.
setup from distutils is deprecated and will be removed eventually.
@v4hn
Copy link
Contributor Author

v4hn commented Nov 9, 2022

I added multiple other patches needed to compile on debian testing.

k-okada added a commit to k-okada/jsk_common that referenced this pull request Dec 8, 2022
k-okada added a commit to k-okada/jsk_common that referenced this pull request Dec 8, 2022
k-okada added a commit to k-okada/jsk_common that referenced this pull request Dec 8, 2022
k-okada added a commit to k-okada/jsk_common that referenced this pull request Dec 8, 2022
k-okada added a commit to k-okada/jsk_common that referenced this pull request Dec 8, 2022
k-okada added a commit to k-okada/jsk_common that referenced this pull request Dec 8, 2022
k-okada added a commit to k-okada/jsk_common that referenced this pull request Dec 8, 2022
k-okada added a commit to k-okada/jsk_common that referenced this pull request Dec 8, 2022
k-okada added a commit to k-okada/jsk_common that referenced this pull request Dec 8, 2022
k-okada added a commit to k-okada/jsk_common that referenced this pull request Dec 8, 2022
k-okada added a commit to k-okada/jsk_common that referenced this pull request Dec 9, 2022
k-okada added a commit to k-okada/jsk_common that referenced this pull request Dec 9, 2022
k-okada added a commit to k-okada/jsk_common that referenced this pull request Dec 9, 2022
k-okada added a commit to k-okada/jsk_common that referenced this pull request Dec 9, 2022
k-okada added a commit to k-okada/jsk_common that referenced this pull request Dec 9, 2022
k-okada added a commit to k-okada/jsk_common that referenced this pull request Dec 9, 2022
k-okada added a commit to k-okada/jsk_common that referenced this pull request Dec 10, 2022
k-okada added a commit to k-okada/jsk_common that referenced this pull request Dec 10, 2022
k-okada added a commit to k-okada/jsk_common that referenced this pull request Dec 10, 2022
k-okada added a commit to k-okada/jsk_common that referenced this pull request Dec 10, 2022
k-okada added a commit to k-okada/jsk_common that referenced this pull request Dec 12, 2022
@k-okada
Copy link
Member

k-okada commented Dec 13, 2022

@v4hn thanks for patches, i have createned new PR #1773 with CI test as well as backward compatibilies. BTW, how did you setup ROS-O build environment, for example are you compile all ros tools from source? or are you using Debian-installed libraries?

here is my settings ->
https://github.com/jsk-ros-pkg/jsk_common/pull/1773/files#diff-33cc4c925f7fd36575f5e5b61d1e9c942fea5189e2c67d09720d714e19151404

@v4hn
Copy link
Contributor Author

v4hn commented Dec 13, 2022

Great work!
Please watch my roscon talk linked from https://github.com/ros-o/ros-o if you didn't already.
There are three orthogonal ways to build with different pros/cons each.
CI testing against RoboStack and Debian can both make sense. Debian is more likely to fail as they include more upstream library changes.
Your approach is pretty much how to build on the Debian packages at the moment.
We still need a simple interface to overlay entries from rosinstall_generator with versions we patched in the ros-o organization because many patches are not merged upstream anymore (definitely not in a timely fashion)...
The core issue is that someone should write a tool to automatically extract a rosdistro file based on active ros-o repositories and hook it up with rosinstall_generator. That would help a lot.
Maintaining four different package origins (packaged, ros-gbp, upstream noetic-devel, ros-o) for a workspace is of course a burden, so I would personally go for an approach that does not use noetic upstream-develop at all, but only releases and ros-o branches of overlays exist and are up2date.
That also works better with the envisioned RoboStack integration of ros-o, where they don't consider upstream-devel at all.

@v4hn
Copy link
Contributor Author

v4hn commented Dec 13, 2022

That being said @jspricke worked on an approach recently to build debs directly from github actions and that might move more and more packages into debian and a custom deb repository in the future.

@k-okada
Copy link
Member

k-okada commented Dec 14, 2022

@v4hn thanks for explanation and I found that I missed your great talk at ROScon.
Integrating rosinstall_generator with ros-o and also deb build make sense to me too. Where should we discussion these isseus or find the current development status? https://github.com/ros-o/ros-o/discussions ??
I am also considering if we could contribute to provide computers to host deb files.

@v4hn
Copy link
Contributor Author

v4hn commented Dec 14, 2022

Please open a discussion there, yes. I can ping a few other involved people if there is something concrete to discuss there.

k-okada added a commit that referenced this pull request Dec 15, 2022
add test to compile on 22.04, see #1770
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.

None yet

2 participants