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.

Sign up for GitHub

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

Jump to bottom

SITL: add missing includes #25673

Merged
merged 1 commit into from
Dec 3, 2023
Merged

SITL: add missing includes #25673

merged 1 commit into from
Dec 3, 2023

Conversation

peterbarker
Copy link
Contributor

fixes SITL-on-HW builds

@Ryanf55
Copy link
Collaborator

Ryanf55 commented Dec 1, 2023

What command did you run to get these to be problematic on master, and is there any way we can protect against it in the future?

@khancyr
Copy link
Contributor

khancyr commented Dec 1, 2023

we can probably add a CI compile test for SITL-HW

@peterbarker peterbarker merged commit 0d9c658 into ArduPilot:master Dec 3, 2023
84 of 87 checks passed
@peterbarker peterbarker deleted the pr/fixes branch December 3, 2023 20:34
@rmackay9 rmackay9 mentioned this pull request Dec 4, 2023
Closed
@itayguy
Copy link

itayguy commented Jan 30, 2024

HI , I tried compiling with :
./libraries/SITL/examples/on-hardware/sitl-on-hw.py on plane-4.4.4.
but it hangs on the configure subprocess and doesn't continue.
I compile with cygwin on windows.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
MergeOnCIPass
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@peterbarker @Ryanf55 @khancyr @itayguy