-
Notifications
You must be signed in to change notification settings - Fork 26
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
Merge toolchain-2.9 into master and discard the toolchain-2.x branches? #135
Changes from all commits
84abd86
79ee4d7
9bf3cdd
d189e72
2b29e3a
612e58f
256c46f
648fdfd
7236c21
ac08bc9
9830140
70c2dc3
18d797c
adb619f
a227f95
d48d352
1282b06
602a819
42848ec
ff417e6
7d28003
598eed9
b7fd688
5026fb5
251439b
6d1b101
e6a4902
237f73c
fc85335
5a14326
c01a6c1
48089c8
6339de6
40cd3a8
1d9b7fc
3d95ed9
57fa5b1
9f7310d
5e8b7c8
d4779e6
08c1123
9db89cf
f39fbf5
5191957
1c60a27
20c56cf
b58c8d9
89063c5
b374765
32bcec8
4b05231
69476e5
5a4f821
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -1000,6 +1000,11 @@ def self.gccxml(file, options) | |
if raw = options[:rawflags] | ||
cmdline.concat(raw) | ||
end | ||
if 1.size == 4 | ||
cmdline << "-m32" | ||
else | ||
cmdline << "-m64" | ||
end | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Likely can be discarded if gccxml is obsolete anyway. |
||
|
||
if defs = options[:define] | ||
defs.each do |str| | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -23,12 +23,12 @@ | |
<depend_optional package="castxml" /> | ||
<depend_optional package="gccxml" /> | ||
--> | ||
<depend package="boost" /> | ||
<depend package="facets" /> | ||
<depend package="libxml2" /> | ||
<depend package="pkg-config" /> | ||
<depend package="ruby-dev" /> | ||
<depend package="ruby-backports" /> | ||
<rosdep name="boost" /> | ||
<rosdep name="facets" /> | ||
<rosdep name="libxml2" /> | ||
<rosdep name="pkg-config" /> | ||
<rosdep name="ruby-dev" /> | ||
<rosdep name="ruby-backports" /> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This is required to find the matching keys in the rosdep database. |
||
|
||
<tags>stable</tags> | ||
|
||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -15,7 +15,7 @@ endif (NOT TYPELIB_HARDCODED_PLUGIN_PATH) | |
set_source_files_properties(endianness.cc PROPERTIES | ||
COMPILE_FLAGS -fno-strict-aliasing) | ||
|
||
target_link_libraries(typeLib ${CMAKE_DL_LIBS} ${Boost_FILESYSTEM_LIBRARIES} ${Boost_SYSTEM_LIBRARIES}) | ||
target_link_libraries(typeLib pthread ${CMAKE_DL_LIBS} ${Boost_FILESYSTEM_LIBRARIES} ${Boost_SYSTEM_LIBRARIES} ${Boost_THREAD_LIBRARIES}) | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Boost mutexes are used in typelib/utilmm/singleton/server.hh and hence the library must be linked to Boost Thread. Could be that |
||
#TARGET_LINK_LIBRARIES(typeLib typeLang_c typeLang_tlb typeLang_idl) | ||
|
||
install(TARGETS typeLib | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Installation of
package.xml
is required to make ROS tools happy.