OpenAFS Master Repository branch, openafs-stable-1_8_x, updated. openafs-stable-1_8_8_1-48-g7c8127c

Gerrit Code Review gerrit@openafs.org
Thu, 4 Aug 2022 13:03:12 -0400


The following commit has been merged in the openafs-stable-1_8_x branch:
commit 7c8127c5a12a94457251b35643a9fa2f284d2a11
Author: Andrew Deason <adeason@sinenomine.net>
Date:   Wed Jul 10 12:42:54 2019 -0500

    LINUX: Honor --enable-checking for libafs
    
    When we build the kernel module on LINUX, we don't pass in any of our
    CFLAGS, since the Linux buildsystem itself figures out what flags are
    needed. However, this means that we don't pass in -Werror when
    --enable-checking is turned on, so warnings may not cause the build to
    fail.
    
    To fix this, create a new autoconf variable, called CFLAGS_WERROR,
    that only contains -Werror if --enable-checking is turned on. We then
    pass that into the Linux module buildsystem, so -Werror is given to
    the compiler when building our module.
    
    Reviewed-on: https://gerrit.openafs.org/13682
    Reviewed-by: Cheyenne Wills <cwills@sinenomine.net>
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
    (cherry picked from commit 6e0f1c3b45102e7644d25cf34395ca980414317f)
    
    [cwills@sinenomine.net] On master gerrit 14106 comes after this commit.
    In the 1.8.x branch the 14106 commit is already pulled in (gerrit 14217)
    which causes a conflict due a difference ('-fno-common') in the line
    above the change in osconf.m4.
    
    Change-Id: I4e49d5c5fdf26399eb04d2f76196f3c3f4e7baf6
    Reviewed-on: https://gerrit.openafs.org/14989
    Reviewed-by: Andrew Deason <adeason@sinenomine.net>
    Reviewed-by: Marcio Brito Barbosa <mbarbosa@sinenomine.net>
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Michael Meffie <mmeffie@sinenomine.net>
    Reviewed-by: Stephan Wiesand <stephan.wiesand@desy.de>

 src/cf/osconf.m4                  | 2 ++
 src/libafs/MakefileProto.LINUX.in | 2 +-
 2 files changed, 3 insertions(+), 1 deletion(-)

-- 
OpenAFS Master Repository