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

Define test repo as module. #7

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bigzed
Copy link
Contributor

@bigzed bigzed commented Feb 10, 2021

The change allows pronto-simplecov to be included in a Gemfile with

gem 'pronto_simplecov', github: 'dsander/pronto-simplecov'', branch: 'master'

before bundle failed with:

fatal: No url found for submodule path 'spec/data/test' in .gitmodules

@dsander
Copy link
Owner

dsander commented Feb 19, 2021

Hmm this is odd, I tried adding the current master branch to both a normal Gemfile and one that is a submodule of a different git repository and didn't have the issue. Is it specific to our setup at work?

@bigzed
Copy link
Contributor Author

bigzed commented Feb 23, 2021

This is the error I get, when I add pronto-simplecov with gem 'pronto-simplecov', github: 'dsander/pronto-simplecov', branch: 'master' to view-farm. Even if I removed the directory the error suggests, the next bundle still fails.

Fetching https://github.com/dsander/pronto-simplecov.git
fatal: No url found for submodule path 'spec/data/test' in .gitmodules

Retrying `git submodule foreach --quiet git\ -C\ \$toplevel\ submodule\ deinit\ --force\ \$sm_path` at /home/steved/.rbenv/versions/2.5.8/lib/ruby/gems/2.5.0/bundler/gems/pronto-simplecov-092b000edc8e due to error (2/4): Bundler::Source::Git::GitCommandError Git error: command `git submodule foreach --quiet git\ -C\ \$toplevel\ submodule\ deinit\ --force\ \$sm_path` in directory /home/steved/.rbenv/versions/2.5.8/lib/ruby/gems/2.5.0/bundler/gems/pronto-simplecov-092b000edc8e has failed.

If this error persists you could try removing the cache directory '/home/steved/.rbenv/versions/2.5.8/lib/ruby/gems/2.5.0/bundler/gems/pronto-simplecov-092b000edc8e'

Retrying `git submodule foreach --quiet git\ -C\ \$toplevel\ submodule\ deinit\ --force\ \$sm_path` at /home/steved/.rbenv/versions/2.5.8/lib/ruby/gems/2.5.0/bundler/gems/pronto-simplecov-092b000edc8e due to error (3/4): Bundler::Source::Git::GitCommandError Git error: command `git submodule foreach --quiet git\ -C\ \$toplevel\ submodule\ deinit\ --force\ \$sm_path` in directory /home/steved/.rbenv/versions/2.5.8/lib/ruby/gems/2.5.0/bundler/gems/pronto-simplecov-092b000edc8e has failed.

If this error persists you could try removing the cache directory '/home/steved/.rbenv/versions/2.5.8/lib/ruby/gems/2.5.0/bundler/gems/pronto-simplecov-092b000edc8e'

Retrying `git submodule foreach --quiet git\ -C\ \$toplevel\ submodule\ deinit\ --force\ \$sm_path` at /home/steved/.rbenv/versions/2.5.8/lib/ruby/gems/2.5.0/bundler/gems/pronto-simplecov-092b000edc8e due to error (4/4): Bundler::Source::Git::GitCommandError Git error: command `git submodule foreach --quiet git\ -C\ \$toplevel\ submodule\ deinit\ --force\ \$sm_path` in directory /home/steved/.rbenv/versions/2.5.8/lib/ruby/gems/2.5.0/bundler/gems/pronto-simplecov-092b000edc8e has failed.

If this error persists you could try removing the cache directory '/home/steved/.rbenv/versions/2.5.8/lib/ruby/gems/2.5.0/bundler/gems/pronto-simplecov-092b000edc8e'

Git error: command `git submodule foreach --quiet git\ -C\ \$toplevel\ submodule\ deinit\ --force\ \$sm_path` in directory /home/steved/.rbenv/versions/2.5.8/lib/ruby/gems/2.5.0/bundler/gems/pronto-simplecov-092b000edc8e has failed.

If this error persists you could try removing the cache directory '/home/steved/.rbenv/versions/2.5.8/lib/ruby/gems/2.5.0/bundler/gems/pronto-simplecov-092b000edc8e'

@dsander
Copy link
Owner

dsander commented Feb 28, 2021

I could not reproduce this neither when bundle installing natively (ruby is installed with asdf) nor with our dockerized development environment. The change kind of makes sense to me, but I'd like to understand why it's required in some cased, but isn't in others.

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.

2 participants