From owner-svn-src-head@freebsd.org Tue Sep 3 14:06:03 2019 Return-Path: Delivered-To: svn-src-head@mailman.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.nyi.freebsd.org (Postfix) with ESMTP id 37C99DC11B; Tue, 3 Sep 2019 14:05:59 +0000 (UTC) (envelope-from yuripv@freebsd.org) Received: from freefall.freebsd.org (freefall.freebsd.org [96.47.72.132]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) server-signature RSA-PSS (4096 bits) client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "freefall.freebsd.org", Issuer "Let's Encrypt Authority X3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 46N7yt4Lhtz4P36; Tue, 3 Sep 2019 14:05:58 +0000 (UTC) (envelope-from yuripv@freebsd.org) Received: by freefall.freebsd.org (Postfix, from userid 1452) id A7F4419EC3; Tue, 3 Sep 2019 14:05:52 +0000 (UTC) X-Original-To: yuripv@localmail.freebsd.org Delivered-To: yuripv@localmail.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [96.47.72.80]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) (Client CN "mx1.freebsd.org", Issuer "Let's Encrypt Authority X3" (verified OK)) by freefall.freebsd.org (Postfix) with ESMTPS id EEF0C1D128; Sat, 30 Mar 2019 04:10:27 +0000 (UTC) (envelope-from owner-src-committers@freebsd.org) Received: from freefall.freebsd.org (freefall.freebsd.org [96.47.72.132]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) server-signature RSA-PSS (4096 bits) client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "freefall.freebsd.org", Issuer "Let's Encrypt Authority X3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 5D0D56C886; Sat, 30 Mar 2019 04:10:27 +0000 (UTC) (envelope-from owner-src-committers@freebsd.org) Received: by freefall.freebsd.org (Postfix, from userid 538) id 0F8E91D0E8; Sat, 30 Mar 2019 04:10:27 +0000 (UTC) Delivered-To: src-committers@localmail.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) (Client CN "mx1.freebsd.org", Issuer "Let's Encrypt Authority X3" (verified OK)) by freefall.freebsd.org (Postfix) with ESMTPS id 0E9181D0E0 for ; Sat, 30 Mar 2019 04:10:24 +0000 (UTC) (envelope-from wlosh@bsdimp.com) Received: from mail-qt1-x82d.google.com (mail-qt1-x82d.google.com [IPv6:2607:f8b0:4864:20::82d]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (Client CN "smtp.gmail.com", Issuer "GTS CA 1O1" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id E72516C86F for ; Sat, 30 Mar 2019 04:10:22 +0000 (UTC) (envelope-from wlosh@bsdimp.com) Received: by mail-qt1-x82d.google.com with SMTP id w30so4766039qta.8 for ; Fri, 29 Mar 2019 21:10:22 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bsdimp-com.20150623.gappssmtp.com; s=20150623; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=/h33n9/OH8gsuUYw2JJGYJtNFYReSU0fvCH/PkoJJT0=; b=Xd/JkF1D7jtHlVJtRfgVo0x/QEu7qLr8JIddOdGZ9XhJDKRSgHzKJ5hyMsghKZa+Zi bxmExtnfykVsbksgSaLqGWqq/efE7casIXFDLMnrKyxNAaeZtz3Bk2LpyyoIyMYz69Ux R4fol/ARYc2/9PdhahHRz/LOtsr076RAPSn9L8CItuHZ6fwNwbhDaxgNfFrNcSTG0GP+ wJ3yaH9zyAykb+cOK2Zjrlq3GYoHqVBHuIb29ihEeLIvNMiMOqIaJPTDHR7+hH5GJGDU usreJf7hH5PnZ7KTDRXtth2ipqRgE3z7bzeUw5pVQQYCzeQIiC4bOJn5yuHtIE4Wn0S8 FNcA== 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=/h33n9/OH8gsuUYw2JJGYJtNFYReSU0fvCH/PkoJJT0=; b=tE61cu9lCYyxnYkGukPCjwLFIJ2SBJbTBaE/f9yABjRuraNiWw65iFyqAYtDKLKRgN VZqAj6XSuPtwi2bqjg93I0bDXHlymI73RRpdFx+aSDCa17IP1HUZ8orJqIdrab4eUv/H LIKfiGf1CMR63g1UpoOpSjRK7grulLUUHQ9QITKqn9FlwrjKD9nDE6Ua5Me2OoKy2YOp 36aERS7rKuJIVFsI9J8ZEI5sfBLxEfNt7bl1XC2TzYEOkhE3rauKrN3QXWCqIX0mMeO7 ByJClB7PpAeQYw/62+5En/q3XTMrDJUQNwDTeKbGRm4fXtUzjuDL3H94fnpk3mBxxLoh 71nQ== X-Gm-Message-State: APjAAAX1J0qVig9g7yv0FyGo6lb1ZYphPVJ0Q66wM0wb5HWA5qXMqnCv 3fdK8lQmQc2dBUuKXyzRTgmS9CD6uM/K9T2zndrM0g== X-Google-Smtp-Source: APXvYqyaGuq6e9/6MTKs0oR1i7mB46gfp/cbIAlIKyQP0JG/DF+bJ20xspXEHsrx4EmJuLxq2Ho7zFj1C+EMuioM5U4= X-Received: by 2002:ac8:304a:: with SMTP id g10mr45047889qte.118.1553919022055; Fri, 29 Mar 2019 21:10:22 -0700 (PDT) MIME-Version: 1.0 References: <201903300309.x2U39DtW002526@gndrsh.dnsmgr.net> In-Reply-To: From: Warner Losh Message-ID: Subject: Re: svn commit: r345707 - in head: lib/clang lib/libc++ lib/libc++experimental lib/libc++fs lib/libc/tests/stdlib lib/libclang_rt lib/libcxxrt lib/libgcc_eh lib/libomp lib/ofed/libibnetdisc share/mk us... To: Garrett Cooper Cc: "Rodney W. Grimes" , Garrett Cooper , src-committers , svn-src-all , svn-src-head@freebsd.org Precedence: bulk X-Loop: FreeBSD.org Sender: owner-src-committers@freebsd.org X-Rspamd-Queue-Id: 5D0D56C886 X-Spamd-Bar: -- Authentication-Results: mx1.freebsd.org X-Spamd-Result: default: False [-2.87 / 15.00]; local_wl_from(0.00)[freebsd.org]; NEURAL_HAM_MEDIUM(-1.00)[-0.998,0]; NEURAL_HAM_SHORT(-0.87)[-0.871,0]; NEURAL_HAM_LONG(-1.00)[-1.000,0]; ASN(0.00)[asn:11403, ipnet:96.47.64.0/20, country:US] Status: O 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-head@freebsd.org X-Mailman-Version: 2.1.29 List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Date: Tue, 03 Sep 2019 14:06:03 -0000 X-Original-Date: Fri, 29 Mar 2019 22:10:10 -0600 X-List-Received-Date: Tue, 03 Sep 2019 14:06:03 -0000 On Fri, Mar 29, 2019, 9:15 PM Enji Cooper wrote: > > > On Mar 29, 2019, at 8:09 PM, Rodney W. Grimes > wrote: > > > >> Author: ngie > >> Date: Fri Mar 29 18:43:46 2019 > >> New Revision: 345707 > >> URL: https://svnweb.freebsd.org/changeset/base/345707 > >> > >> Log: > >> Revert r345706: the third time will be the charm > >> > >> When a review is closed via Phabricator it updates the patch attached > to the > >> review. I downloaded the raw patch from Phabricator, applied it, and > repeated > >> my mistake from r345704 by accident mixing content from D19732 and > D19738. > > > > Which, arguable is a feature or mis feature depending on the point > > of view. I do not like it when I go to look at someone elses > > committed code siting a review, as I want to actually see what > > it was that was committed. You can find the pre-commit diff, > > but it takes a bit of probling. The upside is you can get > > both diffs from the same place and diff the diffs :-) > > > >> For my own personal sanity, I will try not to mix reviews like this i= n > the > >> future. > > > > :-) Been there, almost did that too. > > Pre commit last minute svn diff saved me. > > =E2=80=A6 > > This is why I=E2=80=99m doing the following from here on out: > > $ arc patch > $ svn ci > > Unfortunately svn doesn=E2=80=99t support all of the niceties of =E2=80= =9Carc land=E2=80=9D. > Otherwise, I would have used that. > > The Facebook version of =E2=80=9Carc land=E2=80=9D (before their new non-= public variation) > supported verifying diffs in local repos vs Phabricator to make sure that > the diff content was consistent/correct. > > * Pro: it would catch issues like what I did the first time. > * Con: I couldn=E2=80=99t make last minute changes (I would need to resub= mit the > change and have it re-reviewed, which I argue is a good feature). > > Just some food for thought. For now, arc patch/svn ci works for me. > I use git svn and git show every commit so I'm sure when I want to do multiple at once. Phab doesn't make that easy, alas, so I often choose the lesser evil of just getting a review on the squashed branch. I only split when there is a good reason. If we had better tools, I wouldn't be forced to pick, but you go to wat with the tools you have... Warner >