Fimrware: Packages/Feeds Patches

So Jul 5 19:45:47 CEST 2015

Hallo Steffen

* Steffen Pankratz <kratz00 at gmx.de> [28-06-15 15:48]:
> Hi
> 
> 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. ;-)

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

> 
[..]
> 
> -- 
> Hermes powered by Manjaro Linux (Linux 4.1.0)
> 
> Best regards, Steffen Pankratz.

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

>  
>  #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. Schon blöd, dass man OpenWRT nicht out-of-tree bauen
kann (oder geht das doch irgendwie?).

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

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?

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

Tim

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

> -- 
> franken-dev mailing list
> franken-dev at freifunk.net
> http://lists.freifunk.net/mailman/listinfo/franken-dev-freifunk.net

-------------- 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/20150705/cd4e2d2b/attachment-0002.sig>