-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
wet orocos_kdl removed from groovy #4792
Comments
ping @tfoote who made the problematic commit |
Those packages are in a problem spot as they need catkinized dependencies but they don't actually exist. They happened to be building by luck of side effects. I think the only real solution is to roll back to non catkin versions of those packages. |
Is there a simple way to list all wet groovy packages that have (recursive) dry dependencies? As it stands it seems groovy can't be built from source (at least desktop-full), so I don't think removing the packages will make it much worse. Are the current debians of those offending packages built from the dry ROS packages? |
No the debian builds slip through a crack since the packaging is mostly 3rdparty and the interface is the same for downstream packages and with the debian packages it only relies on the debian package being installed. And the dry or wet ones are close enough to work. |
Not sure I understand. If all wet packages depending on |
There's only one version of anything in the distro wet or dry. If you remove the wet ones, they're gone. |
So what is the solution here? bringing back wet As it is, groovy cannot be built properly from source. |
All wet packages depending on it need to be rolled back to their dry versions and re-released. Or we add instructions to inject orocos_kdl early such that they build through the same loop as the debian packages. |
I'm not sure how to do neither of those things, so I'm out. |
I guess if we ever expect to fix this, now would be the time before the last groovy release. It seems it is still impossible to build groovy from source with the standard instructions. Unfortunately I don't know how to fix this. |
Yeah, I said I would take a look at this, but I never had time. @tfoote I'm a little fuzzy on the history of this, why can't we go back to the wet
|
One problem with experimenting with a fix for this is that it will probably break new packages in Groovy. We could wait for the final community sync of Groovy, and after I can try to fix this, addressing the fallout without interaction with the community. This way we don't disrupt the current effort to get Groovy debs in a non-regression state, and later I can try to fix it without maintainers getting emailed and what not. |
So I haven't had time to look into this. Also the Groovy build farm jobs are down now, so I won't be able to make the grand unified fix to this, but since the binary debs are not going to be generated anymore, I can now do what I want/need to the release files to make the from source work without trying to make the debs work. I'll try to spend some time this weekend building Groovy on my laptop and fixing this issue for the source case. It maybe that I can get it into a working state for from source and binaries now that the farm is shutdown for Groovy. |
Ok, I restored the wet entry for Now I think there still maybe a problem if you mix in some dry packages which use Final steps may include removing the dry entry for I'm going to close this for now, to any future readers if you run into a problem which you think is related to this please speak up in a comment here. |
d422d53 removed wet
orocos_kdl
from groovy, however some other wet packages indesktop-full
depend on it. In particular after populating a workspace withrosinstall_generator --rosdistro groovy desktop --wet-only --tar --deps
and then runningrosdep --rosdistro groovy check --from-paths src --ignore-src
I get:This makes source builds of groovy fail, see for example: http://answers.ros.org/question/169441/installation-groovy-on-fedora-rosdep-error-cannot-locate-rosdep-definition-for-orocos_kdl/?comment=174098
The text was updated successfully, but these errors were encountered: