From: Prarit Bhargava <prarit(a)redhat.com>
The 'dist-configs' is not working properly as it only creates ELN
configs. The 'rh-configs' and 'fedora-configs' targets are not
working properly and should be creating ELN and Fedora configs but they
only create some temporary/intermediate build files. These targets
should output final .config files for each supported arch.
There are several fixes necessary to get the *-configs targets working
properly:
- Fix dist-configs to use a specified flavor instead of only ELN. By
default, dist-configs will build ELN configs.
- Add a flavor parameter to build_config.sh. By default build_config.sh
will use still the configs/flavors file, o/w build_config will only build
the flavor specified.
- Silence a warning in generate_all_configs.sh
Fix the '*-configs' targets to output final .config files for each
supported flavor arch.
Cleanup: Remove unused DIST_TARGET and TARGET variables.
Signed-off-by: Prarit Bhargava <prarit(a)redhat.com>
Reported-by: mleitner(a)redhat.com
Cc: dzickus(a)redhat.com
Cc: jforbes(a)redhat.com
Cc: mleitner(a)redhat.com
Cc: jbenc(a)redhat.com
---
redhat/Makefile | 15 ++++-----------
redhat/configs/build_configs.sh | 9 ++++++++-
redhat/configs/generate_all_configs.sh | 2 +-
3 files changed, 13 insertions(+), 13 deletions(-)
diff --git a/redhat/Makefile b/redhat/Makefile
index 7cdf9e3bd0bf..c813936e098c 100644
--- a/redhat/Makefile
+++ b/redhat/Makefile
@@ -20,13 +20,6 @@ else
endif
endif
-
-ifeq ("$(IS_FEDORA)", "1")
- DIST_TARGET=fedora
-else
- DIST_TARGET=rhel
-endif
-
ifeq ("$(NO_CONFIGCHECKS)","1")
PROCESS_CONFIGS_OPTS=
PROCESS_CONFIGS_CHECK_OPTS=
@@ -138,20 +131,20 @@ dist-configs-commit: dist-configs-prep
./process_configs.sh -z $(PACKAGE_NAME) $(KVERSION) "" $(FLAVOR)
dist-configs: dist-configs-prep
- @cd $(REDHAT)/configs; VERSION=$(KVERSION) ./generate_all_configs.sh rhel 1; \
+ @cd $(REDHAT)/configs; VERSION=$(KVERSION) ./generate_all_configs.sh
"$(FLAVOR)" 1; \
./process_configs.sh $(PROCESS_CONFIGS_OPTS) $(PACKAGE_NAME) $(KVERSION)
-fedora-configs: DIST_TARGET=fedora
+fedora-configs: FLAVOR = fedora
fedora-configs: dist-configs
-rh-configs: DIST_TARGET=rhel
+rh-configs: FLAVOR = rhel
rh-configs: dist-configs
dist-configs-check: dist-configs-prep
cd $(REDHAT)/configs; ./process_configs.sh $(PROCESS_CONFIGS_CHECK_OPTS)
$(PACKAGE_NAME)
dist-configs-prep: dist-clean-configs
- cd $(REDHAT)/configs; TARGET=$(DIST_TARGET) ./build_configs.sh $(PACKAGE_NAME)
$(ARCH_MACH)
+ cd $(REDHAT)/configs; ./build_configs.sh "$(PACKAGE_NAME)"
"$(ARCH_MACH)" "$(FLAVOR)"
dist-configs-arch: ARCH_MACH = $(MACH)
dist-configs-arch: dist-configs
diff --git a/redhat/configs/build_configs.sh b/redhat/configs/build_configs.sh
index 18610e1fc097..90f2be85f49d 100755
--- a/redhat/configs/build_configs.sh
+++ b/redhat/configs/build_configs.sh
@@ -10,6 +10,12 @@ SCRIPT="$(readlink -f $0)"
OUTPUT_DIR="$PWD"
SCRIPT_DIR="$(dirname $SCRIPT)"
+if [ -z "$3" ]; then
+ cat flavors > .flavors
+else
+ echo "$3" > .flavors
+fi
+
LANG=en_US.UTF-8
# to handle this script being a symlink
@@ -21,6 +27,7 @@ set nounset
cleanup()
{
rm -f config-*
+ rm -f .flavors
}
die()
@@ -142,6 +149,6 @@ function build_flavor()
while read line
do
build_flavor $line
-done < flavors
+done < .flavors
cleanup
diff --git a/redhat/configs/generate_all_configs.sh
b/redhat/configs/generate_all_configs.sh
index 14f0f3a24de5..15a39a8fccc8 100755
--- a/redhat/configs/generate_all_configs.sh
+++ b/redhat/configs/generate_all_configs.sh
@@ -25,7 +25,7 @@ for i in kernel-*-$PRIMARY.config; do
mv $i $NEW
done
-rm kernel-*-$SECONDARY.config
+rm -f kernel-*-$SECONDARY.config
if [ $DEBUGBUILDSENABLED -eq 0 ]; then
for i in kernel-*debug*.config; do
--
GitLab