[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/5] nsis: don't install files into /tmp
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH 1/5] nsis: don't install files into /tmp |
Date: |
Tue, 29 Jan 2019 15:07:24 -0600 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.4.0 |
On 1/29/19 5:39 AM, Daniel P. Berrangé wrote:
> The nsis installer target has to run 'make install' to populate a
> directory tree with content for the package. Replace the current
> usage of '/tmp/qemu-nsis' with a location underneath the build
> directory. This ensures that when a developer cleans their build
> directory, any left over files from the installer build process
> are also purged.
>
> Signed-off-by: Daniel P. Berrangé <address@hidden>
> ---
> Makefile | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
Should "qemu-nsis-vroot/" be added to .gitignore for the sake of in-tree
builds?
> diff --git a/Makefile b/Makefile
> index de898eab62..362a98d275 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -907,7 +907,7 @@ endif
> .PHONY: installer
> installer: $(INSTALLER)
>
> -INSTDIR=/tmp/qemu-nsis
> +INSTDIR=$(BUILD_DIR)/qemu-nsis-vroot
>
> $(INSTALLER): $(SRC_PATH)/qemu.nsi
> $(MAKE) install prefix=${INSTDIR}
>
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3226
Virtualization: qemu.org | libvirt.org
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PATCH 0/5] misc fixes to deal with icon location changes, Daniel P . Berrangé, 2019/01/29
- [Qemu-devel] [PATCH 5/5] configure: stop trying to link non-existant bmp file from bios dir, Daniel P . Berrangé, 2019/01/29
- [Qemu-devel] [PATCH 2/5] make: don't insert a '/' after $(DESTDIR), Daniel P . Berrangé, 2019/01/29
- [Qemu-devel] [PATCH 1/5] nsis: don't install files into /tmp, Daniel P . Berrangé, 2019/01/29
- Re: [Qemu-devel] [PATCH 1/5] nsis: don't install files into /tmp,
Eric Blake <=
- [Qemu-devel] [PATCH 3/5] nsis: ensure we always pass -W64 flag to nsis installer script, Daniel P . Berrangé, 2019/01/29
- [Qemu-devel] [PATCH 4/5] nsis: fix location of icons to be bundled in the installer, Daniel P . Berrangé, 2019/01/29