On Tue, Apr 22, 2025, at 10:17 AM, Aaron Merey wrote:
> dwfl_st or dwflst prefixes work for me. I think I slightly prefer
> dwfl_st. As for where to define Dwfl_Process_tracker let's try to keep
> it to the new dwfl_stacktraceP.h and if possible use forward
> declarations to avoid circular dependencies. If it's necessary to
> include more in libdwflP.h that should be ok since it's not publicly
> exposed,
Reworked 6/12 patches so far, but running into a linking problem:
Making all in libdw
make all-am
CCLD libdw.so
/usr/bin/ld: ../libdwfl/libdwfl_pic.a(dwfl_module_getdwarf.os): in function
`open_elf':
/home/serhei/Documents/elfutils/libdwfl/dwfl_module_getdwarf.c:86:(.text+0xa13):
undefined reference to `dwflst_tracker_cache_elf'
This seems to be an issue specific to libdwfl_pic.a rather than libdwfl.a.
The public function dwflst_tracker_cache_elf is called from
dwfl_module_getdwarf.c.
Calling a public function across the library boundary should be no problem,
but we run into this -- circular dependency?
I'm not knowledgeable enough about how the Automake stuff works to know the
immediate resolution; concerned this delays testing the reworked patches.
Let's see if I can figure this out in the next 30min.
--
All the best,
Serhei
http://serhei.io