OpenAFS Master Repository branch, master, updated. openafs-devel-1_9_2-346-geac6ff3
Gerrit Code Review
gerrit@openafs.org
Tue, 28 Oct 2025 15:47:46 -0400
The following commit has been merged in the master branch:
commit eac6ff336ff7e4cd948dd8485750aaf4a5d2e0b3
Author: Mark Vitale <mvitale@sinenomine.net>
Date: Thu Jul 2 11:22:46 2020 -0400
DAFS: Remove misleading dead code
Commit 51ec2670116d "dafs-20060317" added the following lines in
src/config/stds.h:
/* for now, demand attach fileserver is only support on unix pthreads builds */
#if defined(DEMAND_ATTACH_ENABLE) && defined(AFS_PTHREAD_ENV) && !defined(AFS_NT40_ENV)
#define AFS_DEMAND_ATTACH_FS 1
#endif
Although the comment is correct, the logic has been dead code since
commit 3610ba191fd0 "demand attach build always". This commit removed
(almost) all references to DEMAND_ATTACH_ENABLE, including the autoconf
test that defined it. They were replaced by adding compiler option
-DAFS_DEMAND_ATTACH_FS to various make files as needed.
A subsequent commit b9ecf28c6c7c "DAFS: allow salvager to detect whether
FSYNC server is DAFS" replaced all remaining references except the one
which is the subject of this commit.
Remove the final dead reference to DEMAND_ATTACH_ENABLE.
No functional change is incurred by this commit.
Change-Id: I1b22f594dc9e83887f2cd0f76e70ad763544b192
Reviewed-on: https://gerrit.openafs.org/16577
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Cheyenne Wills <cwills@sinenomine.net>
Reviewed-by: Marcio Brito Barbosa <mbarbosa@sinenomine.net>
Reviewed-by: Andrew Deason <adeason@sinenomine.net>
src/config/stds.h | 5 -----
1 file changed, 5 deletions(-)
--
OpenAFS Master Repository