On Sun, 2 May 2021 09:38:40 -0700, Paul E. McKenney wrote:
> On Sun, May 02, 2021 at 08:59:43PM +0900, Akira Yokosawa wrote:
>> On Sun, 2 May 2021 20:40:42 +0900, Akira Yokosawa wrote:
>>> "kpsewhich tcolorbox.sty" can be empty if tcolorbox is not
>>> available.
>>>
>>> If this is the case, "grep" in `grep ProvidePackage | sed ...`
>>> will wait forever.
>>>
>>> Add an empty check in front of "grep" with exit code of 1.
>>
>> Paul,
>>
>> Please ignore this patch.
>> I'll respin soon.
>>
>> Sorry for the noise.
>
> Ignored, and no problem! Been there, done that! ;-)
Patch v4 I just sent out should be good enough.
I'm afraid you don't have a handy environment to test the error path,
but I did uninstall texlive-tcolorbox on a Fedora container
and confirmed the error check works as expected.
Thanks, Akira
>
> Thanx, Paul
>
>> -- Akira
>>
>>
>>>
>>> Fixes: b2f19231bcc5 ("utilities/autodate.sh: Make newer tcolorbox behave as before")
>>> Signed-off-by: Akira Yokosawa <akiyks@xxxxxxxxx>
>>> ---
>>> utilities/autodate.sh | 7 +++++++
>>> 1 file changed, 7 insertions(+)
>>>
>>> diff --git a/utilities/autodate.sh b/utilities/autodate.sh
>>> index 7bd7cbb8..2b9c1117 100644
>>> --- a/utilities/autodate.sh
>>> +++ b/utilities/autodate.sh
>>> @@ -97,6 +97,13 @@ env printf '\\IfQqzBg{}{\\setboolean{qqzbg}{%s}}\n' $qqzbg
>>>
>>> # command for newer tcolorbox (4.40 or later) to have backward-compatible skips
>>> tcolorbox_sty=`kpsewhich tcolorbox.sty`
>>> +
>>> +if [ "$tcolorbox_sty" == "" ]
>>> +then
>>> + echo "Error: package 'tcolorbox' not found. See See #9 in FAQ-BUILD.txt" >&2
>>> + exit 1
>>> +fi
>>> +
>>> tcbversion=`grep ProvidesPackage $tcolorbox_sty | sed -e 's/.*version \([0-9]\+\.[0-9]\+\).*/\1/g'`
>>> tcbold=4.39
>>> env printf '%% tcolorbox version: %s\n' $tcbversion
>>>
