Alexei,

> +     * builds/toplevel.mk: Place `FTMODULE_H' in include/.

Have you tested this commit on a Unix box with a build where 'srcdir'
!= 'builddir', and where 'srcdir' is read-only?  My gut feeling says
that your commit is not correct.

We have the following rules, which I consider the right way.

* Files that can be configured by the user should be read first from
  `builddir`, overriding the locations in `srcdir` (examples:
  `modules.cfg`, `ftoption.h`).

* Files that are auto-generated at configure time must always be
  placed into `builddir` (example: `config.mk`).

I want that this scheme works for *all* build systems (i.e., autoconf,
GNU make, meson, cmake) – well, I guess that VC++ is different...

In other words, this change

  -INCLUDES := $(subst /,$(COMPILER_SEP),$(OBJ_DIR) \
  -                                      $(DEVEL_DIR) \
  +INCLUDES := $(subst /,$(COMPILER_SEP),$(DEVEL_DIR) \
                                         $(BUILD_DIR) \
                                         $(TOP_DIR)/include)

looks very suspicious :-)


    Werner

Reply via email to