riscv-software-src / riscv-tests

Other
906 stars 463 forks source link

fix CI for OpenOCD/debugger tests #542

Closed aap-sc closed 9 months ago

aap-sc commented 9 months ago

git ls-remote lists both refs/head and refs/remotes. riscv-openocd got a new entry in refs/remotes that matches riscv pattern around Jan 25. This breaks gitlab pipeline. The commit restricts ls-remote only to refs/head.

In addition this commit changes the address of riscv-openocd repository, since it was moved to riscv-collab space a while ago.

aap-sc commented 9 months ago

This PR should fix the issue: https://github.com/riscv-software-src/riscv-tests/issues/543

aap-sc commented 9 months ago

@lz-bro , @aswaterman FYI.

@en-sc could you please review this one?

en-sc commented 9 months ago

Seems like the other match, which cased trouble (refs/remotes/riscv-openocd/riscv) was added to riscv-openocd by mistake. I have deleted it and the pipelines should work without the patch from now on (re-run example).

However, the link to riscv-openocd repo should still be updated.

As for the part regarding revisions, I do think we can use a more robust mechanism here. I see the following alternatives:

aap-sc commented 9 months ago

I have deleted it and the pipelines should work without the patch from now on ([re-run example]

Yep. It works now. I'll abandon the patch then