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

Tools: ardupilotwaf: add entry for 64bit versions of Navigator #28239

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

Tools: ardupilotwaf: add entry for 64bit versions of Navigator #28239

wants to merge 2 commits into from
+24 −0

Conversation

Williangalvani
Copy link
Contributor

No description provided.

@Williangalvani Williangalvani mentioned this pull request Sep 26, 2024
Open
peterbarker
peterbarker requested changes Sep 26, 2024
View reviewed changes
Copy link
Contributor

@peterbarker peterbarker left a comment

Choose a reason for hiding this comment

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

Failed to build the navigator64 target:

Could not find the program ['aarch64-none-linux-gnu-ar']

@Williangalvani Williangalvani force-pushed the navigator_64 branch 2 times, most recently from 075b27d to 03c0d76 Compare September 30, 2024 14:39
@Williangalvani Williangalvani mentioned this pull request Sep 30, 2024
Draft
@tridge
Copy link
Contributor

tridge commented Oct 1, 2024

needs to be added to board_list.py

peterbarker
peterbarker requested changes Oct 1, 2024
View reviewed changes
Copy link
Contributor

@peterbarker peterbarker left a comment

Choose a reason for hiding this comment

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

Needs patch next to existing "navigator" board in board_list.py

@tridge
Copy link
Contributor

tridge commented Oct 1, 2024

I ran this on firmware.ardupilot.org:
apt install g++-10-aarch64-linux-gnu

@peterbarker peterbarker self-assigned this Oct 1, 2024
tridge
tridge requested changes Oct 1, 2024
View reviewed changes
Copy link
Contributor

@tridge tridge left a comment

Choose a reason for hiding this comment

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

please remove the COLCON_IGNORE change

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tridge tridge tridge requested changes

@peterbarker peterbarker peterbarker requested changes

Assignees

@peterbarker peterbarker

Labels
MergeOnCIPass
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Williangalvani @tridge @peterbarker @CraigElder