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.

Sign up for GitHub

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

Jump to bottom

[1.0.1] Rewrite Branch is not necessary and should not be created by the application #5

Closed
renelink opened this issue Jan 5, 2015 · 0 comments
Closed

[1.0.1] Rewrite Branch is not necessary and should not be created by the application #5

renelink opened this issue Jan 5, 2015 · 0 comments
Assignees
@renelink
Labels
enhancement

Comments

@renelink
Copy link
Contributor

renelink commented Jan 5, 2015

GitDirStat creates a rewrite branch that is resettted during the rewrite and points to the commits to rewrite.

The rewrite branch was introduced, because the implementation was easier when using the https://wiki.eclipse.org/JGit/User_Guide#Porcelain_API or parts of that api.

Now that I explored the JGIT api in depth and therefore know how to use the lower level api the rewrite branch can be removed. Rewrite the history can be easily done in detatched HEAD state.

Removing the rewrite branch also removes error sources, because a branch with the same name might already exist.
renelink added a commit that referenced this issue Jan 5, 2015
@renelink
Fix for #5
c1b0abc 
Refactored RewriteBranch to RewriteRef in order to prepare for detached
HEAD rewrite.

Moved branch recovery to HistoryUpdate

Improved HistoryUpdate, CacheCommitUpdate and RewriteRef

Renamed RewriteRef to IndexUpdate

Rewrite branch is not needed anymore, because we only use the index now
@renelink renelink closed this as completed Jan 5, 2015
@renelink renelink self-assigned this Jan 5, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@renelink renelink

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@renelink