Re: [PATCH spice-server 1/3] Add support for building with meson

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> 
> On Mon, Jun 18, 2018 at 12:04:18PM -0400, Frediano Ziglio wrote:
> > Not much replies to this specific patch, but I agree was better to
> > integrate dependencies (spice-protocol and spice-common) projects
> > first.
> > 
> 
> > > diff --git a/subprojects/spice-common.wrap
> > > b/subprojects/spice-common.wrap
> > > new file mode 100644
> > > index 00000000..6117c631
> > > --- /dev/null
> > > +++ b/subprojects/spice-common.wrap
> > > @@ -0,0 +1,4 @@
> > > +[wrap-git]
> > > +directory=spice-common
> > > +url=https://gitlab.com/etrunko/spice-common.git
> > > +revision=meson
> > 
> > Don't like much the full URL. Is it not possible to have something
> > relative to the current one?
> > Also, does this points just to master? Would be a regression to
> > current autoconf/submodule behaviour.
> 
> This is one part of the patch which imo does not make sense without
> autotools removal (autotools build -> spice-common/ submodule, meson build ->
> meson-specific subprojects/spice-common/ through this .wrap file). I've
> already sent a patch to avoid this to Eduardo, so I'm expecting at some
> point an updated patch with this change, and probably a few more smaller
> ones.
> 
> Christophe
> 

Can't be use git modules with Meson?
Maybe a workaround could be creating a "subprojects" directory with a link
to spice-common git module?
Would be a pity if this were a stopover to merge Meson integration.
By the way the revision should be a commit hash to avoid to have ABI
(as this would require a release procedure on spice-common we don't
have/want).
Still the problem of relative URL remains. Doing a quick search this
seems not supported by Meson.

Frediano
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux Virtualization]     [Linux Virtualization]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]