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

Remove check for gem subdirectories. #456

Merged

Conversation

voxik
Copy link
Contributor

@voxik voxik commented Feb 15, 2013

This check is done later in Installer#pre_install_checks anyway.

This check is done later in Installer#pre_install_checks anyway.
@saturnflyer
Copy link

Looks good to me!

drbrain added a commit that referenced this pull request Apr 4, 2013
…ries

Remove check for gem subdirectories.
@drbrain drbrain merged commit aaf2566 into rubygems:master Apr 4, 2013
drbrain added a commit that referenced this pull request Apr 4, 2013
drbrain added a commit that referenced this pull request Apr 4, 2013
drbrain added a commit that referenced this pull request Apr 4, 2013
* pietro-key_passphrase: (154 commits)
  Update missed tests for rebase of #447
  Fixed pull request number type for #461
  Improve documentation of DependencyInstaller
  Alphabetize Gem::DependencyInstaller
  Removed commented out DependencyInstaller code
  Alter #489 to use GEM_SPEC_CACHE
  fix tests when GEM_SPEC is set in environment
  add support for ENV GEM_SPEC, fix #430
  Updated history for #443
  Don't alter Gem::Specification.dirs during install
  Default to Gem.dir as late as possible.
  Updated history for #455
  Update history for #456
  Update history for #462
  Add tests and alter output of #514
  add PATH to gem env
  Update History for #514
  Restore backwards-compatibility for #517
  Alphabetize RequestSet
  Undent RequestSet
  ...

Conflicts:
	test/rubygems/test_gem_commands_cert_command.rb
	test/rubygems/test_gem_package.rb
drbrain added a commit that referenced this pull request Jun 27, 2013
drbrain added a commit that referenced this pull request Jun 27, 2013
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.

3 participants