OpenAFS Master Repository branch, master, updated. openafs-devel-1_9_2-61-ge4d8329

Gerrit Code Review gerrit@openafs.org
Tue, 7 Jan 2025 10:27:06 -0500


The following commit has been merged in the master branch:
commit e4d8329619f31c0e234ec0ccd8325b28e33086cf
Author: Andrew Deason <adeason@sinenomine.net>
Date:   Fri Jan 3 12:49:56 2025 -0600

    lwp: Build src/lwp/test
    
    Make the src/lwp/test directory built by default, by 'cd'ing into 'test'
    from src/lwp after building the lwp library. Also fix numerous errors
    and warnings to get the directory buildable, including:
    
    - Change many functions to be declared with proper prototypes
    - Change many functions and variables to be 'static', since they are
      only used inside one compilation unit
    - Remove several unused variables
    - Include stdlib.h for exit() et al, string.h for strcmp(), fcntl.h for
      open(), time.h for localtime() et al, and arpa/inet.h for inet_ntoa().
    - Make main() return an int
    - Change various signal handler callbacks to be the proper type
    - Change LWP_CreateProcess callbacks to be the proper type
    - Use %p to print various pointers, instead of 0x%x
    - Cast various time-based variables to int for printing with %d
    - Remove extra prototypes for IOMGR_Select()
    - Give accept() a socklen_t* instead of an int*
    - Remove the unused make targets 'system' and 'noversion'
    - Update the make rule for the 'test' program to use modern conventions
      and use AFS_LDRULE, and avoid passing -p to the compiler
    - Remove unnecessary links to util.a
    - Remove testlwp.c, which has no main() and isn't linked by
      anything else
    - In selserver.c:handleWrite(), declare 'c' as an unsigned char (so we can
      compare against END_DATA, which is larger than a signed char), and use
      'c' as the byte read in from read(), so 'c' is not used uninitialized.
    - Change test.c to use LWP_WaitProcess() instead of LWP_MwaitProcess().
      LWP_MwaitProcess() has no callers outside of lwp, and was declared
      static, effectively making it private, in commit ff988ead4e
      (lwp-protoize-20080310).
    
    Also add the built test programs to .gitignore.
    
    Change-Id: I6958a78ae271dbcb865df5d7f30019ba05160b56
    Reviewed-on: https://gerrit.openafs.org/16014
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Mark Vitale <mvitale@sinenomine.net>
    Reviewed-by: Michael Meffie <mmeffie@sinenomine.net>

 src/lwp/Makefile.in      |   7 +-
 src/lwp/test/.gitignore  |   4 +
 src/lwp/test/Makefile.in |  12 +-
 src/lwp/test/selclient.c |  34 ++-
 src/lwp/test/selserver.c |  58 ++---
 src/lwp/test/selsubs.c   |  26 +-
 src/lwp/test/test.c      |  21 +-
 src/lwp/test/test_key.c  |  32 +--
 src/lwp/test/testlwp.c   | 646 -----------------------------------------------
 9 files changed, 91 insertions(+), 749 deletions(-)

-- 
OpenAFS Master Repository