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

Define portable basename function #53

Closed
wants to merge 1 commit into from
Closed

Conversation

kraj
Copy link

@kraj kraj commented May 21, 2024

Newer version of musl have removed prototype for basename in string.h [1] which now makes it fail to compile with GCC14+ compiler therefore define local basename utility function.

[1] https://git.musl-libc.org/cgit/musl/commit/?id=725e17ed6dff4d0cd22487bb64470881e86a92e7

Newer version of musl have removed prototype for basename in string.h [1]
which now makes it fail to compile with GCC14+ compiler therefore
define local basename utility function.

[1] https://git.musl-libc.org/cgit/musl/commit/?id=725e17ed6dff4d0cd22487bb64470881e86a92e7

Signed-off-by: Khem Raj <[email protected]>
@bluca
Copy link
Member

bluca commented May 21, 2024

Sorry, but as per previous PRs we do not want to carry these kind of changes. Please patch your local musl build instead to provide the needed interfaces.

@bluca bluca closed this May 21, 2024
@kraj
Copy link
Author

kraj commented May 21, 2024

ideally, the implementation from systemd might be better choice for porting to this project as well but I found this was easier.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants