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

Haiku fixes #1603

Merged
merged 3 commits into from
Aug 8, 2023
Merged

Haiku fixes #1603

merged 3 commits into from
Aug 8, 2023

Conversation

mmuman
Copy link
Collaborator

@mmuman mmuman commented Aug 5, 2023

This fixes build on Haiku:

  • we don't use X11 so the headers aren't installed by default,
  • asprintf needs _GNU_SOURCE and is in stdio.h as on Linux btw, but we probably don't include this one in the existing includes already,
  • added a stub for get_battery_power_draw().

On Haiku it complained about missing X11 header.
@netlify
Copy link

netlify bot commented Aug 5, 2023

Deploy Preview for conkyweb canceled.

Name Link
🔨 Latest commit 497734f
🔍 Latest deploy log https://app.netlify.com/sites/conkyweb/deploys/64ce30d275180d0008ff4411

@github-actions github-actions bot added the sources PR modifies project sources label Aug 5, 2023
It needs _GNU_SOURCE and is declared in stdio.h as on GNU/Linux,
but I suppose it's not included by the existing includes.
Missing new OS-specific call, stubbed for now.
@brndnmtthws brndnmtthws merged commit 16b8907 into brndnmtthws:main Aug 8, 2023
53 checks passed
@mmuman mmuman deleted the haiku-fixes branch August 8, 2023 23:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sources PR modifies project sources
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants