Kaleb KEITHLEY has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/453378 Change subject: build: use a version script for libganesha_nfsd.so ...................................................................... build: use a version script for libganesha_nfsd.so Only expose symbols needed by applications, e.g. ganesha.nfsd and the FSALs. Add link flag to the FSALs and the applications to not allow undefined symbols as an aid to developers. If a new API is added and it is used in an app or FSAL they will discover it immediately if they forget to add it to the version script. Tested (superficially) with FSAL_VFS and FSAL_GLUSTER. Note that Fedora tried, circa f28 or so, to require "-z defs" (a.k.a. --no-undefined). It will probably come back at some point, so we will be ahead of the curve here. Signed-off-by: Kaleb S. KEITHLEY <kkeithle@redhat.com> Change-Id: I0ef14d96dc1953a340f30fb3043e1cb16a0b98e4 --- M src/CMakeLists.txt M src/FSAL/FSAL_CEPH/CMakeLists.txt M src/FSAL/FSAL_GLUSTER/CMakeLists.txt M src/FSAL/FSAL_GPFS/CMakeLists.txt M src/FSAL/FSAL_MEM/CMakeLists.txt M src/FSAL/FSAL_PROXY/CMakeLists.txt M src/FSAL/FSAL_RGW/CMakeLists.txt M src/FSAL/FSAL_VFS/panfs/CMakeLists.txt M src/FSAL/FSAL_VFS/vfs/CMakeLists.txt M src/FSAL/FSAL_VFS/xfs/CMakeLists.txt M src/FSAL/Stackable_FSALs/FSAL_NULL/CMakeLists.txt M src/MainNFSD/CMakeLists.txt A src/MainNFSD/libganesha_nfsd.ver 13 files changed, 263 insertions(+), 29 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/78/453378/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/453378 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: I0ef14d96dc1953a340f30fb3043e1cb16a0b98e4 Gerrit-Change-Number: 453378 Gerrit-PatchSet: 1 Gerrit-Owner: Kaleb KEITHLEY <kaleb@redhat.com> Gerrit-MessageType: newchange