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 README.md #4256

Merged
merged 2 commits into from
Nov 22, 2024
Merged

Update README.md #4256

merged 2 commits into from
Nov 22, 2024

Conversation

GitHubEmploy
Copy link
Contributor

Adding instruction to download the installer file for the linux distro, in case it was not obvious before

Requested here:
https://discord.com/channels/868013470023548938/868013470518505494/1309394857181315142
https://discord.com/channels/868013470023548938/868013470518505494/1309395939320135710

Copy link

netlify bot commented Nov 22, 2024

Deploy Preview for origin-betaflight-app ready!

Name Link
🔨 Latest commit 1e9ec36
🔍 Latest deploy log https://app.netlify.com/sites/origin-betaflight-app/deploys/6740ad686252690008eb6889
😎 Deploy Preview https://deploy-preview-4256.dev.app.betaflight.com
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@thespova
Copy link

This still isn't 100% clear to a newbie that they need to also install that.
Just adding something like "sudo apt install betaflight-configurator_10.10.0_amd64.deb" after the download would be nice. Unless that should not be done until after the other dependencies.

I think

wget https://github.com/betaflight/betaflight-configurator/releases/download/10.10.0/betaflight-configurator_10.10.0_amd64.deb
sudo apt install betaflight-configurator_10.10.0_amd64.deb

@GitHubEmploy
Copy link
Contributor Author

This still isn't 100% clear to a newbie that they need to also install that. Just adding something like "sudo apt install betaflight-configurator_10.10.0_amd64.deb" after the download would be nice. Unless that should not be done until after the other dependencies.

I think

wget https://github.com/betaflight/betaflight-configurator/releases/download/10.10.0/betaflight-configurator_10.10.0_amd64.deb
sudo apt install betaflight-configurator_10.10.0_amd64.deb

Hello,

Not adding that step right after was extremely intentional, as the install will fail on ubuntu 24.10 if you do these directly as shown in the screenshot below:
image

The intent is that they would first download that file, and the continue with the steps needed for their distro.

@thespova
Copy link

Ah got it.

@thespova
Copy link

thespova commented Nov 22, 2024

Maybe both those steps should be added after the dependencies then? Or maybe its fine the way it is. IDK thanks.

@nerdCopter
Copy link
Member

This is no longer relevant as there will be no further binary releases.
From 11.0, there will only be https://app.betaflight.com/, https://master.dev.app.betaflight.com/, and maybe version specific URL's.

@nerdCopter nerdCopter closed this Nov 22, 2024
@nerdCopter
Copy link
Member

nerdCopter commented Nov 22, 2024

i'll redact my closing since the how-to was recently added. but really this makes no sense, since this readme is attached to current master at 11.0, not legacy 10.10.

@nerdCopter nerdCopter reopened this Nov 22, 2024
README.md Outdated Show resolved Hide resolved
Copy link

sonarcloud bot commented Nov 22, 2024

@haslinghuis haslinghuis merged commit a57299d into betaflight:master Nov 22, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants