[Libosinfo] [osinfo-db PATCH] Only use pytest --log-level arg if it is supported

Cole Robinson crobinso at redhat.com
Tue Mar 26 16:21:02 UTC 2019


On 3/26/19 11:30 AM, Fabiano Fidêncio wrote:
> On Tue, Mar 26, 2019 at 4:26 PM Daniel P. Berrangé <berrange at redhat.com> wrote:
>>
>> Older versions of pytest don't have the --log-level arg
>>
>> Signed-off-by: Daniel P. Berrangé <berrange at redhat.com>
>> ---
>>  Makefile | 4 +++-
>>  1 file changed, 3 insertions(+), 1 deletion(-)
>>
>> diff --git a/Makefile b/Makefile
>> index 254b7af..c63cb6e 100644
>> --- a/Makefile
>> +++ b/Makefile
>> @@ -34,6 +34,8 @@ ZANATA = zanata
>>
>>  PYTHON = python3
>>
>> +PYTEST_LOG_LEVEL = $(shell $(PYTHON) -m pytest --help | grep log-level >/dev/null && echo "--log-level=info")
> 
> This seems the easiest path to take, indeed.
> I'm fine with this but I'd also like to hear from Cole.
> 

ACK from me. As I said on IRC I think switching from log.info to plain
print() is better for other reasons as well but we can discuss that
later. This gets us closer to having working CI so let's push it

Thanks,
Cole




More information about the Libosinfo mailing list