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

sdjournal: drop conditional static specifier #358

Merged
merged 2 commits into from
Mar 17, 2021

Conversation

lucab
Copy link
Contributor

@lucab lucab commented Mar 12, 2021

_SD_ARRAY_STATIC is a custom / conditional macro for static C99
array size, which got introduced in recent libsystemd and thus cannot
build on older versions.
Due to how it is currently used in our Go wrapper, it can be safely
omitted from here.

Ref: systemd/systemd@4199f68
Fixes: #355

lucab added 2 commits March 12, 2021 16:06
`_SD_ARRAY_STATIC` is a custom / conditional macro for static C99
array size, which got introduced in recent libsystemd and thus cannot
build on older versions.
Due to how it is currently used in our Go wrapper, it can be safely
omitted from here.

Ref: systemd/systemd@4199f68
@lucab lucab force-pushed the ups/sdjournal-conditional-static branch from c6ca3f3 to cb9eb60 Compare March 12, 2021 16:06
Copy link

@giuseppe giuseppe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

it solves an issue we are seeing with Podman as well as the build on RHEL 8.x is currently failing

@lucab lucab merged commit 47c10f0 into coreos:master Mar 17, 2021
@lucab lucab deleted the ups/sdjournal-conditional-static branch March 17, 2021 17:14
@lucab
Copy link
Contributor Author

lucab commented Mar 19, 2021

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.

could not determine kind of name for C.my_sd_id128_to_string
2 participants