Fix README_JP6.2.md - #358
Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates the installation instructions in the Japanese README for Jetpack 6.2, specifically improving the module installation and initramfs update procedures.
Changes:
- Modified kernel module installation to completely replace
/lib/modulesinstead of copying a specific version - Added initramfs update commands and symbolic link creation for the boot process
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| sudo rm -r /lib/modules | ||
| sudo cp -r ./lib/modules/ /lib/ |
There was a problem hiding this comment.
Removing the entire /lib/modules directory without confirmation is risky and could break the system if the copy operation fails. Consider using sudo rm -r /lib/modules/5.15.148-tegra to remove only the specific module version being replaced, or add the -f flag and proper error handling.
| sudo rm -r /lib/modules | |
| sudo cp -r ./lib/modules/ /lib/ | |
| sudo rm -rf /lib/modules/5.15.148-tegra | |
| sudo cp -r ./lib/modules/5.15.148-tegra /lib/modules/. |
| tar xf rootfs.tar.gz | ||
| sudo cp -r ./lib/modules/5.15.148-tegra /lib/modules/. | ||
| sudo rm -r /lib/modules | ||
| sudo cp -r ./lib/modules/ /lib/ |
There was a problem hiding this comment.
The trailing slash in ./lib/modules/ may cause inconsistent behavior across different shell environments. For clarity and consistency with line 136, use ./lib/modules without the trailing slash.
| sudo cp -r ./lib/modules/ /lib/ | |
| sudo cp -r ./lib/modules /lib/ |
No description provided.