This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: Commit hook annoyance
- From: Eli Zaretskii <eliz at gnu dot org>
- To: "H.J. Lu" <hjl dot tools at gmail dot com>
- Cc: brobecker at adacore dot com, gdb-patches at sourceware dot org
- Date: Fri, 16 Jan 2015 18:38:44 +0200
- Subject: Re: Commit hook annoyance
- Authentication-results: sourceware.org; auth=none
- References: <83ppae1zsq dot fsf at gnu dot org> <CAMe9rOrGkdBEyQ8O-z_Ud3YuVRGwuGS62eYJos82iL_kRgC-MQ at mail dot gmail dot com>
- Reply-to: Eli Zaretskii <eliz at gnu dot org>
> Date: Fri, 16 Jan 2015 08:10:55 -0800
> From: "H.J. Lu" <hjl.tools@gmail.com>
> Cc: Joel Brobecker <brobecker@adacore.com>, GDB <gdb-patches@sourceware.org>
>
> On Fri, Jan 16, 2015 at 8:00 AM, Eli Zaretskii <eliz@gnu.org> wrote:
> > Is this really necessary?
> >
> > $ git push
> > Counting objects: 94, done.
> > Compressing objects: 100% (4/4), done.
> > Writing objects: 100% (4/4), 742 bytes | 0 bytes/s, done.
> > Total 4 (delta 3), reused 0 (delta 0)
> > remote: *** Pattern "Conflicts:" has been detected.
> > remote: *** (in commit 7fa8e2a29c9357024d21bdf09faa5cb930a4ffe5)
> > remote: ***
> > remote: *** This usually indicates a merge commit where some merge conflicts
> > remote: *** had to be resolved, but where the "Conflicts:" section has not
> > remote: *** been deleted from the revision history.
> > remote: ***
> > remote: *** Please edit the commit's revision history to either delete
> > remote: *** the section, or to avoid using the pattern above by itself.
> > remote: error: hook declined to update refs/heads/gdb-7.9-branch
> > To git+ssh://sourceware.org/git/binutils-gdb.git
> > ! [remote rejected] gdb-7.9-branch -> gdb-7.9-branch (hook declined)
> > error: failed to push some refs to 'git+ssh://sourceware.org/git/binutils-gdb.git'
> >
> > There was no unresolved conflicts in the commit I tried to push. Why
> > does the hook insist on forcing me to remove the "Conflicts:" part,
> > instead of looking for the merge-commit conflict markers? Why do we
> > care about leaving the reference to original conflicts in the log?
>
> Do you have a commit which has a conflict and is fixed by another
> commit?
No, it was a cherry-pick that created a conflict, whih I resolved
before committing. But I didn't bother to remove the "Conflicts:"
part from the automatic commit message that Git lets me edit, when I
invoke "git cherry-pick -e".