all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: Stephen Berman <stephen.berman@gmx.net>
Cc: emacs-devel@gnu.org
Subject: Re: Commit mistake?
Date: Sun, 25 Nov 2018 18:34:50 +0200	[thread overview]
Message-ID: <83bm6djetx.fsf@gnu.org> (raw)
In-Reply-To: <87pnutqol6.fsf@gmx.net> (message from Stephen Berman on Sun, 25 Nov 2018 14:22:13 +0100)

> From: Stephen Berman <stephen.berman@gmx.net>
> Date: Sun, 25 Nov 2018 14:22:13 +0100
> 
> I just committed a fix for bug#33423 to master (29ef7d1), and somehow
> the commit included an entire unrelated file, nt/gnulib.mk.  I don't
> know how this happened, because that is a file I'm not aware of ever
> having touched, let alone changed (it's a generated file).  I don't know
> if this is a problem but if it is, can someone who knows what to do
> please take care of it, or tell me how to?

Just delete that file, it should not be in the repository.  "git rm"
etc. (Let me know if you need more detailed instructions.)

> And also tell me how to avoid a possible reoccurrence of in future
> commits?  Thanks and sorry for the nuisance.

The only reliable way I know of is to say "git show" after committing
and before pushing, and review the diffs carefully making sure the
commit only includes what I meant it to.



  reply	other threads:[~2018-11-25 16:34 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-11-25 13:22 Commit mistake? Stephen Berman
2018-11-25 16:34 ` Eli Zaretskii [this message]
2018-11-25 16:47   ` Stephen Berman
2018-11-25 17:06     ` Eli Zaretskii
2018-11-25 16:49 ` Stefan Monnier
2018-11-25 17:09   ` Stephen Berman
2018-11-25 18:16     ` Stefan Monnier
2018-11-25 18:42       ` Stephen Berman
2018-11-25 19:53         ` Stefan Monnier

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=83bm6djetx.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=emacs-devel@gnu.org \
    --cc=stephen.berman@gmx.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.