[OpenWrt-Devel] sysupgrade broken by 30f61a34b4cf ("base-files: always use staged sysupgrade")
zajec5 at gmail.com
Mon Jun 11 06:20:00 EDT 2018
On Sun, 10 Jun 2018 at 15:53, Matthias Schiffer
<mschiffer at universe-factory.net> wrote:
> On 06/09/2018 02:54 PM, Rafał Miłecki wrote:
> > brcm47xx and bcm53xx targets support multiple firmware formats, most
> > of them are TRX files with vendor-specific header prepended.
> > They are handled in 
> > target/linux/bcm53xx/base-files/lib/upgrade/platform.sh
> > by the platform_do_upgrade() which builds a proper command for
> > extracting TRX out of vendor-specific format.
> > Example:
> > To sysupgrade using Linksys's CyberTAN format something like this is called:
> > default_do_upgrade "/tmp/linksys-foo.bin" "dd bs=64 skip=1"
> > The problem is above commit introduced a regression breaking
> > sysupgrade using all these formats. The string returned by get_image()
> > has changed
> > Before:
> > cat /tmp/linksys-foo.bin | dd skip=32 iflag=skip_bytes
> > After:
> > dd skip=32 iflag=skip_bytes /tmp/linksys-foo.bin 2>/dev/null
> > As you can see command generated by get_image() after commit 
> > 30f61a34b4cf ("base-files: always use staged sysupgrade") has changed,
> > is wrong and can't work.
> > Matthias: can you look at this, please?
> please check if the fix in my staging fixes the issue.
If does, thank you!
openwrt-devel mailing list
openwrt-devel at lists.openwrt.org
More information about the openwrt-devel