[LEDE-DEV] Release in view?

2016-08-16 Thread Jan-Tarek Butt
Hi, I think this question is not new .. and many people are looking curious to the LEDE project. I search on this list for discussions about a first release, but maybe I didn't find it or it does not exist. So I have decided to ask here for a schedule of a first LEDE Release. If there any data

Re: [LEDE-DEV] Documentation for router support means the famous Table of Hardware

2016-09-08 Thread Jan-Tarek Butt
On 09/08/16 11:08, Russell Senior wrote: >> "Eric" == Eric Schultz writes: > > Eric> I think the idea of having the hardware support documentation > Eric> managed by experts is a good idea. Non-experts can, and should, > Eric> have plenty of areas to contribute

Re: [LEDE-DEV] Documentation for router support means the famous Table of Hardware

2016-09-08 Thread Jan-Tarek Butt
> Since I was referenced in that thread on the OpenWrt forum, I'll give my > two cents. The documentation initiative I tried to start seemed to > fizzle out from the difficulty in getting the front page of the wiki > changed to be more useful. When changes didn't seem possible (and it > wasn't

Re: [LEDE-DEV] Documentation for router support means the famous Table of Hardware

2016-09-08 Thread Jan-Tarek Butt
On 09/08/16 12:59, Karl Palsson wrote: > > Jan-Tarek Butt <ta...@ring0.de> wrote: >>> Since I was referenced in that thread on the OpenWrt forum, I'll give my >>> two cents. The documentation initiative I tried to start seemed to >>> fizzle out from t

Re: [LEDE-DEV] Documentation for router support means the famous Table of Hardware

2016-09-08 Thread Jan-Tarek Butt
On 09/08/16 19:38, Thomas Endt wrote: > [Sorry if this message is out-of-thread. I joined the mailinglist only > today] No problem :) > Russel> If you want something that's pretty, hire someone to watch > Russel> RecentChanges and clean stuff up. > > I did this quite intensively in the past,

Re: [LEDE-DEV] A Wiki for LEDE Documentation

2016-09-13 Thread Jan-Tarek Butt
Hi, > I would definetely wait for Jow to answer on this subject. AFAIK he is quite > busy these days. > A LEDE wiki should be part of the LEDE infrastructure, with proper control > and multiple admins, in order to avoid situations like at the beginning of > this year. > > Give Jow a little time

Re: [LEDE-DEV] A Wiki for LEDE Documentation

2016-09-12 Thread Jan-Tarek Butt
Hi, > Yeah, I'm also willing to dedicate some hours per week to wiki gardening, > whatever wiki technology is chosen. Shall I do set up a dokuwiki on our open wireless community infrastructure? cheers Tarek signature.asc Description: OpenPGP digital signature

[LEDE-DEV] [PATCH v2] nvram: fix globbing, word splitting and arithmetic variables

2016-10-05 Thread Jan-Tarek Butt
Double quote to prevent globbing and word splitting. $/${} is unnecessary on arithmetic variables. Remove unused variables Signed-off-by: Jan-Tarek Butt <ta...@ring0.de> --- package/utils/nvram/files/nvram.init | 25 - 1 file changed, 12 insertions(+), 13 del

[LEDE-DEV] [PATCH v2] include: add luaSrcDiet call define to package.mk

2016-10-06 Thread Jan-Tarek Butt
Signed-off-by: Jan-Tarek Butt <ta...@ring0.de> --- include/package.mk | 17 - 1 file changed, 16 insertions(+), 1 deletion(-) diff --git a/include/package.mk b/include/package.mk index fa5c4e7..fb76057 100644 --- a/include/package.mk +++ b/include/package.mk @@ -5,

[LEDE-DEV] [PATCH v2] uboot-envtools: remove function vals and fix indentation

2016-10-06 Thread Jan-Tarek Butt
Signed-off-by: Jan-Tarek Butt <ta...@ring0.de> --- package/boot/uboot-envtools/files/uboot-envtools.sh | 21 - 1 file changed, 8 insertions(+), 13 deletions(-) diff --git a/package/boot/uboot-envtools/files/uboot-envtools.sh b/package/boot/uboot-envtools/files

Re: [LEDE-DEV] [PATCH v2] ct-bugcheck: fix globbing, word splitting and formatting

2016-10-06 Thread Jan-Tarek Butt
Hi John, > On 05/10/2016 15:37, Jan-Tarek Butt wrote: >> Double quote to prevent globbing and word splitting. >> { cmd1; cmd2; } >> file instead of individual redirects. >> simplifying syntax >> >> Signed-off-by: Jan-Tarek Butt <ta...@ring0.de>

[LEDE-DEV] [PATCH v3 04/13] scripts: $/${} is unnecessary on arithmetic variables

2016-10-06 Thread Jan-Tarek Butt
Signed-off-by: Jan-Tarek Butt <ta...@ring0.de> --- scripts/om-fwupgradecfg-gen.sh | 10 +- scripts/ubinize-image.sh | 10 +- 2 files changed, 10 insertions(+), 10 deletions(-) diff --git a/scripts/om-fwupgradecfg-gen.sh b/scripts/om-fwupgradecfg-gen.sh index 9

[LEDE-DEV] [PATCH v3 00/13] shell code improvement for /scripts

2016-10-06 Thread Jan-Tarek Butt
Hi, hopefully now all is clear. SoB should now set by all patches. Also I change the commit messages a bit. If something unclear please tell me. For me that is a good praxis to learn it properly :) cheers Tarek Jan-Tarek Butt (13): scripts: Double quote to prevent globbing and word splitting

[LEDE-DEV] [PATCH v3 12/13] scripts: use * instead of @ to concatenate

2016-10-06 Thread Jan-Tarek Butt
Signed-off-by: Jan-Tarek Butt <ta...@ring0.de> --- scripts/config/lxdialog/check-lxdialog.sh | 4 ++-- scripts/ubinize-image.sh | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/scripts/config/lxdialog/check-lxdialog.sh b/scripts/config/lxdialog

[LEDE-DEV] [PATCH v3 06/13] scripts: redirect matters, the 2>&1 has to be last

2016-10-06 Thread Jan-Tarek Butt
Signed-off-by: Jan-Tarek Butt <ta...@ring0.de> --- scripts/make-ipkg-dir.sh | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/scripts/make-ipkg-dir.sh b/scripts/make-ipkg-dir.sh index 529e430..ba1a8df 100755 --- a/scripts/make-ipkg-dir.sh +++ b/scripts/make-ipkg-

[LEDE-DEV] [PATCH v3 01/13] scripts: Double quote to prevent globbing and word splitting

2016-10-06 Thread Jan-Tarek Butt
Signed-off-by: Jan-Tarek Butt <ta...@ring0.de> --- scripts/arm-magic.sh | 11 ++- scripts/clean-package.sh | 2 +- scripts/combined-ext-image.sh| 10 +- scripts/combined-image.sh| 2 +- scripts/deptest.sh | 12 ++-- s

[LEDE-DEV] [PATCH v3 08/13] scripts: Use $(..) instead of legacy `..`

2016-10-06 Thread Jan-Tarek Butt
Signed-off-by: Jan-Tarek Butt <ta...@ring0.de> --- scripts/deptest.sh | 4 ++-- scripts/gen-dependencies.sh | 4 ++-- scripts/ipkg-make-index.sh | 4 ++-- scripts/mkits.sh| 4 ++-- scripts/patch-kernel.sh | 2 +- scripts/strip-kmod.sh | 2 +- 6 files chang

[LEDE-DEV] [PATCH v3 13/13] scripts: read without -r will mangle backslashes

2016-10-06 Thread Jan-Tarek Butt
Signed-off-by: Jan-Tarek Butt <ta...@ring0.de> --- scripts/clean-package.sh | 4 ++-- scripts/rstrip.sh| 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/scripts/clean-package.sh b/scripts/clean-package.sh index 2de3c51..0940732 100755 --- a/scripts/clean-pack

[LEDE-DEV] [PATCH v2] ct-bugcheck: fix globbing, word splitting and formatting

2016-10-05 Thread Jan-Tarek Butt
Double quote to prevent globbing and word splitting. { cmd1; cmd2; } >> file instead of individual redirects. simplifying syntax Signed-off-by: Jan-Tarek Butt <ta...@ring0.de> --- package/utils/ct-bugcheck/src/bugcheck.sh | 177 ++ 1 file changed, 83 inser

Re: [LEDE-DEV] Automated shell code debugging

2016-10-04 Thread Jan-Tarek Butt
On 10/02/16 18:10, Bastian Bittorf wrote: > * Jan-Tarek Butt <ta...@ring0.de> [02.10.2016 17:48]: >> As a part of code quallity enhancemend and prevenition. We can do an >> ondemand daemon as engine to >> check all shell scrips. We can use the opensource Project sh

Re: [LEDE-DEV] package/boot/uboot-envtools/files/uboot-envtools.sh

2016-10-04 Thread Jan-Tarek Butt
On 10/04/16 07:35, Mathias Kresin wrote: > 04.10.2016 01:49, Jan-Tarek Butt: >> Hi, >> >> Does anyone understand what this "variables" in line 24 to 28 are use [0]? >> >> [0] >> https://git.lede-project.org/?p=source.git;a=blob;f=pack

[LEDE-DEV] [PATCH 1/3] nvram: fix globbing, word splitting and arithmetic variables

2016-10-04 Thread Jan-Tarek Butt
Double quote to prevent globbing and word splitting. $/${} is unnecessary on arithmetic variables. Remove unused variables --- package/utils/nvram/files/nvram.init | 27 +-- 1 file changed, 13 insertions(+), 14 deletions(-) diff --git

[LEDE-DEV] [PATCH 2/3] ct-bugcheck: fix globbing, word splitting and formatting

2016-10-04 Thread Jan-Tarek Butt
Double quote to prevent globbing and word splitting. { cmd1; cmd2; } >> file instead of individual redirects. --- package/utils/ct-bugcheck/src/bugcheck.sh | 154 +++--- 1 file changed, 77 insertions(+), 77 deletions(-) diff --git

Re: [LEDE-DEV] [PATCH 1/3] nvram: fix globbing, word splitting and arithmetic variables

2016-10-04 Thread Jan-Tarek Butt
On 10/04/16 14:51, John Crispin wrote: > > > On 04/10/2016 14:47, Jan-Tarek Butt wrote: >> Double quote to prevent globbing and word splitting. >> $/${} is unnecessary on arithmetic variables. >> Remove unused variables > > SoB missing here aswell >

Re: [LEDE-DEV] [PATCH v3 04/13] scripts: $/${} is unnecessary on arithmetic variables

2016-10-06 Thread Jan-Tarek Butt
> > On 06/10/2016 16:44, Bastian Bittorf wrote: >> * Jo-Philipp Wich [06.10.2016 16:35]: >>> removing the dollar signs lets expressions succeed which would otherwise >>> fail - which means it introduces a behavior change. >> >> thanks for showing this. i must repeat myself: >> >>

Re: [LEDE-DEV] A Wiki for LEDE Documentation

2016-09-19 Thread Jan-Tarek Butt
Hi, On 09/15/16 20:02, Thomas Endt wrote: >> I think we should avoid forums for now and should lay out the wiki's >> plans in a wiki page and activate a plugin (there are at least a couple >> for docuwiki) to make a "talk" page or "discussion" page like >> Wikipedia's for wiki volunteer

Re: [LEDE-DEV] LEDE Wiki Teambuilding

2016-09-19 Thread Jan-Tarek Butt
NameFunction == Martin Tippmann (m...@i3o.de), Jan-Tarek Butt t...@ring0.deInfrastructure / I could take care of nginx, caching, let's encrypt, mysql, php, memcache whatever. Thomas Endt, wiki admin / enabler / maintenance

[LEDE-DEV] Documentation for router support means the famous Table of Hardware

2016-09-07 Thread Jan-Tarek Butt
Hi, I am Tarek. ndb and I had discuss about the hardware support documentation for LEDE. So I send these proposal to create a asciidoc documentation nearly similar to the OpenWRT table of hardware. The idea is to create a Main table were users can fast and easily search for devices. Each device

Re: [LEDE-DEV] Call defines for minifying scripting languages

2016-10-03 Thread Jan-Tarek Butt
On 10/03/16 13:14, Karl Palsson wrote: > > Jan-Tarek Butt <ta...@ring0.de> wrote: >> Hi, >> >> My Idea ist to create calldefs to minify all scriptes there are >> interpreted by run time. >> >> As example see the mail "[PATCH] LuaSrcDiet

Re: [LEDE-DEV] Call defines for minifying scripting languages

2016-10-03 Thread Jan-Tarek Butt
On 10/03/16 14:18, Piotr Dymacz wrote: > Hello, > > 2016-10-03 13:14 GMT+02:00 Karl Palsson <ka...@tweak.net.au>: >> >> Jan-Tarek Butt <ta...@ring0.de> wrote: > [snip] >>> 1. Reducing memory size on firmware images. >> >> But

[LEDE-DEV] package/boot/uboot-envtools/files/uboot-envtools.sh

2016-10-03 Thread Jan-Tarek Butt
Hi, Does anyone understand what this "variables" in line 24 to 28 are use [0]? [0] https://git.lede-project.org/?p=source.git;a=blob;f=package/boot/uboot-envtools/files/uboot-envtools.sh;h=e21b28367751fdd86897f3a47911f3a3ffbdbdfb;hb=HEAD cheers Tarek signature.asc Description: OpenPGP

[LEDE-DEV] [PATCH] package/utils/nvram/files/nvram.init * Double quote to prevent globbing and word splitting. * $/${} is unnecessary on arithmetic variables. * remove unused variables

2016-10-03 Thread Jan-Tarek Butt
Signed-off-by: Jan-Tarek Butt <ta...@ring0.de> --- package/utils/nvram/files/nvram.init | 27 +-- 1 file changed, 13 insertions(+), 14 deletions(-) diff --git a/package/utils/nvram/files/nvram.init b/package/utils/nvram/files/nvram.init index 467ab28..ec06e7a

[LEDE-DEV] [PATCH] package/utils/ct-bugcheck/src/bugcheck.sh * Double quote to prevent globbing and word splitting. * { cmd1; cmd2; } >> file instead of individual redirects. * fix formating

2016-10-03 Thread Jan-Tarek Butt
Signed-off-by: Jan-Tarek Butt <ta...@ring0.de> --- package/utils/ct-bugcheck/src/bugcheck.sh | 154 +++--- 1 file changed, 77 insertions(+), 77 deletions(-) diff --git a/package/utils/ct-bugcheck/src/bugcheck.sh b/package/utils/ct-bugcheck/src/bugcheck.sh index 8

[LEDE-DEV] [patch master 14/15] Expressions don't expand in single quotes, use double quotes for that

2016-09-30 Thread Jan-Tarek Butt
--- scripts/deptest.sh | 2 +- scripts/ext-toolchain.sh | 24 2 files changed, 13 insertions(+), 13 deletions(-) diff --git a/scripts/deptest.sh b/scripts/deptest.sh index 0b0e7a9..2b156b3 100755 --- a/scripts/deptest.sh +++ b/scripts/deptest.sh @@ -88,7 +88,7 @@

[LEDE-DEV] [patch master 01/15] Double quote to prevent globbing and word splitting

2016-09-30 Thread Jan-Tarek Butt
--- scripts/arm-magic.sh | 12 ++-- scripts/clean-package.sh | 2 +- scripts/combined-ext-image.sh| 10 +- scripts/combined-image.sh| 2 +- scripts/deptest.sh | 12 ++-- scripts/ext-toolchain.sh | 16

[LEDE-DEV] [patch master 02/15] code style: remove whitespaces on end of lines

2016-09-30 Thread Jan-Tarek Butt
--- scripts/combined-ext-image.sh | 4 ++-- scripts/flashing/flash.sh | 6 +++--- scripts/patch-kernel.sh | 20 ++-- scripts/rstrip.sh | 2 +- 4 files changed, 16 insertions(+), 16 deletions(-) diff --git a/scripts/combined-ext-image.sh

[LEDE-DEV] [patch master 00/15] shell code improvement for /scripts

2016-09-30 Thread Jan-Tarek Butt
Hi, I did some bugfixing and code quality improvement for all shell scrips inside the dir ./scripts. cheers Tarek Jan-Tarek Butt (15): Double quote to prevent globbing and word splitting code style: remove whitespaces on end of lines Add and fix a shebang $/${} is unnecessary

[LEDE-DEV] [patch master 10/15] quote literal {

2016-09-30 Thread Jan-Tarek Butt
--- scripts/getver.sh | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/scripts/getver.sh b/scripts/getver.sh index f8b4ed3..27f246e 100755 --- a/scripts/getver.sh +++ b/scripts/getver.sh @@ -25,8 +25,8 @@ try_git() { ;; *)

[LEDE-DEV] [patch master 11/15] Consider using { cmd1; cmd2; } >> file instead of individual redirects

2016-09-30 Thread Jan-Tarek Butt
--- scripts/config/lxdialog/check-lxdialog.sh | 16 +++- scripts/diffconfig.sh | 16 +++- scripts/ext-toolchain.sh | 43 +-- 3 files changed, 42 insertions(+), 33 deletions(-) diff --git

[LEDE-DEV] [patch master 09/15] Argument mixes string and array. Use $*

2016-09-30 Thread Jan-Tarek Butt
--- scripts/mkits.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/mkits.sh b/scripts/mkits.sh index cf176af..937f712 100755 --- a/scripts/mkits.sh +++ b/scripts/mkits.sh @@ -41,7 +41,7 @@ do k ) KERNEL=$OPTARG;; o ) OUTPUT=$OPTARG;;

[LEDE-DEV] [patch master 03/15] Add and fix a shebang

2016-09-30 Thread Jan-Tarek Butt
--- scripts/freebsd.sh | 1 + scripts/om-fwupgradecfg-gen.sh | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/scripts/freebsd.sh b/scripts/freebsd.sh index 3107861..a6cca0e 100644 --- a/scripts/freebsd.sh +++ b/scripts/freebsd.sh @@ -1 +1,2 @@ +#!/bin/sh env

[LEDE-DEV] [patch master 04/15] $/${} is unnecessary on arithmetic variables

2016-09-30 Thread Jan-Tarek Butt
--- scripts/om-fwupgradecfg-gen.sh | 10 +- scripts/ubinize-image.sh | 10 +- 2 files changed, 10 insertions(+), 10 deletions(-) diff --git a/scripts/om-fwupgradecfg-gen.sh b/scripts/om-fwupgradecfg-gen.sh index 93f6794..5cbb8d1 100644 --- a/scripts/om-fwupgradecfg-gen.sh

[LEDE-DEV] [patch master 12/15] use * instead of @ to concatenate

2016-09-30 Thread Jan-Tarek Butt
--- scripts/config/lxdialog/check-lxdialog.sh | 4 ++-- scripts/ubinize-image.sh | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/scripts/config/lxdialog/check-lxdialog.sh b/scripts/config/lxdialog/check-lxdialog.sh index 02f05ad..0791372 100644 ---

Re: [LEDE-DEV] [patch master 00/15] shell code improvement for /scripts

2016-09-30 Thread Jan-Tarek Butt
On 09/30/16 22:19, John Crispin wrote: > On 30/09/2016 22:02, Jan-Tarek Butt wrote: >> Hi, >> >> I did some bugfixing and code quality improvement for >> all shell scrips inside the dir ./scripts. >> >> cheers >> Tarek > > > Hi Tarek, &g

Re: [LEDE-DEV] [patch master 01/15] Double quote to prevent globbing and word splitting

2016-09-30 Thread Jan-Tarek Butt
l now I was always > stopped by the sheer amount of changes necessary. Thank you! I just do some small stuff :) Butt I plan to continiure work on it. > Am Fri, 30 Sep 2016 22:02:15 +0200 > schrieb Jan-Tarek Butt <ta...@ring0.de>: > >> [..] >> diff --git a/scripts/arm-

[LEDE-DEV] [PATCH] LuaSrcDiet call define for lua code Minifying

2016-10-02 Thread Jan-Tarek Butt
://git.nordwest.freifunk.net/ffnw-firmware/packages/raw/master/hoodselector/luasrc/hoodselector Jan-Tarek Butt (1): add luaSrcDiet call define to package.mk include/package.mk | 17 - 1 file changed, 16 insertions(+), 1 deletion(-) -- 2.10.0 ___ Lede

[LEDE-DEV] [PATCH] add luaSrcDiet call define to package.mk

2016-10-02 Thread Jan-Tarek Butt
--- include/package.mk | 17 - 1 file changed, 16 insertions(+), 1 deletion(-) diff --git a/include/package.mk b/include/package.mk index fa5c4e7..fb76057 100644 --- a/include/package.mk +++ b/include/package.mk @@ -5,6 +5,8 @@ # See /LICENSE for more information. # +#

[LEDE-DEV] [PATCH] package/uboot-envtools.sh remove function vals and fix indentation

2016-10-02 Thread Jan-Tarek Butt
--- package/boot/uboot-envtools/files/uboot-envtools.sh | 21 - 1 file changed, 8 insertions(+), 13 deletions(-) diff --git a/package/boot/uboot-envtools/files/uboot-envtools.sh b/package/boot/uboot-envtools/files/uboot-envtools.sh index e21b283..ad41e07 100644 ---

