Skip to content
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

waf: move chibios hwdef generation to explicit call #27991

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

joshanne
Copy link
Contributor

@joshanne joshanne commented Sep 3, 2024

This change would explicitly call the chibios_hwdef.ChibiOSHWDef run function - instead of palming it off to a subprocess.call

This replaces the work from #27955 which builds upon #27988

@joshanne joshanne marked this pull request as draft September 3, 2024 07:14
Tools/ardupilotwaf/chibios.py Show resolved Hide resolved
Tools/ardupilotwaf/chibios.py Show resolved Hide resolved
@peterbarker
Copy link
Contributor

I do like this.

This is possible because chibiois_hwdef.py was factored to create that object - relatively recently.

@joshanne
Copy link
Contributor Author

joshanne commented Sep 3, 2024

I do like this.

Excellent, then i'll move this from draft to PR.

@joshanne joshanne force-pushed the pr/move-chibios-hwdef-generation-to-explicit-call branch from c870c0a to a63fafc Compare September 3, 2024 11:27
@joshanne joshanne marked this pull request as ready for review September 3, 2024 11:27
@joshanne joshanne force-pushed the pr/move-chibios-hwdef-generation-to-explicit-call branch from e600168 to 0714abe Compare September 4, 2024 00:42
@joshanne joshanne force-pushed the pr/move-chibios-hwdef-generation-to-explicit-call branch from 0714abe to 0a453bf Compare September 16, 2024 20:41
If  is not provided at build time, then AP_SIGNED_FIRMWARE results in empty list.
If  is provided at build time, then AP_SIGNED_FIRMWARE results in .
@joshanne joshanne force-pushed the pr/move-chibios-hwdef-generation-to-explicit-call branch from 0a453bf to b002b6b Compare October 31, 2024 06:23
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