libcare-patch-make: fix some bugs
This patch fix following bugs: 1. origin build should not add '-Wl,-q' flags 2. recover config-host.mak after make kpatch for next building Signed-off-by: Bihong Yu <yubihong@huawei.com> Signed-off-by: yezengruan <yezengruan@huawei.com> (cherry picked from commit 932be3ded6490136ae3a1e79a96f389498b41a42)
This commit is contained in:
yezengruan
openeuler-sync-bot
parent
6a44912faf
commit
f94071cd9b
130
libcare-patch-make-fix-some-bugs.patch
Normal file
130
libcare-patch-make-fix-some-bugs.patch
Normal file
@ -0,0 +1,130 @@
|
|||||||
|
From 326fe36893de32fe8a8e95fcb5aee6df5c9a3fa7 Mon Sep 17 00:00:00 2001
|
||||||
|
From: Bihong Yu <yubihong@huawei.com>
|
||||||
|
Date: Sat, 19 Mar 2022 15:20:27 +0800
|
||||||
|
Subject: [PATCH] libcare-patch-make: fix some bugs
|
||||||
|
|
||||||
|
This patch fix following bugs:
|
||||||
|
1. origin build should not add '-Wl,-q' flags
|
||||||
|
2. recover config-host.mak after make kpatch for next building
|
||||||
|
|
||||||
|
Signed-off-by: Bihong Yu <yubihong@huawei.com>
|
||||||
|
Signed-off-by: yezengruan <yezengruan@huawei.com>
|
||||||
|
---
|
||||||
|
src/libcare-patch-make | 54 +++++++++++++++++++++++++-----------------
|
||||||
|
1 file changed, 32 insertions(+), 22 deletions(-)
|
||||||
|
|
||||||
|
diff --git a/src/libcare-patch-make b/src/libcare-patch-make
|
||||||
|
index 9a75e1f..03aa1d6 100755
|
||||||
|
--- a/src/libcare-patch-make
|
||||||
|
+++ b/src/libcare-patch-make
|
||||||
|
@@ -53,9 +53,9 @@ prepare_env() {
|
||||||
|
LPMAKE_PATCHED_DIR="${LPMAKE_PATCHED_DIR-$PWD/.lpmaketmp/patched}"
|
||||||
|
LPMAKE_PATCHROOT="${LPMAKE_PATCHROOT-$PWD/patchroot}"
|
||||||
|
|
||||||
|
- LIBCARE_CC=$CC
|
||||||
|
- SYMBOLINK_CC=$(which cc)
|
||||||
|
- REAL_CC=$(realpath $SYMBOLINK_CC)
|
||||||
|
+ LIBCARE_CC=$CC
|
||||||
|
+ SYMBOLINK_CC=$(which cc)
|
||||||
|
+ REAL_CC=$(realpath $SYMBOLINK_CC)
|
||||||
|
|
||||||
|
export LPMAKE_ORIGINAL_DIR LPMAKE_PATCHED_DIR LPMAKE_PATCHROOT
|
||||||
|
mkdir -p "$LPMAKE_ORIGINAL_DIR" "$LPMAKE_PATCHED_DIR" "$LPMAKE_PATCHROOT"
|
||||||
|
@@ -83,24 +83,30 @@ restore_origs() {
|
||||||
|
|
||||||
|
trap "restore_origs" 0
|
||||||
|
|
||||||
|
-change_qemu_ld_flags() {
|
||||||
|
- qemu_ld_flags=$(grep "^QEMU_LDFLAGS=" config-host.mak)
|
||||||
|
- ret=$(echo $qemu_ld_flags | grep "\-Wl,-q")
|
||||||
|
- if [[ "$ret" == "" ]];then
|
||||||
|
- qemu_ld_flags="${qemu_ld_flags} -Wl,-q"
|
||||||
|
- echo "change QEMU_LDFLAGS to '${qemu_ld_flags}'"
|
||||||
|
- sed -i "/^QEMU_LDFLAGS=/c\\${qemu_ld_flags}" config-host.mak
|
||||||
|
- fi
|
||||||
|
+replace_qemu_ld_flags() {
|
||||||
|
+ local qemu_ld_flags_old=$1
|
||||||
|
+ ret=$(echo $qemu_ld_flags_old | grep "\-Wl,-q")
|
||||||
|
+ if [[ "$ret" == "" ]]; then
|
||||||
|
+ local qemu_ld_flags="${qemu_ld_flags_old} -Wl,-q"
|
||||||
|
+ echo "replace QEMU_LDFLAGS to '${qemu_ld_flags}'"
|
||||||
|
+ sed -i "/^QEMU_LDFLAGS=/c\\${qemu_ld_flags}" config-host.mak
|
||||||
|
+ fi
|
||||||
|
+}
|
||||||
|
+
|
||||||
|
+recover_qemu_ld_flags() {
|
||||||
|
+ local qemu_ld_flags=$1
|
||||||
|
+ echo "recover QEMU_LDFLAGS to '${qemu_ld_flags}'"
|
||||||
|
+ sed -i "/^QEMU_LDFLAGS=/c\\${qemu_ld_flags}" config-host.mak
|
||||||
|
}
|
||||||
|
|
||||||
|
replace_cc_symbolink() {
|
||||||
|
- unlink $SYMBOLINK_CC
|
||||||
|
- ln -s $LIBCARE_CC $SYMBOLINK_CC
|
||||||
|
+ unlink $SYMBOLINK_CC
|
||||||
|
+ ln -s $LIBCARE_CC $SYMBOLINK_CC
|
||||||
|
}
|
||||||
|
|
||||||
|
recover_cc_symbolink() {
|
||||||
|
- unlink $SYMBOLINK_CC
|
||||||
|
- ln -s $REAL_CC $SYMBOLINK_CC
|
||||||
|
+ unlink $SYMBOLINK_CC
|
||||||
|
+ ln -s $REAL_CC $SYMBOLINK_CC
|
||||||
|
}
|
||||||
|
|
||||||
|
build_objects() {
|
||||||
|
@@ -117,9 +123,6 @@ build_objects() {
|
||||||
|
export KPATCH_STAGE=original
|
||||||
|
export KPCC_DBGFILTER_ARGS=""
|
||||||
|
|
||||||
|
- #add '-Wl,-q' to LD_FLAGS
|
||||||
|
- change_qemu_ld_flags
|
||||||
|
-
|
||||||
|
echo "${green}BUILDING ORIGINAL CODE${reset}"
|
||||||
|
make $LPMAKEFILE $JOBS_MAKE >$MAKE_OUTPUT 2>&1
|
||||||
|
|
||||||
|
@@ -146,6 +149,10 @@ build_objects() {
|
||||||
|
export KPATCH_STAGE=patched
|
||||||
|
export KPCC_APPEND_ARGS="-Wl,-q"
|
||||||
|
|
||||||
|
+ qemu_ld_flags_bak=$(grep "^QEMU_LDFLAGS=" config-host.mak)
|
||||||
|
+ #add '-Wl,-q' to LD_FLAGS
|
||||||
|
+ replace_qemu_ld_flags "$qemu_ld_flags_bak"
|
||||||
|
+
|
||||||
|
echo "${green}BUILDING PATCHED CODE${reset}"
|
||||||
|
make $LPMAKEFILE $JOBS_MAKE >$MAKE_OUTPUT 2>&1
|
||||||
|
|
||||||
|
@@ -153,6 +160,9 @@ build_objects() {
|
||||||
|
make $LPMAKEFILE $JOBS_MAKE install \
|
||||||
|
"$destdir=$LPMAKE_PATCHED_DIR" \
|
||||||
|
>$MAKE_OUTPUT 2>&1
|
||||||
|
+
|
||||||
|
+ # recover LD_FLAGS
|
||||||
|
+ recover_qemu_ld_flags "$qemu_ld_flags_bak"
|
||||||
|
}
|
||||||
|
|
||||||
|
build_kpatches() {
|
||||||
|
@@ -264,16 +274,16 @@ main() {
|
||||||
|
|
||||||
|
prepare_env
|
||||||
|
|
||||||
|
- # replace cc
|
||||||
|
- replace_cc_symbolink
|
||||||
|
+ # replace cc
|
||||||
|
+ replace_cc_symbolink
|
||||||
|
|
||||||
|
if test -z "$only_update"; then
|
||||||
|
build_objects "$@"
|
||||||
|
fi
|
||||||
|
build_kpatches
|
||||||
|
|
||||||
|
- # recover cc
|
||||||
|
- recover_cc_symbolink
|
||||||
|
+ # recover cc
|
||||||
|
+ recover_cc_symbolink
|
||||||
|
}
|
||||||
|
|
||||||
|
main "$@"
|
||||||
|
--
|
||||||
|
2.27.0
|
||||||
|
|
||||||
Loading…
x
Reference in New Issue
Block a user