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

Namespace override with ROS2_NAMESPACE environment variable #993

Open
wants to merge 5 commits into
base: rolling
Choose a base branch
from
Open

Namespace override with ROS2_NAMESPACE environment variable #993

wants to merge 5 commits into from

Commits on Jul 11, 2022

  1. add namespace environment variable support

    @alpylmz
    alpylmz committed Jul 11, 2022
    Configuration menu
    Copy the full SHA
    f87c083 View commit details
    Browse the repository at this point in the history

Commits on Jul 14, 2022

  1. condition typo fix

    @alpylmz
    alpylmz committed Jul 14, 2022
    Configuration menu
    Copy the full SHA
    e03b151 View commit details
    Browse the repository at this point in the history

  2. a test is added for namespace override

    @alpylmz
    alpylmz committed Jul 14, 2022
    Configuration menu
    Copy the full SHA
    ab39576 View commit details
    Browse the repository at this point in the history

  3. environment variable changed from ROS2_NAMESPACE to ROS_NAMESPACE

    @alpylmz
    alpylmz committed Jul 14, 2022
    Configuration menu
    Copy the full SHA
    7de72e2 View commit details
    Browse the repository at this point in the history

  4. const declarations moved close to actual use

    @alpylmz
    alpylmz committed Jul 14, 2022
    Configuration menu
    Copy the full SHA
    c44042f View commit details
    Browse the repository at this point in the history