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

sonoff rfbridge direct hack - first commit #837

Merged
merged 5 commits into from
Dec 24, 2020

Conversation

brahmafear
Copy link
Contributor

Adds environment directive to platform.io for Sonoff RF Bridge modified with "direct hack" allowing bypass of efm8bb1 microcontroller. ( https://github.com/xoseperez/espurna/wiki/Hardware-Itead-Sonoff-RF-Bridge---Direct-Hack ).

@1technophile
Copy link
Owner

Thanks for the PR, could you isolate the change to the added environment, please.
It will make the diff more comprehensive and avoid regressions.

@1technophile 1technophile added this to the v0.9.6 milestone Dec 22, 2020
@brahmafear
Copy link
Contributor Author

Moved new environment to end of file to prevent other changes. Removed "uncomment" listing for this environment from top section. Sorry, I'm not very good with git.

@1technophile
Copy link
Owner

No problem, you need also to add
;default_envs = sonoff-rfbridge-direct
at the end of the environment list (beginning of the file)

@1technophile
Copy link
Owner

Perfect, thanks

platformio.ini Outdated
lib_deps =
${com-esp.lib_deps}
${libraries.esp8266_mdns}
puuu/ESPiLight@^0.17.0
Copy link
Owner

Choose a reason for hiding this comment

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

Could you call
${libraries.esppilight}
instead please

@1technophile
Copy link
Owner

Just one comment :-)

platformio.ini Outdated
lib_deps =
${com-esp.lib_deps}
${libraries.esppilight}
puuu/ESPiLight@^0.17.0
Copy link
Owner

Choose a reason for hiding this comment

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

You are right mdns is not necessary here.
You should also remove :

puuu/ESPiLight@^0.17.0

Since it is called by

${libraries.esppilight}

@1technophile
Copy link
Owner

Thanks !

@1technophile 1technophile merged commit 85dc3ce into 1technophile:development Dec 24, 2020
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.

2 participants