Skip to content

docs: build note for funasr-common (A1 follow-up)#2996

Merged
LauraGPT merged 1 commit into
mainfrom
docs/build-note
Jun 20, 2026
Merged

docs: build note for funasr-common (A1 follow-up)#2996
LauraGPT merged 1 commit into
mainfrom
docs/build-note

Conversation

@LauraGPT

Copy link
Copy Markdown
Collaborator

Adds the missing build step: copy funasr-common/ (shared miniaudio audio loader) into examples/ alongside the example dir; each example's CMake includes ../funasr-common. Closes the A1 follow-up requested in review.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request updates the README.md for llama.cpp runtime by adding a step to copy the shared audio loader (funasr-common) to the examples/ directory. The feedback suggests moving the inline bash comment to its own line above the command to prevent copy-paste issues in different terminal environments.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

## Build (shared)
```bash
git clone https://github.com/ggml-org/llama.cpp && cd llama.cpp
cp -r /path/to/runtime/llama.cpp/funasr-common examples/ # shared audio loader (miniaudio); each example CMake adds ../funasr-common

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

Placing comments inline within a bash code block can sometimes cause issues when users copy and paste the entire block directly into their terminal (especially across different shells or terminal emulators). Moving the comment to its own line above the command improves readability and copy-paste safety.

Suggested change
cp -r /path/to/runtime/llama.cpp/funasr-common examples/ # shared audio loader (miniaudio); each example CMake adds ../funasr-common
# Copy the shared audio loader (miniaudio); each example's CMake adds ../funasr-common
cp -r /path/to/runtime/llama.cpp/funasr-common examples/

@LauraGPT LauraGPT merged commit 4a93f29 into main Jun 20, 2026
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