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
