Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 4 Apr 2019 17:21:31 +0000 (UTC)
From:      Kyle Evans <kevans@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-12@freebsd.org
Subject:   svn commit: r345878 - in stable: 11/usr.bin/patch 12/usr.bin/patch
Message-ID:  <201904041721.x34HLVxW030072@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: kevans
Date: Thu Apr  4 17:21:30 2019
New Revision: 345878
URL: https://svnweb.freebsd.org/changeset/base/345878

Log:
  MFC r344677: patch(1): Exit successfully if we're fed a 0-length patch
  
  This change is made in the name of GNU patch compatibility. If GNU patch is
  fed a zero-length patch, it will exit successfully with no output. This is
  used in at least one port to date (comms/wsjtx), and we break on this usage.
  
  It seems unlikely that anyone relies on patch(1) calling their completely
  empty patch garbage and failing, and GNU compatibility is a plus if it helps
  with porting, so make the switch.

Modified:
  stable/12/usr.bin/patch/common.h
  stable/12/usr.bin/patch/patch.c
  stable/12/usr.bin/patch/pch.c
Directory Properties:
  stable/12/   (props changed)

Changes in other areas also in this revision:
Modified:
  stable/11/usr.bin/patch/common.h
  stable/11/usr.bin/patch/patch.c
  stable/11/usr.bin/patch/pch.c
Directory Properties:
  stable/11/   (props changed)

Modified: stable/12/usr.bin/patch/common.h
==============================================================================
--- stable/12/usr.bin/patch/common.h	Thu Apr  4 17:20:55 2019	(r345877)
+++ stable/12/usr.bin/patch/common.h	Thu Apr  4 17:21:30 2019	(r345878)
@@ -64,6 +64,7 @@ extern size_t	buf_size;	/* size of general purpose buf
 
 extern bool	using_plan_a;	/* try to keep everything in memory */
 extern bool	out_of_mem;	/* ran out of memory in plan a */
+extern bool	nonempty_patchf_seen;	/* seen a non-zero-length patch file? */
 
 #define	MAXFILEC 2
 

Modified: stable/12/usr.bin/patch/patch.c
==============================================================================
--- stable/12/usr.bin/patch/patch.c	Thu Apr  4 17:20:55 2019	(r345877)
+++ stable/12/usr.bin/patch/patch.c	Thu Apr  4 17:21:30 2019	(r345878)
@@ -53,6 +53,7 @@ size_t		buf_size;		/* size of the general purpose buff
 
 bool		using_plan_a = true;	/* try to keep everything in memory */
 bool		out_of_mem = false;	/* ran out of memory in plan a */
+bool		nonempty_patchf_seen = false;	/* seen nonempty patch file? */
 
 #define MAXFILEC 2
 
@@ -419,7 +420,7 @@ main(int argc, char *argv[])
 		set_signals(1);
 	}
 
-	if (!patch_seen)
+	if (!patch_seen && nonempty_patchf_seen)
 		error = 2;
 
 	my_exit(error);

Modified: stable/12/usr.bin/patch/pch.c
==============================================================================
--- stable/12/usr.bin/patch/pch.c	Thu Apr  4 17:20:55 2019	(r345877)
+++ stable/12/usr.bin/patch/pch.c	Thu Apr  4 17:21:30 2019	(r345878)
@@ -179,6 +179,9 @@ there_is_another_patch(void)
 			say("done\n");
 		return false;
 	}
+	if (p_filesize == 0)
+		return false;
+	nonempty_patchf_seen = true;
 	if (verbose)
 		say("Hmm...");
 	diff_type = intuit_diff_type();



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201904041721.x34HLVxW030072>