8000 V1.20.0.rc7: Build for Fipy fails with undefined reference · Issue #263 · pycom/pycom-micropython-sigfox · GitHub
[go: up one dir, main page]
Skip to content
8000
This repository was archived by the owner on Sep 16, 2024. It is now read-only.

V1.20.0.rc7: Build for Fipy fails with undefined reference #263

Closed
robert-hh opened this issue Feb 9, 2019 · 4 comments
Closed

V1.20.0.rc7: Build for Fipy fails with undefined reference #263

robert-hh opened this issue Feb 9, 2019 · 4 comments

Comments

@robert-hh
Copy link
Contributor
robert-hh commented Feb 9, 2019 •
edited
Loading

Fails with OPENTHREAD = off. Error message:

LINK build/FIPY/release/application.elf
build/FIPY/release/application.a(modlte.o):(.literal.TASK_LTE_UPGRADE+0xc): undefined reference to `xMeshTaskHndl'

xMeshTaskHndl is not defined in the sources and not in esp-idf, branch idf_v3.1. Updating esp-idf does not change the picture.
Building for LoPy4, LOPY, GPY and WiPy works.
@iwahdan88
Copy link
Contributor

Hi @robert-hh , thanks for reporting this, you are correct
in modlte.c
line 319 should be conditioned like this

#ifdef LORA_OPENTHREAD_ENABLED
    vTaskSuspend(xMeshTaskHndl);
#endif

we will update it asap

@robert-hh
Copy link
Contributor Author

Tested & it works

@Xykon
Copy link
Member
Xykon commented Feb 9, 2019

I also created a PR for this #264

@robert-hh
Copy link
Contributor Author
robert-hh commented Feb 11, 2019 •
edited
Loading

Thanks. But If I was the only one noticing that as a problem, then a separate branch would not have to be used. I made that change anyhow already before posting the issue. Now there is another branch in the repository, which will be abandoned soon.
At least, GIT makes it easy to deal with such variants.

peter-pycom pushed a commit that referenced this issue Oct 22, 2021
@VladPetrovici @peter-pycom
Fix for the missing certificated in WPA2 Enterprise connection (#263)
68c0063 
* Fix for the missing certificated in WPA2 Enterprise connection

* Added verification check for the certificate content, updated comment

Co-authored-by: VladP <vlad.XT@pycom.io>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@Xykon @iwahdan88 @robert-hh
0