From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: =?utf-8?Q?=C3=93scar_Fuentes?= Newsgroups: gmane.emacs.devel Subject: Re: Recommend these .gitconfig settings for git integrity. Date: Tue, 02 Feb 2016 19:48:02 +0100 Message-ID: <878u33gd99.fsf@wanadoo.es> References: <87a8nlfqj9.fsf@red-bean.com> <83h9ht1o8q.fsf@gnu.org> <87mvrle8ho.fsf@red-bean.com> <56AE8126.9090708@cs.ucla.edu> <87wpqo77yc.fsf@red-bean.com> <87bn80ifmh.fsf@wanadoo.es> <87k2mo75bx.fsf@red-bean.com> <56AFB869.1080206@cs.ucla.edu> <83si1cyy3e.fsf@gnu.org> <56B06282.2050701@cs.ucla.edu> <87twlrgljm.fsf@wanadoo.es> <56B0ED78.5030209@cs.ucla.edu> NNTP-Posting-Host: plane.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit X-Trace: ger.gmane.org 1454438912 32035 80.91.229.3 (2 Feb 2016 18:48:32 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Tue, 2 Feb 2016 18:48:32 +0000 (UTC) To: emacs-devel@gnu.org Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Tue Feb 02 19:48:23 2016 Return-path: Envelope-to: ged-emacs-devel@m.gmane.org Original-Received: from lists.gnu.org ([208.118.235.17]) by plane.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1aQfzy-0003XH-83 for ged-emacs-devel@m.gmane.org; Tue, 02 Feb 2016 19:48:22 +0100 Original-Received: from localhost ([::1]:58977 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aQfzx-000623-OR for ged-emacs-devel@m.gmane.org; Tue, 02 Feb 2016 13:48:21 -0500 Original-Received: from eggs.gnu.org ([2001:4830:134:3::10]:44863) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aQfzu-00061c-7c for emacs-devel@gnu.org; Tue, 02 Feb 2016 13:48:19 -0500 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1aQfzp-0000ih-8y for emacs-devel@gnu.org; Tue, 02 Feb 2016 13:48:18 -0500 Original-Received: from plane.gmane.org ([80.91.229.3]:38316) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aQfzp-0000id-1R for emacs-devel@gnu.org; Tue, 02 Feb 2016 13:48:13 -0500 Original-Received: from list by plane.gmane.org with local (Exim 4.69) (envelope-from ) id 1aQfzn-0003Hu-4M for emacs-devel@gnu.org; Tue, 02 Feb 2016 19:48:11 +0100 Original-Received: from 1.red-83-38-42.dynamicip.rima-tde.net ([83.38.42.1]) by main.gmane.org with esmtp (Gmexim 0.1 (Debian)) id 1AlnuQ-0007hv-00 for ; Tue, 02 Feb 2016 19:48:11 +0100 Original-Received: from ofv by 1.red-83-38-42.dynamicip.rima-tde.net with local (Gmexim 0.1 (Debian)) id 1AlnuQ-0007hv-00 for ; Tue, 02 Feb 2016 19:48:11 +0100 X-Injected-Via-Gmane: http://gmane.org/ Original-Lines: 73 Original-X-Complaints-To: usenet@ger.gmane.org X-Gmane-NNTP-Posting-Host: 1.red-83-38-42.dynamicip.rima-tde.net User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/25.0.50 (gnu/linux) Cancel-Lock: sha1:oXWlG0soP6j8tHHtTCwKDucACrs= X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. X-Received-From: 80.91.229.3 X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Original-Sender: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.devel:199186 Archived-At: Paul Eggert writes: > On 02/02/2016 07:49 AM, Óscar Fuentes wrote: > >> I mean, it look like it does a good thing, if you read the >> description. So why it is not activated by default? Maybe it has some >> drawbacks. > Yes, it has a drawback. I mentioned it in my previous message. On some > larger projects, it slows down some git operations. However, this does > not seem to be a significant problem with Emacs development. Care to explain this in more detail? Emacs is no small project. If the slow down affects large transfers ("large" meaning either many objects or big objects) what happens if an Emacs hackers pauses his activity for several months and then pulls? (after monitoring emacs-diffs for several years, I can attest that this scenario is quite frequent.) > In contrast, omitting the option can cause significant problems later, > problems that can be hard to discover and even harder to repair. For > an example of this, please see the following bugreport on GitHub: > > https://github.com/wp-cli/php-cli-tools/issues/70 > > Apparently the wp-cli project has a corrupted Git history that could > have been prevented by the new setting. It is so much trouble to fix > this that the project's maintainer and the GitHub staff think that > fixing it is more trouble than it's worth, and will just live with the > corrupted repository. I'd hate to see the same thing happen to the > Emacs source code repository. So the GitHub folks, on their early days, had a bug on their Git re-implementation and caused some hosted project to become "infected". This looks like a far-fetched case from the Emacs POV. If we wish to avoid tainted objects created by whatever cause, the check can be enabled on the Savannah's repo, hence limiting the problem to the "infected" user. The problem seems to be so rare that a single Emacs hacker experiencing it every decade or so (if it ever happens) doesn't warrant the risks and inconveniences associated with using the setting (see my previous messages.) >> Anyone cared about that possibility, apart from the "I >> activated it and so far, so good" testimony from*one* individual? > > I've been using it. So has Stefan, and Karl. Nobody who has used it > has reported problems. If there were real problems with this option we > should not make it the default. So you, Karl and Stefan used the feature... for how much time? Two days? Are you a reprensetative sample of the overwhelming majority of the Emacs devel populationt? (No, you aren't). Sorry but imposing the config change to everybody's repo so soon and without discussion looks like a mindlessly hurried-up move, to say it bluntly. > I will look into modifying autogen.sh to make this flag optional, > since there seems to be significant opposition to it. Thanks, although it is now a bit late. It is already installed on several repos, for sure. And anybody that stumbles on the revisions that contained your change (by bisecting some bug, for example) will have his .git/config file modified. I'm pretty sure that you didn't think of this issues when you made the change (neither I did at first.) > I still don't > understand why there's so much opposition, though. If the Emacs > repository becomes corrupted because this option was omitted, quite > possibly we won't notice, and even if we notice quite possibly we > won't fix it. I would hate to see that happen. I appreciate your concern but that has an easy solution: enable it on the server (and on your own machine, to be extra sure.) See, problem solved. Now that you are at it, convince the Savannah admins to enable the check on all git repos they host. That is much more reasonable than automatically changing the configuration of each clone out there.