Skip to content

Ensure that conflicts are staged and never ignored#1062

Merged
nulltoken merged 4 commits intovNextfrom
ethomson/stage_ignores
Jun 11, 2015
Merged

Ensure that conflicts are staged and never ignored#1062
nulltoken merged 4 commits intovNextfrom
ethomson/stage_ignores

Conversation

@ethomson
Copy link
Member

@ethomson ethomson commented May 29, 2015

Like libgit2/libgit2#3139, LibGit2Sharp erroneously does not stage conflicted files, if the filenames match .gitignore. Update stage to take conflict data into account.

@ethomson
stage tests: stage ignored files properly
2b03d65 
Staging should ignore files that are ignored, unless the file
already exists in the repository.  Additional tests to validate
this.
@ethomson ethomson force-pushed the ethomson/stage_ignores branch 2 times, most recently from 168aa2c to 92946fe Compare May 30, 2015 16:06
@ethomson
stage tests: ensure we can stage conflicts
02940ce 
A file should be ignored when it does not exist in the repository,
but conflicts should absolutely qualify as "existing in the
repository".  Ensure that we can stage ignored files correctly.
@ethomson ethomson force-pushed the ethomson/stage_ignores branch from 92946fe to 6b0a0a5 Compare May 30, 2015 16:18
@ethomson
Copy link
Member Author

ethomson commented May 30, 2015

You know, some days I forget that we're building a tool that actively punishes Windows users for using two bytes to represent line endings instead of one.

@ethomson ethomson force-pushed the ethomson/stage_ignores branch from 6b0a0a5 to 3d5ddee Compare June 10, 2015 22:29
@ethomson
Copy link
Member Author

ethomson commented Jun 10, 2015

So after digging in here: the mergedrepo_wd repository was created and has no explicit core.autocrlf settings. The actual contents of the repository have CRLF checked in. This (thankfully) matches the working directory copies. This is akin to core.autocrlf=false.

@ethomson
Copy link
Member Author

ethomson commented Jun 11, 2015

In case it's not clear, I favor a comprehensive solution to the tests issue at hand (the issue being that it's not super clear what the EOL settings are). But this seems like a reasonable stopgap measure until we can be more comprehensive.

nulltoken
nulltoken reviewed Jun 11, 2015
View reviewed changes
LibGit2Sharp/TreeEntryChanges.cs Outdated
Copy link
Member

@nulltoken nulltoken Jun 11, 2015

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would there be a way to improve this documentation to help the user understand what we mean by "exists"?

I had to go through libgit2/libgit2#3139 in order to actually get it.

Copy link
Member Author

@ethomson ethomson Jun 11, 2015

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added additional documentation here, does this clarify this concept?

@nulltoken
Copy link
Member

nulltoken commented Jun 11, 2015

Would this fix #881?

@Therzok
Copy link
Member

Therzok commented Jun 11, 2015

3cfb6f7 Fixes #881.

@ethomson ethomson force-pushed the ethomson/stage_ignores branch from 3d5ddee to 0a72fc9 Compare June 11, 2015 13:56
@nulltoken nulltoken mentioned this pull request Jun 11, 2015
Merged
@ethomson ethomson mentioned this pull request Jun 11, 2015
Closed
nulltoken
nulltoken reviewed Jun 11, 2015
View reviewed changes
LibGit2Sharp/TreeEntryChanges.cs Outdated
Copy link
Member

@nulltoken nulltoken Jun 11, 2015

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s/flase/false/

Copy link
Member Author

@ethomson ethomson Jun 11, 2015

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed.

Edward Thomson added 2 commits June 11, 2015 11:01
Introduce FileStatus.Conflicted
47b7c58 
Introduce conflict data to status and diff information.  Introduce
staging of conflicts.
mergedrepo_wd: turn on core.autocrlf=false
49f9530
@ethomson ethomson force-pushed the ethomson/stage_ignores branch from 0a72fc9 to 49f9530 Compare June 11, 2015 15:16
@nulltoken
Copy link
Member

nulltoken commented Jun 11, 2015

Awesome. Thanks!

nulltoken added a commit that referenced this pull request Jun 11, 2015
@nulltoken
Merge pull request #1062 from libgit2/ethomson/stage_ignores
49346f8 
Ensure that conflicts are staged and never ignored
@nulltoken nulltoken merged commit 49346f8 into vNext Jun 11, 2015
@nulltoken nulltoken deleted the ethomson/stage_ignores branch June 11, 2015 15:50
@nulltoken nulltoken added this to the v0.22 milestone Jun 11, 2015
@rubberduck203 rubberduck203 mentioned this pull request Mar 5, 2016
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

v0.22

Development

Successfully merging this pull request may close these issues.

3 participants

@ethomson @nulltoken @Therzok