From owner-freebsd-current@FreeBSD.ORG Tue Mar 21 21:22:33 2006 Return-Path: X-Original-To: freebsd-current@FreeBSD.org Delivered-To: freebsd-current@FreeBSD.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id D766F16A400 for ; Tue, 21 Mar 2006 21:22:33 +0000 (UTC) (envelope-from dougb@FreeBSD.org) Received: from mail2.fluidhosting.com (mx21.fluidhosting.com [204.14.89.4]) by mx1.FreeBSD.org (Postfix) with SMTP id 0797943D68 for ; Tue, 21 Mar 2006 21:22:30 +0000 (GMT) (envelope-from dougb@FreeBSD.org) Received: (qmail 29565 invoked by uid 399); 21 Mar 2006 21:22:29 -0000 Received: from localhost (HELO ?192.168.0.6?) (dougb@dougbarton.us@127.0.0.1) by localhost with SMTP; 21 Mar 2006 21:22:29 -0000 Message-ID: <44206E9F.6020702@FreeBSD.org> Date: Tue, 21 Mar 2006 13:22:39 -0800 From: Doug Barton Organization: http://www.FreeBSD.org/ User-Agent: Thunderbird 1.5 (Windows/20051201) MIME-Version: 1.0 To: Ruslan Ermilov References: <20060320221741.GA13699@troutmask.apl.washington.edu> <20060320230009.GA55254@troutmask.apl.washington.edu> <20060320231604.GH83781@ip.net.ua> <20060320232746.GI83781@ip.net.ua> <20060320234006.GV35129@funkthat.com> <20060320234145.GB33003@troutmask.apl.washington.edu> <20060321064528.GD77849@ip.net.ua> In-Reply-To: <20060321064528.GD77849@ip.net.ua> X-Enigmail-Version: 0.94.0.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Cc: freebsd-current@FreeBSD.org, rwatson@freebsd.org, Steve Kargl Subject: Re: mergemaster broken? X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 21 Mar 2006 21:22:33 -0000 I'm including rwatson here since the MACHINE_ARCH stuff was his idea. Ruslan Ermilov wrote: >> On Mon, Mar 20, 2006 at 03:40:06PM -0800, John-Mark Gurney wrote: >>> Should we also document that -m is suppose to be src's etc dir instead >>> of src? I've accidentally pointed -m at src, and then it does a make >>> which is quite ammuzing as it's completely the wrong thing... Or now >>> that we call outside of /etc, should we make -m really point to src, >>> and have the proper calls add etc to the directory? I strongly dislike the idea of changing the semantics of the -m option. It's been the way it is since day 1, and I really hate to make changes to something like that. I can see a case for making the man page more clear, but I'd rather work around the problem with -m than change the semantics. > Doesn't really matter, mergemaster(8) was broken because it was > written when we didn't have correct wrappers for "distrib-dirs" > and "distribution", upgrade and cross-arch friendly, at the top > level. > > Anyway, attached is the patch I'd like to commit after a nod > from Doug. It fixes mergemaster(8) to use src/Makefile wrappers > for distrib-dirs and distribution targets, and makes it use > TARGET_ARCH instead of faking up MACHINE_ARCH, now that it uses > the correct wrappers. It also makes ${SOURCEDIR} and -m point > at the src/ top, as documented in a manpage. Forgive me if I'm being dense here, but why do the changes you describe require that we run make in src/? Or, alternatively, if it is _absolutely_ necessary to do so, why do we have to redefine SOURCEDIR to be src, and why can't we just strip /etc from SOURCEDIR where needed? In short, I have no objections to fixing mergemaster to work with the new world order, but it needs to be done in a way that does not change semantics of an existing option. I'd also like confirmation from Robert that we're not breaking any of the behavior that he added by doing it the way you propose. Thanks, Doug -- This .signature sanitized for your protection