On Thu, 2024-04-04 at 14:25 +0200, Alexandre Truong wrote: > The problem is the following: > AssertionError: > 'gitsm://[email protected]/ARMmbed/mbedtls.git;protocol=https;branch=master' != > 'git://[email protected]/ARMmbed/mbedtls.git;protocol=https;branch=master' > > Mbedlts made changes to their repository, adding a sub-module, thus the > assert triggers an error with the url: > git://[email protected]/ARMmbed/mbedtls.git;protocol=https;branch=master > > To fix the issue, the url has been changed to: > gitsm://[email protected]/ARMmbed/mbedtls.git;protocol=https;branch=master > > Signed-off-by: Alexandre Truong <[email protected]> > Reviewed-by: Yoann Congal <[email protected]> > --- > meta/lib/oeqa/selftest/cases/devtool.py | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-)
Thanks! I think this is fine to merge and avoid the current test failures but it is a workaround since there is a version being passed in this test and that version doesn't include the submodules commit. It seems as if devtool/recipetool's submodule detection is checking HEAD for submodules and not the specific revision/version we're passing. Could you file a bug please in our bugzilla so we don't forget this issue and can come back to fixing it properly? FWIW, this is the problematic change upstream: https://github.com/Mbed-TLS/mbedtls/commit/456a54da8ef44c8b725b524567c08cffec6a7214 which we should mention in the commit message too. Cheers, Richard
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#197962): https://lists.openembedded.org/g/openembedded-core/message/197962 Mute This Topic: https://lists.openembedded.org/mt/105327620/21656 Group Owner: [email protected] Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
