Skip to content
  • Emma Anholt's avatar
    ci: Enable -Werror on the meson-i386 build. · f6e59911
    Emma Anholt authored
    
    
    I find warnings to be very disruptive to my workflow (using emacs's "go to
    next error" feature), and I periodically have to go clean up other
    people's drivers to get back to finding my own warnings in the noise.  I
    know I'm not the only one doing something like this.
    
    We don't want to enable -Werror by default in builds, since it means that
    end users will have builds spuriously fail based on what compiler version
    and opt flags they have compared to what the devs are using.  However, it
    is quite easy to have CI ensure that we at least don't introduce warnings
    on the compiler version that it uses.
    
    For now I've just enabled it on meson-i386 to cover a bunch of Mesa core
    and get us started on ratcheting up warnings-cleanliness in the tree,
    without me having to fix up all the drivers at once.
    
    Reviewed-by: default avatarDaniel Stone <daniels@collabora.com>
    Reviewed-by: default avatarKristian H. Kristensen <hoegsberg@google.com>
    Reviewed-by: default avatarAlyssa Rosenzweig <alyssa.rosenzweig@collabora.com>
    Reviewed-by: Eric Engestrom's avatarEric Engestrom <eric@engestrom.ch>
    Tested-by: Marge Bot <mesa/mesa!3539>
    Part-of: <mesa/mesa!3539>
    f6e59911
Validating GitLab CI configuration… Learn more