| 20 Feb 2025 |
 John Ericson |
also, I think nix/ rather than include/nix/ is still kinda bad because you're still going to be including .cc files etc. in the include path, unless I'm misreading
yes it is!
| 22:12:51 |
| John Ericson | I suppose if we are moving them, we might as well move them harder | 22:13:04 |
| John Ericson | however I will still need an include/nix/meson.build | 22:14:08 |
| John Ericson | can't use a include/meson.build | 22:14:14 |
 emily | do you need a separate Meson file for the headers at all? | 22:14:34 |
| John Ericson | emily: yes you do unfortunately | 22:17:23 |
 Robert Hensing (roberth) | did a fresh worktree, and meson compile is not happy: <command-line>: fatal error: config-util.hh: No such file or directory | 22:19:44 |
| Robert Hensing (roberth) | clangd also seems confused | 22:19:57 |
| John Ericson | the generated header has to be generated in the (build dir mirror of the) directory | 22:20:16 |
| John Ericson | so if we want to ahve one list of headers we append to | 22:20:25 |
| emily | couldn't fs.copyfile solve that? | 22:20:33 |
| John Ericson | we do need to have that nix/meson.build | 22:20:36 |
| John Ericson | I think the dest of fs.copyfile must abide by the same rule? | 22:20:55 |
| Robert Hensing (roberth) | might have done that with a mix of gcc and clang. retrying with clang only | 22:22:44 |
| Robert Hensing (roberth) | oh same | 22:22:54 |
| Robert Hensing (roberth) | ok, was perhaps premature to test. Will try again tomorrow? | 22:25:26 |
| John Ericson | sorry about that, it worked for me! | 22:32:31 |
| John Ericson | it was the last thing I fixed, however | 22:32:59 |
| Robert Hensing (roberth) | np. I shouldn't complain | 22:33:01 |
| John Ericson | src/libutil/nix/meson.build is config_h defined in there? | 22:33:06 |
| Robert Hensing (roberth) | seems so | 22:33:40 |
| John Ericson | I'll clean and start again | 22:33:57 |
| John Ericson | ah, reproduced it now | 22:37:42 |
| emily | John Ericson: I believe you do want <nix/…> because that will suppress warnings for the headers | 23:03:38 |
| emily | (don't quote me on this though) | 23:03:42 |
| John Ericson | emily: OK | 23:04:58 |
| John Ericson | I just did "..." caux lix did heh | 23:05:06 |
| John Ericson | I do think once it is installed, it is a "system header" | 23:05:20 |
| John Ericson | so it certainly it should be <,..> | 23:05:53 |
| John Ericson | * so it certainly it should be <....> | 23:06:00 |
