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

Update Installation Instructions for Linux #5004

Merged
merged 1 commit into from
May 8, 2023
Merged

Conversation

billbonney
Copy link
Member

No description provided.

@Hwurzburg
Copy link
Contributor

I dont think this is correct....all APMPlanner releases should be via the firmware server, I think.... if they are not there, then there is a problem...@peterbarker please confirm....thanks

@billbonney
Copy link
Member Author

I dont think this is correct....all APMPlanner releases should be via the firmware server, I think.... if they are not there, then there is a problem...@peterbarker please confirm....thanks

Why would that be a problem? You requested the links be updated. And this matches that request.

see ArduPilot/apm_planner#1280

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.

@Hwurzburg releasing straight-from-github is a legitimate project thing now-adays.

I reckon it's a change, so I've marked it DevCallTopic to make sure everyone's aware - but I'm pretty sure everyone will be, "Someone's doing fresh releases?! Great stuff!" and we'll want to merge this :-)

planner2/source/docs/installation-for-linux.rst Outdated Show resolved Hide resolved
@billbonney billbonney force-pushed the billbonney-patch-1 branch 2 times, most recently from a30d479 to 2762ad8 Compare March 12, 2023 09:21
@peterbarker
Copy link
Contributor

@billbonney I've dismissed my stale review. I wonder if the mention of OpenSuse repo could also go away - I guess it's probably very stale. Not trying to stretch this PR, just observing.

@billbonney
Copy link
Member Author

@billbonney I've dismissed my stale review. I wonder if the mention of OpenSuse repo could also go away - I guess it's probably very stale. Not trying to stretch this PR, just observing.

Yep, we can remove that. Since AppImages are the 'run anywhere' solution

@Hwurzburg Hwurzburg merged commit d35a35f into master May 8, 2023
@Hwurzburg Hwurzburg deleted the billbonney-patch-1 branch May 8, 2023 20:07
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.

4 participants