Skip to content
This repository has been archived by the owner on May 12, 2018. It is now read-only.

Add new method to Gitlab::Git::Repository #52

Merged
merged 1 commit into from Jun 20, 2015

Conversation

mr-vinn
Copy link
Contributor

@mr-vinn mr-vinn commented Oct 18, 2014

Add a new convenience method, #enable_autocrlf, to the Repository class. The new method sets the core.autocrlf config option to true for a repo. This can be used by GitLab in satellite repos to handle the \r\n line endings that get written when editing via the web interface. This is part of the fix for gitlabhq/gitlabhq#7950.

Add a new convenience method, `#enable_autocrlf`, to the Repository
class.  The new method sets the `core.autocrlf` config option to true
for a repo.
@coveralls
Copy link

Coverage Status

Coverage increased (+0.03%) when pulling bfbd5a5 on mr-vinn:enable-autocrlf into cba8de9 on gitlabhq:master.

@dosire
Copy link
Member

dosire commented Jun 17, 2015

Thanks for the @mr-vinn

@jacobvosmaer
Copy link
Contributor

Cool, this might fix https://gitlab.com/gitlab-org/gitlab-ce/issues/1823

The only thing I am not sure about is whether Rugged/libgit2 respects core.autocrlf internally. We might want to try that first. cc @DouweM

@DouweM
Copy link
Contributor

DouweM commented Jun 20, 2015

@jacobvosmaer Confirmed to work. I'm merging this, and going through the motions to get this into gitlab-ce and use it there.

Edit: Whoops, I was too quick to say it worked. It's not working (yet). I'll continue to look into it.

DouweM added a commit that referenced this pull request Jun 20, 2015
Add new method to Gitlab::Git::Repository
@DouweM DouweM merged commit bc81982 into gitlabhq:master Jun 20, 2015
@DouweM
Copy link
Contributor

DouweM commented Jun 20, 2015

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
5 participants