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.

Sign up for GitHub

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

Jump to bottom

Extension of #481 #499

Merged
merged 2 commits into from
Apr 28, 2017
Merged

Extension of #481 #499

merged 2 commits into from
Apr 28, 2017

Conversation

wjwwood
Copy link
Contributor

@wjwwood wjwwood commented Feb 9, 2017

I would have pushed directly to #481, but I didn't have permission.

In this extension I:

  • restored --version to the current behavior
  • added --all-versions that implements the new behavior
  • cleaned up the output formatting
  • changed the function name to get_version_strings and updated docs to better indicate support for multiple version strings per installer
  • added support for Homebrew and Macports

This is what the output looks like on my machine now:

william@farl ~
% rosdep --version
0.11.5

william@farl ~
% rosdep --all-versions
0.11.5

Versions of installers:
  Homebrew 1.1.6-45-g9cce341
  Homebrew/homebrew-core (git revision 9c73d; last commit 2017-01-09)
  Macports 2.4.0
  pip 9.0.1
  setuptools 32.3.1

wkentaro and others added 2 commits October 24, 2016 17:08
@wkentaro
Show installers version with --version option
6d4d73c 
Specifically, pip and setuptools version is crucial because
pip distributed on apt is very old on Ubuntu 14.04.
I think showing the version of installer helps user to resolve
their issue caused by old package managing tools.

Usage:

  $ rosdep --version
  gem version unknown
  source version unknown
  rosdep version 0.11.5
  apt-get version 1.0.1ubuntu2
  pip version 8.1.2
  setuptools version 28.6.1
@wjwwood
add option --all-versions
ff76050 
other fix ups as well and added Homebrew/Macports
@wjwwood wjwwood mentioned this pull request Feb 9, 2017
Closed
@wkentaro
Copy link
Contributor

+1

@wjwwood wjwwood merged commit 8c276a4 into master Apr 28, 2017
@wjwwood wjwwood deleted the wkentaro-show-installers-version branch April 28, 2017 06:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@wjwwood @wkentaro