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 по дате отправления: