Skip to content

Conversation

klutvott123
Copy link
Member

Added instructions for background script.
Removed 4.1.0 requirement.
Merged "How to download the scripts" into "Installing" section.
some minor changes.

Added instructions for background script.
Removed 4.1.0 requirement.
Merged "How to download the scripts" into "Installing" section.
some minor changes.
@sonarqubecloud
Copy link

sonarqubecloud bot commented May 9, 2020

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@jforhan
Copy link

jforhan commented May 10, 2020

Added instructions for background script.
Removed 4.1.0 requirement.
Merged "How to download the scripts" into "Installing" section.
some minor changes.

I see the changes add the info about not including the "OBJ" directory, but...

Looking at the latest 1.4.1 release ZIP directory structure, your bf.lua/bf.luac files will never end up in /SCRIPTS/TOOLS as those files are in a directory named "BF" under the root "OBJ" directory. The other directories will merge when you paste but you end up with a "BF" folder in the root of your SD card. Either the instructions should be changed or there should be additional steps added to copy these two files to the /SCRIPTS/TOOLS directory...Unless the directory structure will be changing in the next release?

Thanks
Jeff

@klutvott123
Copy link
Member Author

@jforhan These instructions are for 1.5.0. In 1.5 bf.lua will be in /SCRIPTS/TOOLS. The BF folder has been repurposed for storing vtx tables.

@jforhan
Copy link

jforhan commented May 10, 2020

@jforhan These instructions are for 1.5.0. In 1.5 bf.lua will be in /SCRIPTS/TOOLS. The BF folder has been repurposed for storing vtx tables.

Thanks for the info. I am guessing the existing instructions are not going to be updated as the 1.5 will be out soon enough?

Thanks
Jeff

@mikeller mikeller merged commit 6a97e05 into betaflight:master May 12, 2020
@klutvott123 klutvott123 deleted the update-readme branch October 24, 2021 19:45
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.

3 participants