[OpenWrt-Devel] [PATCH] toolchain/gdb: Don't use gdb-arc

2019-11-22 Thread Rosen Penev
GDB got support for ARC with version 8.2. No need for this fork. Signed-off-by: Rosen Penev --- toolchain/gdb/Makefile | 11 --- 1 file changed, 11 deletions(-) diff --git a/toolchain/gdb/Makefile b/toolchain/gdb/Makefile index 41ba9853fd..c25d181990 100644 --- a/toolchain/gdb/Makefile

[OpenWrt-Devel] [PATCH 02/17] gen_mvebu_sdcard_img.sh: Don't use format directly in printf

2019-11-25 Thread Rosen Penev
Enables proper checking. Matches printf behavior in C. Found with shellcheck. Signed-off-by: Rosen Penev --- target/linux/mvebu/image/gen_mvebu_sdcard_img.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target/linux/mvebu/image/gen_mvebu_sdcard_img.sh b/target/linux

[OpenWrt-Devel] [PATCH 01/17] gen_mvebu_sdcard_img.sh: Replace let with $(())

2019-11-25 Thread Rosen Penev
let is a bashism. Found with shellcheck. Signed-off-by: Rosen Penev --- target/linux/mvebu/image/gen_mvebu_sdcard_img.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target/linux/mvebu/image/gen_mvebu_sdcard_img.sh b/target/linux/mvebu/image/gen_mvebu_sdcard_img.sh

[OpenWrt-Devel] [PATCH 03/17] gen_mvebu_sdcard_img.sh: Use /bin/sh

2019-11-25 Thread Rosen Penev
There is nothing that needs bash anymore. Signed-off-by: Rosen Penev --- target/linux/mvebu/image/gen_mvebu_sdcard_img.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target/linux/mvebu/image/gen_mvebu_sdcard_img.sh b/target/linux/mvebu/image/gen_mvebu_sdcard_img.sh

[OpenWrt-Devel] [PATCH 06/17] rb532/gen_image.sh: Use /bin/sh

2019-11-25 Thread Rosen Penev
Nothing here needs bash. Signed-off-by: Rosen Penev --- target/linux/rb532/image/gen_image.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target/linux/rb532/image/gen_image.sh b/target/linux/rb532/image/gen_image.sh index a9d48d86ca..e46539fd85 100755 --- a/target/linux

[OpenWrt-Devel] [PATCH 05/17] gen_sunxi_sdcard_img.sh: Switch to /bin/sh

2019-11-25 Thread Rosen Penev
This has nothing that needs bash. Signed-off-by: Rosen Penev --- target/linux/sunxi/image/gen_sunxi_sdcard_img.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target/linux/sunxi/image/gen_sunxi_sdcard_img.sh b/target/linux/sunxi/image/gen_sunxi_sdcard_img.sh index

[OpenWrt-Devel] [PATCH 07/17] gen_omap_sdcard_img.sh: Use /bin/sh

2019-11-25 Thread Rosen Penev
Nothing here needs bash. Signed-off-by: Rosen Penev --- target/linux/omap/image/gen_omap_sdcard_img.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target/linux/omap/image/gen_omap_sdcard_img.sh b/target/linux/omap/image/gen_omap_sdcard_img.sh index 881df9a05d

[OpenWrt-Devel] [PATCH 04/17] gen_zynq_sdcard_img.sh: Switch to /bin/sh

2019-11-25 Thread Rosen Penev
This has nothing that needs bash. Signed-off-by: Rosen Penev --- target/linux/zynq/image/gen_zynq_sdcard_img.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target/linux/zynq/image/gen_zynq_sdcard_img.sh b/target/linux/zynq/image/gen_zynq_sdcard_img.sh index 460a336e69

[OpenWrt-Devel] [PATCH 08/17] mxs/image/gen_sdcard_vfat_ext4.sh: Use /bin/sh

2019-11-25 Thread Rosen Penev
Nothing here needs bash. Signed-off-by: Rosen Penev --- target/linux/mxs/image/gen_sdcard_vfat_ext4.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target/linux/mxs/image/gen_sdcard_vfat_ext4.sh b/target/linux/mxs/image/gen_sdcard_vfat_ext4.sh index bca664c780

[OpenWrt-Devel] [PATCH 12/17] dns313_gen_hdd_img.sh: Switch to /bin/sh

2019-11-25 Thread Rosen Penev
Nothing here needs bash. Signed-off-by: Rosen Penev --- target/linux/gemini/image/dns313_gen_hdd_img.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target/linux/gemini/image/dns313_gen_hdd_img.sh b/target/linux/gemini/image/dns313_gen_hdd_img.sh index 6e899ec840

[OpenWrt-Devel] [PATCH 09/17] mxs/image/gen_sdcard_ext4_ext4.sh: Use /bin/sh

2019-11-25 Thread Rosen Penev
Nothing here needs bash. Signed-off-by: Rosen Penev --- target/linux/mxs/image/gen_sdcard_ext4_ext4.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target/linux/mxs/image/gen_sdcard_ext4_ext4.sh b/target/linux/mxs/image/gen_sdcard_ext4_ext4.sh index 99bd3bf573

