search

orocos-toolchain / rtt

Orocos Real-Time Toolkit
http://www.orocos.org
Other
72 stars 79 forks source link

Added option, to load typekits on demand #162

Closed jmachowinski closed 8 years ago

jmachowinski commented 8 years ago

This commit adds the option, to load typekits on demand.

This is especially useful, if one uses RTT::corba::TaskContexProxy, as the ports are not reported, if no typekit for them is present.

jmachowinski commented 8 years ago

Any one ? If not I assume this is ok and merge.

meyerj commented 8 years ago

I can see how this could be useful, but the function or functor that implements the auto loading has to be written manually. Do you have an example how you would use this new interface?

jmachowinski commented 8 years ago

A concrete usecase can be found in https://github.com/rock-cpp/rock-display/blob/master/src/Main.cpp There I implemented a functor, that utilizes the fact, that orogen generated typekits provide a list of 'contained' types on disk.

smits commented 8 years ago

It's definitely not related to this PR, but we should investigate the failed build: https://travis-ci.org/orocos-toolchain/rtt/builds/147143467#L2127

lgtm

meyerj commented 8 years ago

It's definitely not related to this PR, but we should investigate the failed build: https://travis-ci.org/orocos-toolchain/rtt/builds/147143467#L2127

I already pushed a patch to master: https://github.com/orocos-toolchain/rtt/commit/9140db41cb326dbf86fdd8839f9f7df8aa7047bb, but it's not yet included in this PR. I am not completely sure whether it fixes the issue on Travis.

jmachowinski commented 8 years ago

So, merge ?