[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 5/6] meson: install $localstatedir/run for qga
From: |
Paolo Bonzini |
Subject: |
Re: [PATCH 5/6] meson: install $localstatedir/run for qga |
Date: |
Sun, 30 Aug 2020 11:44:50 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.9.0 |
On 26/08/20 15:06, marcandre.lureau@redhat.com wrote:
> diff --git a/qga/meson.build b/qga/meson.build
> index 3f28f74b52..e5c5778a3e 100644
> --- a/qga/meson.build
> +++ b/qga/meson.build
> @@ -82,6 +82,8 @@ if targetos == 'windows'
> all_qga += [qga_msi]
> alias_target('msi', qga_msi)
> endif
> +else
> + install_subdir('run', install_dir: get_option('localstatedir'))
> endif
This doesn't work if /var/run exists and is a symlink:
Installing subdir /home/pbonzini/ff/run to
/home/pbonzini/ff/build/test/var/local/run
Traceback (most recent call last):
File "/usr/lib/python3.8/site-packages/mesonbuild/mesonmain.py", line 131, in
run
return options.run_func(options)
File "/usr/lib/python3.8/site-packages/mesonbuild/minstall.py", line 538, in
run
installer.do_install(datafilename)
File "/usr/lib/python3.8/site-packages/mesonbuild/minstall.py", line 357, in
do_install
self.install_subdirs(d) # Must be first, because it needs to delete the old
subtree.
File "/usr/lib/python3.8/site-packages/mesonbuild/minstall.py", line 383, in
install_subdirs
d.dirmaker.makedirs(full_dst_dir, exist_ok=True)
File "/usr/lib/python3.8/site-packages/mesonbuild/minstall.py", line 60, in
makedirs
os.makedirs(path, exist_ok=exist_ok)
File "/usr/lib64/python3.8/os.py", line 223, in makedirs
mkdir(name, mode)
FileExistsError: [Errno 17] File exists:
'/home/pbonzini/ff/build/test/var/local/run'
Since DESTDIR is not available, there isn't really a good way to do
this check. We can:
- keep the makefile
- use an install script
- create it in qemu-ga just like on win32
Probably the last one is the best.
Paolo
- [PATCH 0/6] meson: convert the remaining of install target, marcandre . lureau, 2020/08/26
- [PATCH 1/6] meson: install pc-bios blobs, marcandre . lureau, 2020/08/26
- [PATCH 2/6] meson: install scripts/qemu-trace-stap, marcandre . lureau, 2020/08/26
- [PATCH 3/6] meson: install icons, marcandre . lureau, 2020/08/26
- [PATCH 4/6] meson: install desktop file, marcandre . lureau, 2020/08/26
- [PATCH 5/6] meson: install $localstatedir/run for qga, marcandre . lureau, 2020/08/26
- Re: [PATCH 5/6] meson: install $localstatedir/run for qga,
Paolo Bonzini <=
- [PATCH 6/6] build-sys: remove install target from Makefile, marcandre . lureau, 2020/08/26