Fix broken kerserver url by replacing with wget#529
Merged
ahcorde merged 1 commit intogazebosim:gz-cmake5from Feb 26, 2026
Merged
Fix broken kerserver url by replacing with wget#529ahcorde merged 1 commit intogazebosim:gz-cmake5from
ahcorde merged 1 commit intogazebosim:gz-cmake5from
Conversation
ahcorde
requested changes
Feb 26, 2026
Contributor
ahcorde
left a comment
There was a problem hiding this comment.
Do you mind to sign the commit https://github.com/gazebosim/gz-cmake/pull/529/checks?check_run_id=64992631652 ?
Replaced the keyserver url in install.md for gz-cmake with the wget from the packages website of osrf fixes gazebosim#470 Signed-off-by: BhuvanB <bhuvanb1408@gmail.com>
4a1605b to
f02815a
Compare
Contributor
Author
|
my bad,fixed |
ahcorde
approved these changes
Feb 26, 2026
Member
|
we should copy the installation approach from https://github.com/gazebosim/docs/blob/e95648edda66cbf83a04ae568b3a6778fb7b96c1/jetty/install_ubuntu.md?plain=1#L18-L19 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🦟 Bug fix
Fixes #470
Summary
Replaced the keyserver url in install.md for gz-cmake with the wget , removing the broken link.
Current install instructions provide the link for keyserver for downloading binaries through this link :
which results in the output :
so updated it to
as suggested and used in source install section.
Though it does generate warning that apt-get is depreciated. So should those warning be removed using gpg signoff ?
Checklist
codecheckpassed (See contributing)