Message ID | 20191001222907.30588-1-freifunk@adrianschmutzler.de |
---|---|
State | Superseded |
Headers | show |
diff --git a/bsp/board_ar71xx.bsp b/bsp/board_ar71xx.bsp index 37130d05..f84b10a1 100644 --- a/bsp/board_ar71xx.bsp +++ b/bsp/board_ar71xx.bsp @@ -1,7 +1,6 @@ machine=ar71xx chipset=ar71xx subtarget=tiny -target=$builddir/$machine images=("openwrt-${chipset}-${subtarget}-archer-c25-v1-squashfs-sysupgrade.bin" "openwrt-${chipset}-${subtarget}-archer-c60-v1-squashfs-sysupgrade.bin" "openwrt-${chipset}-${subtarget}-archer-c7-v2-squashfs-sysupgrade.bin" diff --git a/bsp/board_mt7621.bsp b/bsp/board_mt7621.bsp index e9f67765..340ec0b6 100644 --- a/bsp/board_mt7621.bsp +++ b/bsp/board_mt7621.bsp @@ -1,7 +1,6 @@ machine=mt7621 chipset=ramips subtarget=mt7621 -target=$builddir/$machine images=( "openwrt-ramips-mt7621-ubnt-erx-squashfs-sysupgrade.tar" diff --git a/bsp/board_wdr4900.bsp b/bsp/board_wdr4900.bsp index 8520c6e5..8671c26e 100644 --- a/bsp/board_wdr4900.bsp +++ b/bsp/board_wdr4900.bsp @@ -1,7 +1,6 @@ machine=wdr4900 chipset=mpc85xx subtarget=generic -target=$builddir/$machine images=( "openwrt-${chipset}-${subtarget}-tl-wdr4900-v1-squashfs-sysupgrade.bin" diff --git a/buildscript b/buildscript index c2de5759..419bf7b5 100755 --- a/buildscript +++ b/buildscript @@ -169,7 +169,7 @@ prepare() { #saves ~200MB for each build test -d ./src/dl || mkdir ./src/dl - ln -s ../../src/dl "$target"/dl + ln -s ../src/dl "$target"/dl } prebuild() { @@ -339,6 +339,7 @@ loadBSP() { echo "Working with $(/bin/ls -l selected_bsp | awk '{ print $11 }')" . selected_bsp + target=$builddir } loadVariant()
Hey Adrian, über so etwas hatte ich häufiger nachgedacht, war mir aber nicht sicher, ob das Randfälle haben kann. Aber das muss man ggf. einfach beobachten. Danke dafür! Ich würde die target-Variable im buildscript dann aber gerne ganz los werden. Weiterhin sollten wir dann die bsps dann (bei uns heißt das "machine") nach den OpenWRT targets richten, damit da im Builddir nicht unerwartete Dinge passieren. (z.B. "mpc85xx" statt "wdr4900") Hierfür wäre dann statt "$machine" "$target" geeignet. Gruß Fabian
Hallo,
die target Variable hat halt den Vorteil, dass man das Verzeichnis an einer einzelnen Stelle ändern kann.
Deshalb hat der Patch auch nur ein paar Zeilen.
Eine andere Frage ist, ob die in die Funktion gehört. Im Prinzip kann man die auch an den Anfang des Skripts schreiben.
Das mit den bsps habe ich bei mir inzwischen ganz anders, da hält sich mein Bedarf in Grenzen, da in der off. Firmware dran rumzuspielen.
Grüße
Adrian
From: Fabian Bläse [mailto:fabian@blaese.de]
Sent: Mittwoch, 2. Oktober 2019 13:53
To: Adrian Schmutzler <freifunk@adrianschmutzler.de>; franken-dev@freifunk.net
Subject: Re: [PATCH 1/2] buildscript: Do not use target-dependent build directory
Hey Adrian,
über so etwas hatte ich häufiger nachgedacht, war mir aber nicht sicher, ob das Randfälle haben kann. Aber das muss man ggf. einfach beobachten.
Danke dafür!
Ich würde die target-Variable im buildscript dann aber gerne ganz los werden.
Weiterhin sollten wir dann die bsps dann (bei uns heißt das "machine") nach den OpenWRT targets richten, damit da im Builddir nicht unerwartete Dinge passieren. (z.B. "mpc85xx" statt "wdr4900")
Hierfür wäre dann statt "$machine" "$target" geeignet.
Gruß
Fabian
Hey Adrian, das Verzeichnis wird über die "builddir" Variable festgelegt, die "target" Variable wird dann ja auch darauf gesetzt.. Soweit ich nichts übersehen habe, ist diese Variable dann redundant. Gruß Fabian
Achso, du meinst alle „$target“ durch „$builddir“ ersetzen …
Das könnte man in der Tat machen.
From: Fabian Bläse [mailto:fabian@blaese.de]
Sent: Mittwoch, 2. Oktober 2019 14:48
To: Adrian Schmutzler <mail@adrianschmutzler.de>; franken-dev@freifunk.net
Subject: Re: [PATCH 1/2] buildscript: Do not use target-dependent build directory
Hey Adrian,
das Verzeichnis wird über die "builddir" Variable festgelegt, die "target" Variable wird dann ja auch darauf gesetzt..
Soweit ich nichts übersehen habe, ist diese Variable dann redundant.
Gruß
Fabian
So far, building the firmware happens in parallel subdirectories of "build", one for each subtarget. However, OpenWrt itself is capable of hosting several target builds in its directory, so there is no need for this extra separation. This patch thus build all targets/subtargets directly in the "build" folder. Since most of the time during build is spent for the toolchain, this will also significantly boost build time if more than one target is used. Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> --- bsp/board_ar71xx.bsp | 1 - bsp/board_mt7621.bsp | 1 - bsp/board_wdr4900.bsp | 1 - buildscript | 3 ++- 4 files changed, 2 insertions(+), 4 deletions(-)