From owner-freebsd-arch  Wed Jan 24 13:18:28 2001
Delivered-To: freebsd-arch@freebsd.org
Received: from ringworld.nanolink.com (pool84-tch-2.Sofia.0rbitel.net [212.95.171.84])
	by hub.freebsd.org (Postfix) with SMTP id 49B5537B400
	for <freebsd-arch@FreeBSD.ORG>; Wed, 24 Jan 2001 13:18:09 -0800 (PST)
Received: (qmail 4687 invoked by uid 1000); 24 Jan 2001 21:16:28 -0000
Date: Wed, 24 Jan 2001 23:16:28 +0200
From: Peter Pentchev <roam@orbitel.bg>
To: clefevre@noos.fr
Cc: freebsd-arch@FreeBSD.ORG
Subject: Re: patch for bsd.lib.mk to create include and lib dirs
Message-ID: <20010124231628.A442@ringworld.oblivion.bg>
Mail-Followup-To: clefevre@noos.fr, freebsd-arch@FreeBSD.ORG
References: <20010124113902.B332@ringworld.oblivion.bg> <zoggyfoy.fsf@gits.dyndns.org>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
User-Agent: Mutt/1.2.5i
In-Reply-To: <zoggyfoy.fsf@gits.dyndns.org>; from clefevre@citeweb.net on Wed, Jan 24, 2001 at 07:21:01PM +0100
Sender: owner-freebsd-arch@FreeBSD.ORG
Precedence: bulk
X-Loop: FreeBSD.ORG

On Wed, Jan 24, 2001 at 07:21:01PM +0100, Cyrille Lefevre wrote:
> Peter Pentchev <roam@orbitel.bg> writes:
> 
> [snip]
> 
> would not be better to use install -d instead of mkdir -p which permit,
> if needed alsewhere, to also set ownership ?

D'oh.. great idea!  This simplifies things, because there's no need
for the sys.mk patch to define MKDIR; INSTALL is already defined.

New patch attached.

G'luck,
Peter

-- 
This inert sentence is my body, but my soul is alive, dancing in the sparks of your brain.

Index: src/share/mk/bsd.lib.mk
===================================================================
RCS file: /home/ncvs/src/share/mk/bsd.lib.mk,v
retrieving revision 1.93
diff -u -r1.93 bsd.lib.mk
--- src/share/mk/bsd.lib.mk	2000/10/02 08:48:49	1.93
+++ src/share/mk/bsd.lib.mk	2001/01/24 09:30:01
@@ -260,10 +260,16 @@
 .if !target(install)
 .if !target(beforeinstall)
 beforeinstall: _includeinstall
+.if defined(CREATEDESTDIRS)
+	${INSTALL} -d -o ${LIBOWN} -g ${LIBGRP} ${DESTDIR}${LIBDIR}
 .endif
+.endif
 
 _includeinstall:
 .if defined(INCS)
+.if defined(CREATEDESTDIRS)
+	${INSTALL} -d -o ${INCOWN} -g ${INCGRP} ${DESTDIR}${LIBDIR}
+.endif
 .for header in ${INCS}
 	cd ${.CURDIR} && \
 	${INSTALL} -C -o ${INCOWN} -g ${INCGRP} -m ${INCMODE} \


To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe freebsd-arch" in the body of the message