[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