Re: [PATCH v4] ath79: add support for Ubiquiti PowerBeam M (XW)

2021-07-04 Thread Russell Senior
Merged in:

  commit 12eb5b2384a63427c0062856331fda60a1ebb309
  reboot-17087-g12eb5b2384

Thanks Adrian!

-- 
Russell Senior
russ...@personaltelco.net

On Fri, Jul 2, 2021 at 9:43 PM Russell Senior  wrote:
>
> It has been about a month now since v4. Is it time to apply this? I am
> using it in production. It doesn't touch anything else, so I am not
> seeing a downside in merging.
>
> --
> Russell Senior
> russ...@personaltelco.net
>
> On Sun, Jun 13, 2021 at 8:33 PM Russell Senior
>  wrote:
> >
> > Is it time to apply this?
> >
> > --
> > Russell Senior
> > russ...@personaltelco.net
> >
> > On Fri, Jun 4, 2021 at 5:29 PM Russell Senior  
> > wrote:
> > >
> > > This patch adds support for the Ubiquiti PowerBeam M (XW), e.g. 
> > > PBE-M5-400,
> > > a 802.11n wireless with a feed+dish form factor. This device was 
> > > previously
> > > supported by the ar71xx loco-m-xw firmware.
> > >
> > > Specifications:
> > >  - Atheros AR9342 SoC
> > >  - 64 MB RAM
> > >  - 8 MB SPI flash
> > >  - 1x 10/100 Mbps Ethernet port, 24 Vdc PoE-in
> > >  - Power and LAN green LEDs
> > >  - 4x RSSI LEDs (red, orange, green, green)
> > >  - UART (115200 8N1)
> > >
> > > Flashing via stock GUI:
> > >  - Downgrade to AirOS v5.5.x (latest available is 5.5.10-u2) first (see
> > >https://openwrt.org/toh/ubiquiti/powerbeam installation instructions)
> > >  - Upload the factory image via AirOS web GUI.
> > >
> > > Flashing via TFTP:
> > >  - Use a pointy tool (e.g., unbent paperclip) to keep the
> > >reset button pressed.
> > >  - Power on the device (keep reset button pressed).
> > >  - Keep pressing until LEDs flash alternatively LED1+LED3 =>
> > >LED2+LED4 => LED1+LED3, etc.
> > >  - Release reset button.
> > >  - The device starts a TFTP server at 192.168.1.20.
> > >  - Set a static IP on the computer (e.g., 192.168.1.21/24).
> > >  - Upload via tftp the factory image:
> > > $ tftp 192.168.1.20
> > > tftp> bin
> > > tftp> trace
> > > tftp> put 
> > > openwrt-ath79-generic-x-ubnt_powerbeam-m-xw-squashfs-factory.bin
> > >
> > > WARNING: so far, no non-destructive method has been discovered for
> > > opening the enclosure to reach the serial console. Internal photos
> > > are available here: https://fcc.io/SWX-NBM5HP
> > >
> > > since v1:
> > >  * renamed from ubnt,powerbeam-m to ubnt,powerbeam-m-xw
> > > since v2:
> > >  * removed redundant mtd-mac-address from dts
> > > since v3:
> > >  * based on testimony of Joe Ayers :
> > >   * the bricking problem was resolved by 076d58d3440f (2016-02)
> > >   * the tftp flashing problem was resolved by 6009b3fd586a (2018-11)
> > >so, removed the associated warnings.
> > >
> > > Signed-off-by: Russell Senior 
> > > ---
> > >  .../ath79/dts/ar9342_ubnt_powerbeam-m-xw.dts  | 34 +++
> > >  .../generic/base-files/etc/board.d/01_leds|  1 +
> > >  .../generic/base-files/etc/board.d/02_network |  1 +
> > >  target/linux/ath79/image/generic-ubnt.mk  |  8 +
> > >  4 files changed, 44 insertions(+)
> > >  create mode 100644 target/linux/ath79/dts/ar9342_ubnt_powerbeam-m-xw.dts
> > >
> > > diff --git a/target/linux/ath79/dts/ar9342_ubnt_powerbeam-m-xw.dts 
> > > b/target/linux/ath79/dts/ar9342_ubnt_powerbeam-m-xw.dts
> > > new file mode 100644
> > > index 00..6db058f887
> > > --- /dev/null
> > > +++ b/target/linux/ath79/dts/ar9342_ubnt_powerbeam-m-xw.dts
> > > @@ -0,0 +1,34 @@
> > > +// SPDX-License-Identifier: GPL-2.0-or-later OR MIT
> > > +
> > > +#include "ar9342_ubnt_xw.dtsi"
> > > +
> > > +/ {
> > > +   compatible = "ubnt,powerbeam-m-xw", "ubnt,xw", "qca,ar9342";
> > > +   model = "Ubiquiti PowerBeam M (XW)";
> > > +};
> > > +
> > > + {
> > > +   status = "okay";
> > > +
> > > +   phy-mask = <4>;
> > > +
> > > +   phy4: ethernet-phy@4 {
> > > +   reg = <4>;
> > > +   };
> > > +};
> > > +
> > > + {
> > > +   status = "okay";
> > > +
> > > +   /* default for ar934x, except for 1000M and 10M */
> > > +   pll-data = <0x0200 0x0101 0x1313>;
> > > +
> > > +   phy-mode = "rgmii-id";
> > > +   phy-handle = <>;
> > > +
> > > +   gmac-config {
> > > +   device = <>;
> > > +   rxd-delay = <3>;
> > > +   rxdv-delay = <3>;
> > > +   };
> > > +};
> > > diff --git a/target/linux/ath79/generic/base-files/etc/board.d/01_leds 
> > > b/target/linux/ath79/generic/base-files/etc/board.d/01_leds
> > > index 1990353394..35ce90aeb4 100644
> > > --- a/target/linux/ath79/generic/base-files/etc/board.d/01_leds
> > > +++ b/target/linux/ath79/generic/base-files/etc/board.d/01_leds
> > > @@ -383,6 +383,7 @@ ubnt,nanostation-loco-m-xw|\
> > >  ubnt,nanostation-m|\
> > >  ubnt,nanostation-m-xw|\
> > >  ubnt,picostation-m|\
> > > +ubnt,powerbeam-m-xw|\
> > >  ubnt,powerbridge-m|\
> > >  ubnt,rocket-m)
> > > ucidef_set_rssimon "wlan0" "20" "1"
> > > diff --git a/target/linux/ath79/generic/base-files/etc/board.d/02_network 
> > > 

RE: [PATCH] gemini: Add hdparm setting

2021-07-04 Thread Adrian Schmutzler
Hi,

> -Original Message-
> From: openwrt-devel [mailto:openwrt-devel-boun...@lists.openwrt.org]
> On Behalf Of Linus Walleij
> Sent: Montag, 5. Juli 2021 00:01
> To: Sebastian Luft ; Hans Ulli Kroll
> ; Hauke Mehrtens ;
> Christian Lamparter 
> Cc: Florian Fainelli ; openwrt-devel@lists.openwrt.org;
> Linus Walleij 
> Subject: [PATCH] gemini: Add hdparm setting
> 
> This uses "hdparm" (if present) to get the harddisk into low power mode on
> NAS set-ups.
> 
> Signed-off-by: Linus Walleij 
> ---
>  .../linux/gemini/base-files/etc/board.d/03_hdparm | 15 +++
>  1 file changed, 15 insertions(+)
>  create mode 100755 target/linux/gemini/base-files/etc/board.d/03_hdparm
> 
> diff --git a/target/linux/gemini/base-files/etc/board.d/03_hdparm
> b/target/linux/gemini/base-files/etc/board.d/03_hdparm
> new file mode 100755
> index ..68d76f0acdf8
> --- /dev/null
> +++ b/target/linux/gemini/base-files/etc/board.d/03_hdparm
> @@ -0,0 +1,15 @@
> +#!/bin/sh

board.d files are not executable anymore. Please remove shebang and execute bit.

Please add an SPDX license identifier instead.

> +# Spin down drives after one minute if inactive
> +
> +which hdparm > /dev/null

I think using which has been discouraged lately in favor of command -v ...

> +if [ ! $? -eq 0 ] ; then

"$?" != 0 ?

> + exit 0
> +fi
> +
> +DISKS=`find /dev/sd[a-z] 2>/dev/null`

Please use $() instead of backticks.

Best

Adrian

