Add Arch Linux Milk-V Duo S test report - #394
Conversation
Reviewer's GuideUpdates the Milk-V Duo S Arch Linux report in English and Chinese for a newer rootfs/image test run, refreshing build commands, boot logs, metadata, and recording evidence. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Hey - I've found 2 issues
Prompt for AI Agents
Please address the comments from this code review:
## Individual Comments
### Comment 1
<location path="Duo_S/ArchLinux/README.md" line_range="7" />
<code_context>
status: basic
-last_update: 2025-08-31
+last_update: 2025-09-06
---
</code_context>
<issue_to_address>
**issue:** The report is dated 2025-09-06 but all updated rootfs references point to the future version `archriscv-2026-08-27`; when users follow the report at its stated update date, the `wget` URL requests an unavailable archive and the documented tested version cannot be reproduced.
**Triggers:** When following this report before 2026-08-27.
**Suggested fix:** Use the actual tested 2025 rootfs/build dates consistently, or update `last_update` and the report only when the 2026 archive actually exists.
</issue_to_address>
### Comment 2
<location path="Duo_S/ArchLinux/README.md" line_range="130" />
<code_context>
```shell
-dd if=milkv-duos-sd_2025-0831-0547.img of=/dev/sdc bs=1M status=progress
+dd if=milkv-duos-sd_2026-0906-2125.img of=/dev/sdc bs=1M status=progress
```
</code_context>
<issue_to_address>
**issue:** The English flashing command looks for `milkv-duos-sd_2026-0906-2125.img` in the current directory, while the build and loop-setup instructions produce and reference it under `out/`; following the documented steps therefore makes `dd` fail with a missing input file unless the user manually changes directories or adds the prefix.
**Triggers:** When the command is run from the repository root or from the `/mnt/duo-rootfs` directory left by the preceding steps.
**Suggested fix:** Use `dd if=out/milkv-duos-sd_2026-0906-2125.img ...`, matching the Chinese instructions and the build output path.
```suggestion
dd if=out/milkv-duos-sd_2026-0906-2125.img of=/dev/sdc bs=1M status=progress
```
</issue_to_address>Sourcery assessment
Approval pending. 2 findings to address first.
Blocking findings: Duo_S/ArchLinux/README.md:7, Duo_S/ArchLinux/README.md:130
f2f51e9 to
82c58a3
Compare
Sourcery withdrew this approval because it has stopped reviewing this pull request.
|
Sourcery has withdrawn its approval of this pull request. It auto-reviews a pull request 5 times, and this push is past that limit, so the approval no longer reflects code Sourcery has read. Comment |
|
感谢更新测试报告~目前还有以下问题需要调整:
以上修改完成并确认步骤可复现后即可合入,感谢。 |
acb6c1c to
9546241
Compare
Signed-off-by: ZihanCheng63 <3886855436@qq.com>
Welcome to Pull Request
Thank you for your contribution! Please refer to the contributing guidelines for more details.
If you are working on the support-matrix report
We appreciate your effort in improving the support matrix. To generate the SVG image locally, you can use:
For internationalization (i18n) SVG generation test, simply add the
-l zhoption.If you're creating a new report, please refer to our templates:
Checklist
If you are working on the assets toolchain
Fixes #<issue_number_goes_here>
Checklist
Summary by Sourcery
Refresh the Milk-V Duo S Arch Linux support report with results from the latest tested image.
Enhancements:
Documentation:
Tests: