git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [GUILT v3 00/31] Teach guilt import-commit how to create legal patch names, and more
@ 2014-05-16 14:45 Per Cederqvist
  2014-05-16 14:45 ` [GUILT v3 01/31] The tests should not fail if guilt.diffstat is set Per Cederqvist
                   ` (30 more replies)
  0 siblings, 31 replies; 41+ messages in thread
From: Per Cederqvist @ 2014-05-16 14:45 UTC (permalink / raw)
  To: Jeff Sipek; +Cc: git, Per Cederqvist

This is version 3 of the patch series I sent back on 21 Mar 2014.  I
have addressed all feedback to date, updated the coding style, and
added signed-off-by lines from Jeff Sipek for those commits that I
have received an explicit approval from him (but only if I have not
made any other change to that particular commit).

I have added two new patches:

* Patch 28/31 fixes coding style issues in t-061.sh.  I inserted it
before the new patch 29/31 since 29/31 copies t-061.sh as t-062.sh,
and by fixing the style issues in the origin it is easier to track the
changes.  This means that v2 patches 28 and 29 are now numbered 29 and
30.  Sorry about that.

* Patch 31/31 removes all use of "git log -p" in the test suite.

To see how the patches have evolved, you might find
http://www.lysator.liu.se/~ceder/guilt-oslo-2014-v3/ useful.  It
displays diffs of all the patches between v2 and v3, in pdiffdiff
output format.

Here is the original blurb for the series, slightly edited:

I recently found myself sitting on a train with a computer in front of
me.  I tried to use "guilt import-commit", which seemed to work, but
when I tried to "guilt push" the commits I had just imported I got
some errors.  It turned out that "guilt import-commit" had generated
invalid patch names.

I decided to fix the issue, and write a test case that demonstrated
the problem.

One thing led to another, and here I am, a few late nights at a hotel
and a return trip on the train later, submitting a patch series in 28
parts.  Sorry about the number of patches, but this is what happens
when you uncover a bug while writing a test case for the bug you
uncovered while writing a test case for your original problem.

The patch series consists of:

 - A number of fixes to the test suite.

 - A number of bug fixes which I hope are non-controversial.  Most of
   the fixes have test cases.

 - Changed behavior: "guilt push" when there is nothing more to push
   now uses exit status 1.  This makes it possible to write shell
   loops such as "while guilt push; do make test||break; done".  Also,
   "guilt pop" when no patches are applied also uses exit status 1.
   (This aligns "guilt push" and "guilt pop" with how "hg qpush" and
   "hg qpop" has worked for several years.)

 - Changed behavior: by default, guilt no longer changes branch when
   you push a patch.  You need to do "git config guilt.reusebranch
   false" to re-enable that.  This patch sets the default value of
   guilt.reusebranch to true; it should in my opinion change to false
   a year or two after the next release.

 - The humble beginnings of a coding style guide.

A more detailed overview of the patches:

1. Some tests fail if "git config guilt.diffstat true" is in effect.

2-4. Some commands fail if run from a directory with spaces in its
     name.

5. "guilt new" had an overly restrictive argument parser.

6-8. guilt.diffstat could break "guilt fold" and "guilt new".

9-10. The test suite did not test exit status properly.

11. Remove pointless redirections in the test suite.

12-13. "guilt header" tried to check if a patch existed, but the check
        was broken.

14-16. Various parts of guilt tried to ensure that patch names were
       legal git branch names, but failed.

17-20. "guilt graph" failed when no patch was applied, and when a
       branch name contained a comma or a quote.

21-23. "git config log.decorate short" caused "guilt import-commit",
       "guilt patchbomb" and "guilt rebase" to fail in various ways.

24. Patches may contain backslashes, but various informative messages
    from guilt did backslash processing.

25-26. "guilt push" and "guilt pop" should fail when there is nothing
       to do.

28. Fix coding style problems in a test case.

27, 29. These two commits were things I intended to contribute a while
       back, when contributing the "Change git branch when patches are
       applied" change (commit 67d3af63f422).  These commits makes
       that behavior optional, and it defaults to being disabled, so
       that you can use both Guilt v0.35 (and earlier) and the current
       Guilt code against the same repo.  These commits add some code
       complexity, and you might want to skip them if you think the
       current behavior is better.

30. A coding style guide, with Emacs support.

31. Avoid using "git log -p".

This patch series is also available on
http://repo.or.cz/w/guilt/ceder.git in the "oslo-2014-v3" branch.  If
you already have a copy of guilt, you should be able to fetch that
branch with something like:

    git remote add ceder http://repo.or.cz/r/guilt/ceder.git
    git fetch ceder refs/heads/oslo-2014-v3:refs/remotes/ceder/oslo-2014-v3

A few of the regression/t-*.out files contain non-ASCII characters.  I
hope they survive the mail transfer; if not, please use the repo above
to fetch the commits.

Per Cederqvist (31):
  The tests should not fail if guilt.diffstat is set.
  Allow "guilt delete -f" to run from a dir which contains spaces.
  Added test case for "guilt delete -f".
  Allow "guilt import-commit" to run from a dir which contains spaces.
  "guilt new": Accept more than 4 arguments.
  Fix the do_get_patch function.
  Added test cases for "guilt fold".
  Added more test cases for "guilt new": empty patches.
  Test suite: properly check the exit status of commands.
  Run test_failed if the exit status of a test script is bad.
  test suite: remove pointless redirection.
  "guilt header": more robust header selection.
  Check that "guilt header '.*'" fails.
  Use "git check-ref-format" to validate patch names.
  Produce legal patch names in guilt-import-commit.
  Fix backslash handling when creating names of imported patches.
  "guilt graph" no longer loops when no patches are applied.
  guilt-graph: Handle commas in branch names.
  Check that "guilt graph" works when working on a branch with a comma.
  "guilt graph": Handle patch names containing quotes.
  The log.decorate setting should not influence import-commit.
  The log.decorate setting should not influence patchbomb.
  The log.decorate setting should not influence guilt rebase.
  disp no longer processes backslashes.
  "guilt push" now fails when there are no more patches to push.
  "guilt pop" now fails when there are no more patches to pop.
  Minor testsuite fix.
  Fix coding style errors in t-061.sh.
  Added guilt.reusebranch configuration option.
  Added a short style guide, and Emacs settings.
  Don't use "git log -p" in the test suite.

 .dir-locals.el             |   3 +
 Documentation/Contributing |  15 +
 guilt                      |  64 +++-
 guilt-delete               |   2 +-
 guilt-fork                 |   2 +-
 guilt-graph                |  13 +-
 guilt-header               |  29 +-
 guilt-import               |   2 +-
 guilt-import-commit        |  30 +-
 guilt-new                  |   9 +-
 guilt-patchbomb            |   2 +-
 guilt-pop                  |  17 +-
 guilt-push                 |  19 +-
 guilt-rebase               |   2 +-
 regression/run-tests       |  10 +-
 regression/scaffold        |  19 +-
 regression/t-020.out       | 924 +++++++--------------------------------------
 regression/t-020.sh        |  73 +++-
 regression/t-021.out       | 901 +------------------------------------------
 regression/t-021.sh        |  18 +-
 regression/t-025.out       | 426 ++++++++++++++++++++-
 regression/t-025.sh        |  14 +-
 regression/t-026.out       |  15 +
 regression/t-026.sh        |   5 +-
 regression/t-028.out       |   7 +
 regression/t-028.sh        |   6 +-
 regression/t-032.out       |   4 +-
 regression/t-032.sh        |   2 +-
 regression/t-033.out       |  90 +++++
 regression/t-033.sh        |  61 +++
 regression/t-034.out       | 569 ++++++++++++++++++++++++++++
 regression/t-034.sh        |  73 ++++
 regression/t-035.out       | 467 +++++++++++++++++++++++
 regression/t-035.sh        |  61 +++
 regression/t-061.out       |   1 -
 regression/t-061.sh        |  12 +-
 regression/t-062.out       | 420 +++++++++++++++++++++
 regression/t-062.sh        | 130 +++++++
 38 files changed, 2745 insertions(+), 1772 deletions(-)
 create mode 100644 .dir-locals.el
 create mode 100644 regression/t-033.out
 create mode 100755 regression/t-033.sh
 create mode 100644 regression/t-034.out
 create mode 100755 regression/t-034.sh
 create mode 100644 regression/t-035.out
 create mode 100755 regression/t-035.sh
 create mode 100644 regression/t-062.out
 create mode 100755 regression/t-062.sh

-- 
1.8.3.1

^ permalink raw reply	[flat|nested] 41+ messages in thread

end of thread, other threads:[~2014-05-18 19:13 UTC | newest]

Thread overview: 41+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-05-16 14:45 [GUILT v3 00/31] Teach guilt import-commit how to create legal patch names, and more Per Cederqvist
2014-05-16 14:45 ` [GUILT v3 01/31] The tests should not fail if guilt.diffstat is set Per Cederqvist
2014-05-16 14:45 ` [GUILT v3 02/31] Allow "guilt delete -f" to run from a dir which contains spaces Per Cederqvist
2014-05-16 14:45 ` [GUILT v3 03/31] Added test case for "guilt delete -f" Per Cederqvist
2014-05-16 14:45 ` [GUILT v3 04/31] Allow "guilt import-commit" to run from a dir which contains spaces Per Cederqvist
2014-05-16 14:45 ` [GUILT v3 05/31] "guilt new": Accept more than 4 arguments Per Cederqvist
2014-05-16 14:45 ` [GUILT v3 06/31] Fix the do_get_patch function Per Cederqvist
2014-05-16 14:45 ` [GUILT v3 07/31] Added test cases for "guilt fold" Per Cederqvist
2014-05-16 14:45 ` [GUILT v3 08/31] Added more test cases for "guilt new": empty patches Per Cederqvist
2014-05-16 15:01   ` Jeff Sipek
2014-05-16 14:45 ` [GUILT v3 09/31] Test suite: properly check the exit status of commands Per Cederqvist
2014-05-16 15:45   ` Jeff Sipek
2014-05-18 19:12     ` Per Cederqvist
2014-05-16 14:45 ` [GUILT v3 10/31] Run test_failed if the exit status of a test script is bad Per Cederqvist
2014-05-16 14:45 ` [GUILT v3 11/31] test suite: remove pointless redirection Per Cederqvist
2014-05-16 14:45 ` [GUILT v3 12/31] "guilt header": more robust header selection Per Cederqvist
2014-05-16 15:22   ` Jeff Sipek
2014-05-16 14:46 ` [GUILT v3 13/31] Check that "guilt header '.*'" fails Per Cederqvist
2014-05-16 14:46 ` [GUILT v3 14/31] Use "git check-ref-format" to validate patch names Per Cederqvist
2014-05-16 15:20   ` Jeff Sipek
2014-05-18 18:59     ` Per Cederqvist
2014-05-16 14:46 ` [GUILT v3 15/31] Produce legal patch names in guilt-import-commit Per Cederqvist
2014-05-16 14:46 ` [GUILT v3 16/31] Fix backslash handling when creating names of imported patches Per Cederqvist
2014-05-16 14:46 ` [GUILT v3 17/31] "guilt graph" no longer loops when no patches are applied Per Cederqvist
2014-05-16 14:46 ` [GUILT v3 18/31] guilt-graph: Handle commas in branch names Per Cederqvist
2014-05-16 14:46 ` [GUILT v3 19/31] Check that "guilt graph" works when working on a branch with a comma Per Cederqvist
2014-05-16 14:46 ` [GUILT v3 20/31] "guilt graph": Handle patch names containing quotes Per Cederqvist
2014-05-16 14:46 ` [GUILT v3 21/31] The log.decorate setting should not influence import-commit Per Cederqvist
2014-05-16 14:46 ` [GUILT v3 22/31] The log.decorate setting should not influence patchbomb Per Cederqvist
2014-05-16 14:46 ` [GUILT v3 23/31] The log.decorate setting should not influence guilt rebase Per Cederqvist
2014-05-16 14:46 ` [GUILT v3 24/31] disp no longer processes backslashes Per Cederqvist
2014-05-16 14:46 ` [GUILT v3 25/31] "guilt push" now fails when there are no more patches to push Per Cederqvist
2014-05-16 14:46 ` [GUILT v3 26/31] "guilt pop" now fails when there are no more patches to pop Per Cederqvist
2014-05-16 14:46 ` [GUILT v3 27/31] Minor testsuite fix Per Cederqvist
2014-05-16 14:46 ` [GUILT v3 28/31] Fix coding style errors in t-061.sh Per Cederqvist
2014-05-16 15:23   ` Jeff Sipek
2014-05-16 14:46 ` [GUILT v3 29/31] Added guilt.reusebranch configuration option Per Cederqvist
2014-05-16 15:26   ` Jeff Sipek
2014-05-16 14:46 ` [GUILT v3 30/31] Added a short style guide, and Emacs settings Per Cederqvist
2014-05-16 14:46 ` [GUILT v3 31/31] Don't use "git log -p" in the test suite Per Cederqvist
2014-05-16 15:36   ` Jeff Sipek

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).