Skip to content

fix(wasi-nn): disable shallow clone for BitNet dependency#4638

Merged
hydai merged 2 commits intoWasmEdge:masterfrom
Sujanian1304:fix/wasinn-bitnet-fetch
Feb 10, 2026
Merged

fix(wasi-nn): disable shallow clone for BitNet dependency#4638
hydai merged 2 commits intoWasmEdge:masterfrom
Sujanian1304:fix/wasinn-bitnet-fetch

Conversation

@Sujanian1304
Copy link
Copy Markdown
Contributor

Description

This PR fixes a build failure in the WASI-NN BitNet backend caused by the FetchContent configuration.

The Issue

The build was failing with the error:
fatal: invalid reference: 404980e

This occurred because cmake/WASINNDeps.cmake combined GIT_SHALLOW TRUE (depth 1) with a specific, older commit hash (404980e). Since the upstream microsoft/BitNet repository has moved forward significantly (the HEAD is now several months ahead of 404980e), the required commit is no longer present in a shallow clone of the default branch.

The Fix

I have disabled the shallow clone for the BitNet dependency (GIT_SHALLOW FALSE). This ensures the full history is fetched so that the specific commit 404980e can be successfully checked out during the build configuration.

Related Issues

Fixes #4635

@github-project-automation github-project-automation bot moved this to Triage-required in WasmEdge Roadmap Feb 10, 2026
@github-actions github-actions bot added c-Test An issue/PR to enhance the test suite c-CI An issue related to CI c-CMake An issue related to CMake mechanism or options labels Feb 10, 2026
@Sujanian1304 Sujanian1304 force-pushed the fix/wasinn-bitnet-fetch branch from 70ab8fb to b540226 Compare February 10, 2026 09:23
Signed-off-by: Piyush kumar <piyush1304kumar@gmail.com>
@Sujanian1304 Sujanian1304 force-pushed the fix/wasinn-bitnet-fetch branch from b540226 to ce73498 Compare February 10, 2026 09:25
@github-actions github-actions bot removed c-Test An issue/PR to enhance the test suite c-CI An issue related to CI labels Feb 10, 2026
@codecov
Copy link
Copy Markdown

codecov bot commented Feb 10, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.98%. Comparing base (273ac6e) to head (c7328d3).
⚠️ Report is 3 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #4638   +/-   ##
=======================================
  Coverage   80.97%   80.98%           
=======================================
  Files         300      300           
  Lines       40633    40632    -1     
  Branches     6720     6719    -1     
=======================================
+ Hits        32901    32904    +3     
+ Misses       5930     5928    -2     
+ Partials     1802     1800    -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Copy Markdown
Member

@hydai hydai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@hydai hydai merged commit 3c0932b into WasmEdge:master Feb 10, 2026
102 of 107 checks passed
@github-project-automation github-project-automation bot moved this from Triage-required to Done in WasmEdge Roadmap Feb 10, 2026
@Sujanian1304
Copy link
Copy Markdown
Contributor Author

@hydai ,thanks for merging
Can you take a look at these PR's as well
#4542
#4625

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

Labels

c-CMake An issue related to CMake mechanism or options

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

bug: WASI-NN Bitnet CI no longer works

2 participants