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 build today. Some issues with submodules. #12

Open
dalao3 opened this issue Mar 12, 2020 · 6 comments
Open

Can't build today. Some issues with submodules. #12

dalao3 opened this issue Mar 12, 2020 · 6 comments

Comments

@dalao3
Copy link

dalao3 commented Mar 12, 2020

I'm following this guide (https://blog.conor-burns.com/lenovo-t440p-coreboot/) but I get some errors below. It appears there are issues with submodules.

Skipping submodule '3rdparty/amd_blobs'
Skipping submodule '3rdparty/blobs'
Skipping submodule '3rdparty/fsp'
Skipping submodule '3rdparty/amd_blobs'
Skipping submodule '3rdparty/blobs'
Skipping submodule '3rdparty/fsp'
    HOSTCC     util/sconfig/main.o
    HOSTCC     util/sconfig/sconfig (link)
    SCONFIG    mainboard/lenovo/t440p/devicetree.cb
    OPTION     option_table.h
    CC         bootblock/mainboard/lenovo/t440p/static.o
cc1: error: 3rdparty/vboot/firmware/include: Not a directory
cc1: error: 3rdparty/vboot/firmware/2lib/include: Not a directory
cc1: error: 3rdparty/vboot/firmware/lib21/include: Not a directory
cc1: error: unrecognized command line option '-Wno-address-of-packed-member' [-Werror]
cc1: all warnings being treated as errors
make: *** [Makefile:365: build/bootblock/mainboard/lenovo/t440p/static.o] Error 1

If I copy the 3rdparty folder from coreboot's latest repo, I get the follow error. How to get a snapshot of the submodules when it was working?

Failed to clone 'util/nvidia/cbootimage' a second time, aborting
    CC         bootblock/lib/bootmode.o
In file included from src/security/vboot/misc.h:21,
                 from src/lib/bootmode.c:18:
src/security/vboot/vboot_common.h:21:10: fatal error: vboot_struct.h: No such file or directory
 #include <vboot_struct.h>
          ^~~~~~~~~~~~~~~~
compilation terminated.
make: *** [Makefile:365: build/bootblock/lib/bootmode.o] Error 1
@KratosPS
Copy link

I get the same error when I attempt to run make. Instead of replacing the entire 3rdparty folder i just replaced the vboot folder. Everything appears to run correctly up until vboot_struct.h can't be found. Tried running the 4.11_branch but don't have permissions to download the submodules. Tried to edit the .gitmodules file pointing to the same directories as t440p-main with vboot added and it appears to not download anything into vboot still.

@elkaps
Copy link

elkaps commented Mar 31, 2020

I managed to run make by using vboot from the following branch:
https://github.com/coreboot/coreboot/tree/4.11_branch
For some reason I got the error
make: *** No rule to make target '".car.data"', needed by 'build/coreboot.pre'. Stop.
which was resolved after copying the Makefile.inc as well.

@KratosPS
Copy link

I managed to run make by using vboot from the following branch:
https://github.com/coreboot/coreboot/tree/4.11_branch
For some reason I got the error
make: *** No rule to make target '".car.data"', needed by 'build/coreboot.pre'. Stop.
which was resolved after copying the Makefile.inc as well.

This worked for me also. Thanks for the help elkaps, I was using the official build of coreboot which is still missing fixes for the touchpad buttons and other minor things.

@sudo-gedit
Copy link

Can you please provide me with your configuration. unfortunately I can't do it. Thank you

@acaleechurn
Copy link

Did someone manage to find out why the build is failing? I wanted to use this repo as it had the latest patches [which may not be true anymore] but ended up using the latest official release. Unfortunately my test clip broke and I could not test the generated ROM but as I'm getting new ones on Friday I wanted to give it a try again.

@acaleechurn
Copy link

If anyone is looking at Corebooting their T440p, the official repo now has the necessary patches to make everything work except for audio. Mine works well with tianocore compiled from the mainstream repo.

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

No branches or pull requests

5 participants