diff mbox

[2/2] build: add pc-bios to config-host.mak deps

Message ID 1467372867-473-2-git-send-email-kraxel@redhat.com
State New
Headers show

Commit Message

Gerd Hoffmann July 1, 2016, 11:34 a.m. UTC
... so configure re-runs on pc-bios updates such as new pxe roms.
Needed because configure symlinks the prebuilt roms from src
into build tree.

Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
---
 Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Peter Maydell July 4, 2016, 12:36 p.m. UTC | #1
On 1 July 2016 at 12:34, Gerd Hoffmann <kraxel@redhat.com> wrote:
> ... so configure re-runs on pc-bios updates such as new pxe roms.
> Needed because configure symlinks the prebuilt roms from src
> into build tree.
>
> Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
> ---
>  Makefile | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Makefile b/Makefile
> index 5ea13bc..c1ac21d 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -30,7 +30,7 @@ CONFIG_ALL=y
>  -include config-all-devices.mak
>  -include config-all-disas.mak
>
> -config-host.mak: $(SRC_PATH)/configure
> +config-host.mak: $(SRC_PATH)/configure $(SRC_PATH)/pc-bios
>         @echo $@ is out-of-date, running configure
>         @# TODO: The next lines include code which supports a smooth
>         @# transition from old configurations without config.status.
> --
> 1.8.3.1

This is ok as a quick fix for the moment, but I think longer term
the right way to fix this problem is to stop doing build-time
stuff (copying files from the src to the build dir) in configure,
and do it in make instead.

thanks
-- PMM
diff mbox

Patch

diff --git a/Makefile b/Makefile
index 5ea13bc..c1ac21d 100644
--- a/Makefile
+++ b/Makefile
@@ -30,7 +30,7 @@  CONFIG_ALL=y
 -include config-all-devices.mak
 -include config-all-disas.mak
 
-config-host.mak: $(SRC_PATH)/configure
+config-host.mak: $(SRC_PATH)/configure $(SRC_PATH)/pc-bios
 	@echo $@ is out-of-date, running configure
 	@# TODO: The next lines include code which supports a smooth
 	@# transition from old configurations without config.status.