public inbox for gwml@vger.gnuweeb.org
 help / color / mirror / Atom feed
* [PATCH liburing] src/Makefile: Fix missing bpf_filter.h installation
@ 2026-02-13 21:05 Ammar Faizi
  2026-02-13 21:07 ` Jens Axboe
  0 siblings, 1 reply; 4+ messages in thread
From: Ammar Faizi @ 2026-02-13 21:05 UTC (permalink / raw)
  To: Jens Axboe; +Cc: Ammar Faizi, io-uring Mailing List, GNU/Weeb Mailing List

After a "make install" command, liburing.h fails to compile because
bpf_filter.h is not copied to the destination include directory:

    In file included from .github/workflows/test_build.c:1:
    /usr/include/liburing.h:21:10: fatal error: liburing/io_uring/bpf_filter.h: No such file or directory
    21 | #include "liburing/io_uring/bpf_filter.h"
        |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    compilation terminated.

Add the header to the Makefile's install list to satisfy the dependency.

Fixes: 46b5c4d66232dcadd0f46c875e6fabce3b3dea85 ("src/include/liburing.h: add bpf_filter.h header")
Signed-off-by: Ammar Faizi <ammarfaizi2@gnuweeb.org>
---
 src/Makefile | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/Makefile b/src/Makefile
index 7febcf3c223b..9f45e1999b09 100644
--- a/src/Makefile
+++ b/src/Makefile
@@ -108,6 +108,7 @@ install: $(all_targets)
 	install -D -m 644 include/liburing/barrier.h $(includedir)/liburing/barrier.h
 	install -D -m 644 include/liburing/io_uring_version.h $(includedir)/liburing/io_uring_version.h
 	install -D -m 644 include/liburing/io_uring/query.h $(includedir)/liburing/io_uring/query.h
+	install -D -m 644 include/liburing/io_uring/bpf_filter.h $(includedir)/liburing/io_uring/bpf_filter.h
 	install -D -m 644 liburing.a $(libdevdir)/liburing.a
 	install -D -m 644 liburing-ffi.a $(libdevdir)/liburing-ffi.a
 ifeq ($(ENABLE_SHARED),1)

base-commit: 9b7c673fecf8f6043dbc132cadbf5570769efd65
-- 
Ammar Faizi


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH liburing] src/Makefile: Fix missing bpf_filter.h installation
  2026-02-13 21:05 [PATCH liburing] src/Makefile: Fix missing bpf_filter.h installation Ammar Faizi
@ 2026-02-13 21:07 ` Jens Axboe
  2026-02-13 21:10   ` Jens Axboe
  0 siblings, 1 reply; 4+ messages in thread
From: Jens Axboe @ 2026-02-13 21:07 UTC (permalink / raw)
  To: Ammar Faizi; +Cc: io-uring Mailing List, GNU/Weeb Mailing List


On Sat, 14 Feb 2026 04:05:48 +0700, Ammar Faizi wrote:
> After a "make install" command, liburing.h fails to compile because
> bpf_filter.h is not copied to the destination include directory:
> 
>     In file included from .github/workflows/test_build.c:1:
>     /usr/include/liburing.h:21:10: fatal error: liburing/io_uring/bpf_filter.h: No such file or directory
>     21 | #include "liburing/io_uring/bpf_filter.h"
>         |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>     compilation terminated.
> 
> [...]

Applied, thanks!

[1/1] src/Makefile: Fix missing bpf_filter.h installation
      commit: 364a7b561fa13cffdd7771978dc5509ec4d9d7f9

Best regards,
-- 
Jens Axboe




^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH liburing] src/Makefile: Fix missing bpf_filter.h installation
  2026-02-13 21:07 ` Jens Axboe
@ 2026-02-13 21:10   ` Jens Axboe
  2026-02-13 21:18     ` Ammar Faizi
  0 siblings, 1 reply; 4+ messages in thread
From: Jens Axboe @ 2026-02-13 21:10 UTC (permalink / raw)
  To: Ammar Faizi; +Cc: io-uring Mailing List, GNU/Weeb Mailing List

On 2/13/26 2:07 PM, Jens Axboe wrote:
> 
> On Sat, 14 Feb 2026 04:05:48 +0700, Ammar Faizi wrote:
>> After a "make install" command, liburing.h fails to compile because
>> bpf_filter.h is not copied to the destination include directory:
>>
>>     In file included from .github/workflows/test_build.c:1:
>>     /usr/include/liburing.h:21:10: fatal error: liburing/io_uring/bpf_filter.h: No such file or directory
>>     21 | #include "liburing/io_uring/bpf_filter.h"
>>         |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>>     compilation terminated.
>>
>> [...]
> 
> Applied, thanks!
> 
> [1/1] src/Makefile: Fix missing bpf_filter.h installation
>       commit: 364a7b561fa13cffdd7771978dc5509ec4d9d7f9

Thanks, I missed that!

BTW, for the future, for:

Fixes: 46b5c4d66232dcadd0f46c875e6fabce3b3dea85 ("src/include/liburing.h: add bpf_filter.h header")

shorten the sha to 12 chars, we don't need the full sha.

For your ~/.gitconfig:

[core]
	abbrev = 12
[pretty]
	fixes = Fixes: %h (\"%s\")
[alias]
	fixes = log -1 --format=fixes

and then you can just do:

axboe@m2max ~/gi/liburing (master)> git fixes 46b5c4d66232dcadd0f46c875e6fabce3b3dea85
Fixes: 46b5c4d66232 ("src/include/liburing.h: add bpf_filter.h header")

and it gives you the right format.

-- 
Jens Axboe

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH liburing] src/Makefile: Fix missing bpf_filter.h installation
  2026-02-13 21:10   ` Jens Axboe
@ 2026-02-13 21:18     ` Ammar Faizi
  0 siblings, 0 replies; 4+ messages in thread
From: Ammar Faizi @ 2026-02-13 21:18 UTC (permalink / raw)
  To: Jens Axboe; +Cc: io-uring Mailing List, GNU/Weeb Mailing List

On Sat, Feb 14, 2026 at 4:10 AM Jens Axboe wrote:
> BTW, for the future, for:
>
> Fixes: 46b5c4d66232dcadd0f46c875e6fabce3b3dea85 ("src/include/liburing.h: add bpf_filter.h header")
>
> shorten the sha to 12 chars, we don't need the full sha.

I see, well noted, thanks for the reminder. Greg told me the same
thing, but I clearly let it slip through the cracks.

I've updated my .gitconfig now.

-- 
Ammar Faizi

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2026-02-13 21:18 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-02-13 21:05 [PATCH liburing] src/Makefile: Fix missing bpf_filter.h installation Ammar Faizi
2026-02-13 21:07 ` Jens Axboe
2026-02-13 21:10   ` Jens Axboe
2026-02-13 21:18     ` Ammar Faizi

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox