October 2015 - Pacman-dev - lists.archlinux.org

[pacman-dev] [PATCH v2 1/3] package.c: Remove obsolete param from doc
by Pierre Neidhardt 20 Oct '15

20 Oct '15

Signed-off-by: Pierre Neidhardt <ambrevar(a)gmail.com> --- src/pacman/package.c | 1 - 1 file changed, 1 deletion(-) diff --git a/src/pacman/package.c b/src/pacman/package.c index 2fd0a9c..0e4977f 100644 --- a/src/pacman/package.c +++ b/src/pacman/package.c @@ -79,7 +79,6 @@ static void optdeplist_display(alpm_pkg_t *pkg, unsigned short cols) * Extra information entails 'required by' info for sync packages and backup * files info for local packages. * @param pkg package to display information for - * @param from the type of package we are dealing with * @param extra should we show extra information */ void dump_pkg_full(alpm_pkg_t *pkg, int extra) -- 2.6.1

3 10

[pacman-dev] pacman-5.0 planning
by Allan McRae 20 Oct '15

20 Oct '15

Hi, Hooks have landed! That means we need to have a release. So here is what is done so far. 1) HOOKS! Andrew has implemented hooks. So far they can not be passed the file names triggered the hook, but there are some WIP patches available. The release is still a while off, so these can be included 9if they don't take months!). 2) File database operations. Listing, searching the sync files databases. 3) Libmakepkg. A lot has been split from makepkg. makepkg has gone from 3838 lines in the last to 2380 lines in current git. More can be split, but the work is done to allow most the most important areas of makepkg to be extended as wanted. Hooks has documentation written, files operations has no documentation yet, and I am not sure what documentation is needed for libmakepkg (suggestions welcome). Is there anything else that people are planning that they would like to land for pacman-5.0? Ping any patches that have not been applied yet, or bug reports that need addressed in reply to this. I will go through patchwork and the bug tracker at some stage anyway. Cheers, Allan

3 4

[pacman-dev] [PATCH] makepkg: Implement 'autolibprovides' option
by Mohammad Alsaleh 20 Oct '15

20 Oct '15

The option tries to find all real (no symlinks) library names in the paths a compiler/linker search for by default. And add those libraries to libprovides. The option is disabled by default. Signed-off-by: Mohammad Alsaleh <CE.Mohammad.AlSaleh(a)gmail.com> --- Side notes: 1- I wasn't sure where to define the option. Since all options were moved to libmakepkg/tidy. 2- A side benefit of this approach is that the developer does not have to guess what to put exactly in provides. A discussion from 2.5 years ago: https://lists.archlinux.org/pipermail/pacman-dev/2013-May/017147.html https://lists.archlinux.org/pipermail/pacman-dev/2013-May/017152.html etc/makepkg.conf.in | 23 ++++++++++++----------- scripts/makepkg.sh.in | 22 ++++++++++++++++++++++ 2 files changed, 34 insertions(+), 11 deletions(-) diff --git a/etc/makepkg.conf.in b/etc/makepkg.conf.in index 71ec624..da6e7ad 100644 --- a/etc/makepkg.conf.in +++ b/etc/makepkg.conf.in @@ -74,18 +74,19 @@ BUILDENV=(!distcc color !ccache check !sign) # Default: OPTIONS=(strip docs libtool staticlibs emptydirs zipman purge !upx !optipng !debug) # A negated option will do the opposite of the comments below. # -#-- strip: Strip symbols from binaries/libraries -#-- docs: Save doc directories specified by DOC_DIRS -#-- libtool: Leave libtool (.la) files in packages +#-- strip: Strip symbols from binaries/libraries +#-- docs: Save doc directories specified by DOC_DIRS +#-- libtool: Leave libtool (.la) files in packages #-- staticlibs: Leave static library (.a) files in packages -#-- emptydirs: Leave empty directories in packages -#-- zipman: Compress manual (man and info) pages in MAN_DIRS with gzip -#-- purge: Remove files specified by PURGE_TARGETS -#-- upx: Compress binary executable files using UPX -#-- optipng: Optimize PNG images with optipng -#-- debug: Add debugging flags as specified in DEBUG_* variables -# -OPTIONS=(strip docs libtool staticlibs emptydirs zipman purge !upx !optipng !debug) +#-- emptydirs: Leave empty directories in packages +#-- zipman: Compress manual (man and info) pages in MAN_DIRS with gzip +#-- purge: Remove files specified by PURGE_TARGETS +#-- upx: Compress binary executable files using UPX +#-- optipng: Optimize PNG images with optipng +#-- debug: Add debugging flags as specified in DEBUG_* variables +#-- autolibprovides: Auto-add so provides +# +OPTIONS=(strip docs libtool staticlibs emptydirs zipman purge !upx !optipng !debug !autolibprovides) #-- File integrity checks to use. Valid: md5, sha1, sha256, sha384, sha512 INTEGRITY_CHECK=(md5) diff --git a/scripts/makepkg.sh.in b/scripts/makepkg.sh.in index 6ededa3..5475bcc 100644 --- a/scripts/makepkg.sh.in +++ b/scripts/makepkg.sh.in @@ -48,6 +48,7 @@ declare -r startdir="$PWD" LIBRARY=${LIBRARY:-'@libmakepkgdir@'} +packaging_options+=('autolibprovides') build_options=('ccache' 'distcc' 'buildflags' 'makeflags') splitpkg_overrides=('pkgdesc' 'arch' 'url' 'license' 'groups' 'depends' 'optdepends' 'provides' 'conflicts' 'replaces' 'backup' @@ -944,6 +945,24 @@ find_libdepends() { printf '%s\n' "${libdepends[@]}" } +auto_libprovides() { + local f solib search_paths autolibprovides + + msg2 "$(gettext "Auto-adding libprovides...")" + + # Only search default library paths + search_paths=($(${CC:-gcc} -print-search-dirs | grep '^lib' | sed "s|^libraries: =|:|;s|:| ${pkgdir}|g")) + + for f in $(find ${search_paths[@]} -maxdepth 1 -type f 2>/dev/null); do + # Match solibs, but exclude ones with a relative path to avoid dups + if [[ "$f" =~ \.so ]] && [[ ! "$f" =~ \.\./ ]]; then + solib="${f##*/}" + autolibprovides+=("${solib%.so.*}.so") + fi + done + + printf '%s\n' "${autolibprovides[@]}" +} find_libprovides() { local p libprovides missing @@ -1130,6 +1149,9 @@ write_pkginfo() { printf "size = %s\n" "$size" printf "arch = %s\n" "$pkgarch" + if check_option "autolibprovides" "y"; then + mapfile -t provides < <(auto_libprovides) + fi mapfile -t provides < <(find_libprovides) mapfile -t depends < <(find_libdepends) -- 2.6.1

4 7

[pacman-dev] [PATCH 0/3] Fix translation issues reported on Transifex
by Allan McRae 19 Oct '15

19 Oct '15

Thanks to the translators who pointed these out. I will deal with removing the spaces before the ellipses in the translation files before pushing this to Transifex with the next release. Allan McRae (3): Pluralize malloc failure string Remove space before ellipses makepkg: separate "sourcefile" into two words scripts/libmakepkg/source/bzr.sh.in | 4 ++-- scripts/makepkg.sh.in | 2 +- scripts/pacman-key.sh.in | 2 +- scripts/repo-add.sh.in | 2 +- src/pacman/conf.c | 4 +++- 5 files changed, 8 insertions(+), 6 deletions(-) -- 2.6.1

1 3

[pacman-dev] [PATCH 1/3] libmakepkg: Add check_buildoption for distcc and ccache
by Jan Alexander Steffens (heftig) 19 Oct '15

19 Oct '15

This reduces code duplication a bit. It also highlights how these options are checked. Signed-off-by: Jan Alexander Steffens (heftig) <jan.steffens(a)gmail.com> --- scripts/libmakepkg/util/option.sh | 22 ++++++++++++++++++++++ scripts/makepkg.sh.in | 8 ++++---- 2 files changed, 26 insertions(+), 4 deletions(-) diff --git a/scripts/libmakepkg/util/option.sh b/scripts/libmakepkg/util/option.sh index 1b2d94d..1266e05 100644 --- a/scripts/libmakepkg/util/option.sh +++ b/scripts/libmakepkg/util/option.sh @@ -106,3 +106,25 @@ check_buildenv() { # not found return 127 } + +## +# Check if option is present in BUILDENV and not inverted in options +# +# usage : check_buildoption( $option, $expected_val ) +# return : 0 - matches expected +# 1 - does not match expected +# 127 - not found +## +check_buildoption() { + case "$2" in + y) # assert enabled + check_buildenv "$1" "y" && ! check_option "$1" "n" + return ;; + n) # assert disabled + check_buildenv "$1" "n" && ! check_option "$1" "y" + return ;; + esac + + # not found + return 127 +} diff --git a/scripts/makepkg.sh.in b/scripts/makepkg.sh.in index ec8218f..a8bac02 100644 --- a/scripts/makepkg.sh.in +++ b/scripts/makepkg.sh.in @@ -857,13 +857,13 @@ run_prepare() { run_build() { # use distcc if it is requested (check buildenv and PKGBUILD opts) - if check_buildenv "distcc" "y" && ! check_option "distcc" "n"; then + if check_buildoption "distcc" "y"; then [[ -d /usr/lib/distcc/bin ]] && export PATH="/usr/lib/distcc/bin:$PATH" export DISTCC_HOSTS fi # use ccache if it is requested (check buildenv and PKGBUILD opts) - if check_buildenv "ccache" "y" && ! check_option "ccache" "n"; then + if check_buildoption "ccache" "y"; then [[ -d /usr/lib/ccache/bin ]] && export PATH="/usr/lib/ccache/bin:$PATH" fi @@ -1562,7 +1562,7 @@ check_software() { fi # distcc - compilation with distcc - if check_buildenv "distcc" "y" && ! check_option "distcc" "n"; then + if check_buildoption "distcc" "y"; then if ! type -p distcc >/dev/null; then error "$(gettext "Cannot find the %s binary required for distributed compilation.")" "distcc" ret=1 @@ -1570,7 +1570,7 @@ check_software() { fi # ccache - compilation with ccache - if check_buildenv "ccache" "y" && ! check_option "ccache" "n"; then + if check_buildoption "ccache" "y"; then if ! type -p ccache >/dev/null; then error "$(gettext "Cannot find the %s binary required for compiler cache usage.")" "ccache" ret=1 -- 2.5.2

2 6

[pacman-dev] [PATCH] Ignore cov-int directory for Coverity scan
by Allan McRae 18 Oct '15

18 Oct '15

Signed-off-by: Allan McRae <allan(a)archlinux.org> --- .gitignore | 1 + 1 file changed, 1 insertion(+) diff --git a/.gitignore b/.gitignore index 09ed787..499d499 100644 --- a/.gitignore +++ b/.gitignore @@ -10,6 +10,7 @@ config.status config.status.lineno configure configure.lineno +cov-int cscope.in.out cscope.out cscope.po.out -- 2.6.1

1 0

[pacman-dev] [GIT] The official pacman repository branch, master, updated. v4.2.1-219-g0ee1bec
by allan＠archlinux.org 18 Oct '15

18 Oct '15

This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "The official pacman repository". The branch, master has been updated via 0ee1beca12f6070d406bbf09c3036124018e57bf (commit) via c99ebca83e5c7ac59d9489ea98d310ee41c91de5 (commit) via 002acb467425755ea6203cce8c77733140489c18 (commit) via cbb7e180c755d1e84983b583518626125781da8a (commit) via a5759cb0d7a65d3637e0065258b7478530b75b7a (commit) via b76409609c051e236a849746db79bd438e353b60 (commit) via 4ceb1c5bf91dea2453d9b888138c40e4f97d408d (commit) via 60ebee7a6e1984f14ea3f88159c28fa9b99d7f5c (commit) via 7000bf919812f3ed56969a86b4aebf90e98ec6a3 (commit) via e8e872c8f93ba13f67b882d64c16606973f0ff67 (commit) via 16718a216ee5607c56d220a198fdb48a923aaafc (commit) via 568a89d58018e89e3a5bb3c28cb0ca3b845aae1b (commit) via c261210ccf0a7d08c7b3f24aee40b7c8d900e6ac (commit) via 46a1ac642907039a64f2ec93d352508b2ba3ba79 (commit) via e8deba3b87784ca14c9afc908046f36a3ad7578c (commit) via 0adb74b29395a727672e4af828a9e5426c738913 (commit) via dd3fe853b30069df19b8d18474c1e94a4038c369 (commit) via 19e61a9e0726d7524b13596da1a07e50ad72d21f (commit) via be503c6eacca4c24878f4aaa928751bc051ebb1f (commit) via 6946d7d1b77a550c3f4de48e4cc6ad1094ac6b10 (commit) via af747ef34a9a152614fcde18abcfc6e1e3ec5d0e (commit) via edbe6c2bdc6a3011e5aea208c44be5eee02f120b (commit) via 75fe6ef104b1071bf763354177f3c4339c1c7854 (commit) via 486643083c853a1d541d6f8e93c68b7a542f5a2d (commit) via 1586b23080f77abf2ffd8c42027dbf8180126cdd (commit) via ec50ec8a1587613a1f602c2e67c4cab33ac3115d (commit) from 578035075108a90d20f084f077badf05d1c8527c (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log ----------------------------------------------------------------- commit 0ee1beca12f6070d406bbf09c3036124018e57bf Author: Allan McRae <allan(a)archlinux.org> Date: Sun Oct 18 11:27:56 2015 +1000 contrib/checkupdates: separate fakeroot call and command Separating the fakeroot command and the pacman call with "--" prevents weird interactions with some locales. See FS#46405. Signed-off-by: Allan McRae <allan(a)archlinux.org> commit c99ebca83e5c7ac59d9489ea98d310ee41c91de5 Author: Pierre Neidhardt <ambrevar(a)gmail.com> Date: Sat Oct 17 19:02:56 2015 +0200 package.c: Comment style Signed-off-by: Pierre Neidhardt <ambrevar(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 002acb467425755ea6203cce8c77733140489c18 Author: Allan McRae <allan(a)archlinux.org> Date: Sat Oct 17 19:50:05 2015 +1000 Do not print any root prefix in files database operations Combining with the --root flag and outputing a consistent root prefix leads to many situations that make no sense. Instead, do not print any prefix for any --files operations. Signed-off-by: Allan McRae <allan(a)archlinux.org> commit cbb7e180c755d1e84983b583518626125781da8a Author: Pierre Neidhardt <ambrevar(a)gmail.com> Date: Sat Oct 17 19:02:55 2015 +0200 package.c: Remove obsolete param from doc Signed-off-by: Pierre Neidhardt <ambrevar(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit a5759cb0d7a65d3637e0065258b7478530b75b7a Author: Andrew Gregory <andrew.gregory.8(a)gmail.com> Date: Fri Oct 16 20:28:31 2015 -0400 add hook tests Signed-off-by: Allan McRae <allan(a)archlinux.org> commit b76409609c051e236a849746db79bd438e353b60 Author: Andrew Gregory <andrew.gregory.8(a)gmail.com> Date: Fri Oct 16 20:28:30 2015 -0400 pactest: add hook/script support Signed-off-by: Andrew Gregory <andrew.gregory.8(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 4ceb1c5bf91dea2453d9b888138c40e4f97d408d Author: Andrew Gregory <andrew.gregory.8(a)gmail.com> Date: Fri Oct 16 20:28:29 2015 -0400 util.py: return the created path Signed-off-by: Andrew Gregory <andrew.gregory.8(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 60ebee7a6e1984f14ea3f88159c28fa9b99d7f5c Author: Andrew Gregory <andrew.gregory.8(a)gmail.com> Date: Fri Oct 16 20:28:28 2015 -0400 pactest: use pacman --hookdir option Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 7000bf919812f3ed56969a86b4aebf90e98ec6a3 Author: Andrew Gregory <andrew.gregory.8(a)gmail.com> Date: Fri Oct 16 20:28:27 2015 -0400 pacman: add user hook directories Signed-off-by: Allan McRae <allan(a)archlinux.org> commit e8e872c8f93ba13f67b882d64c16606973f0ff67 Author: Andrew Gregory <andrew.gregory.8(a)gmail.com> Date: Fri Oct 16 20:28:26 2015 -0400 validate hooks after parsing Signed-off-by: Andrew Gregory <andrew.gregory.8(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 16718a216ee5607c56d220a198fdb48a923aaafc Author: Andrew Gregory <andrew.gregory.8(a)gmail.com> Date: Fri Oct 16 20:28:25 2015 -0400 run hooks during trans_commit Signed-off-by: Andrew Gregory <andrew.gregory.8(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 568a89d58018e89e3a5bb3c28cb0ca3b845aae1b Author: Andrew Gregory <andrew.gregory.8(a)gmail.com> Date: Fri Oct 16 20:28:24 2015 -0400 add hook data types and parser Signed-off-by: Andrew Gregory <andrew.gregory.8(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit c261210ccf0a7d08c7b3f24aee40b7c8d900e6ac Author: Andrew Gregory <andrew.gregory.8(a)gmail.com> Date: Fri Oct 16 20:28:23 2015 -0400 handle: add hookdirs option Signed-off-by: Andrew Gregory <andrew.gregory.8(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 46a1ac642907039a64f2ec93d352508b2ba3ba79 Author: Pierre Neidhardt <ambrevar(a)gmail.com> Date: Sat Oct 17 11:36:13 2015 +0200 PKGBUILD.5: post_*() are functions, not scripts Signed-off-by: Allan McRae <allan(a)archlinux.org> commit e8deba3b87784ca14c9afc908046f36a3ad7578c Author: Allan McRae <allan(a)archlinux.org> Date: Wed Sep 23 00:02:12 2015 +1000 makepkg: remove ability to build individual packages Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 0adb74b29395a727672e4af828a9e5426c738913 Author: Allan McRae <allan(a)archlinux.org> Date: Tue Sep 22 16:11:32 2015 +1000 libmakepkg: remove declaration of unused local variables Signed-off-by: Allan McRae <allan(a)archlinux.org> commit dd3fe853b30069df19b8d18474c1e94a4038c369 Author: Allan McRae <allan(a)archlinux.org> Date: Tue Sep 22 16:06:29 2015 +1000 libmakepkg: return 1 on error in arch array The return value was being assigned when an error was found in a PKGBUILD's arch array but it never was returned. Also remove error message explaining about adding the arch array to a PKGBUILD. That was added a long time ago when the arch array first became compulsory. Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 19e61a9e0726d7524b13596da1a07e50ad72d21f Author: Allan McRae <allan(a)archlinux.org> Date: Tue Sep 22 15:11:52 2015 +1000 libmakepkg: update .gitignore Signed-off-by: Allan McRae <allan(a)archlinux.org> commit be503c6eacca4c24878f4aaa928751bc051ebb1f Author: Allan McRae <allan(a)archlinux.org> Date: Tue Sep 22 14:54:03 2015 +1000 makepkg: remove FORCE_VER variable This variable is no longer in VCS building. Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 6946d7d1b77a550c3f4de48e4cc6ad1094ac6b10 Author: Andrew Gregory <andrew.gregory.8(a)gmail.com> Date: Mon Sep 14 18:37:40 2015 -0400 check fileconflicts and diskspace outside commit This is necessary in order to be able to run PreTransaction hooks as close to the actual commit as possible so that we don't prematurely run hooks for a transaction that ultimately never happens. Signed-off-by: Andrew Gregory <andrew.gregory.8(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit af747ef34a9a152614fcde18abcfc6e1e3ec5d0e Author: Andrew Gregory <andrew.gregory.8(a)gmail.com> Date: Mon Sep 14 18:37:39 2015 -0400 move ini parser into common Signed-off-by: Andrew Gregory <andrew.gregory.8(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit edbe6c2bdc6a3011e5aea208c44be5eee02f120b Author: Andrew Gregory <andrew.gregory.8(a)gmail.com> Date: Mon Sep 14 18:37:38 2015 -0400 move strtrim to util-common Signed-off-by: Andrew Gregory <andrew.gregory.8(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 75fe6ef104b1071bf763354177f3c4339c1c7854 Author: Andrew Gregory <andrew.gregory.8(a)gmail.com> Date: Tue Sep 15 01:08:16 2015 -0400 sortbydeps: skip local packages being updated Signed-off-by: Andrew Gregory <andrew.gregory.8(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 486643083c853a1d541d6f8e93c68b7a542f5a2d Author: Allan McRae <allan(a)archlinux.org> Date: Mon Jul 20 15:46:22 2015 +1000 libmakepkg: check if PKGBUILD variables are arrays or not as appropriate When extracting variables from PKGBUILD (e.g. for .SRCINFO creation) we make assumptions about whether variables are arrays or not. This adds a check to the PKGBUILD linter to ensure variables are arrays or not as appropriate. Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 1586b23080f77abf2ffd8c42027dbf8180126cdd Author: Allan McRae <allan(a)archlinux.org> Date: Sat Aug 8 13:42:07 2015 +1000 libmakepkg: util/pkgbuild.sh needs processing on build pkgbuild.sh contained @DEBUGSUFFIX@ and so needs to be run through the sed command on build. Signed-off-by: Allan McRae <allan(a)archlinux.org> commit ec50ec8a1587613a1f602c2e67c4cab33ac3115d Author: Allan McRae <allan(a)archlinux.org> Date: Sat Aug 8 10:47:13 2015 +1000 pacman.8: Consistently refer to the package database We use "package database" everywhere apart from in the description of "-y". Signed-off-by: Allan McRae <allan(a)archlinux.org> ----------------------------------------------------------------------- Summary of changes: contrib/checkupdates.sh.in | 2 +- doc/PKGBUILD.5.txt | 2 +- doc/makepkg.8.txt | 4 - doc/pacman.8.txt | 6 +- doc/pacman.conf.5.txt | 8 + lib/libalpm/Makefile.am | 3 + lib/libalpm/alpm.c | 5 + lib/libalpm/alpm.h | 10 + lib/libalpm/deps.c | 11 +- lib/libalpm/error.c | 2 + lib/libalpm/handle.c | 59 +++ lib/libalpm/handle.h | 1 + lib/libalpm/hook.c | 498 +++++++++++++++++++++ lib/libalpm/{group.h => hook.h} | 20 +- lib/libalpm/ini.c | 1 + lib/libalpm/ini.h | 1 + lib/libalpm/sync.c | 18 +- lib/libalpm/sync.h | 3 +- lib/libalpm/trans.c | 12 +- scripts/Makefile.am | 3 +- scripts/libmakepkg/.gitignore | 5 +- scripts/libmakepkg/lint_pkgbuild/arch.sh.in | 4 +- scripts/libmakepkg/lint_pkgbuild/changelog.sh.in | 2 +- scripts/libmakepkg/lint_pkgbuild/variable.sh.in | 65 +++ .../util/{pkgbuild.sh => pkgbuild.sh.in} | 0 scripts/makepkg.sh.in | 12 - scripts/po/POTFILES.in | 1 + src/{pacman => common}/ini.c | 0 src/{pacman => common}/ini.h | 0 src/common/util-common.c | 39 ++ src/common/util-common.h | 2 + src/pacman/Makefile.am | 2 + src/pacman/conf.c | 22 + src/pacman/conf.h | 2 + src/pacman/files.c | 28 +- src/pacman/ini.c | 117 +---- src/pacman/ini.h | 31 +- src/pacman/package.c | 3 +- src/pacman/pacman.c | 10 + src/pacman/util.c | 39 -- src/pacman/util.h | 1 - src/util/pactree.c | 36 -- test/pacman/pmfile.py | 27 +- test/pacman/pmpkg.py | 3 +- test/pacman/pmtest.py | 30 +- test/pacman/tests/TESTS | 10 + .../tests/dependency-cycle-fixed-by-upgrade.py | 32 ++ test/pacman/tests/hook-abortonfail.py | 25 ++ test/pacman/tests/hook-file-change-packages.py | 32 ++ .../pacman/tests/hook-file-remove-trigger-match.py | 24 + test/pacman/tests/hook-file-upgrade-nomatch.py | 27 ++ test/pacman/tests/hook-invalid-trigger.py | 25 ++ .../pacman/tests/hook-pkg-install-trigger-match.py | 23 + test/pacman/tests/hook-pkg-remove-trigger-match.py | 23 + .../pacman/tests/hook-pkg-upgrade-trigger-match.py | 26 ++ test/pacman/tests/hook-upgrade-trigger-no-match.py | 23 + test/pacman/util.py | 5 +- 57 files changed, 1144 insertions(+), 281 deletions(-) create mode 100644 lib/libalpm/hook.c copy lib/libalpm/{group.h => hook.h} (69%) create mode 120000 lib/libalpm/ini.c create mode 120000 lib/libalpm/ini.h create mode 100644 scripts/libmakepkg/lint_pkgbuild/variable.sh.in rename scripts/libmakepkg/util/{pkgbuild.sh => pkgbuild.sh.in} (100%) copy src/{pacman => common}/ini.c (100%) copy src/{pacman => common}/ini.h (100%) mode change 100644 => 120000 src/pacman/ini.c mode change 100644 => 120000 src/pacman/ini.h create mode 100644 test/pacman/tests/dependency-cycle-fixed-by-upgrade.py create mode 100644 test/pacman/tests/hook-abortonfail.py create mode 100644 test/pacman/tests/hook-file-change-packages.py create mode 100644 test/pacman/tests/hook-file-remove-trigger-match.py create mode 100644 test/pacman/tests/hook-file-upgrade-nomatch.py create mode 100644 test/pacman/tests/hook-invalid-trigger.py create mode 100644 test/pacman/tests/hook-pkg-install-trigger-match.py create mode 100644 test/pacman/tests/hook-pkg-remove-trigger-match.py create mode 100644 test/pacman/tests/hook-pkg-upgrade-trigger-match.py create mode 100644 test/pacman/tests/hook-upgrade-trigger-no-match.py hooks/post-receive -- The official pacman repository

1 0

[pacman-dev] [PATCH 0/4] hooks
by Andrew Gregory 18 Oct '15

18 Oct '15

= TODO = * documentation * run PreTransaction hooks *after* conflict checks * hook/trigger validation * masking hooks with an empty file or symlink to /dev/null * tests = Hook Directories = ALPM handles keep a list of hook directories to scan. Initially, this list is populated with a single directory: /usr/share/alpm/hooks/. This directory is intended for packages to be able to place hooks in a front-end agnostic manner. Front-ends may specify any number of additional directories to search, with later directories overriding earlier ones. Front-ends can disable hooks altogether by setting an empty list. = Parsing Hooks = Configured directories are scanned for files matching '*.hook'. Hook files must be in pacman-style INI format with the following available options: [Trigger] (multiple Trigger sections allowed) Operation = Sync|Remove (Required) Type = File|Package (Required) Target = <Path|PkgName> (Required, multiple allowed) [Action] (only one Action section allowed) When = PreTransaction|PostTransaction (Required) Exec = <Command> (Required) Depends = <PkgName> (Optional) AbortOnFail (Optional, PreTransaction only) Unlike pacman.conf multiple values are *not* allowed on a single line. Exec is currently limited to the path of the executable to run, arguments are not allowed. This is intended to be a temporary limitation until I decide the best way to go about splitting the arguments, which depends in part on whether we want to try to provide hooks with the packages/files that triggered them. Targets are matched the same as IgnorePkg in pacman.conf, meaning multiple targets can be specified and targets can be negated with a leading '!'. If multiple Trigger sections are specified the hook will run if any of them match the transaction. = Running Hooks = Hooks are not cached after running. Directories are rescanned and hooks reloaded each time. This is to make sure that any hooks added/removed during a transaction are appropriately accounted for. Hooks are run exactly the same as install scriptlets (chroot, output passed to the front-end, etc.). They do *not* have access to which package/file triggered the hook. For the time being, run order is officially undefined. apg Andrew Gregory (4): move strtim to util-common move ini parser into common wip add hooks add example hooks hooks/checkboot.hook | 17 +++ hooks/checkmount | 13 ++ hooks/checkmount.conf | 1 + hooks/checkmount.hook | 17 +++ hooks/sync.hook | 16 +++ lib/libalpm/Makefile.am | 2 + lib/libalpm/alpm.c | 1 + lib/libalpm/alpm.h | 10 ++ lib/libalpm/handle.c | 59 ++++++++ lib/libalpm/handle.h | 1 + lib/libalpm/hook.c | 354 +++++++++++++++++++++++++++++++++++++++++++++++ lib/libalpm/hook.h | 34 +++++ lib/libalpm/trans.c | 6 + src/common/ini.c | 116 ++++++++++++++++ src/common/ini.h | 30 ++++ src/common/util-common.c | 39 ++++++ src/common/util-common.h | 2 + src/pacman/ini.c | 117 +--------------- src/pacman/ini.h | 31 +---- src/pacman/util.c | 39 ------ src/pacman/util.h | 1 - src/util/pactree.c | 36 ----- 22 files changed, 720 insertions(+), 222 deletions(-) create mode 100644 hooks/checkboot.hook create mode 100644 hooks/checkmount create mode 100644 hooks/checkmount.conf create mode 100644 hooks/checkmount.hook create mode 100644 hooks/sync.hook create mode 100644 lib/libalpm/hook.c create mode 100644 lib/libalpm/hook.h create mode 100644 src/common/ini.c create mode 100644 src/common/ini.h mode change 100644 => 120000 src/pacman/ini.c mode change 100644 => 120000 src/pacman/ini.h -- 2.4.5

3 49

[pacman-dev] [PATCH] contrib/checkupdates: separate fakeroot call and command
by Allan McRae 18 Oct '15

18 Oct '15

Separating the fakeroot command and the pacman call with "--" prevents weird interactions with some locales. See FS#46405. Signed-off-by: Allan McRae <allan(a)archlinux.org> --- contrib/checkupdates.sh.in | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/contrib/checkupdates.sh.in b/contrib/checkupdates.sh.in index e8a8198..ff62891 100644 --- a/contrib/checkupdates.sh.in +++ b/contrib/checkupdates.sh.in @@ -51,7 +51,7 @@ eval $(awk -F' *= *' '$1 ~ /DBPath/ { print $1 "=" $2 }' @sysconfdir@/pacman.con mkdir -p "$CHECKUPDATES_DB" ln -s "${DBPath}/local" "$CHECKUPDATES_DB" &> /dev/null -fakeroot pacman -Sy --dbpath "$CHECKUPDATES_DB" --logfile /dev/null &> /dev/null +fakeroot -- pacman -Sy --dbpath "$CHECKUPDATES_DB" --logfile /dev/null &> /dev/null pacman -Qu --dbpath "$CHECKUPDATES_DB" 2> /dev/null | grep -v '\[.*\]' exit 0 -- 2.6.1

1 0

[pacman-dev] [PATCH] Do not print any root prefix in files database operations
by Allan McRae 18 Oct '15

18 Oct '15

Combining with the --root flag and outputing a consistent root prefix leads to many situations that make no sense. Instead, do not print any prefix for any --files operations. Signed-off-by: Allan McRae <allan(a)archlinux.org> --- src/pacman/files.c | 28 +++++++++++++++++++++++++--- 1 file changed, 25 insertions(+), 3 deletions(-) diff --git a/src/pacman/files.c b/src/pacman/files.c index ceec342..31ce9e0 100644 --- a/src/pacman/files.c +++ b/src/pacman/files.c @@ -61,7 +61,7 @@ static int files_fileowner(alpm_list_t *syncs, alpm_list_t *targets) { if(alpm_filelist_contains(files, f)) { if(!config->quiet) { - printf(_("%s is owned by %s/%s %s\n"), filename, + printf(_("%s is owned by %s/%s %s\n"), f, alpm_db_get_name(repo), alpm_pkg_get_name(pkg), alpm_pkg_get_version(pkg)); } else { @@ -166,6 +166,28 @@ notfound: return 0; } +static void dump_file_list(alpm_pkg_t *pkg) { + const char *pkgname; + alpm_filelist_t *pkgfiles; + size_t i; + + pkgname = alpm_pkg_get_name(pkg); + pkgfiles = alpm_pkg_get_files(pkg); + + for(i = 0; i < pkgfiles->count; i++) { + const alpm_file_t *file = pkgfiles->files + i; + /* Regular: '<pkgname> <filepath>\n' + * Quiet : '<filepath>\n' + */ + if(!config->quiet) { + printf("%s%s%s ", config->colstr.title, pkgname, config->colstr.nocolor); + } + printf("%s\n", file->name); + } + + fflush(stdout); +} + static int files_list(alpm_list_t *syncs, alpm_list_t *targets) { alpm_list_t *i, *j; int ret = 0, found = 0; @@ -200,7 +222,7 @@ static int files_list(alpm_list_t *syncs, alpm_list_t *targets) { if((pkg = alpm_db_get_pkg(db, targ)) != NULL) { found = 1; - dump_pkg_files(pkg, config->quiet); + dump_file_list(pkg); break; } } @@ -218,7 +240,7 @@ static int files_list(alpm_list_t *syncs, alpm_list_t *targets) { for(j = alpm_db_get_pkgcache(db); j; j = alpm_list_next(j)) { alpm_pkg_t *pkg = j->data; - dump_pkg_files(pkg, config->quiet); + dump_file_list(pkg); } } } -- 2.6.1

1 0