Skip to content
This repository was archived by the owner on May 31, 2025. It is now read-only.

Fixes a compile error on OS X due the symbol MSG_NOSIGNAL not being defined on OS X.#3

Merged
dirk-thomas merged 1 commit intoros:masterfrom
wjwwood:xmlrpcsock_osx_bug
Aug 29, 2012
Merged

Fixes a compile error on OS X due the symbol MSG_NOSIGNAL not being defined on OS X.#3
dirk-thomas merged 1 commit intoros:masterfrom
wjwwood:xmlrpcsock_osx_bug

Conversation

@wjwwood
Copy link
Copy Markdown
Member

@wjwwood wjwwood commented Aug 28, 2012

Fixes a compile error on OS X due the symbol MSG_NOSIGNAL not being defined on OS X.

When I try to compile ros_comm on OS X 10.7.4 I get:

Scanning dependencies of target xmlrpcpp
[ 10%] Building CXX object ros_comm/utilities/xmlrpcpp/CMakeFiles/xmlrpcpp.dir/src/XmlRpcClient.cpp.o
[ 10%] Building CXX object ros_comm/utilities/xmlrpcpp/CMakeFiles/xmlrpcpp.dir/src/XmlRpcDispatch.cpp.o
[ 10%] Building CXX object ros_comm/utilities/xmlrpcpp/CMakeFiles/xmlrpcpp.dir/src/XmlRpcServer.cpp.o
[ 10%] Building CXX object ros_comm/utilities/xmlrpcpp/CMakeFiles/xmlrpcpp.dir/src/XmlRpcServerConnection.cpp.o
[ 10%] Building CXX object ros_comm/utilities/xmlrpcpp/CMakeFiles/xmlrpcpp.dir/src/XmlRpcServerMethod.cpp.o
[ 10%] Building CXX object ros_comm/utilities/xmlrpcpp/CMakeFiles/xmlrpcpp.dir/src/XmlRpcSocket.cpp.o
/Users/william/ros-underlay/ros_comm/utilities/xmlrpcpp/src/XmlRpcSocket.cpp:268:36: error: use of undeclared identifier 'MSG_NOSIGNAL'
    int n = send(fd, sp, nToWrite, MSG_NOSIGNAL);
                                   ^
1 error generated.
make[2]: *** [ros_comm/utilities/xmlrpcpp/CMakeFiles/xmlrpcpp.dir/src/XmlRpcSocket.cpp.o] Error 1
make[1]: *** [ros_comm/utilities/xmlrpcpp/CMakeFiles/xmlrpcpp.dir/all] Error 2
make: *** [all] Error 2

This pull request fixes the problem by defining the symbol MSG_NOSIGNAL on OS X to an appropriate alternative define. See: http://lists.apple.com/archives/macnetworkprog/2002/Dec/msg00091.html

@ghost ghost assigned wjwwood Aug 29, 2012
dirk-thomas added a commit that referenced this pull request Aug 29, 2012
@dirk-thomas
Merge pull request #3 from wjwwood/xmlrpcsock_osx_bug
4a5a5ad 
Fixes a compile error on OS X due the symbol MSG_NOSIGNAL not being defined on OS X.
@dirk-thomas dirk-thomas merged commit 4a5a5ad into ros:master Aug 29, 2012
@NikolausDemmel NikolausDemmel mentioned this pull request Jan 15, 2013
Closed
@dirk-thomas
Copy link
Copy Markdown
Member

dirk-thomas commented Jan 15, 2013

Ref #124. Merged to fuerte-devel in 9cd2b75

@tfoote tfoote mentioned this pull request Jan 31, 2013
Closed
@jasonimercer jasonimercer mentioned this pull request Jul 6, 2015
Closed
tfoote pushed a commit that referenced this pull request Jun 17, 2016
@codebot
Merge pull request #3 from ruffsl/sros-apparmor2
c6a2714 
Adding roslaunch profile
tfoote pushed a commit that referenced this pull request Aug 11, 2016
@ruffsl
Merge pull request #3 from ruffsl/sros_keyserver
dea58c1 
Sros keyserver
contradict pushed a commit to contradict/ros_comm that referenced this pull request Aug 12, 2016
@wjwwood
Merge pull request ros#3 from ros-visualization/merge_robot_monitor
69da72b 
rqt_robot_monitor: initial import from external
ggallagher01 pushed a commit to ggallagher01/ros_comm that referenced this pull request Dec 27, 2016
Merge pull request ros#3 from clearpathrobotics/core_5948
ef69101 
Avoid deleting XmlRpcClient's while they are being still in use on another thread
rsinnet pushed a commit to MisoRobotics/ros_comm that referenced this pull request Jun 19, 2017
@dirk-thomas
Merge pull request ros#3 from wjwwood/xmlrpcsock_osx_bug
7ab03e1 
Fixes a compile error on OS X due the symbol MSG_NOSIGNAL not being defined on OS X.
Nickolaim added a commit to Nickolaim/ros_comm that referenced this pull request Mar 2, 2018
@Nickolaim
Experimenting with python installation ros#3
2f93c2f
@pbeeson pbeeson mentioned this pull request Jul 23, 2018
Closed
johnsonshih added a commit to johnsonshih/ros_comm that referenced this pull request Sep 18, 2018
@johnsonshih
use os.linesep for portable across platforms (ros#3)
c0ea62b
@fujitatomoya fujitatomoya mentioned this pull request Nov 3, 2018
Closed
@rishabh900 rishabh900 mentioned this pull request Sep 17, 2019
Closed
@meng-atomic meng-atomic mentioned this pull request Jul 3, 2020
Closed
@Sunriseyms Sunriseyms mentioned this pull request Dec 29, 2021
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

@wjwwood wjwwood

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@wjwwood @dirk-thomas