]> git.sesse.net Git - bcachefs-tools-debian/commit
build: require explicit include paths for libbcachefs/
authorAustin Seipp <aseipp@pobox.com>
Mon, 27 Nov 2017 01:29:00 +0000 (19:29 -0600)
committerAustin Seipp <aseipp@pobox.com>
Sun, 10 Dec 2017 22:06:01 +0000 (16:06 -0600)
commit997bb216aa37010826067d0cfa33b4fde5aaa225
tree63e968981b6be6ffa17ccf9612b8c5d2cf639a25
parent04035f0c7eaf26ef0a74e36e0edf1adddecb62b5
build: require explicit include paths for libbcachefs/

This removes the implicit `-I libbcachefs` argument to $(CC), which in turn
requires a set of minor changes throughout the tools. There are two advantages
to this setup:

    1) It is (arguably) easier to read, since the location of bcachefs includes
    are easier to understand at a glance ("where does util.h live?")

    2) It removes the need for a hack to include glibc's copy of
    dirent.h explicitly via '/usr/include/dirent.h', because libbcachefs/
    *also* has a dirent.h file and the compiler cannot disambiguate them.
    This has some ramifications on systems where /usr/include may not
    exist, such as NixOS.

Signed-off-by: Austin Seipp <aseipp@pobox.com>
13 files changed:
Makefile
cmd_assemble.c
cmd_debug.c
cmd_device.c
cmd_format.c
cmd_fsck.c
cmd_key.c
cmd_migrate.c
cmd_run.c
crypto.c
libbcachefs.c
libbcachefs.h
tools-util.c