Skip to content

Fixed broken deps on a fresh clone #3100

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 28, 2025

Conversation

ehughes
Copy link
Contributor

@ehughes ehughes commented Apr 26, 2025

@hathach

I was preparing to do work on the nxp/mcx bsps to add more boards and noticed some of the dependencies could not be fetched on a fresh clone:

I found that the get_deps script from tinyuf2 solves the issue

Everything worked ok on my test cases (mcx, lpc55 and lpc43) once this fix was in place.

Error before the patch:

C:\ELI\tinyusb_dev\tinyusb-dev\tools> .\get_deps.py mcx
cloning lib/FreeRTOS-Kernel with https://github.com/FreeRTOS/FreeRTOS-Kernel.git
cloning tools/uf2 with https://github.com/microsoft/uf2.git
cloning hw/mcu/nxp/mcux-sdk with https://github.com/hathach/mcux-sdk.git
cloning lib/lwip with https://github.com/lwip-tcpip/lwip.git
cloning lib/CMSIS_5 with https://github.com/ARM-software/CMSIS_5.git
Command Error: git -C C:\ELI\tinyusb_dev\tinyusb-dev\hw\mcu\nxp\mcux-sdk rev-parse HEAD
fatal: ambiguous argument 'HEAD': unknown revision or path not in the working tree.
Use '--' to separate paths from revisions, like this:
'git <command> [<revision>...] -- [<file>...]'
HEAD

Command Error: git -C C:\ELI\tinyusb_dev\tinyusb-dev\tools\uf2 rev-parse HEAD
fatal: ambiguous argument 'HEAD': unknown revision or path not in the working tree.
Use '--' to separate paths from revisions, like this:
'git <command> [<revision>...] -- [<file>...]'
HEAD
Command Error: git -C C:\ELI\tinyusb_dev\tinyusb-dev\lib\FreeRTOS-Kernel rev-parse HEAD
Command Error: git -C C:\ELI\tinyusb_dev\tinyusb-dev\lib\lwip rev-parse HEAD

Command Error: git -C C:\ELI\tinyusb_dev\tinyusb-dev\lib\CMSIS_5 rev-parse HEAD
fatal: ambiguous argument 'HEAD': unknown revision or path not in the working tree.
Use '--' to separate paths from revisions, like this:
'git <command> [<revision>...] -- [<file>...]'
HEAD
fatal: ambiguous argument 'HEAD': unknown revision or path not in the working tree.
Use '--' to separate paths from revisions, like this:
'git <command> [<revision>...] -- [<file>...]'
HEAD
fatal: ambiguous argument 'HEAD': unknown revision or path not in the working tree.
Use '--' to separate paths from revisions, like this:
'git <command> [<revision>...] -- [<file>...]'
HEAD

Copy link
Owner

@hathach hathach left a comment

Choose a reason for hiding this comment

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

thank you for the fix. @ehughes the current mcu sdk is pointed to my fork, which is used when upstream does not include mcx-n mcu. Since you are working on the port, maybe take the time to make the sdk upstream again (if mcx is now supported).

@hathach hathach merged commit a22e45b into hathach:master Apr 28, 2025
105 checks passed
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.

2 participants