[LEDE-DEV] Call defines for minifying scripting languages

2016-10-03 Thread Jan-Tarek Butt
Hi, My Idea ist to create calldefs to minify all scriptes there are interpreted by run time. As example see the mail "[PATCH] LuaSrcDiet call define for lua code Minifying". I plan to write also calldefs for shellscripts, python and perl. They brings vareous of features. 1. Reducing memory

Re: [LEDE-DEV] [patch master 00/15] shell code improvement for /scripts

2016-10-01 Thread Jan-Tarek Butt
Hi Alberto, On 09/30/16 23:02, Alberto Bursi wrote: > I'm suspecting he did use shellcheck.net (or its local version, > shellcheck) to find the errors. > > That's a great tool to find all dumb mistakes and bad practices in scripts. > > It's also opensource: > >

Re: [LEDE-DEV] Automated shell code debugging

2016-10-01 Thread Jan-Tarek Butt
Hi, On 10/01/16 01:00, Lars Kruse wrote: > Am Sat, 1 Oct 2016 00:11:23 +0200 > schrieb Martin Tippmann : > >> [..] >> I'm not a shell pro but I found it very difficult to impossible to >> write something that passes without any hints/errors. >> >> There is a way to avoid this by

[LEDE-DEV] mailman force using ssl

2016-10-01 Thread Jan-Tarek Butt
Hi, I reconize that mailman does not redirect automaticaly to ssl Maybe we can change it? Also the default settings sends a password remembering every month. cheers Tarek signature.asc Description: OpenPGP digital signature ___ Lede-dev mailing

Re: [LEDE-DEV] [patch master 05/15] Use cd ... || exit in case cd fails

2016-10-01 Thread Jan-Tarek Butt
On 10/01/16 00:05, Lars Kruse wrote: > Hi, > > > Am Fri, 30 Sep 2016 22:02:19 +0200 > schrieb Jan-Tarek Butt <ta...@ring0.de>: > > >> diff --git a/scripts/clean-package.sh b/scripts/clean-package.sh >> index 5cae341..9efcac5 100755 >> --- a/sc

[LEDE-DEV] [patch master 01/15 v2] Double quote to prevent globbing and word splitting

2016-10-01 Thread Jan-Tarek Butt
--- scripts/arm-magic.sh | 11 ++- scripts/clean-package.sh | 2 +- scripts/combined-ext-image.sh| 10 +- scripts/combined-image.sh| 2 +- scripts/deptest.sh | 12 ++-- scripts/ext-toolchain.sh | 15 ---

[LEDE-DEV] [PATCH v2] read without -r will mangle backslashes

2016-10-02 Thread Jan-Tarek Butt
--- scripts/clean-package.sh | 4 ++-- scripts/rstrip.sh| 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/scripts/clean-package.sh b/scripts/clean-package.sh index 2de3c51..0940732 100755 --- a/scripts/clean-package.sh +++ b/scripts/clean-package.sh @@ -10,14 +10,14

Re: [LEDE-DEV] [patch master 13/15] To read lines rather than words, pipe/redirect to a 'while read' loop

2016-10-02 Thread Jan-Tarek Butt
Hi, On 10/01/16 11:09, Oswald Buddenhagen wrote: > On Sat, Oct 01, 2016 at 12:44:36AM +0200, Lars Kruse wrote: >> Am Fri, 30 Sep 2016 22:02:27 +0200 schrieb Jan-Tarek Butt <ta...@ring0.de>: >> >>> --- >>> scripts/deptest.sh | 8 >>&g

Re: [LEDE-DEV] [patch master 14/15] Expressions don't expand in single quotes, use double quotes for that

2016-10-02 Thread Jan-Tarek Butt
Hi Lars, > > I think, this is just a change based on style preferences, or? > Using single quotes specifically for situation where expansion is not intended > feels quite explicite and clear to me - thus personally I would prefer to > stick > with single quotes instead of double quotes combined

Re: [LEDE-DEV] [patch master 05/15] Use cd ... || exit in case cd fails

2016-10-01 Thread Jan-Tarek Butt
Hi, On 10/01/16 10:54, Felix Fietkau wrote: > On 2016-10-01 09:44, Oswald Buddenhagen wrote: >> On Fri, Sep 30, 2016 at 10:02:19PM +0200, Jan-Tarek Butt wrote: >>> diff --git a/scripts/get_source_date_epoch.sh >>> b/scripts/get_source_date_epoch.sh >>> -[ -n &

[LEDE-DEV] [PATCH v2] Use cd ... || exit in case cd fails

2016-10-01 Thread Jan-Tarek Butt
--- scripts/clean-package.sh | 1 + scripts/deptest.sh | 4 ++-- scripts/ext-toolchain.sh | 12 ++-- scripts/get_source_date_epoch.sh | 4 +++- scripts/getver.sh| 4 +++- scripts/patch-specs.sh | 2 +- scripts/sysupgrade-tar.sh

[LEDE-DEV] [patch master 13/15] To read lines rather than words, pipe/redirect to a 'while read' loop

2016-09-30 Thread Jan-Tarek Butt
--- scripts/deptest.sh | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/scripts/deptest.sh b/scripts/deptest.sh index 8c859ef..0b0e7a9 100755 --- a/scripts/deptest.sh +++ b/scripts/deptest.sh @@ -86,9 +86,9 @@ test_package() # $1=pkgname [ -n "$pkg" -a -z

[LEDE-DEV] [patch master 05/15] Use cd ... || exit in case cd fails

2016-09-30 Thread Jan-Tarek Butt
--- scripts/clean-package.sh | 4 ++-- scripts/deptest.sh | 4 ++-- scripts/ext-toolchain.sh | 12 ++-- scripts/get_source_date_epoch.sh | 2 +- scripts/getver.sh| 2 +- scripts/patch-specs.sh | 2 +- scripts/sysupgrade-tar.sh

[LEDE-DEV] Automated shell code debugging

2016-09-30 Thread Jan-Tarek Butt
Hi thogeter, I have a realy nice idea :) As a part of code quallity enhancemend and prevenition. We can do an ondemand daemon as engine to check all shell scrips. We can use the opensource Project shellcheck [0]. My idea is to do an insatnce on a server: This insatnce can pull the lede repo

[LEDE-DEV] [PATCH v2] redirect matters, the 2>&1 has to be last.

2016-10-01 Thread Jan-Tarek Butt
--- scripts/make-ipkg-dir.sh | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/scripts/make-ipkg-dir.sh b/scripts/make-ipkg-dir.sh index 529e430..ba1a8df 100755 --- a/scripts/make-ipkg-dir.sh +++ b/scripts/make-ipkg-dir.sh @@ -8,10 +8,10 @@ ARCH=$4 WD=$(pwd) mkdir -p

[LEDE-DEV] [PATCH v2] Quote to prevent word splitting

2016-10-01 Thread Jan-Tarek Butt
--- scripts/combined-ext-image.sh | 2 +- scripts/combined-image.sh | 2 +- scripts/ext-toolchain.sh | 8 scripts/strip-kmod.sh | 1 + 4 files changed, 7 insertions(+), 6 deletions(-) diff --git a/scripts/combined-ext-image.sh b/scripts/combined-ext-image.sh index

Re: [LEDE-DEV] [patch master 10/15] quote literal {

2016-10-01 Thread Jan-Tarek Butt
Hi On 10/01/16 10:57, Oswald Buddenhagen wrote: > On Sat, Oct 01, 2016 at 12:30:22AM +0200, Lars Kruse wrote: >> Am Fri, 30 Sep 2016 22:02:24 +0200 schrieb Jan-Tarek Butt <ta...@ring0.de>: >> >>> --- >>> scripts/getver.sh | 4 ++-- >>> 1 file chang

[LEDE-DEV] [PATCH] busybox: prevent globbing, word splitting

2016-10-14 Thread Jan-Tarek Butt
Double quote to prevent globbing and word splitting. Signed-off-by: Jan-Tarek Butt <ta...@ring0.de> --- package/utils/busybox/files/cron | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/package/utils/busybox/files/cron b/package/utils/busybox/files/cron index 1

[LEDE-DEV] interests for meeting on 33c3 ?

2016-12-28 Thread Jan-Tarek Butt
Hi there, does anybody has interests to have a LEDE meat on 33c3 :) ? nbd, NeoRaider and I had meet together yesterday. cheers Tarek signature.asc Description: OpenPGP digital signature ___ Lede-dev mailing list Lede-dev@lists.infradead.org