From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on dcvr.yhbt.net X-Spam-Level: X-Spam-Status: No, score=-4.0 required=3.0 tests=AWL,BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_PASS, SPF_PASS shortcircuit=no autolearn=ham autolearn_force=no version=3.4.2 Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by dcvr.yhbt.net (Postfix) with ESMTP id F0D0B1F5AE for ; Thu, 16 Jul 2020 10:56:34 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726898AbgGPK4b (ORCPT ); Thu, 16 Jul 2020 06:56:31 -0400 Received: from cloud.peff.net ([104.130.231.41]:58926 "EHLO cloud.peff.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726239AbgGPK4b (ORCPT ); Thu, 16 Jul 2020 06:56:31 -0400 Received: (qmail 22633 invoked by uid 109); 16 Jul 2020 10:56:30 -0000 Received: from Unknown (HELO peff.net) (10.0.1.2) by cloud.peff.net (qpsmtpd/0.94) with ESMTP; Thu, 16 Jul 2020 10:56:30 +0000 Authentication-Results: cloud.peff.net; auth=none Received: (qmail 22791 invoked by uid 111); 16 Jul 2020 10:56:29 -0000 Received: from coredump.intra.peff.net (HELO sigill.intra.peff.net) (10.0.0.2) by peff.net (qpsmtpd/0.94) with (TLS_AES_256_GCM_SHA384 encrypted) ESMTPS; Thu, 16 Jul 2020 06:56:29 -0400 Authentication-Results: peff.net; auth=none Date: Thu, 16 Jul 2020 06:56:29 -0400 From: Jeff King To: Jonathan Nieder Cc: Junio C Hamano , Johannes Schindelin , Derrick Stolee , Johannes Schindelin via GitGitGadget , git@vger.kernel.org, delphij@google.com, Huan Huan Chen Subject: Re: [PATCH 1/2] Revert "check_repository_format_gently(): refuse extensions for old repositories" Message-ID: <20200716105629.GC376357@coredump.intra.peff.net> References: <0bede821-139a-d805-934a-142004abaa4c@gmail.com> <20200716062054.GA3242764@google.com> <20200716062429.GB3242764@google.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20200716062429.GB3242764@google.com> Sender: git-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org On Wed, Jul 15, 2020 at 11:24:29PM -0700, Jonathan Nieder wrote: > The behavior introduced in 14c7fa269e4 might be a good behavior if we > were traveling back in time to 2015, but we're far too late. For some > reason I thought that it was what had been originally implemented and > that it had regressed. Apologies for not doing my research when > 14c7fa269e4 was under development. Thanks for a good summary of the situation. I agree that the current (well, pre-14c7fa269e4) behavior is a bug (mine) from 2015, and we probably have to accept that state of affairs to some degree in order to avoid breaking existing cases. It is unfortunate that this means that somebody with version=0 and extensions.preciousObjects is in danger of running a pre-2015 version of Git and having that extension totally ignored, which could be a data-safety issue. But the farther we get from 2015 the less likely that is to be a problem (and the more likely somebody is to be depending on the current behavior of v0+preciousObjects). > Let's return to the behavior we've had since 2015: always act on > extensions.* settings, regardless of repository format version. While > we're here, include some tests to describe the effect on the "upgrade > repository version" code path. So this makes sense to me as a first step. -Peff