Folgendes ist dabei rausgekommen:
Code: Alles auswählen
Index: Makefile.am
===================================================================
RCS file: /cvs/tuxbox/cdk/Makefile.am,v
retrieving revision 1.558
diff -b -B -u -r1.558 Makefile.am
--- a/Makefile.am 10 Jan 2009 09:13:31 -0000 1.558
+++ b/Makefile.am 11 Jan 2009 11:06:31 -0000
@@ -5,9 +5,12 @@
# Define high-level and yadd targets
include make/high-level-yadd.mk
-# Set up some default values (used only by serversetup).
+# Set up some default values (used by serversetup, default kernel/busybox config).
include make/defaultvalues.mk
+# Override default values if needed
+-include $(customizationsdir)/user.mk
+
# Set up the build environment
include make/buildenv.mk
Index: make/busybox.mk
===================================================================
RCS file: /cvs/tuxbox/cdk/make/busybox.mk,v
retrieving revision 1.2
diff -b -B -u -r1.2 busybox.mk
--- a/make/busybox.mk 6 Jan 2009 21:40:15 -0000 1.2
+++ b/make/busybox.mk 11 Jan 2009 11:06:31 -0000
@@ -8,9 +8,9 @@
POSSIBLY_KERNEL26=-Dkernel26
endif
-$(DEPDIR)/busybox: bootstrap @DEPENDS_busybox@ Patches/busybox.config.m4 Patches/busybox.diff
+$(DEPDIR)/busybox: bootstrap @DEPENDS_busybox@ $(BUSYBOX_CONFIG_FILE) Patches/busybox.diff
@PREPARE_busybox@
- m4 -Dyadd $(POSSIBLY_IDE) $(POSSIBLY_EXT3) $(POSSIBLY_KERNEL26) -DPREFIX="\"$(targetprefix)\"" Patches/busybox.config.m4 > @DIR_busybox@/.config
+ m4 -Dyadd $(POSSIBLY_IDE) $(POSSIBLY_EXT3) $(POSSIBLY_KERNEL26) -DPREFIX="\"$(targetprefix)\"" $(BUSYBOX_CONFIG_FILE) > @DIR_busybox@/.config
cd @DIR_busybox@ && \
$(MAKE) all install \
CROSS_COMPILE=$(target)- \
@@ -21,9 +21,9 @@
if TARGETRULESET_FLASH
-flash-busybox: bootstrap $(flashprefix)/root @DEPENDS_busybox@ Patches/busybox.config.m4 Patches/busybox.diff
+flash-busybox: bootstrap $(flashprefix)/root @DEPENDS_busybox@ $(BUSYBOX_CONFIG_FILE) Patches/busybox.diff
@PREPARE_busybox@
- m4 -Dflash $(POSSIBLY_IDE) $(POSSIBLY_EXT3) $(POSSIBLY_KERNEL26) -DPREFIX="\"$(flashprefix)/root\"" Patches/busybox.config.m4 > @DIR_busybox@/.config
+ m4 -Dflash $(POSSIBLY_IDE) $(POSSIBLY_EXT3) $(POSSIBLY_KERNEL26) -DPREFIX="\"$(flashprefix)/root\"" $(BUSYBOX_CONFIG_FILE) > @DIR_busybox@/.config
cd @DIR_busybox@ && \
$(MAKE) all install \
CROSS_COMPILE=$(target)- \
Index: make/defaultvalues.mk
===================================================================
RCS file: /cvs/tuxbox/cdk/make/defaultvalues.mk,v
retrieving revision 1.2
diff -b -B -u -r1.2 defaultvalues.mk
--- a/make/defaultvalues.mk 6 Jan 2009 21:40:15 -0000 1.2
+++ b/make/defaultvalues.mk 11 Jan 2009 11:06:31 -0000
@@ -8,3 +8,11 @@
DBOX_SUBNETMASK = 255.255.255.0
DBOX_MAC = 00:50:9c:xx:xx:x
+
+# These variables are default values and can be overwritten in $(customizationsdir)/user.mk
+BUSYBOX_CONFIG_FILE:=Patches/busybox-1.13.config.m4
+
+LINUX_CONFIG_FILE_DIR:=Patches
+LINUX_DBOX2_FLASH_FILE_DIR:=Patches
+
+YADD_ROOTFS:=
Index: make/linuxkernel.mk
===================================================================
RCS file: /cvs/tuxbox/cdk/make/linuxkernel.mk,v
retrieving revision 1.2
diff -b -B -u -r1.2 linuxkernel.mk
--- a/make/linuxkernel.mk 6 Jan 2009 21:40:15 -0000 1.2
+++ b/make/linuxkernel.mk 11 Jan 2009 11:06:31 -0000
@@ -125,15 +125,15 @@
kernel-cdk: $(bootprefix)/kernel-cdk
if KERNEL26
-$(bootprefix)/kernel-cdk: linuxdir $(hostprefix)/bin/mkimage Patches/linux-$(KERNELVERSION).config Patches/dbox2-flash.c-26.m4
- sed $(IDE_SED_CONF) $(EXT3_SED_CONF) $(XFS_SED_CONF) $(NFSSERVER_SED_CONF) Patches/linux-$(KERNELVERSION).config \
+$(bootprefix)/kernel-cdk: linuxdir $(hostprefix)/bin/mkimage $(LINUX_CONFIG_FILE_DIR)/linux-$(KERNELVERSION).config $(LINUX_DBOX2_FLASH_FILE_DIR)/dbox2-flash.c-26.m4
+ sed $(IDE_SED_CONF) $(EXT3_SED_CONF) $(XFS_SED_CONF) $(NFSSERVER_SED_CONF) $(LINUX_CONFIG_FILE_DIR)/linux-$(KERNELVERSION).config \
> $(KERNEL_DIR)/.config
- m4 Patches/dbox2-flash.c-26.m4 > linux/drivers/mtd/maps/dbox2-flash.c
+ m4 $(YADD_ROOTFS) $(LINUX_DBOX2_FLASH_FILE_DIR)/dbox2-flash.c-26.m4 > linux/drivers/mtd/maps/dbox2-flash.c
else
-$(bootprefix)/kernel-cdk: linuxdir $(hostprefix)/bin/mkimage Patches/linux-2.4.35.5-dbox2.config Patches/dbox2-flash.c.m4
- sed $(IDE_SED_CONF) $(EXT3_SED_CONF) $(XFS_SED_CONF) $(NFSSERVER_SED_CONF) Patches/linux-2.4.35.5-dbox2.config \
+$(bootprefix)/kernel-cdk: linuxdir $(hostprefix)/bin/mkimage $(LINUX_CONFIG_FILE_DIR)/linux-2.4.35.5-dbox2.config $(LINUX_DBOX2_FLASH_FILE_DIR)/dbox2-flash.c.m4
+ sed $(IDE_SED_CONF) $(EXT3_SED_CONF) $(XFS_SED_CONF) $(NFSSERVER_SED_CONF) $(LINUX_CONFIG_FILE_DIR)/linux-2.4.35.5-dbox2.config \
> $(KERNEL_DIR)/.config
- m4 Patches/dbox2-flash.c.m4 > linux/drivers/mtd/maps/dbox2-flash.c
+ m4 $(YADD_ROOTFS) $(LINUX_DBOX2_FLASH_FILE_DIR)/dbox2-flash.c.m4 > linux/drivers/mtd/maps/dbox2-flash.c
endif
$(MAKE) $(KERNEL_BUILD_FILENAME)
if KERNEL26
Code: Alles auswählen
# define local versions of linux kernel and busybox settings here
# default settings are in make/defaultvalues.mk
# define rootfs type for yadd kernels - you can then mount the flash root partifion within the yadd
YADD_ROOTFS= --define=rootfstype=jffs2
# define the busybox confg file here
BUSYBOX_CONFIG_FILE=$(customizationsdir)/busybox-1.13.config.m4
# define the path to the kernel stuff file here
LINUX_CONFIG_FILE_DIR=$(customizationsdir)
LINUX_DBOX2_FLASH_FILE_DIR=$(customizationsdir)
Die Konfigurationsdatei der Busybox als Konfigurationsparameter scheint mir eine gute Kompromisse zwischen theoretisch elegantes und praktisches machbares zu sein. Dagegen habe ich aber ein Problem mit noch eine Ablage für Benutzerparameter, hier, eine neu zugefügte Datei user.mk. Die mehr traditionelle (in Sinn von auto[conf|make]) Alternative wäre --with-busybox-conf=<blabla>.m4. (Ich bin mir sicher, sämtliche bisherige Teilnehmer des Threads könnte sowas implementieren 
) Die "configure-Zeile" ist ja nicht wirklich Benutzerfreundlich, aber dies wird ja normalerweise nicht als ein Problem betrachtet. "Benutzerkonfiguration" besteht z.Z. von der Eingabe eines configure-Befehls (normalerweise in einer selbstgestrichen Skript), eventuell zusammen mit diverse customization-scripts. Wir sollten dies nicht ohne sehr guten Grunden erweitern, z.B. mit "Anlegen eines user.mk". Ich bin der Meinung, eine configure-option wäre besser.