Skip to content

Remove orphan cocotblib submodule gitlink - #20

Open
dnltz wants to merge 1 commit into
SpinalHDL:masterfrom
dnltz:fix/remove-orphan-cocotblib-gitlink
Open

Remove orphan cocotblib submodule gitlink#20
dnltz wants to merge 1 commit into
SpinalHDL:masterfrom
dnltz:fix/remove-orphan-cocotblib-gitlink

Conversation

@dnltz

@dnltz dnltz commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

Commit 77f8d29 ("HMAC_MD5 spinalSim done + remove all cocotb test") deleted .gitmodules, whose only entry was tester/src/python/cocotblib, but left the gitlink itself in the tree.

A gitlink without a matching .gitmodules entry has no URL to resolve, so any recursive clone of this repository, or of a project that vendors it as a submodule, aborts:

fatal: No url found for submodule path
'tester/src/python/cocotblib' in .gitmodules
fatal: Failed to recurse into submodule path '...'

Plain clones are unaffected, which is why this went unnoticed. Nothing in the repository references cocotb any more, so drop the gitlink.

Commit 77f8d29 ("HMAC_MD5 spinalSim done + remove all cocotb test")
deleted .gitmodules, whose only entry was tester/src/python/cocotblib,
but left the gitlink itself in the tree.

A gitlink without a matching .gitmodules entry has no URL to resolve, so
any recursive clone of this repository, or of a project that vendors it
as a submodule, aborts:

  fatal: No url found for submodule path
         'tester/src/python/cocotblib' in .gitmodules
  fatal: Failed to recurse into submodule path '...'

Plain clones are unaffected, which is why this went unnoticed. Nothing
in the repository references cocotb any more, so drop the gitlink.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Signed-off-by: Daniel Schultz <dnltz@aesc-silicon.de>
@dnltz

dnltz commented Aug 27, 2026

Copy link
Copy Markdown
Contributor Author

@Dolu1990 github action don't work anymore, but this is a bug fix for loading SpinalCrypto recursive as sub-module.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant