OpenAFS Master Repository branch, master, updated. openafs-devel-1_9_2-25-g13ef511
Gerrit Code Review
gerrit@openafs.org
Fri, 6 Dec 2024 10:12:34 -0500
The following commit has been merged in the master branch:
commit 13ef51154413fde3662d90d125eed1c50bb7af40
Author: Mark Vitale <mvitale@sinenomine.net>
Date: Wed Apr 7 22:50:13 2021 -0400
DARWIN: Use -Werror=objc-method-access for objc
The default Xcode compiler options are insufficient to fail the build if
a method is not found. For example, a typo in the name of method
'componentsJoinedByString' results in the following build warning:
/TaskUtil.m:70:62: warning: instance method '-componentsJoinedBySting:'
not found (return type defaults to 'id') [-Wobjc-method-access]
NSLog(@"Task failed: %@ %@ status:%d.", taskName,
[args componentsJoinedBySting:@", "], status);
Because this is only flagged as a warning, the build completes
successfully. When this code runs, the AFSBackgrounder merely logs a
runtime exception every time it passes through the erroneous code. This
is a silent failure, unless you happen to know how to check for
AFSBackgrounder log messages:
$ log show --predicate 'process=="AFSBackgrounder"'
Add compiler flag '-Werror=objc-method-access' to all of our xcode-built
project files to treat this case as an error instead of a warning, so
the build will fail.
[adeason@sinenomine.net: Update all xcode project files.]
Change-Id: I69a9f45deb6710a50590bd79daf07466332a6ad1
Reviewed-on: https://gerrit.openafs.org/14586
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Mark Vitale <mvitale@sinenomine.net>
Reviewed-by: Cheyenne Wills <cwills@sinenomine.net>
Reviewed-by: Andrew Deason <adeason@sinenomine.net>
.../DARWIN/AFSPreference/OpenAFS.xcodeproj/project.pbxproj | 10 ++++++++++
.../AklogAuthPlugin/AklogAuthPlugin.xcodeproj/project.pbxproj | 4 ++++
src/platform/DARWIN/afscell/afscell.xcodeproj/project.pbxproj | 4 ++++
3 files changed, 18 insertions(+)
--
OpenAFS Master Repository