From 3d7040b7d69cefaccbd7f58c686853e6d3db31f8 Mon Sep 17 00:00:00 2001 From: Markus Stockhausen Date: Fri, 11 Oct 2024 15:11:43 -0400 Subject: [PATCH] realtek: scripts: fix error in belkin-header For some reason the new belkin-header.py script works without issues in a local Fedora build environment. In the OpenWrt build pipeline it produces the following errors: Traceback (most recent call last): File "/builder/shared-workdir/build/scripts/belkin-header.py", line 92, in head = create_header(buf, args.belkin_header, args.belkin_model) File "/builder/shared-workdir/build/scripts/belkin-header.py", line 68, in create_header head[28:29] = VERSION1.to_bytes(1) TypeError: to_bytes() missing required argument 'byteorder' (pos 2) This may be related due to different python version. Fix this by handing over the needed parameters Signed-off-by: Markus Stockhausen Link: https://github.com/openwrt/openwrt/pull/16667 Signed-off-by: Robert Marko --- scripts/belkin-header.py | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/scripts/belkin-header.py b/scripts/belkin-header.py index 4dc3d2652e..f62d55cab7 100755 --- a/scripts/belkin-header.py +++ b/scripts/belkin-header.py @@ -35,7 +35,7 @@ COMPANY = "belkin" MODULE = "IMG" def xcrc32(buf): - return (0xffffffff - zlib.crc32(buf, 0xffffffff)).to_bytes(4, 'big') + return (0xffffffff - zlib.crc32(buf, 0xffffffff)).to_bytes(4, byteorder='big') def encode_model(model): map = " 0123456789ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz-" @@ -63,12 +63,12 @@ def create_header(buf, belkin_header, belkin_model): head[0:4] = int(belkin_header, 0).to_bytes(4, 'big') head[8:12] = int(time.time()).to_bytes(4, 'big') - head[12:16] = len(buf).to_bytes(4, 'big') + head[12:16] = len(buf).to_bytes(4, byteorder='big') head[24:28] = xcrc32(buf) - head[28:29] = VERSION1.to_bytes(1) - head[29:30] = VERSION2.to_bytes(1) - head[30:31] = VERSION3.to_bytes(1) - head[31:32] = VERSION4.to_bytes(1) + head[28:29] = VERSION1.to_bytes(1, byteorder='big') + head[29:30] = VERSION2.to_bytes(1, byteorder='big') + head[30:31] = VERSION3.to_bytes(1, byteorder='big') + head[31:32] = VERSION4.to_bytes(1, byteorder='big') head[16:16 + len(COMPANY)] = bytes(COMPANY,'ascii') mod = MODULE + "-{:1d}.{:02d}.{:02d}.{:02d}".format(VERSION1, VERSION2, VERSION3, VERSION4)