Skip to content

Conversation

@fzakaria
Copy link
Contributor

Motivation

I ran include-what-you-use over libflake as it seemed small to start.

steps:

> iwyu_tool.py -p build/ src/libflake-tests/ -j 100 -- -Xiwyu --mapping_file=/nix/store/0zqzzg1mm1fman18svxkf5p4km4yq85p-include-what-you-use-0.24/share/include-what-you-use/boost-all.imp -Xiwyu --cxx17ns > iwyu.out

> cd build

> fix_includes.py < ../iwyu.out

I then went through the header files and now I can more safely remove the header files clangd warns that are "not used".

Context

There is an ongoing debate on whether to do include-what-you-use (iwyu).

For:

Against:


Add 👍 to pull requests you find important.

The Nix maintainer team uses a GitHub project board to schedule and track reviews.

@fzakaria fzakaria requested a review from edolstra as a code owner October 15, 2025 03:45
@github-actions github-actions bot added the c api Nix as a C library with a stable interface label Oct 15, 2025
@fzakaria fzakaria force-pushed the fzakaria/iwyu-libflake branch from 3f178b8 to e8b126f Compare October 15, 2025 03:48
@Ericson2314 Ericson2314 added this pull request to the merge queue Oct 15, 2025
Merged via the queue into NixOS:master with commit e07754d Oct 15, 2025
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c api Nix as a C library with a stable interface

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants