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.

Sign up for GitHub

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

Jump to bottom

Replace GNU error(3) with POSIX perror and fprintf to add support for non-gnu operating systems. #223

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Replace GNU error(3) with POSIX perror and fprintf to add support for non-gnu operating systems. #223

wants to merge 1 commit into from

Conversation

fvalasiad
Copy link
Collaborator

No description provided.

@fvalasiad fvalasiad requested a review from zvr August 26, 2024 01:04
@fvalasiad
Replace GNU error(3) with POSIX perror and fprintf to add support for…
817860c 
… non-gnu operating systems.

Signed-off-by: Fotios Valasiadis <[email protected]>
@zvr
Copy link
Collaborator

zvr commented Dec 10, 2024

We should keep all the calls to error().

If we want, autoconf may check for its presence, and if it's not there, include an dummy implementation of the function.

@fvalasiad
Copy link
Collaborator Author

What about creating a header that is to be included by anyone expecting error(3), and that header either includes error.h if present(as reported by autoconf) or alternatively implements its own dummy error(3)?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zvr zvr Awaiting requested review from zvr

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@fvalasiad @zvr