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

Pull request fail to merge with BIN #1922

Closed
wsantos opened this issue Nov 10, 2015 · 3 comments
Closed

Pull request fail to merge with BIN #1922

wsantos opened this issue Nov 10, 2015 · 3 comments
Labels
💊 bug Something isn't working status: needs feedback Tell me more about it
Milestone

Comments

@wsantos
Copy link

wsantos commented Nov 10, 2015

I've tried to merge a PR within a binary and it says that PR has confict, after futher investigation I've found this problems:

1 - Patch file is incorrect, maye we should use --binary and --full-index in patch creation

diff --git a/containers/galera.ca/files/xz-5.2.2.tar.xz b/containers/galera.ca/files/xz-5.2.2.tar.xz
new file mode 100644
index 0000000..2997653
Binary files /dev/null and b/containers/galera.ca/files/xz-5.2.2.tar.xz differ

2 - If you try to apply the patch manually we receive this error:

error: cannot apply binary patch to 'containers/galera.ca/files/xz-5.2.2.tar.xz'; without full index line
error: containers/galera.ca/files/xz-5.2.2.tar.xz: patch does not apply

Suggestions: We should be able to see git commands output and also the full command in logs if debug is enabled, it would help a lot to track down problems between git+gogs.

@unknwon unknwon added the 💊 bug Something isn't working label Nov 11, 2015
@unknwon unknwon added this to the 0.7.x milestone Nov 11, 2015
@unknwon
Copy link
Member

unknwon commented Nov 11, 2015

Thanks your feedback!

@unknwon unknwon added the status: needs feedback Tell me more about it label Nov 17, 2015
@unknwon
Copy link
Member

unknwon commented Nov 17, 2015

Hi, I've pushed fix to develop, add --binary should solve the problem (it includes --full-index by default I think). And print full output when there is conflict. You can test on https://try.gogs.io

@unknwon
Copy link
Member

unknwon commented Nov 20, 2015

Close as fixed.

@unknwon unknwon closed this as completed Nov 20, 2015
@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 11, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
💊 bug Something isn't working status: needs feedback Tell me more about it
Projects
None yet
Development

No branches or pull requests

2 participants