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

Removed repeated installation instructions #160

Merged
merged 3 commits into from
Aug 15, 2020
Merged

Removed repeated installation instructions #160

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
c48be88
Removed repeated installation instructions
ahcorde Aug 13, 2020
5de4346
Updated install link
ahcorde Aug 14, 2020
96d2087
Wording tweak
chapulina Aug 15, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
23 changes: 1 addition & 22 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -59,28 +59,7 @@ combination of custom code and ZeroMQ.

# Install

We recommend following the [Binary Install](#markdown-header-binary-install)
instructions to get up and running as quickly and painlessly as possible.

The [Source Install](#markdown-header-source-install) instructions should be
used if you need the very latest software improvements, you need to modify the
code, or you plan to make a contribution.

## Binary Install

On Ubuntu systems, `apt-get` can be used to install `ignition-transport`:

```
$ sudo apt install libignition-transport<#>-dev
```

Be sure to replace `<#>` with a number value, such as `1` or `2`, depending on
which version you need.

## Source Install

See the [install](https://ignitionrobotics.org/api/transport/8.0/installation.html)
section of the documentation.
See the [installation tutorial](https://ignitionrobotics.org/api/transport/9.0/installation.html).

# Usage

Expand Down
8 changes: 5 additions & 3 deletions tutorials/02_installation.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,13 +25,15 @@ Setup keys:
wget http://packages.osrfoundation.org/gazebo.key -O - | sudo apt-key add -
```

Install Ignition Transport:
Install Ignition Transport, `apt-get` can be used to install `ignition-transport`:

```
sudo apt-get update
sudo apt-get install libignition-transport9-dev
$ sudo apt install libignition-transport<#>-dev
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change don't seems to reflect in installation tutorial like I wondered that should happen.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah that's a good point. The Ignition Transport 9.0 documents shouldn't have been uploaded yet because that version hasn't been released. I ticketed an issue here: #162

These tutorial updates should also be made to the Blueprint and Citadel branches (CC @ahcorde )

```

Be sure to replace `<#>` with a number value, such as `1` or `2`, depending on
which version you need.

## Mac OS X

Ignition Transport and several of its dependencies can be compiled on OS
Expand Down