|
From: | David Ayers |
Subject: | Re: gnustep-make frameworks: multiple entries in FRAMEWORK_NAME |
Date: | Thu, 05 Dec 2002 13:27:25 +0100 |
User-agent: | Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.2) Gecko/20021126 |
Nicola Pero wrote:
Hello Nicola, could I PING you on this patch? http://mail.gnu.org/pipermail/gnustep-dev/2002-November/001070.htmlIf 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 - ); \
[Prev in Thread] | Current Thread | [Next in Thread] |