- Pacman-dev - lists.archlinux.org

[Git][pacman/pacman][master] lint_pkgbuild/arch: always perform some checks
by Allan McRae (＠allan) 30 Jan '25

30 Jan '25

Allan McRae pushed to branch master at Pacman / Pacman Commits: bc7b8e2e by Allan McRae at 2025-01-30T01:17:40+00:00 lint_pkgbuild/arch: always perform some checks When using --ignorearch or options that imply it (e.g. --printsrcinfo), all checks of the arch array were skipped. Instead, perform all checks apart from confirming that the package can be built on that architecture. Signed-off-by: Allan McRae <allan(a)archlinux.org> - - - - - 1 changed file: - scripts/libmakepkg/lint_pkgbuild/arch.sh.in View it on GitLab: https://gitlab.archlinux.org/pacman/pacman/-/commit/bc7b8e2eb22a2c125735901… -- View it on GitLab: https://gitlab.archlinux.org/pacman/pacman/-/commit/bc7b8e2eb22a2c125735901… You're receiving this email because of your account on gitlab.archlinux.org.

1 0

[Git][pacman/pacman][master] Add a .gdb_index section before splitting debug info
by Allan McRae (＠allan) 19 Jan '25

19 Jan '25

Allan McRae pushed to branch master at Pacman / Pacman Commits: faea4735 by Allan McRae at 2025-01-19T04:27:42+00:00 Add a .gdb_index section before splitting debug info Using gdb-add-index to add a .gdb_index section before splitting debug info (together with enabling "maintenance set debuginfod download-sections" in GDB) can dramatically reduce the amount of data GDB has to download. Fixes #205. Signed-off-by: Allan McRae <allan(a)archlinux.org> - - - - - 1 changed file: - scripts/libmakepkg/tidy/strip.sh.in View it on GitLab: https://gitlab.archlinux.org/pacman/pacman/-/commit/faea4735505c7aae4a06934… -- View it on GitLab: https://gitlab.archlinux.org/pacman/pacman/-/commit/faea4735505c7aae4a06934… You're receiving this email because of your account on gitlab.archlinux.org.

1 0

[Git][pacman/pacman][master] libmakepkg: set CCACHE_PREFIX to absolute path of distcc when using both
by Allan McRae (＠allan) 18 Jan '25

18 Jan '25

Allan McRae pushed to branch master at Pacman / Pacman Commits: 680169fb by Guoxin Pu at 2025-01-18T05:08:44+00:00 libmakepkg: set CCACHE_PREFIX to absolute path of distcc when using both The current logic sets CCACHE_PREFIX to distcc when both distcc and ccache are enabled. However, according to the source of ccache, it would execute the command with execv, which would not look up arg0 from PATH, unlike those exec functions with _p suffix. This would result in the following error, when building a package with both ccache and distcc enabled: ``` ccache: error: execute_noreturn of distcc failed: No such file or directory ``` This breaks package builds in different ways: packages that use make/cc directly would yield the actual error which is the same as the above line, packages that rely on other build systems wouldn't go through compiler check and complain on a bad compiler. To reproduce the problem, try to build a simple package: ``` git clone https://gitlab.archlinux.org/archlinux/packaging/packages/abc.git cd abc cp /etc/makepkg.conf . echo 'BUILDENV=(distcc color ccache check !sign)' >> makepkg.conf makepkg --config makepkg.conf ``` refs: https://github.com/ccache/ccache/blob/f887434d35bf6544ce3b56dd7684de804cdaa… https://man.archlinux.org/man/exec.3.en#v_-_execv(),_execvp(),_execvpe() Signed-off-by: Guoxin Pu <pugokushin(a)gmail.com> - - - - - 1 changed file: - scripts/libmakepkg/buildenv/compiler.sh.in View it on GitLab: https://gitlab.archlinux.org/pacman/pacman/-/commit/680169fb61ede7e084adef8… -- View it on GitLab: https://gitlab.archlinux.org/pacman/pacman/-/commit/680169fb61ede7e084adef8… You're receiving this email because of your account on gitlab.archlinux.org.

1 0

[Git][pacman/pacman][master] Detect repeated values in the arch array
by Allan McRae (＠allan) 17 Jan '25

17 Jan '25

Allan McRae pushed to branch master at Pacman / Pacman Commits: e767cf9a by Allan McRae at 2025-01-16T23:14:22+00:00 Detect repeated values in the arch array Repeated values in the arch array can result in architecture specific fields being repeated when using --printsrcinfo. Signed-off-by: Allan McRae <allan(a)archlinux.org> - - - - - 1 changed file: - scripts/libmakepkg/lint_pkgbuild/arch.sh.in View it on GitLab: https://gitlab.archlinux.org/pacman/pacman/-/commit/e767cf9a549f9be835fc5a5… -- View it on GitLab: https://gitlab.archlinux.org/pacman/pacman/-/commit/e767cf9a549f9be835fc5a5… You're receiving this email because of your account on gitlab.archlinux.org.

1 0

RFC: alternate approach to split packages
by Allan McRae 11 Jan '25

11 Jan '25

Hi all, When package splitting was implemented in makepkg, it was done so under two premises: 1) Arch Linux did not split packages out of principle, and there would be very few exceptions 2) KDE was the target package to be split, and its build system provided nice make targets for doing so This meant we targetted splitting packages where upstream provided direct methods for installing the split components. So looking like: pkgname=('pkg1' 'pkg2') package_pkg1() { cd "$pkgbase-$pkgver" make DESTDIR="$pkgdir/" install-pkg1 } package_pkg2() { cd "$pkgbase-$pkgver" make DESTDIR="$pkgdir/" install-pkg2 } As time went by, Arch started splitting packages more - mostly to avoid pulling large dependencies onto a system for wider components. Packagers even started using (abusing) optional dependencies to avoid splitting. The current splitting approach leads to monstrosities like the Arch Linux gcc PKGBUILD [1] (of which I completely blame the current packagers and take no responsibility!). Recently, some packagers have been using a custom _pick function [2] to allow them to move specified files from the first package's "make install" and save them for a later split package. The _pick approach is more similar to how rpm/deb approaches work, but relies on the (non-guaranteed) stability of makepkg internals. I have roughly implemented an additional approach to package splitting that will improve this situation. Essentially, the "make install" is done in the stash() function, and the files are stashed for future use in packaging. The individual package_*() functions then provide the files to include in the package in a "filelist" array. This array allows the '*' wildcard, and '!' to exclude files. A simple example: pkgname=('foo' 'foo-docs') stash() { cd $pkgbase-$pkgver make DESTDIR=$stashdir install } package_foo() { filelist=('*' '!usr/share/docs/') ... } package_foo-docs() { filelist=('usr/share/docs/') ... } I'm seeking feedback on this approach before I finish the implementation and push this. Either replying to the email, or on the gitlab snippet [3]. Thanks! Allan [1] https://gitlab.archlinux.org/archlinux/packaging/packages/gcc/-/blob/main/P… [2] https://gitlab.archlinux.org/archlinux/packaging/packages/gvfs/-/blob/main/… [3] https://gitlab.archlinux.org/-/snippets/3770

1 0

[Git][pacman/pacman][master] 2 commits: Add accessor for handle->disable_sandbox
by Allan McRae (＠allan) 10 Jan '25

10 Jan '25

Allan McRae pushed to branch master at Pacman / Pacman Commits: e6b3eb25 by Allan McRae at 2025-01-10T04:36:49+00:00 Add accessor for handle->disable_sandbox Signed-off-by: Allan McRae <allan(a)archlinux.org> - - - - - e65b7d42 by Allan McRae at 2025-01-10T04:36:49+00:00 Add accessor for handle->disable_dl_timeout Signed-off-by: Allan McRae <allan(a)archlinux.org> - - - - - 2 changed files: - lib/libalpm/alpm.h - lib/libalpm/handle.c View it on GitLab: https://gitlab.archlinux.org/pacman/pacman/-/compare/7ccf316ceb767ddbd2c967… -- View it on GitLab: https://gitlab.archlinux.org/pacman/pacman/-/compare/7ccf316ceb767ddbd2c967… You're receiving this email because of your account on gitlab.archlinux.org.

1 0

[Git][pacman/pacman][master] Provide a default sandboxuser
by Allan McRae (＠allan) 10 Jan '25

10 Jan '25

Allan McRae pushed to branch master at Pacman / Pacman Commits: 7ccf316c by Allan McRae at 2025-01-09T12:14:25+10:00 Provide a default sandboxuser If sandboxuser is not set, pacman/libalpm does not handle moving incomplete download files out of the temporary download directories and into the cache. This leave download_XXXXXX directories in the cache that cause warnings on -Sc operations. Initialise the sandboxuser with the username of UID 0 (root on most systems). Fixes #209. Signed-off-by: Allan McRae <allan(a)archlinux.org> - - - - - 1 changed file: - lib/libalpm/alpm.c View it on GitLab: https://gitlab.archlinux.org/pacman/pacman/-/commit/7ccf316ceb767ddbd2c967a… -- View it on GitLab: https://gitlab.archlinux.org/pacman/pacman/-/commit/7ccf316ceb767ddbd2c967a… You're receiving this email because of your account on gitlab.archlinux.org.

1 0

[Git][pacman/pacman][master] pacman.conf.5: replace {sysconfdir} with /etc in code block
by Allan McRae (＠allan) 08 Jan '25

08 Jan '25

Allan McRae pushed to branch master at Pacman / Pacman Commits: ae603e35 by Allan McRae at 2025-01-08T14:25:09+10:00 pacman.conf.5: replace {sysconfdir} with /etc in code block Variable replacements are not performed on code blocks leaving the example Include usage with '{sysconfdir}/pacman.d/mirrorlist'. Replace this with '/etc' consistent with other example blocks in the man page. Signed-off-by: Allan McRae <allan(a)archlinux.org> - - - - - 1 changed file: - doc/pacman.conf.5.asciidoc View it on GitLab: https://gitlab.archlinux.org/pacman/pacman/-/commit/ae603e35d7f60e686109ca0… -- View it on GitLab: https://gitlab.archlinux.org/pacman/pacman/-/commit/ae603e35d7f60e686109ca0… You're receiving this email because of your account on gitlab.archlinux.org.

1 0

[Git][pacman/pacman][master] Remove executable permissions from source file
by Allan McRae (＠allan) 08 Jan '25

08 Jan '25

Allan McRae pushed to branch master at Pacman / Pacman Commits: 4908eed5 by Allan McRae at 2025-01-08T03:08:18+00:00 Remove executable permissions from source file Signed-off-by: Allan McRae <allan(a)archlinux.org> - - - - - 1 changed file: - scripts/libmakepkg/lint_config/source_date_epoch.sh.in View it on GitLab: https://gitlab.archlinux.org/pacman/pacman/-/commit/4908eed57dd837a53f5ceb8… -- View it on GitLab: https://gitlab.archlinux.org/pacman/pacman/-/commit/4908eed57dd837a53f5ceb8… You're receiving this email because of your account on gitlab.archlinux.org.

1 0

[Git][pacman/pacman][master] 2 commits: lint_config: ensure NPROC is set to something usable
by Allan McRae (＠allan) 08 Jan '25

08 Jan '25

Allan McRae pushed to branch master at Pacman / Pacman Commits: cc87999d by Allan McRae at 2025-01-08T03:06:37+00:00 lint_config: ensure NPROC is set to something usable We use NPROC for managing the number of parallel jobs to run, so it is essentially that this is a valid number. Add a lint pass, and move the setting of the default value to this location. Signed-off-by: Allan McRae <allan(a)archlinux.org> - - - - - 286e776e by Allan McRae at 2025-01-08T03:06:37+00:00 lint_config: move PACKAGER check into its own function The check for the format of the PACKAGER variable did not align with the rest of the function where it was located. Move to its own function. Signed-off-by: Allan McRae <allan(a)archlinux.org> - - - - - 4 changed files: - scripts/libmakepkg/lint_config/meson.build - + scripts/libmakepkg/lint_config/nproc.sh.in - + scripts/libmakepkg/lint_config/packager.sh.in - scripts/libmakepkg/lint_config/variable.sh.in View it on GitLab: https://gitlab.archlinux.org/pacman/pacman/-/compare/425ff046edb861b52cce9f… -- View it on GitLab: https://gitlab.archlinux.org/pacman/pacman/-/compare/425ff046edb861b52cce9f… You're receiving this email because of your account on gitlab.archlinux.org.

1 0