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

actionlib headers fail to compile on OS X #15

Merged
merged 1 commit into from
Aug 28, 2013
Merged

Conversation

wjwwood
Copy link
Member

@wjwwood wjwwood commented Aug 28, 2013

When compiling actionlib_tutorials on OS X with the latest hydro I get:

In file included from /Users/william/hydro/src/actionlib_tutorials/src/averaging_server.cpp:3:
In file included from /Users/william/hydro/new_install/include/actionlib/server/simple_action_server.h:42:
In file included from /Users/william/hydro/new_install/include/actionlib/server/action_server.h:52:
/Users/william/hydro/new_install/include/actionlib/server/action_server_base.h:163:12: error: default arguments cannot be added to an out-of-line definition of a member of a
      class template
      bool auto_start = false)  :
           ^            ~~~~~
In file included from /Users/william/hydro/src/actionlib_tutorials/src/fibonacci_server.cpp:2:
In file included from /Users/william/hydro/new_install/include/actionlib/server/simple_action_server.h:42:
In file included from /Users/william/hydro/new_install/include/actionlib/server/action_server.h:52:
/Users/william/hydro/new_install/include/actionlib/server/action_server_base.h:163:12: error: default arguments cannot be added to an out-of-line definition of a member of a
      class template
      bool auto_start = false)  :
           ^            ~~~~~
1 error generated.
1 error generated.
make[2]: *** [CMakeFiles/fibonacci_server.dir/src/fibonacci_server.cpp.o] Error 1
make[1]: *** [CMakeFiles/fibonacci_server.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....
make[2]: *** [CMakeFiles/averaging_server.dir/src/averaging_server.cpp.o] Error 1
make[1]: *** [CMakeFiles/averaging_server.dir/all] Error 2
Linking CXX executable /Users/william/hydro/devel_isolated/actionlib_tutorials/lib/actionlib_tutorials/averaging_client
Linking CXX executable /Users/william/hydro/devel_isolated/actionlib_tutorials/lib/actionlib_tutorials/fibonacci_client
[100%] Built target fibonacci_client
[100%] Built target averaging_client
make: *** [all] Error 2
<== Failed to process package 'actionlib_tutorials':
  Command '/Users/william/hydro/new_install/env.sh make -j4 -l4' returned non-zero exit status 2

Reproduce this error by running:
==> cd /Users/william/hydro/build_isolated/actionlib_tutorials && /Users/william/hydro/new_install/env.sh make -j4 -l4

Command failed, exiting.

@wjwwood
Copy link
Member Author

wjwwood commented Aug 28, 2013

Upgraded to a pull request, @jbohren @dirk-thomas can you review?

@dirk-thomas
Copy link
Member

Reference #11 to keep track where the change was made - does not affect the groovy branch.

+1 @wjwwood Can you go ahead and merge/release 1.10.3 into Hydro?

@jbohren
Copy link
Member

jbohren commented Aug 28, 2013

👍 Looks good to me.

wjwwood added a commit that referenced this pull request Aug 28, 2013
actionlib headers fail to compile on OS X
@wjwwood wjwwood merged commit f1c37b0 into hydro-devel Aug 28, 2013
@wjwwood wjwwood deleted the fix_osx_build branch August 28, 2013 00:32
@wjwwood
Copy link
Member Author

wjwwood commented Aug 28, 2013

Done. ros/rosdistro#1872

severin-lemaignan referenced this pull request in severin-lemaignan/robotpkg Aug 18, 2014
Changes since 1.9.12:

1.11.2 (2014-05-20)
-------------------
* Update python publishers to define queue_size.
* Use the correct queue for processing MessageEvents
* Contributors: Esteve Fernandez, Michael Ferguson, Nican

1.11.1 (2014-05-08)
-------------------
* Fix uninitialised execute_thread_ member pointer
* Make rostest in CMakeLists optional
* Use catkin_install_python() to install Python scripts
* Contributors: Dirk Thomas, Esteve Fernandez, Jordi Pages, Lukas Bulwahn

1.11.0 (2014-02-13)
-------------------
* replace usage of __connection_header with MessageEvent (`#20
<https://github.com/ros/actionlib/issues/20>`_)

1.10.3 (2013-08-27)
-------------------
* Merged pull request `#15 <https://github.com/ros/actionlib/issues/15>`_
  Fixes a compile issue for actionlib headers on OS X

1.10.2 (2013-08-21)
-------------------
* separating ActionServer implementation into base class and
ros-publisher-based class (`#11 <https://github.com/ros/actionlib/issues/11>`_)
* support CATKIN_ENABLE_TESTING
* add isValid to ServerGoalHandle (`#14
<https://github.com/ros/actionlib/issues/14>`_)
* make operators const (`#10 <https://github.com/ros/actionlib/issues/10>`_)
* add counting of connections to avoid reconnect problem when callbacks are
invoked in different order (`#7 <https://github.com/ros/actionlib/issues/7>`_)
* fix deadlock in simple_action_server.py (`#4
<https://github.com/ros/actionlib/issues/4>`_)
* fix missing runtime destination for library (`#3
<https://github.com/ros/actionlib/issues/3>`_)

1.10.1 (2013-06-06)
-------------------
* fix location of library before installation (`#1
<https://github.com/ros/actionlib/issues/1>`_)

1.10.0 (2013-04-11)
-------------------
* define DEPRECATED only if not defined already
* modified dependency type of catkin to buildtool
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants