Re: HEADER_FILES_INSTALL_DIR

[Nicola Pero]

> headers.make in 1.11 does this:
>    HEADER_FILES_INSTALL_DIR = \
>      $(HEADER_FILES_DIR)/$($(GNUSTEP_INSTANCE)_HEADER_FILES_INSTALL_DIR)

Ehm ... thanks for your bug report, :-) 

... I'm a bit puzzled though because I couldn't find this line in
gnustep-make's CVS or 1.11 ... Can you point me to the .tgz or CVS version
you used that contained those lines ?  Maybe the 1.11 .tgz is broken ?

--

As far as I can see, Instance/Shared/headers.make does the following:


HEADER_FILES_INSTALL_DIR = $($(GNUSTEP_INSTANCE)_HEADER_FILES_INSTALL_DIR)

# Please use `.' to force it to stay empty
ifeq ($(HEADER_FILES_INSTALL_DIR),)
  HEADER_FILES_INSTALL_DIR = $(GNUSTEP_INSTANCE)
endif


 
> I think it should be just:
>    HEADER_FILES_INSTALL_DIR = \
>      $($(GNUSTEP_INSTANCE)_HEADER_FILES_INSTALL_DIR)
> 
> why is it prefixed by $(HEADER_FILES_DIR)/?

I don't know. :-)

You are certainly right that it shouldn't be prefixed with
$(HEADER_FILES_DIR), and in fact ... well it isn't ... and I tested it
briefly and it all works for me.

Possibly it's a locally modified version (to work around some other
problem maybe ?) ?

Thanks