From owner-svn-src-all@FreeBSD.ORG Mon Oct 22 16:42:44 2012 Return-Path: Delivered-To: svn-src-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id B34C7EC7; Mon, 22 Oct 2012 16:42:44 +0000 (UTC) (envelope-from yanegomi@gmail.com) Received: from mail-la0-f54.google.com (mail-la0-f54.google.com [209.85.215.54]) by mx1.freebsd.org (Postfix) with ESMTP id 6CC378FC16; Mon, 22 Oct 2012 16:42:43 +0000 (UTC) Received: by mail-la0-f54.google.com with SMTP id e12so2241734lag.13 for ; Mon, 22 Oct 2012 09:42:42 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type; bh=vsumpZDPJq8u3muZ8Szpv6objBGllHdqiFqANOqkIis=; b=cne8BE6CrdvQt8kacTefpBMCAjP2cIcKOo13YCoNmgZFp0TNMnnnkimnkVz7QxAJNH OcqhjugZN3OmSR74++gadpDw3NgMe/FZbgUHaqzjjaUz3Id5SFctkK63Jd+nkwJCsaee ZUvyymWRAyc2+KloyG3+pdRQrEY9FLmQtivFJB7A766QIiEkzvViUzALDc1kpcxVO/pE tKrF42XzB0b8WvG0FipEMBtvAsT6cmjtP3WexwnINjmnCNflo5nv+i/7RjnNwqfe4ILN 8nud8S2GqIKfJPc1VShCNxOIMF3EgVglOwXMrnYBIxRWCHDQLdRTWFaBrPu9PY4jJ9D+ fcSg== MIME-Version: 1.0 Received: by 10.152.104.115 with SMTP id gd19mr8870830lab.13.1350924162291; Mon, 22 Oct 2012 09:42:42 -0700 (PDT) Received: by 10.112.43.232 with HTTP; Mon, 22 Oct 2012 09:42:42 -0700 (PDT) In-Reply-To: <20121022150706.GD70741@FreeBSD.org> References: <201210220118.q9M1Ifh5098857@svn.freebsd.org> <20121022150706.GD70741@FreeBSD.org> Date: Mon, 22 Oct 2012 09:42:42 -0700 Message-ID: Subject: Re: svn commit: r241823 - in head: . etc etc/atf etc/mtree lib lib/atf lib/atf/libatf-c lib/atf/libatf-c++ libexec libexec/atf libexec/atf/atf-check share share/atf share/doc share/doc/atf share/exampl... From: Garrett Cooper To: Gleb Smirnoff Content-Type: text/plain; charset=ISO-8859-1 Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, Marcel Moolenaar , src-committers@freebsd.org X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.14 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, 22 Oct 2012 16:42:44 -0000 On Mon, Oct 22, 2012 at 8:07 AM, Gleb Smirnoff wrote: > On Mon, Oct 22, 2012 at 01:18:41AM +0000, Marcel Moolenaar wrote: > M> Tinderbox breakages that are the result of this commit are entirely > M> the committer's fault -- in other words: buildworld testing on amd64 > M> only. > > Taking into account the fact that last couple of weeks head was usually > not buildable rather than buildable, I strongly disappreciate this. > > It looks to me as we are again treating head/ as a pile of stuff, > not as an operating system one can install and use. > > Now that we have a big choice of VCSes: svn user branches, git and > perforce, why isn't it possible to settle things in private branch > and only then commit them to the head/? I had been checking in changes to p4 over the last 4 months and then I ditched it for a local svn checkout when preparing the final patch that I sent to Marcel; I had run the patch in svn in two different sets of VMs in a make tinderbox manner several times. All but a handful of non-build critical items made it into this commit, and the only breakage that was present was accidental and affected ATF runtime. Maintaining all these moving pieces has proved challenging (esp. because I lack the scripts to track all of the changes that I had at IronPort with p4), and even then tracking new files in p4 is entertaining compared to svn, git, etc. This unnecessary complexity plus the fact that one needs p4 in order to collaborate with the changes I'm working on is the primary reason why I'm ditching it for git. The only breakage that's occurring now we've found is people using clang/libc++ (and libc++ looks like it has bugs that would have been caught with C++ applications written in a C++ standards-compliant manner) and various optimization levels which would not have raised red flags with make tinderbox in the first place. If there was a widespread (gcc and/or standard compiler/linker flags) issue with the build a) we would have seen it be now in the tinderbox emails and b) I would have CCed the current set of mailing lists so the issue would have been resolved quickly. The fact that it wasn't caught illustrates the fact that although we're trying to pilot clang as the default compiler by next month, there's a huge gap in required testing for commits. Looking forward, the major items for ATF have been committed. The rest of the patches which will be committed will be considerably smaller, targeted to specific components, and [for the most part, minus test integration in the tinderbox builds which will only be done after extensively testing is performed] will not affect the standard build process. Thank you for the concern though and I understand where you're coming from. -Garrett