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

Centos Unknow Option -c #220

Closed
sogos opened this issue Dec 11, 2012 · 4 comments
Closed

Centos Unknow Option -c #220

sogos opened this issue Dec 11, 2012 · 4 comments

Comments

@sogos
Copy link

sogos commented Dec 11, 2012

Hello,

Runtime Error at first Lunch

In
vendor/klaussilveira/gitter/lib/Gitter/Client.php

Line 136
$process = new Process($this->getPath() . ' -c "color.ui"=false ' . $command, $repository->getPath());

If I remove "-c "color.ui"=false ", it's ok

@DannyvdSluijs
Copy link
Collaborator

This isn't a Gitlist issue, it's cause can be found in Gitter, a package used by Gitlist. Also it isn't an issue, but having version v1.7. of git on your system is a requirement. This requirement needs to be added to Gitter.

See: klaussilveira/gitter#9

@sogos
Copy link
Author

sogos commented Dec 11, 2012

Hum probably this requirements is not a good option (as long git 1.7.1 is in Centos 6.3 repo), according to me it only concern color in command line.
It would be easier for community to have a git version checker to remove the -c option

@klaussilveira
Copy link
Owner

This will be fixed in the next gitter version, therefore in future versions of GitList as well. For a quick fix, upgrade your git version.

@MSylvia
Copy link

MSylvia commented Jan 11, 2013

I just edited line # 134 in Client.php to

//$process = new Process($this->getPath() . ' -c "color.ui"=false ' . $command, $repository->getPath());
$process = new Process($this->getPath() . ' ' . $command, $repository->getPath());

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

4 participants