Fimrware: Packages/Feeds Patches

Mo Jul 6 20:52:55 CEST 2015

Hallo Steffen,

* Steffen Pankratz <kratz00 at gmx.de> [05-07-15 21:15]:
> On Sun, 5 Jul 2015 19:45:47 +0200
> Tim Niemeyer <tim.niemeyer at mastersword.de> wrote:
> 
> Hi Tim
> 
> danke fuers Review und deine Anmerkungen.
Kein Thema. Gern.

> 
> 
> > > 1. Inhalt der Patches
> > > Dieses Patches dienen als Vorbereitung zum Upgrade der Feed-Versionen.
> > > Sie verwenden aktuell aber noch dieselben Feed-Versionen.
> > > Alle Feeds verwenden ab sofort Git anstatt SVN.
> > > Ich habe geschaut passende Git revisions fuer die SVN revisions zu finden.
> > > Nicht verwendete Feeds/Packages habe ich entfernt.
> > > 
> > 
> > So, nun habe ich mir alle Patches angeschaut. Erstmal vielen Dank für
> > die Arbeit. Zusammenfassend kann man sagen, du hast da ein riesen Loch
> > gefunden, was wohl auch noch einige Arbeit benötigt.
> 
> :)
> 
> 
> > Grundsätzlich stört mich svn nicht, aber ich habe auch nichts dagegen,
> > wenn man git verwendet. Wir sollten das nehmen, was die Primärquelle
> > ist. Ich weiß gerade gar nicht wie der Stand ist, ob OpenWRT inzwischen
> > richtig umgezogen ist, oder ob die noch immer nur ein git-svn mirror
> > verwenden. Aber im Grunde ist es mir auch relativ egal. ;-)
> 
> Afaik, wird nur noch Git verwendet.
Ok, gut. Danke für die Info.

> 
> 
> > Patch 1 und 2 sind so also erstmal super. Ich hab trotzdem ein paar
> > Kommentare reingemalt, die sind aber rein FYI. Einige der Punkte
> > resultieren aus dem Problem, was du mit Patch 4 aufgedeckt hast.
> > Vieleicht sollten wir dieses Problem auch erst fixen und dann richtig
> > aufräumen. Es spricht aber auch nichts dagegen Patch 1 und 2 anzuwenden,
> > Die fehlenden Features können dann ja bei Bedarf wieder nachgezogen
> > werden.
> 
> Die fehlenden Features, wie du es nennst, sind in der aktuellen Firmware gar nicht drin.
> Es wurden damals nur die Packages-Feeds hinzugefuegt, ohne die jeweiligen Packages zu verwenden.
Ja, das war ein Fehler.

