[OpenWrt-Devel] [PATCH 2/2] wireguard.sh: run through shellcheck

2020-01-05 Thread Rosen Penev
Missing local variables and quoting fixes. Signed-off-by: Rosen Penev --- package/network/utils/wireguard-tools/Makefile | 2 +- .../network/utils/wireguard-tools/files/wireguard.sh | 10 +++--- 2 files changed, 8 insertions(+), 4 deletions(-) diff --git a/package/network/utils/w

[OpenWrt-Devel] [PATCH 1/2] wireguard.sh: batch file writes

2020-01-05 Thread Rosen Penev
Speeds up config generation. Signed-off-by: Rosen Penev --- v3: Fix title v2: Added PKG_RELEASE bump. .../utils/wireguard-tools/files/wireguard.sh | 64 +-- 1 file changed, 30 insertions(+), 34 deletions(-) diff --git a/package/network/utils/wireguard-tools/files/wireguard.sh

[OpenWrt-Devel] [PATCH 4/4] scripts/env: use read -r instead of read

2020-01-05 Thread Rosen Penev
read mangles backslashes. Found with shellcheck. Signed-off-by: Rosen Penev --- scripts/env | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/env b/scripts/env index 848850a778..c81fbf8ddc 100755 --- a/scripts/env +++ b/scripts/env @@ -42,7 +42,7 @@ ask_bool() {

[OpenWrt-Devel] [PATCH 1/4] scripts/env: use command -v instead of which

2020-01-05 Thread Rosen Penev
Simpler and built in to the shell. Found with shellcheck. Signed-off-by: Rosen Penev --- scripts/env | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/env b/scripts/env index 7bfca58c5d..563f39c1cf 100755 --- a/scripts/env +++ b/scripts/env @@ -57,7 +57,7 @@ env_init()

[OpenWrt-Devel] [PATCH 3/4] scripts/env: exit in case of failure to cd

2020-01-05 Thread Rosen Penev
Found with shellcheck. Signed-off-by: Rosen Penev --- scripts/env | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/env b/scripts/env index 19cfa13841..848850a778 100755 --- a/scripts/env +++ b/scripts/env @@ -147,7 +147,7 @@ env_clear() { else r

[OpenWrt-Devel] [PATCH 2/4] scripts/env: replace \! with !

2020-01-05 Thread Rosen Penev
The latter is more standard. The former throws an error under shellcheck. Signed-off-by: Rosen Penev --- scripts/env | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/scripts/env b/scripts/env index 563f39c1cf..19cfa13841 100755 --- a/scripts/env +++ b/scripts/env @@ -73,7

[OpenWrt-Devel] [PATCH 2/4] scripts/mkits.sh: add missing quotes

2020-01-05 Thread Rosen Penev
Found with shellcheck. Signed-off-by: Rosen Penev --- scripts/mkits.sh | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/scripts/mkits.sh b/scripts/mkits.sh index def3d3c7c2..53b9ec5f20 100755 --- a/scripts/mkits.sh +++ b/scripts/mkits.sh @@ -55,7 +55,7 @@ if [ -z "${ARCH}"

[OpenWrt-Devel] [PATCH 3/4] scripts/mkits.sh: fix improper string and array concatenation

2020-01-05 Thread Rosen Penev
Found with shellcheck. Signed-off-by: Rosen Penev --- scripts/mkits.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/mkits.sh b/scripts/mkits.sh index 53b9ec5f20..c09e537dbd 100755 --- a/scripts/mkits.sh +++ b/scripts/mkits.sh @@ -43,7 +43,7 @@ do k

[OpenWrt-Devel] [PATCH 4/4] scripts/mkits.sh: switch from bash to sh

2020-01-05 Thread Rosen Penev
This no longer needs bash. Signed-off-by: Rosen Penev --- scripts/mkits.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/mkits.sh b/scripts/mkits.sh index c09e537dbd..62cfc21c6b 100755 --- a/scripts/mkits.sh +++ b/scripts/mkits.sh @@ -1,4 +1,4 @@ -#!/usr/bin/env bas

[OpenWrt-Devel] [PATCH 1/4] scripts/mkits.sh: replace echo -e with printf

2020-01-05 Thread Rosen Penev
echo flags are not POSIX. printf does the same with added \n. Signed-off-by: Rosen Penev --- scripts/mkits.sh | 24 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/scripts/mkits.sh b/scripts/mkits.sh index 32a978e18d..def3d3c7c2 100755 --- a/scripts/mkits

[OpenWrt-Devel] [PATCH 1/3] scripts/flashing/flash.sh: Add missing quotes

2020-01-05 Thread Rosen Penev
These are in if statements (where they should be), and echo output that is not piped to anything. These should be safe. Found with shellcheck. Signed-off-by: Rosen Penev --- scripts/flashing/flash.sh | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/scripts/flashing

[OpenWrt-Devel] [PATCH 3/3] scripts/flashing/flash.sh: Switch to /bin/sh

2020-01-05 Thread Rosen Penev
The script no longer uses any features that necessitate bash. Also removed trailing whitespace. Signed-off-by: Rosen Penev --- scripts/flashing/flash.sh | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/scripts/flashing/flash.sh b/scripts/flashing/flash.sh index aea

[OpenWrt-Devel] [PATCH 2/3] scripts/flashing/flash.sh: switch from echo -en to printf

2020-01-05 Thread Rosen Penev
echo flags are an extension and not available with sh. printf does the same as -en. Signed-off-by: Rosen Penev --- scripts/flashing/flash.sh | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/scripts/flashing/flash.sh b/scripts/flashing/flash.sh index 7f2cdf5db9..aea1bf1

[OpenWrt-Devel] [PATCH 2/2] scripts/gen_image_generic.sh: use /bin/sh

2020-01-05 Thread Rosen Penev
This has nothing bash specific. Signed-off-by: Rosen Penev --- scripts/gen_image_generic.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/gen_image_generic.sh b/scripts/gen_image_generic.sh index 527ca5bacc..5162be11dd 100755 --- a/scripts/gen_image_generic.sh +++ b

openwrt-devel@lists.openwrt.org

2020-01-05 Thread Rosen Penev
-o is not well defined. Found with shellcheck. Signed-off-by: Rosen Penev --- scripts/gen_image_generic.sh | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/scripts/gen_image_generic.sh b/scripts/gen_image_generic.sh index 2c57d56f07..527ca5bacc 100755 --- a/scripts/gen_im

Re: [OpenWrt-Devel] [PATCH] ath79: add support for TP-Link TL-MR6400

2020-01-05 Thread mail
Hi Enrico, > -Original Message- > From: Enrico Mioso [mailto:mrkiko...@gmail.com] > Sent: Dienstag, 17. September 2019 19:59 > To: m...@adrianschmutzler.de > Cc: Filip Moc ; openwrt-devel@lists.openwrt.org > Subject: RE: [OpenWrt-Devel] [PATCH] ath79: add support for TP-Link TL- > MR6400 >

[OpenWrt-Devel] [PATCH] hostapd: disable ft_psk_generate_local for non-PSK networks

2020-01-05 Thread David Bauer
Without this commit, ft_psk_generate_local is enabled for non-PSK networks by default. This breaks 802.11r for EAP networks. Disable ft_psk_generate_local by default for non-PSK networks resolves this misbehavior. Reported-by: Martin Weinelt Signed-off-by: David Bauer Tested-by: Martin Weinelt

[OpenWrt-Devel] [sdwalker/sdwalker.github.io] 9a4ea2: This week's update

2020-01-05 Thread Stephen Walker
Branch: refs/heads/master Home: https://github.com/sdwalker/sdwalker.github.io Commit: 9a4ea277be13410f72ab6a36931401cbb886960f https://github.com/sdwalker/sdwalker.github.io/commit/9a4ea277be13410f72ab6a36931401cbb886960f Author: Stephen Walker Date: 2020-01-05 (Sun, 05 Jan 2

[OpenWrt-Devel] Google Summer of Code - ideas wanted

2020-01-05 Thread Andreas Bräu
Hi there, I wish you all the best for 2020! We're planning to apply for Google Summer of Code (GSoC) again. GSoC is a program by Google to bring student developers into Open Source projects. tl;dr: Google pays students for 3 months in summer developing software for an open source project. Therefo

Re: [OpenWrt-Devel] [PATCH 0/5] Remove kernel 4.9 targets and kernel 4.9 support

2020-01-05 Thread mail
> -Original Message- > From: openwrt-devel [mailto:openwrt-devel-boun...@lists.openwrt.org] > On Behalf Of Adrian Schmutzler > Sent: Sonntag, 5. Januar 2020 17:38 > To: openwrt-devel@lists.openwrt.org > Subject: [OpenWrt-Devel] [PATCH 0/5] Remove kernel 4.9 targets and kernel > 4.9 support

Re: [OpenWrt-Devel] [PATCH] ramips: Fix sysupgrade for Xiaomi mir3g

2020-01-05 Thread mail
Hi, > When I use sysupgrade in 18.06 it executes the platform_check_image() > functions which calls nand_do_platform_check("mir3g", > "/tmp/openwrt-19.07-ramips-mt7621-xiaomi_mir3g-squashfs-sysupgrade.bin") > https://git.openwrt.org/?p=openwrt/openwrt.git;a=blob;f=target/linux/ramips/base-files

Re: [OpenWrt-Devel] [PATCH] ramips: Fix sysupgrade for Xiaomi mir3g

2020-01-05 Thread Hauke Mehrtens
On 1/5/20 5:37 PM, m...@adrianschmutzler.de wrote: > Hi Hauke, > >> -Original Message- >> From: Hauke Mehrtens [mailto:ha...@hauke-m.de] >> Sent: Sonntag, 5. Januar 2020 15:18 >> To: openwrt-devel@lists.openwrt.org >> Cc: m...@adrianschmutzler.de; d...@kresin.me; Hauke Mehrtens >> >> Subj

[OpenWrt-Devel] [PATCH 5/5] kernel: remove obsolete kernel version switches

2020-01-05 Thread Adrian Schmutzler
After kernel 4.9 has been removed, this removes all (now obsolete) kernel version switches that deal with versions before 4.14. Package kmod-crypto-iv is empty now and thus removed entirely. Signed-off-by: Adrian Schmutzler --- include/netfilter.mk | 26 +++

[OpenWrt-Devel] [PATCH 0/5] Remove kernel 4.9 targets and kernel 4.9 support

2020-01-05 Thread Adrian Schmutzler
This removes support for the remaining three kernel 4.9 targets, and after that for the now "unused" kernel 4.9 support. The targets seem to be not maintained anymore, for two of them there were even patches available to bump them to 4.14, but nobody even attempted to test them. Support will stay

[OpenWrt-Devel] [PATCH 3/5] orion: remove unmaintained target

2020-01-05 Thread Adrian Schmutzler
This target is still on kernel 4.9, and it looks like there is no active maintainer for this target anymore. Remove the code and all the packages which are only used by this target. To add this target to OpenWrt again port it to a recent and supported kernel version. Signed-off-by: Adrian Schmutz

Re: [OpenWrt-Devel] [PATCH] ramips: Fix sysupgrade for Xiaomi mir3g

2020-01-05 Thread mail
Hi Hauke, > -Original Message- > From: Hauke Mehrtens [mailto:ha...@hauke-m.de] > Sent: Sonntag, 5. Januar 2020 15:18 > To: openwrt-devel@lists.openwrt.org > Cc: m...@adrianschmutzler.de; d...@kresin.me; Hauke Mehrtens > > Subject: [PATCH] ramips: Fix sysupgrade for Xiaomi mir3g > > With

Re: [OpenWrt-Devel] Kernel version for OpenWrt 20.X

2020-01-05 Thread Piotr Dymacz
Hi Adrian, Tom, On 05.01.2020 16:57, m...@adrianschmutzler.de wrote: -Original Message- From: Tom Psyborg [mailto:pozega.tomis...@gmail.com] Sent: Sonntag, 5. Januar 2020 16:33 To: m...@adrianschmutzler.de Cc: Hauke Mehrtens ; OpenWrt Development List ; Koen Vandeputte Subject: Re: [Ope

Re: [OpenWrt-Devel] Kernel version for OpenWrt 20.X

2020-01-05 Thread mail
> -Original Message- > From: Tom Psyborg [mailto:pozega.tomis...@gmail.com] > Sent: Sonntag, 5. Januar 2020 16:33 > To: m...@adrianschmutzler.de > Cc: Hauke Mehrtens ; OpenWrt Development List > ; Koen Vandeputte > > Subject: Re: [OpenWrt-Devel] Kernel version for OpenWrt 20.X > > On 05/0

Re: [OpenWrt-Devel] Kernel version for OpenWrt 20.X

2020-01-05 Thread Tom Psyborg
On 05/01/2020, m...@adrianschmutzler.de wrote: > Hi, > >> -Original Message- >> From: openwrt-devel [mailto:openwrt-devel-boun...@lists.openwrt.org] >> On Behalf Of Hauke Mehrtens >> Sent: Samstag, 4. Januar 2020 18:26 >> To: m...@adrianschmutzler.de; 'OpenWrt Development List' > de...@lis

Re: [OpenWrt-Devel] [PATCH] ramips: Fix sysupgrade for Xiaomi mir3g

2020-01-05 Thread Hauke Mehrtens
On 1/5/20 3:17 PM, Hauke Mehrtens wrote: > Without this change sysupgrade from 18.06 to 19.07 is only possible with > the -F option. > In OpenWrt 18.06 the nand_do_platform_check() function is called with > the board name mir3g only, if the tar does not use mir3g it will fail. > OpenWrt 19.07 and l

[OpenWrt-Devel] [PATCH] ramips: Fix sysupgrade for Xiaomi mir3g

2020-01-05 Thread Hauke Mehrtens
Without this change sysupgrade from 18.06 to 19.07 is only possible with the -F option. In OpenWrt 18.06 the nand_do_platform_check() function is called with the board name mir3g only, if the tar does not use mir3g it will fail. OpenWrt 19.07 and later support the metadata with the supported_device

Re: [OpenWrt-Devel] Kernel version for OpenWrt 20.X

2020-01-05 Thread Hauke Mehrtens
On 1/5/20 2:43 PM, m...@adrianschmutzler.de wrote: > On 11/28/19 7:11 PM, Adrian Schmutzler wrote: >> Hi Hauke, >> >>> The following are still on kernel 4.9: >>> * ar7 >>> * ixp4xx >>> * orion >> >> There are patches (actually from you, May 2019) on the list w

Re: [OpenWrt-Devel] Kernel version for OpenWrt 20.X

2020-01-05 Thread mail
> > >> On 11/28/19 7:11 PM, Adrian Schmutzler wrote: > > >>> Hi Hauke, > > >>> > > The following are still on kernel 4.9: > > * ar7 > > * ixp4xx > > * orion > > >>> > > >>> There are patches (actually from you, May 2019) on the list which > > >>> claim to bump ar7 and orion t

Re: [OpenWrt-Devel] Kernel version for OpenWrt 20.X

2020-01-05 Thread mail
Hi, > -Original Message- > From: openwrt-devel [mailto:openwrt-devel-boun...@lists.openwrt.org] > On Behalf Of Hauke Mehrtens > Sent: Samstag, 4. Januar 2020 18:26 > To: m...@adrianschmutzler.de; 'OpenWrt Development List' de...@lists.openwrt.org> > Cc: 'Koen Vandeputte' > Subject: Re: [

Re: [OpenWrt-Devel] [PATCH procd 3/4] system: sysupgrade: rework firmware validation

2020-01-05 Thread Petr Štetiar
Hauke Mehrtens [2020-01-04 20:41:44]: > > + vjson_error(err, "unhandled error"); > > In which case is this returned, aren't there specific error handlers in > call cases now and vjson_parse() would overwrite it again? It should be returned in unhandled cases, in case I've missed to handle some

Re: [OpenWrt-Devel] [PATCH procd 3/4] system: sysupgrade: rework firmware validation

2020-01-05 Thread Hauke Mehrtens
On 1/5/20 11:40 AM, Petr Štetiar wrote: > Hauke Mehrtens [2020-01-04 20:41:44]: > > Hi, > > thanks for the review! > >> Please annotate the function with: >> __attribute__ ((format (printf, 2, 3))); > > Done. > >>> + va_start(va, fmt); >>> + r = vsnprintf(buf+r, sizeof(buf)-r, fmt, va); >

Re: [OpenWrt-Devel] [PATCH procd 3/4] system: sysupgrade: rework firmware validation

2020-01-05 Thread Petr Štetiar
Hauke Mehrtens [2020-01-04 20:41:44]: Hi, thanks for the review! > Please annotate the function with: > __attribute__ ((format (printf, 2, 3))); Done. > > + va_start(va, fmt); > > + r = vsnprintf(buf+r, sizeof(buf)-r, fmt, va); > > Please check here for truncation: > > rv = vsnprintf(buf+

[OpenWrt-Devel] [PATCH v2 2/2] ath79: Remove mtd cfi_cmdset_0002 status check patches

2020-01-05 Thread Tokunori Ikegami
Currently the patch only changes break to use goto statement instead. But not necessary acutually since the ret value checked after the for loop. So it is okay for the break case before changed by the patch also. This patch only reverts the following commit partially. https://github.com/openwrt/

[OpenWrt-Devel] [PATCH v2 0/2] Remove mtd cfi_cmdset_0002 status check patches

2020-01-05 Thread Tokunori Ikegami
The following commit changes are mainly applied into the linux kernel upstream. https://github.com/openwrt/openwrt/commit/ddc11c3932c7b7b7df7d5fbd48f207e77619eaa7 So it is not necessary to remain current patches and remove them. Tokunori Ikegami (2): ar71xx: Remove mtd cfi_cmdset_0002 status

[OpenWrt-Devel] [PATCH v2 1/2] ar71xx: Remove mtd cfi_cmdset_0002 status check patch

2020-01-05 Thread Tokunori Ikegami
Currently the patch only changes break to use goto statement instead. But not necessary acutually since the ret value checked after the for loop. So it is okay for the break case before changed by the patch also. This patch only reverts the following commit partially. https://github.com/openwrt