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
gazebo: remove #88466
gazebo: remove #88466
Conversation
I haven't used Gazebo in a long time. Removing these and pulling me off the maintainers list seems fair to me. The others used them more recently, so their input is probably more important. |
Being broken for that long is a pretty good indicator that it should be removed. Hopefully someone looking to get it working again can find the git history to have a starting place. Thank you for taking action on this! |
Ok, agree. |
Thank you all for your answers! @therealpxc: Ok, I'll remove |
To summarize: |
I have working versions of Gazebo in my nixpkgs fork, which I maintain as part of my project for ROS support in Nix. I have begun to upstream some of the dependencies (see #85983), and will hopefully be able to create PRs for the rest over the next few weeks. I'm also willing to take over maintenance of QGroundControl as I use it regularly. |
Sorry I wasn't paying enough attentions to the github notifications and late to the thread. Thank you for taking care of the broken package. Also I am really grateful about @lopsided98 's work, which should work for people who want to use Gazebo or other ROS packages in Nix. |
Motivation for this change
The packages
gazebo{6,7}(-headless)
have had a broken dependencysdformat
since 2016 66b6c2c. They were not marked as broken themselves, but of course they don't build either.Searching for "gazebo" in this repo results in many unrelated issues where they annoyingly turn up as a dependent packages which failed to build.
There are no dependencies for these packages, I think it's safe to remove them.
@pxc
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)