On 24/07/16 07:52PM, Thomas Weißschuh wrote: > pacman is the package manager used by Arch Linux and its derivates. > Creating native packages from the kernel tree has multiple advantages: > > * The package triggers the correct hooks for initramfs generation and > bootloader configuration > * Uninstallation is complete and also invokes the relevant hooks > * New UAPI headers can be installed without any manual bookkeeping Additionally, to what already has been mentioned above, this would also greatly simplify the instructions and the needed setup for bisection if people run into regressions, as for example in the following documents: - https://docs.kernel.org/admin-guide/verify-bugs-and-bisect-regressions.html - https://wiki.archlinux.org/title/Bisecting_bugs_with_Git So thank you (up front) for your effort's here, this could greatly help to improve the debugging experience! I have tested this patch by just running "make pacman-pkg" and everything worked as expected (currently booted from the generated kernel). As noted in another mail in this thread it would also be cool if just plain makepkg could be executed (i.e. -s/--syncdeps, -i/--install and --packagelist) would be interesting, but I think with the current setup this could become a bit complicated 🤔 Maybe it's a possiblity to have a "make pacman-pkgbuild" and then expect the user to interact with the created PKGBUILD..? Idk if that is actually better Also is there a canonical way to be notified whenever people report issues with this or there are patches to it? I'd love to help out if that is desirable from your side .. Also I have written some more comments inline below... > --- > .gitignore | 6 +++ > Makefile | 2 +- > scripts/Makefile.package | 14 +++++++ > scripts/package/PKGBUILD | 99 ++++++++++++++++++++++++++++++++++++++++++++++++ > 4 files changed, 120 insertions(+), 1 deletion(-) > > diff --git a/.gitignore b/.gitignore > index c59dc60ba62e..7902adf4f7f1 100644 > --- a/.gitignore > +++ b/.gitignore > @@ -92,6 +92,12 @@ modules.order > # > /tar-install/ > > +# > +# pacman files (make pacman-pkg) > +# > +/PKGBUILD > +/pacman/ > + > # > # We don't want to ignore the following even if they are dot-files > # > diff --git a/Makefile b/Makefile > index 7372ea45ed3f..768d3dc107f8 100644 > --- a/Makefile > +++ b/Makefile > @@ -1481,7 +1481,7 @@ CLEAN_FILES += vmlinux.symvers modules-only.symvers \ > # Directories & files removed with 'make mrproper' > MRPROPER_FILES += include/config include/generated \ > arch/$(SRCARCH)/include/generated .objdiff \ > - debian snap tar-install \ > + debian snap tar-install PKGBUILD pacman \ > .config .config.old .version \ > Module.symvers \ > certs/signing_key.pem \ > diff --git a/scripts/Makefile.package b/scripts/Makefile.package > index bf016af8bf8a..0aaa0832279c 100644 > --- a/scripts/Makefile.package > +++ b/scripts/Makefile.package > @@ -141,6 +141,19 @@ snap-pkg: > cd $(objtree)/snap && \ > snapcraft --target-arch=$(UTS_MACHINE) > > +# pacman-pkg > +# --------------------------------------------------------------------------- > + > +PHONY += pacman-pkg > +pacman-pkg: > + @ln -srf $(srctree)/scripts/package/PKGBUILD $(objtree)/PKGBUILD > + +objtree="$(realpath $(objtree))" \ > + BUILDDIR=pacman \ > + CARCH="$(UTS_MACHINE)" \ > + KBUILD_MAKEFLAGS="$(MAKEFLAGS)" \ > + KBUILD_REVISION="$(shell $(srctree)/scripts/build-version)" \ > + makepkg > + > # dir-pkg tar*-pkg - tarball targets > # --------------------------------------------------------------------------- > > @@ -221,6 +234,7 @@ help: > @echo ' bindeb-pkg - Build only the binary kernel deb package' > @echo ' snap-pkg - Build only the binary kernel snap package' > @echo ' (will connect to external hosts)' > + @echo ' pacman-pkg - Build only the binary kernel pacman package' > @echo ' dir-pkg - Build the kernel as a plain directory structure' > @echo ' tar-pkg - Build the kernel as an uncompressed tarball' > @echo ' targz-pkg - Build the kernel as a gzip compressed tarball' > diff --git a/scripts/package/PKGBUILD b/scripts/package/PKGBUILD > new file mode 100644 > index 000000000000..eb3957fad915 > --- /dev/null > +++ b/scripts/package/PKGBUILD > @@ -0,0 +1,99 @@ > +# SPDX-License-Identifier: GPL-2.0-only > +# Maintainer: Thomas Weißschuh <linux@xxxxxxxxxxxxxx> > +# Contributor: Jan Alexander Steffens (heftig) <heftig@xxxxxxxxxxxxx> > + > +pkgbase=linux-upstream > +pkgname=("${pkgbase}" "${pkgbase}-headers" "${pkgbase}-api-headers") > +pkgver="${KERNELRELEASE//-/_}" I just wondered whether the version format was chosen deliberatly here, as it yields something like 6.10.0_04474_ga1cc539bf882-6 for example. A similar output is generated by the pkgver() function suggested in the Wiki which would give something like this: v6.8.rc5.r84.g9b9c280-1. https://wiki.archlinux.org/title/VCS_package_guidelines#Git The most notable difference is the usage of underscores vs. dots, which is not an issue but also not usually a thing with arch packages. > +# The PKGBUILD is evaluated multiple times. > +# Running scripts/build-version from here would introduce inconsistencies. > +pkgrel="${KBUILD_REVISION}" As far as I understand this means that the pkgrel is incremented on every build, which is not really usual. The 'pkgrel' variable is typically used to indicate changes to the binary package that are besides an upstream release, i.e. when one changes a compile flag or similar. Also on a new upstream release this flag would be reset to 1, while here we increment the variable monotonically. So this is not an issue per-se, just a bit unusual compared to how we handle packages in Arch itself. See https://wiki.archlinux.org/title/PKGBUILD#pkgrel > +pkgdesc='Linux' Maybe expand the description here a little to something like "The Linux kernel and modules as built from the upstream PKGBUILD", just "Linux" seems to be a bit short. > +url='https://www.kernel.org/' > +# Enable flexible cross-compilation > +arch=(${CARCH}) > +license=(GPL-2.0-only) > +makedepends=( > + base-devel > + bc > + cpio > + gettext > + libelf > + openssl > + pahole > + perl > + python > + rsync > + tar > +) Including base-devel is to the makedepends is pretty sneaky, although none of the make dependencies will ever be installed as we do not sync the dependencies for the packagebuild as I have noted on the beginning of this mail (--syncdeps). > [...]
Attachment:
signature.asc
Description: PGP signature
