[arch-projects] [devtools] [GIT] The official devtools repository branch master updated. 20171108-9-g38c7a39
BartÅomiej Piotrowski
bpiotrowski at archlinux.org
Tue Jan 23 12:43:10 UTC 2018
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
via 38c7a391b043547b946a99731a56a233458ba7a2 (commit)
via 75ad2aca572d36c4df1b07153e2403534fbd89ed (commit)
via aee72cae3231811058d6993d71f97e449e477d8f (commit)
via 5ab8f8430a2c865656cd800c03ac21cb17a86d3c (commit)
via c9e287e845973e4c04e21fc513caaea4907e604e (commit)
via 48b2f8dcc49d88e60bd9e4ee97c92fea8ac90721 (commit)
via 3b725b58434b92e93ff90164fae8d76c4761706a (commit)
from 7a3c5085017987b6ef934cf9d9b098b4994ba21a (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 38c7a391b043547b946a99731a56a233458ba7a2
Author: Bartłomiej Piotrowski <bpiotrowski at archlinux.org>
Date: Sun Jan 21 14:18:43 2018 +0100
makechrootpkg: make sure that makepkg.conf is always parsed as text
commit 75ad2aca572d36c4df1b07153e2403534fbd89ed
Author: Luke Shumaker <lukeshu at parabola.nu>
Date: Mon Jan 15 17:57:00 2018 +0100
makechrootpkg: Adjust to work properly with `set -e`
This worked properly until eab5aba.
commit aee72cae3231811058d6993d71f97e449e477d8f
Author: Luke Shumaker <lukeshu at parabola.nu>
Date: Mon Jan 15 17:57:00 2018 +0100
lib/common.sh: Adjust to work properly with `set -u`
Support for working with `set -u` was broken by 94160d6. Egg on my
face; I'm the one who wants `set -u` support, and I'm the author of
that commit!
libmakepkg does not work with `set -u`; but mostly because of the include
guards! So we just need to temporarily disable `set -u` (nounset) while
loading libmakepkg. Instead of introducing a new variable, just store the
initial nounset status in _INCLUDE_COMMON_SH; rather than a useless
fixed-string "true".
While we're at it, disable POSIX-mode (just in case we're running as "sh"
instead of "bash"), since libmakepkg uses bash-isms that won't parse in
POSIX mode.
commit 5ab8f8430a2c865656cd800c03ac21cb17a86d3c
Author: Luke Shumaker <lukeshu at parabola.nu>
Date: Mon Jan 15 17:57:00 2018 +0100
arch-nspawn: Remove pointless $(echo ...) subshell
commit c9e287e845973e4c04e21fc513caaea4907e604e
Author: Luke Shumaker <lukeshu at parabola.nu>
Date: Mon Jan 15 17:57:00 2018 +0100
arch-nspawn: make sure that makepkg.conf is always parsed as text
https://lists.parabola.nu/pipermail/dev/2017-June/005576.html
commit 48b2f8dcc49d88e60bd9e4ee97c92fea8ac90721
Author: Eli Schwartz <eschwartz at archlinux.org>
Date: Sun Sep 3 03:53:40 2017 -0400
makechrootpkg: Fix anti-pattern when checking for enabled features
Don't use error-prone logic e.g.
foo=true; if $foo ...
This completely fails to act as expected when the variable is unset
because of unrelated bugs.
While this merely causes the default behavior to be "false" rather than
"true" in such cases, it is better to fail to enable explicitly
requested behavior (which will be noticed by the user) than to simply
upgrade to this behavior for free (which may not seem to have any
obvious cause).
Signed-off-by: Eli Schwartz <eschwartz at archlinux.org>
commit 3b725b58434b92e93ff90164fae8d76c4761706a
Author: Eli Schwartz <eschwartz at archlinux.org>
Date: Sun Sep 3 03:53:39 2017 -0400
makechrootpkg: Fix unconditionally running namcap
Fixes regression in 2fd5931a8c67289a8a4acd327b3ce99a5d64c8c7
$run_namcap will always be set to ""
`if $not_a_var; then ...; fi` is always truthful when $not_a_var is
unset or equal to "" and the `then` clause will always be run.
I'm not sure why global state variables need to be cloned locally for
their sole explicit purpose.
But for now this patch implements the minimum necessary work to properly
pass the "do I want namcap" variable into prepare_chroot() according to
the current logic flow.
Note that I have still not thorougly tested makechrootpkg.
Signed-off-by: Eli Schwartz <eschwartz at archlinux.org>
-----------------------------------------------------------------------
Summary of changes:
arch-nspawn.in | 4 ++--
lib/common.sh | 4 +++-
makechrootpkg.in | 10 +++++-----
3 files changed, 10 insertions(+), 8 deletions(-)
hooks/post-receive
--
The official devtools repository
More information about the arch-projects
mailing list