Re: [PATCH] kill: fix signal number to name lookup on AIX

[Michael Felt]

I think both patches are working as expected. Had to find the "patched" 
8.27 tarball you sent previously to get rid of the missing function.

Test performed on AIX 6.1 (6100-09-08-1642), on NFS filesystem (trying 
on local filesystem on AIX 5.3, but will probably fail (initially) 
because of /bin/sh glob issues. Will retry using SHELL=/bin/ksh and "bash-4"

Hint: I expect the issue here is that the ln is done on a NFS server 
rather than "local" filesystem.

Also, not run (here) as root, so those tests were skipped here.

========================================================
   GNU coreutils 8.27.4-3c9d7: ./tests/test-suite.log
========================================================

# TOTAL: 592
# PASS:  414
# SKIP:  177
# XFAIL: 0
# FAIL:  1
# XPASS: 0
# ERROR: 0

from ./tests/test-suite.log

FAIL: tests/ln/sf-1
===================

++ initial_cwd_=/data/prj/gnu/coreutils/coreutils-8.27.4-3c9d7
+++ testdir_prefix_
+++ printf gt
++ pfx_=gt
+++ mktempd_ /data/prj/gnu/coreutils/coreutils-8.27.4-3c9d7 gt-sf-1.sh.XXXX
+++ case $# in
+++ destdir_=/data/prj/gnu/coreutils/coreutils-8.27.4-3c9d7
+++ template_=gt-sf-1.sh.XXXX
+++ MAX_TRIES_=4
+++ case $destdir_ in
+++ destdir_slash_=/data/prj/gnu/coreutils/coreutils-8.27.4-3c9d7/
+++ case $template_ in
++++ unset TMPDIR
+++ d=/data/prj/gnu/coreutils/coreutils-8.27.4-3c9d7/gt-sf-1.sh.tVrQ
+++ case $d in
+++ :
+++ test -d /data/prj/gnu/coreutils/coreutils-8.27.4-3c9d7/gt-sf-1.sh.tVrQ
++++ ls -dgo /data/prj/gnu/coreutils/coreutils-8.27.4-3c9d7/gt-sf-1.sh.tVrQ
+++ perms='drwx--S--- 2 4096 Jul 27 08:08 
/data/prj/gnu/coreutils/coreutils-8.27.4-3c9d7/gt-sf-1.sh.tVrQ'
+++ case $perms in
+++ :
+++ echo /data/prj/gnu/coreutils/coreutils-8.27.4-3c9d7/gt-sf-1.sh.tVrQ
+++ return
++ test_dir_=/data/prj/gnu/coreutils/coreutils-8.27.4-3c9d7/gt-sf-1.sh.tVrQ
++ cd /data/prj/gnu/coreutils/coreutils-8.27.4-3c9d7/gt-sf-1.sh.tVrQ
++ gl_init_sh_nl_='
'
++ IFS='
'
++ for sig_ in 1 2 3 13 15
+++ expr 1 + 128
++ eval 'trap '\''Exit 129'\'' 1'
+++ trap 'Exit 129' 1
++ for sig_ in 1 2 3 13 15
+++ expr 2 + 128
++ eval 'trap '\''Exit 130'\'' 2'
+++ trap 'Exit 130' 2
++ for sig_ in 1 2 3 13 15
+++ expr 3 + 128
++ eval 'trap '\''Exit 131'\'' 3'
+++ trap 'Exit 131' 3
++ for sig_ in 1 2 3 13 15
+++ expr 13 + 128
++ eval 'trap '\''Exit 141'\'' 13'
+++ trap 'Exit 141' 13
++ for sig_ in 1 2 3 13 15
+++ expr 15 + 128
++ eval 'trap '\''Exit 143'\'' 15'
+++ trap 'Exit 143' 15
++ trap remove_tmp_ 0
+ path_prepend_ ./src
+ test 1 '!=' 0
+ path_dir_=./src
+ case $path_dir_ in
+ abs_path_dir_=/data/prj/gnu/coreutils/coreutils-8.27.4-3c9d7/./src
+ case $abs_path_dir_ in
+ 
PATH=/data/prj/gnu/coreutils/coreutils-8.27.4-3c9d7/./src:/data/prj/gnu/coreutils/coreutils-8.27.4-3c9d7/src:/opt/bin:/
usr/bin:/etc:/usr/sbin:/usr/ucb:/usr/bin/X11:/sbin:/usr/java6/jre/bin:/usr/java6/bin:/usr/vacpp/bin:/home/michael/bin:/op
t/svn/bin
+ create_exe_shims_ /data/prj/gnu/coreutils/coreutils-8.27.4-3c9d7/./src
+ case $EXEEXT in
+ return 0
+ shift
+ test 0 '!=' 0
+ export PATH
+ print_ver_ ln
+ require_built_ ln
+ skip_=no
+ for i in '"$@"'
+ case " $built_programs " in
+ test no = yes
+ test yes = yes
+ local i
+ for i in '$*'
+ env ln --version
ln (GNU coreutils) 8.27.4-3c9d7
Copyright (C) 2017 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later 
<http://gnu.org/licenses/gpl.html>.
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.

Written by Mike Parker and David MacKenzie.
+ echo foo
+ ln -s . b
+ ln -sf a b
+ case $(cat err) in
++ cat err
+++ stat -f -c %l .
++ expr 4294967295 + 1
+ name_max_plus1=4294967296
+ test 4294967296 -gt 1
++ printf '%0*d' 4294967296 0
./tests/ln/sf-1.sh: line 38: printf: warning: 0: Result too large
./tests/ln/sf-1.sh: xrealloc: cannot allocate 133652480 bytes (134088304 
bytes allocated)
FAIL tests/ln/sf-1.sh (exit status: 2)



On 7/23/2017 12:37 PM, Pádraig Brady wrote:
> On 29/06/17 05:22, Michael Felt wrote:
> > FAIL: tests/misc/kill
> > ++ env kill -l HUP TERM
> > ++ tail -n1
> > + SIGTERM=15
> > ++ env kill -l 15
> > + test EXIT = TERM
> > + fail=1
> I think our hack of using the wait() status access macros for
> manipulating the exit status from shells is failing for AIX.
> The attached avoids the use of WTERMSIG() on AIX.
> Perhaps we should do this for all platforms?
>
> cheers,
> Pádraig