- Pacman-dev - lists.archlinux.org

[pacman-dev] [PATCH 1/2] meson: use better check for debug builds
by Eli Schwartz 05 May '20

05 May '20

meson 0.48 added the 'debug' and 'optimization' builtin options, which bidirectionally map to the buildtype, but in some cases where debug is enabled, the builtype may be custom. Checking the 'debug' option lets us detect every case currently detected, plus a few more, and does so in a shorter and more concise manner. Signed-off-by: Eli Schwartz <eschwartz(a)archlinux.org> --- meson.build | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/meson.build b/meson.build index 572526b2..680cf62b 100644 --- a/meson.build +++ b/meson.build @@ -177,7 +177,7 @@ elif conf.has('HAVE_STRUCT_STATFS_F_FLAGS') conf.set('FSSTATSTYPE', 'struct statfs') endif -if get_option('buildtype').startswith('debug') +if get_option('debug') extra_cflags = [ '-Wcast-align', '-Wclobbered', -- 2.26.2

2 3

[pacman-dev] [PATCH] Convert gitlab valgrind runner to meson
by Allan McRae 05 May '20

05 May '20

Turns out environmental variables do get passed through fakechroot! Signed-off-by: Allan McRae <allan(a)archlinux.org> --- .gitlab-ci.yml | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index 4961352e..a53d60cf 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -62,11 +62,10 @@ arch-clang: arch-valgrind: extends: .arch-test script: - - pacman -Syu --needed --noconfirm autoconf-archive valgrind - - autoreconf -i - - ./configure --enable-warningflags --disable-doc - - make - - fakechroot make PY_LOG_FLAGS=--valgrind check + - pacman -Syu --needed --noconfirm valgrind + - meson build + - ninja -C build + - PACTEST_VALGRIND=1 fakechroot meson test -C build arch-nettle: extends: .arch-test -- 2.25.1

1 0

[pacman-dev] Bug: using --program-prefix configure option breaks makepkg
by Wouter Wijsman 04 May '20

04 May '20

Hi Pacman team, In the last couple of days I've been trying to build pacman for use with a homebrew sdk for the Playstation portable. The idea is for it to allow sharing of libraries for the system. To make sure this version of pacman doesn't collide with the system's, I use the --program-prefix option for the configure script, which adds "psp-" in front of all binary names. Now when using the --program-prefix option, makepkg is no longer able to work. This because it has the name for the pacman executable name hardcoded it seems. I compile with the following commands: ./configure --prefix=${PSPDEV} --with-buildscript=PSPBUILD --with-root-dir=${PSPDEV}/psp --program-prefix="psp-" --disable-doc make make install For this to work, I require the following patch: https://github.com/sharkwouter/psptoolchain/blob/psp-pacman/patches/pacman-… Is there a way to prevent me from needing this patch? Besides this patch, the pacman 5.2.1 tarball build and works fine for my purposes. P.S. I did find another small annoyance. Which is that "make install" fails if the bash completion files are already installed. Kind regards, Wouter Wijsman

4 4

[pacman-dev] [GIT] The official pacman repository branch, master, updated. v5.2.1-63-gcebda6d0
by Allan McRae 04 May '20

04 May '20

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 cebda6d0ee4c58521cafd9d0980c8b79cc068602 (commit) via 0b81fb64d3f36710d3988ab49feae8c7cc37c1af (commit) from 969e1ab996e79f3e121d0fb3bfd4e91ac7d6f488 (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 cebda6d0ee4c58521cafd9d0980c8b79cc068602 Author: Dave Reisner <dreisner(a)archlinux.org> Date: Sat May 2 12:12:32 2020 -0400 meson: handle XFAIL tests outside of TAP This change causes expected fail tests to actually fail by eliding the '# TODO' from the test plan. In turn, we can now properly use 'should_fail' in the meson test() rule and see these expected fail tests in the output: Before: ... 320/332 upgrade077.py OK 0.12679290771484375 s 321/332 upgrade078.py OK 0.12620115280151367 s 322/332 upgrade080.py OK 0.1252129077911377 s ... Ok: 332 Expected Fail: 0 Fail: 0 Unexpected Pass: 0 Skipped: 0 Timeout: 0 After: ... 320/332 upgrade077.py OK 0.12679290771484375 s 321/332 upgrade078.py EXPECTEDFAIL0.12620115280151367 s 322/332 upgrade080.py OK 0.1252129077911377 s ... Ok: 326 Expected Fail: 6 Fail: 0 Unexpected Pass: 0 Skipped: 0 Timeout: 0 Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 0b81fb64d3f36710d3988ab49feae8c7cc37c1af Author: Allan McRae <allan(a)archlinux.org> Date: Sat May 2 17:48:48 2020 +1000 Add Gitlab Runner configuration Arch Linux is setting up a Gitlab instance. This adds CI for the pacman project, testing a range of configurations on Arch and basic builds on Fedora and Debian. Note that asciidoc is specifically not installed on the Debian run because it is all sorts of broken... Also, the defaults have been set to meson, with two autotools tests that will soon be removed. Original-file from: Andrew Gregory <andrew.gregory.8(a)gmail.com> Altered-to-run-on-Arch-Gitlab by: Sven-Hendrik Haase <svenstaro(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> ----------------------------------------------------------------------- Summary of changes: .gitlab-ci.yml | 128 ++++++++++++++++++++++++++++++++++++++++++++++++ test/pacman/meson.build | 15 +++++- test/pacman/pmenv.py | 7 ++- 3 files changed, 148 insertions(+), 2 deletions(-) create mode 100644 .gitlab-ci.yml hooks/post-receive -- The official pacman repository

1 0

[pacman-dev] [PATCH] Fix program name transformation options for configure script
by Wouter Wijsman 04 May '20

04 May '20

The configure script which is generated when running autotools has the following options for modifying the name of the executables build: 1. --program-prefix 2. --program-suffix 3. --program-transform-name Using any of these options gives autotools the program_transform_name variable, but it wasn't being used for the makepkg script, which made these options cause issues with starting makepkg. This commit addresses this. Signed-off-by: Wouter Wijsman <wwijsman(a)live.nl> --- scripts/Makefile.am | 8 +++++++- scripts/makepkg.sh.in | 6 ++++-- 2 files changed, 11 insertions(+), 3 deletions(-) diff --git a/scripts/Makefile.am b/scripts/Makefile.am index 47455ed2..113ad86c 100644 --- a/scripts/Makefile.am +++ b/scripts/Makefile.am @@ -164,6 +164,10 @@ else REAL_PACKAGE_VERSION = $(PACKAGE_VERSION) endif +# Real executable names using program_transform_name +PACMAN_INSTALL_NAME := $(shell echo pacman|sed '$(program_transform_name)') +PACMAN_CONF_INSTALL_NAME := $(shell echo pacman-conf|sed '$(program_transform_name)') + #### Taken from the autoconf scripts Makefile.am #### edit = sed \ -e 's|@rootdir[@]|$(ROOTDIR)|g' \ @@ -182,7 +186,9 @@ edit = sed \ -e 's|@DEBUGSUFFIX[@]|$(DEBUGSUFFIX)|g' \ -e "s|@INODECMD[@]|$(INODECMD)|g" \ -e "s|@FILECMD[@]|$(FILECMD)|g" \ - -e 's|@SCRIPTNAME[@]|$@|g' + -e 's|@SCRIPTNAME[@]|$@|g' \ + -e 's|@PACMAN_INSTALL_NAME[@]|$(PACMAN_INSTALL_NAME)|g' \ + -e 's|@PACMAN_CONF_INSTALL_NAME[@]|$(PACMAN_CONF_INSTALL_NAME)|g' ## All the scripts depend on Makefile so that they are rebuilt when the ## prefix etc. changes. Use chmod -w to prevent people from editing the diff --git a/scripts/makepkg.sh.in b/scripts/makepkg.sh.in index d1416d15..38b6eda9 100644 --- a/scripts/makepkg.sh.in +++ b/scripts/makepkg.sh.in @@ -43,6 +43,8 @@ unset GREP_OPTIONS declare -r makepkg_version='@PACKAGE_VERSION@' declare -r confdir='@sysconfdir@' declare -r BUILDSCRIPT='@BUILDSCRIPT@' +declare -r PACMAN_INSTALL_NAME='@PACMAN_INSTALL_NAME@' +declare -r PACMAN_CONF_INSTALL_NAME='@PACMAN_CONF_INSTALL_NAME@' declare -r startdir="$(pwd -P)" LIBRARY=${LIBRARY:-'@libmakepkgdir@'} @@ -235,7 +237,7 @@ run_pacman() { else cmd=(su root -c "$(printf '%q ' "${cmd[@]}")") fi - local lockfile="$(pacman-conf DBPath)/db.lck" + local lockfile="$(${PACMAN_CONF_INSTALL_NAME} DBPath)/db.lck" while [[ -f $lockfile ]]; do local timer=0 msg "$(gettext "Pacman is currently in use, please wait...")" @@ -1123,7 +1125,7 @@ unset var PACKAGER=${PACKAGER:-"Unknown Packager"} # set pacman command if not already defined -PACMAN=${PACMAN:-pacman} +PACMAN=${PACMAN:-${PACMAN_INSTALL_NAME}} # save full path to command as PATH may change when sourcing /etc/profile PACMAN_PATH=$(type -P $PACMAN) -- 2.20.1

1 0

[pacman-dev] Showing optional dependencies in pactree output
by Frederick Zhang 04 May '20

04 May '20

Hi all, I noticed that there is a patch by Johannes available at [1] which adds a new flag to pactree to show optional dependencies as well. I think this is a nice feature to have however it seems that the patch was never merged. I just tried it out locally and managed to compile with a few minor changes. If it wasn't rejected for some reason guess I could submit a patch if needed? [1] https://lists.archlinux.org/pipermail/pacman-dev/2016-February/020922.html -- Best regards, Frederick Zhang Email: frederick888(a)tsundere.moe PGP: 8BFB EA5B 4C44 BFAC C8EC 5F93 1F92 8BE6 0D8B C11D

2 1

[pacman-dev] [PATCH] Add makedepends to -Si/-Qi output
by Carson Black 04 May '20

04 May '20

Fairly self explanatory patch. pacman didn't show makedepends, now it does. Signed-off-by: Carson Black <uhhadd(a)gmail.com> --- src/pacman/package.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/pacman/package.c b/src/pacman/package.c index 4c93a443..c3c3dbf6 100644 --- a/src/pacman/package.c +++ b/src/pacman/package.c @@ -70,6 +70,7 @@ enum { T_URL, T_VALIDATED_BY, T_VERSION, + T_BUILD_DEPENDS_ON, /* the following is a sentinel and should remain in last position */ _T_MAX }; @@ -124,6 +125,7 @@ static void make_aligned_titles(void) buf[T_URL] = _("URL"); buf[T_VALIDATED_BY] = _("Validated By"); buf[T_VERSION] = _("Version"); + buf[T_BUILD_DEPENDS_ON] = _("Build Depends On"); for(i = 0; i < ARRAYSIZE(wbuf); i++) { wlen[i] = mbstowcs(wbuf[i], buf[i], strlen(buf[i]) + 1); @@ -271,6 +273,7 @@ void dump_pkg_full(alpm_pkg_t *pkg, int extra) list_display(titles[T_GROUPS], alpm_pkg_get_groups(pkg), cols); deplist_display(titles[T_PROVIDES], alpm_pkg_get_provides(pkg), cols); deplist_display(titles[T_DEPENDS_ON], alpm_pkg_get_depends(pkg), cols); + deplist_display(titles[T_BUILD_DEPENDS_ON], alpm_pkg_get_makedepends(pkg), cols); optdeplist_display(pkg, cols); if(extra || from == ALPM_PKG_FROM_LOCALDB) { -- 2.26.2

2 1

[pacman-dev] [PATCH] meson: handle XFAIL tests outside of TAP
by Dave Reisner 02 May '20

02 May '20

This change causes expected fail tests to actually fail by eliding the '# TODO' from the test plan. In turn, we can now properly use 'should_fail' in the meson test() rule and see these expected fail tests in the output: Before: ... 320/332 upgrade077.py OK 0.12679290771484375 s 321/332 upgrade078.py OK 0.12620115280151367 s 322/332 upgrade080.py OK 0.1252129077911377 s ... Ok: 332 Expected Fail: 0 Fail: 0 Unexpected Pass: 0 Skipped: 0 Timeout: 0 After: ... 320/332 upgrade077.py OK 0.12679290771484375 s 321/332 upgrade078.py EXPECTEDFAIL0.12620115280151367 s 322/332 upgrade080.py OK 0.1252129077911377 s ... Ok: 326 Expected Fail: 6 Fail: 0 Unexpected Pass: 0 Skipped: 0 Timeout: 0 --- test/pacman/meson.build | 15 ++++++++++++++- test/pacman/pmenv.py | 7 ++++++- 2 files changed, 20 insertions(+), 2 deletions(-) diff --git a/test/pacman/meson.build b/test/pacman/meson.build index 9de4f5a1..fd0dbf90 100644 --- a/test/pacman/meson.build +++ b/test/pacman/meson.build @@ -330,6 +330,17 @@ pacman_tests = [ 'tests/xfercommand001.py', ] +xfail_tests = { + 'tests/deprange001.py': true, + # expect failure on 32 bit machines + 'tests/query006.py': cc.sizeof('ssize_t') < 8, + 'tests/replace110.py': true, + 'tests/sync-update-package-removing-required-provides.py': true, + 'tests/sync403.py': true, + 'tests/sync406.py': true, + 'tests/upgrade078.py': true, +} + foreach input : pacman_tests test_name = input.split('/')[1] args = [ @@ -351,7 +362,9 @@ foreach input : pacman_tests test_name, PYTHON, protocol : 'tap', + env : ['RUNNING_UNDER_MESON=1'], args : args, - timeout: 120, + timeout : 120, + should_fail : xfail_tests.get(input, false), depends : [pacman_bin]) endforeach diff --git a/test/pacman/pmenv.py b/test/pacman/pmenv.py index 24437b61..d87193f2 100644 --- a/test/pacman/pmenv.py +++ b/test/pacman/pmenv.py @@ -70,5 +70,10 @@ def run(self): t.run(self.pacman) tap.diag("==> Checking rules") - tap.todo = t.expectfailure + # When running under meson, we don't emit 'todo' in the plan and instead + # handle expected failures in the test() objects. This really should be + # fixed in meson: + # https://github.com/mesonbuild/meson/issues/2923#issuecomment-614647076 + tap.todo = (t.expectfailure and + not 'RUNNING_UNDER_MESON' in os.environ) tap.subtest(lambda: t.check(), t.description) -- 2.26.2

1 0

[pacman-dev] [PATCH v2] Add Gitlab Runner configuration
by Allan McRae 02 May '20

02 May '20

Arch Linux is setting up a Gitlab instance. This adds CI for the pacman project, testing a range of configurations on Arch and basic builds on Fedora and Debian. Note that asciidoc is specifically not installed on the Debian run because it is all sorts of broken... Also, the defaults have been set to meson, with two autotools tests that will soon be removed. Original-file from: Andrew Gregory <andrew.gregory.8(a)gmail.com> Altered-to-run-on-Arch-Gitlab by: Sven-Hendrik Haase <svenstaro(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> --- I should actually send the updated version... .gitlab-ci.yml | 128 +++++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 128 insertions(+) create mode 100644 .gitlab-ci.yml diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml new file mode 100644 index 00000000..4961352e --- /dev/null +++ b/.gitlab-ci.yml @@ -0,0 +1,128 @@ +variables: + MAKEFLAGS: "-j10" + VERBOSE: 1 + +.arch-test: + image: archlinux/base + before_script: + - > + pacman -Syu --needed --noconfirm + base-devel git + gpgme libarchive curl + python + fakeroot fakechroot + meson + +arch: + extends: .arch-test + script: + - meson build + - ninja -C build + - fakechroot meson test -C build + +arch-debug: + extends: .arch-test + script: + - meson --buildtype=debug build + - ninja -C build + - fakechroot meson test -C build + +arch-docs: + extends: .arch-test + script: + - pacman -Syu --needed --noconfirm asciidoc + - meson -Ddoc=enabled build + - ninja -C build + +arch-autoconf: + extends: .arch-test + script: + - pacman -Syu --needed --noconfirm autoconf-archive + - autoreconf -i + - ./configure --enable-warningflags --disable-doc + - make + - fakechroot make check + +arch-distcheck: + extends: .arch-test + script: + - pacman -Syu --needed --noconfirm autoconf-archive asciidoc + - autoreconf -i + - ./configure --enable-warningflags + - fakechroot make distcheck + +arch-clang: + extends: .arch-test + script: + - pacman -Syu --needed --noconfirm clang + - CC=clang meson build + - ninja -C build + - fakechroot meson test -C build + +arch-valgrind: + extends: .arch-test + script: + - pacman -Syu --needed --noconfirm autoconf-archive valgrind + - autoreconf -i + - ./configure --enable-warningflags --disable-doc + - make + - fakechroot make PY_LOG_FLAGS=--valgrind check + +arch-nettle: + extends: .arch-test + script: + - meson -Dcrypto=nettle --buildtype=debug build + - ninja -C build + - fakechroot meson test -C build + +arch-no-gpg: + extends: .arch-test + script: + - meson -Dgpgme=disabled --buildtype=debug build + - ninja -C build + - fakechroot meson test -C build + +arch-no-curl: + extends: .arch-test + script: + - meson -Dcurl=disabled --buildtype=debug build + - ninja -C build + - fakechroot meson test -C build + +arch-no-nls: + extends: .arch-test + script: + - meson -Di18n=false --buildtype=debug build + - ninja -C build + - fakechroot meson test -C build + +debian: + image: debian:bullseye + before_script: + - apt update + - > + apt -y install --no-install-recommends + git pkg-config meson gcc libtool + libgpgme-dev libarchive-dev libcurl4-openssl-dev libssl-dev curl + gettext python3 python3-setuptools dash gawk ca-certificates + fakeroot fakechroot + script: + - meson --buildtype=debug build + - ninja -C build + - fakechroot meson test -C build + +fedora: + image: fedora + before_script: + - > + dnf -y install + git findutils patch sed + meson gcc libtool + gpgme-devel libarchive-devel libcurl-devel openssl-devel gettext-devel + asciidoc python3 dash gawk + fakeroot fakechroot + perl-Module-Load-Conditional + script: + - meson --buildtype=debug build + - ninja -C build + - fakechroot meson test -C build -- 2.25.1

1 0

[pacman-dev] [PATCH] Add Gitlab Runner configuration
by Allan McRae 02 May '20

02 May '20

Arch Linux is setting up a Gitlab instance. This adds CI for the pacman project, testing a range of configurations on Arch and basic builds on Fedora and Debian. Note that asciidoc is specifically not installed on the Debian run because it is all sorts of broken... Also, the defaults have been set to meson, with two autotools tests that will soon be removed. Original-file from: Andrew Gregory <andrew.gregory.8(a)gmail.com> Altered-to-run-on-Arch-Gitlab by: Sven-Hendrik Haase <svenstaro(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> --- .gitlab-ci.yml | 116 +++++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 116 insertions(+) create mode 100644 .gitlab-ci.yml diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml new file mode 100644 index 00000000..5979fa61 --- /dev/null +++ b/.gitlab-ci.yml @@ -0,0 +1,116 @@ +variables: + MAKEFLAGS: "-j10" + VERBOSE: 1 + +.arch-test: + image: archlinux/base + before_script: + - > + pacman -Syu --needed --noconfirm + base-devel git + gpgme libarchive curl + asciidoc python + fakeroot fakechroot autoconf-archive + +arch-default: + extends: .arch-test + script: + - autoreconf -i + - ./configure --enable-warningflags + - make + - fakechroot make check + +arch-clang: + extends: .arch-test + script: + - pacman -Syu --needed --noconfirm clang + - autoreconf -i + - CC=clang ./configure --enable-warningflags + - make + - fakechroot make check + +arch-meson: + extends: .arch-test + script: + - pacman -Syu --needed --noconfirm meson + - meson build + - ninja -C build + - fakechroot meson test -C build + +arch-distcheck: + extends: .arch-test + script: + - autoreconf -i + - ./configure --enable-warningflags + - fakechroot make distcheck + +arch-valgrind: + extends: .arch-test + script: + - pacman -Syu --needed --noconfirm valgrind + - autoreconf -i + - ./configure --enable-warningflags + - make + - fakechroot make PY_LOG_FLAGS=--valgrind check + +arch-no-gpg: + extends: .arch-test + script: + - autoreconf -i + - ./configure --enable-warningflags --without-gpgme + - make + - fakechroot make check + +arch-no-curl: + extends: .arch-test + script: + - autoreconf -i + - ./configure --enable-warningflags --without-libcurl + - make + - fakechroot make check + +arch-no-nls: + extends: .arch-test + script: + - autoreconf -i + - ./configure --enable-warningflags --disable-nls + - make + - fakechroot make check + +debian-test: + image: debian + before_script: + - apt update + - > + apt -y install --no-install-recommends + git make autoconf autopoint automake pkg-config patch gcc libtool + libgpgme-dev libarchive-dev libcurl4-openssl-dev libssl-dev curl + gettext python3 dash gawk ca-certificates + asciidoc docbook-xsl xsltproc libxml2-utils + fakeroot fakechroot autoconf-archive + script: + - autoreconf -i + - ./configure --enable-warningflags + - make + - fakechroot make check + +debian-gcc: + extends: debian-test + image: gcc + +fedora: + image: fedora + before_script: + - > + dnf -y install + git findutils patch sed + make automake autoconf gcc libtool + gpgme-devel libarchive-devel libcurl-devel openssl-devel gettext-devel + asciidoc python3 dash gawk + fakeroot fakechroot autoconf-archive + perl-Module-Load-Conditional + script: + - autoreconf -i + - ./configure --enable-warningflags + - make + - fakechroot make check -- 2.25.1

1 0