Arch-projects March 2018

arch-projects@lists.archlinux.org

4 participants
14 discussions

[arch-projects] [mkinitcpio] [PATCH] --rsyncable option
by Charles Duffy 15 Dec '18

15 Dec '18

At present, mkinitcpio outputs are not particularly stable: Generating two consecutive initramfs images from the same inputs can generate archives which are quite entirely unlike each other from the perspective of binary diff tools (rsync, casync, rdiff-backup, xdelta, etc). Making some adjustments to compression and archival can greatly reduce the size of this delta. A ticket with a proposed patch (adding a --rsyncable option to mkinitcpio) has been opened at https://bugs.archlinux.org/task/56128 -- however, as the advice there has been to consult this mailing list, it may no longer be open as of this writing. :) The change consists of the following: - Sort the list of files (in the C locale) before passing it to cpio. - When --rsyncable is passed, set all datestamps to the UNIX epoch before generating the cpio archive. - When --rsyncable is passed, pass the --reproducible option to cpio (this creates a dependency on GNU cpio 2.12, prevents device numbers from being included in the archive, and renumbers inode numbers to be consistent when file ordering is unchanged). - When --rsyncable is passed, pass the --rsyncable and --no-name options to gzip (the former tells it to reset the compression table on occasion, sacrificing compression ratio to prevent the effect from changes from cascading throughout too much of the stream; the latter prevents name and timestamp data from being stored in the gzip header itself). A version of the patch squashing together the two commits already present on the ticket is attached.

1 1

[arch-projects] [devtools] [PATCH v2] makechrootpkg: respect GNUPGHOME
by Emiel Wiedijk 31 May '18

31 May '18

Previously, makechrootpkg hardcoded ~/.gnupg. Therefore, if a user uses a custom GPG home directory, the siganture checking would fail. Now makechrootpkg uses $GNUPGHOME, with a fallback to ~/.gnupg. Signed-off-by: Emiel Wiedijk <me(a)aimileus.nl> --- makechrootpkg.in | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/makechrootpkg.in b/makechrootpkg.in index afcd121..653847f 100644 --- a/makechrootpkg.in +++ b/makechrootpkg.in @@ -252,7 +252,8 @@ download_sources() { chmod 1777 "$builddir" # Ensure sources are downloaded - sudo -u "$makepkg_user" env SRCDEST="$SRCDEST" BUILDDIR="$builddir" \ + sudo -u "$makepkg_user" --preserve-env=GNUPGHOME \ + env SRCDEST="$SRCDEST" BUILDDIR="$builddir" \ makepkg --config="$copydir/etc/makepkg.conf" --verifysource -o || die "Could not download sources." @@ -341,7 +342,7 @@ main() { [[ -n $makepkg_user && -z $(id -u "$makepkg_user") ]] && die 'Invalid makepkg user.' makepkg_user=${makepkg_user:-${SUDO_USER:-$USER}} - check_root SOURCE_DATE_EPOCH + check_root SOURCE_DATE_EPOCH,GNUPGHOME # Canonicalize chrootdir, getting rid of trailing / chrootdir=$(readlink -e "$passeddir") -- 2.16.2

2 1

[arch-projects] [PATCH] [namcap] Treat all non-tar files as PKGBUILD files, regardless of filename
by Luke Shumaker 16 Apr '18

16 Apr '18

From: Luke Shumaker <lukeshu(a)parabola.nu> https://bugs.archlinux.org/task/57193 If the user mistakenly passes in some other (non-package, non-PKGBUILD) file, instead of refusing to process it (saying "Error: Cannot process ${filename}"), it will now try to parse it as a PKGBUILD, probably encounter and print at least one error, but finally say "Error: ${filename} is not a valid PKGBUILD". This is actually probably *more* helpful to a confused user who passed in the wrong file. --- namcap.py | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/namcap.py b/namcap.py index b62a2fa..fd62d3c 100755 --- a/namcap.py +++ b/namcap.py @@ -248,9 +248,7 @@ for package in packages: if os.path.isfile(package) and tarfile.is_tarfile(package): process_realpackage(package, active_modules) - elif package.endswith('PKGBUILD'): + else: process_pkgbuild(package, active_modules) - else: - print("Error: Cannot process %s" % package) # vim: set ts=4 sw=4 noet: -- 2.15.1

3 3

[arch-projects] [devtools] [PATCH] makechrootpkg: fix verifysource with pacman-git
by Eli Schwartz 26 Mar '18

26 Mar '18

In pacman-git commit d8717a6a9666ec80c8645d190d6f9c7ab73084ac makepkg started checking that the setuid/setgid bit could be removed on the $BUILDDIR in order to prevent this propagating to the packages themselves. Unfortunately, this requires the temporary builddir used during the --verifysource stage of makepkg, to be owned by $makepkg_user which was not the case as it is created as root using mktemp (and given world rwx in addition to the restricted deletion bit.) Obviously makepkg cannot chmod a directory that it does not own. Fix this by making $makepkg_user the owner of that directory, as should have been the case all along. (Giving world rwx is illogical on general principle. The fact that this is a workaround for makepkg demanding these directories be writable even when they are not going to be used for the makepkg options in question, is not justification for being careless.) Signed-off-by: Eli Schwartz <eschwartz(a)archlinux.org> --- Yay, I "broke" something. :D makechrootpkg.in | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/makechrootpkg.in b/makechrootpkg.in index afcd121..6bc82a4 100644 --- a/makechrootpkg.in +++ b/makechrootpkg.in @@ -249,7 +249,7 @@ download_sources() { local builddir builddir="$(mktemp -d)" - chmod 1777 "$builddir" + chown "$makepkg_user:$makepkg_user" "$builddir" # Ensure sources are downloaded sudo -u "$makepkg_user" env SRCDEST="$SRCDEST" BUILDDIR="$builddir" \ -- 2.16.2

2 2

[arch-projects] [devtools] [GIT] The official devtools repository branch master updated. 20171108-12-g40f0179
by Bartłomiej Piotrowski 24 Mar '18

24 Mar '18

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 devtools repository". The branch, master has been updated discards 6da90b2028da0bc504be7ff60873dd2719818a1e (commit) via 40f0179a5e74d6d3babbefdeae21fd374be0e090 (commit) via ffb5003fdacaece3540ba167f7e965a122133af0 (commit) This update added new revisions after undoing existing revisions. That is to say, the old revision is not a strict subset of the new revision. This situation occurs when you --force push a change and generate a repository containing something like this: * -- * -- B -- O -- O -- O (6da90b2028da0bc504be7ff60873dd2719818a1e) \ N -- N -- N (40f0179a5e74d6d3babbefdeae21fd374be0e090) When this happens we assume that you've already had alert emails for all of the O revisions, and so we here report only the revisions in the N branch from the common base, B. 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 40f0179a5e74d6d3babbefdeae21fd374be0e090 Author: Eli Schwartz <eschwartz(a)archlinux.org> Date: Sun Mar 18 01:46:44 2018 -0400 makechrootpkg: fix verifysource with pacman-git In pacman-git commit d8717a6a9666ec80c8645d190d6f9c7ab73084ac makepkg started checking that the setuid/setgid bit could be removed on the $BUILDDIR in order to prevent this propagating to the packages themselves. Unfortunately, this requires the temporary builddir used during the --verifysource stage of makepkg, to be owned by $makepkg_user which was not the case as it is created as root using mktemp (and given world rwx in addition to the restricted deletion bit.) Obviously makepkg cannot chmod a directory that it does not own. Fix this by making $makepkg_user the owner of that directory, as should have been the case all along. (Giving world rwx is illogical on general principle. The fact that this is a workaround for makepkg demanding these directories be writable even when they are not going to be used for the makepkg options in question, is not justification for being careless.) Signed-off-by: Eli Schwartz <eschwartz(a)archlinux.org> commit ffb5003fdacaece3540ba167f7e965a122133af0 Author: Emiel Wiedijk <me(a)aimileus.nl> Date: Tue Feb 27 11:16:24 2018 -0500 makechrootpkg: respect GNUPGHOME Previously, makechrootpkg hardcoded ~/.gnupg. Therefore, if a user uses a custom GPG home directory, the siganture checking would fail. Now makechrootpkg uses $GNUPGHOME, with a fallback to ~/.gnupg. Signed-off-by: Emiel Wiedijk <me(a)aimileus.nl> ----------------------------------------------------------------------- Summary of changes: makechrootpkg.in | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) hooks/post-receive -- The official devtools repository

1 0

[arch-projects] [devtools] [GIT] The official devtools repository branch master updated. 20171108-11-g6da90b2
by Bartłomiej Piotrowski 24 Mar '18

24 Mar '18

1 0

[arch-projects] [netctl] [GIT] Network control utility based on systemd annotated tag 1.16 created. 1.16
by Jouke Witteveen 22 Mar '18

22 Mar '18

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 "Network control utility based on systemd". The annotated tag, 1.16 has been created at 82dffc523922a470b222198e49bb2f5ef384ef75 (tag) tagging dbc32d470420396f4af360af9ac17680fe99b368 (commit) replaces 1.15 tagged by Jouke Witteveen on Thu Mar 22 10:54:04 2018 +0100 - Log ----------------------------------------------------------------- Release 1.16 -----BEGIN PGP SIGNATURE----- iQFKBAABCAA0FiEEbql9KtO+F66lkufaVrYfv4JJLBUFAlqzfUEWHGoud2l0dGV2 ZWVuQGdtYWlsLmNvbQAKCRBWth+/gkksFU4vCADmnNsbgSnLiZso5sYcTzg8XhSV I1BIG9PfeDVXuDekpGQMhhCh0+GUyuPE3Pxsem07m6WZJ+O/89QGmisLggb23t7X P1/l97Y8AeSODhPIXAm7f3G1CWa1ciXyxEpFe3PACmajIE6CkTbTko4X2bi3ZYN4 8fSQTeOAfaTpEaaGgbjqRYHj7nlot+qbo1l/pApwv69ey4TKHItjN7umw6S43rvl 7f4NVZhWtulINlFNgFnsGoD9mW3k6mVKBHun6HUTfCfX5w5+O0SccazoKwanEEEL GSxl7CUw/6G3AmGRFL0Yg6Ncqf4vWcLMVRL784ZQkctiXfiGKVTOrKsmIJTg =eByd -----END PGP SIGNATURE----- Erich Eckner (1): Fix quoting of After= in generated service files (FS#57850) Jouke Witteveen (4): Technical documentation fixes Only wait when necessary in PPP cleanup code Reference profile options with a '='-suffix 1.16 release updates ----------------------------------------------------------------------- hooks/post-receive -- Network control utility based on systemd

1 0

[arch-projects] [netctl] [GIT] Network control utility based on systemd branch master updated. 1.15-5-gdbc32d4
by Jouke Witteveen 22 Mar '18

22 Mar '18

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 "Network control utility based on systemd". The branch, master has been updated via dbc32d470420396f4af360af9ac17680fe99b368 (commit) via 6d3644be38aa9d273cdc0989404804dad8818844 (commit) from cef231d762e7bb65bcf9a6a040ba367c5062a8f8 (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 dbc32d470420396f4af360af9ac17680fe99b368 Author: Jouke Witteveen <j.witteveen(a)gmail.com> Date: Thu Mar 22 10:47:26 2018 +0100 1.16 release updates commit 6d3644be38aa9d273cdc0989404804dad8818844 Author: Jouke Witteveen <j.witteveen(a)gmail.com> Date: Thu Mar 22 10:46:20 2018 +0100 Reference profile options with a '='-suffix ----------------------------------------------------------------------- Summary of changes: Makefile | 4 +-- NEWS | 4 +++ docs/netctl.profile.5.txt | 65 ++++++++++++++++++++++++----------------------- 3 files changed, 39 insertions(+), 34 deletions(-) hooks/post-receive -- Network control utility based on systemd

1 0

[arch-projects] [dbscripts] [GIT] Official repo DB scripts branch pacman-git created. 20131102-82-gcda3694
by eschwartz＠archlinux.org 22 Mar '18

22 Mar '18

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 "Official repo DB scripts". The branch, pacman-git has been created at cda3694d0b247849e12edcd56e0ca7a41ec90ae1 (commit) - Log ----------------------------------------------------------------- commit cda3694d0b247849e12edcd56e0ca7a41ec90ae1 Author: Eli Schwartz <eschwartz(a)archlinux.org> Date: Wed Mar 21 12:09:21 2018 -0400 pacman 5.1 compatibility The print_all_package_names function changed its behavior, adapt ----------------------------------------------------------------------- hooks/post-receive -- Official repo DB scripts

1 0

[arch-projects] [dbscripts] [GIT] Official repo DB scripts branch master updated. 20131102-81-g8cc8e9c
by eschwartz＠archlinux.org 22 Mar '18

22 Mar '18

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 "Official repo DB scripts". The branch, master has been updated via 8cc8e9cbbefdf6fac368031d3dacc74d001f23b7 (commit) via 0432cffc42040f852b9a55c27316248da9064b67 (commit) via 36087fbd8b030fa6f908fdbb667292e3c078b615 (commit) via 23c2b82c336bf19b7a29a90d19bca4423d8b8839 (commit) via 97e17a5996425e8b7c3c1765a3c5074e1c4ff38d (commit) via 9672d6ec64ff5773df4b22bca9f2da44303dc03f (commit) via 41333413f19e025d800e02c83511b85bee65ac60 (commit) via 5c867ea3b857cba0993fc18e064c6f02f70ee099 (commit) from da49ea616e162d3cfcb4b6a4242f497e94c36564 (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 8cc8e9cbbefdf6fac368031d3dacc74d001f23b7 Author: Luke Shumaker <lukeshu(a)parabola.nu> Date: Wed Jun 18 20:33:06 2014 -0400 Clean up printf-formatters for user messages - db-functions: getpkgfile: The .sig file extension should be part of the filename parameter, not part of the message format string. - db-functions: arch_repo_modify: Shouldn't use ${action} string-interpolation in the message format string. Since the entire message is a command, and we're using @Q to escape arguments anyway, go ahead and just construct the entire command as a single string that way, and feed it to '%s'. commit 0432cffc42040f852b9a55c27316248da9064b67 Author: Luke Shumaker <lukeshu(a)parabola.nu> Date: Wed Jun 18 12:07:09 2014 -0400 Fixups near unquoted variables Using the following command to find unquoted variables (and ignoring more than a a few false positives), grep -Prn --exclude-dir=.git '(?<!["=]|\[\[ |\[\[ -[zn] )\$(?!{?#|\(|\? )' one is lead to find a few cleanups that are something other than "add double-quotes". That's what these are. We'll leave dumb adding of double-quotes for another commit. Most of these are still fixing quoting issues, just with a better fix. - parse_pkgbuilds.sh: Avoid having to escape quotes in `eval` strings by using `declare -p`. Updates the logic copied from makepkg, with the latest logic copied from makepkg. See https://git.archlinux.org/pacman.git/commit/?id=9e52a36794552b77ecf26f7f34b… - sourceballs: Avoid using ary=($string) to do field separation by using `read` and test that multiple licenses actually work as expected. - sourceballs: Replace `[[ -z ${ary[*]} ]]` with test for the array length - db-functions: Replace mangling echo field separators using sed, with printf formatters - db-functions: Replace for/echo loop to print an array line by line, with `printf '%s\n'` - db-functions: set_repo_permissions: Line up error messages, quote "$group" - db-move: Replace `$(echo ${array[@]})` with `${array[*]}` - testing2x: Use `"$@"` instead of `$*` when looping over an array Also, not really quoting related but on the same line as a quoting issue, optimize: - db-functions: Replace [[ -n "$(... | sort | uniq -D)" ]] with ! ... | awk 'a[$0]++{exit 1}' commit 36087fbd8b030fa6f908fdbb667292e3c078b615 Author: Luke Shumaker <lukeshu(a)parabola.nu> Date: Wed Mar 14 16:52:05 2018 -0400 Remove uses of the "v=true; if $v ..." anti-pattern Instead, compare the value of $v to 'true'. commit 23c2b82c336bf19b7a29a90d19bca4423d8b8839 Author: Luke Shumaker <lukeshu(a)parabola.nu> Date: Sun Apr 17 12:18:02 2016 -0400 Don't use `grep -q` when operating on piped stdin (By default, prefer `grep &>/dev/null`) `grep -q` may exit as soon as it finds a match; this is a good optimization for when the input is a file. However, if the input is the output of another program, then that other program will receive SIGPIPE, and further writes will fail. When this happens, it might (bsdtar does) print a message about a "write error" to stderr. Which is going to confuse and alarm the user. In one of the cases (in common.bash, in the test suite), this had already been mitigated by wrapping bsdtar in "echo "$(bsdtar ...)", as Bash builtin echo doesn't complain if it gets SIGPIPE. However, that means we're storing the entire output of bsdtar in memory, which is silly. Additionally, the way it was implemented is also wrong; because it was being used with `grep -qv` instead of just `grep -q`, it *always* found a non-matching line (even something inconsequential like `%NAME%`), and *never* triggered a test failure. Looking at a few of these cases, it might also make sense to switch to using `bsdtar tf` instead of `bsdtar xf` when checking membership, but that's work for another day. commit 97e17a5996425e8b7c3c1765a3c5074e1c4ff38d Author: Luke Shumaker <lukeshu(a)parabola.nu> Date: Sun Apr 17 02:07:26 2016 -0400 Add "#!/hint/bash" to the beginning of several files It is a method of notifying text-editors that a file is in Bash syntax without giving it a propper shebang (which would be confusing, as it would suggest that the file should be executable), as well as working across virtually all text-editors (unlike "-*- Mode: Bash -*-" or whatever). commit 9672d6ec64ff5773df4b22bca9f2da44303dc03f Author: Luke Shumaker <lukeshu(a)parabola.nu> Date: Wed Jun 18 21:04:02 2014 -0400 Normalize to tab indent commit 41333413f19e025d800e02c83511b85bee65ac60 Author: Luke Shumaker <lukeshu(a)parabola.nu> Date: Fri Jan 17 13:06:53 2014 -0500 Export TMPDIR, and use mktemp -t instead of making it part of the template commit 5c867ea3b857cba0993fc18e064c6f02f70ee099 Author: Eli Schwartz <eschwartz(a)archlinux.org> Date: Fri Mar 16 00:14:17 2018 -0400 fixup! Update messages to make fuller use of printf formatters pkgs should be passed as a single argument ----------------------------------------------------------------------- Summary of changes: config | 4 +++- config.local.svn-community | 2 ++ config.local.svn-packages | 2 ++ cron-jobs/check_archlinux/parse_pkgbuilds.sh | 31 +++++++++------------------- cron-jobs/devlist-mailer | 6 +++--- cron-jobs/ftpdir-cleanup | 6 +++--- cron-jobs/sourceballs | 16 ++++++-------- db-functions | 30 +++++++++++---------------- test/fixtures/pkg-simple-a/PKGBUILD | 2 +- test/lib/common.bash | 6 ++++-- testing2x | 8 +++---- 11 files changed, 50 insertions(+), 63 deletions(-) hooks/post-receive -- Official repo DB scripts

1 0

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

Arch-projects March 2018