From owner-svn-src-head@freebsd.org Sat Sep 26 19:18:41 2015 Return-Path: Delivered-To: svn-src-head@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 3456CA0AD88 for ; Sat, 26 Sep 2015 19:18:41 +0000 (UTC) (envelope-from wlosh@bsdimp.com) Received: from mail-qk0-f170.google.com (mail-qk0-f170.google.com [209.85.220.170]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (Client CN "smtp.gmail.com", Issuer "Google Internet Authority G2" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id E7B38847 for ; Sat, 26 Sep 2015 19:18:40 +0000 (UTC) (envelope-from wlosh@bsdimp.com) Received: by qkap81 with SMTP id p81so54423163qka.2 for ; Sat, 26 Sep 2015 12:18:34 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:sender:in-reply-to:references:date :message-id:subject:from:to:cc:content-type; bh=JR9EcRedyoCKTnMAC0jrg+oqoQHfSuQPEjx3a1HDyzs=; b=M2+5bVrD9ObUo+bKihoajKiA+K0KUMYJl5KI12KWQ0CnPuhW9b+KG+CCl0M5DRH/Nn An5Dc95M5rfbTsH/BVjBOj3C0wxRnTxQ4UuCpbSWiYofFGcTQzI+/HKggifKQjoSXjnx G+8nrBmx7A0oErHX/GwZSam7tbptQ+cnD55XQUYcSv3laSmuStGOWSOl64WbTA7RcbDs Y9TjtB2sSaUINCXviCcCu8k0WLUy6I6Iiv7JrZ3507gJckPOGrOENvLJyo8xzlzwlUKA Dbj7Un38LgOk6L7/r0hpQ/wM6cZ+UfJwJ9LXGNhNlV5xTtDfhczTO+roYmgJiPfBnw7D toHw== X-Gm-Message-State: ALoCoQmJdiEDnoIHcmIBf8ISurYS8UbHhYvrX06GvBTKW7gZYk/TQTjIfnauUGD+RCSVXx8uoXPg MIME-Version: 1.0 X-Received: by 10.55.204.66 with SMTP id r63mr13347048qki.10.1443295114354; Sat, 26 Sep 2015 12:18:34 -0700 (PDT) Sender: wlosh@bsdimp.com Received: by 10.140.80.167 with HTTP; Sat, 26 Sep 2015 12:18:34 -0700 (PDT) X-Originating-IP: [2601:280:4900:3700:70fd:56c:2b2e:19a] In-Reply-To: <5606A9AE.7070303@FreeBSD.org> References: <201509252303.t8PN3Wx0098623@repo.freebsd.org> <5605D4FC.4040205@FreeBSD.org> <5606A9AE.7070303@FreeBSD.org> Date: Sat, 26 Sep 2015 13:18:34 -0600 X-Google-Sender-Auth: dCxC0BiuSDNHjSCxkECMAso5iOk Message-ID: Subject: Re: svn commit: r288241 - head/share/mk From: Warner Losh To: Bryan Drewery Cc: NGie Cooper , "src-committers@freebsd.org" , "svn-src-all@freebsd.org" , "svn-src-head@freebsd.org" Content-Type: text/plain; charset=UTF-8 X-Content-Filtered-By: Mailman/MimeDel 2.1.20 X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 26 Sep 2015 19:18:41 -0000 On Sat, Sep 26, 2015 at 8:20 AM, Bryan Drewery wrote: > On 9/26/2015 7:19 AM, Warner Losh wrote: > > > > > > On Fri, Sep 25, 2015 at 11:11 PM, NGie Cooper > > wrote: > > > > On Fri, Sep 25, 2015 at 4:13 PM, Bryan Drewery > > wrote: > > > On 9/25/2015 4:12 PM, NGie Cooper wrote: > > >> On Fri, Sep 25, 2015 at 4:03 PM, Bryan Drewery < > bdrewery@freebsd.org > wrote: > > >>> Author: bdrewery > > >>> Date: Fri Sep 25 23:03:32 2015 > > >>> New Revision: 288241 > > >>> URL: https://svnweb.freebsd.org/changeset/base/288241 > > >>> > > >>> Log: > > >>> Remove 'set -e' that are no longer needed as it is already > default. > > >>> > > >>> When bmake was initially imported at r241298 shell commands > were no longer > > >>> ran with 'set -e' as they were before. This was fixed in > r254980 so they > > >>> again always use 'set -e'. > > >> > > >> The bsd.subdir.mk portion of the change > looks > > like it would cause > > >> issues depending on what's being called (fmake or an earlier > version > > >> of sys.mk might be used at install time). > > >> > > > > > > We only support bmake in head. And the 'set -e' were only added for > > > bmake compatibility before it was fixed to work like fmake did. > > > > Sorry. Fuzzy memory on the latter item. Yeah, I requested it a > > couple years ago. > > > > We might only support bmake in head, but there's nothing preventing > > someone from doing a source upgrade from one of the older 10 releases > > to 11+. Thinking about this a bit more, this is an extreme edge case > > that doesn't really matter, because people doing source upgrades > > across major releases really should be doing them from the latest > > minor release for the major release > > > > > > I wouldn't state it so glibly. It is not as extreme as you might think. > > For a > > long time compiling -current from a host that was one or two major > > releases old has worked. Currently we advertise that we can upgrade > > from the stable/9 branch point or newer to tip of head (based on values > > in Makefile.inc1). > > > > I don't believe that Bryan's change set changes that in any significant > way, > > but given the large amount of churn he and I (and others) have generated > in > > /usr/share/mk, testing from a 9.x machine would be prudent. I didn't > remove > > some minor bits of code, and also made sys.mk compatible > > with the FreeBSD 9 > > fmake because of issues like this. > > > > Note that fmake compatibility was explicitly removed around the > META_MODE import time. fmake doesn't work at all currently. > That's only partially true. Some of the compatibility was removed, but not all. We still have (and still want) the compatibility for detecting whether or not to bootstrap bmake or not in src/Makefile. We also added compatibility to sys.mk so that we could build older systems on -current hosts with fmake which uses the system sys.mk for the earliest parts of the build.. So there's still some vestiges of it in the system, and they are conscious decisions to leave them in place. What was removed was building world with fmake for the whole shooting match. Also removed were certain hacks in the other files than sys.mk (and what it includes) to support both. But regardless, we need to make sure that the minimal version we support is actually accurate in Makefile.inc1. And that can only be done by direct testing. I tried to remove it all, but there was much gnashing of teeth and such when I proposed it so I backed away. There's too much need and too many use cases for slivers of fmake support in the tree still. Warner