On 24/08/01 08:53PM, Thomas Weißschuh wrote:
> On 2024-08-01 11:36:37+0000, Nathan Chancellor wrote:
> > Hi Jose,
> >
> > On Thu, Aug 01, 2024 at 07:29:40AM -0600, Jose Fernandez wrote:
> > > Add a new -debug package to the pacman PKGBUILD that will contain the
> > > vmlinux image for debugging purposes. This package depends on the
> > > -headers package and will be installed in /usr/src/debug/${pkgbase}.
> > >
> > > The vmlinux image is needed to debug core dumps with tools like crash.
> > >
> > > Signed-off-by: Jose Fernandez <jose.fernandez@xxxxxxxxx>
> > > Reviewed-by: Peter Jung <ptr1337@xxxxxxxxxxx>
> >
> > This appears to add a non-trivial amount of time to the build when benchmarking
> > with Arch Linux's configuration (I measure 9% with hyperfine):
>
> As nothing more is compiled, I guess this is just the additional
> packaging.
I would expect that the overhead stems from the extra compression that
needs to be done for the resulting package and depending on what you
have set in your /etc/makepkg.conf this can add significant overhead.
For packages that are not planned to be distributed it might make sense
to completely disable compression[0] by setting PKGEXT. The referenced
forum discussion also explains that this can be hard-disabled in the
PKGBUILD itself, but I would prefer if we could respect the user config
in that regard (especially for people who would like to save some space
on their system).
Additionally the default makepkg configuration is ZSTD compression with
a few parameters that are pretty time consuming (as they are meant to be
used by packagers), which will most likely be reverted again in favour
of a more widely applicable set of defaults with the next pacman
package release. See [2] for the related discussion on the Arch Linux
bugtracker.
The rest of the changes looks fine to me, adding in some configuration
options on which package is actually built via the other patch also
sounds good.
Reviewed-by: Christian Heusel <christian@xxxxxxxxx>
Cheers,
Christian
[0]: https://bbs.archlinux.org/viewtopic.php?id=127894
[1]: https://gitlab.archlinux.org/archlinux/packaging/packages/pacman/-/issues/23
Attachment:
signature.asc
Description: PGP signature
