On Mon, 18 Aug 2025 11:07:58 -0600, Jonathan Corbet wrote:
> Akira Yokosawa <akiyks@xxxxxxxxx> writes:
>
>> Ah, I have finally understood what 5/11 is trying to do.
>>
>> Its changelog mainly talks about an issue you saw after adding options
>> to xindy in that same commit, and you added
>>
>> \newfontfamily\headingfont{DejaVu Serif}
>>
>> to resolve it.
>>
>> Current changelog didn't make sense at all for me.
>>
>> Can you please reword it and make it easier to follow?
>>
>> With that, feel free to add my
>>
>> Reviewed-by: Akira Yokosawa <akiyks@xxxxxxxxx>
>
> So, if I have managed to understand this conversation, this reword is
> all we need to get this series merged..?
Well, after some thoughts on the conversation took place on xindy,
I think I have to withdraw my Reviewed-by: tag.
I was the one who was totally confused.
Please disregard it.
Mauro, I can't review on 5/11 unless you provide me exact steps to reproduce
the font discovery issue you said you have observed under debian at 4/11 of
this series. That is, without assuming your other series of build-wrapper.
The build-wrapper should be upper compatible with the current way of
running sub-make, without any change in conf.py.
I don't think this is too much to ask. Moving both the goal post
and the build script at the same time is the wrong thing to do
in my opinion.
Regards,
Akira
