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

cmake: make crimson libraries static to prevent cmake error #36907

Merged
merged 2 commits into from Sep 1, 2020

Conversation

yuvalif
Copy link
Contributor

@yuvalif yuvalif commented Aug 31, 2020

Fixes: https://tracker.ceph.com/issues/47209

Signed-off-by: Yuval Lifshitz ylifshit@redhat.com

Checklist

  • References tracker ticket

Show available Jenkins commands
  • jenkins retest this please
  • jenkins test classic perf
  • jenkins test crimson perf
  • jenkins test signed
  • jenkins test make check
  • jenkins test make check arm64
  • jenkins test submodules
  • jenkins test dashboard
  • jenkins test api
  • jenkins test docs
  • jenkins render docs
  • jenkins test ceph-volume all
  • jenkins test ceph-volume tox

Copy link
Contributor

@ronen-fr ronen-fr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I seem to remember that this issue was already handled some short
time ago. @liu-chunmei ?

src/test/crimson/seastore/CMakeLists.txt Show resolved Hide resolved
src/test/crimson/seastore/CMakeLists.txt Outdated Show resolved Hide resolved
src/test/crimson/seastore/CMakeLists.txt Show resolved Hide resolved
yuvalif and others added 2 commits August 31, 2020 19:22
Signed-off-by: Yuval Lifshitz <ylifshit@redhat.com>
@tchaikov tchaikov changed the title crimson/cmake: make libraries static to prevent cmake error cmake: make crimson libraries static to prevent cmake error Aug 31, 2020
@liu-chunmei
Copy link
Contributor

I seem to remember that this issue was already handled some short
time ago. @liu-chunmei ?

I am not sure what you mean, I check master branch, the issue still there.

@tchaikov tchaikov merged commit caa7baa into ceph:master Sep 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants