Re: ci: Improve OpenBSD core dump backtrace handling
| От | Nazir Bilal Yavuz |
|---|---|
| Тема | Re: ci: Improve OpenBSD core dump backtrace handling |
| Дата | |
| Msg-id | CAN55FZ1ZVMPYCOKsXz=gaVdJc0SN-7CgTegQh7juWrCkoUo_Xg@mail.gmail.com обсуждение исходный текст |
| Ответ на | Re: ci: Improve OpenBSD core dump backtrace handling (Thomas Munro <thomas.munro@gmail.com>) |
| Ответы |
Re: ci: Improve OpenBSD core dump backtrace handling
|
| Список | pgsql-hackers |
Hi,
Thank you for looking into this!
On Sat, 25 Oct 2025 at 00:41, Thomas Munro <thomas.munro@gmail.com> wrote:
>
> On Sat, Oct 25, 2025 at 2:20 AM Nazir Bilal Yavuz <byavuz81@gmail.com> wrote:
> > Any feedback would be appreciated.
>
> + filename=$(basename "$corefile")
> + base=$(echo "$filename" | sed 's/\.core.*$//')
> + binary=$(find "$executable_directory" -type f -name "$base"
> 2>/dev/null | head -n 1)
> +
> + if [ -z "$binary" ]; then
> + echo "${base} executable can not be found in
> ${executable_directory}"
> + continue
> + fi
> +
> + lldb "$binary" -c "$corefile" --batch -o 'thread backtrace
> all' -o 'quit'
>
> s/can not/cannot/, I don't make the rules...
>
> Maybe if not found it should just run it the old way without the
> executable? We don't really expect other programs to be dumped (well,
> sometimes we abort cp etc, something I plan to fix...), and they're
> probably stripped anyway, but I guess we might as well try to show as
> much information as we can if it happens? Would it be better to set
> PATH to $executable_directory:$PATH and use "which"?
I agree with all of your points. All of them are addressed in the v2.
--
Regards,
Nazir Bilal Yavuz
Microsoft
Вложения
В списке pgsql-hackers по дате отправления: