[Spice-devel] [PATCH spice-server 1/2] Bump meson requirement to 0.47.2

Frediano Ziglio fziglio at redhat.com
Wed Sep 26 09:05:43 UTC 2018


> 
> Hi,
> 
> On Wed, Sep 26, 2018 at 10:25:56AM +0200, Christophe Fergeau wrote:
> > NB: The newer meson we require, the harder it will be to say "ok, from
> > now on, we'll be switching to building with meson by default".
> 
> What would mean switching to it by default? Removing autotools?
> 
> I agree that we should be careful on bumping it too much (I see
> several projects in GNOME using even unreleased versions
> sometimes!!!)
> 
> Still, seems that the last release was 0.48 and 0.47.2 is in
> Fedora 28 even so I think it is fine..
> 
>     https://bodhi.fedoraproject.org/updates/?packages=meson
> 

RHEL8, RHEL7 and Ubuntu 18.04 have Meson 0.45.1. I won't consider
all these deprecated systems.
But as long as there's Python 3 available Meson can be included
in the build (not that is great having to package the build
system in the distribution in order to build it).

> > 
> > Christophe
> > 
> > On Tue, Sep 25, 2018 at 02:40:00PM -0300, Eduardo Lima (Etrunko) wrote:
> > > This new version ships the fix for the issue where 'check' keyword could
> > > not be used in project definition, so we had to run that command again
> > > only to check if it succeeded.
> > > 
> > > https://github.com/mesonbuild/meson/issues/3944
> > > 
> > > Signed-off-by: Eduardo Lima (Etrunko) <etrunko at redhat.com>
> > > ---
> > >  meson.build | 13 ++-----------
> > >  1 file changed, 2 insertions(+), 11 deletions(-)
> > > 
> > > diff --git a/meson.build b/meson.build
> > > index 762f1d1c..3184a6f5 100644
> > > --- a/meson.build
> > > +++ b/meson.build
> > > @@ -2,22 +2,13 @@
> > >  # project definition
> > >  #
> > >  project('spice', 'c',
> > > -        version : run_command('build-aux/git-version-gen',
> > > '${MESON_SOURCE_ROOT}/.tarball-version').stdout().strip(),
> > > +        version : run_command('build-aux/git-version-gen',
> > > '${MESON_SOURCE_ROOT}/.tarball-version', check : true).stdout().strip(),
> > >          license : 'LGPLv2.1',
> > > -        meson_version : '>= 0.47.0')
> > > -
> > > -# double check meson.project_version()
> > > -# we can not use 'check' keyword in run_command() for git-version-gen
> > > above
> > > -# https://github.com/mesonbuild/meson/issues/3944
> > > -version = run_command('build-aux/git-version-gen',
> > > '${MESON_SOURCE_ROOT}/.tarball-version', check : true).stdout().strip()
> > > -if meson.project_version() != version
> > > -  error('Wrong project version')
> > > -endif
> > > +        meson_version : '>= 0.47.2')
> > >  
> > >  message('Updating submodules')
> > >  run_command('build-aux/meson/check-spice-common', check : true)
> > >  
> > > -
> > >  #
> > >  # soversion
> > >  # The versioning is defined by the forumla (CURRENT-AGE.AGE.REVISION)

Frediano


More information about the Spice-devel mailing list