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

Cant compile dev build since Jan 31st #875

Closed
pharper-aus opened this issue Feb 24, 2021 · 5 comments · Fixed by #877
Closed

Cant compile dev build since Jan 31st #875

pharper-aus opened this issue Feb 24, 2021 · 5 comments · Fixed by #877
Milestone

Comments

@pharper-aus
Copy link

Since the introduction of auto discovery changes I can no longer build and receive the following error

ZgatewayBT:615:13: error: 'disc' was not declared in this scope
if (disc)
^

I think disc is in scope in main but not in this module.

@1technophile
Copy link
Owner

the continuous integration build, so there may be something depending on the environment of the build.

Could you share your environment parameters, Arduino IDE/platformio with respectively user_config.h or .ini file. Don't forget to remove your credentials.

@pharper-aus
Copy link
Author

ok I will do that shortly,

I simply did a git pull of latest dev , opened main.ini in arduino IDE then uncommented this line

//#define ZgatewayBT "BT" //ESP8266, ESP32

by removing the // then choosing compile and I get the error

by commenting the line

//if (disc)

the code will compile but obviously discovery happens every time.

*also do you have a guide to setting up the project in platform IO, it looks like a great environment and I work in VSCode everyday now so any help would be great.

@1technophile 1technophile added this to the v0.9.6 milestone Feb 26, 2021
@1technophile
Copy link
Owner

Thanks for the info, I will take a look.

also do you have a guide to setting up the project in platform IO, it looks like a great environment and I work in VSCode everyday now so any help would be great.

I began with this one:
https://www.youtube.com/watch?v=0poh_2rBq7E

@1technophile
Copy link
Owner

Hi @pharper-aus

I have done a correction on the linked branch, could you test that it solves your problem please

@pharper-aus
Copy link
Author

Thanks , that worked like a charm , also thanks for the youtube link it was perfect

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 a pull request may close this issue.

2 participants