-
Notifications
You must be signed in to change notification settings - Fork 3k
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
[fix] Fixes for all packages and dependencies #1240
Merged
Merged
Changes from all commits
Commits
Show all changes
99 commits
Select commit
Hold shift + click to select a range
0453b6c
Initial Cleanup
amc-nu 9931eeb
fixed also for indigo
amc-nu 1833e5e
Merge branch 'develop' into fix/cmake_cleanup
amc-nu ea1a9e1
Merge branch 'develop' into fix/cmake_cleanup
amc-nu 575874d
kf cjeck
amc-nu 9ba9c61
Merge branch 'develop' into fix/cmake_cleanup
amc-nu b168501
Merge branch 'fix/cmake_cleanup' of https://github.com/CPFL/Autoware …
amc-nu 4102041
Fix road wizard
amc-nu 2030151
Added travis ci
amc-nu bc1cc09
Trigger CI
amc-nu b178f7d
Fixes to cv_tracker and lidar_tracker cmake
amc-nu e9b3a6e
Fix kitti player dependencies
amc-nu 75fa3ef
Removed unnecessary dependencies
amc-nu 0af2b00
messages fixing for can
amc-nu 2724a8a
Update build script travis
amc-nu 77b8ae3
Travis Path
amc-nu 30a392f
Travis Paths fix
amc-nu 6d4f527
Travis test
amc-nu 8b27335
Eigen checks
amc-nu 2d49cc9
removed unnecessary dependencies
amc-nu bea231f
Eigen Detection
amc-nu c999e85
Job number reduced
amc-nu b20a6a5
Eigen3 more fixes
amc-nu 4a6df7c
More Eigen3
amc-nu 5e51436
Even more Eigen
amc-nu e83bbac
find package cmake modules included
amc-nu 92c568a
More fixes to cmake modules
amc-nu bb64504
Removed non ros dependency
amc-nu a311d34
Enable industrial_ci for indidog and kinetic
amc-nu c98bd7d
Wrong install command
amc-nu 54903fd
fix rviz_plugin install
amc-nu 07acaac
FastVirtualScan fix
amc-nu dfc641b
Fix Qt5 Fastvirtualscan
amc-nu 0e32f88
Fixed qt5 system dependencies for rosdep
amc-nu d8bc116
NDT TKU Fix catkin not pacakged
amc-nu 8e6ba29
More in detail dependencies fixes for more packages
amc-nu 50ca492
GLEW library for ORB
amc-nu c42c052
Ignore OrbLocalizer
amc-nu 4a0ad64
Ignore Version checker
amc-nu ef7856e
Fix for driveworks interface
amc-nu fd16c87
driveworks not catkinpackagedd
amc-nu e091647
Missing catkin for driveworks
amc-nu 0ae3269
libdpm opencv not catkin packaged
amc-nu 8514fc8
catkin lib gnss not included in obj_db
amc-nu c581aa3
Points2Polygon fix
amc-nu c40e550
More missing dependencies
amc-nu 44863ff
image viewer not packaged
amc-nu 10c615a
Fixed SSH2 detection, added viewers for all distros
amc-nu e5279e4
Fix gnss localizer incorrect dependency config
amc-nu e429828
Fixes to multiple packages dependencies
amc-nu d4197f7
gnss plib and package
amc-nu 12d7c01
More fixes to gnss
amc-nu 930f327
gnss dependencies for gnss_loclaizer
amc-nu 12753ed
Missing gnss dependency for gnss on localizer
amc-nu 92efa31
More fixes for dependencies
amc-nu 1dfc181
gnss more fixes
amc-nu 6bf66a2
fixes to more dependencies
amc-nu f9bf78b
header dependency
amc-nu 7d0b765
Debug message
amc-nu fed8a91
more debug messages changed back to gnss
amc-nu 633c913
debud messages
amc-nu ff0f1e1
gnss test
amc-nu f27a113
gnss install command
amc-nu 1e0ebda
Several fixes for OpenPlanner and its lbiraries
amc-nu b299b88
Fixes to ROSInterface
amc-nu f3cd1df
More fixes to robotsdk and rosinterface
amc-nu c8a3703
robotsdk calibration fix
amc-nu c64cc34
Fixes to rosinterface robotsdk libraries and its nodes
amc-nu f7adede
Fixes to Qt5 missing dependencies in robotsdk
amc-nu d8d8afd
glviewer missing dependencies
amc-nu 7b7c9a7
Missing qt specific config cmake for robotsdk
amc-nu b1d0d91
disable cv_tracker
amc-nu ec7d242
Fix to open planner un needed dependendecies
amc-nu 65ed856
Fixes for libraries indecision maker
amc-nu 69d3e3d
Fixes to libraries decision_maker installation
amc-nu d4ebdd6
Gazebo on Kinetic
amc-nu d97d110
Added Missing library
amc-nu 8c0254b
* Removed Gazebo and synchonization packages
amc-nu 817f062
Fixes to lane_planner
amc-nu 5b06f16
Added NDT TKU Glut extra dependencies
amc-nu 583c3db
ndt localizer/lib fast pcl fixes
amc-nu 956cba1
Fix kf_lib
amc-nu 24dc292
Merge branch 'develop' into feature/industrial_ci
amc-nu ff59361
Keep industrial_ci
amc-nu 6d8a73e
Fixes for dpm library
amc-nu fa1404a
Fusion lib fixed
amc-nu cf89072
dpm and fusion header should match exported project name
amc-nu 480fda8
Fixes to dpm_ocv ndt_localizer and pcl_omp
amc-nu 8dc2690
no fast_pcl anymore
amc-nu c82aead
Merge branch 'develop' into feature/industrial_ci
amc-nu 91f2403
fixes to libdpm and its package
amc-nu c08b154
CI test
amc-nu d7f2803
test with native travis ci
amc-nu 73be432
missing update for apt
amc-nu 806d818
Fixes to pcl_omp installation and headers
amc-nu bfc1521
Merge branch 'develop' into feature/industrial_ci
yk-fujii 97c535c
Final fixes for tests, modified README
amc-nu fe63b9e
* Fixes to README
amc-nu 97ab81c
re enable native travis tests
amc-nu File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,40 +1,49 @@ | ||
cmake_minimum_required(VERSION 2.8.3) | ||
project(as) | ||
|
||
# If submodule is not cloned this project will not be build. | ||
set(AS_MSG_PATH "${CMAKE_SOURCE_DIR}/msgs/platform_automation_msgs/module_comm_msgs") | ||
if(EXISTS "${AS_MSG_PATH}") | ||
|
||
find_package( | ||
catkin REQUIRED COMPONENTS | ||
roscpp | ||
std_msgs | ||
geometry_msgs | ||
module_comm_msgs | ||
dbw_mkz_msgs | ||
catkin REQUIRED COMPONENTS | ||
roscpp | ||
std_msgs | ||
geometry_msgs | ||
module_comm_msgs | ||
dbw_mkz_msgs | ||
) | ||
|
||
catkin_package( | ||
CATKIN_DEPENDS | ||
roscpp | ||
std_msgs | ||
geometry_msgs | ||
module_comm_msgs | ||
dbw_mkz_msgs | ||
CATKIN_DEPENDS | ||
roscpp | ||
std_msgs | ||
geometry_msgs | ||
module_comm_msgs | ||
dbw_mkz_msgs | ||
) | ||
|
||
SET(CMAKE_CXX_FLAGS "-std=c++11 -O2 -g -Wall ${CMAKE_CXX_FLAGS}") | ||
# If submodule is not cloned this project will not be build. | ||
set(AS_MSG_PATH "${CMAKE_SOURCE_DIR}/msgs/platform_automation_msgs/module_comm_msgs") | ||
if (EXISTS "${AS_MSG_PATH}") | ||
|
||
SET(CMAKE_CXX_FLAGS "-std=c++11 -O2 -g -Wall ${CMAKE_CXX_FLAGS}") | ||
|
||
include_directories( | ||
${catkin_INCLUDE_DIRS} | ||
) | ||
include_directories( | ||
${catkin_INCLUDE_DIRS} | ||
) | ||
|
||
add_executable(pacmod_interface | ||
nodes/pacmod_interface/pacmod_interface_node.cpp | ||
nodes/pacmod_interface/pacmod_interface.cpp | ||
) | ||
target_link_libraries(pacmod_interface ${catkin_LIBRARIES}) | ||
add_dependencies(pacmod_interface ${catkin_EXPORTED_TARGETS}) | ||
|
||
install(TARGETS pacmod_interface | ||
ARCHIVE DESTINATION ${CATKIN_PACKAGE_LIB_DESTINATION} | ||
LIBRARY DESTINATION ${CATKIN_PACKAGE_LIB_DESTINATION} | ||
RUNTIME DESTINATION ${CATKIN_PACKAGE_BIN_DESTINATION} | ||
) | ||
|
||
add_executable(pacmod_interface | ||
nodes/pacmod_interface/pacmod_interface_node.cpp | ||
nodes/pacmod_interface/pacmod_interface.cpp | ||
) | ||
target_link_libraries(pacmod_interface ${catkin_LIBRARIES}) | ||
add_dependencies(pacmod_interface ${catkin_EXPORTED_TARGETS} ) | ||
install(DIRECTORY launch/ | ||
DESTINATION ${CATKIN_PACKAGE_SHARE_DESTINATION}/launch | ||
PATTERN ".svn" EXCLUDE) | ||
|
||
endif() | ||
endif () |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
File renamed without changes.
File renamed without changes.
26 changes: 14 additions & 12 deletions
26
ros/src/computing/perception/detection/lib/fusion/package.xml
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,15 +1,17 @@ | ||
<?xml version="1.0"?> | ||
<package> | ||
<name>fusion</name> | ||
<version>1.6.3</version> | ||
<description>The fusion package</description> | ||
<maintainer email="kondoh@axe-inc.co.jp">Masao KONDOH</maintainer> | ||
<license>BSD</license> | ||
<buildtool_depend>catkin</buildtool_depend> | ||
<build_depend>roscpp</build_depend> | ||
<build_depend>autoware_msgs</build_depend> | ||
<run_depend>roscpp</run_depend> | ||
<run_depend>autoware_msgs</run_depend> | ||
<export> | ||
</export> | ||
<name>fusion</name> | ||
<version>1.6.3</version> | ||
<description>The fusion package</description> | ||
<maintainer email="kondoh@axe-inc.co.jp">Masao KONDOH</maintainer> | ||
<license>BSD</license> | ||
<buildtool_depend>catkin</buildtool_depend> | ||
|
||
<build_depend>roscpp</build_depend> | ||
<build_depend>autoware_msgs</build_depend> | ||
<build_depend>cv_bridge</build_depend> | ||
|
||
<run_depend>roscpp</run_depend> | ||
<run_depend>autoware_msgs</run_depend> | ||
<run_depend>cv_bridge</run_depend> | ||
</package> |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Moving this IF statement always makes catkin search for module_comm_msgs.
So, I got error:
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.
I think you're missing the
as
submodule.Please run:
git submodule update --init --recursive
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.
I know to need
git submodule update --init --recursive
But, I think that autoware does not necessarily need to compile "as" package because problems often occur frequently by not executing this command.
If you want to use "as" default, I think that you need to include this command in catkin_make_release, install manual, or etc.
How about you think?
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.
Then if this package is not needed I recommend to add a CATKIN_IGNORE
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.
👍