This seems reasonable, could you wrap the hanging indent like meson_options with the closing brace on it's own line and with each option on its own line? ie: project( mesa ... )
With that: Reviewed-by: Dylan Baker <dy...@pnwbakers.com> Quoting Eric Engestrom (2017-10-24 07:04:01) > This way, we know what we're allowed to use (no nested include lists > for instance) and users get immediate feedback when trying to use > unsupported versions, rather than a cryptic crash or things being > silently not built correctly. > > Cc: Dylan Baker <dy...@pnwbakers.com> > Signed-off-by: Eric Engestrom <eric.engest...@imgtec.com> > --- > meson.build | 6 +++++- > 1 file changed, 5 insertions(+), 1 deletion(-) > > diff --git a/meson.build b/meson.build > index 9f4812258236d6dfd72d..38038c322082f03236f2 100644 > --- a/meson.build > +++ b/meson.build > @@ -18,7 +18,11 @@ > # OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN > THE > # SOFTWARE. > > -project('mesa', ['c', 'cpp'], version : '17.3.0-devel', license : 'MIT', > +project('mesa', > + ['c', 'cpp'], > + version : '17.3.0-devel', > + license : 'MIT', > + meson_version : '>= 0.42', > default_options : ['c_std=c99', 'cpp_std=c++11']) > > # Arguments for the preprocessor, put these in a separate array from the C > and > -- > Cheers, > Eric >
signature.asc
Description: signature
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev