Conversation
Don't depend on specific openssl version, as this could conflict with an already installed gnutls version.
|
@jspricke I couldn't find libcurl-dev on either package.debian.org or packages.ubuntu.com, could you clarify where you found those packages names ? |
|
oh interesting that the search doesn't return virtual packages o_O. Thanks for the links. |
I actually couldn't make it work easily while trying it. I will leave it up to @tfoote to try / confirm / approve / merge.
|
It does look like this is a problem. I think we resolved the virtual package checks but it doesn't appear that we've resolved the install side issues: This is what rosdep would have run: I tweaked it to use the new rule and it failed to install. We've previously tried the same: #57 but had to revert #58 It looks like we haven't resolved being able to install. I think ros-infrastructure/rosdep#274 needs to be reopened wrt to installing. |
|
I have implemented support for this in ros-infrastructure/rosdep#521 and tested it with this PR: https://gist.github.com/ipa-mdl/401f74bff9bfe64fd359c4593c4e3797 |
|
What is the status of this PR? |
|
Works with ros-infrastructure/rosdep#585 |
|
rosdep should now handle this properly. |
|
@tfoote Cas you please give this another try with the |
tfoote
left a comment
There was a problem hiding this comment.
With the new rosdep this appears to be working correctly.
https://gist.github.com/tfoote/e84c8d7c3f34515df9fa75433d34a201
|
Could this change be the reason for the following job starting to fail? http://build.ros.org/view/Mdev/job/Mdev__rc_visard__ubuntu_bionic_amd64/4/console |
|
Yes, that logic block is going to need to become aware of the virtual package logic as it looks to be implementing it in parallel. |
|
What is the status on this? |
|
I went ahead and reverted the change in #18223. |
Don't depend on specific openssl version, as this could conflict with an
already installed gnutls version.