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

rename silvercrest and nexus to mitigate codebase conflict with ESPiL… #157

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

peshovec
Copy link

Hit the problem with codebase conflict, then saw the remark in the README.
With this change openmqttgateway compiles without warning. However i am not confident the changes are functional ok. What is your opinion?

@NorthernMan54
Copy link
Owner

Tks for the effort, but as the files are a direct copy from rtl_433_ESP, I don't want to make changes to them, that will be overwritten but the next port.

@1technophile
Copy link

The changes could be proposed upstream in rtl_433 repository.

@peshovec
Copy link
Author

Thanks for the feedback to both of you.
Prepared a draft for a potential PR to rtl_433. Please take a look at it, as i am not confident in the proposed changes. Also right now i don't know how to start some testing. from the rtl_433 PR - to rtl_433_esp - to openmqttgateway.
Direct link to the commit with the mentioned potential PR for rtl_433 : peshovec/rtl_433@4554f19

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.

3 participants