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

Can't compile nrf5 lock app since submodule update of openthread without manually mucking with the submodule source tree #1690

Closed
bzbarsky-apple opened this issue Jul 22, 2020 · 2 comments
Labels
bug Something isn't working build issue nrf
Milestone

Comments

@bzbarsky-apple
Copy link
Contributor

bzbarsky-apple commented Jul 22, 2020

Problem

When I try to follow the instructions in examples/lock-app/nrf5/README.md, whether using docker or not, I get a build error like so:

make[7]: *** No rule to make target 'backbone_router/leader.cpp', needed by 'backbone_router/libopenthread_ftd_a-leader.o'.  Stop.

I poked around a bit, and the openthread commit at openthread/openthread@daf8916 renamed that file. It changed the src/core/Makefile.am file in the openthread repo, but looking at our tree, the hird_party/openthread/repo/src/core/Makefile.in file still had the old filename in it (leader.cpp, not bbr_leader.cpp). We pulled that commit into our tree in #1583 as far as I can tell.

Rerunning scripts/build/boostrap.sh does not regenerate the Makefile.in in question.

If I manually delete third_party/openthread/repo/src/core/Makefile.in and then do the nrf5 lock-app build, things work fine...

Proposed Solution

I'm not sure. Presumably something here (either our bootstrap or the nrf5 build) should regenerate the relevant Makefile.in.

@mspang @rwalker-apple

@woody-apple woody-apple added triage bug Something isn't working build issue and removed triage labels Jul 23, 2020
@mspang
Copy link
Contributor

mspang commented Jul 24, 2020

This is a known flaw in the automake builds.

I think @andy31415 found that configuring --enable-maintainer-mode helped with this kind of problem.

@woody-apple woody-apple added this to the V1.0 milestone Jul 31, 2020
@mspang
Copy link
Contributor

mspang commented Sep 1, 2020

This is obsolete with GN

@mspang mspang closed this as completed Sep 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working build issue nrf
Projects
None yet
Development

No branches or pull requests

3 participants