> +for DISK in $DISKS
> +do
> + if [ -b $DISK ] ; then
> + hdparm -S 12 /dev/sda > /dev/null
> + fi
> +done
> --
> 2.31.1
> 
> 
> ___
> openwrt-devel mailing list
> openwrt-devel@lists.openwrt.org
> https://lists.openwrt.org/mailman/listinfo/openwrt-devel


openpgp-digital-signature.asc
Description: PGP signature
___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel


[sdwalker/sdwalker.github.io] 2c602d: This week's update

2021-07-04 Thread Stephen Walker via openwrt-devel
The sender domain has a DMARC Reject/Quarantine policy which disallows
sending mailing list messages using the original "From" header.

To mitigate this problem, the original message has been wrapped
automatically by the mailing list software.--- Begin Message ---
  Branch: refs/heads/master
  Home:   https://github.com/sdwalker/sdwalker.github.io
  Commit: 2c602dbfddbfbe239407200824c8c40c4c94f8ab
  
https://github.com/sdwalker/sdwalker.github.io/commit/2c602dbfddbfbe239407200824c8c40c4c94f8ab
  Author: Stephen Walker 
  Date:   2021-07-04 (Sun, 04 Jul 2021)

  Changed paths:
M uscan/index-18.06.html
M uscan/index-19.07.html
M uscan/index-21.02.html
M uscan/index.html

  Log Message:
  ---
  This week's update



--- End Message ---
___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel


[PATCH] gemini: Add hdparm setting

2021-07-04 Thread Linus Walleij
This uses "hdparm" (if present) to get the harddisk into low
power mode on NAS set-ups.

Signed-off-by: Linus Walleij 
---
 .../linux/gemini/base-files/etc/board.d/03_hdparm | 15 +++
 1 file changed, 15 insertions(+)
 create mode 100755 target/linux/gemini/base-files/etc/board.d/03_hdparm

diff --git a/target/linux/gemini/base-files/etc/board.d/03_hdparm 
b/target/linux/gemini/base-files/etc/board.d/03_hdparm
new file mode 100755
index ..68d76f0acdf8
--- /dev/null
+++ b/target/linux/gemini/base-files/etc/board.d/03_hdparm
@@ -0,0 +1,15 @@
+#!/bin/sh
+# Spin down drives after one minute if inactive
+
+which hdparm > /dev/null
+if [ ! $? -eq 0 ] ; then
+   exit 0
+fi
+
+DISKS=`find /dev/sd[a-z] 2>/dev/null`
+for DISK in $DISKS
+do
+   if [ -b $DISK ] ; then
+   hdparm -S 12 /dev/sda > /dev/null
+   fi
+done
-- 
2.31.1


___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel


RE: [PATCH v3] cn913x: add support for iEi Puzzle-M901/Puzzle-M902

2021-07-04 Thread Adrian Schmutzler
Hi,

> -Original Message-
> From: openwrt-devel [mailto:openwrt-devel-boun...@lists.openwrt.org]
> On Behalf Of eveans2...@gmail.com
> Sent: Freitag, 2. Juli 2021 13:16
> To: openwrt-devel@lists.openwrt.org
> Cc: Ian Chang 
> Subject: [PATCH v3] cn913x: add support for iEi Puzzle-M901/Puzzle-M902

This goes into mvebu target, so the commit title prefix should be "mvebu:"!

Further comments below.

> 
> From: Ian Chang 
> 
>  Hardware specification
>  --
>  * CN9130 SoC, Quad-core ARMv8 Cortex-72 @ 2200 MHz
>  * 4 GB DDR
>  * 4 GB eMMC
>  * mmcblk0
>   - mmcblk0p164M  kernel_1
>   - mmcblk0p264M  kernel_2
>   - mmcblk0p3   512M  rootfs_1
>   - mmcblk0p4   512M  rootfs_2
>   - mmcblk0p5   512M  Reserved
>   - mmcblk0p664M  Reserved
>   - mmcblk0p7   1.8G  rootfs_data
> 
>  * 4 MB (SPI Flash)
>  * 6 x 2.5 Gigabit  ports (Puzzle-M901)
> - External PHY with 6 ports (AQR112R)
>  * 6 x 2.5 Gigabit ports (Puzzle-M902)
> - External PHY with 6 ports (AQR112R)
>3 x 10 Gigabit ports (Puzzle-M902)
> - External PHY with 3 ports (AQR113R)
> 
>  * 4 x Front panel LED
>  * 1 x USB 3.0
>  * Reset button on Rear panel
>  * UART (115200 8N1,header on PCB)
> 
>  Flash instructions:
>   The original firmware is based on OpenWrt.
>   Flash firmware using LuCI and CLI
> 
> Signed-off-by: Ian Chang 
> ---
>  .../base-files/etc/board.d/02_network |   6 +
>  .../cortexa72/base-files/lib/upgrade/emmc.sh  |  40 +++
>  .../base-files/lib/upgrade/platform.sh|   8 +
>  .../boot/dts/marvell/cn9130-puzzle-m901.dts   | 216 
>  .../boot/dts/marvell/cn9130-puzzle-m902.dts   | 236 ++
>  .../arm64/boot/dts/marvell/cn9130-puzzle.dtsi |  37 +++
>  .../boot/dts/marvell/cn9131-puzzle-m901.dts   | 153 
>  .../boot/dts/marvell/cn9131-puzzle-m902.dts   | 136 ++
>  .../boot/dts/marvell/cn9132-puzzle-m902.dts   | 209 
>  target/linux/mvebu/image/cortexa72.mk |  20 ++
>  10 files changed, 1061 insertions(+)
>  create mode 100644 target/linux/mvebu/cortexa72/base-
> files/lib/upgrade/emmc.sh
>  create mode 100644
> target/linux/mvebu/files/arch/arm64/boot/dts/marvell/cn9130-puzzle-
> m901.dts
>  create mode 100644
> target/linux/mvebu/files/arch/arm64/boot/dts/marvell/cn9130-puzzle-
> m902.dts
>  create mode 100644
> target/linux/mvebu/files/arch/arm64/boot/dts/marvell/cn9130-puzzle.dtsi
>  create mode 100644
> target/linux/mvebu/files/arch/arm64/boot/dts/marvell/cn9131-puzzle-
> m901.dts
>  create mode 100644
> target/linux/mvebu/files/arch/arm64/boot/dts/marvell/cn9131-puzzle-
> m902.dts
>  create mode 100644
> target/linux/mvebu/files/arch/arm64/boot/dts/marvell/cn9132-puzzle-
> m902.dts
> 
> diff --git a/target/linux/mvebu/cortexa72/base-
> files/etc/board.d/02_network b/target/linux/mvebu/cortexa72/base-
> files/etc/board.d/02_network
> index 9ab3c8174d..f3c3cda977 100755
> --- a/target/linux/mvebu/cortexa72/base-files/etc/board.d/02_network
> +++ b/target/linux/mvebu/cortexa72/base-files/etc/board.d/02_network
> @@ -21,6 +21,12 @@ marvell,armada8040-db)
>  marvell,armada7040-db)
>   ucidef_set_interfaces_lan_wan "eth0 eth2" "eth1"
>   ;;
> +iei,cn9131-puzzle-m901)
> + ucidef_set_interfaces_lan_wan "eth1 eth2 eth3 eth4 eth5" "eth0"
> + ;;
> +iei,cn9132-puzzle-m902)
> + ucidef_set_interfaces_lan_wan "eth1 eth2 eth3 eth4 eth5 eth10
> eth11 eth12" "eth0"
> + ;;

Please take care of alphabetic sorting. These should be moved before the 
various marvell... entries.

>  *)
>   ucidef_set_interface_lan "eth0"
>   ;;
> diff --git a/target/linux/mvebu/cortexa72/base-files/lib/upgrade/emmc.sh
> b/target/linux/mvebu/cortexa72/base-files/lib/upgrade/emmc.sh
> new file mode 100644
> index 00..bd7eea5c22
> --- /dev/null
> +++ b/target/linux/mvebu/cortexa72/base-files/lib/upgrade/emmc.sh
> @@ -0,0 +1,40 @@
> +platform_do_upgrade_emmc() {
> + local board=$(board_name)
> + local diskdev partdev
> +
> + export_bootdevice && export_partdevice diskdev 0 || {
> + v "Unable to determine upgrade device"
> + return 1

Broken indent

> + }
> +
> + sync
> +
> + if [ "$UPGRADE_OPT_SAVE_PARTITIONS" = "1" ]; then
> + get_partitions "/dev/$diskdev" bootdisk
> +
> + v "Extract boot sector from the image"
> + get_image_dd "$1" of=/tmp/image.bs count=1 bs=512b
> +
> + get_partitions /tmp/image.bs image
> + fi
> +
> + #iterate over each partition from the image and write it to the boot
> disk
> + while read part start size; do
> + if export_partdevice partdev $part; then
> + if [ "$partdev" = "mmcblk0p2" ]; then
> + v "Writing image mmcblk0p3 for
> /dev/$partdev  $start $size"
> + get_image_dd "$1" of="/dev/mmcblk0p3"
> ibs="512" obs=1M skip="$start" count="$size" conv=fsync
> +  

Re: [PATCH v6 1/3] dnsmasq: distinct Ubus names for multiple instances

2021-07-04 Thread Etan Kissling



On 04.07.21, 14:49, "Nick Lowe"  wrote:

> Is this expected to contain thekelleys.org.uk?

Assuming you meant uk.org.thekelleys (reversed).

Yes, it is matching previous behaviour where the default name was used
for all the instances. In situation where this can conflict, this patch
will now only assign the default name to at most one instance, and will
assign unique names to other instances.

> -1, --enable-dbus[=]
> Allow dnsmasq configuration to be updated via DBus method calls.
> The configuration which can be changed is upstream DNS servers (and
> corresponding domains) and cache clear. Requires that dnsmasq has
> been built with DBus support. If the service name is given, dnsmasq
> provides service at that name, rather than the default which is
> uk.org.thekelleys.dnsmasq

See https://thekelleys.org.uk/dnsmasq/docs/dnsmasq-man.html

Etan


___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel


Re: [PATCH v6 1/3] dnsmasq: distinct Ubus names for multiple instances

2021-07-04 Thread Nick Lowe
Hi all,

Is this expected to contain thekelleys.org.uk?

+   [ $dbus -gt 0 ] && xappend
"--enable-dbus=uk.org.thekelleys.$instance_name"

Best,

Nick

___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel


RE: [PATCH 19.07 4/4] treewide: mark selected packages nonshared

2021-07-04 Thread Adrian Schmutzler
> >   PKG_NAME:=json-c
> >   PKG_VERSION:=0.12.1
> > -PKG_RELEASE:=3.1
> > +PKG_RELEASE:=3.2
> I've never seen a non integer release, is there a special reason for this?

I've also used this as standard scheme for changes in stable branches. The 
advantage is that you immediately see up to which version the changes are 
shared with master, and when it starts to deviate.

Best

Adrian 


openpgp-digital-signature.asc
Description: PGP signature
___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel


RE: [PATCH] This patch adds supports for GL-B2200-EMMC.

2021-07-04 Thread Adrian Schmutzler
Hi,

> -Original Message-
> From: openwrt-devel [mailto:openwrt-devel-boun...@lists.openwrt.org]
> On Behalf Of Li Zhang
> Sent: Freitag, 2. Juli 2021 10:25
> To: openwrt-devel@lists.openwrt.org
> Cc: Li Zhang 
> Subject: [PATCH] This patch adds supports for GL-B2200-EMMC.

this patch has multiple formal issues.

Please visit https://openwrt.org/submitting-patches
and also consider checking out
https://openwrt.org/docs/guide-developer/device-support-policies

Some quick additional content-related comments below.

> 
> Specifications:
>   - SOC: Qualcomm IPQ4019 ARM Quad-Core
>   - RAM: 512 MiB
>   - Flash: 16 MiB NOR - SPI0
>   - EMMC: 8GB EMMC
>   - ETH: Qualcomm QCA8075
>   - WLAN1: Qualcomm Atheros QCA4019 2.4GHz 802.11b/g/n 2x2
>   - WLAN2: Qualcomm Atheros QCA4019 5GHz 802.11n/ac W2 2x2
>   - WLAN3: Qualcomm Atheros QCA9886 5GHz 802.11n/ac W2 2x2
>   - INPUT: Reset, WPS
>   - LED: Power, Internet
>   - UART1: On board pin header near to LED (3.3V, TX, RX, GND), 3.3V
> without pin - 115200 8N1
>   - UART2: On board with BLE module
>   - SPI1: On board socket for Zigbee module
> 
> Update firmware instructions
> 
> Pleae update firmware on uboot web(default 192.168.1.1).
> ---
>  package/firmware/ipq-wifi/Makefile |   2 +
>  .../ipq-wifi/board-glinet_gl-b2200-emmc.qca4019| Bin 0 -> 24316 bytes
>  .../ipq-wifi/board-glinet_gl-b2200-emmc.qca9888| Bin 0 -> 12168 bytes
>  target/linux/ipq40xx/Makefile  |   2 +-
>  .../ipq40xx/base-files/etc/board.d/02_network  |   5 +
>  .../etc/hotplug.d/firmware/11-ath10k-caldata   |   3 +
>  .../arm/boot/dts/qcom-ipq4019-gl-b2200-emmc.dts| 366
> +
>  .../arch/arm/boot/dts/qcom-ipq4029-gl-s1300.dts|   7 -
>  target/linux/ipq40xx/image/generic.mk  |  27 ++
>  .../patches-5.4/901-arm-boot-add-dts-files.patch   |   3 +-
>  10 files changed, 406 insertions(+), 9 deletions(-)  create mode 100644
> package/firmware/ipq-wifi/board-glinet_gl-b2200-emmc.qca4019
>  create mode 100644 package/firmware/ipq-wifi/board-glinet_gl-b2200-
> emmc.qca9888
>  create mode 100644 target/linux/ipq40xx/files/arch/arm/boot/dts/qcom-
> ipq4019-gl-b2200-emmc.dts
> 
> diff --git a/package/firmware/ipq-wifi/Makefile b/package/firmware/ipq-
> wifi/Makefile
> index e63e3b3..3d02048 100644
> --- a/package/firmware/ipq-wifi/Makefile
> +++ b/package/firmware/ipq-wifi/Makefile
> @@ -39,6 +39,7 @@ ALLWIFIBOARDS:= \
>   engenius_emr3500 \
>   ezviz_cs-w3-wd1200g-eup \
>   glinet_gl-ap1300 \
> + glinet_gl-b2200-emmc \
>   glinet_gl-s1300 \
>   linksys_ea8300 \
>   linksys_mr8300-v0 \
> @@ -125,6 +126,7 @@ $(eval $(call generate-ipq-wifi-
> package,engenius_emd1,EnGenius EMD1))  $(eval $(call generate-ipq-wifi-
> package,engenius_emr3500,EnGenius EMR3500))  $(eval $(call generate-ipq-
> wifi-package,ezviz_cs-w3-wd1200g-eup,EZVIZ CS-W3-WD1200G EUP))  $(eval
> $(call generate-ipq-wifi-package,glinet_gl-ap1300,GL.iNet GL-AP1300))
> +$(eval $(call generate-ipq-wifi-package,glinet_gl-b2200-emmc,GL.iNet
> +GL-B2200-EMMC))
>  $(eval $(call generate-ipq-wifi-package,glinet_gl-s1300,GL.iNet GL-S1300))
> $(eval $(call generate-ipq-wifi-package,linksys_ea8300,Linksys EA8300))
> $(eval $(call generate-ipq-wifi-package,linksys_mr8300-v0,Linksys MR8300))
> diff --git a/package/firmware/ipq-wifi/board-glinet_gl-b2200-emmc.qca4019
> b/package/firmware/ipq-wifi/board-glinet_gl-b2200-emmc.qca4019
> new file mode 100644
> index
> ..c4721328e1bbaad8722b3b5f7
> 4c88af6c8c5214d
> GIT binary patch
> literal 24316
> zcmeHPdr(tX8b1jj>SE!BfCzXAAv{7MK%g2V@)iR`u&79YB2YzgYm|rKVdz3TJ}5
> *)
> zpcV`fv<%9lDI(+Ii&(6!ol>>oS(N88y)cV^wS9qmp#v%9$G-
> sD0Ei6#PD9`_69
> zobR0bedm1VcTbXgbAH@66XJs7c8kJ7Q-kBv<1!L~OeO 308ua%%FDMF6c+{+
> z%A~<>7fVZmrTe#r61Bo31!ep3DnB%+qM%G#aQNuf?c(5gK0h=xc-
> xL02{Ib=XTiQ$
> zA);a$4cOE~NxQ1G{0C55QV`C#V3x1mLxBLNlFlxoo^E0K=e|q|qCS x?@3
> z44G_T2d^l1js2Q+BZ22o;4ytoZ`EULP1YR=<=#UF;G}<%NAMU)eY%W!^@yiCf
> !bQ4
> zhL6kAzhDE4Xw+v+hG8v+X>V_DXJ=<)AOPIKXKJCiKs&?a3hRc+!fvL_9HC5Am
> bg;V
> zG>9)7z091<*r2Mh`apJSvS93*s lQ_@}{u?sFh1IlHd3R%Kb07
> z{$T7ARYP@Qb}~FTq-
> w5~X75an9_z!+V4DCsG<6@WhX?+FVC<^uG#p_^a^%>j
> z&;cVHdSNHU*{Om6*j-C?Np^-HRPW$FGrpJ9s-
> e8ZWaeSYVJ04?Ys(=%IItv!te
> zm5~ShGd)wsTgz@pt-5JCaTXl)2@14CHh~o??|d7xXq1$Ri8=c5O_)-
> F&!a
> zn=~lLCHjlL>W|V-
> @ya*8eRH6)V@G3frDxH|>Ef#B1Kdm`@)op9#CSQ*(U*Ir?V^U)
> zD?N{WUBQtMUXoWHbCqM{`cu(MzDn{|$ev&;Go9vjIIe1j7IjD3;7e&7W
> Wp3&)a
> z8lKzOz^~<1da=J!mlf*v;N6{kl*93Xf4*#QZy+US~u42dci5P__$j5Ide
> z3y$#f!qy1YBPqUE6 2BRpZ#{?DF0d91Ttz;KDK
> zO9tWf2-
> Heyn+EZOHwNk@0E!<6m_O6EMmU6hY$9I3x@WLrUbwYL9==)Y?8<0Ybg
> p
> zwn|~33?Abl193;)dG2B-)CqAz MM1=BDK9B#9k8DA=@V1CU
> z(NIKyP6eU9WLlkyNhq9(ic)8tA4?!{MxCEHBOa&+<%QqfdisyG}Ce-
> e
> zp;5djvF;=o=EP7s=odW4mu<-
> h>OsTgJ4Z42YB8xAy;bH^T@Kfz)Eq2X0u)LVN@ftC
> zmWCN%W;Cxzlx846RVV}~1SkY31SkY31XeHt>-
> 

Re: Broken 21.02 rc3 imagebuilder [Was: Re: Should ubus be marked as target-specific "nonshared"? (broken 21.02 rc2 imagebuilder)]

2021-07-04 Thread Hannu Nyman

Petr Štetiar kirjoitti 2.7.2021 klo 16.11:

Hannu Nyman  [2021-06-05 13:40:21]:

Hi,


Based on forum discussion, we have currently a broken 21.02.0-rc2
imagebuilder, as libiwinfo can't find the correct libubus version.

seems like there is broken 21.02.0-rc3 imagebuilder as well:

  openwrt-imagebuilder-21.02.0-rc3-ipq40xx-generic.Linux-x86_64$ make image 
PROFILE=avm_fritzbox-7530 PACKAGES=luci-ssl-openssl
  ...
  Collected errors:
...

It seems like that currently any package that depends on nonshared package
needs to be treated as nonshared as well, otherwise such issues would happen.

-- ynezz



My solution attempt aimed just to mitigate the "nonshared static packages 
depend on ABI revisioned shared packages" case, so that at least the 
nonshared package set would remain coherent in releases. iwinfo was one such 
package and apprently @jow- fixed that now separately (by changing it to shared).


Similar common packages are still e.g. mtd, block-mount and blockd, which are 
nonshared but depend on ABI revisioned shared packages like libubox, libubus 
etc., so they can be broken by version bumps. See analysis in 
https://lists.openwrt.org/pipermail/openwrt-devel/2021-June/035442.html


Regarding the generic case, the combination of ABI revisioning and 
shared/nonshared build difference (volatile/fixed) is the killer here. There 
is no real solution.


The easiest mitigation might be to backport changes into these key ABI 
revisioned shared packages in the release branches just shortly before next 
rc or dot releases, so that the imagebuilder remains broken only for a short 
time.




___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel