Skip to content

Conversation

juhaylinen
Copy link
Contributor

Description

Fixes below compilation error:

Compile [  1.3%]: ESP8266Interface.cpp
[Error] ESP8266Interface.cpp@582,33: cannot declare static function inside another function
[Error] ESP8266Interface.cpp@583,13: cannot convert 'ESP8266Interface (*)()' to 'WiFiInterface*' in return
Failed to build library

Pull request type

[X] Fix
[ ] Refactor
[ ] Target update
[ ] Functionality change
[ ] Breaking change

Compile [  1.3%]: ESP8266Interface.cpp
[Error] ESP8266Interface.cpp@582,33: cannot declare static function inside another function
[Error] ESP8266Interface.cpp@583,13: cannot convert 'ESP8266Interface (*)()' to 'WiFiInterface*' in return
Failed to build library
@0xc0170
Copy link
Contributor

0xc0170 commented Sep 3, 2018

@SeppoTakalo this might be a candidate for 5.10 rc2 ?

@teetak01
Copy link
Contributor

teetak01 commented Sep 3, 2018

This looks like something that should be part of 5.10-RC1.

@JanneKiiskila

@JanneKiiskila
Copy link
Contributor

Yep, needs to be fixed.
@SeppoTakalo

@0xc0170
Copy link
Contributor

0xc0170 commented Sep 3, 2018

/morph build

@mbed-ci
Copy link

mbed-ci commented Sep 3, 2018

Build : SUCCESS

Build number : 2996
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/7967/

Triggering tests

/morph test
/morph export-build
/morph mbed2-build

@mbed-ci
Copy link

mbed-ci commented Sep 3, 2018

@mbed-ci
Copy link

mbed-ci commented Sep 3, 2018

@SeppoTakalo
Copy link
Contributor

Error: C9555E: Failed to check out a license.
All suitable licenses are already in use.

@0xc0170
Copy link
Contributor

0xc0170 commented Sep 3, 2018

/morph export-build

@mbed-ci
Copy link

mbed-ci commented Sep 4, 2018

@cmonr
Copy link
Contributor

cmonr commented Sep 4, 2018

/morph test

@mbed-ci
Copy link

mbed-ci commented Sep 4, 2018

@0xc0170 0xc0170 merged commit eb3cbb8 into ARMmbed:master Sep 4, 2018
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.

8 participants