diff options
author | Michael Laß | 2021-07-30 23:48:07 +0200 |
---|---|---|
committer | Michael Laß | 2021-07-30 23:48:07 +0200 |
commit | e42764fe12e919ee22ac1c80c9abd0d1abf35ccb (patch) | |
tree | e98e3f69ba2d63abcd1c56181131984da51e56d2 /0002-Avoid-duplicate-definitions-of-globals.patch | |
parent | 7471f5e5477b133c2fd317e4883e84a4d0e6ee2c (diff) | |
download | aur-e42764fe12e919ee22ac1c80c9abd0d1abf35ccb.tar.gz |
Update to 1.8.8
Diffstat (limited to '0002-Avoid-duplicate-definitions-of-globals.patch')
-rw-r--r-- | 0002-Avoid-duplicate-definitions-of-globals.patch | 353 |
1 files changed, 0 insertions, 353 deletions
diff --git a/0002-Avoid-duplicate-definitions-of-globals.patch b/0002-Avoid-duplicate-definitions-of-globals.patch deleted file mode 100644 index 2465de7ebf47..000000000000 --- a/0002-Avoid-duplicate-definitions-of-globals.patch +++ /dev/null @@ -1,353 +0,0 @@ -From 828174efa41aa7f35899b3dd0be190b4f15132c5 Mon Sep 17 00:00:00 2001 -From: Cheyenne Wills <cwills@sinenomine.net> -Date: Fri, 22 May 2020 12:16:48 -0600 -Subject: [PATCH 2/3] Avoid duplicate definitions of globals - -GCC 10 changed a default flag from -fcommon to -fno-common. See -https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85678 for some background. - -The change in gcc 10 results in build link-time errors. For example: - ../../src/xstat/.libs/liboafs_xstat_cm.a(xstat_cm.o):(.bss+0x2050): - multiple definition of `numCollections'; - -Ensure that only one definition for global data objects exist and change -references to use "extern" as needed. - -To ensure that future changes do not introduce duplicated global -definitions, add the -fno-common flag to XCFLAGS when using the -configure --enable-checking setting. - -[cwills@sinenomine.net: Note for 1.8.x: renamed terminationEvent -to cm_terminationEvent/fs_terminationEvent instead of deleting it.] - -Reviewed-on: https://gerrit.openafs.org/14106 -Tested-by: BuildBot <buildbot@rampaginggeek.com> -Reviewed-by: Andrew Deason <adeason@sinenomine.net> -Reviewed-by: Benjamin Kaduk <kaduk@mit.edu> -(cherry picked from commit 0e2072ae386d4111bef161eb955964b649c31386) - -Change-Id: I54ca61d372cf763e4a28c0b0829ea361219f6203 -Reviewed-on: https://gerrit.openafs.org/14217 -Reviewed-by: Andrew Deason <adeason@sinenomine.net> -Reviewed-by: Mark Vitale <mvitale@sinenomine.net> -Tested-by: BuildBot <buildbot@rampaginggeek.com> -Reviewed-by: Stephan Wiesand <stephan.wiesand@desy.de> ---- - src/bucoord/main.c | 6 ++---- - src/butc/dbentries.c | 2 +- - src/butc/dump.c | 7 +++---- - src/butc/lwps.c | 2 +- - src/butc/tcmain.c | 2 +- - src/cf/osconf.m4 | 2 +- - src/fsprobe/fsprobe.h | 1 - - src/uss/uss_vol.c | 2 +- - src/viced/host.c | 2 ++ - src/viced/host.h | 2 +- - src/vol/fssync-server.c | 3 --- - src/vol/volume.c | 3 --- - src/xstat/xstat_cm.c | 7 +++---- - src/xstat/xstat_cm.h | 3 +-- - src/xstat/xstat_cm_test.c | 4 ++-- - src/xstat/xstat_fs.c | 7 +++---- - src/xstat/xstat_fs.h | 3 +-- - src/xstat/xstat_fs_test.c | 4 ++-- - 18 files changed, 25 insertions(+), 37 deletions(-) - -diff --git a/src/bucoord/main.c b/src/bucoord/main.c -index d2a5f368b..a453cc9dd 100644 ---- a/src/bucoord/main.c -+++ b/src/bucoord/main.c -@@ -47,9 +47,9 @@ char tcell[64]; - /* - * Global configuration information for the Backup Coordinator. - */ --struct bc_config *bc_globalConfig; /*Ptr to global BC configuration info */ -+extern struct bc_config *bc_globalConfig; /*Ptr to global BC configuration info */ - --struct ubik_client *cstruct; /* Ptr to Ubik client structure */ -+extern struct ubik_client *cstruct; /* Ptr to Ubik client structure */ - time_t tokenExpires; /* The token's expiration time */ - - static const char *DefaultConfDir; /*Default backup config directory */ -@@ -162,8 +162,6 @@ bc_InitTextConfig(void) - udbClientTextP ctPtr; - int i; - -- extern struct bc_config *bc_globalConfig; -- - mkdir(DefaultConfDir, 777); /* temporary */ - - /* initialize the client text structures */ -diff --git a/src/butc/dbentries.c b/src/butc/dbentries.c -index 37b35d307..29ddcbd01 100644 ---- a/src/butc/dbentries.c -+++ b/src/butc/dbentries.c -@@ -31,7 +31,7 @@ - #include "error_macros.h" - - dlqlinkT savedEntries; --dlqlinkT entries_to_flush; -+static dlqlinkT entries_to_flush; - - int dbWatcherinprogress; - -diff --git a/src/butc/dump.c b/src/butc/dump.c -index 1f4eb9b8f..f8fc943a6 100644 ---- a/src/butc/dump.c -+++ b/src/butc/dump.c -@@ -42,8 +42,7 @@ extern int isafile; - extern int forcemultiple; - - extern struct ubik_client *cstruct; --dlqlinkT savedEntries; --dlqlinkT entries_to_flush; -+extern dlqlinkT savedEntries; - - extern afs_int32 groupId; - extern afs_int32 BufferSize; -@@ -61,8 +60,8 @@ extern char *xbsalGName; - extern char *globalButcLog; - #endif /*xbsa */ - --afs_int32 dataSize; /* Size of data to read on each rx_Read() call */ --afs_int32 tapeblocks; /* Number of 16K tape datablocks in buffer (!CONF_XBSA) */ -+extern afs_int32 dataSize; /* Size of data to read on each rx_Read() call */ -+extern afs_int32 tapeblocks; /* Number of 16K tape datablocks in buffer (!CONF_XBSA) */ - - /* TBD - * -diff --git a/src/butc/lwps.c b/src/butc/lwps.c -index d262991d8..aff5e9e07 100644 ---- a/src/butc/lwps.c -+++ b/src/butc/lwps.c -@@ -51,7 +51,7 @@ extern int forcemultiple; - #endif - - /* XBSA Global Parameters */ --afs_int32 xbsaType; -+extern afs_int32 xbsaType; - #ifdef xbsa - struct butx_transactionInfo butxInfo; - #endif -diff --git a/src/butc/tcmain.c b/src/butc/tcmain.c -index e8a20938e..a0298addb 100644 ---- a/src/butc/tcmain.c -+++ b/src/butc/tcmain.c -@@ -66,7 +66,7 @@ - #define TL_PREFIX "TL" - #define CFG_PREFIX "CFG" - --struct ubik_client *cstruct; -+extern struct ubik_client *cstruct; - FILE *logIO, *ErrorlogIO, *centralLogIO, *lastLogIO; - char lFile[AFSDIR_PATH_MAX]; - char logFile[AFSDIR_PATH_MAX + 256]; -diff --git a/src/cf/osconf.m4 b/src/cf/osconf.m4 -index 5f0d73fc2..e36152bd4 100644 ---- a/src/cf/osconf.m4 -+++ b/src/cf/osconf.m4 -@@ -672,7 +672,7 @@ if test "x$GCC" = "xyes"; then - XCFLAGS="${XCFLAGS} -Wall -Wstrict-prototypes -Wold-style-definition -Wpointer-arith" - fi - if test "x$enable_checking" != "xno"; then -- XCFLAGS="${XCFLAGS} -Wall -Wstrict-prototypes -Wold-style-definition -Werror -fdiagnostics-show-option -Wpointer-arith" -+ XCFLAGS="${XCFLAGS} -Wall -Wstrict-prototypes -Wold-style-definition -Werror -fdiagnostics-show-option -Wpointer-arith -fno-common" - if test "x$enable_checking" != "xall"; then - CFLAGS_NOERROR="-Wno-error" - CFLAGS_NOUNUSED="-Wno-unused" -diff --git a/src/fsprobe/fsprobe.h b/src/fsprobe/fsprobe.h -index 94083bfe9..3060371ae 100644 ---- a/src/fsprobe/fsprobe.h -+++ b/src/fsprobe/fsprobe.h -@@ -103,7 +103,6 @@ struct fsprobe_ProbeResults { - - extern int fsprobe_numServers; /*# servers connected */ - extern struct fsprobe_ConnectionInfo *fsprobe_ConnInfo; /*Ptr to connections */ --extern int numCollections; /*Num data collections */ - extern struct fsprobe_ProbeResults fsprobe_Results; /*Latest probe results */ - - extern int fsprobe_Init(int, struct sockaddr_in *, int, int (*)(void), int ); -diff --git a/src/uss/uss_vol.c b/src/uss/uss_vol.c -index c9c13941d..b31a42a06 100644 ---- a/src/uss/uss_vol.c -+++ b/src/uss/uss_vol.c -@@ -61,7 +61,7 @@ extern int line; - * will work and we can avoid nasty little core dumps. - */ - struct ubik_client *uconn_vldbP; /*Ubik connection struct */ --struct ubik_client *cstruct; /*Required name for above */ -+extern struct ubik_client *cstruct; /*Required name for above */ - - /* - * ------------------------ Private globals ----------------------- -diff --git a/src/viced/host.c b/src/viced/host.c -index 36f9e88c0..e7657caac 100644 ---- a/src/viced/host.c -+++ b/src/viced/host.c -@@ -67,6 +67,8 @@ int hostCount = 0; /* number of hosts in hostList */ - int rxcon_ident_key; - int rxcon_client_key; - -+struct host *(hosttableptrs[h_MAXHOSTTABLES]); -+ - static struct rx_securityClass *sc = NULL; - static int h_quota_limit; - -diff --git a/src/viced/host.h b/src/viced/host.h -index fead948f6..272dcdbe7 100644 ---- a/src/viced/host.h -+++ b/src/viced/host.h -@@ -254,7 +254,7 @@ extern int h_RestoreState(void); - #define H_ENUMERATE_BAIL(flags) ((flags)|0x80000000) - #define H_ENUMERATE_ISSET_BAIL(flags) ((flags)&0x80000000) - --struct host *(hosttableptrs[h_MAXHOSTTABLES]); /* Used by h_itoh */ -+extern struct host *(hosttableptrs[h_MAXHOSTTABLES]); /* Used by h_itoh */ - #define h_htoi(host) ((host)->index) /* index isn't zeroed, no need to lock */ - #define h_itoh(hostindex) (hosttableptrs[(hostindex)>>h_HTSHIFT]+((hostindex)&(h_HTSPERBLOCK-1))) - -diff --git a/src/vol/fssync-server.c b/src/vol/fssync-server.c -index 2d7cd6011..69d902f1b 100644 ---- a/src/vol/fssync-server.c -+++ b/src/vol/fssync-server.c -@@ -79,9 +79,6 @@ - #endif /* USE_UNIX_SOCKETS */ - - #ifdef FSSYNC_BUILD_SERVER -- --int (*V_BreakVolumeCallbacks) (VolumeId volume); -- - #define MAXHANDLERS 4 /* Up to 4 clients; must be at least 2, so that - * move = dump+restore can run on single server */ - #define MAXOFFLINEVOLUMES 128 /* This needs to be as big as the maximum -diff --git a/src/vol/volume.c b/src/vol/volume.c -index f1ff10a7b..3d9e47ef3 100644 ---- a/src/vol/volume.c -+++ b/src/vol/volume.c -@@ -124,9 +124,6 @@ pthread_cond_t vol_vinit_cond; - int vol_attach_threads = 1; - #endif /* AFS_PTHREAD_ENV */ - --/* start-time configurable I/O parameters */ --ih_init_params vol_io_params; -- - #ifdef AFS_DEMAND_ATTACH_FS - pthread_mutex_t vol_salvsync_mutex; - -diff --git a/src/xstat/xstat_cm.c b/src/xstat/xstat_cm.c -index d8600c124..3f5bfc843 100644 ---- a/src/xstat/xstat_cm.c -+++ b/src/xstat/xstat_cm.c -@@ -32,9 +32,8 @@ - int xstat_cm_numServers; /*Num connected servers */ - struct xstat_cm_ConnectionInfo - *xstat_cm_ConnInfo; /*Ptr to connection array */ --int numCollections; /*Number of data collections */ - struct xstat_cm_ProbeResults xstat_cm_Results; /*Latest probe results */ --char terminationEvent; /*One-shot termination event */ -+char cm_terminationEvent; /*One-shot termination event */ - - afs_int32 xstat_cmData[AFSCB_MAX_XSTAT_LONGS]; /*Buffer for collected data */ - -@@ -315,8 +314,8 @@ xstat_cm_LWP(void *unused) - */ - if (xstat_cm_debug) - printf("[%s] Signalling main process at %" AFS_PTR_FMT "\n", rn, -- &terminationEvent); -- oneShotCode = LWP_SignalProcess(&terminationEvent); -+ &cm_terminationEvent); -+ oneShotCode = LWP_SignalProcess(&cm_terminationEvent); - if (oneShotCode) - fprintf(stderr, "[%s] Error %d from LWP_SignalProcess()", rn, - oneShotCode); -diff --git a/src/xstat/xstat_cm.h b/src/xstat/xstat_cm.h -index efbba531c..569011cc2 100644 ---- a/src/xstat/xstat_cm.h -+++ b/src/xstat/xstat_cm.h -@@ -86,10 +86,9 @@ struct xstat_cm_ProbeResults { - extern int xstat_cm_numServers; /*# connected servers */ - extern struct xstat_cm_ConnectionInfo - *xstat_cm_ConnInfo; /*Ptr to connections */ --extern int numCollections; /*Num data collections */ - extern struct xstat_cm_ProbeResults - xstat_cm_Results; /*Latest probe results */ --extern char terminationEvent; /*One-shot termination event */ -+extern char cm_terminationEvent; /*One-shot termination event */ - - /* - * ------------------------ Exported functions ------------------------ -diff --git a/src/xstat/xstat_cm_test.c b/src/xstat/xstat_cm_test.c -index b284237aa..7c5e56864 100644 ---- a/src/xstat/xstat_cm_test.c -+++ b/src/xstat/xstat_cm_test.c -@@ -881,8 +881,8 @@ RunTheTest(struct cmd_syndesc *a_s, void *arock) - */ - if (debugging_on) - printf("[%s] Calling LWP_WaitProcess() on event %" AFS_PTR_FMT -- "\n", rn, &terminationEvent); -- waitCode = LWP_WaitProcess(&terminationEvent); -+ "\n", rn, &cm_terminationEvent); -+ waitCode = LWP_WaitProcess(&cm_terminationEvent); - if (debugging_on) - printf("[%s] Returned from LWP_WaitProcess()\n", rn); - if (waitCode) { -diff --git a/src/xstat/xstat_fs.c b/src/xstat/xstat_fs.c -index 51dbe3847..dff930472 100644 ---- a/src/xstat/xstat_fs.c -+++ b/src/xstat/xstat_fs.c -@@ -33,9 +33,8 @@ - int xstat_fs_numServers; /*Num connected servers */ - struct xstat_fs_ConnectionInfo - *xstat_fs_ConnInfo; /*Ptr to connection array */ --int numCollections; /*Number of data collections */ - struct xstat_fs_ProbeResults xstat_fs_Results; /*Latest probe results */ --char terminationEvent; /*One-shot termination event */ -+char fs_terminationEvent; /*One-shot termination event */ - - afs_int32 xstat_fsData[AFS_MAX_XSTAT_LONGS]; /*Buffer for collected data */ - -@@ -334,8 +333,8 @@ xstat_fs_LWP(void *unused) - */ - if (xstat_fs_debug) - printf("[%s] Signalling main process at %" AFS_PTR_FMT "\n", rn, -- &terminationEvent); -- oneShotCode = LWP_SignalProcess(&terminationEvent); -+ &fs_terminationEvent); -+ oneShotCode = LWP_SignalProcess(&fs_terminationEvent); - if (oneShotCode) - fprintf(stderr, "[%s] Error %d from LWP_SignalProcess()", rn, - oneShotCode); -diff --git a/src/xstat/xstat_fs.h b/src/xstat/xstat_fs.h -index 6e2b77b85..a800ba679 100644 ---- a/src/xstat/xstat_fs.h -+++ b/src/xstat/xstat_fs.h -@@ -87,10 +87,9 @@ struct xstat_fs_ProbeResults { - extern int xstat_fs_numServers; /*# connected servers */ - extern struct xstat_fs_ConnectionInfo - *xstat_fs_ConnInfo; /*Ptr to connections */ --extern int numCollections; /*Num data collections */ - extern struct xstat_fs_ProbeResults - xstat_fs_Results; /*Latest probe results */ --extern char terminationEvent; /*One-shot termination event */ -+extern char fs_terminationEvent; /*One-shot termination event */ - - /* - * ------------------------ Exported functions ------------------------ -diff --git a/src/xstat/xstat_fs_test.c b/src/xstat/xstat_fs_test.c -index 853c2121e..d58ecf014 100644 ---- a/src/xstat/xstat_fs_test.c -+++ b/src/xstat/xstat_fs_test.c -@@ -778,8 +778,8 @@ RunTheTest(struct cmd_syndesc *a_s, void *dummy) - */ - if (debugging_on) - printf("[%s] Calling LWP_WaitProcess() on event %" AFS_PTR_FMT "\n", rn, -- &terminationEvent); -- waitCode = LWP_WaitProcess(&terminationEvent); -+ &fs_terminationEvent); -+ waitCode = LWP_WaitProcess(&fs_terminationEvent); - if (debugging_on) - printf("[%s] Returned from LWP_WaitProcess()\n", rn); - if (waitCode) { --- -2.30.0 - |