[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