Received-SPF: pass (google.com: domain of linux-kernel+bounces-97789-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249;
Date: Sat, 9 Mar 2024 13:54:35 +0700
From: Bagas Sanjaya <bagasdotme@gmail.com>
To: "Linux regression tracking (Thorsten Leemhuis)" <regressions@leemhuis.info>,
	Jonathan Corbet <corbet@lwn.net>
Cc: regressions@lists.linux.dev, linux-doc@vger.kernel.org,
	linux-kernel@vger.kernel.org,
	Petr =?utf-8?B?VGVzYcWZw61r?= <petr@tesarici.cz>
Subject: Re: [PATCH v1] docs: verify/bisect: fixes, finetuning, and support
 for Arch
Message-ID: <ZewHqzDi5Juou1rU@archie.me>
References: <6592c9ef4244faa484b4113f088dbc1beca61015.1709716794.git.linux@leemhuis.info>
 <ZehvJv3fDZIPs9Kc@archie.me>
 <feba810d-1391-475e-9b56-fbb6c52912a0@leemhuis.info>
Precedence: bulk
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
	protocol="application/pgp-signature"; boundary="W/axohnCOtDbrhTh"
Content-Disposition: inline
In-Reply-To: <feba810d-1391-475e-9b56-fbb6c52912a0@leemhuis.info>


--W/axohnCOtDbrhTh
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Thu, Mar 07, 2024 at 09:56:47AM +0100, Linux regression tracking (Thorst=
en Leemhuis) wrote:
> 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-regressi=
ons.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.
> >> =20
> >>         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.
> >=20
> > Consult documentation of your distribution for how to manually install/=
remove
> > the kernel.
>=20
> 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.
>=20
> 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.
>=20
> >>  Install build requirements
> >>  --------------------------
> >> @@ -1076,72 +1078,103 @@ about to build.
> >>  Here are a few examples what you typically need on some mainstream
> >>  distributions:
> >> =20
> >> +* Arch Linux and derivatives::
> >> +
> >> +    sudo pacman --needed -S bc binutils bison flex gcc git kmod libel=
f openssl \
> >> +      pahole perl zlib ncurses qt6-base
> >> +
> >>  * Debian, Ubuntu, and derivatives::
> >> =20
> >> -    sudo apt install bc binutils bison dwarves flex gcc git make open=
ssl \
> >> -      pahole perl-base libssl-dev libelf-dev
> >> +    sudo apt install bc binutils bison dwarves flex gcc git kmod libe=
lf-dev \
> >> +      libssl-dev make openssl pahole perl-base pkg-config zlib1g-dev \
> >> +      libncurses-dev qt6-base-dev g++
> >> =20
> >>  * Fedora and derivatives::
> >> =20
> >> -    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,rpmbui=
ld} \
> >> +      /usr/include/{libelf.h,openssl/pkcs7.h,zlib.h,ncurses.h,qt6/QtG=
ui/QAction}
> >=20
> > IMO qt packages are for people who wish to do xconfig instead of menuco=
nfig
> > or nconfig.
>=20
> 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.
>=20

OK, thanks!

--=20
An old man doll... just what I always wanted! - Clara

--W/axohnCOtDbrhTh
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iHUEABYKAB0WIQSSYQ6Cy7oyFNCHrUH2uYlJVVFOowUCZewHpwAKCRD2uYlJVVFO
oyN9AP4qiCxkqPRLMtJeAgSU6Kg8PssrdGDymIdNwE5I7KC74QD9GHIDsxA2aDGg
BGfaFSW2UxE+kZzALs5qmS+05hqoVwE=
=ZICm
-----END PGP SIGNATURE-----

--W/axohnCOtDbrhTh--