Re: [PATCH 00/15] Translate sphinx-pre-install to Python

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 08 Jul 2025 08:25:08 -0600, Jonathan Corbet wrote:
> Akira Yokosawa <akiyks@xxxxxxxxx> writes:
> 
>> Hi Mauro,
>>
>> On Mon, 30 Jun 2025 23:34:48 +0200, Mauro Carvalho Chehab wrote:
>> [...]
>>
>>> The test script also ran the install procedure for system,
>>> venv and native Sphinx install.
>>
>> Which install procedure did you test?  The short one with the "--no-pdf"
>> option?
>>
>> I am asking because installing the full list of packages in podman run
>> of opensuse/leap:15.6 didn't complete successfully for me.
>>
>> And by the look of things, you stopped at installation, because you are
>> well aware of all the issues in running "make htmldocs" and its friends
>> after the install.
>>
>> I assume you (or somebody else) are going to update the script once this
>> series is applied to make the suggested lists of packages be useful for
>> newcomers.
> 
> Thanks for testing this out.
> 
> My question would be: are the results somehow worse than those provided
> by the existing, Perl script?

Not at all.  Mauro made some improvements WRT opensuse/leap over the perl
one, but there remain a lot of rooms for improvements here and there. 

I've been ignoring sphinx-pre-install all these years, but the impressive
test results presented in this cover-letter made me test it.

>                                If this change regresses things, we
> obviously do not want to proceed.  If it reproduces the old behavior in
> a more maintainable language, then hopefully it is a step toward fixing
> things in the long run ... ?

Yes, translation into python would help a lot, I guess.

        Thanks, Akira





[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux FS]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux