Fimrware: Packages/Feeds Patches

So Jul 5 21:15:44 CEST 2015

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.

> > 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.

> 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.

> 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.

> >  
> >  #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.

> 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.

> >  
> >  	## 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 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.
-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : 0003-use-the-OpenWrt-git-repository-instead-of-the-svn-re.patch
Dateityp    : text/x-patch
Dateigröße  : 1429 bytes
Beschreibung: nicht verfügbar
URL         : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20150705/3b421126/attachment-0004.bin>
-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : 0005-added-OpenWrt-oldpackages-feed-to-resolve-missing-pa.patch
Dateityp    : text/x-patch
Dateigröße  : 1461 bytes
Beschreibung: nicht verfügbar
URL         : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20150705/3b421126/attachment-0005.bin>
-------------- 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/20150705/3b421126/attachment-0002.sig>