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

add missing glog flag that caused problems in some systems #685

Merged
merged 1 commit into from
Feb 20, 2023

Conversation

marinagmoreira
Copy link
Member

@Bckempa can you confirm this was the change in your system?

@Bckempa
Copy link
Contributor

Bckempa commented Feb 18, 2023

@Bckempa can you confirm this was the change in your system?

Yep, this fixes the build for me - thanks!

For sake of a complete and searchable record, without this patch I hit a build error on a fresh Ubuntu 16.04.7 install following the install guide for NASA users:

Errors     << calibration:make /home/bkempa/astrobee/logs/calibration/build.make.003.log
/usr/bin/ld: CMakeFiles/run_camera_target_based_intrinsics_calibrator.dir/tools/run_camera_target_based_intrinsics_calibrator.cc.o: undefined reference to symbol '_ZN6google4base21CheckOpMessageBuilder7ForVar2Ev'
//usr/lib/x86_64-linux-gnu/libglog.so.0: error adding symbols: DSO missing from command line
collect2: error: ld returned 1 exit status
make[2]: *** [/home/bkempa/astrobee/devel/.private/calibration/lib/calibration/run_camera_target_based_intrinsics_calibrator] Error 1
make[1]: *** [CMakeFiles/run_camera_target_based_intrinsics_calibrator.dir/all] Error 2
make: *** [all] Error 2

which contains the mangled symbol name google::base::CheckOpMessageBuilder::ForVar2() and was caused by a missing link flag for glog.

@marinagmoreira marinagmoreira merged commit aec9796 into nasa:develop Feb 20, 2023
marinagmoreira added a commit that referenced this pull request Mar 9, 2023
* Documentation update (#668)

* docker docs update

* make build instructions ROS1

* updating install instructions, hopefully making them easier to follow

* updated from github PR reviews

* fix typo

* replace list with append (#671)

* ci: bump to checkout@v3 (#673)

* Unlatching topics (#674)

* Changed heartbeat topic to be unlatched.

* Changed echoed command to not latching.

* no subscribers and info is obtained by service, so no latched topic needed

* removing latched on frequently published topics

* Made command topic to not atching.

* no need for these to be latched

* sub and pub in same node, no need to latch

* no need to be latched no subs and not recorded, rviz should be opened when published

* Made ground bridge topics not latched.

* Fixed heartbeat topic latched comment.

---------

Co-authored-by: Katie Browne <kathryn.browne@nasa.gov>

* support multiple versions for docs (#675)

* Make coverage_analyzer.py support Python 3 (#680)

Refactor to simplify and make output identical between Python 2 and 3

* Release 0.17.0 (#683)

* add missing glog flag that caused problems in some systems (#685)

* Make isort repeatable (#694)

* make isort repeatable

* isort now consistently groups astrobee imports as first party; one-time update to apply to old files

* moving ff_names to ff_common, updating headers (#696)

---------

Co-authored-by: Trey Smith <trey.smith@nasa.gov>
Co-authored-by: Katie Browne <kathryn.browne@nasa.gov>
trey0 pushed a commit to trey0/astrobee that referenced this pull request Mar 15, 2023
luisa-mao pushed a commit to luisa-mao/astrobee-1 that referenced this pull request Jul 11, 2023
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