[PATCH v2 2/2] Changed patch behavior because if was not comfortable. Now you don't need to add every patch into buildscript.
Steffen Pankratz
kratz00 at gmx.de
Mi Mär 16 19:44:26 CET 2016
On Tue, 15 Mar 2016 20:37:33 +0100
Jan Kraus <mayosemmel at googlemail.com> wrote:
Hi Jan,
Anmerkungen "inline".
> Signed-off-by: Jan Kraus <mayosemmel at gmail.com>
> ---
> buildscript | 16 +++++++---------
> 1 file changed, 7 insertions(+), 9 deletions(-)
>
> diff --git a/buildscript b/buildscript
> index 5937d9b..c5e487a 100755
> --- a/buildscript
> +++ b/buildscript
> @@ -116,6 +116,12 @@ get_source() {
> cd .. # src
> }
>
> +patch() {
> + for patch in "$PWD"/build_patches/*.patch; do
> + patch -p0 -d "$target" -i "$PWD"/build_patches/$patch
> + done
> +}
> +
Das Skript verwendet zum Einruecken 4 Spaces, keine Tabs.
Deine Aenderungen enthalten Tabs und auch Spaces.
Bitte konsequent Spaces verwenden.
$patch besser "double quoten", um ungewollten Bash Eigenheiten zu vermeiden.
patch -p0 -d "$target" -i "$PWD"/build_patches/"$patch"
> prepare() {
> get_source
>
> @@ -155,15 +161,7 @@ prepare() {
> fi
> done
>
> - # todo evaluate this with the new openwrt
> - # This changes the default behavior of sysupgrade to not save the config
> - patch -p0 -d "$target" -i "$PWD"/build_patches/sysupgrade_no_config_save.patch
> -
> - # set default password to 'ffol'
> - patch -p0 -d "$target" -i "$PWD"/build_patches/set_root_password.patch
> -
> - #add Support for GL-AR150 see also http://git.openwrt.org/?p=openwrt.git;a=commit;h=8211e81bce57392635fa3ad6572514387118e4a5
> - patch -p0 -d "$target" -i "$PWD"/build_patches/ar71xx-add-support-for-GL-AR150.patch
> + patch
Auch hier Tab vs. Spaces.
Gruss
-Steffen
--
Hermes powered by Manjaro Linux (Linux 4.4.5)
Best regards, Steffen Pankratz.
-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname : nicht verfügbar
Dateityp : application/pgp-signature
Dateigröße : 181 bytes
Beschreibung: OpenPGP digital signature
URL : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20160316/c35a699a/attachment-0002.sig>
Mehr Informationen über die Mailingliste franken-dev