Received-SPF: pass (google.com: domain of linux-kernel+bounces-95168-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
Message-ID: <feba810d-1391-475e-9b56-fbb6c52912a0@leemhuis.info>
Date: Thu, 7 Mar 2024 09:56:47 +0100
Precedence: bulk
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH v1] docs: verify/bisect: fixes, finetuning, and support
 for Arch
Content-Language: en-US, de-DE
To: Bagas Sanjaya <bagasdotme@gmail.com>, Jonathan Corbet <corbet@lwn.net>
Cc: regressions@lists.linux.dev, linux-doc@vger.kernel.org,
 linux-kernel@vger.kernel.org, =?UTF-8?B?UGV0ciBUZXNhxZnDrWs=?=
 <petr@tesarici.cz>
References: <6592c9ef4244faa484b4113f088dbc1beca61015.1709716794.git.linux@leemhuis.info>
 <ZehvJv3fDZIPs9Kc@archie.me>
From: "Linux regression tracking (Thorsten Leemhuis)"
 <regressions@leemhuis.info>
Reply-To: Linux regressions mailing list <regressions@lists.linux.dev>
In-Reply-To: <ZehvJv3fDZIPs9Kc@archie.me>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit

On 06.03.24 14:27, Bagas Sanjaya wrote:
> On Wed, Mar 06, 2024 at 10:21:12AM +0100, Thorsten Leemhuis wrote:
>> diff --git a/Documentation/admin-guide/verify-bugs-and-bisect-regressions.rst b/Documentation/admin-guide/verify-bugs-and-bisect-regressions.rst
>> index 54bde8bac95c67..58211840ac6ffb 100644
>> --- a/Documentation/admin-guide/verify-bugs-and-bisect-regressions.rst
>> +++ b/Documentation/admin-guide/verify-bugs-and-bisect-regressions.rst
>> @@ -192,8 +192,8 @@ will be considered the 'good' release and used to prepare the .config file.
>>  
>>         sudo rm -rf /lib/modules/6.0-rc1-local-gcafec0cacaca0
>>         sudo kernel-install -v remove 6.0-rc1-local-gcafec0cacaca0
>> -       # * Note, if kernel-install is missing, you will have to
>> -       #   manually remove the kernel image and related files.
>> +       # * Note, on some distributions kernel-install is missing
>> +       #   or does only part of the job.
> 
> Consult documentation of your distribution for how to manually install/remove
> the kernel.

That would sent the wrong message in the TLDR, as the step-by-step guide
and the reference section would be the first place to look for details,
which is kinda implicit for a TLDR.

That being said: that made me look closer at the install section of the
guide and the reference section. I think they should mention "consult
documentation of your distribution" more clearly. I just looked into
this, but that requires a few bigger changes, so I'll take care of that
in a later patch I hope to send in the next few days.

>>  Install build requirements
>>  --------------------------
>> @@ -1076,72 +1078,103 @@ about to build.
>>  Here are a few examples what you typically need on some mainstream
>>  distributions:
>>  
>> +* Arch Linux and derivatives::
>> +
>> +    sudo pacman --needed -S bc binutils bison flex gcc git kmod libelf openssl \
>> +      pahole perl zlib ncurses qt6-base
>> +
>>  * Debian, Ubuntu, and derivatives::
>>  
>> -    sudo apt install bc binutils bison dwarves flex gcc git make openssl \
>> -      pahole perl-base libssl-dev libelf-dev
>> +    sudo apt install bc binutils bison dwarves flex gcc git kmod libelf-dev \
>> +      libssl-dev make openssl pahole perl-base pkg-config zlib1g-dev \
>> +      libncurses-dev qt6-base-dev g++
>>  
>>  * Fedora and derivatives::
>>  
>> -    sudo dnf install binutils /usr/include/{libelf.h,openssl/pkcs7.h} \
>> -      /usr/bin/{bc,bison,flex,gcc,git,openssl,make,perl,pahole}
>> +    sudo dnf install binutils \
>> +      /usr/bin/{bc,bison,flex,gcc,git,openssl,make,perl,pahole,rpmbuild} \
>> +      /usr/include/{libelf.h,openssl/pkcs7.h,zlib.h,ncurses.h,qt6/QtGui/QAction}
> 
> IMO qt packages are for people who wish to do xconfig instead of menuconfig
> or nconfig.

Well, yes, that's why the patch also added a few lines to point this out
and mention that installing the qt dev headers is optional. Guess you
just missed that.

Ciao, Thorsten