[PATCH v2 2/2] Changed patch behavior because if was not comfortable. Now you don't need to add every patch into buildscript.

[mayosemmel]

Hallo Tim,

leider kann ich dem nicht ganz folgen.

Im buildscript habe ich folgende Definition gefunden:
## Feed definition [0]: name aka directory, [1]: url, [2]: revision, [3..n]: patches
Am Beispiel von OpenWrt sieht das dann so aus:
OPENWRT=(openwrt
         $PACKAGEURL
         $PACKAGEREV)
OPENWRT_PKGS="gpioctl-sysfs libugpio fastd haserl"

Allerdings ist für keinen einzigen Feed ein Patch definiert, deshalb
fällt es mir etwas schwer, die Definition zu verstehen.

Ich würde nun folgendes machen:
das [3..n] für die patches in [3] ändern und da den Pfad zu den
entsprechenden Patches reinschreiben.
Passt das dann?

BTW: Wo wird die Variable OPENWRT_PCKGS bzw. die der anderen Feeds
verarbeitet/benötigt?

Grüße Jan

Am Dienstag, den 15.03.2016, 22:30 +0100 schrieb Tim Niemeyer:
> Hi Mayosemmel
> 
> Den Patchset finde ich soweit gut. Könntest du aber bitte die OpenWRT
> Patches dann in ein Unterverzeichnis "build_patches/openwrt" packen. 
> 
> Aktuell haben wir zwar keine anderen, aber das buildscript sieht vor,
> dass man beliebige Patches eben auch an die Feed kleben kann. Das würde
> dann hier zu einem Konflikt kommen. Deluxe wäre, wenn jeder Feed anhand
> dessen Namen ein Verzeichnis im build_patches Verzeichnis bekommt, wo
> dann die Patches drin sind. Aber das musst du nicht unbedingt einbauen.
> 
> Am Dienstag, den 15.03.2016, 20:37 +0100 schrieb Jan Kraus:
> > 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
> Hier ist die Einrückung kaputt.
> 
> Tim
> 
> > +}
> > +
> >  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
> >  
> >      #saves ~200MB for each build
> >      test -d ./src/dl || mkdir ./src/dl
> > -- 
> > 2.1.4
> > 
> 

-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : signature.asc
Dateityp    : application/pgp-signature
Dateigröße  : 473 bytes
Beschreibung: This is a digitally signed message part
URL         : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20160316/f55546d3/attachment-0002.sig>