> 
> > Bei Patch 3 habe ich noch ne Rückfrage.
> > Bei Patch 4 würde gern in die Diskussion gehen.
> > Patch 5 scheint mir kaputt zu sein, magst du da nochmal gucken? (Oder
> > ggfs auch abwarten, was die Diskussion von Patch 4 ergibt?)
> 
> Ich habe jeweils unter deinen Anmerkungen kommentiert.
> 
> 
> > > From 471b24589abd3387843f33e6b3f617d4b69ccea0 Mon Sep 17 00:00:00 2001
> > > From: Steffen Pankratz <kratz00 at gmx.de>
> > > Date: Sun, 28 Jun 2015 14:01:47 +0200
> > > Subject: [PATCH 1/5] - removed unused packages and feeds
> > > 
> > > Signed-off-by: Steffen Pankratz <kratz00 at gmx.de>
> > Reviewed-by: Tim Niemeyer <tim.niemeyer at mastersword.de>
> > > ---
> > >  buildscript                            | 14 ++++------
> > >  src/packages/ffol/libwlocate/Makefile  | 51 ----------------------------------
> > >  src/packages/ffol/lwtrace/Makefile     | 48 --------------------------------
> > >  src/packages/ffol/nodewatcher/Makefile | 40 --------------------------
> > >  4 files changed, 5 insertions(+), 148 deletions(-)
> > >  delete mode 100644 src/packages/ffol/libwlocate/Makefile
> > >  delete mode 100644 src/packages/ffol/lwtrace/Makefile
> > >  delete mode 100644 src/packages/ffol/nodewatcher/Makefile
> > > 
> > > diff --git a/buildscript b/buildscript
> > > index 5488803..b704daf 100755
> > > --- a/buildscript
> > > +++ b/buildscript
> > > @@ -14,7 +14,7 @@ OPENWRT=(openwrt
> > >           svn
> > >           svn://svn.openwrt.org/openwrt/packages
> > >           $OPENWRTREV)
> > > -OPENWRT_PKGS="bridge ethtool gpioctl-sysfs horst libugpio mii-tool wavemon kmod-gpiotoggling"
> > > +OPENWRT_PKGS="bridge gpioctl-sysfs libugpio mii-tool kmod-gpiotoggling"
> > host und wavemon werden nicht verwendet? Hm, ja.
> > 
> > Dann ist das zwar ok, aber im Grunde sollten wir die Tools aufnehmen,
> > ohne Werkzeuge kann man keine Antenne ausrichten. So lange das
> > allerdings nicht benutzt wird ist es richtig das hier raus zu nehmen.
> 
> Ich habe alle Package-Feeds geloescht, wenn dessen Packages nicht im Image landen wuerde.
> Fuer die neue Firmware koennen wir die Packages dann aber gerne aufnehmen.
Ja, gern. Scheint aber offenbar bisher niemanden wirklich gestört zu
haben, also nicht die höchste Prio.

> 
> 
> > >  
> > >  #official openwrt routing packages
> > >  ROUTING=(routing
> > > @@ -22,20 +22,15 @@ ROUTING=(routing
> > >           https://github.com/openwrt-routing/packages.git
> > >           44318d05898069300ce2ad6eb05eb40f232e7123
> > >           ../../../build_patches/0001-batman-adv-no-rebroadcast-option.patch)
> > > -ROUTING_PKGS="alfred kmod-batman-adv kmod-batgat"
> > > +ROUTING_PKGS="kmod-batman-adv"
> > 
> > FYI: Vermutlich war mal angedacht die batman gateway selection zu
> > verwenden. Das könnte auch in Zukunft nochmal ein Thema sein, aber
> > ansonsten wie oben.
> > 
> > >  FASTD=(fastd
> > >         git
> > >         git://git.metameute.de/lff/pkg_fastd
> > >         v14-1)
> > > -FASTD_PKGS="fastd libuecc"
> > Eine (offenbar alte) Abhängigkeit von fastd.
> > 
> > > +FASTD_PKGS="fastd"
> > >  
> > > -FFOL=(ffol
> > > -      'local'
> > > -      ${PWD}/src/packages/ffol/)
> > > -FFOL_PKGS="nodewatcher"
> > > -
> > > -FEEDS=(OPENWRT ROUTING FASTD FFOL)
> > > +FEEDS=(OPENWRT ROUTING FASTD)
> > >  
> > >  checkout_svn(){
> > >  	local DIRECTORY=$1
> > > @@ -93,6 +88,7 @@ checkout_git(){
> > >  }
> > >  
> > >  get_source() {
> > > +	test -d src || mkdir src
> > >  	cd src
> > >  
> > >  	#Get the OpenWrt Core Source for Firmware
> > > diff --git a/src/packages/ffol/libwlocate/Makefile b/src/packages/ffol/libwlocate/Makefile
> > > deleted file mode 100644
> > > index 7946da3..0000000
> > [..] ja
> > > -- 
> > > 2.4.4
> > > 
> > 
> > > From 45cd812ffda41ee82da7bc32dffb99bf0c419e70 Mon Sep 17 00:00:00 2001
> > > From: Steffen Pankratz <kratz00 at gmx.de>
> > > Date: Sun, 28 Jun 2015 14:35:15 +0200
> > > Subject: [PATCH 2/5] - fixed usage of 'PACKAGEREV' and 'PACKAGEURL' variables
> > > 
> > > Signed-off-by: Steffen Pankratz <kratz00 at gmx.de>
> > Reviewed-by: Tim Niemeyer <tim.niemeyer at mastersword.de>
> > > ---
> > >  buildscript | 4 ++--
> > >  1 file changed, 2 insertions(+), 2 deletions(-)
> > > 
> > > diff --git a/buildscript b/buildscript
> > > index b704daf..c35dd65 100755
> > > --- a/buildscript
> > > +++ b/buildscript
> > > @@ -12,8 +12,8 @@ PACKAGEURL=svn://svn.openwrt.org/openwrt/packages
> > >  #official openwrt packages
> > >  OPENWRT=(openwrt
> > >           svn
> > > -         svn://svn.openwrt.org/openwrt/packages
> > > -         $OPENWRTREV)
> > > +         $PACKAGEURL
> > > +         $PACKAGEREV)
> > ja
> > 
> > >  OPENWRT_PKGS="bridge gpioctl-sysfs libugpio mii-tool kmod-gpiotoggling"
> > >  
> > >  #official openwrt routing packages
> > > -- 
> > > 2.4.4
> > > 
> > 
> > > From 2eea5cbb3934442afb94b9acb64ac4553432513c Mon Sep 17 00:00:00 2001
> > > From: Steffen Pankratz <kratz00 at gmx.de>
> > > Date: Sun, 28 Jun 2015 14:39:23 +0200
> > > Subject: [PATCH 3/5] - use the OpenWrt git repository instead of the svn
> > >  repository
> > > 
> > > Signed-off-by: Steffen Pankratz <kratz00 at gmx.de>
> > > ---
> > >  buildscript | 10 +++++-----
> > >  1 file changed, 5 insertions(+), 5 deletions(-)
> > > 
> > > diff --git a/buildscript b/buildscript
> > > index c35dd65..626acbd 100755
> > > --- a/buildscript
> > > +++ b/buildscript
> > > @@ -2,9 +2,9 @@
> > >  
> > >  builddir=./build
> > >  
> > > -OPENWRTREV=42096
> > > -OPENWRTURL=svn://svn.openwrt.org/openwrt/branches/barrier_breaker
> > > -PACKAGEREV=${OPENWRTREV}
> > > +OPENWRTREV="e77adae70c09298b871d074b35142d4845a0a7ca"
> > > +OPENWRTURL="git://git.openwrt.org/14.07/openwrt.git"
> > > +PACKAGEREV=42096
> > >  PACKAGEURL=svn://svn.openwrt.org/openwrt/packages
> > >  
> > >  ## Feed definition [0]: name aka directory [1]: type git vs svn [2]: url [3]: revision evtl [4]: Patches
> > > @@ -92,7 +92,7 @@ get_source() {
> > >  	cd src
> > >  
> > >  	#Get the OpenWrt Core Source for Firmware
> > > -	checkout_svn openwrt $OPENWRTURL $OPENWRTREV
> > > +	checkout_git openwrt $OPENWRTURL $OPENWRTREV
> > >  
> > >  	test -d packages || mkdir packages
> > >  	cd packages
> > > @@ -131,7 +131,7 @@ prepare() {
> > >  	test -d $builddir || mkdir $builddir
> > >  
> > >  	/bin/rm -rf $target
> > > -	svn export src/openwrt $target
> > > +	cp -R src/openwrt $target
> > In git könnten Dateirechte und Links einhalten sein. Mmn würde hier ein
> > cp -a besser sein. 
> 
> Ich denke du hast Recht, ist die bessere Loesung.
> Aktualisierter Patch anbei.
I see..

> 
> > Schon blöd, dass man OpenWRT nicht out-of-tree bauen
> > kann (oder geht das doch irgendwie?).
> 
> Noch nicht geschaut, sollten wir fuer die Firmware aber auf jeden Fall tun.
Ist vielleicht etwas, was man sich gesondert mal angucken sollte.

> 
> 
> > >  
> > >  	## generate own feeds.conf
> > >  	#this local variable should be globally configure variable used in get_source and here
> > > -- 
> > > 2.4.4
> > > 
> > 
> > > From cc9492907e63753db375b40b105a601b15bd7830 Mon Sep 17 00:00:00 2001
> > > From: Steffen Pankratz <kratz00 at gmx.de>
> > > Date: Sun, 28 Jun 2015 14:48:10 +0200
> > > Subject: [PATCH 4/5] - use the OpenWrt packages git repository instead of the
> > >  svn repository
> > > 
> > > Signed-off-by: Steffen Pankratz <kratz00 at gmx.de>
> > > ---
> > >  buildscript | 6 +++---
> > >  1 file changed, 3 insertions(+), 3 deletions(-)
> > > 
> > > diff --git a/buildscript b/buildscript
> > > index 626acbd..70b95ec 100755
> > > --- a/buildscript
> > > +++ b/buildscript
> > > @@ -4,14 +4,14 @@ builddir=./build
> > >  
> > >  OPENWRTREV="e77adae70c09298b871d074b35142d4845a0a7ca"
> > >  OPENWRTURL="git://git.openwrt.org/14.07/openwrt.git"
> > > -PACKAGEREV=42096
> > > -PACKAGEURL=svn://svn.openwrt.org/openwrt/packages
> > > +PACKAGEREV="36fe1d34e682299434e09e09947148b3bf7217a3"
> > > +PACKAGEURL="https://github.com/openwrt/packages.git"
> > Ich glaube das deckt sich gar nicht. Es zeigt aber ein gravierendes
> > Problem auf. Offenbar wurde irgendwann mal die OpenWRT SVN Version
> > hochgezogen, damit dann auch die Package-Version und in genau dieser ist
> > die Hälfte aus dem Repo gelöscht und auf github gezogen. Vermutlich sind
> > deswegen auch wavemon und horst nicht mehr in der Firmware drin.
> 
> Es sollte sich schon decken.
> Ich habe geschaut passende Git revisions fuer die SVN revisions zu
> finden.
> Und ja, ich denke auch, dass dies der Grund ist wieso einige
> Package-Feeds 'verschwunden' sind.
> Teil der Firmware waren diese Packages aber nie.
Ich schau nochmal ..
Ne, das passt einfach nicht.

2991 files changed, 173209 insertions(+), 304539 deletions(-), 13 modifications(!)

Das kommt zwar vom selben Datum, aber die Zweige haben unterschiedlich
entwickelt. Im svn wurde nach und nach gelöscht und im git nach und nach
hinzugefügt.

Ich denke an dieser Stelle ist das oldpackages Repo (aus dem Patch 5) das richtig:
88 files changed, 5803 insertions(+), 693 deletions(-)

Das wurde von svn://svn.openwrt.org/openwrt/branches/packages_14.07@42224
abgezweigt. Ist also etwas weiter als das svn was wir genutzt haben.

Mein Vorschlag:
-PACKAGEREV=42096
-PACKAGEURL=svn://svn.openwrt.org/openwrt/packages
+PACKAGEREV="ee10403aab0b9b23096532662471e8052377444c"
+PACKAGEURL="http://git.openwrt.org/14.07/packages.git"

Und dann Patch 5 einfach weg lassen.

> > Ich denke wir müssen in einem zukünftigen Schritt eine Package-Version
> > finden, die tatsächlich zu unserer OpenWRT Version passt.
> > Sinnvollerweise aktualisieren wir bei der Gelegenheit das OpenWRT.
> > Bis dahin können wir diesen Patch mMn aber gern aufnehmen.
> > Meinungen?
> 
> OpenWrt zu aktualisieren waere mein naechster Schritt.
> Ich wollte ja wie gesagt erst mal den SVN zu Git Schritt machen
> und dabei inhaltlich noch auf dem selben Stand bleiben.
> 
> 
> > >  
> > >  ## Feed definition [0]: name aka directory [1]: type git vs svn [2]: url [3]: revision evtl [4]: Patches
> > >  
> > >  #official openwrt packages
> > >  OPENWRT=(openwrt
> > > -         svn
> > > +         git
> > >           $PACKAGEURL
> > >           $PACKAGEREV)
> > >  OPENWRT_PKGS="bridge gpioctl-sysfs libugpio mii-tool kmod-gpiotoggling"
> > > -- 
> > > 2.4.4
> > > 
> > 
> > > From 131a3dabaf0b8339b6738627227f4e026095ad20 Mon Sep 17 00:00:00 2001
> > > From: Steffen Pankratz <kratz00 at gmx.de>
> > > Date: Sun, 28 Jun 2015 15:07:26 +0200
> > > Subject: [PATCH 5/5] - added OpenWrt oldpackages feed, to resolve missing
> > >  packages like 'bridge', 'gpioctl-sysfs', 'libugpio', 'mii-tool' and
> > >  'kmod-gpiotoggling'
> > > 
> > > Signed-off-by: Steffen Pankratz <kratz00 at gmx.de>
> > > ---
> > >  .gitignore  | 1 +
> > >  buildscript | 7 ++++++-
> > >  2 files changed, 7 insertions(+), 1 deletion(-)
> > > 
> > > diff --git a/.gitignore b/.gitignore
> > > index d2bf6cb..e0adea8 100644
> > > --- a/.gitignore
> > > +++ b/.gitignore
> > > @@ -7,5 +7,6 @@ src/openwrt/
> > >  src/packages/fastd/
> > >  src/packages/libremap/
> > >  src/packages/openwrt/
> > > +src/packages/openwrt_oldpackages/
> > >  src/packages/routing/
> > >  .project
> > > \ No newline at end of file
> > > diff --git a/buildscript b/buildscript
> > > index 70b95ec..0bba857 100755
> > > --- a/buildscript
> > > +++ b/buildscript
> > > @@ -16,6 +16,11 @@ OPENWRT=(openwrt
> > >           $PACKAGEREV)
> > >  OPENWRT_PKGS="bridge gpioctl-sysfs libugpio mii-tool kmod-gpiotoggling"
> > >  
> > > +OPENWRT_OLDPACKAGES=(openwrt_oldpackages
> > > +                     git
> > > +                     http://git.openwrt.org/14.07/packages.git
> > > +                     d70e69464ec00006b26ac684e808a6118dd4177a)
> > 
> > $ git config --local --get remote.origin.url
> > http://git.openwrt.org/14.07/packages.git
> > $ git show d70e69464ec00006b26ac684e808a6118dd4177a
> > fatal: bad object d70e69464ec00006b26ac684e808a6118dd4177a
> > 
> > Öhm?
> 
> Strange, keine Ahnung was ich da gemacht habe, die Revision ist wohl weg geschmolzen :)
> Patch angepasst und angehaengt.
> 
> Gruss
> -Steffen
> 
> -- 
> Hermes powered by Manjaro Linux (Linux 4.1.0)
> 
> Best regards, Steffen Pankratz.

> From 2eea5cbb3934442afb94b9acb64ac4553432513c Mon Sep 17 00:00:00 2001
> From: Steffen Pankratz <kratz00 at gmx.de>
> Date: Sun, 28 Jun 2015 14:39:23 +0200
> Subject: [PATCH 3/5] - use the OpenWrt git repository instead of the svn
>  repository

Schaut gut aus. Danke.

> 
> Signed-off-by: Steffen Pankratz <kratz00 at gmx.de>
Reviewed-by: Tim Niemeyer <tim.niemeyer at mastersword.de>
> ---
>  buildscript | 10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/buildscript b/buildscript
> index c35dd65..626acbd 100755
> --- a/buildscript
> +++ b/buildscript
> @@ -2,9 +2,9 @@
>  
>  builddir=./build
>  
> -OPENWRTREV=42096
> -OPENWRTURL=svn://svn.openwrt.org/openwrt/branches/barrier_breaker
> -PACKAGEREV=${OPENWRTREV}
> +OPENWRTREV="e77adae70c09298b871d074b35142d4845a0a7ca"
> +OPENWRTURL="git://git.openwrt.org/14.07/openwrt.git"
> +PACKAGEREV=42096
>  PACKAGEURL=svn://svn.openwrt.org/openwrt/packages
>  
>  ## Feed definition [0]: name aka directory [1]: type git vs svn [2]: url [3]: revision evtl [4]: Patches
> @@ -92,7 +92,7 @@ get_source() {
>  	cd src
>  
>  	#Get the OpenWrt Core Source for Firmware
> -	checkout_svn openwrt $OPENWRTURL $OPENWRTREV
> +	checkout_git openwrt $OPENWRTURL $OPENWRTREV
>  
>  	test -d packages || mkdir packages
>  	cd packages
> @@ -131,7 +131,7 @@ prepare() {
>  	test -d $builddir || mkdir $builddir
>  
>  	/bin/rm -rf $target
> -	svn export src/openwrt $target
> +	cp -a src/openwrt $target
>  
>  	## generate own feeds.conf
>  	#this local variable should be globally configure variable used in get_source and here
> -- 
> 2.4.4
> 

> From 131a3dabaf0b8339b6738627227f4e026095ad20 Mon Sep 17 00:00:00 2001
> From: Steffen Pankratz <kratz00 at gmx.de>
> Date: Sun, 28 Jun 2015 15:07:26 +0200
> Subject: [PATCH 5/5] - added OpenWrt oldpackages feed, to resolve missing
>  packages like 'bridge', 'gpioctl-sysfs', 'libugpio', 'mii-tool' and
>  'kmod-gpiotoggling'

Also der Patch schaut in sich konsistent aus. Allerdings halte es für besser
dieses oldpackages Repo für das obige packages Repo zu nehmen. Imho ist das hier
aktuell nur nötig, weil die Pakete in dem damaligen Stand noch nicht komplett
umgezogen waren.

Tim

> 
> Signed-off-by: Steffen Pankratz <kratz00 at gmx.de>
> ---
>  .gitignore  | 1 +
>  buildscript | 7 ++++++-
>  2 files changed, 7 insertions(+), 1 deletion(-)
> 
> diff --git a/.gitignore b/.gitignore
> index d2bf6cb..e0adea8 100644
> --- a/.gitignore
> +++ b/.gitignore
> @@ -7,5 +7,6 @@ src/openwrt/
>  src/packages/fastd/
>  src/packages/libremap/
>  src/packages/openwrt/
> +src/packages/openwrt_oldpackages/
>  src/packages/routing/
>  .project
> \ No newline at end of file
> diff --git a/buildscript b/buildscript
> index 70b95ec..0bba857 100755
> --- a/buildscript
> +++ b/buildscript
> @@ -16,6 +16,11 @@ OPENWRT=(openwrt
>           $PACKAGEREV)
>  OPENWRT_PKGS="bridge gpioctl-sysfs libugpio mii-tool kmod-gpiotoggling"
>  
> +OPENWRT_OLDPACKAGES=(openwrt_oldpackages
> +                     git
> +                     http://git.openwrt.org/14.07/packages.git
> +                     ee10403aab0b9b23096532662471e8052377444c)
> +
>  #official openwrt routing packages
>  ROUTING=(routing
>           git
> @@ -30,7 +35,7 @@ FASTD=(fastd
>         v14-1)
>  FASTD_PKGS="fastd"
>  
> -FEEDS=(OPENWRT ROUTING FASTD)
> +FEEDS=(OPENWRT OPENWRT_OLDPACKAGES ROUTING FASTD)
>  
>  checkout_svn(){
>  	local DIRECTORY=$1
> -- 
> 2.4.4
> 
-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : signature.asc
Dateityp    : application/pgp-signature
Dateigröße  : 836 bytes
Beschreibung: Digital signature
URL         : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20150706/63dcc760/attachment-0002.sig>