From 65be7042a876ffe186a42ced04cde60ed81d3136 Mon Sep 17 00:00:00 2001 From: Frantisek Sumsal Date: Mon, 10 Feb 2020 18:35:36 +0100 Subject: [PATCH 1/2] lgtm: set TMPDIR to /var/tmp `/tmp` has `noexec` set in the LGTM infrastructure which breaks meson's compilation checks. Let's use `/var/tmp` for now, until the issue is properly resolved See: https://discuss.lgtm.com/t/meson-fails-to-determine-sizes-of-basic-data-types/2639 --- .lgtm.yml | 13 +++++++++++++ 1 file changed, 13 insertions(+) diff --git a/.lgtm.yml b/.lgtm.yml index 1c7be044cb..5eb24f4a4d 100644 --- a/.lgtm.yml +++ b/.lgtm.yml @@ -1,3 +1,6 @@ +--- +# vi: ts=2 sw=2 et: + extraction: cpp: prepare: @@ -12,6 +15,16 @@ extraction: after_prepare: - pip3 install meson - export PATH="$HOME/.local/bin/:$PATH" + before_index: + # /tmp has `noexec` set in the LGTM infra which breaks meson's + # compilation checks. Let's temporarily override TMPDIR to /var/tmp which + # should work as expected + # See: https://discuss.lgtm.com/t/meson-fails-to-determine-sizes-of-basic-data-types/2639 + - export TMPDIR=/var/tmp + index: + build_command: + - meson build + - ninja -C build python: python_setup: version: 3 From 7ad1f043984386a8bb56a4a556942dcb02b1fd84 Mon Sep 17 00:00:00 2001 From: Frantisek Sumsal Date: Mon, 10 Feb 2020 19:42:44 +0100 Subject: [PATCH 2/2] lgtm: use the system version of meson LGTM was upgraded to Ubuntu Eoan, so the meson there is pretty new. Also, the pip command didn't work anyway, since it didn't attempt to update the previously installed meson. --- .lgtm.yml | 6 ------ 1 file changed, 6 deletions(-) diff --git a/.lgtm.yml b/.lgtm.yml index 5eb24f4a4d..eb17bd52de 100644 --- a/.lgtm.yml +++ b/.lgtm.yml @@ -5,16 +5,10 @@ extraction: cpp: prepare: packages: - - python3-pip - - python3-setuptools - - python3-wheel - libpwquality-dev - libfdisk-dev - libp11-kit-dev - libssl-dev - after_prepare: - - pip3 install meson - - export PATH="$HOME/.local/bin/:$PATH" before_index: # /tmp has `noexec` set in the LGTM infra which breaks meson's # compilation checks. Let's temporarily override TMPDIR to /var/tmp which