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

[NVIDIA] Add ssl crypto dep on brpc #36181

Closed
wants to merge 3 commits into from

Conversation

zlsh80826
Copy link
Collaborator

PR types

Bug fixes

PR changes

Others

Describe

修正 issue #36179

device_worker_test 在 WITH_DISTRIBUTED=ON 時會動態連接 brpc, 同時 brpc 會需要 ssl 和 crypto, 但 dependency 只有 link brpc 沒有 link ssl, crypto 導致無法在編譯 device_worker_test 時找到 libssl, libcrypto 相關函式

@paddle-bot-old
Copy link

Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

@paddle-bot-old
Copy link

Sorry to inform you that 8ac9f9e's CIs have passed for more than 7 days. To prevent PR conflicts, you need to re-run all CIs manually.

@zlsh80826
Copy link
Collaborator Author

Close this bug because #37064 has fixed the problem

@zlsh80826 zlsh80826 closed this Dec 25, 2021
@zlsh80826 zlsh80826 deleted the brpc-ssl-crypto branch December 25, 2021 07:04
@zlsh80826 zlsh80826 changed the title add ssl crypto dep on brpc [NVIDIA] add ssl crypto dep on brpc Jan 6, 2022
@zlsh80826 zlsh80826 changed the title [NVIDIA] add ssl crypto dep on brpc [NVIDIA] Add ssl crypto dep on brpc Jan 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant