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
cmake: link ceph-fuse against librt #31531
Conversation
ceph-fuse: fix error of building ceph-fuse fix linking error of without -lrt for ceph-fuse. error message: ``` [100%] Building CXX object src/CMakeFiles/ceph-fuse.dir/ceph_fuse.cc.o [100%] Building CXX object src/CMakeFiles/ceph-fuse.dir/client/fuse_ll.cc.o [100%] Linking CXX executable ../bin/ceph-fuse /opt/rh/devtoolset-7/root/usr/libexec/gcc/x86_64-redhat-linux/7/ld: CMakeFiles/ceph-fuse.dir/ceph_fuse.cc.o: undefined reference to symbol 'clock_gettime@@GLIBC_2.2.5' //lib64/librt.so.1: error adding symbols: DSO missing from command line collect2: error: ld returned 1 exit status make[3]: *** [bin/ceph-fuse] Error 1 make[2]: *** [src/CMakeFiles/ceph-fuse.dir/all] Error 2 make[1]: *** [src/CMakeFiles/ceph-fuse.dir/rule] Error 2 make: *** [ceph-fuse] Error 2 ``` Signed-off-by: Yong Wang <wy7980@sina.com>
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.
LGTM
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'd suggest revise the title of commit message to something like
cmake: link ceph-fuse against librt
Yes, the new title can be more clear. |
@wy7980 could you squash these two commits into a single one? |
@wy7980 instead of creating a new PR for addressing the reviewers' comments, i'd recommend reusing the existing one. you can always use |
@wy7980 also, would be great if you could use your full name in your "Signed-off-by" line. i don't think it's "wy7980", otherwise it's just very unusual.. |
fede9ab
to
aabfe40
Compare
got it. |
* refs/pull/31531/head: cmake: link ceph-fuse against librt Reviewed-by: Patrick Donnelly <pdonnell@redhat.com> Reviewed-by: Kefu Chai <kchai@redhat.com>
ceph-fuse: fix error of building ceph-fuse
fix linking error of without -lrt for ceph-fuse.
error message:
Signed-off-by: Yong Wang wy7980@sina.com
Checklist
Show available Jenkins commands
jenkins retest this please
jenkins test crimson perf
jenkins test signed
jenkins test make check
jenkins test make check arm64
jenkins test submodules
jenkins test dashboard
jenkins test dashboard backend
jenkins test docs
jenkins render docs
jenkins test ceph-volume all
jenkins test ceph-volume tox