Re: [PATCH spice-gtk] build-sys: Use always --buildtype=release

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

 



> 
> On Thu, 2019-02-14 at 12:56 +0000, Frediano Ziglio wrote:
> > Allows to catch some errors which the compiler is not able to
> > detect in debug mode (the default).
> 
> Seems fine, but what kind of errors are you talking about?
> Reviewed-by: Jonathon Jongsma <jjongsma@xxxxxxxxxx>
> 

Like "this variable is not used" or "this variable is not initialized"
for instance.
Not sure if people prefers "release" or "debugoptimized" but without
optimizations compilers cannot detect some potential issues.

Frediano

> 
> > 
> > Signed-off-by: Frediano Ziglio <fziglio@xxxxxxxxxx>
> > ---
> >  .gitlab-ci.yml | 10 +++++-----
> >  1 file changed, 5 insertions(+), 5 deletions(-)
> > 
> > https://gitlab.freedesktop.org/fziglio/spice-gtk/pipelines/19157
> > (only Meson builds)
> > 
> > diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
> > index 8ffc6542..adf53e1c 100644
> > --- a/.gitlab-ci.yml
> > +++ b/.gitlab-ci.yml
> > @@ -57,15 +57,15 @@ fedora-meson:
> >    before_script:
> >      - dnf install -y $DEPS_COMMON $DEPS_FEDORA
> >      - git clone ${CI_REPOSITORY_URL/spice-gtk/spice-protocol}
> > -    - meson spice-protocol build-spice-protocol --prefix=/usr
> > +    - meson --buildtype=release spice-protocol build-spice-protocol
> > --prefix=/usr
> >      - ninja -C build-spice-protocol install
> >  
> >    script:
> > -    - meson build-default
> > +    - meson --buildtype=release build-default
> >      - ninja -C build-default
> >      - ninja -C build-default test
> >  
> > -    - meson build-feat-disabled -Dauto_features=disabled
> > +    - meson --buildtype=release build-feat-disabled
> > -Dauto_features=disabled
> >      - ninja -C build-feat-disabled
> >      - ninja -C build-feat-disabled test
> >  
> > @@ -92,11 +92,11 @@ windows-meson:
> >      - dnf install -y $DEPS_COMMON $DEPS_MINGW
> >      - git clone ${CI_REPOSITORY_URL/spice-gtk/spice-protocol}
> >      - mkdir spice-protocol/build-spice-protocol && cd spice-
> > protocol/build-spice-protocol
> > -    - mingw64-meson --prefix=/usr
> > +    - mingw64-meson --buildtype=release --prefix=/usr
> >      - ninja install
> >  
> >    script:
> >      - cd $CI_PROJECT_DIR
> >      - mkdir build-win64 && cd build-win64
> > -    - mingw64-meson -Dgtk_doc=disabled
> > +    - mingw64-meson --buildtype=release -Dgtk_doc=disabled
> >      - ninja install
> 
> 
_______________________________________________
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]