[Mesa-dev] [PATCH mesa] meson: be explicit about the version required

Dylan Baker dylan at pnwbakers.com
Tue Oct 24 16:40:22 UTC 2017


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 <dylan at 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 <dylan at pnwbakers.com>
> Signed-off-by: Eric Engestrom <eric.engestrom at 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
> 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: signature
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20171024/d7f53e74/attachment.sig>


More information about the mesa-dev mailing list