Fimrware: Packages/Feeds Patches
Tim Niemeyer
tim.niemeyer at mastersword.de
Do Jul 9 20:54:06 CEST 2015
Patch 5 reviewed und applied.
Tim
Am Sonntag, den 05.07.2015, 21:15 +0200 schrieb Steffen Pankratz:
> 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
>
-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname : signature.asc
Dateityp : application/pgp-signature
Dateigröße : 819 bytes
Beschreibung: This is a digitally signed message part
URL : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20150709/454033be/attachment-0002.sig>
Mehr Informationen über die Mailingliste franken-dev