Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: upgrade freebsd tasks. #2014

Closed
wants to merge 1 commit into from
Closed

Conversation

nekonomicon
Copy link
Member

No description provided.

@a1batross
Copy link
Member

Oh, nice, they package it!

I will merge this after I will add check for system-provided libbacktrace.

@nekonomicon
Copy link
Member Author

Oh... there no dynamic linking yet...

Oh, nice, they package it!

Yep, because FreeBSD does not have glibc.

@a1batross
Copy link
Member

Oh... there no dynamic linking yet...

Oh, nice, they package it!

Yep, because FreeBSD does not have glibc.

The backtrace from glibc (which is in execinfo.h) is not libbacktrace by Ian Lance Taylor. The latter actually can print file and line from debug info.

Because we don't link to zlib, I didn't enable compression right away, it's either we should switch to zlib or add support for miniz in libbacktrace. And again I quite like miniz, so I'm looking how to add support for it, which shouldn't be complicated as miniz provides compatible interface.

@nekonomicon
Copy link
Member Author

Hmm... It seems I'm confusing libexecinfo with libbacktrace.
But libexecinfo is a part of base system already.

@a1batross
Copy link
Member

I think we should prefer libbacktrace if it turns out to be stable enough to be used in signal handler.

@a1batross
Copy link
Member

Had to manually apply it because it seems you commited changes to xash-extras and mainui.

Merged, thanks!

@a1batross a1batross closed this Feb 9, 2025
@nekonomicon nekonomicon deleted the cirrus branch February 10, 2025 06:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants