FreeBSD/poudriere 000a75a — src/share/poudriere/include hash.sh
hash_isset_var: mapfile can break since specializing /dev/stdin
Delta | File | |
---|---|---|
+5 | -9 | src/share/poudriere/include/hash.sh |
+5 | -9 | 1 files |
hash_isset_var: mapfile can break since specializing /dev/stdin
Delta | File | |
---|---|---|
+5 | -9 | src/share/poudriere/include/hash.sh |
+5 | -9 | 1 files |
Add stack_isset()
Delta | File | |
---|---|---|
+16 | -0 | test/stack.sh |
+8 | -0 | src/share/poudriere/include/hash.sh |
+24 | -0 | 2 files |
eargs: Support showing passed in vars Requires adding to the end of the eargs line: EARGS: "$@" This passes in the positional parameters which allows seeing them properly grouped. # ./poudriere api 'hash_get' Error: hash_get: 3 arguments expected, got 0: Expected: hash_get var key var_return Missing: var key var_return # ./poudriere api 'hash_get 1' Error: hash_get: 3 arguments expected, got 1: Expected: hash_get var key var_return Received: hash_get "1" Missing: key var_return # ./poudriere api 'hash_get 1 2' Error: hash_get: 3 arguments expected, got 2: Expected: hash_get var key var_return [6 lines not shown]
Delta | File | |
---|---|---|
+102 | -12 | src/share/poudriere/common.sh |
+1 | -1 | src/share/poudriere/include/hash.sh |
+103 | -13 | 2 files |
issetvar(): Rename to more consistent 'isset'
Delta | File | |
---|---|---|
+12 | -8 | src/share/poudriere/include/util.sh |
+4 | -4 | src/share/poudriere/include/hash.sh |
+4 | -4 | test/stack.sh |
+2 | -2 | src/poudriere-sh/helpers.c |
+2 | -2 | src/share/poudriere/common.sh |
+2 | -2 | test/getvar.sh |
+26 | -22 | 1 files not shown |
+27 | -23 | 7 files |
setvar can fail, return error if so
Avoid || return on last line. This may change behavior since || disables set -e. The caller can control this if it cares about the error.
Delta | File | |
---|---|---|
+6 | -7 | src/share/poudriere/include/hash.sh |
+6 | -7 | 1 files |
Fix some getvar/setvar functions to avoid shadowing setvar
Delta | File | |
---|---|---|
+262 | -255 | src/share/poudriere/include/util.sh |
+78 | -78 | src/share/poudriere/include/hash.sh |
+340 | -333 | 2 files |
gsub sh: Fix special case of replacing '*' This fixes an infinite loop in _msg_fmt_n() when aligning build summary.
Delta | File | |
---|---|---|
+6 | -0 | src/share/poudriere/include/hash.sh |
+4 | -0 | test/gsub.sh |
+10 | -0 | 2 files |
hash: Cleanup shellcheck warnings
Delta | File | |
---|---|---|
+28 | -20 | src/share/poudriere/include/hash.sh |
+28 | -20 | 1 files |
hash_set: Support noclobber
Delta | File | |
---|---|---|
+10 | -1 | src/share/poudriere/include/hash.sh |
+6 | -2 | test/hash_basic.sh |
+16 | -3 | 2 files |
hash_remove: Support no return var
Delta | File | |
---|---|---|
+19 | -9 | src/share/poudriere/include/hash.sh |
+9 | -1 | test/hash_basic.sh |
+28 | -10 | 2 files |
hash_remove: Avoid colliding with parent vars
Delta | File | |
---|---|---|
+9 | -9 | src/share/poudriere/include/hash.sh |
+9 | -9 | 1 files |
Add a hash_vars() mostly for test assertions
Delta | File | |
---|---|---|
+65 | -0 | src/share/poudriere/include/hash.sh |
+12 | -0 | test/hash_basic.sh |
+77 | -0 | 2 files |
gsub_badchars: Fix with /bin/sh for '-' replacements
Delta | File | |
---|---|---|
+4 | -1 | src/share/poudriere/include/hash.sh |
+3 | -0 | test/gsub.sh |
+7 | -1 | 2 files |
Merge branch 'tests-and-framework-and-misc' * tests-and-framework-and-misc: (208 commits) Update ports from freebsd/freebsd-ports/main at 9d8ea2600730961c87b13ff051f26c4e88fa136d Support gmake jobserver for 'make check' New test assert framework Tests now support running the same test with different data. More SH builtins SH error/debug improvements 77 new tests, 36 being package build tests Tests for pkgclean and incremental build Processes are now tracked as jobs rather than pids if [] conditions changed to use case where possible Some shellcheck finding fixes
Delta | File | |
---|---|---|
+2,967 | -14,532 | test-ports/default/MOVED |
+7,402 | -0 | test-ports/default/lang/python39/pkg-plist |
+0 | -7,018 | test-ports/default/lang/python37/pkg-plist |
+3,262 | -2,190 | src/share/poudriere/common.sh |
+5,402 | -0 | test-ports/default/Mk/bsd.port.mk.orig |
+1,843 | -1,637 | test-ports/default/Mk/bsd.port.mk |
+20,876 | -25,377 | 619 files not shown |
+54,108 | -42,093 | 625 files |
stack: Add stack_size()
Delta | File | |
---|---|---|
+29 | -2 | src/share/poudriere/include/hash.sh |
+25 | -5 | test/stack.sh |
+54 | -7 | 2 files |
array/readarray() stuff
Delta | File | |
---|---|---|
+155 | -0 | src/share/poudriere/include/hash.sh |
+80 | -0 | test/array.sh |
+51 | -0 | test/readarray.sh |
+30 | -15 | test/Makefile.in |
+20 | -0 | src/share/poudriere/include/util.sh |
+2 | -0 | test/Makefile.am |
+338 | -15 | 6 files |
stack: Add stack_expand() and stack_set()
Delta | File | |
---|---|---|
+117 | -12 | test/stack.sh |
+98 | -0 | src/share/poudriere/include/hash.sh |
+215 | -12 | 2 files |
stack: Support spaced values
Delta | File | |
---|---|---|
+20 | -42 | test/stack.sh |
+26 | -0 | src/share/poudriere/include/asserts.sh |
+13 | -13 | test/hash_stack.sh |
+7 | -6 | src/share/poudriere/include/hash.sh |
+66 | -61 | 4 files |
SC2295: Avoid expansion becoming a pattern
Delta | File | |
---|---|---|
+5 | -5 | src/share/poudriere/common.sh |
+3 | -3 | src/share/poudriere/include/util.sh |
+1 | -0 | src/share/poudriere/include/hash.sh |
+9 | -8 | 3 files |
SC2145: Pass $@ properly as its own argument
SC2254: Fix and document where case globbing is intended
case is significantly faster than test(1) for simple string checks.
Delta | File | |
---|---|---|
+1,512 | -902 | src/share/poudriere/common.sh |
+188 | -109 | src/share/poudriere/include/util.sh |
+93 | -61 | src/share/poudriere/include/fs.sh |
+59 | -35 | src/share/poudriere/include/pkg.sh |
+56 | -22 | src/share/poudriere/jail.sh |
+42 | -24 | src/share/poudriere/include/pkgqueue.sh |
+1,950 | -1,153 | 15 files not shown |
+2,124 | -1,239 | 21 files |
stack: Add basic push/pop functions stack_*() and for hash_*()
Delta | File | |
---|---|---|
+222 | -0 | src/share/poudriere/include/hash.sh |
+149 | -0 | test/stack.sh |
+91 | -0 | test/hash_stack.sh |
+16 | -2 | test/Makefile.in |
+2 | -0 | test/Makefile.am |
+480 | -2 | 5 files |
Pipefail fixes
Delta | File | |
---|---|---|
+109 | -50 | src/share/poudriere/jail.sh |
+53 | -26 | src/share/poudriere/common.sh |
+27 | -11 | src/share/poudriere/bulk.sh |
+21 | -8 | src/share/poudriere/testport.sh |
+14 | -12 | src/share/poudriere/include/fs.sh |
+18 | -6 | src/share/poudriere/include/util.sh |
+242 | -113 | 4 files not shown |
+258 | -122 | 10 files |
Merge branch 'display_encode_args'
Delta | File | |
---|---|---|
+677 | -0 | test/display.sh |
+242 | -53 | src/share/poudriere/include/display.sh |
+84 | -36 | src/share/poudriere/include/hash.sh |
+49 | -31 | src/share/poudriere/include/util.sh |
+65 | -7 | test/encode_args.sh |
+17 | -9 | test/Makefile.in |
+1,134 | -136 | 4 files not shown |
+1,154 | -138 | 10 files |
hash: Reduce code duplication
Delta | File | |
---|---|---|
+4 | -7 | src/share/poudriere/include/hash.sh |
+4 | -7 | 1 files |
hash: Avoid name collisions with callers
Delta | File | |
---|---|---|
+43 | -43 | src/share/poudriere/include/hash.sh |
+43 | -43 | 1 files |
hash: Add hash_isset_var() and hash_unset_var()
Delta | File | |
---|---|---|
+51 | -0 | src/share/poudriere/include/hash.sh |
+17 | -0 | test/hash_basic.sh |
+68 | -0 | 2 files |
hash/list: Hide unhelpful set -x debugging
Delta | File | |
---|---|---|
+4 | -0 | src/share/poudriere/include/hash.sh |
+4 | -0 | 1 files |