Skip to content

Conversation

rlewczuk
Copy link
Contributor

@rlewczuk rlewczuk commented Sep 22, 2025

This PR adds additional information to an error message when loading backend library via ld_load_library() fails. This helps spotting why backend library did not load (missing library, missing dependency or unresolved symbol etc.).

@github-actions github-actions bot added the ggml changes relating to the ggml tensor library for machine learning label Sep 22, 2025
@rlewczuk
Copy link
Contributor Author

Since @ericcurtin does not have possibility to merge this change anymore, who has ? @slaren ?

@rlewczuk rlewczuk force-pushed the rle/ggml-ld-error-msgs branch from 830daaa to 5bc6480 Compare September 29, 2025 06:41
@rlewczuk rlewczuk requested a review from slaren as a code owner September 29, 2025 06:41
@ericcurtin
Copy link
Collaborator

ericcurtin commented Sep 29, 2025

I wonder if we rebase this code and push will the build go green @rlewczuk ?

It would be useful to know who has merge rights also.

I'm going to be less active reviewing PRs these days. If I could ask people to avoid tagging me, that would be good, because best I can do is tag someone else.

I will continue to contribute where appropriate.

@rlewczuk rlewczuk force-pushed the rle/ggml-ld-error-msgs branch from 5bc6480 to 2322195 Compare September 29, 2025 09:42
@rlewczuk
Copy link
Contributor Author

rlewczuk commented Sep 29, 2025

Did it right now, albeit it was also rebased today morning (CET). Let's see if CI still fails.

Are other PRs also showing similar errors ?

Edit: seems to be tests flapping due to SVE changes, see #16314

@danbev danbev merged commit 02463ab into ggml-org:master Sep 29, 2025
64 of 67 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ggml changes relating to the ggml tensor library for machine learning
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants