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-ASN: AS31976 209.132.180.0/23 X-Spam-Status: No, score=-4.0 required=3.0 tests=AWL,BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,RCVD_IN_DNSWL_HI,RCVD_IN_MSPIKE_H3, RCVD_IN_MSPIKE_WL,SPF_HELO_NONE,SPF_NONE shortcircuit=no autolearn=ham autolearn_force=no version=3.4.2 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by dcvr.yhbt.net (Postfix) with ESMTP id AC6161F619 for ; Fri, 20 Mar 2020 18:08:43 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726866AbgCTSIm (ORCPT ); Fri, 20 Mar 2020 14:08:42 -0400 Received: from pb-smtp1.pobox.com ([64.147.108.70]:63821 "EHLO pb-smtp1.pobox.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725446AbgCTSIm (ORCPT ); Fri, 20 Mar 2020 14:08:42 -0400 Received: from pb-smtp1.pobox.com (unknown [127.0.0.1]) by pb-smtp1.pobox.com (Postfix) with ESMTP id CC2B048870; Fri, 20 Mar 2020 14:08:41 -0400 (EDT) (envelope-from junio@pobox.com) DKIM-Signature: v=1; a=rsa-sha1; c=relaxed; d=pobox.com; h=from:to:cc :subject:references:date:in-reply-to:message-id:mime-version :content-type; s=sasl; bh=hhcKipFMwySWY/q9yUOX2lJxm8M=; b=qKyTcN 6zuWRq19NaI1BJK/p7phXuZW9GdsgIOVXGqfMhLuXl6TD3CW14U550ThVph/JPN1 V7kvHFy5QcvCOFzvvt/3c4tmaepFGx4oyOwUGdRwe2k4n7dAVvslIua/ZnTosWMZ DOEP4GhEWIusIlcDL7VYyX2vfmy8mFspxZOL0= DomainKey-Signature: a=rsa-sha1; c=nofws; d=pobox.com; h=from:to:cc :subject:references:date:in-reply-to:message-id:mime-version :content-type; q=dns; s=sasl; b=WoNSTqMPCB0Mh2jdAY3etHD5oF/18uig OYh4vWx8An+A7cJSpsTX/1IfddP0CrFjwXv4Of6gf+YZaoq8cmKRRK/2qY5XUdbz R1BAxkBtpVtQrfbV7ij7qVpShQz+o6MqeQEIDNht77J2snfibKLj62I2chgs8lyo GBXPan3WULg= Received: from pb-smtp1.nyi.icgroup.com (unknown [127.0.0.1]) by pb-smtp1.pobox.com (Postfix) with ESMTP id C35FB4886F; Fri, 20 Mar 2020 14:08:41 -0400 (EDT) (envelope-from junio@pobox.com) Received: from pobox.com (unknown [34.74.119.39]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by pb-smtp1.pobox.com (Postfix) with ESMTPSA id 0B3AD4886B; Fri, 20 Mar 2020 14:08:41 -0400 (EDT) (envelope-from junio@pobox.com) From: Junio C Hamano To: git@vger.kernel.org Cc: peff@peff.net, Adrian Wijaya Subject: Re: [GSOC][PATCH v2] t1300: convert "test -f" with "test_path_is_file" References: <20200320160723.15190-1-adrianwijaya100@gmail.com> Date: Fri, 20 Mar 2020 11:08:40 -0700 In-Reply-To: <20200320160723.15190-1-adrianwijaya100@gmail.com> (Adrian Wijaya's message of "Fri, 20 Mar 2020 23:07:23 +0700") Message-ID: User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/26.3 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain X-Pobox-Relay-ID: D44DAE12-6AD5-11EA-AAAB-C28CBED8090B-77302942!pb-smtp1.pobox.com Sender: git-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org Adrian Wijaya writes: > test "z$(git config --file=notyet test.frotz)" = znitfol && We see 130+ instances of this rather antiquated construct that presumably avoids to compare an empty string with something else using "test X = Y" tool. It might have been a good idea in 2005 (I do not remember why we started doing this), but perhaps it is time to clean them up by listing it as another microproject idea for the next year?