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

Gem::Specification#to_ruby needs OpenSSL #2937

Merged
1 commit merged into from Sep 30, 2019
Merged

Conversation

nobu
Copy link
Contributor

@nobu nobu commented Sep 30, 2019

Description:

Since b1d825a, lib/rubygems/specification.rb should require openssl.

Tasks:

  • Describe the problem / feature
  • Write tests
  • Write code to solve the problem
  • Get code review from coworkers / friends

I will abide by the code of conduct.

@deivid-rodriguez
Copy link
Member

This is a duplicate of #2929, but the other PR doesn't have passing tests because of an unrelated issue, so I'm fine merging this as it is. I think the place where the require has been put is good. I placed it right were it's used, but I don't think it matters much?

@bronzdoc
Copy link
Member

@bundlerbot r+

ghost pushed a commit that referenced this pull request Sep 30, 2019
2937: Gem::Specification#to_ruby needs OpenSSL r=bronzdoc a=nobu

# Description:

Since b1d825a, lib/rubygems/specification.rb should require openssl.

# Tasks:

- [x] Describe the problem / feature
- [ ] Write tests
- [x] Write code to solve the problem
- [ ] Get code review from coworkers / friends

I will abide by the [code of conduct](https://github.com/rubygems/rubygems/blob/master/CODE_OF_CONDUCT.md).


Co-authored-by: Nobuyoshi Nakada <nobu@ruby-lang.org>
@ghost
Copy link

ghost commented Sep 30, 2019

Build succeeded

@ghost ghost merged commit 0dd5b88 into rubygems:master Sep 30, 2019
@nobu nobu deleted the openssl-in-to_ruby branch October 1, 2019 00:29
@hsbt hsbt added this to the 3.0.8 milestone Feb 18, 2020
hsbt pushed a commit that referenced this pull request Feb 18, 2020
2937: Gem::Specification#to_ruby needs OpenSSL r=bronzdoc a=nobu

# Description:

Since b1d825a, lib/rubygems/specification.rb should require openssl.

# Tasks:

- [x] Describe the problem / feature
- [ ] Write tests
- [x] Write code to solve the problem
- [ ] Get code review from coworkers / friends

I will abide by the [code of conduct](https://github.com/rubygems/rubygems/blob/master/CODE_OF_CONDUCT.md).


Co-authored-by: Nobuyoshi Nakada <nobu@ruby-lang.org>
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants