Re: gnustep-make frameworks: multiple entries in FRAMEWORK_NAME

[David Ayers]

Nicola Pero wrote:

> > Hello Nicola,
> >
> > could I PING you on this patch?
> >
> > http://mail.gnu.org/pipermail/gnustep-dev/2002-November/001070.html
> >
> > If you don't have time for it soon, then I will start on handling it 
> > specially in the gsweb makefiles for now.
> >    
>
> Thanks - I applied this patch.
>
> -- but not the part doing 'rm -Rf
> $(GNUSTEP_HEADERS)/$(HEADER_FILES_INSTALL_DIR)' to uninstall because if
> HEADER_FILES_INSTALL_DIR happens to be ./ [which is legitimate] it might
> produce kind of a problem ... :-)
>
> It would be better if we added a script which would remove the directory
> if empty when uninstalling (but then we might need to remove the parent as
> well if empty, etc ... well it gets messy ... and not terribly useful at
> the end of the day, nobody is really using this stuff, which is why I've
> never spent time doing it).
Thanks Nicola,

I just got back today from a small trip, so please excuse my delay. 
Hmm, if ./ is a legitamate HEADER_FILES_INSTALL_DIR then I can follow 
you here, eventhough I have no idea what this script would look like. 
But I guess very few people use it. so no worry.

BTW: this seemed to have got lost some where, or is there a reason for 
not doing this?:
Index: core/make/Instance/framework.make
===================================================================
RCS file: /cvsroot/gnustep/gnustep/core/make/Instance/framework.make,v
retrieving revision 1.28
diff -u -r1.28 framework.make
--- core/make/Instance/framework.make   3 Dec 2002 16:11:34 -0000       1.28
+++ core/make/Instance/framework.make   5 Dec 2002 12:31:48 -0000
@@ -429,7 +429,7 @@
         if test -d "$(HEADER_FILES_INSTALL_DIR)"; then \
           rm -Rf $(HEADER_FILES_INSTALL_DIR); \
         fi; \
-          $(MKINSTALLDIRS) $(GNUSTEP_INSTANCE); \
+          $(MKINSTALLDIRS) $(HEADER_FILES_INSTALL_DIR); \
         cd 
$(FRAMEWORK_INSTALL_DIR)/$(FRAMEWORK_VERSION_DIR_NAME)/Headers ; \
            $(TAR) cf - . | (cd  
$(GNUSTEP_HEADERS)/$(HEADER_FILES_INSTALL_DIR); \
            $(TAR) xf - ); \