Skip to content

Fix #341. #342

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

Merged
merged 1 commit into from
Sep 12, 2018
Merged

Fix #341. #342

merged 1 commit into from
Sep 12, 2018

Conversation

me21
Copy link
Contributor

@me21 me21 commented Dec 1, 2017

No description provided.

@couchninja
Copy link

Hey hey! Not sure what the status of this pull request is, and if this is a idiosyncratic way of fixing it, but it fixed my problem with building in platformio on a wemos mini d1 pro. So thanks @me21, and it would be great if it can be merged!

@me21
Copy link
Contributor Author

me21 commented Mar 4, 2018

@couchninja Looks like this patch will break other, non-ESP implementations. It will likely not be merged and some other patch will need to be prepared.

@piratebriggs
Copy link

Hey @me21 the fix you posted in the issue works for me on the esp platform and as you've added an ifdef it looks like a good fix. Is there a reason you haven't created a PR for it? I'm happy to submit it and reference your fix.

@me21
Copy link
Contributor Author

me21 commented Sep 12, 2018

@piratebriggs I've forgotten already what ifdef I've added, but please go ahead with PR :)

Signed-off-by: Alexandr Zarubkin <[email protected]>
@me21
Copy link
Contributor Author

me21 commented Sep 12, 2018

@jrowberg @piratebriggs I've updated my PR, it builds for me, can you take a look please?

@jrowberg jrowberg merged commit c20f68e into jrowberg:master Sep 12, 2018
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.

4 participants