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.

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

env-hooks: install env-hook to etc/orocos/profile.d #160

Merged
merged 1 commit into from
Dec 22, 2016

Conversation

meyerj
Copy link
Member

@meyerj meyerj commented Jul 15, 2016

The env-hook sets the OROCOS_TARGET environment variable if not set and adds the lib/orocos directory of the install-space and all entries of the CMAKE_PREFIX_PATH (for catkin workspaces) to the
RTT_COMPONENT_PATH.

The scripts in etc/orocos/OROCOS_TARGET/profile.de will be sourced in lexical order by a new setup.sh
script located in the orocos_toolchain repository.

@meyerj meyerj added this to the 2.9 milestone Jul 15, 2016
@meyerj meyerj self-assigned this Jul 15, 2016
@jmachowinski
Copy link
Member

Should this not only be installed, if catkin is present ?

@smits
Copy link
Member

smits commented Aug 10, 2016

lgtm

@meyerj
Copy link
Member Author

meyerj commented Aug 10, 2016

Should this not only be installed, if catkin is present ?

We are working on an alternative light-weight installation procedure including a new setup.sh script to setup the environment in the installation-script branch of orocos_toolchain. The script mimics catkin's env-hooks and allows to setup the environment based on what packages (or hooks) are found in the install-space without depending on Ruby or catkin.

Eventually we could remove the catkin env-hook in favor of the new one that is installed unconditionally. At the moment the same hook is installed at two different locations for backwards compatibility.

We can wait with merging this patch until the installation-script branch has settled. I created a new PR here: orocos-toolchain/orocos_toolchain#13

@meyerj meyerj force-pushed the add-orocos-env-hook branch from 8afc07e to a437e40 Compare November 11, 2016 10:50
The env-hook sets the OROCOS_TARGET environment variable if not set and adds the lib/orocos directory
of the install-space and all entries of the CMAKE_PREFIX_PATH (for catkin workspaces) to the
RTT_COMPONENT_PATH.

The scripts in etc/orocos/OROCOS_TARGET/profile.de will be sourced in lexical order by a new setup.sh
script located in the orocos_toolchain repository.

Signed-off-by: Johannes Meyer <[email protected]>
@meyerj meyerj force-pushed the add-orocos-env-hook branch from a437e40 to 421839c Compare November 11, 2016 13:35
@meyerj meyerj merged commit 98e46a4 into orocos-toolchain:master Dec 22, 2016
@meyerj meyerj deleted the add-orocos-env-hook branch December 22, 2016 10:41
@meyerj meyerj changed the title env-hooks: install env-hook to etc/orocos/OROCOS_TARGET/profile.d env-hooks: install env-hook to etc/orocos/profile.d Oct 25, 2017
meyerj added a commit to meyerj/rtt that referenced this pull request Oct 25, 2017
…ok into rdt-toolchain-2.9-cherry-pick

env-hooks: install env-hook to etc/orocos/OROCOS_TARGET/profile.d
meyerj added a commit to orocos-toolchain/orocos_toolchain that referenced this pull request Jun 21, 2019
…os_toolchain/CMakeLists.txt

The setup scripts will set the required environment variables to run Orocos RTT and other toolchain
packages. The actual env-hooks have been moved into the individual packages that need it:

- orocos-toolchain/rtt#160
- orocos-toolchain/ocl#43
- orocos-toolchain/utilrb@8ac984f#diff-47236f28d8c7e670852c13dcc723ed57
- orocos-toolchain/typelib@d4779e6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants