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

Macos #93

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Macos #93

wants to merge 2 commits into from

Conversation

D-Alex
Copy link
Member

@D-Alex D-Alex commented Feb 19, 2016

There are still a lot of rough edges which have to be sorted out before rock is running on mac os out of the box.

  • linking against typelib_ruby fails because on mac os it is not allowed to link against modules. In any case I think it would be more straight forward to install typelib also in the lib folder as shared library - any objections?

@@ -64,7 +67,14 @@ struct LocalTaskContext : public RTT::TaskContext

boost::int32_t __orogen_getTID() const
{
#ifdef __APPLE__
std::string thread = boost::lexical_cast<std::string>(boost::this_thread::get_id());
Copy link
Member

Choose a reason for hiding this comment

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

Shouldn't we simply do that for everyone ? Does that give the same result than SYS_gettid on Linux ?

Copy link
Member Author

Choose a reason for hiding this comment

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

I will check if this is giving the same result on linux systems

Alexander Duda added 2 commits April 2, 2016 14:44
SYS_gettid is not available on mac os
rtt plays around with the visibility flags and does not export
the rtt corba exception classes. The result is that all rtt corba
exceptions are only seen as CORBA::UserException in the client
library.
@doudou
Copy link
Member

doudou commented Apr 2, 2016

FYI, I've merged the two obvious commits and rebased

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.

2 participants