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

oxidized does not check if git is installed #21

Closed
RichiH opened this issue Jul 11, 2014 · 1 comment
Closed

oxidized does not check if git is installed #21

RichiH opened this issue Jul 11, 2014 · 1 comment

Comments

@RichiH
Copy link
Contributor

RichiH commented Jul 11, 2014

If grit is installed and git usage configured, but Git is not installed, this happens:

oxidized@ubuntu:~$ oxidized -d
Puma 2.8.2 starting...
* Min threads: 0, max threads: 16
* Environment: development
* Listening on tcp://<redacted>:8888
== Sinatra/1.4.5 has taken the stage on 8888 for development with backup from Puma
can't convert nil into String
== Sinatra has ended his set (crowd applauds)
== Sinatra has ended his set (crowd applauds)
/var/lib/gems/1.9.1/gems/posix-spawn-0.3.8/lib/posix/spawn.rb:187:in `_pspawn': can't convert nil into String (TypeError)
    from /var/lib/gems/1.9.1/gems/posix-spawn-0.3.8/lib/posix/spawn.rb:187:in `pspawn'
    from /var/lib/gems/1.9.1/gems/posix-spawn-0.3.8/lib/posix/spawn.rb:160:in `spawn'
    from /var/lib/gems/1.9.1/gems/posix-spawn-0.3.8/lib/posix/spawn.rb:312:in `popen4'
    from /var/lib/gems/1.9.1/gems/posix-spawn-0.3.8/lib/posix/spawn/child.rb:105:in `exec!'
    from /var/lib/gems/1.9.1/gems/posix-spawn-0.3.8/lib/posix/spawn/child.rb:80:in `initialize'
    from /var/lib/gems/1.9.1/gems/grit-2.5.0/lib/grit/git.rb:345:in `new'
    from /var/lib/gems/1.9.1/gems/grit-2.5.0/lib/grit/git.rb:345:in `native'
    from /var/lib/gems/1.9.1/gems/grit-2.5.0/lib/grit/git.rb:372:in `method_missing'
    from /var/lib/gems/1.9.1/gems/grit-2.5.0/lib/grit/git-ruby.rb:16:in `init'
    from /var/lib/gems/1.9.1/gems/grit-2.5.0/lib/grit/repo.rb:101:in `init_bare'
    from /var/lib/gems/1.9.1/gems/oxidized-0.0.63/lib/oxidized/output/git.rb:38:in `rescue in store'
    from /var/lib/gems/1.9.1/gems/oxidized-0.0.63/lib/oxidized/output/git.rb:33:in `store'
    from /var/lib/gems/1.9.1/gems/oxidized-0.0.63/lib/oxidized/worker.rb:36:in `process'
    from /var/lib/gems/1.9.1/gems/oxidized-0.0.63/lib/oxidized/worker.rb:13:in `block in work'
    from /var/lib/gems/1.9.1/gems/oxidized-0.0.63/lib/oxidized/worker.rb:13:in `each'
    from /var/lib/gems/1.9.1/gems/oxidized-0.0.63/lib/oxidized/worker.rb:13:in `work'
    from /var/lib/gems/1.9.1/gems/oxidized-0.0.63/lib/oxidized/core.rb:36:in `run'
    from /var/lib/gems/1.9.1/gems/oxidized-0.0.63/lib/oxidized/core.rb:29:in `initialize'
    from /var/lib/gems/1.9.1/gems/oxidized-0.0.63/lib/oxidized/core.rb:11:in `new'
    from /var/lib/gems/1.9.1/gems/oxidized-0.0.63/lib/oxidized/core.rb:11:in `new'
    from /var/lib/gems/1.9.1/gems/oxidized-0.0.63/lib/oxidized/cli.rb:9:in `run'
    from /var/lib/gems/1.9.1/gems/oxidized-0.0.63/bin/oxidized:8:in `<top (required)>'
    from /usr/local/bin/oxidized:23:in `load'
    from /usr/local/bin/oxidized:23:in `<main>'
oxidized@ubuntu:~$
@ytti
Copy link
Owner

ytti commented Jul 11, 2014

Will be fixed by grit => rugged migration, as rugged does not require git being installed

@ytti ytti closed this as completed in 2110aa9 Jul 13, 2014
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

No branches or pull requests

2 participants