Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Alexander Kanavin <alexander.kanavin@linux.intel.com>
To: openembedded-core@lists.openembedded.org
Subject: [PATCH 31/42] automake: refresh patches
Date: Thu,  8 Mar 2018 20:17:53 +0200	[thread overview]
Message-ID: <20180308181804.31261-31-alexander.kanavin@linux.intel.com> (raw)
In-Reply-To: <20180308181804.31261-1-alexander.kanavin@linux.intel.com>

From: Ross Burton <ross.burton@intel.com>

The patch tool will apply patches by default with "fuzz", which is where if the
hunk context isn't present but what is there is close enough, it will force the
patch in.

Whilst this is useful when there's just whitespace changes, when applied to
source it is possible for a patch applied with fuzz to produce broken code which
still compiles (see #10450).  This is obviously bad.

We'd like to eventually have do_patch() rejecting any fuzz on these grounds. For
that to be realistic the existing patches with fuzz need to be rebased and
reviewed.

Signed-off-by: Ross Burton <ross.burton@intel.com>
Signed-off-by: Alexander Kanavin <alexander.kanavin@linux.intel.com>
---
 ...tion-in-shebangs-with-modern-use-warnings.patch | 41 ++++++++++------------
 .../automake/automake/buildtest.patch              | 32 +++++++++--------
 2 files changed, 36 insertions(+), 37 deletions(-)

diff --git a/meta/recipes-devtools/automake/automake/automake-replace-w-option-in-shebangs-with-modern-use-warnings.patch b/meta/recipes-devtools/automake/automake/automake-replace-w-option-in-shebangs-with-modern-use-warnings.patch
index 64eb253a6ea..96b5a3d6b2a 100644
--- a/meta/recipes-devtools/automake/automake/automake-replace-w-option-in-shebangs-with-modern-use-warnings.patch
+++ b/meta/recipes-devtools/automake/automake/automake-replace-w-option-in-shebangs-with-modern-use-warnings.patch
@@ -22,10 +22,10 @@ Signed-off-by: Serhii Popovych <spopovyc@cisco.com>
  t/wrap/automake.in | 3 ++-
  4 files changed, 8 insertions(+), 4 deletions(-)
 
-diff --git a/bin/aclocal.in b/bin/aclocal.in
-index 349f24a..50cb8d3 100644
---- a/bin/aclocal.in
-+++ b/bin/aclocal.in
+Index: automake-1.15.1/bin/aclocal.in
+===================================================================
+--- automake-1.15.1.orig/bin/aclocal.in
++++ automake-1.15.1/bin/aclocal.in
 @@ -1,4 +1,4 @@
 -#!@PERL@ -w
 +#!@PERL@
@@ -40,17 +40,17 @@ index 349f24a..50cb8d3 100644
  
  use Automake::Config;
  use Automake::General;
-diff --git a/bin/automake.in b/bin/automake.in
-index eedc8bc..e0a01cf 100644
---- a/bin/automake.in
-+++ b/bin/automake.in
+Index: automake-1.15.1/bin/automake.in
+===================================================================
+--- automake-1.15.1.orig/bin/automake.in
++++ automake-1.15.1/bin/automake.in
 @@ -1,4 +1,4 @@
 -#!@PERL@ -w
 +#!@PERL@
  # -*- perl -*-
  # @configure_input@
  
-@@ -28,6 +28,7 @@ eval 'case $# in 0) exec @PERL@ -S "$0";; *) exec @PERL@ -S "$0" "$@";; esac'
+@@ -28,6 +28,7 @@ eval 'case $# in 0) exec @PERL@ -S "$0";
  package Automake;
  
  use strict;
@@ -58,16 +58,16 @@ index eedc8bc..e0a01cf 100644
  
  BEGIN
  {
-diff --git a/t/wrap/aclocal.in b/t/wrap/aclocal.in
-index e64b546..9996899 100644
---- a/t/wrap/aclocal.in
-+++ b/t/wrap/aclocal.in
+Index: automake-1.15.1/t/wrap/aclocal.in
+===================================================================
+--- automake-1.15.1.orig/t/wrap/aclocal.in
++++ automake-1.15.1/t/wrap/aclocal.in
 @@ -1,4 +1,4 @@
 -#!@PERL@ -w
 +#!@PERL@
  # @configure_input@
  
- # Copyright (C) 2012-2014 Free Software Foundation, Inc.
+ # Copyright (C) 2012-2017 Free Software Foundation, Inc.
 @@ -19,6 +19,7 @@
  BEGIN
  {
@@ -76,16 +76,16 @@ index e64b546..9996899 100644
    @Aclocal::perl_libdirs = ('@abs_top_srcdir@/lib');
    unshift @Aclocal::perl_libdirs, '@abs_top_builddir@/lib'
      if '@srcdir@' ne '.';
-diff --git a/t/wrap/automake.in b/t/wrap/automake.in
-index 8b943b1..be61226 100644
---- a/t/wrap/automake.in
-+++ b/t/wrap/automake.in
+Index: automake-1.15.1/t/wrap/automake.in
+===================================================================
+--- automake-1.15.1.orig/t/wrap/automake.in
++++ automake-1.15.1/t/wrap/automake.in
 @@ -1,4 +1,4 @@
 -#!@PERL@ -w
 +#!@PERL@
  # @configure_input@
  
- # Copyright (C) 2012-2014 Free Software Foundation, Inc.
+ # Copyright (C) 2012-2017 Free Software Foundation, Inc.
 @@ -19,6 +19,7 @@
  BEGIN
  {
@@ -94,6 +94,3 @@ index 8b943b1..be61226 100644
    @Automake::perl_libdirs = ('@abs_top_srcdir@/lib');
    unshift @Automake::perl_libdirs, '@abs_top_builddir@/lib'
      if '@srcdir@' ne '.';
--- 
-2.3.0
-
diff --git a/meta/recipes-devtools/automake/automake/buildtest.patch b/meta/recipes-devtools/automake/automake/buildtest.patch
index 1dd5337849f..c32cd1fb11e 100644
--- a/meta/recipes-devtools/automake/automake/buildtest.patch
+++ b/meta/recipes-devtools/automake/automake/buildtest.patch
@@ -4,21 +4,23 @@ be run separately.
 Signed-off-by: Björn Stenberg <bjst@enea.com>
 Upstream-Status: Pending
 
---- a/lib/am/check.am	2012-11-14 13:46:16.335475995 +0100
-+++ b/lib/am/check.am	2012-08-13 18:40:12.000000000 +0200
-@@ -44,7 +44,7 @@
- am__tty_colors = $(am__tty_colors_dummy)
- endif !%?COLOR%
-
+Index: automake-1.15.1/lib/am/check.am
+===================================================================
+--- automake-1.15.1.orig/lib/am/check.am
++++ automake-1.15.1/lib/am/check.am
+@@ -41,7 +41,7 @@ am__tty_colors = { \
+   fi; \
+ }
+ 
 -.PHONY: check-TESTS
 +.PHONY: check-TESTS buildtest-TESTS runtest-TESTS
-
- if %?PARALLEL_TESTS%
-
-@@ -465,7 +465,14 @@
-
- else !%?PARALLEL_TESTS%
-
+ 
+ if !%?SERIAL_TESTS%
+ 
+@@ -466,7 +466,14 @@ else %?SERIAL_TESTS%
+ 
+ ## Obsolescent serial testsuite driver.
+ 
 -check-TESTS: $(TESTS)
 +AM_RECURSIVE_TARGETS += buildtest runtest
 +
@@ -28,6 +30,6 @@ Upstream-Status: Pending
 +	$(MAKE) $(AM_MAKEFLAGS) runtest-TESTS
 +
 +runtest-TESTS:
-	@failed=0; all=0; xfail=0; xpass=0; skip=0; \
-	srcdir=$(srcdir); export srcdir; \
+ 	@failed=0; all=0; xfail=0; xpass=0; skip=0; \
+ 	srcdir=$(srcdir); export srcdir; \
  ## Make sure Solaris VPATH-expands all members of this list, even
-- 
2.16.1



  parent reply	other threads:[~2018-03-08 18:25 UTC|newest]

Thread overview: 42+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-03-08 18:17 [PATCH 01/42] expat: refresh patches Alexander Kanavin
2018-03-08 18:17 ` [PATCH 02/42] grub: " Alexander Kanavin
2018-03-08 18:17 ` [PATCH 03/42] shared-mime-info: " Alexander Kanavin
2018-03-08 18:17 ` [PATCH 04/42] serf: " Alexander Kanavin
2018-03-08 18:17 ` [PATCH 05/42] nspr: " Alexander Kanavin
2018-03-08 18:17 ` [PATCH 06/42] libical: " Alexander Kanavin
2018-03-08 18:17 ` [PATCH 07/42] gmp: " Alexander Kanavin
2018-03-08 18:17 ` [PATCH 08/42] gnutls: " Alexander Kanavin
2018-03-08 18:17 ` [PATCH 09/42] libffi: " Alexander Kanavin
2018-03-08 18:17 ` [PATCH 10/42] libksba: " Alexander Kanavin
2018-03-08 18:17 ` [PATCH 11/42] libtiff: " Alexander Kanavin
2018-03-08 18:17 ` [PATCH 12/42] gstreamer: " Alexander Kanavin
2018-03-08 18:17 ` [PATCH 13/42] kmod: " Alexander Kanavin
2018-03-08 18:17 ` [PATCH 14/42] dtc: " Alexander Kanavin
2018-03-08 18:17 ` [PATCH 15/42] lttng: " Alexander Kanavin
2018-03-08 18:17 ` [PATCH 16/42] wayland: " Alexander Kanavin
2018-03-08 18:17 ` [PATCH 17/42] clutter: " Alexander Kanavin
2018-03-08 18:17 ` [PATCH 18/42] xset: " Alexander Kanavin
2018-03-08 18:17 ` [PATCH 19/42] at: " Alexander Kanavin
2018-03-08 18:17 ` [PATCH 20/42] bash: " Alexander Kanavin
2018-03-08 18:17 ` [PATCH 21/42] ethtool: " Alexander Kanavin
2018-03-08 18:17 ` [PATCH 22/42] ghostscript: " Alexander Kanavin
2018-03-08 18:17 ` [PATCH 23/42] ghostscript: drop incorrectly applied patch Alexander Kanavin
2018-03-08 18:17 ` [PATCH 24/42] libidn: refresh patches Alexander Kanavin
2018-03-08 18:17 ` [PATCH 25/42] mdadm: " Alexander Kanavin
2018-03-08 18:17 ` [PATCH 26/42] pam: " Alexander Kanavin
2018-03-08 18:17 ` [PATCH 27/42] parted: " Alexander Kanavin
2018-03-08 18:17 ` [PATCH 28/42] tcp-wrappers: " Alexander Kanavin
2018-03-08 18:17 ` [PATCH 29/42] iptables: drop unnecessary patches Alexander Kanavin
2018-03-08 18:17 ` [PATCH 30/42] apt: refresh patches Alexander Kanavin
2018-03-08 18:17 ` Alexander Kanavin [this message]
2018-03-08 18:17 ` [PATCH 32/42] intltool: " Alexander Kanavin
2018-03-08 18:17 ` [PATCH 33/42] mtd: " Alexander Kanavin
2018-03-08 18:17 ` [PATCH 34/42] syslinux: " Alexander Kanavin
2018-03-08 18:17 ` [PATCH 35/42] ppp: " Alexander Kanavin
2018-03-08 18:17 ` [PATCH 36/42] util-linux: " Alexander Kanavin
2018-03-08 18:17 ` [PATCH 37/42] zlib: " Alexander Kanavin
2018-03-08 18:18 ` [PATCH 38/42] irda-utils: " Alexander Kanavin
2018-03-08 18:18 ` [PATCH 39/42] python-numpy: update to 1.14.1 Alexander Kanavin
2018-03-08 18:18 ` [PATCH 40/42] python: refresh patches Alexander Kanavin
2018-03-08 18:18 ` [PATCH 41/42] perl: " Alexander Kanavin
2018-03-08 18:18 ` [PATCH 42/42] perl: remove perl-enable-gdbm.patch Alexander Kanavin

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=20180308181804.31261-31-alexander.kanavin@linux.intel.com \
    --to=alexander.kanavin@linux.intel.com \
    --cc=openembedded-core@lists.openembedded.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.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox