all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Stephen Leake <stephen_leake@stephe-leake.org>
To: emacs-devel <emacs-devel@gnu.org>
Subject: Re: master has switched from Automake to GNU Make
Date: Fri, 24 Mar 2017 18:35:08 -0500	[thread overview]
Message-ID: <86inmyp7tf.fsf@stephe-leake.org> (raw)
In-Reply-To: <58D56B33.9050408@gmx.at> (martin rudalics's message of "Fri, 24 Mar 2017 19:53:39 +0100")

martin rudalics <rudalics@gmx.at> writes:

> All this is about the final result of what a branch was supposed to
> accomplish and I agree with you here.  

Ok.

> My concerns were with what happens in the branch as long as it is not
> final. Why throw away its history when pushing? Because it contains
> silly code its author would be ashamed of?

Yes, but I would phrase it "it contained a failed approach, so here's a
better approach".

Or, "the sequence of commits is too hard to follow, since it has false
starts and back-tracking, so here's a cleaner sequence".

One could argue that new approach should have yet another branch. Which
is a good idea if you are not _really_ sure the second approach is
better, or will work.

Partly it depends on how closely people are cooperating on the branch.
If it's two or more developers actively working on all the code, then
pushing a totally new set of commits is a Bad Idea; the other developers
will lose work (or time in recovering).

But our presumed use case is one developer actively working on the code,
and asking for reviews. In which case, a clean sequence of
understandable commits is a big plus.

-- 
-- Stephe



  reply	other threads:[~2017-03-24 23:35 UTC|newest]

Thread overview: 90+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-03-17  8:33 No rule to make target `../../build-aux/snippet/arg-nonnull.h' martin rudalics
2017-03-17  9:16 ` Eli Zaretskii
2017-03-17  9:39   ` martin rudalics
2017-03-17 10:00     ` Paul Eggert
2017-03-17 10:37       ` martin rudalics
2017-03-17 15:55         ` Paul Eggert
2017-03-17 18:01           ` martin rudalics
2017-03-17 18:58           ` master has switched from Automake to GNU Make Paul Eggert
2017-03-17 19:26             ` Eli Zaretskii
2017-03-17 19:33               ` Paul Eggert
2017-03-17 19:51                 ` Eli Zaretskii
2017-03-17 20:21                   ` Kaushal Modi
2017-03-17 21:45                     ` Paul Eggert
2017-03-17 21:33                   ` Paul Eggert
2017-03-18  8:01                     ` Eli Zaretskii
2017-03-18 16:26                       ` Paul Eggert
2017-03-18 16:47                         ` Eli Zaretskii
2017-03-18  8:11             ` martin rudalics
2017-03-18  9:28               ` Eli Zaretskii
2017-03-18 10:33                 ` martin rudalics
2017-03-20 11:26             ` Robert Marshall
2017-03-20 15:59               ` Paul Eggert
2017-03-21  8:31                 ` Robert Marshall
2017-03-21 17:42                 ` Tom Tromey
2017-03-21 18:23                   ` Paul Eggert
2017-03-21 19:01                   ` Eli Zaretskii
2017-03-23  8:02             ` martin rudalics
2017-03-23 11:41               ` Stefan Monnier
2017-03-23 13:06                 ` martin rudalics
2017-03-23 15:44                   ` Stefan Monnier
2017-03-24  9:03                     ` martin rudalics
2017-03-24 12:04                       ` Stefan Monnier
2017-03-24 13:25                         ` martin rudalics
2017-03-24 13:42                           ` Stefan Monnier
2017-03-24 13:42                       ` Lars Brinkhoff
2017-03-24 18:53                         ` martin rudalics
2017-03-24 23:35                           ` Stephen Leake [this message]
2017-03-25  9:28                             ` martin rudalics
2017-03-26 12:58                               ` Stephen Leake
2017-03-26 14:31                                 ` Eli Zaretskii
2017-03-24 14:16                       ` Filipp Gunbin
2017-03-24 18:53                         ` martin rudalics
2017-03-24 23:27                       ` Stephen Leake
2017-03-24 23:40                         ` Stefan Monnier
2017-03-25  9:14                           ` Andreas Schwab
2017-03-25 17:36                             ` Stefan Monnier
2017-03-25 17:57                               ` Elias Mårtenson
2017-03-25  9:28                           ` martin rudalics
2017-03-25  9:26                         ` martin rudalics
2017-03-25 15:24                           ` Paul Eggert
2017-03-25 15:55                             ` Eli Zaretskii
2017-03-26  8:39                               ` martin rudalics
2017-03-26 14:18                                 ` Eli Zaretskii
2017-03-26 19:01                                   ` martin rudalics
2017-03-26 19:16                                     ` Eli Zaretskii
2017-03-26 19:33                                       ` martin rudalics
2017-03-27 14:53                                         ` Eli Zaretskii
2017-03-27 19:29                                           ` martin rudalics
2017-03-27 19:33                                             ` Eli Zaretskii
2017-03-26  8:38                             ` martin rudalics
2017-03-26 13:03                               ` Stephen Leake
2017-03-26 14:31                                 ` Eli Zaretskii
2017-04-11 10:55                             ` martin rudalics
2017-04-11 12:13                               ` Andreas Schwab
2017-04-11 13:17                                 ` Yuri Khan
2017-04-11 13:53                                   ` martin rudalics
2017-04-11 14:07                                     ` Eli Zaretskii
2017-04-11 15:13                                     ` Yuri Khan
2017-04-11 15:31                                       ` Sven Axelsson
2017-04-12  9:30                                         ` martin rudalics
2017-04-12  9:30                                       ` martin rudalics
2017-04-11 18:19                                     ` Sven Joachim
2017-04-11 13:23                                 ` martin rudalics
2017-04-11 15:45                                   ` Andreas Schwab
2017-04-11 18:00                                     ` martin rudalics
2017-04-11 18:06                                       ` martin rudalics
2017-04-11 13:34                               ` Noam Postavsky
2017-04-11 13:54                                 ` martin rudalics
2017-04-11 14:09                                   ` Noam Postavsky
2017-04-12  9:28                                     ` martin rudalics
2017-04-12 12:18                                       ` Michal Nazarewicz
2017-04-12 14:42                                         ` Thien-Thi Nguyen
2017-04-12 14:38                                       ` Paul Eggert
2017-04-11 16:21                                   ` Michal Nazarewicz
2017-04-11 16:24                                     ` Noam Postavsky
2017-04-12 12:11                                       ` Michal Nazarewicz
2017-03-25 16:46                           ` Andreas Politz
2017-03-24 17:56                     ` Stephen Leake
2017-03-23 15:27               ` Eli Zaretskii
2017-03-24  9:02                 ` martin rudalics

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=86inmyp7tf.fsf@stephe-leake.org \
    --to=stephen_leake@stephe-leake.org \
    --cc=emacs-devel@gnu.org \
    /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.