>>>>> On Mon, 11 Jul 2022, Ionen Wolkens wrote:
>> -    ebegin "  python_check_deps"
>> -    python_check_deps
>> -    eend ${?}
>> +    einfo "  python_check_deps"
>> +    if python_check_deps; then
>> +            einfo "  python_check_deps succeeded"
>> +    else
>> +            einfo "  python_check_deps failed"
>> +    fi
>> }

> I was about to go about merging this as suggested, but this masks the
> return value, and then things like this always succeed:

> if _python_run_check_deps "${impl}"; then

Maybe leave ebegin/eend in place then, which was invented precisely for
this use case? What's so bad about nesting it?

Ulrich

Attachment: signature.asc
Description: PGP signature

Reply via email to