diff options
author | Emily Shaffer <emilyshaffer@google.com> | 2021-07-22 03:27:06 +0200 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2021-07-22 22:35:20 +0200 |
commit | b7e6a4162207785c66a1de6f4530499925b762b6 (patch) | |
tree | e4d4e75422a2435439211246b53c106eda46adba /trace2.h | |
parent | Merge branch 'ds/sparse-index-protections' (diff) | |
download | git-b7e6a4162207785c66a1de6f4530499925b762b6.tar.xz git-b7e6a4162207785c66a1de6f4530499925b762b6.zip |
tr2: make process info collection platform-generic
To pave the way for non-Windows platforms to define
trace2_collect_process_info(), reorganize the stub-or-definition schema
to something which doesn't directly reference Windows.
Platforms which want to collect parent process information in the
future should:
1. Add an implementation to compat/ (e.g. compat/somearch/procinfo.c)
2. Add that object to COMPAT_OBJS to config.mak.uname
(e.g. COMPAT_OBJS += compat/somearch/procinfo.o)
3. Define HAVE_PLATFORM_PROCINFO in config.mak.uname
In the Windows case, this definition lives in
compat/win32/trace2_win32_process_info.c, which is already conditionally
added to COMPAT_OBJS; so let's add HAVE_PLATFORM_PROCINFO to hint to the
build that compat/stub/procinfo.c should not be used.
Signed-off-by: Emily Shaffer <emilyshaffer@google.com>
Helped-by: Jonathan Nieder <jrnieder@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'trace2.h')
-rw-r--r-- | trace2.h | 6 |
1 files changed, 0 insertions, 6 deletions
@@ -492,13 +492,7 @@ enum trace2_process_info_reason { TRACE2_PROCESS_INFO_EXIT, }; -#if defined(GIT_WINDOWS_NATIVE) void trace2_collect_process_info(enum trace2_process_info_reason reason); -#else -#define trace2_collect_process_info(reason) \ - do { \ - } while (0) -#endif const char *trace2_session_id(void); |