'make -C roms efi' does not update edk2 submodules

Bug #1858814 reported by Philippe Mathieu-Daudé
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
QEMU
Expired
Undecided
Unassigned

Bug Description

On a fresh clone, 'make -C roms efi' fails because submodule is not initialized [1]:

/builds/philmd/qemu/roms/edk2/CryptoPkg/Library/OpensslLib/OpensslLibCrypto.inf(-1): error 000E: File/directory not found in workspace
/builds/philmd/qemu/roms/edk2/CryptoPkg/Library/OpensslLib/openssl/e_os.h
- Failed -

Laszlo suggested [2] it is possibly a regression from commit f3e330e3c319:
"roms/Makefile.edk2: don't pull in submodules when building from tarball"

[1] https://gitlab.com/philmd/qemu/-/jobs/395644357#L436
[2] https://<email address hidden>/msg668929.html

Tags: edk2
Changed in qemu:
assignee: nobody → Philippe Mathieu-Daudé (philmd)
Revision history for this message
Laszlo Ersek (Red Hat) (lersek) wrote :

Symptom persists as of v5.1.0, but I don't think it really matters. We should close the ticket as wontfix.

Changed in qemu:
status: New → Confirmed
Revision history for this message
Philippe Mathieu-Daudé (philmd) wrote :

There might be a big buildsys change in QEMU if we switch to Meson,
so I'm waiting for that to happen first and then update this ticket.

Revision history for this message
Thomas Huth (th-huth) wrote :

The QEMU project is currently considering to move its bug tracking to another system. For this we need to know which bugs are still valid and which could be closed already. Thus we are setting older bugs to "Incomplete" now.
If you still think this bug report here is valid, then please switch the state back to "New" within the next 60 days, otherwise this report will be marked as "Expired". Or mark it as "Fix Released" if the problem has been solved with a newer version of QEMU already. Thank you and sorry for the inconvenience.

Changed in qemu:
status: Confirmed → Incomplete
Changed in qemu:
assignee: Philippe Mathieu-Daudé (philmd) → nobody
Revision history for this message
Launchpad Janitor (janitor) wrote :

[Expired for QEMU because there has been no activity for 60 days.]

Changed in qemu:
status: Incomplete → Expired
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.