Re: [cip-dev] [isar-cip-core][PATCH v1 1/2] swupdate-handler-roundrobin: Keep Lua version 5.5 for sid

Hi Quirin, 

Thank you for the review. 

> It would be better to pin the version for the previous version instead of sid." 

Does this mean keep SWUPDATE_LUA_VERSION = "5.5" and add version specific variables like below?
 
SWUPDATE_LUA_VERSION:trixie = "5.4"
SWUPDATE_LUA_VERSION:bookworm = "5.3"

Thanks & regards
Shivanand K

-----Original Message-----
From: [email protected] <[email protected]> On Behalf Of Quirin Gylstorff via lists.cip-project.org
Sent: 20 July 2026 13:44
To: [email protected]
Subject: Re: [cip-dev] [isar-cip-core][PATCH v1 1/2] swupdate-handler-roundrobin: Keep Lua version 5.5 for sid

Hi,

On 7/20/26 6:59 AM, Kunijadar Shivanand wrote:
> From: Shivanand Kunijadar <[email protected]>
> 
> Update the Lua version to 5.5 as the SWUpdate package in sid checks 
> the lua script at /usr/share/lua/5.5/
> 
> Signed-off-by: Shivanand Kunijadar 
> <[email protected]>
> ---
>   .../swupdate-handler-roundrobin_0.1.bb                           | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git 
> a/recipes-core/swupdate-handler-roundrobin/swupdate-handler-roundrobin
> _0.1.bb 
> b/recipes-core/swupdate-handler-roundrobin/swupdate-handler-roundrobin
> _0.1.bb
> index e3cc7f8..3505a35 100644
> --- 
> a/recipes-core/swupdate-handler-roundrobin/swupdate-handler-roundrobin
> _0.1.bb
> +++ b/recipes-core/swupdate-handler-roundrobin/swupdate-handler-roundr
> +++ obin_0.1.bb
> @@ -26,6 +26,7 @@ SRC_URI += "${@('file://' + d.getVar('SWUPDATE_ROUND_ROBIN_HANDLER_CONFIG')) if
>   # The lua version used by swupdate to search for additional handler is hard coded in debian/rules
>   # see https://salsa.debian.org/debian/swupdate/-/blob/6ccd44a8539ebb880bf0dac408d5db5de7e2de99/debian/rules#L13
>   SWUPDATE_LUA_VERSION ??= "5.4"
> +SWUPDATE_LUA_VERSION:sid = "5.5"
>   SWUPDATE_ADDITIONAL_LUA_VERSIONS ??= "5.3"

It would be better to pin the version for the previous version instead of sid.

Quirin
>   
>   do_prepare_build:append() {
> 
> 
> 
> 
>