Skip to content

[FEA] Enable building static libs#602

Merged
rapids-bot[bot] merged 8 commits into
NVIDIA:branch-22.06from
trxcllnt:fea/enable-static-libs
Mar 31, 2022
Merged

[FEA] Enable building static libs#602
rapids-bot[bot] merged 8 commits into
NVIDIA:branch-22.06from
trxcllnt:fea/enable-static-libs

Conversation

@trxcllnt

@trxcllnt trxcllnt commented Mar 30, 2022

Copy link
Copy Markdown
Member

This PR allows building libcugraphops static libs via CMake option -DBUILD_SHARED_LIBS=ON|OFF.

I was seeing a linker error due to not having -fPIC enabled, but I wouldn't have expected this to affect static libs. I'll investigate some more and turn it off if possible, but for now -fPIC is enabled.

@trxcllnt trxcllnt requested a review from a team as a code owner March 30, 2022 20:24
@trxcllnt trxcllnt requested a review from robertmaynard March 30, 2022 20:24
@trxcllnt trxcllnt added 3 - Ready for Review improvement Improvement / enhancement to an existing function non-breaking Non-breaking change and removed cpp CMake labels Mar 30, 2022

@robertmaynard robertmaynard left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM to me

@trxcllnt

Copy link
Copy Markdown
Member Author

rerun tests

@cjnolet cjnolet left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@trxcllnt

Copy link
Copy Markdown
Member Author

rerun tests

@cjnolet

cjnolet commented Mar 31, 2022

Copy link
Copy Markdown
Contributor

@gpucibot merge

@rapids-bot rapids-bot Bot merged commit 4a3dfb9 into NVIDIA:branch-22.06 Mar 31, 2022
loulankxh pushed a commit to loulankxh/raft that referenced this pull request Oct 14, 2025
This PR allows building libcugraphops static libs via CMake option [`-DBUILD_SHARED_LIBS=ON|OFF`](https://cmake.org/cmake/help/latest/variable/BUILD_SHARED_LIBS.html).

~I was seeing a linker error due to not having `-fPIC` enabled, but I wouldn't have expected this to affect static libs. I'll investigate some more and turn it off if possible, but for now `-fPIC` is enabled.~

Authors:
  - Paul Taylor (https://github.com/trxcllnt)

Approvers:
  - Corey J. Nolet (https://github.com/cjnolet)

URL: NVIDIA#602
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3 - Ready for Review improvement Improvement / enhancement to an existing function non-breaking Non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants