| Message ID | 20201015201840.282956-1-brogers@suse.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | meson: Only install icons and qemu.desktop if have_system | expand |
On 15/10/20 22:18, Bruce Rogers wrote: > These files are not needed for a linux-user only install. > > Signed-off-by: Bruce Rogers <brogers@suse.com> > --- > ui/meson.build | 7 +++++-- > 1 file changed, 5 insertions(+), 2 deletions(-) > > diff --git a/ui/meson.build b/ui/meson.build > index 78ad792ffb..fb36d305ca 100644 > --- a/ui/meson.build > +++ b/ui/meson.build > @@ -113,8 +113,11 @@ if have_system or xkbcommon.found() > endif > > subdir('shader') > -subdir('icons') > > -install_data('qemu.desktop', install_dir: config_host['qemu_desktopdir']) > +if have_system > + subdir('icons') > + > + install_data('qemu.desktop', install_dir: config_host['qemu_desktopdir']) > +endif > > modules += {'ui': ui_modules} > Queued, thanks. Paolo
Le jeu. 15 oct. 2020 22:22, Bruce Rogers <brogers@suse.com> a écrit : > These files are not needed for a linux-user only install. > Signed-off-by: Bruce Rogers <brogers@suse.com> > --- > ui/meson.build | 7 +++++-- > 1 file changed, 5 insertions(+), 2 deletions(-) > > diff --git a/ui/meson.build b/ui/meson.build > index 78ad792ffb..fb36d305ca 100644 > --- a/ui/meson.build > +++ b/ui/meson.build > @@ -113,8 +113,11 @@ if have_system or xkbcommon.found() > endif > > subdir('shader') > -subdir('icons') > > -install_data('qemu.desktop', install_dir: config_host['qemu_desktopdir']) > +if have_system > Some tools could have an icon, QSD later? For now: Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org> + subdir('icons') > + > + install_data('qemu.desktop', install_dir: > config_host['qemu_desktopdir']) > +endif > > modules += {'ui': ui_modules} > -- > 2.28.0 > > >
On Fri, Oct 16, 2020 at 10:26:16AM +0200, Philippe Mathieu-Daudé wrote: > Le jeu. 15 oct. 2020 22:22, Bruce Rogers <brogers@suse.com> a écrit : > > > These files are not needed for a linux-user only install. > > > > Signed-off-by: Bruce Rogers <brogers@suse.com> > > --- > > ui/meson.build | 7 +++++-- > > 1 file changed, 5 insertions(+), 2 deletions(-) > > > > diff --git a/ui/meson.build b/ui/meson.build > > index 78ad792ffb..fb36d305ca 100644 > > --- a/ui/meson.build > > +++ b/ui/meson.build > > @@ -113,8 +113,11 @@ if have_system or xkbcommon.found() > > endif > > > > subdir('shader') > > -subdir('icons') > > > > -install_data('qemu.desktop', install_dir: config_host['qemu_desktopdir']) > > +if have_system > > > > Some tools could have an icon, QSD later? The icons are only needed for the tools with graphical display (ie SDL) right now. Regards, Daniel
diff --git a/ui/meson.build b/ui/meson.build index 78ad792ffb..fb36d305ca 100644 --- a/ui/meson.build +++ b/ui/meson.build @@ -113,8 +113,11 @@ if have_system or xkbcommon.found() endif subdir('shader') -subdir('icons') -install_data('qemu.desktop', install_dir: config_host['qemu_desktopdir']) +if have_system + subdir('icons') + + install_data('qemu.desktop', install_dir: config_host['qemu_desktopdir']) +endif modules += {'ui': ui_modules}
These files are not needed for a linux-user only install. Signed-off-by: Bruce Rogers <brogers@suse.com> --- ui/meson.build | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-)