[PATCH v2 1/4] rules: add AUTORELEASE and COMMITCOUNT variables

2021-01-19 Thread Paul Spooren
on change, using a 1 based counter. - $(COMMITCOUNT): For non-traditional versioning (x.y.z), most prominent `base-files`, it is possible to pass an argument to get the total number of commits. The new functionality can also be used by other feeds like packages.git. Suggested-by: Daniel Golle

Re: [PATCH] autoconf: update to 2.70

2021-01-18 Thread Paul Spooren
On So, Jan 17, 2021 at 17:50, Rosen Penev wrote: Refreshed patches. Removed emacs patch as upstream removed emacs completely. Removed musl host patch. Upstream seems to have fixed it differently. Signed-off-by: Rosen Penev --- Please disable the doc building, at least that's how I

Re: [PATCH] iperf3: move over to packages.git

2021-01-18 Thread Paul Spooren
feed. Agree, patch sent. CC: Felix Fietkau Signed-off-by: Paul Spooren Reviewed-by: Rosen Penev Reviewed-by: Florian Eckert --- package/network/utils/iperf3/Makefile | 83 --- 1 file changed, 83 deletions(-) delete mode 100644 package/network/utils/iperf3

Re: [PATCH v2 3/4] image-commands: support Chromium OS image-type creation

2021-01-17 Thread Paul Spooren
On So, Jan 17, 2021 at 09:20, Brian Norris wrote: Hi Paul, On Sun, Jan 17, 2021 at 2:43 AM Paul Spooren wrote: On Sat Jan 16, 2021 at 5:07 PM HST, Brian Norris wrote: > See the previous patches, which implemented the cros-vbutil > verified-boot payload-packing tool, and extended

Re: [PATCH v2 0/4] Add support for Chromium OS and Google WiFi

2021-01-17 Thread Paul Spooren
On Sat Jan 16, 2021 at 5:07 PM HST, Brian Norris wrote: > Hi, > > This series adds support for both Chromium OS (or particularly, its > kernel-payload signing and disk layout) and for a device using it (the > first generation Google WiFi). > > Google WiFi (code-named "Gale") is an IPQ4019-based

Re: [PATCH v2 3/4] image-commands: support Chromium OS image-type creation

2021-01-17 Thread Paul Spooren
On Sat Jan 16, 2021 at 5:07 PM HST, Brian Norris wrote: > See the previous patches, which implemented the cros-vbutil > verified-boot payload-packing tool, and extended ptgen for the CrOS > kernel partition type. With these, it's now possible to package kernel + > rootfs to make disk images that

Re: [PATCH 1/2] libnl: add libnl-cli library

2021-01-16 Thread Paul Spooren
Jan 16, 2021 2:19:15 AM Pavel Šimerda : On 1/16/21 12:50 PM, Petr Štetiar wrote: Pavel Šimerda [2021-01-16 00:23:01]: Hi, My perspective is that we are working on *hardware enablement* here. At least some switch chips support Link Aggregation in addition to hardware switching and VLAN

[PATCH] iperf3: move over to packages.git

2021-01-15 Thread Paul Spooren
That package is not required for building OpenWrt so the maintenance should happen over at packages.git. CC: Felix Fietkau Signed-off-by: Paul Spooren --- package/network/utils/iperf3/Makefile | 83 --- 1 file changed, 83 deletions(-) delete mode 100644 package

[PATCH v3] scripts: target-metadata don't add PROFILES twice

2021-01-12 Thread Paul Spooren
`. This patch removes duplicate profile IDs and only adds them once to `.profiles.mk`. Signed-off-by: Paul Spooren --- v3: don't use the Apache-2 licensed uniq function from a Perl lib This duplicate entries were reported recently via this PR: https://github.com/openwrt/openwrt/pull/3777 scripts

Re: [PATCH] base-files: sysupgrade: store status of system-services

2021-01-10 Thread Paul Spooren
On So, Jan 10, 2021 at 22:40, Sven Roederer wrote: Am Sonntag, 10. Januar 2021, 09:47:27 CET schrieb Andre Heider: > Same. I would personally like this as default sysupgrade procedure, as > that's what makes most sense imho. > If I have disabled a service it makes sense that after a

Re: [PATCH buildbot 3/3] docker,worker: install qemu-utils

2021-01-10 Thread Paul Spooren
On So, Jan 10, 2021 at 10:25, Paul Spooren wrote: To convert x86 images to VDI/VMDK the `qemu-img` tools is required, which is part of Debians `qemu-utils` package. The goal is to remove the ancient in-tree `qemu-img` version and replace it with a recent host installed package. Related

[PATCH buildbot 2/3] docker,worker: install gcc-mulitlib

2021-01-10 Thread Paul Spooren
wrt/docker/pull/74/commits/f9da188d53fd9809ca4911089611e1f42422d049 Signed-off-by: Paul Spooren --- docker/buildslave/Dockerfile | 1 + 1 file changed, 1 insertion(+) diff --git a/docker/buildslave/Dockerfile b/docker/buildslave/Dockerfile index 20ec53f..408db3f 100644 --- a/docker/buildsl

[PATCH buildbot 1/3] docker,worker: alphabetically sort packages

2021-01-10 Thread Paul Spooren
Improve readability of installed packages in the worker container. Signed-off-by: Paul Spooren --- docker/buildslave/Dockerfile | 22 +++--- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/docker/buildslave/Dockerfile b/docker/buildslave/Dockerfile index 9ee3ae4

[PATCH buildbot 3/3] docker,worker: install qemu-utils

2021-01-10 Thread Paul Spooren
-by: Paul Spooren --- docker/buildslave/Dockerfile | 1 + 1 file changed, 1 insertion(+) diff --git a/docker/buildslave/Dockerfile b/docker/buildslave/Dockerfile index 408db3f..9cee882 100644 --- a/docker/buildslave/Dockerfile +++ b/docker/buildslave/Dockerfile @@ -25,6 +25,7 @@ RUN echo 'deb http

Re: [PATCH 1/3] build/json: generate json file for initramfs images

2021-01-10 Thread Paul Spooren
On Sa, Jan 9, 2021 at 00:51, Moritz Warning wrote: The initramfs images are missing from the profiles.json files. Signed-off-by: Moritz Warning --- Superseded by GitHub patch[1]. Overall I'm not happy with this approach as it complicates multiple ends, instead I suggested a simpler

[PATCH v3 2/3] base-files: replace PKG_RELEASE with commitcount

2021-01-10 Thread Paul Spooren
The newly added `commitcount` function does automatic versioning based on the number of Git commits of a package. Replace *tedious to bump* and *merge conflict causing* `PKG_RELEASE` and replace it with `commitcount`. Signed-off-by: Paul Spooren --- package/base-files/Makefile | 3 +-- 1 file

[PATCH v3 1/3] rules: add commitcount function

2021-01-10 Thread Paul Spooren
-r15385+75-c5d033a34d_x86_64.ipk In cases where no Git system is used, the timestamp of SOURCE_DATE_EPOCH is used instead, allowing reproducible builds. base-files_210107.30634-r1538581-c5d033a34d_x86_64.ipk Signed-off-by: Paul Spooren --- v2: - reproducible versions via SOURCE_DATE_EPOCH

[PATCH v3 3/3] uboot-envtools: use `commitcount` for release

2021-01-10 Thread Paul Spooren
Use `commitcount` function rather than manually setting a PKG_RELEASE. Signed-off-by: Paul Spooren --- package/boot/uboot-envtools/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/package/boot/uboot-envtools/Makefile b/package/boot/uboot-envtools/Makefile index

Re: [PATCH] base-files: sysupgrade: store status of system-services

2021-01-09 Thread Paul Spooren
On So, Jan 3, 2021 at 23:14, Sven Roederer wrote: When saving the list of installed pkgs, also store the status of the system services. The list is created in the etc/backup folder also and formated as: /etc/init.d/ {enable|disable} This way it can be sourced after sysupgrade, to restore

[PATCH v2 3/3] uboot-envtools: use `commitcount` for release

2021-01-07 Thread Paul Spooren
Use `commitcount` function rather than manually setting a PKG_RELEASE. Signed-off-by: Paul Spooren --- package/boot/uboot-envtools/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/package/boot/uboot-envtools/Makefile b/package/boot/uboot-envtools/Makefile index

[PATCH v2 2/3] base-files: replace PKG_RELEASE with commitcount

2021-01-07 Thread Paul Spooren
The newly added `commitcount` function does automatic versioning based on the number of Git commits of a package. Replace *tedious to bump* and *merge conflict causing* `PKG_RELEASE` and replace it with `commitcount`. Signed-off-by: Paul Spooren --- package/base-files/Makefile | 3 +-- 1 file

[PATCH v2 1/3] rules: add commitcount function

2021-01-07 Thread Paul Spooren
-r15385+75-c5d033a34d_x86_64.ipk In cases where no Git system is used, the timestamp of SOURCE_DATE_EPOCH is used instead, allowing reproducible builds. base-files_210107.30634-r1538581-c5d033a34d_x86_64.ipk Signed-off-by: Paul Spooren --- v2: - reproducible versions via SOURCE_DATE_EPOCH

[PATCH 2/3] base-files: replace PKG_RELEASE with commitcount

2021-01-07 Thread Paul Spooren
The newly added `commitcount` function does automatic versioning based on the number of Git commits of a package. Replace *tedious to bump* and *merge conflict causing* `PKG_RELEASE` and replace it with `commitcount`. Signed-off-by: Paul Spooren --- package/base-files/Makefile | 3 +-- 1 file

[PATCH 3/3] uboot-envtools: use `commitcount` for release

2021-01-07 Thread Paul Spooren
Use `commitcount` function rather than manually setting a PKG_RELEASE. Signed-off-by: Paul Spooren --- package/boot/uboot-envtools/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/package/boot/uboot-envtools/Makefile b/package/boot/uboot-envtools/Makefile index

[PATCH 1/3] rules: add commitcount function

2021-01-07 Thread Paul Spooren
-r15385+75-c5d033a34d_x86_64.ipk In cases where no Git system is used, a folder timestamp is used in combination with the hardcoded $(REVISION) variable (set in releases). base-files_210107.30634-r1538581-c5d033a34d_x86_64.ipk Signed-off-by: Paul Spooren --- rules.mk | 22

Re: [PATCH] tools/fakeroot: update to 1.25.3

2021-01-05 Thread Paul Spooren
Merged, thank you. On So, Jan 3, 2021 at 22:57, Paul Spooren wrote: On Mo, Jan 4, 2021 at 16:34, Syrone Wong wrote: This crashes on buildbots with the follow error: bash: /buildbot/shared-workdir/build/staging_dir/host/bin/autoreconf: No such file or directory You'll need

Re: [PATCH] base-files: read all 3 bytes in get_magic_vfat() at once

2021-01-04 Thread Paul Spooren
On Mo, Jan 4, 2021 at 01:28, Adrian Schmutzler wrote: While the speed improvement might be negligible, there is still no reason to read individual bytes. Signed-off-by: Adrian Schmutzler --- I'm wondering how many times dd needs to be run to sum up to the same amount of time this took

Re: [PATCH] tools/fakeroot: update to 1.25.3

2021-01-04 Thread Paul Spooren
On Mo, Jan 4, 2021 at 16:34, Syrone Wong wrote: This crashes on buildbots with the follow error: bash: /buildbot/shared-workdir/build/staging_dir/host/bin/autoreconf: No such file or directory You'll need to set autoreconf as a dependency. I think in tools/Makefile something like

Re: [PATCH] tools/fakeroot: update to 1.25.3

2021-01-03 Thread Paul Spooren
On Sa, Jan 2, 2021 at 05:28, Syrone Wong wrote: use PKG_FIXUP:=autoreconf to generate configure 200-hide-dlsym-error.patch deleted due to fixed upstream in another way other patches refreshed to reflect latest changes Signed-off-by: Syrone Wong --- tools/fakeroot/Makefile

Re: LuCI: split luci.mk into a common and a feed-internal file

2021-01-01 Thread Paul Spooren
On Sa, Jan 2, 2021 at 03:31, Sven Roederer wrote: Hi, based on Pauls recent idea "rules: add findrev function from LuCI" of reusing some LuCI-code, I'd like to point to a github-PR with a related suggestion: https://github.com/openwrt/luci/pull/2637 As noted in the PR-message, it

Re: build-requirements when using the Imagebuilder

2021-01-01 Thread Paul Spooren
On Fr, Jan 1, 2021 at 17:39, Sven Roederer wrote: Hi, based on the recent changes to the tools required when using the imagebuilder, I wonder If there are more tools that can be excluded for this case. Candidates I see here are: * patch * unzip * bzip2 * Perl 5.x (including Data::Dumper,

[PATCH 2/2] base-files: replace PKG_RELEASE with findrev

2021-01-01 Thread Paul Spooren
The newly added `findrev` function does automatic versioning based on Git commits of the package. Replace tedious to bump and merge conflict causing `PKG_RELEASE` and replace it with `findrev`. CC: Adrian Schmutzler Signed-off-by: Paul Spooren --- package/base-files/Makefile | 3 +-- 1 file

[PATCH 1/2] rules: add findrev function from LuCI

2021-01-01 Thread Paul Spooren
. CC: Jo-Philipp Wich Signed-off-by: Paul Spooren --- rules.mk | 24 1 file changed, 24 insertions(+) diff --git a/rules.mk b/rules.mk index f79340b124..dcefd92761 100644 --- a/rules.mk +++ b/rules.mk @@ -408,6 +408,30 @@ endef # file extension ext=$(word $(words

Re: [PATCH] tools/fakeroot: update to 1.25.3

2021-01-01 Thread Paul Spooren
Please don't double post on github and mailing list at the same time. I'll review this soonish! -- Jan 1, 2021 11:30:39 AM Syrone Wong : use PKG_FIXUP:=autoreconf to generate configure 200-hide-dlsym-error.patch deleted due to fixed upstream in another way other patches refreshed to reflect

Re: Persistent HTTP(S) connections in opkg

2020-12-31 Thread Paul Spooren
On Mi, Dez 30, 2020 at 19:14, Baptiste Jonglez wrote: Following the discussion in http://lists.openwrt.org/pipermail/openwrt-devel/2020-November/032297.html There are basically two options to implement persistent connections in opkg: 1) keep calling "wget", but pass the list of all

Re: [PATCH] build/prereq: merge ifndef IB block together

2020-12-31 Thread Paul Spooren
On Mi, Dez 30, 2020 at 18:10, Paul Spooren wrote: On Mi, Dez 30, 2020 at 18:28, Baptiste Jonglez wrote: Hi, On 29-12-20, Paul Spooren wrote: Multiple prereq checks are only required within the build system but not for the ImageBuilder. These checks are excluded by using ifndef IB

Re: [PATCH] usbutils: remove

2020-12-31 Thread Paul Spooren
On Do, Dez 31, 2020 at 11:47, Bjørn Mork wrote: Paul Spooren writes: On Mi, Dez 30, 2020 at 10:03, Bjørn Mork wrote: Rosen Penev writes: I have git send-email preset to send stuff to this email address. I usually don't CC anyone. I don't understand what "maintainer&q

[PATCH] usbreset: remove redundant package

2020-12-30 Thread Paul Spooren
The package `usbutils` already offers an USB reset function, this package is therefore not really required standalone. CC: Jo-Philipp Wich Signed-off-by: Paul Spooren --- package/utils/usbreset/Makefile | 39 - package/utils/usbreset/src/usbreset.c | 236

Re: [PATCH] usbutils: remove

2020-12-30 Thread Paul Spooren
On Mi, Dez 30, 2020 at 10:03, Bjørn Mork wrote: Rosen Penev writes: I have git send-email preset to send stuff to this email address. I usually don't CC anyone. I don't understand what "maintainer" means in an OpenWrt context... My understanding is that a maintainer is a reasonable

Re: [PATCH] build/prereq: merge ifndef IB block together

2020-12-30 Thread Paul Spooren
On Mi, Dez 30, 2020 at 18:28, Baptiste Jonglez wrote: Hi, On 29-12-20, Paul Spooren wrote: Multiple prereq checks are only required within the build system but not for the ImageBuilder. These checks are excluded by using ifndef IB. This commit merges the three ifndef IB blocks

[PATCH] build/prereq: merge ifndef IB block together

2020-12-29 Thread Paul Spooren
Multiple prereq checks are only required within the build system but not for the ImageBuilder. These checks are excluded by using ifndef IB. This commit merges the three ifndef IB blocks together. Signed-off-by: Paul Spooren --- include/prereq-build.mk | 6 +- 1 file changed, 1 insertion

Re: [PATCH] usbutils: remove

2020-12-29 Thread Paul Spooren
via IRC: > I have a version bump to 013, but it requires libudev-zero from the packages feed Acked-by: Paul Spooren package/utils/usbutils/Makefile | 60 - 1 file changed, 60 deletions(-) delete mode 100644 package/utils/usbutils/Makefile diff --

[PATCH] scripts/qemustart: use squashfs instead of ext4

2020-12-29 Thread Paul Spooren
The qemustart script currently picks the ext4 filesystem rather than squashfs, while the latter is default for nearly all OpenWrt targets. Change the default behaviour of qemustart to be in line with the rest. Signed-off-by: Paul Spooren --- scripts/qemustart | 2 +- 1 file changed, 1

RE: [PATCH] scripts/feed: no warn on toolchain/linux overwrite

2020-12-28 Thread Paul Spooren
On Mo, Dez 28, 2020 at 01:02, Adrian Schmutzler wrote: -Original Message- From: openwrt-devel [mailto:openwrt-devel-boun...@lists.openwrt.org] On Behalf Of Paul Spooren Sent: Sonntag, 27. Dezember 2020 22:31 To: openwrt-devel@lists.openwrt.org Cc: Paul Spooren ; ~aparcar

Re: [PATCH] glibc: add arc700 patch

2020-12-28 Thread Paul Spooren
Hi Alexey, could you please test this patch? I have no devboard at hand. Kind regards, Paul On So, Dez 27, 2020 at 15:29, Rosen Penev wrote: On Sat, Dec 26, 2020 at 11:50 AM Hans Dedecker wrote: Hi, On Fri, Dec 25, 2020 at 1:37 AM Rosen Penev wrote: > > glibc does not officially

Re: [RFC PATCH 1/2] base-files: sysupgrade: add function for conffiles retrieval

2020-12-27 Thread Paul Spooren
-by: Adrian Schmutzler Acked-by: Paul Spooren --- package/base-files/files/sbin/sysupgrade | 18 +++--- 1 file changed, 11 insertions(+), 7 deletions(-) diff --git a/package/base-files/files/sbin/sysupgrade b/package/base-files/files/sbin/sysupgrade index ca2eb298dc..4ecea5a303

Re: [RFC PATCH 2/2] base-files: sysupgrade: fine-tune preserved files during upgrade

2020-12-27 Thread Paul Spooren
or providing --no-keepd ... the files managed by opkg accessible via /usr/lib/opkg/status by setting SAVE_CONFIG_OPKG=0 or providing --no-opkg If a file is listed in multiple locations, it is kept until it is "disabled" everywhere. Signed-off-by: Adrian Schmutzler Acked-by: Pa

[PATCH] scripts/feed: no warn on toolchain/linux overwrite

2020-12-27 Thread Paul Spooren
he warning string. Signed-off-by: Paul Spooren --- scripts/feeds | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/scripts/feeds b/scripts/feeds index 122f8568cb..eee0a50717 100755 --- a/scripts/feeds +++ b/scripts/feeds @@ -537,7 +537,9 @@ sub install_src { my $overrid

GitHub contact email address

2020-12-18 Thread Paul Spooren
Hi team, the current contact mail on OpenWrt is openwrt-devel[at]lists.openwrt.org, I'd suggest to change it to contact[at]openwrt.org, as I've never heard anyone using the former. Best, Paul ___ openwrt-devel mailing list

[RFC] raise gcc/make versions for 20.x

2020-12-15 Thread Paul Spooren
Hi, I've seen two patches for version raises of build requirements and would like to know if we should merge them before or after 20.x. make: 3.81.x -> 4.1.x gcc: 4.8 -> 6.x I'm in favor to merge both *before* the branch. Best, Paul ___

Re: [PATCH] toolchain: gcc: Remove support for GCC 5

2020-12-15 Thread Paul Spooren
will not support this old GCC version any more. Signed-off-by: Hauke Mehrtens --- Acked-by: Paul Spooren This goes hand in hand with Petrs gcc6+ patches. I'm wondering if these major version requirement changes should happen before or after the 20.x change, I'd like the former

Re: [PATCH] mvebu/cortexa53: add targz rootfs feature

2020-12-14 Thread Paul Spooren
On Mo, Dez 14, 2020 at 21:22, Daniel Golle wrote: On Mon, Dec 14, 2020 at 10:10:32PM +0100, Josef Schlehofer wrote: On 14. 12. 20 21:59, Paul Spooren wrote: > The rootfs is required to create rootfs Docker containers. With this > feature enable a new runtime test can

[PATCH] mvebu/cortexa53: add targz rootfs feature

2020-12-14 Thread Paul Spooren
The rootfs is required to create rootfs Docker containers. With this feature enable a new runtime test can happen for the aarch64_cortex-a53 architecture. CC: Josef Schlehofer Signed-off-by: Paul Spooren --- target/linux/mvebu/cortexa53/target.mk | 2 +- 1 file changed, 1 insertion(+), 1

Re: [PATCH] uhttpd: don't redirect to HTTPS by default

2020-12-14 Thread Paul Spooren
automatic redirect permanent: $ uci set uhttpd.main.redirect_https=1 $ uci commit uhttpd $ service uhttpd reload Signed-off-by: Petr Štetiar --- Acked-by: Paul Spooren package/network/services/uhttpd/files/uhttpd.config | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [PATCH] utils: simplify mkdir_p boolean conditions

2020-12-14 Thread Paul Spooren
Please add the specific project to the subject like [PATCH foobar] On So, Dez 13, 2020 at 11:57, Rui Salvaterra wrote: Just a trivial simplification. Signed-off-by: Rui Salvaterra --- utils.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/utils.c b/utils.c index

[PATCH] build/json: add filesystem information

2020-12-13 Thread Paul Spooren
Some images are created using different filesystems, most popular squashfs and ext4. To allow downstream projects to distinguesh between those, add the `filesystem` information to created json files. Signed-off-by: Paul Spooren --- include/image.mk | 1 + scripts

[PATCH v2] build/prereq: require make 4.1 or later

2020-12-13 Thread Paul Spooren
was introduced in make 4.1. All major distributions seem to updated to 4.1 or later, so this ideally dosen't break anything. Signed-off-by: Paul Spooren --- Fixed two typos include/prereq-build.mk | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/include/prereq-build.mk b/

[PATCH] build/prereq: requie make 4.1 or later

2020-12-12 Thread Paul Spooren
was introduced in make 4.1. All major distributions seem to updated to 4.1 or later, so this ideally dosn't break anything. Signed-off-by: Paul Spooren --- include/prereq-build.mk | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/include/prereq-build.mk b/include/prereq-bu

[PATCH] iftop: remove package

2020-12-09 Thread Paul Spooren
The package has no reason to be in openwrt.git. Move it to packages.git. Signed-off-by: Paul Spooren Acked-by: Jo-Philipp Wich --- package/network/utils/iftop/Makefile | 45 1 file changed, 45 deletions(-) delete mode 100644 package/network/utils/iftop/Makefile

[RFC] remove x86 qemu images

2020-12-06 Thread Paul Spooren
Hi, openwrt.git includes an old version of QEMU (0.14 vs 5.1.0 in packages.git) only to convert x86 images to vdi and vmdk. Is there anyone actively using the vanilla x86 QEMU images from the upstream servers or can can we remove that "feature"? This would allow to remove tools/qemu and

Re: [PATCH 2/2] openvpn: remove

2020-12-01 Thread Paul Spooren
Felix you're the maintainer, more blessings? Also would you like to stay the maintainer or would e.g. Rosen like to adopt yet another package? Best On Di, Dez 1, 2020 at 12:37, Rosen Penev wrote: This will be moved to packages. Signed-off-by: Rosen Penev --- Acked-by: Paul Spooren

Re: [PATCH] libroxml: remove

2020-11-27 Thread Paul Spooren
Done, thank you. On Fr, Nov 27, 2020 at 11:29, Felix Fietkau wrote: On 2020-11-27 10:36, Paul Spooren wrote: I'm happy to handle this but you're the maintainer. Please give your blessing. I hereby bless you :) Thanks for taking care of this - Felix

Re: [PATCH] libroxml: remove

2020-11-27 Thread Paul Spooren
I'm happy to handle this but you're the maintainer. Please give your blessing. On Do, Nov 26, 2020 at 23:54, Rosen Penev wrote: This will be moved to the packages feed as nothing here uses it. Signed-off-by: Rosen Penev --- package/libs/libroxml/Makefile | 48

Re: [PATCH] Revert "build: switch VERSION_REPO to HTTPS"

2020-11-25 Thread Paul Spooren
On Wed Nov 25, 2020 at 4:11 AM HST, Petr Štetiar wrote: > Baptiste Jonglez [2020-11-25 12:41:18]: > > Hi, > > > For the imagebuilder, it increases the *total* build time (not just > > download time!) by +50%: > > > > http://lists.openwrt.org/pipermail/openwrt-devel/2020-September/031406.html > >

Re: [PATCH v2] tools/sstrip: update to latest version

2020-11-25 Thread Paul Spooren
On Wed Nov 25, 2020 at 6:48 AM HST, Rui Salvaterra wrote: > Drop our local sstrip copy and use the current ELFKickers upstream > version. > > Patch the original makefile in order to avoid building elftoc, since it > fails > with musl's elf.h. This is fine, since we only need sstrip anyway. > >

[PATCH] Revert "build: switch VERSION_REPO to HTTPS"

2020-11-25 Thread Paul Spooren
obsolete. It is still possible to manually change feeds to HTTPS if desired, but the default can be HTTP. Signed-off-by: Paul Spooren --- This was already requested via IRC and accepted somewhat accepted as the current ustream-wolfssl implementation is broken. Ref: https

[PATCH] config: clean double whitespace in Config-build.in

2020-11-25 Thread Paul Spooren
Trivial cosmetic cleanup. This also helps for script that parse for options in Config files. Signed-off-by: Paul Spooren --- config/Config-build.in | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/config/Config-build.in b/config/Config-build.in index 4148180cd5

Re:

2020-11-22 Thread Paul Spooren
Not sure if it works for DMARC emails, but: Acked-by: Paul Spooren On Sun Nov 22, 2020 at 4:23 AM HST, Paulo Machado via openwrt-devel wrote: > The sender domain has a DMARC Reject/Quarantine policy which disallows > sending mailing list messages using the original "F

Re: [PATCH opkg 0/5] Purge packages from cache when they have incorrect checksum

2020-11-20 Thread Paul Spooren
a9/arm_cortex-a9_vfpv3-d16 increased a tiny bit, but that seems totally worth it. Comparing package sizes... Change Local Remote Package +98 66602 66504 opkg Therefore, Acked-by: Paul Spooren > > Thanks, > Baptiste > > On 25-08-20, Baptiste Jonglez wrote: >

RE: 20.xx: postponse LuCI HTTPS per default

2020-11-20 Thread Paul Spooren
On Fri Nov 20, 2020 at 7:35 AM HST, Adrian Schmutzler wrote: > Hi, > > > -Original Message- > > From: openwrt-devel [mailto:openwrt-devel-boun...@lists.openwrt.org] > > On Behalf Of Alberto Bursi > > Sent: Freitag, 20. November 2020 17:32 > > To: openwrt-devel@lists.openwrt.org > >

20.xx: state of the DSA

2020-11-19 Thread Paul Spooren
Hi all, DSA (Distributed Switch Architecture)[0] is a main feature of 20.xx and one of the last blockers for a branch. The goal states[1] support where possible, not necessarily every target. This mail thread should be used to get an overview of the missing bits. The following targets seem use

20.xx: postponse LuCI HTTPS per default

2020-11-19 Thread Paul Spooren
Hi, The current list of release goals for 20.xx states[0] that LuCI should use HTTPS per default. This works by creating on-device a self-signed certificate. Self-signed certificates result in warnings and may cause more harm than good, multiple discussion are found in the mail archive. As no

Re: [PATCH] busybox: enable whois by default

2020-11-18 Thread Paul Spooren
t take up any extra space for the default case. [2]: https://github.com/openwrt/packages/pull/13944 Sunshine, Paul > > [1] > https://forum.openwrt.org/t/whois-binary-cannot-find-providing-package/31274 > > Regards, > > Josef > > > On 17. 11. 20 21:18, Paul Spooren wrote:

Re: [PATCH v2] build: create $(PKG_SYMVERS_DIR) if non-existent

2020-11-18 Thread Paul Spooren
On Tue Nov 17, 2020 at 9:05 PM HST, Felix Fietkau wrote: > > On 2020-11-18 06:54, Oldřich Jedlička wrote: > > út 17. 11. 2020 v 23:00 odesílatel Sebastian Kemper > > napsal: > >> > >> Commit 5d76065 moved the creation of the symvers directory to > >> include/kernel-build.mk. This is fine when

Re: [PATCH] tools: sstrip: update to latest version

2020-11-17 Thread Paul Spooren
On Wed Nov 11, 2020 at 1:53 PM HST, Rui Salvaterra wrote: > Drop our local sstrip copy and use the current ELFKickers upstream > version. > Patch the original makefile in order to avoid building elftoc, since it > fails > with musl's elf.h. This is fine, since we only need sstrip anyway. >

Re: [PATCH v2] build: create $(PKG_SYMVERS_DIR) if non-existent

2020-11-17 Thread Paul Spooren
built (again) this directory will not be created by the build > system, causing build failure if make tries to copy files into it. > > This moves the creation of the symvers directory back into > include/kernel.mk so that the directory is created in any case. > > Signed-off-by: Seb

Re: [PATCH] build: create $(PKG_SYMVERS_DIR) if non-existent

2020-11-17 Thread Paul Spooren
On Tue Nov 17, 2020 at 11:36 AM HST, Sebastian Kemper wrote: > Commit 5d76065 moved the creation of the symvers directory to > include/kernel-build.mk. This is fine when building from scratch. But > when unpacking an SDK the directory doesn't exist and because the kernel > won't be built (again)

Re: Merging /usr

2020-11-17 Thread Paul Spooren
Hi, > It appears most of the major distros (Debian/Ubuntu, Fedora, Arch, > Solaris) have moved to merge the contents of /{bin,sbin,lib}/ into > /usr/{bin,sbin,lib}/, with top-level symbolic links added for > compatibility. Sounds good to me, do you have an estimate how much work this is? >

Re: [PATCH] busybox: enable whois by default

2020-11-17 Thread Paul Spooren
On Mon Nov 16, 2020 at 9:14 PM HST, Petr Štetiar wrote: > Josef Schlehofer [2020-11-17 02:07:09]: > > Hi, > > > Whois can identify who owns a domain and how to get reach owner. Providing > > this tool in OpenWrt someone does not need to use websites for everything. > > I don't think, that this

Re: [PATCH v2] build: store buildsystem revision in packages

2020-11-16 Thread Paul Spooren
On Mon Jul 20, 2020 at 9:53 AM HST, Paul Spooren wrote: > Buildinfo files were added to allow reproducible builds testing. The > combination of the three buildinfo files > {feeds,config,version}.buildinfo allows to recreate the buildsystem and > check if compiled binaries are bit for

Re: [PATCH 2/2] LICENSE: add SPDX identifiers

2020-11-16 Thread Paul Spooren
On Tue Jul 28, 2020 at 11:32 PM HST, Paul Spooren wrote: > SPDX license header follow a machine readable and versioned identifier > format for licenses. The adoption to such system would simplify > scanning of licenses within OpenWrt. > > Adopted from the Linux Kernel[0] s

RE: [PATCH] scripts: add size_compare.sh

2020-11-16 Thread Paul Spooren
On Thu Sep 3, 2020 at 3:36 AM HST, Adrian Schmutzler wrote: > > -Original Message- > > From: openwrt-devel [mailto:openwrt-devel-boun...@lists.openwrt.org] > > On Behalf Of Paul Spooren > > Sent: Freitag, 31. Juli 2020 05:15 > > To: openwrt-devel@l

[PATCH buildbot] phase2: use full git history for reproducibility

2020-11-12 Thread Paul Spooren
work if the full git history is available. Until now only a shallow copy (--depth 1) would be cloned. With this commit the `feeds.conf.default` is changed to use the full git repository. Signed-off-by: Paul Spooren --- phase2/master.cfg | 7 +++ 1 file changed, 7 insertions(+) diff --git

Re: [PATCH openwrt-19.07] feeds: add freifunk feed

2020-11-09 Thread Paul Spooren
On Thu Nov 5, 2020 at 1:44 PM HST, Daniel Golle wrote: > Read the freifunk packages, that have been moved from the LuCI feed > into its own feed in January 2019. > Use openwrt-19.07 branch of that repository for openwrt-19.07. > > Signed-off-by: Sven Roederer > (cherry picked from commit

Re: x86/64 SDK segfaults on some hosts [Was: Re: [PATCH 1/2] fakeroot: make fakeroot script relocatable]

2020-11-09 Thread Paul Spooren
On Mon Nov 9, 2020 at 7:33 AM HST, Jo-Philipp Wich wrote: > I probably missed the point of your mail - do you meant to imply that my > patch introduced this regression? Since the CI builds start failing the same day as the patches were commited there could be a correlation. There is also a GitHub

[PATCH] bcm27xx: add bit variant to redundant RaspberryPi

2020-11-06 Thread Paul Spooren
it is possible to tell the architecture. Signed-off-by: Paul Spooren --- target/linux/bcm27xx/image/Makefile | 2 ++ 1 file changed, 2 insertions(+) diff --git a/target/linux/bcm27xx/image/Makefile b/target/linux/bcm27xx/image/Makefile index f009b997e9..6b533cad42 100644 --- a/target/linux/bcm27xx

[PATCH opkg v2] pkg_hash: don't suggest incompatible packages

2020-10-26 Thread Paul Spooren
in the kmods archive and the target specific packages feed. This commit fixes the issue by dropping all package problematic candidates by checking if all their dependencies could be installed. Signed-off-by: Paul Spooren --- v2: - Only resolve dependencies if architecture is selected and package

[PATCH buildbot] phase1: don't add kmod feed via buildbot

2020-10-21 Thread Paul Spooren
of *hacked* into the images by the buildbot. Signed-off-by: Paul Spooren --- Patch for openwrt.git is here: https://github.com/openwrt/openwrt/pull/3425/commits/9343e7d65494b6b9fd7bf0d76b0d5371f7e6dbd2 phase1/master.cfg | 27 --- 1 file changed, 27 deletions(-) diff --git

[PATCH opkg] pkg_hash: don't suggest incompatible packages

2020-10-17 Thread Paul Spooren
in the kmods archive and the target specific packages feed. This commit fixes the issue by dropping all package problematic candidates by checking if all their dependencies could be installed. Signed-off-by: Paul Spooren --- libopkg/pkg_hash.c | 20 +--- 1 file changed, 17

[RFC]Secure ImageBuilders

2020-10-13 Thread Paul Spooren
Hi, Over the last month I suggested two different approaches to improve the ImageBuilder security. The amount of feedback was fairly low, maybe partly because it's not trivial to understand the issue. I wrote a summary of the two approaches and the current problem, please spend the 5 minutes to

Re: [PATCH v2] imagebuilder: add package signature verification

2020-10-01 Thread Paul Spooren
On Tue Sep 15, 2020 at 3:24 PM HST, Paul Spooren wrote: > The ImageBuilder downloads pre-built packages and adds them to images. > This process uses `opkg` which has the capability to verify package list > signatures via `usign`, as enabled per default on running OpenWrt > devices. &

Re: [PATCH] build, imagebuilder: Do not require compilers

2020-09-29 Thread Paul Spooren
ip the checking for > the compilers if running as ImageBuilder. A similar change has been > made for libncurses-dev in 4a1a58a3e2d2. > > Signed-off-by: Sven Roederer Acked-by: Paul Spooren > --- > include/prereq-build.mk | 4 > 1 file changed, 4 insertions(+) > > diff --git

[PATCH 2/2] libopkg: remove "extra_data" option

2020-09-24 Thread Paul Spooren
;__dummy__" it is cleaner to "comment out" the specific repository and Debian repository structure is nowhere used in the OpenWrt ecosystem, therefore it seem save to remove "extra_data". Signed-off-by: Paul Spooren --- libopkg/opkg.c | 17 - libopkg

[PATCH 1/2] libopkg: remove support for "dist" config

2020-09-24 Thread Paul Spooren
The "dist" config option is legacy to imitate Debians repository structure. However this is never used within the OpenWrt ecosystem and can therefore be removed. Signed-off-by: Paul Spooren --- libopkg/opkg_conf.c | 32 +++- libopkg/opkg_conf.h | 1

Policy for adding new files to OpenWrt

2020-09-21 Thread Paul Spooren
Hi team, Based on PR#3427 on GitHub[0] I'd suggest we add a policy that every added file needs a SPDX header. This likely simplifies merging patches into mainline Kernel and probably makes some lawyers happy. There are fancy tools to do so, like the `reuse` project[1]. Best, Paul [0]:

[PATCH v2] imagebuilder: add package signature verification

2020-09-15 Thread Paul Spooren
-by: Paul Spooren --- target/imagebuilder/Makefile | 10 +- target/imagebuilder/files/Makefile | 2 ++ 2 files changed, 11 insertions(+), 1 deletion(-) diff --git a/target/imagebuilder/Makefile b/target/imagebuilder/Makefile index ad19ab2b53..0cdc1f4d93 100644 --- a/target

[PATCH] conf: add src/{,gz}/trusted option

2020-09-15 Thread Paul Spooren
are unsigned. The two new config options are: src/trusted src/gz/trusted Signed-off-by: Paul Spooren --- ImageBuilders offer a folder called "packages/" which includes at least the packages kernel_*.ipk and libc*.ipk, additionally packages provided by the user. It is no

Re: HTTPS performance issue with opkg (Was: [PATCH 3/3] build: switch VERSION_REPO to HTTPS)

2020-09-15 Thread Paul Spooren
On Mon Sep 14, 2020 at 10:23 PM HST, Daniel Golle wrote: > On Tue, Sep 15, 2020 at 08:49:51AM +0200, Baptiste Jonglez wrote: > > On 27-08-20, Paul Spooren wrote: > > > The variable VERSION_REPO is used by opkg to download package(list)s. > > > Now that the default ins

Re: [PATCH] imagebuilder: add package signature verification

2020-09-14 Thread Paul Spooren
Hi, On Mon Sep 14, 2020 at 12:27 PM HST, Baptiste Jonglez wrote: > Hi, > > Thanks for the patch, it looks good but comments below: > > On 25-08-20, Paul Spooren wrote: > > The ImageBuilder downloads pre-built packages and adds them to images. > > This process uses `opk

Re: [PATCH] imagebuilder: add package signature verification

2020-09-13 Thread Paul Spooren
Hi, On Tue Aug 25, 2020 at 10:24 PM HST, Paul Spooren wrote: > The ImageBuilder downloads pre-built packages and adds them to images. > This process uses `opkg` which has the capability to verify package list > signatures, as enabled per default on running OpenWrt devices. >

<    1   2   3   4   5   6   >