Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Alexander Kanavin <alexander.kanavin@linux.intel.com>
To: openembedded-core@lists.openembedded.org
Subject: [PATCH 02/42] grub: refresh patches
Date: Thu,  8 Mar 2018 20:17:24 +0200	[thread overview]
Message-ID: <20180308181804.31261-2-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>
---
 ...odule-explicitly-keeps-symbole-.module_license.patch | 17 +++++++----------
 1 file changed, 7 insertions(+), 10 deletions(-)

diff --git a/meta/recipes-bsp/grub/files/grub-module-explicitly-keeps-symbole-.module_license.patch b/meta/recipes-bsp/grub/files/grub-module-explicitly-keeps-symbole-.module_license.patch
index 935c472cf8f..ffc2d40d893 100644
--- a/meta/recipes-bsp/grub/files/grub-module-explicitly-keeps-symbole-.module_license.patch
+++ b/meta/recipes-bsp/grub/files/grub-module-explicitly-keeps-symbole-.module_license.patch
@@ -43,19 +43,16 @@ Signed-off-by: Hongxu Jia <hongxu.jia@windriver.com>
  grub-core/genmod.sh.in | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)
 
-diff --git a/grub-core/genmod.sh.in b/grub-core/genmod.sh.in
-index 76df0bb..39a9cfc 100644
---- a/grub-core/genmod.sh.in
-+++ b/grub-core/genmod.sh.in
-@@ -59,7 +59,7 @@ if test x@TARGET_APPLE_CC@ != x1; then
+Index: grub-2.02/grub-core/genmod.sh.in
+===================================================================
+--- grub-2.02.orig/grub-core/genmod.sh.in
++++ grub-2.02/grub-core/genmod.sh.in
+@@ -56,7 +56,7 @@ if test x@TARGET_APPLE_LINKER@ != x1; th
  	if test x@platform@ != xemu; then
- 	    @STRIP@ --strip-unneeded \
+ 	    @TARGET_STRIP@ --strip-unneeded \
  		-K grub_mod_init -K grub_mod_fini \
 -		-K _grub_mod_init -K _grub_mod_fini \
 +		-K _grub_mod_init -K _grub_mod_fini -K .module_license \
  		-R .note.gnu.gold-version -R .note.GNU-stack \
- 		-R .note -R .comment $tmpfile || exit 1
+ 		-R .note -R .comment -R .ARM.exidx $tmpfile || exit 1
  	fi
--- 
-2.8.1
-
-- 
2.16.1



  reply	other threads:[~2018-03-08 18:24 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 ` Alexander Kanavin [this message]
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 ` [PATCH 31/42] automake: " Alexander Kanavin
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-2-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