Re: [PATCH spice-gtk 4/4] ci: Try Meson dist

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

 



> On 7/8/19 4:10 PM, Frediano Ziglio wrote:
> >>
> >> On 5/20/19 10:39 AM, Frediano Ziglio wrote:
> >>> Make sure Meson is able to generate a correct tarball.
> >>>
> >>> Signed-off-by: Frediano Ziglio <fziglio@xxxxxxxxxx>
> >>> ---
> >>>    .gitlab-ci.yml | 6 ++++++
> >>>    1 file changed, 6 insertions(+)
> >>>
> >>> diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
> >>> index a6cb2cda..5ddb4db8 100644
> >>> --- a/.gitlab-ci.yml
> >>> +++ b/.gitlab-ci.yml
> >>> @@ -82,6 +82,12 @@ fedora-meson:
> >>>    
> >>>      script:
> >>>        - meson --buildtype=release build-default
> >>> +    # Meson does not update submodules recursively
> >>
> >> Autotools do not do that either, it's done in the autogen.sh script
> >>
> > 
> > But Meson is supposed to deal with submodules, Autotools has nothing
> > to do with the repositories, it's quite a different mindset.
> > 
> >>> +    - git submodule update --init --recursive
> >>> +    # this fix an issue with Meson dist
> >>> +    - if ! test -r ../spice-common.git; then DIR=`basename "$PWD"`; ln
> >>> -s
> >>> "$DIR/.git/modules/spice-common" ../spice-common.git; fi
> >>
> >> That's weird. It does work for me locally.
> > 
> > Maybe you already have the link. Or they fixed the issue on a later
> > version.
> 
> Sorry. I meant that your fix works for me (and without it, the
> build fails).
> 
> > 
> >> It is expected that ../spice-common.git would be relative to 'origin'.
> >>
> > 
> > What do you mean? ../spice-common.git is a directory link and origin
> > is a branch.
> 
> I mean I expect ../spice-common.git to not be a directory but a (remote)
> git repository that it's location is based on remote 'origin' url
> 

Yes, it's a bug in Meson. It tries to clone from the local repository
but something is wrong for submodules. I had some patches but I need to
rework them and write also a test for the issue.

> > 
> >>> +    - rm -rf meson-dist
> >>> +    - ninja -C build-default dist
> >>
> >> Note, that the generated tarball does not contain a configure script.
> >> This is probably expected as autotools are not used, but I wanted
> >> to emphasize that the two tarballs are different.
> >>
> > 
> > Yes, Meson "dist" is not powerful enough to generated such a tarball
> > and a manual script would be quite an hack.
> 
> OK, no problem, as long as we keep that in mind.
> 
> Thanks,
>      Uri.
> 

We had some talk and we are of the idea of removing Autotools from spice-gtk.
If not the official distribution will stay with Autotools (and in this case
yes, we have to remember it).

Welcome,
  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]