Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

[Dependency Update] Upgrade openssl to 1.1.1b #14837

Merged
merged 4 commits into from
May 6, 2019

Conversation

stu1130
Copy link
Contributor

@stu1130 stu1130 commented Apr 29, 2019

Description

Upgrade the openssl package to 1.1.1b due to the issue list at 1.0.2l
And the openssl < 1.1.0 didn't support -j (multithreading)
Also fix a bug where the lib folder is missing when we done compiling the mkldnn

Checklist

Essentials

Please feel free to remove inapplicable items for your PR.

  • CI check for scala publish build
  • CI check for python publish build
  • Ubuntu 16.04 cu100mkl build
  • Ubuntu 16.04 mkl build

Changes

Comments

@stu1130 stu1130 requested a review from szha as a code owner April 29, 2019 20:22
@stu1130 stu1130 changed the title [Dependency Update] Upgrade openssl to 1.1.0k [WIP][Dependency Update] Upgrade openssl to 1.1.0k Apr 29, 2019
@stu1130 stu1130 closed this Apr 29, 2019
@stu1130 stu1130 changed the title [WIP][Dependency Update] Upgrade openssl to 1.1.0k [WIP][Dependency Update] Upgrade openssl to 1.1.0j Apr 29, 2019
@stu1130 stu1130 reopened this Apr 29, 2019
@stu1130 stu1130 closed this May 1, 2019
@stu1130 stu1130 changed the title [WIP][Dependency Update] Upgrade openssl to 1.1.0j [WIP][Dependency Update] Upgrade openssl to 1.1.1b May 1, 2019
@stu1130 stu1130 reopened this May 1, 2019
@stu1130 stu1130 changed the title [WIP][Dependency Update] Upgrade openssl to 1.1.1b [Dependency Update] Upgrade openssl to 1.1.1b May 1, 2019
@stu1130
Copy link
Contributor Author

stu1130 commented May 1, 2019

ping @lanking520 @szha for review

@anirudhacharya
Copy link
Member

@mxnet-label-bot add [pr-awaiting-review]

@marcoabreu marcoabreu added the pr-awaiting-review PR is waiting for code review label May 2, 2019
Copy link
Member

@wkcn wkcn left a comment

Choose a reason for hiding this comment

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

LGTM. Thank you!

@wkcn wkcn added pr-awaiting-merge Review and CI is complete. Ready to Merge and removed pr-awaiting-review PR is waiting for code review labels May 6, 2019
@lanking520 lanking520 merged commit a722db4 into apache:master May 6, 2019
access2rohit pushed a commit to access2rohit/incubator-mxnet that referenced this pull request May 14, 2019
* upgrade openssl to 1.1.0k

* fix the wrong version

* upgrade to the latest version

* avoid lib to be treated as file instead of folder
haohuanw pushed a commit to haohuanw/incubator-mxnet that referenced this pull request Jun 23, 2019
* upgrade openssl to 1.1.0k

* fix the wrong version

* upgrade to the latest version

* avoid lib to be treated as file instead of folder
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
pr-awaiting-merge Review and CI is complete. Ready to Merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants