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

common/Makefile: Fix dependencies for audit.proto #522

Merged
merged 1 commit into from
Feb 24, 2025

Conversation

k0ch4lo
Copy link
Member

@k0ch4lo k0ch4lo commented Feb 15, 2025

This PR adapts the target dependencies for compilation of audit.proto in common/Makefile to ensure the 'protobuf' target actually provides audit.pb-c.{h,c}.

This fixes races during build:
make -C common protobuf
| make[1]: Entering directory /dir/service-static-1.0+999/common' | protoc-c --c_out=. audit.proto
| In file included from audit.c:28:
| audit.h:32:10: fatal error: audit.pb-c.h: No such file or directory
| 32 | #include "audit.pb-c.h"
| | ^~~~~~~~~~~~~~

@k0ch4lo k0ch4lo force-pushed the libcommon_makefile_fix_protobuf branch 11 times, most recently from a990a47 to 62c9b9f Compare February 20, 2025 21:19
@quitschbo quitschbo force-pushed the libcommon_makefile_fix_protobuf branch from 62c9b9f to 72a263c Compare February 21, 2025 07:53
@k0ch4lo k0ch4lo force-pushed the libcommon_makefile_fix_protobuf branch from 72a263c to a5734a2 Compare February 21, 2025 20:40
This commit adapts the target dependencies for compilation of audit.proto in common/Makefile
to ensure the 'protobuf' target actually provides audit.pb-c.{h,c}.

This fixes races during build:
make -C common protobuf
| make[1]: Entering directory /dir/service-static-1.0+999/common'
| protoc-c --c_out=. audit.proto
| In file included from audit.c:28:
| audit.h:32:10: fatal error: audit.pb-c.h: No such file or directory
|    32 | #include "audit.pb-c.h"
|       |          ^~~~~~~~~~~~~~

Signed-off-by: Felix Wruck <[email protected]>
@quitschbo quitschbo force-pushed the libcommon_makefile_fix_protobuf branch from a5734a2 to 6d7d62b Compare February 24, 2025 13:16
@quitschbo quitschbo merged commit 4eb93eb into gyroidos:main Feb 24, 2025
2 checks passed
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.

2 participants