[OpenWrt-Devel] [PATCH 13/17] gen_rpi_sdcard_img.sh: Use /bin/sh

2019-11-25 Thread Rosen Penev
Nothing here needs bash. Signed-off-by: Rosen Penev --- target/linux/brcm2708/image/gen_rpi_sdcard_img.sh | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/target/linux/brcm2708/image/gen_rpi_sdcard_img.sh b/target/linux/brcm2708/image/gen_rpi_sdcard_img.sh index

[OpenWrt-Devel] [PATCH 10/17] gen_mt7623_emmc_img.sh: Use /bin/sh

2019-11-25 Thread Rosen Penev
Nothing here needs bash. Signed-off-by: Rosen Penev --- target/linux/mediatek/image/gen_mt7623_emmc_img.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target/linux/mediatek/image/gen_mt7623_emmc_img.sh b/target/linux/mediatek/image/gen_mt7623_emmc_img.sh index

[OpenWrt-Devel] [PATCH 16/17] gen_axs10x_sdcard_img.sh: Switch to /bin/sh

2019-11-25 Thread Rosen Penev
Nothing here needs bash. Signed-off-by: Rosen Penev --- target/linux/arc770/image/gen_axs10x_sdcard_img.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target/linux/arc770/image/gen_axs10x_sdcard_img.sh b/target/linux/arc770/image/gen_axs10x_sdcard_img.sh index

[OpenWrt-Devel] [PATCH 17/17] mbl_gen_hdd_img.sh: Switch to /bin/sh

2019-11-25 Thread Rosen Penev
Nothing here needs bash. Signed-off-by: Rosen Penev --- target/linux/apm821xx/image/mbl_gen_hdd_img.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target/linux/apm821xx/image/mbl_gen_hdd_img.sh b/target/linux/apm821xx/image/mbl_gen_hdd_img.sh index 1fb868b5b3

[OpenWrt-Devel] [PATCH 11/17] gen_sdcard_head_img.sh: Switch to /bin/sh

2019-11-25 Thread Rosen Penev
Nothing here needs bash. Signed-off-by: Rosen Penev --- target/linux/layerscape/image/gen_sdcard_head_img.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target/linux/layerscape/image/gen_sdcard_head_img.sh b/target/linux/layerscape/image/gen_sdcard_head_img.sh index

[OpenWrt-Devel] [PATCH 14/17] gen_at91_sdcard_img.sh: Switch to /bin/sh

2019-11-25 Thread Rosen Penev
Nothing here needs bash. Signed-off-by: Rosen Penev --- target/linux/at91/image/gen_at91_sdcard_img.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target/linux/at91/image/gen_at91_sdcard_img.sh b/target/linux/at91/image/gen_at91_sdcard_img.sh index 4eb30fa790

[OpenWrt-Devel] [PATCH 15/17] gen_axs10x_sdcard_img.sh: Switch to /bin/sh

2019-11-25 Thread Rosen Penev
Nothing here needs bash. Signed-off-by: Rosen Penev --- target/linux/archs38/image/gen_axs10x_sdcard_img.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target/linux/archs38/image/gen_axs10x_sdcard_img.sh b/target/linux/archs38/image/gen_axs10x_sdcard_img.sh index

Re: [OpenWrt-Devel] [PATCH 03/17] gen_mvebu_sdcard_img.sh: Use /bin/sh

2019-11-25 Thread Rosen Penev
On Mon, Nov 25, 2019 at 11:16 AM Bjørn Mork wrote: > > Rosen Penev writes: > > > +#!/bash/sh > > Eh, that's not /bin/sh Whoops. Will resend. > > > Bjørn ___ openwrt-devel mailing list openwrt-devel@lists.openwrt.or

[OpenWrt-Devel] [PATCHv3] tools/pkg-config: Replace with pkgconf

2019-11-25 Thread Rosen Penev
apper with meson as passing --version with other options fails. Signed-off-by: Rosen Penev --- v3: Added meson patch v2: Rebased against master tools/Makefile| 6 ++-- tools/pkg-config/files/pkg-config | 3 -- tools/{pkg-config => pkgconf}/Makefil

[OpenWrt-Devel] [PATCH 03/17] gen_mvebu_sdcard_img.sh: Use /bin/sh

2019-11-25 Thread Rosen Penev
There is nothing that needs bash anymore. Signed-off-by: Rosen Penev --- target/linux/mvebu/image/gen_mvebu_sdcard_img.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target/linux/mvebu/image/gen_mvebu_sdcard_img.sh b/target/linux/mvebu/image/gen_mvebu_sdcard_img.sh

[OpenWrt-Devel] [PATCH 02/17] gen_mvebu_sdcard_img.sh: Don't use format directly in printf

2019-11-25 Thread Rosen Penev
Enables proper checking. Matches printf behavior in C. Found with shellcheck. Signed-off-by: Rosen Penev --- target/linux/mvebu/image/gen_mvebu_sdcard_img.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target/linux/mvebu/image/gen_mvebu_sdcard_img.sh b/target/linux

[OpenWrt-Devel] [PATCH 01/17] gen_mvebu_sdcard_img.sh: Replace let with $(())

2019-11-25 Thread Rosen Penev
let is a bashism. Found with shellcheck. Signed-off-by: Rosen Penev --- target/linux/mvebu/image/gen_mvebu_sdcard_img.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target/linux/mvebu/image/gen_mvebu_sdcard_img.sh b/target/linux/mvebu/image/gen_mvebu_sdcard_img.sh

[OpenWrt-Devel] [PATCH 06/17] rb532/gen_image.sh: Use /bin/sh

2019-11-25 Thread Rosen Penev
Nothing here needs bash. Signed-off-by: Rosen Penev --- target/linux/rb532/image/gen_image.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target/linux/rb532/image/gen_image.sh b/target/linux/rb532/image/gen_image.sh index a9d48d86ca..e46539fd85 100755 --- a/target/linux

[OpenWrt-Devel] [PATCH 04/17] gen_zynq_sdcard_img.sh: Switch to /bin/sh

2019-11-25 Thread Rosen Penev
This has nothing that needs bash. Signed-off-by: Rosen Penev --- target/linux/zynq/image/gen_zynq_sdcard_img.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target/linux/zynq/image/gen_zynq_sdcard_img.sh b/target/linux/zynq/image/gen_zynq_sdcard_img.sh index 460a336e69

[OpenWrt-Devel] [PATCH 05/17] gen_sunxi_sdcard_img.sh: Switch to /bin/sh

2019-11-25 Thread Rosen Penev
This has nothing that needs bash. Signed-off-by: Rosen Penev --- target/linux/sunxi/image/gen_sunxi_sdcard_img.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target/linux/sunxi/image/gen_sunxi_sdcard_img.sh b/target/linux/sunxi/image/gen_sunxi_sdcard_img.sh index

[OpenWrt-Devel] [PATCH 07/17] gen_omap_sdcard_img.sh: Use /bin/sh

2019-11-25 Thread Rosen Penev
Nothing here needs bash. Signed-off-by: Rosen Penev --- target/linux/omap/image/gen_omap_sdcard_img.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target/linux/omap/image/gen_omap_sdcard_img.sh b/target/linux/omap/image/gen_omap_sdcard_img.sh index 881df9a05d

[OpenWrt-Devel] [PATCH 08/17] mxs/image/gen_sdcard_vfat_ext4.sh: Use /bin/sh

2019-11-25 Thread Rosen Penev
Nothing here needs bash. Signed-off-by: Rosen Penev --- target/linux/mxs/image/gen_sdcard_vfat_ext4.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target/linux/mxs/image/gen_sdcard_vfat_ext4.sh b/target/linux/mxs/image/gen_sdcard_vfat_ext4.sh index bca664c780

[OpenWrt-Devel] [PATCH 09/17] mxs/image/gen_sdcard_ext4_ext4.sh: Use /bin/sh

2019-11-25 Thread Rosen Penev
Nothing here needs bash. Signed-off-by: Rosen Penev --- target/linux/mxs/image/gen_sdcard_ext4_ext4.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target/linux/mxs/image/gen_sdcard_ext4_ext4.sh b/target/linux/mxs/image/gen_sdcard_ext4_ext4.sh index 99bd3bf573

[OpenWrt-Devel] [PATCH 12/17] dns313_gen_hdd_img.sh: Switch to /bin/sh

2019-11-25 Thread Rosen Penev
Nothing here needs bash. Signed-off-by: Rosen Penev --- target/linux/gemini/image/dns313_gen_hdd_img.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target/linux/gemini/image/dns313_gen_hdd_img.sh b/target/linux/gemini/image/dns313_gen_hdd_img.sh index 6e899ec840

[OpenWrt-Devel] [PATCH 10/17] gen_mt7623_emmc_img.sh: Use /bin/sh

2019-11-25 Thread Rosen Penev
Nothing here needs bash. Signed-off-by: Rosen Penev --- target/linux/mediatek/image/gen_mt7623_emmc_img.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target/linux/mediatek/image/gen_mt7623_emmc_img.sh b/target/linux/mediatek/image/gen_mt7623_emmc_img.sh index

[OpenWrt-Devel] [PATCH 11/17] gen_sdcard_head_img.sh: Switch to /bin/sh

2019-11-25 Thread Rosen Penev
Nothing here needs bash. Signed-off-by: Rosen Penev --- target/linux/layerscape/image/gen_sdcard_head_img.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target/linux/layerscape/image/gen_sdcard_head_img.sh b/target/linux/layerscape/image/gen_sdcard_head_img.sh index

[OpenWrt-Devel] [PATCH 13/17] gen_rpi_sdcard_img.sh: Use /bin/sh

2019-11-25 Thread Rosen Penev
Nothing here needs bash. Signed-off-by: Rosen Penev --- target/linux/brcm2708/image/gen_rpi_sdcard_img.sh | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/target/linux/brcm2708/image/gen_rpi_sdcard_img.sh b/target/linux/brcm2708/image/gen_rpi_sdcard_img.sh index

[OpenWrt-Devel] [PATCH 14/17] gen_at91_sdcard_img.sh: Switch to /bin/sh

2019-11-25 Thread Rosen Penev
Nothing here needs bash. Signed-off-by: Rosen Penev --- target/linux/at91/image/gen_at91_sdcard_img.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target/linux/at91/image/gen_at91_sdcard_img.sh b/target/linux/at91/image/gen_at91_sdcard_img.sh index 4eb30fa790

[OpenWrt-Devel] [PATCH 17/17] mbl_gen_hdd_img.sh: Switch to /bin/sh

2019-11-25 Thread Rosen Penev
Nothing here needs bash. Signed-off-by: Rosen Penev --- target/linux/apm821xx/image/mbl_gen_hdd_img.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target/linux/apm821xx/image/mbl_gen_hdd_img.sh b/target/linux/apm821xx/image/mbl_gen_hdd_img.sh index 1fb868b5b3

[OpenWrt-Devel] [PATCH 15/17] gen_axs10x_sdcard_img.sh: Switch to /bin/sh

2019-11-25 Thread Rosen Penev
Nothing here needs bash. Signed-off-by: Rosen Penev --- target/linux/archs38/image/gen_axs10x_sdcard_img.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target/linux/archs38/image/gen_axs10x_sdcard_img.sh b/target/linux/archs38/image/gen_axs10x_sdcard_img.sh index

[OpenWrt-Devel] [PATCH 16/17] gen_axs10x_sdcard_img.sh: Switch to /bin/sh

2019-11-25 Thread Rosen Penev
Nothing here needs bash. Signed-off-by: Rosen Penev --- target/linux/arc770/image/gen_axs10x_sdcard_img.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target/linux/arc770/image/gen_axs10x_sdcard_img.sh b/target/linux/arc770/image/gen_axs10x_sdcard_img.sh index

Re: [OpenWrt-Devel] [PATCHv3] tools/pkg-config: Replace with pkgconf

2019-11-27 Thread Rosen Penev
On Wed, Nov 27, 2019 at 2:49 AM Petr Štetiar wrote: > > Rosen Penev [2019-11-25 14:44:18]: > > Hi, > > you're trying to do a lot of things in one patch, making it hard to review, > please split it up. > > 1. switch to pkgconf > 2. shell check fix > 3. me

[OpenWrt-Devel] [PATCH 3/3] tools/pkgconf: Add patch to fix meson compilation.

2019-11-27 Thread Rosen Penev
The wrapper calls pkgconf with multiple arguments. meson wants to call it with a single --version argument. To keep compatibility with pkg-config, revert the upstream version change. This fixes projects that use meson and pkg-config to find packages. Signed-off-by: Rosen Penev --- ...me

[OpenWrt-Devel] [PATCH 1/3] tools/pkg-config: Replace with pkgconf

2019-11-27 Thread Rosen Penev
pkg-config. For simplicity here, it is renamed to pkg-config.real, as in the original package. Initial results have been positive. As before, pkgconf works as long as the pkg-config files point to the proper paths. Signed-off-by: Rosen Penev --- tools/Makefile

[OpenWrt-Devel] [PATCH 2/3] tools/pkgconf: Run pkg-config wrapper through shellcheck

2019-11-27 Thread Rosen Penev
Mainly quoting fixes. Separated parameters by \ for easier readability. Switched to calling by dirname so that it can be called from any directory. Signed-off-by: Rosen Penev --- tools/pkgconf/files/pkg-config | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/tools

Re: [OpenWrt-Devel] [PATCHv3] tools/pkg-config: Replace with pkgconf

2019-11-27 Thread Rosen Penev
On Wed, Nov 27, 2019 at 6:35 AM Andre Heider wrote: > > On 27/11/2019 13:19, Petr Štetiar wrote: > > Andre Heider [2019-11-27 12:26:39]: > > > >> I didn't pursue upstreaming it because it looks like merge request at > >> github > >> are ignored and I didn't feel like signing up at upstreams own

Re: [OpenWrt-Devel] [PATCH 2/3] tools/pkgconf: Run pkg-config wrapper through shellcheck

2019-11-27 Thread Rosen Penev
On Wed, Nov 27, 2019 at 2:30 PM Rosen Penev wrote: > > Mainly quoting fixes. > > Separated parameters by \ for easier readability. > > Switched to calling by dirname so that it can be called from any > directory. > > Signed-off-by: Rosen Penev > --- > t

[OpenWrt-Devel] [PATCH] ucert: Cast ucert_argv to proper type when passing to execv

2019-11-28 Thread Rosen Penev
Fixes warnings: warning: passing argument 2 of 'execv' from incompatible pointer type [-Wincompatible-pointer-types] 254 | execv(usign_argv[0], usign_argv) Signed-off-by: Rosen Penev --- usign-exec.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --g

[OpenWrt-Devel] [PATCHv2 2/3] tools/pkgconf: Run pkg-config wrapper through shellcheck

2019-11-28 Thread Rosen Penev
Mainly quoting fixes. Separated parameters by \ for easier readability. Signed-off-by: Rosen Penev --- v2: Removed calling by dirname. tools/pkgconf/files/pkg-config | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/tools/pkgconf/files/pkg-config b/tools/pkgconf/files

[OpenWrt-Devel] [PATCHv2 1/3] tools/pkg-config: Replace with pkgconf

2019-11-28 Thread Rosen Penev
pkg-config. For simplicity here, it is renamed to pkg-config.real, as in the original package. Initial results have been positive. As before, pkgconf works as long as the pkg-config files point to the proper paths. Signed-off-by: Rosen Penev --- tools/Makefile

[OpenWrt-Devel] [PATCHv2 3/3] tools/pkgconf: Add patch to fix meson compilation.

2019-11-28 Thread Rosen Penev
The wrapper calls pkgconf with multiple arguments. meson wants to call it with a single --version argument. To keep compatibility with pkg-config, revert the upstream version change. This fixes projects that use meson and pkg-config to find packages. Signed-off-by: Rosen Penev --- ...me

[OpenWrt-Devel] [PATCH] firewall: Fix Wformat-nonliteral warning

2019-11-29 Thread Rosen Penev
Allows GCC to check the formats by switching to a define, which is a constant expression. Fixes: warning: format not a string literal, argument types not checked [-Wformat-nonliteral] 207 | snprintf(buf, sizeof(buf), tmpl, include->path); Signed-off-by: Rosen Penev --- includes.c |

[OpenWrt-Devel] [PATCH] uci: Fix Wformat-nonliteral warning

2019-11-29 Thread Rosen Penev
] 176 |error_info); |^~ error: format not a string literal, argument types not checked [-Werror=format-nonliteral] 185 |error_info); Signed-off-by: Rosen Penev --- libuci.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/libuci.c b

[OpenWrt-Devel] [PATCH] ubus: Fix Wformat-nonliteral warning

2019-11-29 Thread Rosen Penev
t;name, msgstr); | ^~~ Signed-off-by: Rosen Penev --- examples/server.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/examples/server.c b/examples/server.c index e0cde0b..e218efa 100644 --- a/examples/server.c +++ b/examples/server.c @@ -84,7 +84,7 @@ static int test

[OpenWrt-Devel] [PATCH] opkg: Fix -Wformat-overflow warning

2019-11-29 Thread Rosen Penev
ote: in expansion of macro 'opkg_msg' 45 | opkg_msg(l, fmt": %s.\n", ##args, strerror(errno)) | ^~~~ note: in expansion of macro 'opkg_perror' 247 | opkg_perror(ERROR, "Failed to create temporary directory '%s'", Signed-off-b

Re: [OpenWrt-Devel] default compiler hardening options for all C projects [Was: Re: [PATCH] uci: Fix Wformat-nonliteral warning]

2019-11-30 Thread Rosen Penev
> > >> Wouldn't it make sense to enable following hardening flags (maybe consider >> others as well?): >> >> -Werror=format-security >> -Werror=format-nonliteral Actually the flag I used was -Werror=format=2. It includes all of them. The nonliteral warning is not applicable everywhere. In so

Re: [OpenWrt-Devel] default compiler hardening options for all C projects [Was: Re: [PATCH] uci: Fix Wformat-nonliteral warning]

2019-11-30 Thread Rosen Penev
On Sat, Nov 30, 2019 at 7:41 AM Petr Štetiar wrote: > > Petr Štetiar [2019-11-30 15:04:40]: > > > Wouldn't it make sense to enable following hardening flags (maybe consider > > others as well?): > > > > -Werror=format-security > > -Werror=format-nonliteral > > FYI, following uci patch: > > d

[OpenWrt-Devel] [PATCHv2] uci: Fix Wformat-nonliteral warning

2019-11-30 Thread Rosen Penev
-nonliteral] vfprintf(stderr, fmt, ap); Signed-off-by: Rosen Penev --- v2: Fixed extra warning from clang 10. cli.c| 1 + libuci.c | 10 +- 2 files changed, 6 insertions(+), 5 deletions(-) diff --git a/cli.c b/cli.c index 8970f4f..6ba97ea 100644 --- a/cli.c +++ b/cli.c

[OpenWrt-Devel] [PATCH] uci: Fix extra semicolons warnings

2019-11-30 Thread Rosen Penev
tatement has no effect; remove unnecessary ';' to silence this warning [-Werror,-Wextra-semi-stmt] UCI_TRAP_SAVE(ctx, ignore); error: empty expression statement has no effect; remove unnecessary ';' to silence this warning [-Werror,-Wextra-semi-stmt] }; Signe

Re: [OpenWrt-Devel] [PATCH] firewall: Fix Wformat-nonliteral warning

2019-12-01 Thread Rosen Penev
> On Dec 1, 2019, at 5:55 AM, Jonas Gorski wrote: > > Hi, > >> On Fri, 29 Nov 2019 at 22:06, Rosen Penev wrote: >> >> Allows GCC to check the formats by switching to a define, which is a >> constant expression. >> >> Fixes: >> >&

[OpenWrt-Devel] [PATCH] uci: Fix Wformat-nonliteral warning

2019-12-02 Thread Rosen Penev
-nonliteral] vfprintf(stderr, fmt, ap); Signed-off-by: Rosen Penev --- v3: Moved define to top of file. v2: Fixed extra warning from clang 10. cli.c| 1 + libuci.c | 15 --- 2 files changed, 9 insertions(+), 7 deletions(-) diff --git a/cli.c b/cli.c index 8970f4f

[OpenWrt-Devel] [PATCHv4] uci: Fix Wformat-nonliteral warning

2019-12-07 Thread Rosen Penev
-nonliteral] vfprintf(stderr, fmt, ap); Signed-off-by: Rosen Penev --- v4: Use a proper named define v3: Moved define to top of file. v2: Fixed extra warning from clang 10. cli.c| 1 + libuci.c | 15 --- 2 files changed, 9 insertions(+), 7 deletions(-) diff --git a

Re: [OpenWrt-Devel] Inquery

2019-12-11 Thread Rosen Penev
On Wed, Dec 11, 2019 at 11:32 AM Alberto Bursi wrote: > > > > On 11/12/19 18:54, Daniel Golle wrote: > > On Wed, Dec 11, 2019 at 05:37:26PM +0100, WRT Burner wrote: > >> On 11/12/2019 15:22, Daniel Golle wrote: > >>> And it's even needless to say that > >>> replying to a spam email in which ever w

Re: [OpenWrt-Devel] Inquery

2019-12-12 Thread Rosen Penev
On Thu, Dec 12, 2019 at 2:01 AM Daniel Golle wrote: > > Hi Alberto, > > On Wed, Dec 11, 2019 at 08:32:41PM +0100, Alberto Bursi wrote: > > > > > > On 11/12/19 18:54, Daniel Golle wrote: > > > On Wed, Dec 11, 2019 at 05:37:26PM +0100, WRT Burner wrote: > > > > On 11/12/2019 15:22, Daniel Golle wrot

[OpenWrt-Devel] [PATCH] libcxx: Add package

2019-12-13 Thread Rosen Penev
package needs to be cleaned up and updated. Runtime tested with gddrescue, gdisk, dcwapd, bonnie++, and aircrack-ng on a TP-Link Archer C7v2. Signed-off-by: Rosen Penev --- config/Config-build.in | 3 ++ include/uclibc++.mk | 12 +- package/libs/libcxx/Makefile

[OpenWrt-Devel] [PATCH] iperf: Fix compilation with libcxx

2019-12-13 Thread Rosen Penev
Avoids redefining bool. Signed-off-by: Rosen Penev --- package/network/utils/iperf/Makefile | 2 +- package/network/utils/iperf/patches/010-libcxx.patch | 12 2 files changed, 13 insertions(+), 1 deletion(-) create mode 100644 package/network/utils/iperf/patches

[OpenWrt-Devel] [PATCH] perf: Add libunwind only if selected

2019-12-21 Thread Rosen Penev
The depends are totally wrong. libunwind does not work with powerpc and i386 as it needs glibc. Instead of duplicating the platforms, just change the dependency. Signed-off-by: Rosen Penev --- package/devel/perf/Makefile | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a

[OpenWrt-Devel] [PATCH] libcxx: Depenency fixes

2019-12-22 Thread Rosen Penev
Don't build with uClibc-ng. It's totally unsupported as several functions are missing. Make the musl libc support conditional. Fix hash with make check FIXUP=1. Apparently I based the Makefile off of libedit and forgot to fix the hash. Signed-off-by: Rosen Penev --- config/Confi

Re: [OpenWrt-Devel] [PATCHv2 1/3] tools/pkg-config: Replace with pkgconf

2019-12-25 Thread Rosen Penev
On Wed, Dec 25, 2019 at 3:17 AM Petr Štetiar wrote: > > Rosen Penev [2019-11-28 11:19:31]: > > > pkgconf is a newer, actively maintained implementation of pkg-config > > I'm wondering if that `actively maintained` part still holds true. It does. The team has managemen

Re: [OpenWrt-Devel] [PATCH] use strncpy instead of strcpy

2019-12-26 Thread Rosen Penev
On Thu, Dec 26, 2019 at 2:32 PM Khem Raj wrote: > > On Thu, Dec 26, 2019 at 4:50 AM Hauke Mehrtens wrote: > > > > On 12/24/19 10:50 PM, Petr Štetiar wrote: > > > Khem Raj [2019-12-24 10:02:13]: > > > > > > Hi, > > > > > > use "PATCH libubox" subject prefix, because there is no blobmsg.c file in

[OpenWrt-Devel] [PATCH] nettle: Disable ARMEB assembly

2019-12-27 Thread Rosen Penev
It's broken for ARMv5, which is the only armeb target in OpenWrt. Signed-off-by: Rosen Penev --- package/libs/nettle/Makefile | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/package/libs/nettle/Makefile b/package/libs/nettle/Makefile index 62c39702ed..b9324e4793 1

[OpenWrt-Devel] [PATCH 2/3] toolchain/wrapper.sh: Replace read with read -r

2019-12-29 Thread Rosen Penev
Without -r, backslashes would get mangled. Found with shellcheck. Signed-off-by: Rosen Penev --- target/toolchain/files/wrapper.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target/toolchain/files/wrapper.sh b/target/toolchain/files/wrapper.sh index bd4996cac2

[OpenWrt-Devel] [PATCH 3/3] toolchain/wrapper.sh: Use /bin/sh instead of bash

2019-12-29 Thread Rosen Penev
No bash features are actually used here. Signed-off-by: Rosen Penev --- target/toolchain/files/wrapper.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target/toolchain/files/wrapper.sh b/target/toolchain/files/wrapper.sh index ba2b34778f..d42c6a14d5 100755 --- a/target

[OpenWrt-Devel] [PATCH 1/3] toolchain/wrapper: Replace legacy `` with $()

2019-12-29 Thread Rosen Penev
Found with shellcheck. Signed-off-by: Rosen Penev --- target/toolchain/files/wrapper.sh | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/target/toolchain/files/wrapper.sh b/target/toolchain/files/wrapper.sh index 4452128382..bd4996cac2 100755 --- a/target/toolchain

[OpenWrt-Devel] [PATCH 2/4] scripts/gen_image_generic.sh: Remove $ in arithmetic

2019-12-29 Thread Rosen Penev
Fixes shellcheck warning: SC2004: $/${} is unnecessary on arithmetic variables. 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 1e35b1b349

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

2019-12-29 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 0832bf91de..6a8065e15a 100755 --- a/scripts/gen_image_generic.sh

[OpenWrt-Devel] [PATCH 3/4] scripts/gen_image_generic.sh: Replace deprecated `` with $()

2019-12-29 Thread Rosen Penev
Found with shellcheck. 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 9f3e4d024c..0832bf91de 100755 --- a/scripts/gen_image_generic.sh +++ b/scripts

[OpenWrt-Devel] [PATCH 1/4] scripts/gen_image_generic.sh: Replace -o with ||

2019-12-29 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 2d2fb120ce..1e35b1b349 100755 --- a/scripts

[OpenWrt-Devel] [PATCH 2/4] scripts/clean-package.sh: Use read -r

2019-12-29 Thread Rosen Penev
read will mangle backslashes. Found with shellcheck. Signed-off-by: Rosen Penev --- scripts/clean-package.sh | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/scripts/clean-package.sh b/scripts/clean-package.sh index 987e6685a8..e849c41afa 100755 --- a/scripts/clean

[OpenWrt-Devel] [PATCH 3/4] scripts/clean-package.sh: Remove useless cat

2019-12-29 Thread Rosen Penev
Found with shellcheck. Signed-off-by: Rosen Penev --- scripts/clean-package.sh | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/scripts/clean-package.sh b/scripts/clean-package.sh index e849c41afa..b64d7ad21a 100755 --- a/scripts/clean-package.sh +++ b/scripts/clean

openwrt-devel@lists.openwrt.org

2019-12-29 Thread Rosen Penev
-a is not well defined. Signed-off-by: Rosen Penev --- scripts/clean-package.sh | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/scripts/clean-package.sh b/scripts/clean-package.sh index b64d7ad21a..2c9d4169ba 100755 --- a/scripts/clean-package.sh +++ b/scripts/clean

[OpenWrt-Devel] [PATCH 1/4] scripts/clean-package.sh: exit in case cd fails

2019-12-29 Thread Rosen Penev
Found with shellcheck. Signed-off-by: Rosen Penev --- scripts/clean-package.sh | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/scripts/clean-package.sh b/scripts/clean-package.sh index 6dd9bf7306..987e6685a8 100755 --- a/scripts/clean-package.sh +++ b/scripts/clean

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

2019-12-29 Thread Rosen Penev
On Sun, Dec 29, 2019 at 8:19 PM Stefan Lippers-Hollmann wrote: > > Hi > > On 2019-12-29, Rosen Penev wrote: > > This has nothing bash specific. > [...] > > +++ b/scripts/gen_image_generic.sh > > @@ -1,4 +1,4 @@ > > -#!/usr/bin/env bash > > +#!/bin/

[OpenWrt-Devel] [PATCHv2 2/4] scripts/gen_image_generic.sh: Remove $ in arithmetic

2019-12-29 Thread Rosen Penev
Fixes shellcheck warning: SC2004: $/${} is unnecessary on arithmetic variables. 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 1e35b1b349

[OpenWrt-Devel] [PATCHv2 1/4] scripts/gen_image_generic.sh: Replace -o with ||

2019-12-29 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 2d2fb120ce..1e35b1b349 100755 --- a/scripts

[OpenWrt-Devel] [PATCHv2 3/4] scripts/gen_image_generic.sh: Replace deprecated `` with $()

2019-12-29 Thread Rosen Penev
Found with shellcheck. 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 9f3e4d024c..0832bf91de 100755 --- a/scripts/gen_image_generic.sh +++ b/scripts

[OpenWrt-Devel] [PATCHv2 4/4] scripts/gen_image_generic.sh: use /bin/sh

2019-12-29 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 0832bf91de..6a8065e15a 100755 --- a/scripts/gen_image_generic.sh

[OpenWrt-Devel] [PATCH 2/2] scripts/flashing: Switch to /bin/sh

2019-12-30 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

[OpenWrt-Devel] [PATCH 1/2] scripts/flashing: Switch from echo -en to printf

2019-12-30 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

[OpenWrt-Devel] [PATCH 1/2] scripts/strip-kmod.sh: Replace legacy `` with $()

2019-12-30 Thread Rosen Penev
Found with shellcheck. Signed-off-by: Rosen Penev --- scripts/strip-kmod.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/strip-kmod.sh b/scripts/strip-kmod.sh index 313015b909..7886cd907f 100755 --- a/scripts/strip-kmod.sh +++ b/scripts/strip-kmod.sh @@ -50,6

[OpenWrt-Devel] [PATCH 2/2] scripts/strip-kmod.sh: Use sh instead of bash

2019-12-30 Thread Rosen Penev
There's nothing bash specific here. Signed-off-by: Rosen Penev --- scripts/strip-kmod.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/strip-kmod.sh b/scripts/strip-kmod.sh index 7886cd907f..28be35ea7c 100755 --- a/scripts/strip-kmod.sh +++ b/scripts/strip-km

[OpenWrt-Devel] [PATCH] scripts/symlink-tree.sh: Use /bin/sh

2019-12-30 Thread Rosen Penev
This uses nothing bash specific. Signed-off-by: Rosen Penev --- scripts/symlink-tree.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/symlink-tree.sh b/scripts/symlink-tree.sh index aa169e2455..6ed91dfb5a 100755 --- a/scripts/symlink-tree.sh +++ b/scripts/symlink

[OpenWrt-Devel] [PATCH 1/2] scripts/strip-kmod.sh: Replace legacy `` with $()

2019-12-30 Thread Rosen Penev
Found with shellcheck. Signed-off-by: Rosen Penev --- scripts/strip-kmod.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/strip-kmod.sh b/scripts/strip-kmod.sh index 313015b909..7886cd907f 100755 --- a/scripts/strip-kmod.sh +++ b/scripts/strip-kmod.sh @@ -50,6

[OpenWrt-Devel] [PATCH 1/5] scripts/mkits.sh: Replace legacy `` with $()

2019-12-30 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 93c8cedaed..32a978e18d 100755 --- a/scripts/mkits.sh +++ b/scripts/mkits.sh @@ -15,7 +15,7 @@ # usage

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

2019-12-30 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 3/5] scripts/mkits.sh Add missing quotes

2019-12-30 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 "$

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

2019-12-30 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

[OpenWrt-Devel] [PATCH 5/5] scripts/mkits.sh: Switch from bash to sh

2019-12-30 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

[OpenWrt-Devel] [PATCH] trace-cmd: Fix compilation without fortify-headers

2019-12-30 Thread Rosen Penev
Upstream backport. Signed-off-by: Rosen Penev --- package/devel/trace-cmd/Makefile | 2 +- .../devel/trace-cmd/patches/120-limits.patch | 26 +++ 2 files changed, 27 insertions(+), 1 deletion(-) create mode 100644 package/devel/trace-cmd/patches/120-limits.patch

[OpenWrt-Devel] [PATCH] lldpd: Fix compilation without fortify-headers

2019-12-30 Thread Rosen Penev
Upstream backport. Signed-off-by: Rosen Penev --- package/network/services/lldpd/Makefile | 2 +- .../services/lldpd/patches/010-limits.patch | 21 +++ 2 files changed, 22 insertions(+), 1 deletion(-) create mode 100644 package/network/services/lldpd/patches/010

[OpenWrt-Devel] [PATCH 1/5] scripts/env: use explicit find location

2019-12-31 Thread Rosen Penev
Some find binaries do not imply the current directory. 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 69b63895b4..fd49e1c817 100755 --- a/scripts/env +++ b/scripts/env

[OpenWrt-Devel] [PATCH 3/5] scripts/env: replace \! with !

2019-12-31 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@lists.openwrt.org

2019-12-31 Thread Rosen Penev
The former are not well defined. Found with shellcheck. Signed-off-by: Rosen Penev --- scripts/env | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/scripts/env b/scripts/env index fd49e1c817..563f39c1cf 100755 --- a/scripts/env +++ b/scripts/env @@ -73,7 +73,7

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

2019-12-31 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

<    1   2   3   4   5   6   7   8   9   10   >