From owner-svn-src-all@freebsd.org Mon Jun 3 16:02:11 2019 Return-Path: Delivered-To: svn-src-all@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 168C415B583F for ; Mon, 3 Jun 2019 16:02:11 +0000 (UTC) (envelope-from sobomax@sippysoft.com) Received: from mail-io1-f41.google.com (mail-io1-f41.google.com [209.85.166.41]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) server-signature RSA-PSS (4096 bits) client-signature RSA-PSS (2048 bits) client-digest SHA256) (Client CN "smtp.gmail.com", Issuer "GTS CA 1O1" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 9DDFD87FC4 for ; Mon, 3 Jun 2019 16:02:10 +0000 (UTC) (envelope-from sobomax@sippysoft.com) Received: by mail-io1-f41.google.com with SMTP id e3so14726706ioc.12 for ; Mon, 03 Jun 2019 09:02:10 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=kxj31LewPvbp6zwgtdAYYqM15GbbsTRrLi3fReZbnb0=; b=dfdFF/Ca4LfYCcoPdXYP35PYOLDe+WdFzc4K+P9g8n7hWJ1lvIrD2YT6hxHmZX6uJn PL0MVD5xMKZziwJeSgZHlNFrTxtX51/99oa66kb/1VJWDHNB9IQjYp0q9Ee0LCKwPblz UgzISPa5Gz11jrErzBeBGGHAjdeOBNQOn7t+ihgjoIfahZk7PyVKPHAlgTIy68YLBO/f KbWeea889yccUwY4/KMn2aNPebY2cQzo4v0rdYxxEfuUxA5NLBzm5UZhlaq4qDngC2PB xd2Nf1csb8QNPopeg3kqjgXHLSSKtY52eky7IIDIQFe49CrRHqljp8VMTWUypURymXVr HyUA== X-Gm-Message-State: APjAAAUi+e2D9Q/Jj0/Jux+0t4H6ian5HLt8bOeXtQMXbiw8u5CRqqkG TRsMQybU5ow1isBum6NZ98tc+soZaJ6ApaWZQSk9Gw== X-Google-Smtp-Source: APXvYqytdRQQWYCja+z4p/mFyf8Qkqw22TO7Laa8ZCQgXgStmIaMMmp+GP/BzapLrq0cD/WlHh/Gi995nmTxKNG0B5Q= X-Received: by 2002:a5d:9b04:: with SMTP id y4mr10830441ion.195.1559577413178; Mon, 03 Jun 2019 08:56:53 -0700 (PDT) MIME-Version: 1.0 References: <201906022338.x52NcJV9070782@repo.freebsd.org> <99965642-23AF-492C-9D3E-19AE069AB323@gmail.com> In-Reply-To: <99965642-23AF-492C-9D3E-19AE069AB323@gmail.com> From: Maxim Sobolev Date: Mon, 3 Jun 2019 08:56:41 -0700 Message-ID: Subject: Re: svn commit: r348521 - in head: . etc sys/conf sys/modules usr.bin/login usr.sbin/services_mkdb To: Enji Cooper Cc: src-committers , svn-src-all , svn-src-head X-Rspamd-Queue-Id: 9DDFD87FC4 X-Spamd-Bar: ------ Authentication-Results: mx1.freebsd.org X-Spamd-Result: default: False [-6.97 / 15.00]; NEURAL_HAM_MEDIUM(-1.00)[-1.000,0]; NEURAL_HAM_SHORT(-0.97)[-0.972,0]; REPLY(-4.00)[]; NEURAL_HAM_LONG(-1.00)[-1.000,0] Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Content-Filtered-By: Mailman/MimeDel 2.1.29 X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 03 Jun 2019 16:02:11 -0000 On Mon, Jun 3, 2019 at 7:28 AM Enji Cooper wrote: > Hi Maxim, > > > > Modified: head/Makefile.inc1 > > > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D > > --- head/Makefile.inc1 Sun Jun 2 22:27:26 2019 (r348520) > > +++ head/Makefile.inc1 Sun Jun 2 23:38:19 2019 (r348521) > > @@ -887,8 +887,8 @@ MTREEFLAGS+=3D -W > > INSTALLFLAGS+=3D -h sha256 > > .endif > > .if defined(DB_FROM_SRC) || defined(NO_ROOT) > > -IMAKE_INSTALL=3D INSTALL=3D"install ${INSTALLFLAGS}" > > -IMAKE_MTREE=3D MTREE_CMD=3D"mtree ${MTREEFLAGS}" > > +IMAKE_INSTALL=3D INSTALL=3D"${INSTALL} ${INSTALLFLAGS}" > > +IMAKE_MTREE=3D MTREE_CMD=3D"${MTREE_CMD} ${MTREEFLAGS}" > > .endif > > Please back out this part of the change as it is unnecessary and adds > a dependency on sys.mk=E2=80=99s definitions. $PATH should contain, i.e.,= be > populated, with these tools when running make installworld. This is part = of > the logic behind how $ITOOLS, etc, works. > Thank you very much for your work and the rest of the change though! > Thanks Enji, for a clarification, will do. I still cannot understand why can't we have those tools being re-defined with top-level make environment? What I am striving at is being able to do say, make INSTALL=3D"abcd" installworld. Yes, I understand I can call "abcd" "install" (or make a symlink to it) and put it into PATH, but there are at least one issue here, namely what if I need to pass some distinct argument to abcd based on usage scenario. E.g. make INSTALL=3D"abcd foo" distributiuon. In your case I shou= ld be resorting to setting up some global environment variable and hope it won't get erased somewhere in the call chain, but that's bit "how you doing" IMHO. My approach makes certain things so much easier by cutting those un-neded steps. At any rate thank for a feedback and some background info, I will do more work on this. I was thinking more in terms of having some sort of bsd.bootools.mk where this can be defined in a centralized manner. -Max